[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] build: Add include check on syscall.h
From: |
Lluís Vilanova |
Subject: |
Re: [Qemu-devel] [PATCH] build: Add include check on syscall.h |
Date: |
Fri, 29 Jan 2016 14:30:44 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Peter Maydell writes:
> On 28 January 2016 at 20:36, Lluís Vilanova <address@hidden> wrote:
>> The LTTng tracing backend includes the system's "syscall.h", but QEMU
>> replaces it with its own for linux-user builds. This results in a double
>> include on some targets (when LTTng is enabled).
>>
>> Signed-off-by: Lluís Vilanova <address@hidden>
> Adding include guards is fine, but it sounds to me like what we
> should actually do to fix this confusion is rename all the linux-user
> local headers to target_syscall.h.
Hmmm, I didn't know if using the same name was on purpose or not. If the
intention was *not* to override the system's syscall.h, then a rename is the
proper solution.
Can you or someone else confirm this?
Thanks,
Lluis