diff options
Diffstat (limited to 'results/classifier/108/other/893367')
| -rw-r--r-- | results/classifier/108/other/893367 | 39 |
1 files changed, 39 insertions, 0 deletions
diff --git a/results/classifier/108/other/893367 b/results/classifier/108/other/893367 new file mode 100644 index 000000000..40b66445e --- /dev/null +++ b/results/classifier/108/other/893367 @@ -0,0 +1,39 @@ +graphic: 0.795 +device: 0.728 +performance: 0.710 +network: 0.619 +semantic: 0.591 +socket: 0.590 +vnc: 0.442 +PID: 0.438 +debug: 0.435 +other: 0.372 +files: 0.338 +boot: 0.307 +permissions: 0.273 +KVM: 0.216 + +HPET supports only one IRQ + +The emulated HPET only supports triggering IRQ 2. Since MSIs are by default disabled, this severely limits the usefulness of the HPET as only one timer block can effectively be used (otherwise they would share IRQ 2). Ideally, the HPET should support as much timer blocks as CPUs and each timer block can be driven by a different IRQ. + +TIMER: HPET at fed00000 -> 0xbf500000. +TIMER: HPET vendor 8086 revision 01: LEGACY 64BIT +TIMER: HPET: cap 8086a201 config 0 period 10000000 +TIMER: HPET Timer[0]: config 30 int 4 +TIMER: HPET Timer[1]: config 30 int 4 +TIMER: HPET Timer[2]: config 30 int 4 + +The offending code seems to be: + + /* advertise availability of ioapic inti2 */ + timer->config |= 0x00000004ULL << 32; + +in hw/hpet.c hpet_reset(). + +I assume this has been fixed by this commit here: +https://git.qemu.org/?p=qemu.git;a=commitdiff;h=7a10ef51c2397ac4323b +Or is there still something left to do here? + +No. Looks good to me. + |