<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On 29 August 2014 08:46, Gwenole Beauchesne <span dir="ltr"><<a href="mailto:gb.devel@gmail.com" target="_blank">gb.devel@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Could you please describe in there the ownership model? I think the<br>
implementation should own the fd, so the clients should dup() it if<br>
ever necessary.<br></blockquote><div><br></div><div>So the fd can be destroyed at any time, particularly with threads? Meaning that if you want to use it, then the first thing you need to do before all others, is to dup() it, but in theory it could be destroyed by then too ...</div>
<div><br></div><div>Much better to hand the client back something that it unambiguously owns, where there's no possibility for interference. Note how EGL_EXT_image_dma_buf_import was revised from the same shared 'implementation owns fd so client should dup() model', to one where the implementation internally dup()s.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Do we have a Query API for EGLImage? I thought that was for EGLSurface<br>
only. I mean, if not, that could be interesting to retrieve all the<br>
necessary other attributes, or extend the eglExportDMABufImageMESA()<br>
function to include the same level of information than what we have<br>
for EXT_image_dma_buf_import. e.g. multi-plane stride information for<br>
instance, up to including the actual format.<br>
<br>
What happens if the EGLImage is not backing a 2D texture?<br></blockquote><div><br></div><div>Or TEXTURE_EXTERNAL, surely? Anyway, isn't that restriction a bit too harsh, that the image has to be bound (or have been bound once?) to TEXTURE_{2D,EXTERNAL}? Given how EGLImage is used as a neutral interchange type/format in places, I think supporting dma_buf_import -> EGLImage -> dma_buf_export, with no need to ever bind it to GLES texture units, is a useful thing to have.</div>
<div><br></div><div>Cheers,</div><div>Daniel</div></div></div></div>