Hello Jim, Thank you for taking the time to review this patch. I've updated the patch with the changes you requested:
* Split NEWS.rst change into separate patch. * Addressed line length issues in docs/formatdomain.rst * Addressed line length issues in src/libxl/libxl_conf.c * Corrected comment style in src/libxl/libxl_conf.c * Rebased onto latest master. * Removed the 'viridian-hvm-none' test." Best Regards, Will Will (2): libxl_conf: Implement hyperv domain flags for Xen Add NEWS item for <hyperv/> flags with Xen NEWS.rst | 5 + docs/formatdomain.rst | 24 +-- src/libxl/libxl_conf.c | 101 ++++++++++++ .../viridian-hvm-full.json | 101 ++++++++++++ .../viridian-hvm-full.xml | 45 +++++ .../libxlxml2domconfigdata/viridian-hvm.json | 99 +++++++++++ tests/libxlxml2domconfigdata/viridian-hvm.xml | 42 +++++ .../viridian-passthrough.json | 155 ++++++++++++++++++ .../viridian-passthrough.xml | 37 +++++ tests/libxlxml2domconfigtest.c | 8 + 10 files changed, 606 insertions(+), 11 deletions(-) create mode 100644 tests/libxlxml2domconfigdata/viridian-hvm-full.json create mode 100644 tests/libxlxml2domconfigdata/viridian-hvm-full.xml create mode 100644 tests/libxlxml2domconfigdata/viridian-hvm.json create mode 100644 tests/libxlxml2domconfigdata/viridian-hvm.xml create mode 100644 tests/libxlxml2domconfigdata/viridian-passthrough.json create mode 100644 tests/libxlxml2domconfigdata/viridian-passthrough.xml -- 2.49.0
