[PATCH 0/2] xen/arm: static event channel

Rahul Singh posted 2 patches 1 year, 7 months ago
Test gitlab-ci failed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/cover.1663928523.git.rahul.singh@arm.com
SUPPORT.md           | 7 +++++++
xen/arch/arm/setup.c | 5 +++--
2 files changed, 10 insertions(+), 2 deletions(-)
[PATCH 0/2] xen/arm: static event channel
Posted by Rahul Singh 1 year, 7 months ago
This patch series fix issues related to static event channel series.

Rahul Singh (2):
  xen: Add static event channel in SUPPORT.md on ARM
  xen/arm: fix booting ACPI based system after static evtchn series

 SUPPORT.md           | 7 +++++++
 xen/arch/arm/setup.c | 5 +++--
 2 files changed, 10 insertions(+), 2 deletions(-)

-- 
2.25.1
Re: [PATCH 0/2] xen/arm: static event channel
Posted by Bertrand Marquis 1 year, 7 months ago
Hi Rahul,

Please give the necessary justification for inclusion in 4.17:
- severity of the bug fixed
- probability and impact of potential issues that the patch could add.

thanks
Bertrand

> On 23 Sep 2022, at 13:02, Rahul Singh <Rahul.Singh@arm.com> wrote:
> 
> This patch series fix issues related to static event channel series.
> 
> Rahul Singh (2):
>  xen: Add static event channel in SUPPORT.md on ARM
>  xen/arm: fix booting ACPI based system after static evtchn series
> 
> SUPPORT.md           | 7 +++++++
> xen/arch/arm/setup.c | 5 +++--
> 2 files changed, 10 insertions(+), 2 deletions(-)
> 
> -- 
> 2.25.1
> 
Re: [PATCH 0/2] xen/arm: static event channel
Posted by Rahul Singh 1 year, 7 months ago
Hi All,

> On 26 Sep 2022, at 1:12 pm, Bertrand Marquis <Bertrand.Marquis@arm.com> wrote:
> 
> Hi Rahul,
> 
> Please give the necessary justification for inclusion in 4.17:
> - severity of the bug fixed

The severity of the bug is high as without this fixed system with ACPI support will fail to boot.

> - probability and impact of potential issues that the patch could add.

As we are not supporting the static event channel for ACPI, it is okay to move
alloc_static_evtchn() under acpi_disabled check.

Regards,
Rahul
RE: [PATCH 0/2] xen/arm: static event channel
Posted by Henry Wang 1 year, 7 months ago
Hi Rahul and all,

> -----Original Message-----
> From: Rahul Singh <Rahul.Singh@arm.com>
> Subject: Re: [PATCH 0/2] xen/arm: static event channel
> 
> Hi All,
> > On 26 Sep 2022, at 1:12 pm, Bertrand Marquis
> <Bertrand.Marquis@arm.com> wrote:
> > Hi Rahul,
> >
> > Please give the necessary justification for inclusion in 4.17:
> > - severity of the bug fixed
> The severity of the bug is high as without this fixed system with ACPI support
> will fail to boot.

I agree on that, dom0 boot from ACPI on Arm is what we currently supported.
So I think we need to fix the current issue for 4.17.

Besides the bugfix patch, the other patch in this series is a pure doc change
so it should be safe.

I think we still need Arm maintainers' R-b or ack for this series, and as long as
This series is properly reviewed:

Release-acked-by: Henry Wang <Henry.Wang@arm.com>

> 
> > - probability and impact of potential issues that the patch could add.
> 
> As we are not supporting the static event channel for ACPI, it is okay to move
> alloc_static_evtchn() under acpi_disabled check.

Agreed.

Kind regards,
Henry

> 
> Regards,
> Rahul
>