workqueue: Work around edge cases for calc of pool's cpumask
authorMichael Bringmann <mwb@linux.vnet.ibm.com>
Thu, 27 Jul 2017 21:27:14 +0000 (16:27 -0500)
committerTejun Heo <tj@kernel.org>
Fri, 28 Jul 2017 15:05:52 +0000 (11:05 -0400)
commit1ad0f0a7aa1bf3bd42dcd108a96713d255eacd9f
treeadad9ec5dd0a10141e3b30b57c2fa83264c77e18
parent0a94efb5acbb6980d7c9ab604372d93cd507e4d8
workqueue: Work around edge cases for calc of pool's cpumask

There is an underlying assumption/trade-off in many layers of the Linux
system that CPU <-> node mapping is static.  This is despite the presence
of features like NUMA and 'hotplug' that support the dynamic addition/
removal of fundamental system resources like CPUs and memory.  PowerPC
systems, however, do provide extensive features for the dynamic change
of resources available to a system.

Currently, there is little or no synchronization protection around the
updating of the CPU <-> node mapping, and the export/update of this
information for other layers / modules.  In systems which can change
this mapping during 'hotplug', like PowerPC, the information is changing
underneath all layers that might reference it.

This patch attempts to ensure that a valid, usable cpumask attribute
is used by the workqueue infrastructure when setting up new resource
pools.  It prevents a crash that has been observed when an 'empty'
cpumask is passed along to the worker/task scheduling code.  It is
intended as a temporary workaround until a more fundamental review and
correction of the issue can be done.

[With additions to the patch provided by Tejun Hao <tj@kernel.org>]

Signed-off-by: Michael Bringmann <mwb@linux.vnet.ibm.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
kernel/workqueue.c