libxl: dm_restrict: Move to domain_build_info
authorIan Jackson <ian.jackson@eu.citrix.com>
Thu, 12 Oct 2017 11:13:48 +0000 (12:13 +0100)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Thu, 12 Oct 2017 12:04:20 +0000 (13:04 +0100)
commitbc1bd8ebc30cffb8a5b3b0bcebf7c99df5d79723
tree8841924f72ca2d6719b769348143250f4a6559d3
parentc4e365a0eb3cb6c9dedfaf0c13b0a2ce62f49cf6
libxl: dm_restrict: Move to domain_build_info

Right now, this is broken because libxl__build_device_model_args_new
is used also for the qemu run for pv guests for qdisk devices, pvfb,
etc.

We can either make this option properly HVM-specific, or make it
generic.

In principle it is a reasonable request, to make the PV qemu
deprivileged (even though it is not likely to be implemented any time
soon).  So make this option generic.

We retain the name "device model" even though it is arguably
inaccurate, because the xl docs already say, for example
  For a PV guest a device-model is sometimes used to provide backends
  for certain PV devices

The documentation patch here is pure code motion.  For ease of review
we will fix up the docs, so the wording to be right for the new
context, in the next patch.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
Reported-by: Roger Pau Monné <roger.pau@citrix.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
---
v2: Change xl too, to avoid breaking the build.
    Fix manpage pod syntax.
docs/man/xl.cfg.pod.5.in
tools/libxl/libxl_create.c
tools/libxl/libxl_dm.c
tools/libxl/libxl_types.idl
tools/xl/xl_parse.c