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
>
>
next prev parent 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).