From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH] remove name length check in a workqueue Date: Wed, 10 Aug 2005 10:05:23 -0700 Message-ID: <20050810100523.0075d4e8.akpm@osdl.org> References: <1123683544.5093.4.camel@mulgrave> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from smtp.osdl.org ([65.172.181.4]:28368 "EHLO smtp.osdl.org") by vger.kernel.org with ESMTP id S965214AbVHJRGr (ORCPT ); Wed, 10 Aug 2005 13:06:47 -0400 In-Reply-To: Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: Ingo Molnar Cc: James.Bottomley@SteelEye.com, linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org Ingo Molnar wrote: > > > yeah ... cannot remember why i have done it originally :-| > Might it be to do with sizeof(task_struct.comm)? > > On Wed, 10 Aug 2005, James Bottomley wrote: > > > Ingo, > > > > This has been in the workqueue code in day one, for no real reason that > > I can see. We just tripped over it in SCSI because the fibre channel > > transport class creates one workqueue per host with the name scsi_wq_%d > > which trips this after we get to 100. Unfortunately we just came across > > someone with > 100 host adapters ... > > > > I think the solution is just to get rid of the artificial limit. > > > > James > > > > diff --git a/kernel/workqueue.c b/kernel/workqueue.c > > --- a/kernel/workqueue.c > > +++ b/kernel/workqueue.c > > @@ -308,8 +308,6 @@ struct workqueue_struct *__create_workqu > > struct workqueue_struct *wq; > > struct task_struct *p; > > > > - BUG_ON(strlen(name) > 10); > > - > > wq = kmalloc(sizeof(*wq), GFP_KERNEL); > > if (!wq) > > return NULL; > > > >