Am 17.09.18 um 18:30 schrieb Emilio G. Cota: > From: Paolo Bonzini <[email protected]> > > Cc: Cornelia Huck <[email protected]> > Cc: Christian Borntraeger <[email protected]> > Cc: Alexander Graf <[email protected]> > Cc: Richard Henderson <[email protected]> > Cc: David Hildenbrand <[email protected]> > Cc: [email protected] > Signed-off-by: Paolo Bonzini <[email protected]> > Signed-off-by: Emilio G. Cota <[email protected]> > --- > 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 <[email protected]> -- Thanks, David / dhildenb
