diff options
| author | Jan Kiszka <jan.kiszka@web.de> | 2011-04-09 13:18:59 +0200 |
|---|---|---|
| committer | Aurelien Jarno <aurelien@aurel32.net> | 2011-04-27 20:04:51 +0200 |
| commit | 47f7be394aa7baf7855fe78f56b8ba4c69bf75d9 (patch) | |
| tree | 9633e60e984359013a156d84e3f6c3c9ac6a0cea /hw/ioapic.c | |
| parent | 661bfc80e876d32da8befe53ba0234d87fc0bcc2 (diff) | |
| download | qemu-47f7be394aa7baf7855fe78f56b8ba4c69bf75d9.zip qemu-47f7be394aa7baf7855fe78f56b8ba4c69bf75d9.tar.gz qemu-47f7be394aa7baf7855fe78f56b8ba4c69bf75d9.tar.bz2 | |
ioapic: Do not set irr for masked edge IRQs
So far we set IRR for edge IRQs even if the pin is masked. If the guest
later on unmasks and switches the pin to level-triggered mode, irr will
remain set, causing an IRQ storm. The point is that setting IRR is not
correct in this case according to the spec, and avoiding this resolves
the issue.
Reported-and-tested-by: Isaku Yamahata <yamahata@valinux.co.jp>
Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
Diffstat (limited to 'hw/ioapic.c')
| -rw-r--r-- | hw/ioapic.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/hw/ioapic.c b/hw/ioapic.c index 569327d..6c26e82 100644 --- a/hw/ioapic.c +++ b/hw/ioapic.c @@ -160,8 +160,9 @@ static void ioapic_set_irq(void *opaque, int vector, int level) s->irr &= ~mask; } } else { - /* edge triggered */ - if (level) { + /* According to the 82093AA manual, we must ignore edge requests + * if the input pin is masked. */ + if (level && !(entry & IOAPIC_LVT_MASKED)) { s->irr |= mask; ioapic_service(s); } |