Patch 1 removes the use of LIBXL_HAVE_* that are present in Xen >= 4.9.
Patch 2 mentions the version bump in News.

Thanks danpb for a little prodding to take a closer look at potential
code reduction! After following through with the exersice, it proved
to be a bit more than I expected.

Jim Fehlig (2):
  Xen: Remove unneeded LIBXL_HAVE_* ifdefs
  News: Mention bump in minimum supported Xen version

 NEWS.rst                       |  7 ++++++
 src/libxl/libxl_capabilities.c | 14 ++---------
 src/libxl/libxl_capabilities.h |  2 --
 src/libxl/libxl_conf.c         | 45 ----------------------------------
 src/libxl/libxl_domain.c       | 20 +--------------
 src/libxl/libxl_driver.c       | 29 +---------------------
 src/libxl/xen_xl.c             | 24 ++----------------
 tests/domaincapsmock.c         | 11 ---------
 tests/libxlxml2domconfigtest.c |  6 -----
 tests/xlconfigtest.c           | 12 ---------
 10 files changed, 13 insertions(+), 157 deletions(-)

-- 
2.31.1

On Thu, Jun 17, 2021 at 02:30:27PM -0600, Jim Fehlig wrote:
> Patch 1 removes the use of LIBXL_HAVE_* that are present in Xen >= 4.9.
> Patch 2 mentions the version bump in News.
> 
> Thanks danpb for a little prodding to take a closer look at potential
> code reduction! After following through with the exersice, it proved
> to be a bit more than I expected.
> 
> Jim Fehlig (2):
>   Xen: Remove unneeded LIBXL_HAVE_* ifdefs
>   News: Mention bump in minimum supported Xen version

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

Am Thu, 17 Jun 2021 14:30:27 -0600
schrieb Jim Fehlig <jfehlig@suse.com>:

> Patch 1 removes the use of LIBXL_HAVE_* that are present in Xen >= 4.9.

Thanks, this compiles for, and looks good to me.

Olaf