broadcom/vc4: Skip BO labeling when in simulator mode.
authorEric Anholt <eric@anholt.net>
Fri, 13 Oct 2017 19:16:04 +0000 (12:16 -0700)
committerEric Anholt <eric@anholt.net>
Tue, 17 Oct 2017 19:35:16 +0000 (12:35 -0700)
It was calling down into i915 trying to label the BO, which is definitely
not the right thing.

src/gallium/drivers/vc4/vc4_bufmgr.c
src/gallium/drivers/vc4/vc4_simulator.c

index b300fa5bd025e43d91e2e1ff48637a2332d50d12..d06d55f864503c6a6d9cf6e56a510ca5ecdaff78 100644 (file)
@@ -69,7 +69,7 @@ vc4_bo_label(struct vc4_screen *screen, struct vc4_bo *bo, const char *fmt, ...)
                 .len = strlen(name),
                 .name = (uintptr_t)name,
         };
-        drmIoctl(screen->fd, DRM_IOCTL_VC4_LABEL_BO, &label);
+        vc4_ioctl(screen->fd, DRM_IOCTL_VC4_LABEL_BO, &label);
 
         ralloc_free(name);
 }
index cf9821236f382644b6b339877a57d8c720f6bfd5..e70ade7d359cc9dc38c9d02ff2ecfd24f5f968e7 100644 (file)
@@ -654,6 +654,10 @@ vc4_simulator_ioctl(int fd, unsigned long request, void *args)
                  */
                 return 0;
 
+        case DRM_IOCTL_VC4_LABEL_BO:
+                /* This is just debug information, nothing to do. */
+                return 0;
+
         case DRM_IOCTL_VC4_GET_TILING:
         case DRM_IOCTL_VC4_SET_TILING:
                 /* Disable these for now, since the sharing with i965 requires