diff options
| author | Breno Leitao <leitao@debian.org> | 2026-05-24 08:19:56 -0700 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2026-05-27 08:22:56 -1000 |
| commit | 611583a76ea97991b0f65ec1ff099eac7fe0bae4 (patch) | |
| tree | 2c9e10e9b439952e87e9dc4b7cde60e0f46f4be9 /include | |
| parent | 1503043fd75e29ad49c7d506232e272f6951d07d (diff) | |
workqueue: drop spurious '*' from print_worker_info() fn declaration
print_worker_info() declares its local 'fn' as work_func_t * but
worker->current_func has type work_func_t (a function pointer). The
extra level of indirection is wrong and only happens to be harmless
today because every supported Linux architecture has
sizeof(work_func_t) == sizeof(work_func_t *):
copy_from_kernel_nofault() reads the correct number of bytes by
accident, and %ps still resolves the printed address because the
stored value is the function address regardless of declared type.
On any future ABI where sizeof(void (*)()) differs from
sizeof(void *), the nofault copy would transfer the wrong number of
bytes and the subsequent %ps would print an incorrect address.
Match the field type so the intent is explicit and the code does not
silently rely on equal pointer sizes.
Fixes: 3d1cb2059d93 ("workqueue: include workqueue info when printing debug dump of a worker task")
Signed-off-by: Breno Leitao <leitao@debian.org>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions
