]> xenbits.xensource.com Git - xen.git/commit
libxl: Use vkb=[] for HVMs
authorJason Andryuk <jandryuk@gmail.com>
Sun, 7 Apr 2024 14:32:08 +0000 (10:32 -0400)
committerAndrew Cooper <andrew.cooper3@citrix.com>
Thu, 11 Apr 2024 15:39:10 +0000 (16:39 +0100)
commit3c7482b73c5aa38e7faecccd649c67093001d3b5
tree3a534be39bc98b309b48f9085e1b58240a089210
parent2b31a1f0557a05a182fe360c49eaa7ae1b231aa4
libxl: Use vkb=[] for HVMs

xl/libxl only applies vkb=[] to PV & PVH guests.  HVM gets only a single
vkb by default, but that can be disabled by the vkb_device boolean.
Notably the HVM vkb cannot be configured, so feature-abs-pointer or the
backend-type cannot be specified.

Re-arrange the logic so that vkb=[] is handled regardless of domain
type.  If vkb is empty or unspecified, follow the vkb_device boolean for
HVMs.  Nothing changes for PVH & PV.  HVMs can now get a configured vkb
instead of just the default one.

The chance for regression is an HVM config with
vkb=["$something"]
vkb_device=false

Which would now get a vkb.

This is useful for vGlass which provides a VKB to HVMs.  vGlass wants to
specify feature-abs-pointer, but that is racily written by vGlass
instead of coming through the xl.cfg.  Unhelpfully, Linux xen-kbdfront
reads the backend nodes without checking that the backend is in
InitWait.

Signed-off-by: Jason Andryuk <jandryuk@gmail.com>
Signed-off-by: Jason Andryuk <jason.andryuk@amd.com>
Reviewed-by: Anthony PERARD <anthony.perard@citrix.com>
CHANGELOG.md
docs/man/xl.cfg.5.pod.in
tools/libs/light/libxl_create.c