On 09/08/2018 15:19, Emanuele wrote: > > > On 08/09/2018 03:02 PM, Laurent Vivier wrote: >> On 06/08/2018 16:33, Emanuele Giuseppe Esposito wrote: >>> +void qpci_init_spapr(QPCIBusSPAPR *ret, QTestState *qts, >>> QGuestAllocator *alloc) >>> +{ >>> assert(qts); >>> + /* tests cannot use spapr, needs to be fixed first */ >>> + ret->bus.has_buggy_msi = TRUE; >>> + >> Perhaps you can revert the logic and call that "msi_is_available"? >> It's more generic than "buggy". It says if we can or not use MSI. > This name was suggested by Paolo, I honestly don't know which is better, > maybe it's better to ask him for this.
The MSIs here are not interrupts, they are just random memory writes. Even if it's not clear if the bug is in libqos or QEMU, there is no reason why they shouldn't work. Paolo