<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 20, 2017 at 4:15 PM, Ben Widawsky <span dir="ltr"><<a href="mailto:ben@bwidawsk.net" target="_blank">ben@bwidawsk.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 17-03-20 12:03:24, Jason Ekstrand wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
My only question here is whether or not we want to add a "supported<br>
modifiers" query to DRIimage before we bump the version number.  It's<br>
probably not super-important because we can always bump the version number<br>
again.  Meh.<br>
<br>
</blockquote>
<br></span>
Remind me what the reasoning is for support modifiers? I thought we agreed to<br>
just use GET_PLANE2? Supported modifiers is difficult because mesa has no idea<br>
of per plane constraints.</blockquote><div><br></div><div>They're for completely different cases.  If you want to scan out, use GET_PLANE2, end of story.  If, however, you're not trying to scan out and your window is always going to get composited (think not full-screen) then you may have a much larger set of modifiers available to you and that set of modifiers is not determined by GET_PLANE2 but by the GL or whatever is being used for compositing.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<div class="HOEnZb"><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sat, Mar 18, 2017 at 1:00 PM, Ben Widawsky <<a href="mailto:ben@bwidawsk.net" target="_blank">ben@bwidawsk.net</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
All the plumbing is in place so the extension just needs to be<br>
advertised.<br>
<br>
Signed-off-by: Ben Widawsky <<a href="mailto:ben@bwidawsk.net" target="_blank">ben@bwidawsk.net</a>><br>
---<br>
 src/mesa/drivers/dri/i965/int<wbr>el_screen.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/mesa/drivers/dri/i965/in<wbr>tel_screen.c<br>
b/src/mesa/drivers/dri/i965/in<wbr>tel_screen.c<br>
index ca725d9202..1ff18b30e4 100644<br>
--- a/src/mesa/drivers/dri/i965/in<wbr>tel_screen.c<br>
+++ b/src/mesa/drivers/dri/i965/in<wbr>tel_screen.c<br>
@@ -948,7 +948,7 @@ intel_from_planar(__DRIimage *parent, int plane, void<br>
*loaderPrivate)<br>
 }<br>
<br>
 static const __DRIimageExtension intelImageExtension = {<br>
-    .base = { __DRI_IMAGE, 13 },<br>
+    .base = { __DRI_IMAGE, 14 },<br>
<br>
     .createImageFromName                = intel_create_image_from_name,<br>
     .createImageFromRenderbuffer        = intel_create_image_from_<br>
renderbuffer,<br>
--<br>
2.12.0<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
<br>
</blockquote></blockquote>
<br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
Ben Widawsky, Intel Open Source Technology Center<br>
</font></span></blockquote></div><br></div></div>