Mesa (virgl_fix_type_v1): virgl: add assert and missing function parameter

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Wed Nov 21 15:16:14 UTC 2018


Module: Mesa
Branch: virgl_fix_type_v1
Commit: 19af208c7de52561ec8ecb99b61e799ffe4ef67c
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=19af208c7de52561ec8ecb99b61e799ffe4ef67c

Author: Robert Foss <robert.foss at collabora.com>
Date:   Tue Nov 20 16:38:27 2018 +0100

virgl: add assert and missing function parameter

Verify the pipe_fd_type to be of PIPE_FD_TYPE_NATIVE_SYNC.

Fixes: d1a1c21e7621b5177feb "virgl: native fence fd support"

Suggested-by: Eric Engestrom <eric.engestrom at intel.com>
Signed-off-by: Robert Foss <robert.foss at collabora.com>
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

---

 src/gallium/drivers/virgl/virgl_context.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/gallium/drivers/virgl/virgl_context.c b/src/gallium/drivers/virgl/virgl_context.c
index 9be7775abd..892fef76c7 100644
--- a/src/gallium/drivers/virgl/virgl_context.c
+++ b/src/gallium/drivers/virgl/virgl_context.c
@@ -1030,8 +1030,11 @@ static void virgl_set_shader_buffers(struct pipe_context *ctx,
 }
 
 static void virgl_create_fence_fd(struct pipe_context *ctx,
-			        struct pipe_fence_handle **fence, int fd)
+                                  struct pipe_fence_handle **fence,
+                                  int fd,
+                                  enum pipe_fd_type type)
 {
+   assert(type == PIPE_FD_TYPE_NATIVE_SYNC);
    struct virgl_screen *rs = virgl_screen(ctx->screen);
 
    *fence = rs->vws->cs_create_fence(rs->vws, fd);




More information about the mesa-commit mailing list