qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 05/10] target/s390x: Remove duplicated ifdef macro


From: Cornelia Huck
Subject: Re: [PATCH v2 05/10] target/s390x: Remove duplicated ifdef macro
Date: Tue, 21 Jan 2020 14:52:51 +0100

On Tue, 21 Jan 2020 12:03:44 +0100
Philippe Mathieu-Daudé <address@hidden> wrote:

> Commit ae71ed8610 replaced the use of global max_cpus variable
> with a machine property, but introduced a unnecessary ifdef, as
> this block is already in the 'not CONFIG_USER_ONLY' branch part:
> 
>    86 #if defined(CONFIG_USER_ONLY)
>    87
>   ...
>   106 #else /* !CONFIG_USER_ONLY */
>   107
>   ...
>   292 static void do_ext_interrupt(CPUS390XState *env)
>   293 {
>   ...
>   313 #ifndef CONFIG_USER_ONLY
>   314         MachineState *ms = MACHINE(qdev_get_machine());
>   315         unsigned int max_cpus = ms->smp.max_cpus;
>   316 #endif
> 
> To ease code review, remove the duplicated preprocessor macro,
> and move the declarations at the beginning of the statement.
> 
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> v2: New patch
> Cc: Like Xu <address@hidden>
> ---
>  target/s390x/excp_helper.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)

Thanks, applied.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]