[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 11/22] i8259: Update IRQ state after reset
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 11/22] i8259: Update IRQ state after reset |
Date: |
Wed, 28 Sep 2011 18:42:13 +0000 |
On Wed, Sep 28, 2011 at 6:09 PM, Peter Maydell <address@hidden> wrote:
> On 28 September 2011 19:01, Blue Swirl <address@hidden> wrote:
>> On Wed, Sep 28, 2011 at 11:00 AM, Jan Kiszka <address@hidden> wrote:
>>> As we clearly modify the PIC state on pic_reset, we also have to update
>>> the IRQ output. This only happened on init so far. Apply this
>>> consistently.
>>
>> Nack, IRQ lines shouldn't be touched on reset. The other side may not
>> be ready for receiving the interrupt change and qemu_irqs are
>> stateless anyway.
>
> So how does this work for a device whose reset state is "this
> irq/gpio line is asserted" ?
The polarity could be reversed but that wouldn't work if the signal is
used in several places.
But the duration of reset in QEMU is infinitely small, nothing is
expected to happen during that. This doesn't match real HW though, the
reset could take several hundred milliseconds but I doubt modeling
that would be interesting.
We also assume that the reset states of devices and their outputs are
coherent i.e. the output of one device during reset would not change
the state of another device from its reset state.
- Re: [Qemu-devel] [PATCH 21/22] monitor: Restrict pic/irq_info to supporting targets, (continued)
[Qemu-devel] [PATCH 17/22] i8259: Eliminate PicState2, Jan Kiszka, 2011/09/28
[Qemu-devel] [PATCH 11/22] i8259: Update IRQ state after reset, Jan Kiszka, 2011/09/28
Re: [Qemu-devel] [PATCH 11/22] i8259: Update IRQ state after reset, Jan Kiszka, 2011/09/28
Re: [Qemu-devel] [PATCH 11/22] i8259: Update IRQ state after reset, Blue Swirl, 2011/09/29
Re: [Qemu-devel] [PATCH 11/22] i8259: Update IRQ state after reset, Jan Kiszka, 2011/09/30
Re: [Qemu-devel] [PATCH 11/22] i8259: Update IRQ state after reset, Peter Maydell, 2011/09/30
Re: [Qemu-devel] [PATCH 11/22] i8259: Update IRQ state after reset, Blue Swirl, 2011/09/30
Re: [Qemu-devel] [PATCH 11/22] i8259: Update IRQ state after reset, Blue Swirl, 2011/09/30
[Qemu-devel] [PATCH 15/22] i8259: PREP: Replace pic_intack_read with pic_read_irq, Jan Kiszka, 2011/09/28