[libdrm 1/4] etnaviv: submit full struct drm_etnaviv_gem_submit
Christian Gmeiner
christian.gmeiner at gmail.com
Fri Jun 9 10:27:34 UTC 2017
It is safe to submit the full struct even on older kernels as such
kernels do not process the full struct. Without this change it
becomes quite challenging to extned the submit struct.
Freedreno has no special treatment too. See git commits
- freedreno: sync uapi header
- freedreno: add fence fd support
Signed-off-by: Christian Gmeiner <christian.gmeiner at gmail.com>
---
etnaviv/etnaviv_cmd_stream.c | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/etnaviv/etnaviv_cmd_stream.c b/etnaviv/etnaviv_cmd_stream.c
index 3c7b0ed..8d0e813 100644
--- a/etnaviv/etnaviv_cmd_stream.c
+++ b/etnaviv/etnaviv_cmd_stream.c
@@ -203,14 +203,8 @@ static void flush(struct etna_cmd_stream *stream, int in_fence_fd,
if (out_fence_fd)
req.flags |= ETNA_SUBMIT_FENCE_FD_OUT;
- /*
- * Pass the complete submit structure only if flags are set. Otherwise,
- * only pass the fields up to, but not including the flags field for
- * backwards compatiblity with older kernels.
- */
ret = drmCommandWriteRead(gpu->dev->fd, DRM_ETNAVIV_GEM_SUBMIT,
- &req, req.flags ? sizeof(req) :
- offsetof(struct drm_etnaviv_gem_submit, flags));
+ &req, sizeof(req));
if (ret)
ERROR_MSG("submit failed: %d (%s)", ret, strerror(errno));
--
2.9.4
More information about the etnaviv
mailing list