[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SM
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI |
Date: |
Wed, 23 Nov 2016 16:48:02 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.0 |
On 11/18/16 15:10, Michael S. Tsirkin wrote:
> On Fri, Nov 18, 2016 at 11:36:56AM +0100, Laszlo Ersek wrote:
>> This is v3 of the series, with updates based on the v2 discussion:
>> <http://lists.nongnu.org/archive/html/qemu-devel/2016-11/msg02687.html>.
>>
>> I've added feature negotiation via the APM_STS ("scratchpad") register.
>> A new spec file called "docs/specs/q35-apm-sts.txt" is included.
>>
>> Tested with new OVMF patches (about to send out those as well).
>> Regression tested with SeaBIOS (beyond simple functional tests with
>> maximum SeaBIOS logging enabled, I used gdb to step through the new
>> ich9_apm_status_changed() callback to see if it was behaving compatibly
>> with SeaBIOS).
>>
>> The series was developed and tested on top of v2.7.0, because v2.8.0-rc0
>> crashes very quickly for me when running OVMF:
>>
>> kvm_io_ioeventfd_add: error adding ioeventfd: File exists
>>
>> It is my understanding that there are patches on the list for this:
>>
>> [Qemu-devel] [PATCH v2 for-2.8 0/3] virtio fixes
>>
>> Anyway, the series rebases to v2.8.0-rc0 without as much as context
>> differences.
>>
>> Cc: "Kevin O'Connor" <address@hidden>
>> Cc: "Michael S. Tsirkin" <address@hidden>
>> Cc: Gerd Hoffmann <address@hidden>
>> Cc: Paolo Bonzini <address@hidden>
>
> I'll review.
Can you please review the
docs/specs/q35-apm-sts.txt
hunks across the series? The OVMF series has been reviewed and it's
ready to go in.
If you approve the interface spec, I can safely commit the OVMF patches
(exactly because the feature is now negotiated), without having to wait
for the QEMU code to converge.
> Pls remember it will have to be re-posted or pinged
> after 2.8 is out.
Sure, will do.
Thanks!
Laszlo
>
>> Thanks
>> Laszlo
>>
>> Laszlo Ersek (3):
>> hw/isa/apm: introduce callback for APM_STS_IOPORT writes
>> hw/isa/lpc_ich9: add SMI feature negotiation via APM_STS
>> hw/isa/lpc_ich9: ICH9_APM_STS_F_BROADCAST_SMI: inject SMI on all VCPUs
>>
>> docs/specs/q35-apm-sts.txt | 80
>> ++++++++++++++++++++++++++++++++++++++++++++++
>> include/hw/i386/ich9.h | 9 ++++++
>> include/hw/isa/apm.h | 9 +++---
>> hw/acpi/piix4.c | 2 +-
>> hw/isa/apm.c | 15 ++++++---
>> hw/isa/lpc_ich9.c | 64 +++++++++++++++++++++++++++++++++++--
>> hw/isa/vt82c686.c | 2 +-
>> 7 files changed, 168 insertions(+), 13 deletions(-)
>> create mode 100644 docs/specs/q35-apm-sts.txt
>>
>> --
>> 2.9.2
- [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Laszlo Ersek, 2016/11/18
- [Qemu-devel] [PATCH v3 for-2.9 1/3] hw/isa/apm: introduce callback for APM_STS_IOPORT writes, Laszlo Ersek, 2016/11/18
- [Qemu-devel] [PATCH v3 for-2.9 2/3] hw/isa/lpc_ich9: add SMI feature negotiation via APM_STS, Laszlo Ersek, 2016/11/18
- [Qemu-devel] [PATCH v3 for-2.9 3/3] hw/isa/lpc_ich9: ICH9_APM_STS_F_BROADCAST_SMI: inject SMI on all VCPUs, Laszlo Ersek, 2016/11/18
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Michael S. Tsirkin, 2016/11/18
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI,
Laszlo Ersek <=
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Paolo Bonzini, 2016/11/23
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Laszlo Ersek, 2016/11/23
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Laszlo Ersek, 2016/11/23
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Kevin O'Connor, 2016/11/23
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Michael S. Tsirkin, 2016/11/23
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Laszlo Ersek, 2016/11/24
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Michael S. Tsirkin, 2016/11/24
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Laszlo Ersek, 2016/11/25
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Laszlo Ersek, 2016/11/25
- Re: [Qemu-devel] [PATCH v3 for-2.9 0/3] q35: add negotiable broadcast SMI, Gerd Hoffmann, 2016/11/28