diff options
author | Helge Deller <deller@gmx.de> | 2020-11-13 03:53:31 +0100 |
---|---|---|
committer | Eric W. Biederman <ebiederm@xmission.com> | 2020-11-23 17:31:04 +0100 |
commit | 8663daeac7a1fd1b200e3365ccc9403f026f2fc8 (patch) | |
tree | ec6763f0663d51b0d9f7ca8a20d432cb5154fb22 /arch/h8300 | |
parent | Linux 5.10-rc3 (diff) | |
download | linux-8663daeac7a1fd1b200e3365ccc9403f026f2fc8.tar.xz linux-8663daeac7a1fd1b200e3365ccc9403f026f2fc8.zip |
parisc: Drop parisc special case for __sighandler_t
I believe we can and *should* drop this parisc-specific typedef for
__sighandler_t when compiling a 64-bit kernel. The reasons:
1. We don't have a 64-bit userspace yet, so nothing (on userspace side)
can break.
2. Inside the Linux kernel, this is only used in kernel/signal.c, in
function kernel_sigaction() where the signal handler is compared against
SIG_IGN. SIG_IGN is defined as (__sighandler_t)1), so only the pointers
are compared.
3. Even when a 64-bit userspace gets added at some point, I think
__sighandler_t should be defined what it is: a function pointer struct.
I compiled kernel/signal.c with and without the patch, and the produced code
is identical in both cases.
Signed-off-by: Helge Deller <deller@gmx.de>
Signed-off-by: Peter Collingbourne <pcc@google.com>
Acked-by: "Eric W. Biederman" <ebiederm@xmission.com>
Reviewed-by: Peter Collingbourne <pcc@google.com>
Link: https://linux-review.googlesource.com/id/I21c43f21b264f339e3aa395626af838646f62d97
Link: https://lkml.kernel.org/r/a75b8eb7bb9eac1cf73fb119eb53e5892d6e9656.1605235762.git.pcc@google.com
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Diffstat (limited to 'arch/h8300')
0 files changed, 0 insertions, 0 deletions