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

Re: [Xen-devel] [PATCH v4 0/3] add hypercall option to temporarily pin a vcpu



Committers,

anything missing on my side to have this series being committed?
All patches have Acked-by and Reviewed-by tags.


Juergen

On 10/03/16 06:27, Juergen Gross wrote:
> Some hardware (e.g. Dell studio 1555 laptops) require SMIs to be
> called on physical cpu 0 only. Linux drivers like dcdbas or i8k try
> to achieve this by pinning the running thread to cpu 0, but in Dom0
> this is not enough: the vcpu must be pinned to physical cpu 0 via
> Xen, too.
> 
> This patch series enhances xen tools to cope with new reactions of
> the hypervisor due to the new feature.
> 
> Changes in V4:
> - removed patches 1-3 (hypervisor patches) as already accepted
> - patch 1 (was 4): minor code modifications as suggested by Dario Faggioli
> - patch 3 (was 6): change "force" variable to bool as suggested by 
>   Dario Faggioli
> - patch 3 (was 6): add #define to libxl indicating availability of
>   libxl_set_vcpuaffinity_force() as requested by Wei Liu
> - patch 3 (was 6): add force option to xl man page as requested by 
>   Dario Faggioli
> 
> Changes in V3:
> - removed old patch 1 as already accepted
> - new patch 1 to correct error in cpupool_unassign_cpu_helper()
> - patch 2: adjust coding style as requested by Jan Beulich
> - patch 2: rename pin_temp to pin_override as requested by Jan Beulich and
>   suggested by David Vrabel
> - patch 2: change return type of do_pin_temp() to int as requested by
>   Jan Beulich
> - patch 2: swap error checks in do_sched_op() as requested by Jan Beulich
> - patch 2: adjust comment in xen/include/public/sched.h as requested by
>   Jan Beulich
> - new patch 3 to add possibility in hypervisor to undo pin override via tools
> - patch 4 (was 3): adjust coding style as requested by Wei Liu
> - new patch 5 to  print message how to recover from xl cpupool-cpu-remove
>   errors
> - new patch 6 to undo pin override via xl vcpu-pin (force option)
> 
> Changes in V2:
> - add patch 1 to silence messages on suspend/resume
> - add patch 3 to handle EBUSY case when removing cpu from cpupool
> - limit operation to hardware domain as suggested by Jan Beulich
> - some style issues corrected as requested by Jan Beulich
> - use fixed width types in interface as requested by Jan Beulich
> - add compat layer checking as requested by Jan Beulich
> 
> 
> Juergen Gross (3):
>   libxc: do some retries in xc_cpupool_removecpu() for EBUSY case
>   libxl: print message how to recover from xl cpupool-cpu-remove errors
>   libxl: add force option for xl vcpu-pin
> 
>  docs/man/xl.pod.1         |  7 ++++++-
>  tools/libxc/xc_cpupool.c  | 20 +++++++++++++++++++-
>  tools/libxl/libxl.c       | 31 +++++++++++++++++++++++++------
>  tools/libxl/libxl.h       | 10 ++++++++++
>  tools/libxl/xl_cmdimpl.c  | 32 +++++++++++++++++++++++++++-----
>  tools/libxl/xl_cmdtable.c |  3 ++-
>  6 files changed, 89 insertions(+), 14 deletions(-)
> 
> Cc: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
> Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> Cc: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> Cc: Wei Liu <wei.liu2@xxxxxxxxxx>
> 


_______________________________________________
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®.