On 03/17/2015 06:39 PM, Andreas Färber wrote:
Am 17.03.2015 um 10:08 schrieb Chen Fan:
ICC bus was invented only to provide hotplug capability to
CPU and APIC because at the time being hotplug was available only for
BUS attached devices.
Now this patch is to drop ICC bus impl, and switch to bus-less
CPU+APIC hotplug, handling them in the same manner as pc-dimm.
Signed-off-by: Chen Fan <[email protected]>
---
hw/i386/pc.c | 21 +++++----------------
hw/i386/pc_piix.c | 9 +--------
hw/i386/pc_q35.c | 9 +--------
hw/intc/apic.c | 6 +++---
hw/intc/apic_common.c | 24 +++++++++++++++++++-----
include/hw/i386/apic_internal.h | 5 ++---
include/hw/i386/pc.h | 2 +-
target-i386/cpu.c | 15 ++-------------
target-i386/cpu.h | 2 +-
9 files changed, 35 insertions(+), 58 deletions(-)
This mini-series is lacking a cover letter and a change log.
Please also find a better subject for this one: Luckily you are not just
dropping things but rather implementing the new hotplug interface.
Is this a resend from Zhu Guihua's series or did anything change?
I think for Soft Freeze and Hard Freeze coming up today it's too late.
I picked up this two patches from Zhu Guihua's series separately and made
some changed to better adapt new interface.
As you may have seen, I've posted a test case to facilitate testing of
the various proposed code movements.
yes, I would use it to test my later version.
Thanks,
Chen
Regards,
Andreas