[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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 invisiblethingslab com>
---
   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 redhat com>

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?

Michal


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]