Re: [qemu-s390x] [PATCH 23/35] target/s390x: access cpu->interrupt_request with atomics

[David Hildenbrand]

Am 17.09.18 um 18:30 schrieb Emilio G. Cota:
> From: Paolo Bonzini <address@hidden>
> 
> Cc: Cornelia Huck <address@hidden>
> Cc: Christian Borntraeger <address@hidden>
> Cc: Alexander Graf <address@hidden>
> Cc: Richard Henderson <address@hidden>
> Cc: David Hildenbrand <address@hidden>
> Cc: address@hidden
> Signed-off-by: Paolo Bonzini <address@hidden>
> Signed-off-by: Emilio G. Cota <address@hidden>
> ---
>  hw/intc/s390_flic.c | 2 +-
>  target/s390x/cpu.c  | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/hw/intc/s390_flic.c b/hw/intc/s390_flic.c
> index 5f8168f0f0..bce07c8a1e 100644
> --- a/hw/intc/s390_flic.c
> +++ b/hw/intc/s390_flic.c
> @@ -189,7 +189,7 @@ static void qemu_s390_flic_notify(uint32_t type)
>      CPU_FOREACH(cs) {
>          S390CPU *cpu = S390_CPU(cs);
>  
> -        cs->interrupt_request |= CPU_INTERRUPT_HARD;
> +        atomic_or(&cs->interrupt_request, CPU_INTERRUPT_HARD);
>  
>          /* ignore CPUs that are not sleeping */
>          if (s390_cpu_get_state(cpu) != S390_CPU_STATE_OPERATING &&
> diff --git a/target/s390x/cpu.c b/target/s390x/cpu.c
> index 8ed4823d6e..d4c93abc29 100644
> --- a/target/s390x/cpu.c
> +++ b/target/s390x/cpu.c
> @@ -65,7 +65,7 @@ static bool s390_cpu_has_work(CPUState *cs)
>          return false;
>      }
>  
> -    if (!(cs->interrupt_request & CPU_INTERRUPT_HARD)) {
> +    if (!(atomic_read(&cs->interrupt_request) & CPU_INTERRUPT_HARD)) {
>          return false;
>      }
>  
> 

Reviewed-by: David Hildenbrand <address@hidden>

-- 

Thanks,

David / dhildenb