arm64: Remove the aux_context structure
authorCatalin Marinas <catalin.marinas@arm.com>
Fri, 4 Apr 2014 14:42:16 +0000 (15:42 +0100)
committerCatalin Marinas <catalin.marinas@arm.com>
Fri, 9 May 2014 14:47:48 +0000 (15:47 +0100)
This patch removes the aux_context structure (and the containing file)
to allow the placement of the _aarch64_ctx end magic based on the
context stored on the signal stack.

Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
arch/arm64/include/asm/sigcontext.h [deleted file]
arch/arm64/kernel/signal.c

diff --git a/arch/arm64/include/asm/sigcontext.h b/arch/arm64/include/asm/sigcontext.h
deleted file mode 100644 (file)
index dca1094..0000000
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * Copyright (C) 2012 ARM Ltd.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program.  If not, see <http://www.gnu.org/licenses/>.
- */
-#ifndef __ASM_SIGCONTEXT_H
-#define __ASM_SIGCONTEXT_H
-
-#include <uapi/asm/sigcontext.h>
-
-/*
- * Auxiliary context saved in the sigcontext.__reserved array. Not exported to
- * user space as it will change with the addition of new context. User space
- * should check the magic/size information.
- */
-struct aux_context {
-       struct fpsimd_context fpsimd;
-       /* additional context to be added before "end" */
-       struct _aarch64_ctx end;
-};
-#endif
index 890a591f75dd1848e28e1ea62413775be5394c6d..7ff2eee96c6b26b883ebf3aae66bf0f6ff084ba8 100644 (file)
@@ -100,8 +100,7 @@ static int restore_sigframe(struct pt_regs *regs,
 {
        sigset_t set;
        int i, err;
-       struct aux_context __user *aux =
-               (struct aux_context __user *)sf->uc.uc_mcontext.__reserved;
+       void *aux = sf->uc.uc_mcontext.__reserved;
 
        err = __copy_from_user(&set, &sf->uc.uc_sigmask, sizeof(set));
        if (err == 0)
@@ -121,8 +120,11 @@ static int restore_sigframe(struct pt_regs *regs,
 
        err |= !valid_user_regs(&regs->user_regs);
 
-       if (err == 0)
-               err |= restore_fpsimd_context(&aux->fpsimd);
+       if (err == 0) {
+               struct fpsimd_context *fpsimd_ctx =
+                       container_of(aux, struct fpsimd_context, head);
+               err |= restore_fpsimd_context(fpsimd_ctx);
+       }
 
        return err;
 }
@@ -167,8 +169,8 @@ static int setup_sigframe(struct rt_sigframe __user *sf,
                          struct pt_regs *regs, sigset_t *set)
 {
        int i, err = 0;
-       struct aux_context __user *aux =
-               (struct aux_context __user *)sf->uc.uc_mcontext.__reserved;
+       void *aux = sf->uc.uc_mcontext.__reserved;
+       struct _aarch64_ctx *end;
 
        /* set up the stack frame for unwinding */
        __put_user_error(regs->regs[29], &sf->fp, err);
@@ -185,12 +187,17 @@ static int setup_sigframe(struct rt_sigframe __user *sf,
 
        err |= __copy_to_user(&sf->uc.uc_sigmask, set, sizeof(*set));
 
-       if (err == 0)
-               err |= preserve_fpsimd_context(&aux->fpsimd);
+       if (err == 0) {
+               struct fpsimd_context *fpsimd_ctx =
+                       container_of(aux, struct fpsimd_context, head);
+               err |= preserve_fpsimd_context(fpsimd_ctx);
+               aux += sizeof(*fpsimd_ctx);
+       }
 
        /* set the "end" magic */
-       __put_user_error(0, &aux->end.magic, err);
-       __put_user_error(0, &aux->end.size, err);
+       end = aux;
+       __put_user_error(0, &end->magic, err);
+       __put_user_error(0, &end->size, err);
 
        return err;
 }