[Mesa-dev] [PATCH v1] virgl: add assert and missing function parameter

Robert Foss robert.foss at collabora.com
Tue Nov 20 16:24:23 UTC 2018


Verify the pipe_fd_type to be of PIPE_FD_TYPE_NATIVE_SYNC.

Suggested-by: Eric Engestrom <eric.engestrom at intel.com>
Signed-off-by: Robert Foss <robert.foss 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 9be7775abd3..892fef76c75 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);
-- 
2.17.1



More information about the mesa-dev mailing list