Re: [PATCH] libxl+hvmloader: extend IGD check part 2

2025-05-21 Thread Anthony PERARD
On Tue, Apr 08, 2025 at 03:23:13PM +0200, Marek Marczykowski-Górecki wrote: > Consider also "Display controller" an IGD, not only "VGA compatible > controller" in few more places. > > Fixes: 4191619e0893 ("libxl: extend IGD check") > Signed-off-by: Marek Marczykowski-Górecki > --- > Do you prefer

Re: [PATCH] libxl+hvmloader: extend IGD check part 2

2025-04-09 Thread Marek Marczykowski-Górecki
On Tue, Apr 08, 2025 at 04:11:36PM +0200, Jan Beulich wrote: > On 08.04.2025 15:23, Marek Marczykowski-Górecki wrote: > > --- a/tools/firmware/hvmloader/pci.c > > +++ b/tools/firmware/hvmloader/pci.c > > @@ -173,6 +173,7 @@ void pci_setup(void) > > switch ( class ) > > { > >

Re: [PATCH] libxl+hvmloader: extend IGD check part 2

2025-04-08 Thread Jan Beulich
On 08.04.2025 15:23, Marek Marczykowski-Górecki wrote: > --- a/tools/firmware/hvmloader/pci.c > +++ b/tools/firmware/hvmloader/pci.c > @@ -173,6 +173,7 @@ void pci_setup(void) > switch ( class ) > { > case 0x0300: > +case 0x0380: > /* If emulated VGA

Re: [PATCH] libxl+hvmloader: extend IGD check part 2

2025-04-08 Thread Andrew Cooper
On 08/04/2025 2:23 pm, Marek Marczykowski-Górecki wrote: > Consider also "Display controller" an IGD, not only "VGA compatible > controller" in few more places. > > Fixes: 4191619e0893 ("libxl: extend IGD check") > Signed-off-by: Marek Marczykowski-Górecki > --- > Do you prefer this to be split in

Re: [PATCH] libxl+hvmloader: extend IGD check part 2

2025-04-08 Thread Jan Beulich
On 08.04.2025 15:23, Marek Marczykowski-Górecki wrote: > --- a/tools/libs/light/libxl_pci.c > +++ b/tools/libs/light/libxl_pci.c > @@ -2575,7 +2575,8 @@ int libxl__grant_vga_iomem_permission(libxl__gc *gc, > const uint32_t domid, > > if (sysfs_dev_get_class(gc, pci, &pci_device_class))

Re: [PATCH] libxl+hvmloader: extend IGD check part 2

2025-04-08 Thread Marek Marczykowski-Górecki
On Tue, Apr 08, 2025 at 02:31:10PM +0100, Andrew Cooper wrote: > On 08/04/2025 2:23 pm, Marek Marczykowski-Górecki wrote: > > Consider also "Display controller" an IGD, not only "VGA compatible > > controller" in few more places. > > > > Fixes: 4191619e0893 ("libxl: extend IGD check") > > Signed-of