All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
To: Richard Henderson <rth@twiddle.net>
Cc: qemu-devel@nongnu.org, aurelien@aurel32.net,
	Richard Henderson <rth@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 1/7] microblaze: target-ify target_ucontext
Date: Tue, 23 Nov 2010 10:05:56 +0100	[thread overview]
Message-ID: <20101123090556.GA2876@edde.se.axis.com> (raw)
In-Reply-To: <1290466678-6337-2-git-send-email-rth@twiddle.net>

On Mon, Nov 22, 2010 at 02:57:52PM -0800, Richard Henderson wrote:
> From: Richard Henderson <rth@redhat.com>
> 
> Rename the members of target_ucontext so that they don't conflict
> with possible host macros for ucontext members.  This has already
> been done for the other targets.

Thanks, I applied this one.

Cheers


> 
> Signed-off-by: Richard Henderson <rth@twiddle.net>
> ---
>  linux-user/signal.c |   18 +++++++++---------
>  1 files changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/linux-user/signal.c b/linux-user/signal.c
> index 77683f7..7c62fac 100644
> --- a/linux-user/signal.c
> +++ b/linux-user/signal.c
> @@ -3071,11 +3071,11 @@ struct target_stack_t {
>  };
>  
>  struct target_ucontext {
> -    abi_ulong uc_flags;
> -    abi_ulong uc_link;
> -    struct target_stack_t uc_stack;
> -    struct target_sigcontext sc;
> -    uint32_t extramask[TARGET_NSIG_WORDS - 1];
> +    abi_ulong tuc_flags;
> +    abi_ulong tuc_link;
> +    struct target_stack_t tuc_stack;
> +    struct target_sigcontext tuc_mcontext;
> +    uint32_t tuc_extramask[TARGET_NSIG_WORDS - 1];
>  };
>  
>  /* Signal frames. */
> @@ -3189,7 +3189,7 @@ static void setup_frame(int sig, struct target_sigaction *ka,
>          goto badframe;
>  
>      /* Save the mask.  */
> -    err |= __put_user(set->sig[0], &frame->uc.sc.oldmask);
> +    err |= __put_user(set->sig[0], &frame->uc.tuc_mcontext.oldmask);
>      if (err)
>          goto badframe;
>  
> @@ -3198,7 +3198,7 @@ static void setup_frame(int sig, struct target_sigaction *ka,
>              goto badframe;
>      }
>  
> -    setup_sigcontext(&frame->uc.sc, env);
> +    setup_sigcontext(&frame->uc.tuc_mcontext, env);
>  
>      /* Set up to return from userspace. If provided, use a stub
>         already in userspace. */
> @@ -3261,7 +3261,7 @@ long do_sigreturn(CPUState *env)
>          goto badframe;
>  
>      /* Restore blocked signals */
> -    if (__get_user(target_set.sig[0], &frame->uc.sc.oldmask))
> +    if (__get_user(target_set.sig[0], &frame->uc.tuc_mcontext.oldmask))
>          goto badframe;
>      for(i = 1; i < TARGET_NSIG_WORDS; i++) {
>          if (__get_user(target_set.sig[i], &frame->extramask[i - 1]))
> @@ -3270,7 +3270,7 @@ long do_sigreturn(CPUState *env)
>      target_to_host_sigset_internal(&set, &target_set);
>      sigprocmask(SIG_SETMASK, &set, NULL);
>  
> -    restore_sigcontext(&frame->uc.sc, env);
> +    restore_sigcontext(&frame->uc.tuc_mcontext, env);
>      /* We got here through a sigreturn syscall, our path back is via an
>         rtb insn so setup r14 for that.  */
>      env->regs[14] = env->sregs[SR_PC];
> -- 
> 1.7.2.3
> 
> 

  reply	other threads:[~2010-11-23  9:25 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-22 22:57 [Qemu-devel] [PATCH 0/7] Fixes for ia64-linux host Richard Henderson
2010-11-22 22:57 ` [Qemu-devel] [PATCH 1/7] microblaze: target-ify target_ucontext Richard Henderson
2010-11-23  9:05   ` Edgar E. Iglesias [this message]
2010-11-22 22:57 ` [Qemu-devel] [PATCH 2/7] tcg: Fix default definition of divu_i32 and remu_i32 Richard Henderson
2010-11-22 22:57 ` [Qemu-devel] [PATCH 3/7] tcg-ia64: Provide default GUEST_BASE Richard Henderson
2010-11-22 22:57 ` [Qemu-devel] [PATCH 4/7] tcg-ia64: Implement qemu_ld32 Richard Henderson
2010-11-22 22:57 ` [Qemu-devel] [PATCH 5/7] tcg-ia64: Fix tlb read error for 32-bit targets Richard Henderson
2010-11-22 22:57 ` [Qemu-devel] [PATCH 6/7] tcg-ia64: Fix address compilation in qemu_st Richard Henderson
2010-11-22 22:57 ` [Qemu-devel] [PATCH 7/7] tcg-ia64: Fix warning in qemu_ld Richard Henderson
2010-11-23  9:09   ` malc
2010-11-23 15:18     ` Richard Henderson
2010-12-01 19:28 ` [Qemu-devel] [PATCH 0/7] Fixes for ia64-linux host Edgar E. Iglesias

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20101123090556.GA2876@edde.se.axis.com \
    --to=edgar.iglesias@gmail.com \
    --cc=aurelien@aurel32.net \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@redhat.com \
    --cc=rth@twiddle.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.