On 06.01.2024 02:08, Stefano Stabellini wrote: > On Fri, 5 Jan 2024, Jiqian Chen wrote: >> --- a/tools/libs/light/libxl_pci.c >> +++ b/tools/libs/light/libxl_pci.c >> @@ -1418,6 +1418,7 @@ static void pci_add_dm_done(libxl__egc *egc, >> unsigned long long start, end, flags, size; >> int irq, i; >> int r; >> + int gsi; >> uint32_t flag = XEN_DOMCTL_DEV_RDM_RELAXED; >> uint32_t domainid = domid; >> bool isstubdom = libxl_is_stubdom(ctx, domid, &domainid); >> @@ -1486,6 +1487,7 @@ static void pci_add_dm_done(libxl__egc *egc, >> goto out_no_irq; >> } >> if ((fscanf(f, "%u", &irq) == 1) && irq) { >> + gsi = irq; > > A question for Roger and Jan: are we always guaranteed that gsi == irq > (also in the PV case)?
Iirc for IO-APIC based IRQs that's always the case; for MSI ones there simply is no GSI, and Xen's IRQ number then normally isn't the same as the pIRQ a domain would see. Jan