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

Re: [libvirt] [PATCH 3/3] virsh: Prohibit all clear cpumap



On 04/01/2013 11:42 PM, Osier Yang wrote:
> This prohibits all clear cpumap eariler in virsh, for both vcpupin
> and emulatorpin.

Is that really necessary?  Sometimes, letting virsh allow obviously
wrong constructs, in order to prove that libvirt itself will flag things
as sensible errors, makes for nicer testing.  Virsh should only filter
out obviously wrong things if it can give a nicer error message than
what the underlying API would produce.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


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