Re: [PATCH] libxl: use b_info->{acpi,acpi} when available

On 9/18/20 1:31 PM, Daniel P. Berrangé wrote: > On Wed, Sep 16, 2020 at 11:09:31AM +0200, Michal Privoznik wrote: > > On 9/10/20 6:18 AM, Marek Marczykowski-Górecki wrote: > > > b_info->u.hvm.{acpi,apic} are deprecated. But also, on recent libxl > > > version (4.14) the old one seems to be broken. While libxl part should > > > be fixed too, update the usage here and at some point drop support for > > > the old version. > > > b_info->acpi was added in Xen 4.8 > > > b_info->apic was added in Xen 4.10 > > > Xen 4.10 is the oldest version that still has security support (until > > > December 2020). > > > > > > Signed-off-by: Marek Marczykowski-Górecki <marmarek(a)invisiblethingslab.com&gt; > > > --- > > > src/libxl/libxl_conf.c | 13 +++++++++++++ > > > tests/libxlxml2domconfigdata/basic-hvm.json | 4 ++-- > > > tests/libxlxml2domconfigdata/cpu-shares-hvm.json | 4 ++-- > > > .../libxlxml2domconfigdata/fullvirt-acpi-slic.json | 4 ++-- > > > .../fullvirt-cpuid-legacy-nest.json | 4 ++-- > > > tests/libxlxml2domconfigdata/fullvirt-cpuid.json | 4 ++-- > > > .../max-eventchannels-hvm.json | 4 ++-- > > > tests/libxlxml2domconfigdata/max-gntframes-hvm.json | 4 ++-- > > > tests/libxlxml2domconfigdata/moredevs-hvm.json | 4 ++-- > > > .../libxlxml2domconfigdata/variable-clock-hvm.json | 4 ++-- > > > .../vnuma-hvm-legacy-nest.json | 4 ++-- > > > tests/libxlxml2domconfigdata/vnuma-hvm.json | 4 ++-- > > > 12 files changed, 35 insertions(+), 22 deletions(-) > > > > This looks good to me. > > > > Reviewed-by: Michal Privoznik <mprivozn(a)redhat.com&gt; > > > > I'll wait a bit with pushing it though in case Jim wants to chime in. > > This broke the build on Ubuntu 1804 due to tests failing > > TEST: libxlxml2domconfigtest > .!!!!.!!!! 10 FAIL Oh, Ubuntu 18.04 has libxen-dev-4.9.2 and I'm not sure about FreeBSD, but probably something old too. So we can't use xen 4.10 APIs even though it was released 3 years ago. Unfortunately, we will have to support Ubuntu 18.04 for quite some time because 20.04 was released only a while ago and we have this two year transition period: https://libvirt.org/platforms.html Marek, are you okay with me reverting the patch?