[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/4] QEMU_PACKED: Remove gcc_struct attribute
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/4] QEMU_PACKED: Remove gcc_struct attribute in Windows non x86 targets |
Date: |
Fri, 3 May 2019 18:23:15 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 |
On 5/3/19 5:56 PM, Marc-André Lureau wrote:
> Hi
>
> Le ven. 3 mai 2019 à 17:23, Peter Maydell <address@hidden> a
> écrit :
>
>> On Fri, 3 May 2019 at 06:07, Thomas Huth <address@hidden> wrote:
>>>
>>> On 03/05/2019 02.36, Cao Jiaxi wrote:
>>>> gcc_struct is for x86 only, and it generates an warning on ARM64
>> Clang/MinGW targets.
>>>>
>>>> Signed-off-by: Cao Jiaxi <address@hidden>
>>>> ---
>>>> contrib/libvhost-user/libvhost-user.h | 2 +-
>>>> include/qemu/compiler.h | 2 +-
>>>> scripts/cocci-macro-file.h | 7 ++++++-
>>>> slirp/src/util.h | 2 +-
>>>> 4 files changed, 9 insertions(+), 4 deletions(-)
>>
>>>> diff --git a/slirp/src/util.h b/slirp/src/util.h
>>>> index 01f1e0e068..278828fe3f 100644
>>>> --- a/slirp/src/util.h
>>>> +++ b/slirp/src/util.h
>>>> @@ -43,7 +43,7 @@
>>>> #include <netinet/in.h>
>>>> #endif
>>>>
>>>> -#if defined(_WIN32)
>>>> +#if defined(_WIN32) && (defined(__x86_64__) || defined(__i386__))
>>>> # define SLIRP_PACKED __attribute__((gcc_struct, packed))
>>>> #else
>>>> # define SLIRP_PACKED __attribute__((packed))
>>>>
>>>
>>> The slirp code is currently on its way into a separate module, so you
>>> might need to provide that hunk to the libslirp folks again... I'm
>>> putting the slirp maintainers on CC:, maybe they can pick it up from
>> here.
>>
>> Yes, the slirp module has now landed in master, so this patch
>> definitely needs to be split into two. I've kept in my
>> target-arm.next tree the parts which are applicable to
>> the QEMU repo itself (ie everything except the slirp/ change),
>> so we just need a new patch for the slirp submodule part.
>>
>> Marc-André, Samuel -- what's the process for submitting and
>> getting reviewed changes to the slirp submodule now it's a
>> separate project ?
>>
>
> It's hosted on gitlab.freedesktop.org, with some CI. It's fine to send
> patches on qemu devel, as long as Samuel or I are in CC. But in the long
> term, I think gitlab MR will be favoured (after a while using it, I think
> gitlab is better than ML for patches & bug tracking tbh)
Except when working offline (or with poor intermittent link).
[Qemu-devel] [PATCH v3 2/4] qga: Fix mingw compilation warnings on enum conversion, Cao Jiaxi, 2019/05/02
[Qemu-devel] [PATCH v3 3/4] util/cacheinfo: Use uint64_t on LLP64 model to satisfy Windows ARM64, Cao Jiaxi, 2019/05/02
[Qemu-devel] [PATCH v3 4/4] osdep: Fix mingw compilation regarding stdio formats, Cao Jiaxi, 2019/05/02
Re: [Qemu-devel] [PATCH v3 0/4] Initial Windows on ARM (AArch64 64-Bit) host support, Philippe Mathieu-Daudé, 2019/05/03