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

[Martin Kletzander]

On Sat, Oct 22, 2016 at 05:49:38PM +0800, Chen Hanxiao wrote:
>From: Chen Hanxiao <chenhanxiao at gmail.com>
>
> 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 at gmail.com>
>---
> 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?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20161027/34ab6425/attachment-0001.sig>