Re: [Qemu-devel] [PATCH] Register usb-uhci reset function.

[Blue Swirl]

On 6/16/09, Paul Brook <address@hidden> wrote:
> On Tuesday 16 June 2009, Gleb Natapov wrote:
>  > Update irq line on reset. Reseting irq line is required because
>  > racing irq from pci device will call piix3_set_irq(). piix3_set_irq()
>  > will remember current level in pci_irq_levels[]. The PIC line will be
>  > triggered if one of pci_irq_levels[] is set (depends on piix3 config).
>  > If for instance pci_irq_levels[0] and pci_irq_levels[1] are mapped to
>  > the same PIC irq and during reset pci_irq_levels[1] == 1, but device
>  > that drives pci_irq_levels[0] is initialized first the device driver
>  > will not be able to lower irq line.
>
>
> This is nonsense.
>
>  The only relevant circumstances are if the devices raises an IRQ, and is then
>  reset by software while the system is running. It's got nothing to do with
>  piix3, PCI bus interrupt sharing or system reset. If you are seeing problems
>  after a system reset then your bug lies elsewhere.

No, the scenario Gleb describes is true, piix_pci.c has internal state
for the OR of the IRQs coming in and the state is not reset because
there is no reset handler. This is buggy.

The real fix is to implement piix3 reset correctly, not like Gleb
proposes now but 3/3 of his earlier patches.

As a less attractive alternative, piix3 could store the IRQ state for
each incoming IRQ, not OR state. Then all the devices would need to
call lower_irq to reach correct reset state at piix3.