[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] linux-user: Add signal handling support for
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v2] linux-user: Add signal handling support for x86_64 |
Date: |
Sun, 26 Feb 2017 12:39:09 +0000 |
On 26 February 2017 at 12:29, Laurent Vivier <address@hidden> wrote:
> Le 26/02/2017 à 00:04, Pranith Kumar a écrit :
>> Note that x86_64 has only _rt signal handlers. This implementation
>> attempts to share code with the x86_32 implementation.
>> + struct target_fpreg st[8];
>
> Why don't you keep the original name "_st"?
I suspect I asked for that in an earlier round of review
on the basis that names with underscores are reserved
namespace. Looking more closely at the rules though that's
a bit over-conservative for a struct field name because
_ followed by lowercase is only reserved in the global
name space (unlike __ or _ followed by uppercase, which
could be a macro). So using _st here isn't wrong, but
on the other hand there's not much reason to use the
underscore I think.
thanks
-- PMM