Re: [PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes

2013-03-12 Thread Tejun Heo
On Fri, Mar 01, 2013 at 07:23:51PM -0800, Tejun Heo wrote: > Finally, here's the unbound workqueue with custom worker attributes > patchset I've been talking about. The goal is simple. We want > unbound workqueues with custom worker attributes with a mechanism to > expose the knobs to userland.

Re: [PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes

2013-03-12 Thread Tejun Heo
Hello, Lai. On Sun, Mar 10, 2013 at 05:01:13AM -0700, Tejun Heo wrote: > > > 0023-workqueue-implement-get-put_pwq.patch > > > > I guess this patch and patch25 may have very deep issue VS RCU. > > Hmmm... scary. I suppose you're gonna elaborate on the review of the > actual patch? What happene

Re: [PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes

2013-03-11 Thread Tejun Heo
On Mon, Mar 11, 2013 at 11:42:29PM +0800, Lai Jiangshan wrote: > >> for 1~13,15~22,26~28, please add Reviewed-by: Lai Jiangshan > >> > > OK, Also add my Reviewed-by to 23~25. > > > > > Done. > > > > I didn't see you updated branch in your tree. Working on something else and also waiting on th

Re: [PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes

2013-03-11 Thread Lai Jiangshan
On Sun, Mar 10, 2013 at 8:01 PM, Tejun Heo wrote: > Hey, Lai. > > On Sun, Mar 10, 2013 at 06:34:33PM +0800, Lai Jiangshan wrote: >> > This patchset contains the following 31 patches. >> > >> > 0001-workqueue-make-sanity-checks-less-punshing-using-WAR.patch >> >> > 0002-workqueue-make-workqueue_l

Re: [PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes

2013-03-11 Thread Lai Jiangshan
On Mon, Mar 11, 2013 at 11:24 PM, Tejun Heo wrote: > On Sun, Mar 10, 2013 at 05:01:13AM -0700, Tejun Heo wrote: >> Hey, Lai. >> >> On Sun, Mar 10, 2013 at 06:34:33PM +0800, Lai Jiangshan wrote: >> > > This patchset contains the following 31 patches. >> > > >> > > 0001-workqueue-make-sanity-checks

Re: [PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes

2013-03-11 Thread Tejun Heo
On Sun, Mar 10, 2013 at 05:01:13AM -0700, Tejun Heo wrote: > Hey, Lai. > > On Sun, Mar 10, 2013 at 06:34:33PM +0800, Lai Jiangshan wrote: > > > This patchset contains the following 31 patches. > > > > > > 0001-workqueue-make-sanity-checks-less-punshing-using-WAR.patch > > > > > 0002-workqueue-

Re: [PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes

2013-03-10 Thread Tejun Heo
Hey, Lai. On Sun, Mar 10, 2013 at 06:34:33PM +0800, Lai Jiangshan wrote: > > This patchset contains the following 31 patches. > > > > 0001-workqueue-make-sanity-checks-less-punshing-using-WAR.patch > > > 0002-workqueue-make-workqueue_lock-irq-safe.patch > > workqueue_lock protects too many th

Re: [PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes

2013-03-10 Thread Lai Jiangshan
On 02/03/13 11:23, Tejun Heo wrote: Hi, Tejun, I agree almost the whole design.(only except some locks) And I found only a little problems for current review. > > This patchset contains the following 31 patches. > > 0001-workqueue-make-sanity-checks-less-punshing-using-WAR.patch > 0002-work

Re: [PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes

2013-03-05 Thread Tejun Heo
On Fri, Mar 01, 2013 at 07:23:51PM -0800, Tejun Heo wrote: > which is scheduled to be rebased on top of v3.9-rc1 once it comes out. > The changes are also available in the following git branch. > > git://git.kernel.org/pub/scm/linux/kernel/git/tj/wq.git review-attrs Branch rebased on top of wq/f

[PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes

2013-03-01 Thread Tejun Heo
Subject: [PATCHSET wq/for-3.10-tmp] workqueue: implement workqueue with custom worker attributes Hello, Finally, here's the unbound workqueue with custom worker attributes patchset I've been talking about. The goal is simple. We want unbound workqueues with custom worker attribu