qemu-devel.nongnu.org archive mirror
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).