Re: [PATCH 2/4 V6] workqueue: split apply_workqueue_attrs() into 3 stages

From: Tejun Heo
Date: Mon Apr 06 2015 - 11:39:34 EST


On Thu, Apr 02, 2015 at 07:14:40PM +0800, Lai Jiangshan wrote:
> The patch doesn't have functionality changed except two minor adjustment:
> 1) free_unbound_pwq() for the error path is removed, we use the
> heavier version put_pwq_unlocked() instead since the error path
> is rare. this adjustment simplifies the code.
> 2) the memory-allocation is also moved into wq_pool_mutex.
> this is needed to avoid to do the further splitting.

And we're dropping online_cpus locking before applying the new pwq's.
Is that safe?

Thanks.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/