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

2020-09-23 Thread Jim Fehlig
On 9/23/20 1:54 AM, Marek Marczykowski-Górecki wrote: On Tue, Sep 22, 2020 at 09:41:00PM -0600, Jim Fehlig wrote: On 9/18/20 9:51 AM, Daniel P. Berrangé wrote: On Fri, Sep 18, 2020 at 05:41:21PM +0200, Marek Marczykowski-Górecki wrote: On Fri, Sep 18, 2020 at 05:12:52PM +0200, Michal

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

2020-09-23 Thread Marek Marczykowski-Górecki
On Tue, Sep 22, 2020 at 09:41:00PM -0600, Jim Fehlig wrote: > On 9/18/20 9:51 AM, Daniel P. Berrangé wrote: > > On Fri, Sep 18, 2020 at 05:41:21PM +0200, Marek Marczykowski-Górecki wrote: > > > On Fri, Sep 18, 2020 at 05:12:52PM +0200, Michal Prívozník wrote: > > > > On 9/18/20 1:31 PM, Daniel P.

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

2020-09-22 Thread Jim Fehlig
On 9/18/20 9:51 AM, Daniel P. Berrangé wrote: On Fri, Sep 18, 2020 at 05:41:21PM +0200, Marek Marczykowski-Górecki wrote: On Fri, Sep 18, 2020 at 05:12:52PM +0200, Michal Prívozník wrote: On 9/18/20 1:31 PM, Daniel P. Berrangé wrote: On Wed, Sep 16, 2020 at 11:09:31AM +0200, Michal Privoznik

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

2020-09-18 Thread Daniel P . Berrangé
On Fri, Sep 18, 2020 at 05:41:21PM +0200, Marek Marczykowski-Górecki wrote: > On Fri, Sep 18, 2020 at 05:12:52PM +0200, Michal Prívozník wrote: > > 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,

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

2020-09-18 Thread Marek Marczykowski-Górecki
On Fri, Sep 18, 2020 at 05:12:52PM +0200, Michal Prívozník wrote: > 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

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

2020-09-18 Thread Michal Prívozník
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

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

2020-09-18 Thread Daniel P . Berrangé
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

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

2020-09-18 Thread Michal Prívozník
On 9/17/20 7:57 PM, Jim Fehlig wrote: On 9/16/20 3:09 AM, 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,

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

2020-09-17 Thread Jim Fehlig
On 9/16/20 3:09 AM, 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

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

2020-09-16 Thread Michal Privoznik
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

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

2020-09-09 Thread Marek Marczykowski-Górecki
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