[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH] fix compilation on FreeBSD for mac_*
From: |
Andreas Tobler |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH] fix compilation on FreeBSD for mac_*world.c |
Date: |
Thu, 28 Nov 2013 23:48:29 +0100 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:24.0) Gecko/20100101 Thunderbird/24.1.1 |
On 28.11.13 23:38, Peter Maydell wrote:
> On 28 November 2013 21:33, Andreas Tobler <address@hidden> wrote:
>> Hello,
>>
>> On 09.11.13 00:15, Andreas Tobler wrote:
>>> Hello,
>>>
>>> the attached patch fixes compilation on FreeBSD for
>>> hw/ppc/mac_new/oldworld.c on FreeBSD.
>>>
>>> The affected function is present in machine/param.h on FreeBSD:
>>>
>>> #define round_page(x) ((((unsigned long)(x)) + PAGE_MASK) & ~(PAGE_MASK))
>>>
>>> To make it compile on FreeBSD I'd like to propose the below patch which
>>> is in use since a longer time on the ports tree (FreeBSD) and in my
>>> personal git tree.
>
> Rather than doing this:
> + #if defined(__FreeBSD__)
> + #undef round_page
> + #endif
>
> personally I'd prefer it if we just replaced the uses of round_page()
> with TARGET_PAGE_ALIGN(), which is the macro we define in
> the headers for aligning up to the next page size.
>
> Up to the ppc submaintainers though...
Whatever you prefer, I'm ready to test....
Thanks for the feedback.
Andreas