[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] per-machine vs per-qemu-version compat_props macros (wa
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] per-machine vs per-qemu-version compat_props macros (was Re: acpi:ich9: add memory hotplug handling) |
Date: |
Tue, 24 Jun 2014 14:55:20 -0300 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Tue, Jun 24, 2014 at 05:47:53PM +0100, Peter Maydell wrote:
> On 24 June 2014 17:44, Eduardo Habkost <address@hidden> wrote:
> > (I would go even further and say that no compat_props bit need to be
> > specific to a machine-type family, and they are simply tied to the code
> > included in a QEMU version. So every PC_COMPAT_* bit could go to a
> > common QEMU_COMPAT_<version> macro, that could be reusable by all
> > machine-types, and there's no need PC-specific compat macros.)
>
> What if different machines in a particular release had the
> property set to different defaults?
I never saw that happen, and I don't think it is even likely: default
values set in instance_init, and compat_props are applied just after
instance_init, before object_new() returns. So a machine-type would need
to find a way to change fields after instance_init but before
compat_props are applied.
Anyway, if a machine-type really needs that, it can still have its own
set of compat macros to implement machine-type-specific compat_props. My
point is that there's no need for that on the PC code today.
--
Eduardo
- [Qemu-devel] [PULL v2 014/106] pc: add 'etc/reserved-memory-end' fw_cfg interface for SeaBIOS, (continued)
- [Qemu-devel] [PULL v2 014/106] pc: add 'etc/reserved-memory-end' fw_cfg interface for SeaBIOS, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 018/106] pc-dimm: add busy slot check and slot auto-allocation, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 021/106] trace: add acpi memory hotplug IO region events, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 022/106] trace: pc: add PC_DIMM slot & address allocation, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 023/106] acpi:piix4: allow plug/unlug callbacks handle not only PCI devices, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 025/106] pc: ich9 lpc: make it work with global/compat properties, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 024/106] acpi:piix4: add memory hotplug handling, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 026/106] acpi:ich9: add memory hotplug handling, Michael S. Tsirkin, 2014/06/18
[Qemu-devel] [PULL v2 027/106] pc: migrate piix4 & ich9 MemHotplugState, Michael S. Tsirkin, 2014/06/18
[Qemu-devel] [PULL v2 030/106] pc: ACPI BIOS: implement memory hotplug interface, Michael S. Tsirkin, 2014/06/18
[Qemu-devel] [PULL v2 029/106] pc: propagate memory hotplug event to ACPI device, Michael S. Tsirkin, 2014/06/18
[Qemu-devel] [PULL v2 033/106] pc: ACPI BIOS: make GPE.3 handle memory hotplug event on PIIX and Q35 machines, Michael S. Tsirkin, 2014/06/18
[Qemu-devel] [PULL v2 034/106] acpi: update generated files, Michael S. Tsirkin, 2014/06/18
[Qemu-devel] [PULL v2 042/106] virtio-net: announce self by guest, Michael S. Tsirkin, 2014/06/18