qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Dou Liyang <douly.fnst@cn.fujitsu.com>
To: Eduardo Otubo <eduardo.otubo@profitbricks.com>
Cc: qemu-devel@nongnu.org, isimatu.yasuaki@jp.fujitsu.com,
	Igor Mammedov <imammedo@redhat.com>,
	afaerber@suse.de
Subject: Re: [Qemu-devel] [PATCH v3 0/7] cpu: add i386 cpu hot remove support
Date: Thu, 23 Jun 2016 16:54:20 +0800	[thread overview]
Message-ID: <576BA3BC.8000403@cn.fujitsu.com> (raw)
In-Reply-To: <20160622121248.GA29390@vader>

Hi,

Thanks to test.

在 2016年06月22日 20:12, Eduardo Otubo 写道:
> On Mon, Jul 13, 2015 at 09=45=57AM +0800, Zhu Guihua wrote:
>> On 07/09/2015 10:25 PM, Eduardo Otubo wrote:
>>> On Fri, Jun 26, 2015 at 11=37=43AM +0800, Zhu Guihua wrote:
>>>> Hi,
>>>> On 06/24/2015 09:28 PM, Eduardo Otubo wrote:
>>>>> Hello Zhu,
>>>>>
>>>>> Are you still working on this feature? Could you provide a rebased
>>>>> version of this series?
>>>> Sorry for late reply.
>>>>
>>>> Yes, we are still working on this feature.
>>>>
>>>> I have updated my github, you can get the rebased version from it.
>>>>
>>>> https://github.com/zhuguihua/qemu.git  cpu-hotplug
>>> Hi, thanks a lot for rebasing it. I didn't have time to test it, though.
>>> I appologise.
>>>
>>> Regarding your branch, your plan is to stick with device_del and
>>> object_del interface?
>> Yeah, we will only stick with device_del interface.
>> There is no need to use object_del interface for cpu hot remove.
>>
>> Thanks,
>> Zhu
>>
> This is almost a year old email, but I decided to git it a try anyway.
> Are you still working on this?
Yes, we are still working on this feature.

but, as we discuss almost one year age and the Abstract CPU core type is 
defined recently, this patch's QMP/HMP interface will be replace.

we are  following Igor's patches to see what we can do for this feature.
> I'm trying your patches and it seems I
> had a simple issue: EJ0 is sent, guest ack's and removes the CPU, but it
> get's stuck somewhere between the BIOS and Qemu. CPU is successfully
> removed from the guest, but it still appears on QOM:
>
>      (qemu) info cpus
>      * CPU #0: pc=0xffffffff8104f596 (halted) thread_id=22208
>        CPU #1: pc=0xffffffff8104f596 (halted) thread_id=22209
>        CPU #2: pc=0xffffffff810429c8 (halted) thread_id=22281
>      (qemu) info qom-tree
>      /machine (pc-i440fx-2.1-machine)
>        /fw_cfg (fw_cfg_io)
>          /fwcfg[0] (qemu:memory-region)
>      
>      [...]
>      
>          /cpu3 (SandyBridge-x86_64-cpu)
>      
>      [...]
>
> Do you have any clues?
we guess Qom may not be updated during the remove event. we will concern it.

Thanks,
Dou.
>

      parent reply	other threads:[~2016-06-23  8:56 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-13 10:40 [Qemu-devel] [PATCH v3 0/7] cpu: add i386 cpu hot remove support Zhu Guihua
2015-02-13 10:40 ` [Qemu-devel] [PATCH v3 1/7] x86: add x86_cpu_unrealizefn() for cpu apic remove Zhu Guihua
2015-02-13 10:40 ` [Qemu-devel] [PATCH v3 2/7] acpi/cpu: add cpu hot unplug request callback function Zhu Guihua
2015-02-13 10:40 ` [Qemu-devel] [PATCH v3 3/7] acpi, pc: add cpu hot unplug request callback support Zhu Guihua
2015-02-13 10:40 ` [Qemu-devel] [PATCH v3 4/7] acpi/cpu: add cpu hot unplug callback function Zhu Guihua
2015-02-13 10:40 ` [Qemu-devel] [PATCH v3 5/7] acpi, pc: add cpu hot unplug callback support Zhu Guihua
2015-02-13 10:40 ` [Qemu-devel] [PATCH v3 6/7] cpu hotplug: implement function cpu_status_write() for vcpu ejection Zhu Guihua
2015-02-13 10:40 ` [Qemu-devel] [PATCH v3 7/7] cpus: reclaim allocated vCPU objects Zhu Guihua
2015-03-04  8:40   ` Bharata B Rao
2015-02-24  1:29 ` [Qemu-devel] [PATCH v3 0/7] cpu: add i386 cpu hot remove support Gu Zheng
2015-06-24 13:28 ` Eduardo Otubo
2015-06-26  3:37   ` Zhu Guihua
2015-06-30 13:47     ` Eduardo Otubo
2015-07-09 14:25     ` Eduardo Otubo
2015-07-13  1:45       ` Zhu Guihua
2016-06-22 12:12         ` Eduardo Otubo
2016-06-22 13:24           ` Igor Mammedov
2016-06-22 13:27             ` Eduardo Otubo
2016-06-22 14:45               ` Igor Mammedov
2016-06-23  8:54           ` Dou Liyang [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=576BA3BC.8000403@cn.fujitsu.com \
    --to=douly.fnst@cn.fujitsu.com \
    --cc=afaerber@suse.de \
    --cc=eduardo.otubo@profitbricks.com \
    --cc=imammedo@redhat.com \
    --cc=isimatu.yasuaki@jp.fujitsu.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).