>>> On 29.05.15 at 17:34, <paul.durr...@citrix.com> wrote: >> From: Paul Durrant >> Sent: 29 May 2015 16:07 >> No, you'd see something. Perhaps our ACPI lid/power switch code is just >> buggy then? > > I already said this to Ian, but for the record... From my reading of the > ACPI spec (v 5.0, page 23 glossary entry) the SCI is an active low, > shareable, level-sensitive interrupt, but our code (pmtimer.c:pmt_update_sci) > seems to treat it as active high. I'll have a look at the QEMU SCI code as > reference, but maybe it is just broken.
While ACPI may prefer it to be that way, I don't think this is a requirement (both physical systems I checked just now have it level-high), and in any case subject to an interrupt source override in MADT (which hvmloader doesn't produce afaict). Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel