[Mesa-dev] [PATCH] egl/dri2: Allow modifiers to add FDs to imports

Daniel Stone daniels at collabora.com
Mon Jul 31 17:35:42 UTC 2017


When using dmabuf import, make sure that the modifier is actually
allowed to add planes to the base format, as implied by the comment.

Signed-off-by: Daniel Stone <daniels at collabora.com>
---
 src/egl/drivers/dri2/egl_dri2.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
index b73dcd72b6..76294897a5 100644
--- a/src/egl/drivers/dri2/egl_dri2.c
+++ b/src/egl/drivers/dri2/egl_dri2.c
@@ -2166,8 +2166,10 @@ dri2_check_dma_buf_format(const _EGLImageAttribs *attrs)
           *  this extension."
           */
          if (attrs->DMABufPlaneModifiersLo[i].IsPresent &&
-             attrs->DMABufPlaneModifiersHi[i].IsPresent)
+             attrs->DMABufPlaneModifiersHi[i].IsPresent) {
+            plane_n = i + 1;
             continue;
+	}
 
          _eglError(EGL_BAD_ATTRIBUTE, "too many plane attributes");
          return 0;
-- 
2.13.3



More information about the mesa-dev mailing list