Re: [Qemu-ppc] device crash test -- isa-fdc on ppc64/powernv

[Philippe Mathieu-Daudé]

Hi,

>>>>>>>> Should the powernv board be initializing something like this, or is it
>>>>>>>> more correct to just disable the isa-fdc device for this board for now
>>>>>>>> and be done with it?
>>>>>>>
>>>>>>> I don't know the powernv board very well, but maybe Cédric could answer
>>>>>>> that question.
>>>>>>>
>>>>>>> For reference, I assume you're talking about this crash here:
>>>>>>>
>>>>>>> qemu-system-ppc64 -M powernv -device isa-fdc
>>>>>>> qemu-system-ppc64: hw/block/fdc.c:2703: isabus_fdc_realize: Assertion
>>>>>>> `fdctrl->dma' failed.
>>>>>>> Aborted (core dumped)
>>>>>>>
>>>>>>> ?
>>>>>>>
>>>>>>> IIRC Eduardo also posted a patch for that one already, not sure about
>>>>>>> the current status, though.
>>>>>>
>>>>>> Eduardo Otubo posted a patch for a different case: assertion
>>>>>> failure inside DMA_init() when another device already called
>>>>>> DMA_init() (e.g. if two i82374 devices are instantiated).  The
>>>>>> fix was to make i82374 non-user-creatable.
[...]
>> PMD, is this fix in your queue?

I just sent it (still not sure this is what Eduardo expected):
http://lists.nongnu.org/archive/html/qemu-devel/2017-10/msg03769.html

This is not related to your isa-fdc bug however.

I just noticed this patch thread and from John mail:
"the way we assign IRQs to ISA devices is kind of simple and not
trivial to undo correctly"
(https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg00347.html)

I also hit this in my series "remove i386/pc dependency from non-PC world".
So far I plan to send my work in 4 parts, the ISA bus codebase would
end PC-agnostic.

My current problem is indeed inter-devices IRQ circular dependencies,
solved in the Malta board using qemu_irq_proxy(). Then not sure yet
how to unrealize them :)

Regards,

Phil.