Re: [PATCH] qemu_driver: don't set cpu pin for TCG domain

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

 



On Sat, Oct 22, 2016 at 05:49:38PM +0800, Chen Hanxiao wrote:
From: Chen Hanxiao <chenhanxiao@xxxxxxxxx>

We don't support pinning cpu for TCG domain.
But we could set it by vcpupin command,
which result in a failed startup.

Signed-off-by: Chen Hanxiao <chenhanxiao@xxxxxxxxx>
---
src/qemu/qemu_driver.c | 8 ++++++++
1 file changed, 8 insertions(+)


Honestly, I think it makes more sense the way it's done now.  The API
allows the same thing you would be able to do by 'virsh edit' anyway.
The fact that TCG doesn't support that now doesn't mean it will not
support that in the future and the way it's done now will ensure it
works in that case, I believe.


So I would vote for keeping it this way...  Maybe a domain capability
element could show whether this is possible or not, but that seems like
an overkill now.  Is that any particular major issue you are trying to
resolve by this?  E.g. in a mgmt app?

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]