Re: [Qemu-devel] [PATCH] msi: remove return code for msi_init()

[Marcel Apfelbaum <marcel@redhat.com>]

On 31/05/2017 11:28, Paolo Bonzini wrote:
>> Ok before I move on let's see whether this is what we want...
>>
>> - firstly, find all machine types:
>>
>>    pxdev:qemu [edu-fix]# grep -R ".parent = TYPE_MACHINE" * | wc
>>       49     196    2269
>>
>>    so now we have 49 kinds of machines.
>>
>> - rename msi_nonbroken into msi_broken, then:
>>
>>    - x86/arm/spapr/s390 machines are the only ones that don't need to
>>      set msi_broken since they support MSI and have msi_nonbroken set,
>>      either in board init function or in irq chip init function
>>
>>    - for all the rest of the machines, I should add "msi_broken" in its
>>      machine init() function.
>>
>> Is this really what we want?
> No, for now I'd rather just go and remove msi_nonbroken.  When someone
> reports a bug, we can add back "msi_broken".

Hi,
I agree with the direction, but I am concerned msi_nonbroken is there 
for a reason.
We might break some (obscure/not in use) machine.
Maybe we should CC all arch machine maintainers/contributors to give 
them a chance
to object...

Thanks,
Marcel

> Paolo