Re: [PATCH 0/7 V6] workqueue: fix hoplug things

2012-09-08 Thread Lai Jiangshan
> > Hmmm... so, I'm having some difficulty communicating with you. We > need two separate patch series. One for for-3.6-fixes and the other > for restructuring on top of for-3.7 after the fixes are merged into > it. > > As you currently posted, the patches are based on for-3.7 and fixes > and res

Re: [PATCH 0/7 V6] workqueue: fix hoplug things

2012-09-08 Thread Tejun Heo
On Sun, Sep 09, 2012 at 01:56:47AM +0800, Lai Jiangshan wrote: > > So, your worry was incorrect and the above is what we're gonna do, no? > > NO BUG found for PATCH4 even idle rebinding is synchronous. Hmmm... so, I'm having some difficulty communicating with you. We need two separate patch seri

Re: [PATCH 0/7 V6] workqueue: fix hoplug things

2012-09-08 Thread Lai Jiangshan
On Sun, Sep 9, 2012 at 1:50 AM, Tejun Heo wrote: > Hello, Lai. > > On Sun, Sep 09, 2012 at 01:46:59AM +0800, Lai Jiangshan wrote: >> > * Instead of MANAGING, add pool->manager. >> > >> > * Fix the idle depletion bug by using pool->manager for exclusion and >> > always grabbing pool->manager_mute

Re: [PATCH 0/7 V6] workqueue: fix hoplug things

2012-09-08 Thread Tejun Heo
On Sat, Sep 08, 2012 at 10:50:07AM -0700, Tejun Heo wrote: > Please create two patches, first introducing pool->manager_mutex and ^ pool->manager -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a me

Re: [PATCH 0/7 V6] workqueue: fix hoplug things

2012-09-08 Thread Tejun Heo
Hello, Lai. On Sun, Sep 09, 2012 at 01:46:59AM +0800, Lai Jiangshan wrote: > > * Instead of MANAGING, add pool->manager. > > > > * Fix the idle depletion bug by using pool->manager for exclusion and > > always grabbing pool->manager_mutex. Hotplug can use pool->manager > > to schedule rebind

Re: [PATCH 0/7 V6] workqueue: fix hoplug things

2012-09-08 Thread Lai Jiangshan
On Sun, Sep 9, 2012 at 1:37 AM, Tejun Heo wrote: > Hello, Lai. > > On Sun, Sep 09, 2012 at 01:27:37AM +0800, Lai Jiangshan wrote: >> On Sun, Sep 9, 2012 at 1:12 AM, Lai Jiangshan wrote: >> > The patch set is based on 3b07e9ca26866697616097044f25fbe53dbab693 of >> > wq.git >> > >> > Patch 1,2 are

Re: [PATCH 0/7 V6] workqueue: fix hoplug things

2012-09-08 Thread Tejun Heo
Hello, Lai. On Sun, Sep 09, 2012 at 01:27:37AM +0800, Lai Jiangshan wrote: > On Sun, Sep 9, 2012 at 1:12 AM, Lai Jiangshan wrote: > > The patch set is based on 3b07e9ca26866697616097044f25fbe53dbab693 of wq.git > > > > Patch 1,2 are accepted. Patch 1 goes to 3.6. tj has a replacement goes > > to

Re: [PATCH 0/7 V6] workqueue: fix hoplug things

2012-09-08 Thread Lai Jiangshan
On Sun, Sep 9, 2012 at 1:12 AM, Lai Jiangshan wrote: > The patch set is based on 3b07e9ca26866697616097044f25fbe53dbab693 of wq.git > > Patch 1,2 are accepted. Patch 1 goes to 3.6. tj has a replacement goes > to 3.6 instead of Patch 2. so Patch2 will go to 3.7. Patch2 will need > to be rebased if

Re: [PATCH 0/7 V6] workqueue: fix hoplug things

2012-09-08 Thread Lai Jiangshan
On Sun, Sep 9, 2012 at 1:27 AM, Lai Jiangshan wrote: > On Sun, Sep 9, 2012 at 1:12 AM, Lai Jiangshan wrote: >> The patch set is based on 3b07e9ca26866697616097044f25fbe53dbab693 of wq.git >> >> Patch 1,2 are accepted. Patch 1 goes to 3.6. tj has a replacement goes >> to 3.6 instead of Patch 2. so

[PATCH 0/7 V6] workqueue: fix hoplug things

2012-09-08 Thread Lai Jiangshan
The patch set is based on 3b07e9ca26866697616097044f25fbe53dbab693 of wq.git Patch 1,2 are accepted. Patch 1 goes to 3.6. tj has a replacement goes to 3.6 instead of Patch 2. so Patch2 will go to 3.7. Patch2 will need to be rebased if the replacement is still in 3.7. (tj, could you help me do the