[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] linux-user: add ppc signal handling, v2
From: |
Nathan Froyd |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] linux-user: add ppc signal handling, v2 |
Date: |
Tue, 12 May 2009 19:15:44 -0700 |
User-agent: |
Mutt/1.5.13 (2006-08-11) |
On Wed, May 13, 2009 at 03:13:10AM +0400, malc wrote:
> On Tue, 12 May 2009, Nathan Froyd wrote:
> > This patch series adds signal handling support for 32-bit PPC linux-user
> > emulation. It requires a bit of shuffling of target-ppc code to ensure
> > that we know what the capabilities of our current CPU are so we know
> > which registers to save.
>
> This one works as is, however the series introduces 3 instances of
> trailing whitespace (git complains) and few new tab characters (my
> eyes do).
Sorry about that; fixed in a reply to part 2/3 of the patch series.
> Plus while building it (this time with gcc 3.4.6) i noticed
> that the warnings around __put_user are gone, but lines 2746 and 2747
> of op_helper.c produce (both originate from d9430add, which is also
> yours):
>
> warning: array subscript is above array bounds
>
> Boils down to:
>
> #define VSHIFT(suffix, leftp) \
> void helper_vs##suffix (ppc_avr_t *r, ppc_avr_t *a, ppc_avr_t *b) \
> { \
> int shift = b->u8[LO_IDX*0x15] & 0x7; \
>
> I'd venture a guess that '*15' instead was meant to be there.
Indeed. I will fix this in a separate patch unless you beat me to
it. :) Thanks for the heads-up.
-Nathan
- [Qemu-devel] [PATCH 0/3] linux-user: add ppc signal handling, v2, Nathan Froyd, 2009/05/12
- [Qemu-devel] [PATCH 1/3] target-ppc: expose cpu capability flags, Nathan Froyd, 2009/05/12
- [Qemu-devel] [PATCH 3/3] support ELF_HWCAP for PPPC, Nathan Froyd, 2009/05/12
- [Qemu-devel] [PATCH 2/3] linux-user: ppc signal handling, Nathan Froyd, 2009/05/12
- Re: [Qemu-devel] [PATCH 0/3] linux-user: add ppc signal handling, v2, malc, 2009/05/12
- Re: [Qemu-devel] [PATCH 0/3] linux-user: add ppc signal handling, v2,
Nathan Froyd <=