Am 17.09.18 um 18:30 schrieb Emilio G. Cota:
> From: Paolo Bonzini <pbonz...@redhat.com>
>
> Cc: Cornelia Huck <coh...@redhat.com>
> Cc: Christian Borntraeger <borntrae...@de.ibm.com>
> Cc: Alexander Graf <ag...@suse.de>
> Cc: Richard Henderson <r...@twiddle.net>
> Cc: David Hildenbrand <da...@redhat.com>
> Cc: qemu-s3...@nongnu.org
> Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
> Signed-off-by: Emilio G. Cota <c...@braap.org>
> ---
> 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 <da...@redhat.com>
--
Thanks,
David / dhildenb
