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

[Xen-devel] [PATCH v3] Fix xl vcpu-set to decrease an guest vCPU amount without complaints.



Hey Ian,

Since v2 [http://lists.xen.org/archives/html/xen-devel/2015-03/msg01787.html]:
 - Fixed up #1 "libxl: Add ERROR_DOMAIN_NOTFOUND for libxl_domain_info" per 
your review.
 - Moved the check from vcpuset to libxl_set_vcpuonline to check if the set 
bits of the
   cpumap is greater than the maximum vCPU that the guest can have (and added 
printks).
 - Added LIBXL_DOMAIN_TYPE_NOTFOUND so that libxl_set_vcpuonline can report 
that the
   domain id is for an non-existent guest.
 - Added an #define in libxl.h to expose the new _NOTFOUND error value.

In short, these patches:
 [PATCH v3 1/7] libxl: Add ERROR_DOMAIN_NOTFOUND for libxl_domain_info
 [PATCH v3 2/7] libxl: Add to libxl__domain_type a new return value
 [PATCH v3 3/7] libxl: In libxl_set_vcpuonline check for maximum
 [PATCH v3 4/7] libxl/vcpuset: Print error if libxl_set_vcpuonline

are new. While these have been redone since v2 review:
 [PATCH v3 5/7] libxl/vcpuset: Return error value if failed.
 [PATCH v3 7/7] libxl/vcpu-set - allow to decrease vcpu count on

And this one is Acked and has not changed since v2 posting:
 [PATCH v3 6/7] libxl/vcpuset: Remove useless limit on max_vcpus.

These patches are also available at:
 
 git://xenbits.xen.org/people/konradwilk/xen.git vcpuset.v3


Thank you for taking the time to review these patches!
 
 tools/libxl/libxl.c         | 97 ++++++++++++++++++++++++++++++---------------
 tools/libxl/libxl.h         | 14 ++++++-
 tools/libxl/libxl_dom.c     |  5 ++-
 tools/libxl/libxl_pci.c     |  6 +++
 tools/libxl/libxl_types.idl |  3 ++
 tools/libxl/xl_cmdimpl.c    | 47 ++++++++++++++--------
 6 files changed, 121 insertions(+), 51 deletions(-)

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.