diff options
| author | Mathieu Malaterre <malat@debian.org> | 2018-05-22 21:47:32 +0200 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2018-05-23 20:16:58 +0200 |
| commit | 66448bc274cadedb71fda7d914e7c29d8dead217 (patch) | |
| tree | a995effcf0fa2a7a5b08381cca9e6a337ab86b6b /kernel | |
| parent | workqueue: Make sure struct worker is accessible for wq_worker_comm() (diff) | |
| download | linux-66448bc274cadedb71fda7d914e7c29d8dead217.tar.xz linux-66448bc274cadedb71fda7d914e7c29d8dead217.zip | |
workqueue: move function definitions within CONFIG_SMP block
In commit 7ee681b25284 ("workqueue: Convert to state machine callbacks"),
three new function definitions were added: ‘workqueue_prepare_cpu’,
‘workqueue_online_cpu’ and ‘workqueue_offline_cpu’.
Move these function definitions within a CONFIG_SMP block since they are
not used outside of it. This will match function declarations in header
<include/linux/workqueue.h>, and silence the following gcc warning (W=1):
kernel/workqueue.c:4743:5: warning: no previous prototype for ‘workqueue_prepare_cpu’ [-Wmissing-prototypes]
kernel/workqueue.c:4756:5: warning: no previous prototype for ‘workqueue_online_cpu’ [-Wmissing-prototypes]
kernel/workqueue.c:4783:5: warning: no previous prototype for ‘workqueue_offline_cpu’ [-Wmissing-prototypes]
Signed-off-by: Mathieu Malaterre <malat@debian.org>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/workqueue.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index 60d6fd2636f3..7ea75529eabb 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -4626,6 +4626,8 @@ void wq_worker_comm(char *buf, size_t size, struct task_struct *task) mutex_unlock(&wq_pool_attach_mutex); } +#ifdef CONFIG_SMP + /* * CPU hotplug. * @@ -4846,8 +4848,6 @@ int workqueue_offline_cpu(unsigned int cpu) return 0; } -#ifdef CONFIG_SMP - struct work_for_cpu { struct work_struct work; long (*fn)(void *); |