linux-kernel - Re: [RFC KERNEL PATCH v6 2/3] xen/pvh: Setup gsi for passthrough device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <9ebdcc0b-3775-4c5e-9ffb-f45d048281dd@suse.com>
Date: Fri, 10 May 2024 09:48:29 +0200
From: Juergen Gross <jgross@...e.com>
To: Jiqian Chen <Jiqian.Chen@....com>,
 Stefano Stabellini <sstabellini@...nel.org>,
 Bjorn Helgaas <bhelgaas@...gle.com>, "Rafael J . Wysocki"
 <rafael@...nel.org>, Roger Pau Monné <roger.pau@...rix.com>
Cc: xen-devel@...ts.xenproject.org, linux-pci@...r.kernel.org,
 linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org,
 Huang Rui <Ray.Huang@....com>
Subject: Re: [RFC KERNEL PATCH v6 2/3] xen/pvh: Setup gsi for passthrough
 device

On 19.04.24 05:36, Jiqian Chen wrote:
> In PVH dom0, the gsis don't get registered, but the gsi of
> a passthrough device must be configured for it to be able to be
> mapped into a domU.
> 
> When assign a device to passthrough, proactively setup the gsi
> of the device during that process.
> 
> Co-developed-by: Huang Rui <ray.huang@....com>
> Signed-off-by: Jiqian Chen <Jiqian.Chen@....com>
> Reviewed-by: Stefano Stabellini <sstabellini@...nel.org>

This patch is breaking the build.

On Arm I get:

In file included from /home/gross/korg/src/drivers/xen/xen-pciback/pci_stub.c:23:0:
/home/gross/korg/src/include/xen/acpi.h: In function 'xen_acpi_sleep_register':
/home/gross/korg/src/include/xen/acpi.h:67:3: error: 'acpi_suspend_lowlevel' 
undeclared (first use in this function); did you mean 'xen_acpi_suspend_lowlevel'?
    acpi_suspend_lowlevel = xen_acpi_suspend_lowlevel;
    ^~~~~~~~~~~~~~~~~~~~~
    xen_acpi_suspend_lowlevel
/home/gross/korg/src/include/xen/acpi.h:67:3: note: each undeclared identifier 
is reported only once for each function it appears in
make[6]: *** [/home/gross/korg/src/scripts/Makefile.build:244: 
drivers/xen/xen-pciback/pci_stub.o] Error 1
make[5]: *** [/home/gross/korg/src/scripts/Makefile.build:485: 
drivers/xen/xen-pciback] Error 2
make[4]: *** [/home/gross/korg/src/scripts/Makefile.build:485: drivers/xen] Error 2

Additionally I'm seeing this warning on x86_64:

/home/gross/korg/src/arch/x86/xen/enlighten_pvh.c:97:5: warning: no previous 
prototype for ‘xen_pvh_passthrough_gsi’ [-Wmissing-prototypes]
  int xen_pvh_passthrough_gsi(struct pci_dev *dev)


Juergen

Download attachment "OpenPGP_0xB0DE9DD628BF132F.asc" of type "application/pgp-keys" (3684 bytes)

Download attachment "OpenPGP_signature.asc" of type "application/pgp-signature" (496 bytes)