exclude vgem node and invalid drm node in vainfo

call drm function to avoid invalid node
check the name of vgem

Signed-off-by: Carl Zhang <[email protected]>

Author: XinfengZhang

common/Makefile.am

@@ -41,7 +41,7 @@ endif
 
 if USE_DRM
 source_c		+= va_display_drm.c
-libva_display_cflags	+= $(DRM_CFLAGS) $(LIBVA_DRM_CFLAGS)
+libva_display_cflags	+= $(DRM_CFLAGS) $(LIBVA_DRM_CFLAGS) $(DRM_CFLAGS)
 libva_display_libs	+= $(DRM_LIBS) $(LIBVA_DRM_LIBS)
 endif
 

common/va_display_drm.c

@@ -24,13 +24,15 @@
 
 #include <stdio.h>
 #include <stdlib.h>
+#include <string.h>
 #include <unistd.h>
 #include <fcntl.h>
 #ifdef IN_LIBVA
 # include "va/drm/va_drm.h"
 #else
 # include <va/va_drm.h>
 #endif
+#include <xf86drm.h>
 #include "va_display.h"
 
 static int drm_fd = -1;
@@ -41,10 +43,12 @@ va_open_display_drm(void)
 {
     VADisplay va_dpy;
     int i;
-
+    drmVersionPtr version;
     static const char *drm_device_paths[] = {
         "/dev/dri/renderD128",
         "/dev/dri/card0",
+        "/dev/dri/renderD129",
+        "/dev/dri/card1",
         NULL
     };
 
@@ -72,6 +76,18 @@ va_open_display_drm(void)
         if (drm_fd < 0)
             continue;
 
+        version = drmGetVersion(drm_fd);
+        if (!version) {
+            close(drm_fd);
+            continue;
+        }
+        if (!strncmp(version->name, "vgem", 4)) {
+            drmFreeVersion(version);
+            close(drm_fd);
+            continue;
+        }
+        drmFreeVersion(version);
+
         va_dpy = vaGetDisplayDRM(drm_fd);
         if (va_dpy)
             return va_dpy;

configure.ac

@@ -144,6 +144,7 @@ AM_CONDITIONAL(USE_SSP, test "$ssp_cc" = "yes")
 
 # Check for DRM (mandatory)
 PKG_CHECK_MODULES([LIBVA_DRM], [libva-drm])
+PKG_CHECK_MODULES([DRM], [libdrm])
 
 # Check for libva (for dynamic linking)
 LIBVA_API_MIN_VERSION=libva_api_min_version

meson.build

@@ -21,6 +21,7 @@ if get_option('drm') != 'false'
   require_drm = get_option('drm') == 'true'
   drm_deps = [
     dependency('libva-drm', required: require_drm),
+    dependency('drm', required: require_drm),
   ]
   use_drm = true
   foreach d : drm_deps