From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42752) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xl9sQ-00028C-Hz for qemu-devel@nongnu.org; Mon, 03 Nov 2014 00:08:32 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Xl9sK-00049s-A0 for qemu-devel@nongnu.org; Mon, 03 Nov 2014 00:08:26 -0500 Received: from mx1.redhat.com ([209.132.183.28]:48814) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xl9sK-00047t-2c for qemu-devel@nongnu.org; Mon, 03 Nov 2014 00:08:20 -0500 Date: Mon, 3 Nov 2014 07:08:09 +0200 From: "Michael S. Tsirkin" Message-ID: <20141103050809.GA11096@redhat.com> References: <1413948272-7765-1-git-send-email-guz.fnst@cn.fujitsu.com> <5456F47F.4040609@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5456F47F.4040609@cn.fujitsu.com> Subject: Re: [Qemu-devel] [PATCH V6 0/8] cpu/acpi: convert cpu hot plug to hotplug_handler API List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gu Zheng Cc: qemu-devel@nongnu.org, tangchen@cn.fujitsu.com, isimatu.yasuaki@jp.fujitsu.com, chen.fan.fnst@cn.fujitsu.com, imammedo@redhat.com, afaerber@suse.de On Mon, Nov 03, 2014 at 11:20:31AM +0800, Gu Zheng wrote: > Hi Michael, > Could you please have a look at this series? > > Thanks, > Gu > On 10/22/2014 11:24 AM, Gu Zheng wrote: Applied in my tree already, will send upstream after a bit of testing. Thanks! > > Previously we use cpu_added_notifiers to register cpu hotplug notifier callback > > which is not able to pass/handle errors, so we switch it to unified hotplug > > handler API which allows to pass errors and would allow to cancel device_add > > in case of error. > > Thanks very much for Igor's review and suggestion. > > > > --- > > v6: > > Patch 8/8: > > -using error_abort instead of local_err, so that we can gain the error report > > before abort (suggested by Igor). > > > > v5: > > -rebase on the latest upstream and fix some comments. > > Patch 4/8: > > -split the check out of acpi_dev block. > > Patch 5/8: > > -move CPU hot-plug notifier cleanup hunk into Patch 6/8. > > Patch 6/8: > > -delete the caller of notifier_list_notify() in this patch. > > Patch 8/8: > > -rename acpi_set_local_sts to acpi_set_cpu_present_bit for better readability. > > > > v4: > > -split removal of CPU hotplug notifier into separate patch (Patch 6/8). > > Patch 1/7: > > -convert CPUState *cpu to DeviceState *dev like it's done for other handlers > > and do cast to CPU inside. > > Patch 5/7: > > -Make rtc_state as a link property in PCMachine rather than the global > > variables. > > -Split out the removal of unused notifier into separate patch. > > -Check the result of plug callback before update rtc_state. > > > > v3: > > -deal with start-up cpus in pc_cpu_plug as Igor suggested. > > > > v2: > > -Add 3 new patches(5/7,6/7,7/7), delete original patch 5/5. > > 1/5-->1/7 > > 2/5-->2/7 > > 3/5-->3/7 > > 4/5-->4/7 > > Patch 1/7: > > -add errp argument to catch error. > > -return error instead of aborting if cpu id is invalid. > > -make acpi_cpu_plug_cb as a wrapper around AcpiCpuHotplug_add. > > Patch 3/7: > > -remove unused AcpiCpuHotplug_add directly. > > Patch 5/7: > > -switch the last user of cpu hotplug notifier to hotplug handler API, and > > remove the unused cpu hotplug notify. > > Patch 6/7: > > -split the function rename (just cleanup) into single patch. > > Patch 7/7: > > -introduce help function acpi_set_local_sts to keep the bit setting in > > one place. > > --- > > > > Gu Zheng (8): > > acpi/cpu: add cpu hotplug callback function to match hotplug_handler > > API > > acpi:ich9: convert cpu hotplug to hotplug_handler API > > acpi:piix4: convert cpu hotplug to hotplug_handler API > > pc: add cpu hotplug handler to PC_MACHINE > > pc: Update rtc_cmos in pc_cpu_plug > > qom/cpu: remove the unused CPU hot-plug notifier > > cpu-hotplug: rename function for better readability > > acpi/cpu-hotplug: introduce helper function to keep bit setting in > > one place > > > > hw/acpi/cpu_hotplug.c | 34 ++++++++++++++------- > > hw/acpi/ich9.c | 17 ++-------- > > hw/acpi/piix4.c | 18 ++--------- > > hw/i386/pc.c | 65 +++++++++++++++++++++++++++-------------- > > hw/i386/pc_piix.c | 2 +- > > hw/i386/pc_q35.c | 2 +- > > include/hw/acpi/cpu_hotplug.h | 7 ++-- > > include/hw/acpi/ich9.h | 1 - > > include/hw/i386/pc.h | 3 +- > > include/sysemu/sysemu.h | 3 -- > > qom/cpu.c | 10 ------ > > 11 files changed, 82 insertions(+), 80 deletions(-) > > >