]> err.no Git - linux-2.6/commitdiff
[ARM] Remove rt_sigframe puc and pinfo pointers
authorRussell King <rmk@dyn-67.arm.linux.org.uk>
Thu, 15 Jun 2006 17:00:34 +0000 (18:00 +0100)
committerRussell King <rmk+kernel@arm.linux.org.uk>
Sun, 18 Jun 2006 15:17:41 +0000 (16:17 +0100)
These two members appear to be surplus to requirements.  Discussing
this issue with glibc folk:

| > Additionally, do you see any need for these weird "puc" and "pinfo"
| > pointers in the kernels rt_sigframe structure?  Can we kill them?
|
| We can kill them.  I checked with Phil B. about them last week, and he
| didn't remember any reason they still needed to be there.  And nothing
| should know where they are on the stack.  Unfortunately, doing this
| will upset GDB, which knows that the saved registers are 0x88 bytes
| above the stack pointer on entrance to an rt signal trampoline; but,
| since puc and pinfo are quite recognizable, I can adapt GDB to support
| the new layout if you want to remove them.

So remove them.

Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
arch/arm/kernel/signal.c

index a0cd0a90a10d06f4bd266db044937796337c1eb3..0ddbf2795c5b058633beb58bde12b3e863337e8f 100644 (file)
@@ -200,8 +200,6 @@ struct sigframe {
 };
 
 struct rt_sigframe {
-       struct siginfo __user *pinfo;
-       void __user *puc;
        struct siginfo info;
        struct ucontext uc;
        unsigned long retcode[2];
@@ -511,8 +509,6 @@ setup_rt_frame(int usig, struct k_sigaction *ka, siginfo_t *info,
        if (!frame)
                return 1;
 
-       __put_user_error(&frame->info, &frame->pinfo, err);
-       __put_user_error(&frame->uc, &frame->puc, err);
        err |= copy_siginfo_to_user(&frame->info, info);
 
        __put_user_error(0, &frame->uc.uc_flags, err);