It is OK for Intel driver. However I am not sure there is no side effect to other drivers or not.
> Some drm calls from va_getDriverName() are required even when > using shell variable to set video driver name. > Otherwise, drm errors occur. > > Change-Id: Iaf3d06d1b36af27de4ec5befc7d6600173aa1264 > Signed-off-by: Lukasz Anaczkowski <[email protected]> > --- > va/va.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/va/va.c b/va/va.c > index ddd105e..03f86e7 100644 > --- a/va/va.c > +++ b/va/va.c > @@ -431,15 +431,15 @@ VAStatus vaInitialize ( > > va_infoMessage("VA-API version %s\n", VA_VERSION_S); > > + vaStatus = va_getDriverName(dpy, &driver_name); > + va_infoMessage("va_getDriverName() returns %d\n", vaStatus); > + /* Look for env variable */ > driver_name_env = getenv("LIBVA_DRIVER_NAME"); > if (driver_name_env && geteuid() == getuid()) { > /* Don't allow setuid apps to use LIBVA_DRIVER_NAME */ > + free(driver_name); // drop name from DRM > driver_name = strdup(driver_name_env); > - vaStatus = VA_STATUS_SUCCESS; > va_infoMessage("User requested driver '%s'\n", driver_name); > - } else { > - vaStatus = va_getDriverName(dpy, &driver_name); > - va_infoMessage("va_getDriverName() returns %d\n", vaStatus); > } > > if (VA_STATUS_SUCCESS == vaStatus) { _______________________________________________ Libva mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libva
