diff mbox

[3/5] user-exec: Delete now-unused hppa and m68k cpu_signal_handler() code

Message ID 1465897927-4719-4-git-send-email-peter.maydell@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Peter Maydell June 14, 2016, 9:52 a.m. UTC
Now that configure blocks attempts to build user-mode code on hppa
and m68k hosts, we can delete the cpu_signal_handler() implementations
for those architectures.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 user-exec.c | 60 ------------------------------------------------------------
 1 file changed, 60 deletions(-)

Comments

Laurent Vivier June 14, 2016, 9:58 a.m. UTC | #1
Le 14/06/2016 à 11:52, Peter Maydell a écrit :
> Now that configure blocks attempts to build user-mode code on hppa
> and m68k hosts, we can delete the cpu_signal_handler() implementations
> for those architectures.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Reviewed-by: Laurent Vivier <laurent@vivier.eu>

> ---
>  user-exec.c | 60 ------------------------------------------------------------
>  1 file changed, 60 deletions(-)
> 
> diff --git a/user-exec.c b/user-exec.c
> index 50e95a6..1e2449e 100644
> --- a/user-exec.c
> +++ b/user-exec.c
> @@ -494,24 +494,6 @@ int cpu_signal_handler(int host_signum, void *pinfo, void *puc)
>                               is_write, &uc->uc_sigmask);
>  }
>  
> -#elif defined(__mc68000)
> -
> -int cpu_signal_handler(int host_signum, void *pinfo,
> -                       void *puc)
> -{
> -    siginfo_t *info = pinfo;
> -    struct ucontext *uc = puc;
> -    unsigned long pc;
> -    int is_write;
> -
> -    pc = uc->uc_mcontext.gregs[16];
> -    /* XXX: compute is_write */
> -    is_write = 0;
> -    return handle_cpu_signal(pc, (unsigned long)info->si_addr,
> -                             is_write,
> -                             &uc->uc_sigmask);
> -}
> -
>  #elif defined(__ia64)
>  
>  #ifndef __ISR_VALID
> @@ -616,48 +598,6 @@ int cpu_signal_handler(int host_signum, void *pinfo,
>                               is_write, &uc->uc_sigmask);
>  }
>  
> -#elif defined(__hppa__)
> -
> -int cpu_signal_handler(int host_signum, void *pinfo,
> -                       void *puc)
> -{
> -    siginfo_t *info = pinfo;
> -    struct ucontext *uc = puc;
> -    unsigned long pc = uc->uc_mcontext.sc_iaoq[0];
> -    uint32_t insn = *(uint32_t *)pc;
> -    int is_write = 0;
> -
> -    /* XXX: need kernel patch to get write flag faster.  */
> -    switch (insn >> 26) {
> -    case 0x1a: /* STW */
> -    case 0x19: /* STH */
> -    case 0x18: /* STB */
> -    case 0x1b: /* STWM */
> -        is_write = 1;
> -        break;
> -
> -    case 0x09: /* CSTWX, FSTWX, FSTWS */
> -    case 0x0b: /* CSTDX, FSTDX, FSTDS */
> -        /* Distinguish from coprocessor load ... */
> -        is_write = (insn >> 9) & 1;
> -        break;
> -
> -    case 0x03:
> -        switch ((insn >> 6) & 15) {
> -        case 0xa: /* STWS */
> -        case 0x9: /* STHS */
> -        case 0x8: /* STBS */
> -        case 0xe: /* STWAS */
> -        case 0xc: /* STBYS */
> -            is_write = 1;
> -        }
> -        break;
> -    }
> -
> -    return handle_cpu_signal(pc, (unsigned long)info->si_addr,
> -                             is_write, &uc->uc_sigmask);
> -}
> -
>  #else
>  
>  #error host CPU specific signal handler needed
>
Richard Henderson June 14, 2016, 4:32 p.m. UTC | #2
On 06/14/2016 02:52 AM, Peter Maydell wrote:
> Now that configure blocks attempts to build user-mode code on hppa
> and m68k hosts, we can delete the cpu_signal_handler() implementations
> for those architectures.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>  user-exec.c | 60 ------------------------------------------------------------
>  1 file changed, 60 deletions(-)

Reviewed-by: Richard Henderson <rth@twiddle.net>


r~
diff mbox

Patch

diff --git a/user-exec.c b/user-exec.c
index 50e95a6..1e2449e 100644
--- a/user-exec.c
+++ b/user-exec.c
@@ -494,24 +494,6 @@  int cpu_signal_handler(int host_signum, void *pinfo, void *puc)
                              is_write, &uc->uc_sigmask);
 }
 
-#elif defined(__mc68000)
-
-int cpu_signal_handler(int host_signum, void *pinfo,
-                       void *puc)
-{
-    siginfo_t *info = pinfo;
-    struct ucontext *uc = puc;
-    unsigned long pc;
-    int is_write;
-
-    pc = uc->uc_mcontext.gregs[16];
-    /* XXX: compute is_write */
-    is_write = 0;
-    return handle_cpu_signal(pc, (unsigned long)info->si_addr,
-                             is_write,
-                             &uc->uc_sigmask);
-}
-
 #elif defined(__ia64)
 
 #ifndef __ISR_VALID
@@ -616,48 +598,6 @@  int cpu_signal_handler(int host_signum, void *pinfo,
                              is_write, &uc->uc_sigmask);
 }
 
-#elif defined(__hppa__)
-
-int cpu_signal_handler(int host_signum, void *pinfo,
-                       void *puc)
-{
-    siginfo_t *info = pinfo;
-    struct ucontext *uc = puc;
-    unsigned long pc = uc->uc_mcontext.sc_iaoq[0];
-    uint32_t insn = *(uint32_t *)pc;
-    int is_write = 0;
-
-    /* XXX: need kernel patch to get write flag faster.  */
-    switch (insn >> 26) {
-    case 0x1a: /* STW */
-    case 0x19: /* STH */
-    case 0x18: /* STB */
-    case 0x1b: /* STWM */
-        is_write = 1;
-        break;
-
-    case 0x09: /* CSTWX, FSTWX, FSTWS */
-    case 0x0b: /* CSTDX, FSTDX, FSTDS */
-        /* Distinguish from coprocessor load ... */
-        is_write = (insn >> 9) & 1;
-        break;
-
-    case 0x03:
-        switch ((insn >> 6) & 15) {
-        case 0xa: /* STWS */
-        case 0x9: /* STHS */
-        case 0x8: /* STBS */
-        case 0xe: /* STWAS */
-        case 0xc: /* STBYS */
-            is_write = 1;
-        }
-        break;
-    }
-
-    return handle_cpu_signal(pc, (unsigned long)info->si_addr,
-                             is_write, &uc->uc_sigmask);
-}
-
 #else
 
 #error host CPU specific signal handler needed