Re: [RFC PATCH 00/12] qemu: add support to hot-plug/unplug cpu device

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jan 21, 2015 at 16:00:52 +0800, Zhu Guihua wrote:
> If you apply the folowing patchset
> [PATCH v3 0/7] cpu: add device_add foo-x86_64-cpu support
> https://lists.nongnu.org/archive/html/qemu-devel/2015-01/msg01552.html,
> and [PATCH v2 00/11] cpu: add i386 cpu hot remove support
> https://lists.nongnu.org/archive/html/qemu-devel/2015-01/msg01557.html,
> qemu can support hotplug and hot-unplug cpu device.
> 
> So this patch series will make libvirt support hotplug and hot-unplug cpu
> device for qemu driver, and now only supports one cpu driver which is
> 'qemu64-x86_64-cpu'.
> 
> The cpu device's xml like this:
> <cpu driver='qemu64-x86_64-cpu' apic_id='3'>
>

Libvirt already implements vCPU hotplug and unplug via
virDomainSetVcpusFlags() API and implements this API also for the qemu
driver. Granted that the existing code uses older comands but that can
be tweaked to support the new stuff too.

Additionally the new <cpu> subelement of <devices> doesn't seem
necessary:

- There's only one model that can be plugged
- Configuring the APIC id doesn't make much sense for users
- nothing else can be configured

I have following questions though:

Is there a difference in specifying the cpu via the -device option and
via the legacy -cpu argument?

Is there a plan to have more than one cpu "model" that can be plugged?

Can more than one cpu model be plugged to the same VM at the same time?

In general I don't think it's necessary to add the new device type and
we should rather improve the existing API to support the new device
type.

More comments will be inline in the patches.

Peter

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]