[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/10] linux-user: Use #if to only call validate
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 01/10] linux-user: Use #if to only call validate_guest_space for 32-bit ARM target |
Date: |
Fri, 23 Feb 2018 18:48:39 +0000 |
On 23 February 2018 at 18:35, Peter Maydell <address@hidden> wrote:
> On 28 December 2017 at 18:08, Luke Shumaker <address@hidden> wrote:
>> From: Luke Shumaker <address@hidden>
>>
>> Instead of defining a bogus validate_guest_space that always returns 1 on
>> targets other than 32-bit ARM, use #if blocks to only call it on 32-bit ARM
>> targets. This makes the "normal" flow control clearer.
>>
>> Signed-off-by: Luke Shumaker <address@hidden>
>> @@ -1845,11 +1835,12 @@ unsigned long init_guest_space(unsigned long
>> host_start,
>> /* If just a starting address is given, then just verify that
>> * address. */
>> if (host_start && !host_size) {
>> +#if defined(TARGET_ARM) && !defined(TARGET_AARCH64)
>
> I would strongly prefer us not to add new "these targets do
> this" ifdefs, please. The current approach means that any
> target can say it needs an implementation of this hook by
> providing one and defining the TARGET_HAS_VALIDATE_GUEST_SPACE
> macro to say so. I think that's a better approach.
Looking through some of the rest of this patchset I might change
my mind on that (the code in master is very confusing). I won't
have time to get to this til Tuesday now, though.
thanks
-- PMM