Re: [PATCH 3/3 V7] workqueue: Allow modifying low level unbound workqueue cpumask

2015-04-23 Thread Mike Galbraith
FWIW on the testing side, I'm running these in 3.12(ish), 4.0 and 4.1 rt trees with NOHZ_FULL, and have yet to meet a problem. -Mike -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH 3/3 V7] workqueue: Allow modifying low level unbound workqueue cpumask

2015-04-23 Thread Mike Galbraith
FWIW on the testing side, I'm running these in 3.12(ish), 4.0 and 4.1 rt trees with NOHZ_FULL, and have yet to meet a problem. -Mike -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH 3/3 V7] workqueue: Allow modifying low level unbound workqueue cpumask

2015-04-22 Thread Frederic Weisbecker
On Wed, Apr 22, 2015 at 03:39:35PM -0400, Tejun Heo wrote: > Hello, > > Generally looks good to me. Some minor things below. > > On Tue, Apr 07, 2015 at 07:26:37PM +0800, Lai Jiangshan wrote: > > diff --git a/kernel/workqueue.c b/kernel/workqueue.c > > index cbccf5d..557612e 100644 > > ---

Re: [PATCH 3/3 V7] workqueue: Allow modifying low level unbound workqueue cpumask

2015-04-22 Thread Tejun Heo
Hello, Generally looks good to me. Some minor things below. On Tue, Apr 07, 2015 at 07:26:37PM +0800, Lai Jiangshan wrote: > diff --git a/kernel/workqueue.c b/kernel/workqueue.c > index cbccf5d..557612e 100644 > --- a/kernel/workqueue.c > +++ b/kernel/workqueue.c > @@ -299,7 +299,7 @@ static

Re: [PATCH 3/3 V7] workqueue: Allow modifying low level unbound workqueue cpumask

2015-04-22 Thread Tejun Heo
Hello, Generally looks good to me. Some minor things below. On Tue, Apr 07, 2015 at 07:26:37PM +0800, Lai Jiangshan wrote: diff --git a/kernel/workqueue.c b/kernel/workqueue.c index cbccf5d..557612e 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -299,7 +299,7 @@ static

Re: [PATCH 3/3 V7] workqueue: Allow modifying low level unbound workqueue cpumask

2015-04-22 Thread Frederic Weisbecker
On Wed, Apr 22, 2015 at 03:39:35PM -0400, Tejun Heo wrote: Hello, Generally looks good to me. Some minor things below. On Tue, Apr 07, 2015 at 07:26:37PM +0800, Lai Jiangshan wrote: diff --git a/kernel/workqueue.c b/kernel/workqueue.c index cbccf5d..557612e 100644 ---

[PATCH 3/3 V7] workqueue: Allow modifying low level unbound workqueue cpumask

2015-04-07 Thread Lai Jiangshan
Allow to modify the low-level unbound workqueues cpumask through sysfs. This is performed by traversing the entire workqueue list and calling apply_wqattrs_prepare() on the unbound workqueues with the low level mask passed in. Only after all the preparation are done, we commit them all together.

[PATCH 3/3 V7] workqueue: Allow modifying low level unbound workqueue cpumask

2015-04-07 Thread Lai Jiangshan
Allow to modify the low-level unbound workqueues cpumask through sysfs. This is performed by traversing the entire workqueue list and calling apply_wqattrs_prepare() on the unbound workqueues with the low level mask passed in. Only after all the preparation are done, we commit them all together.