fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-08 11:18:08 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

loader: fallback to kernel name, if PCI fails

Browse source 
Currently, if the PCI machinery fails, we return a NULL driver name.
In the past this has resulted in various workarounds.

To avoid those, fallback to loader_get_kernel_driver_name(). It's not
perfect, yet perfectly reasonable.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>
Reviewed-by: Eric Engestrom <eric@engestrom.ch>
Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4084>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4084>
This commit is contained in:
Emil Velikov 2020-03-05 14:41:25 +00:00 • committed by Emil Velikov
parent bf1838838a
commit 91478db20d
1 changed files with 33 additions and 23 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

56
src/loader/loader.c
View file

@ -428,33 +428,14 @@ loader_get_device_name_for_fd(int fd)
return result; return result;
} }
char * static char *
loader_get_driver_for_fd(int fd) loader_get_pci_driver(int fd)
{ {
int vendor_id, chip_id, i, j; int vendor_id, chip_id, i, j;
char *driver = NULL; char *driver = NULL;
/* Allow an environment variable to force choosing a different driver if (!loader_get_pci_id_for_fd(fd, &vendor_id, &chip_id))
* binary. If that driver binary can't survive on this FD, that's the return NULL;
* user's problem, but this allows vc4 simulator to run on an i965 host,
* and may be useful for some touch testing of i915 on an i965 host.
*/
if (geteuid() == getuid()) {
driver = getenv("MESA_LOADER_DRIVER_OVERRIDE");
if (driver)
return strdup(driver);
}
#if defined(HAVE_LIBDRM) && defined(USE_DRICONF)
driver = loader_get_dri_config_driver(fd);
if (driver)
return driver;
#endif
if (!loader_get_pci_id_for_fd(fd, &vendor_id, &chip_id)) {
driver = loader_get_kernel_driver_name(fd);
return driver;
}
for (i = 0; i < ARRAY_SIZE(driver_map); i++) { for (i = 0; i < ARRAY_SIZE(driver_map); i++) {
if (vendor_id != driver_map[i].vendor_id) if (vendor_id != driver_map[i].vendor_id)
@ -482,6 +463,35 @@ out:
return driver; return driver;
} }
char *
loader_get_driver_for_fd(int fd)
{
char *driver;
/* Allow an environment variable to force choosing a different driver
* binary. If that driver binary can't survive on this FD, that's the
* user's problem, but this allows vc4 simulator to run on an i965 host,
* and may be useful for some touch testing of i915 on an i965 host.
*/
if (geteuid() == getuid()) {
driver = getenv("MESA_LOADER_DRIVER_OVERRIDE");
if (driver)
return strdup(driver);
}
#if defined(HAVE_LIBDRM) && defined(USE_DRICONF)
driver = loader_get_dri_config_driver(fd);
if (driver)
return driver;
#endif
driver = loader_get_pci_driver(fd);
if (!driver)
driver = loader_get_kernel_driver_name(fd);
return driver;
}
void void
loader_set_logger(loader_logger *logger) loader_set_logger(loader_logger *logger)
{ {