xfs
[Top] [All Lists]

Re: [PATCH 2/6] workqueue: factor out worker_pool from global_cwq

To: Namhyung Kim <namhyung@xxxxxxxxxx>
Subject: Re: [PATCH 2/6] workqueue: factor out worker_pool from global_cwq
From: Tejun Heo <tj@xxxxxxxxxx>
Date: Thu, 12 Jul 2012 10:07:52 -0700
Cc: linux-kernel@xxxxxxxxxxxxxxx, torvalds@xxxxxxxxxxxxxxxxxxxx, joshhunt00@xxxxxxxxx, axboe@xxxxxxxxx, rni@xxxxxxxxxx, vgoyal@xxxxxxxxxx, vwadekar@xxxxxxxxxx, herbert@xxxxxxxxxxxxxxxxxxxx, davem@xxxxxxxxxxxxx, linux-crypto@xxxxxxxxxxxxxxx, swhiteho@xxxxxxxxxx, bpm@xxxxxxx, elder@xxxxxxxxxx, xfs@xxxxxxxxxxx, marcel@xxxxxxxxxxxx, gustavo@xxxxxxxxxxx, johan.hedberg@xxxxxxxxx, linux-bluetooth@xxxxxxxxxxxxxxx, martin.petersen@xxxxxxxxxx
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; bh=fALqSWuEFYgwxPW55GjXb9U4tPscIOjqk16D46Imrjw=; b=iCAYzzhrdnMrO3ABrgspryyfFwpRayL7hZ1E3IOe/PCvgL+xIpdSuGytIHEmxvcrR6 Xr0TeoZDVCYzZcWdkGuxvdL+/39zs0+nxwFRMPPitfQ+wZN/LsUMgaRP3gzQYoGIUrro EMxqVcO1BebjIwscsG9CKDAWjefvFwwQDDemYAX/Il0vNdrHA4DIaOG66zcLLI5r3HXJ +kUqxM4BI/rZJC8RaPK65oXBEvbYmJRu9HzrlWMWQoYXiQVgOKH+AWjxvCCc/46y8jB9 DvnxJc8s5MEWvlwCYVcrt81ntrf7FvQyWuvgI6bNthBmZqy4d/b3MtRCcP6FiNoKwHjs 97nQ==
In-reply-to: <87ehok8bmb.fsf@xxxxxxxxxxxxxxxxxx>
References: <1341859315-17759-1-git-send-email-tj@xxxxxxxxxx> <1341859315-17759-3-git-send-email-tj@xxxxxxxxxx> <87ehok8bmb.fsf@xxxxxxxxxxxxxxxxxx>
Sender: Tejun Heo <htejun@xxxxxxxxx>
User-agent: Mutt/1.5.20 (2009-06-14)
Hello, Namhyung.

Sorry about the delay.

On Tue, Jul 10, 2012 at 01:48:44PM +0900, Namhyung Kim wrote:
> > +   struct list_head        idle_list;      /* X: list of idle workers */
> > +   struct timer_list       idle_timer;     /* L: worker idle timeout */
> > +   struct timer_list       mayday_timer;   /* L: SOS timer for dworkers */
> 
> What is 'dworkers'?

My stupid finger pressing 'd' when I never meant to. :)

> > -   /* workers are chained either in the idle_list or busy_hash */
> > -   struct list_head        idle_list;      /* X: list of idle workers */
> > +   /* workers are chained either in busy_head or pool idle_list */
> 
> s/busy_head/busy_hash/ ?

Will fix.

Thanks.

-- 
tejun

<Prev in Thread] Current Thread [Next in Thread>