<div dir="ltr"><div>NAK.</div><div><br></div><div>On SNB+, we'd like to get Y-tiling when the surface is only being used by the compositor and reserve X-tiling for scanout surfaces.  With the modifiers negotiation framework, the X server should be able to figure out that the window is being composited (or blitted) and request Y-tiling.  If the surface is full-screen, it should know this and either require X-tiling because that's all KMS advertises or not flip the buffer because it uses a modifier that's unsupported by the kernel.  I think you have an X server bug.</div><div><br></div><div>--Jason<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jul 31, 2018 at 7:54 AM Mario Kleiner <<a href="mailto:mario.kleiner.de@gmail.com">mario.kleiner.de@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">At least that's what the Linux 4.18 i915 drm/kms driver<br>
thinks, and the disagreement between kms and Mesa leads to<br>
pageflipping failure with X-Server 1.20's dmabuf modifiers<br>
enabled modesetting-ddx, at least as tested on a gen 7<br>
Ivybridge system.<br>
<br>
Signed-off-by: Mario Kleiner <<a href="mailto:mario.kleiner.de@gmail.com" target="_blank">mario.kleiner.de@gmail.com</a>><br>
---<br>
 src/mesa/drivers/dri/i965/intel_screen.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/mesa/drivers/dri/i965/intel_screen.c b/src/mesa/drivers/dri/i965/intel_screen.c<br>
index cb35741..ef04ffb 100644<br>
--- a/src/mesa/drivers/dri/i965/intel_screen.c<br>
+++ b/src/mesa/drivers/dri/i965/intel_screen.c<br>
@@ -309,7 +309,7 @@ static const struct {<br>
 } supported_modifiers[] = {<br>
    { .modifier = DRM_FORMAT_MOD_LINEAR       , .since_gen = 1 },<br>
    { .modifier = I915_FORMAT_MOD_X_TILED     , .since_gen = 1 },<br>
-   { .modifier = I915_FORMAT_MOD_Y_TILED     , .since_gen = 6 },<br>
+   { .modifier = I915_FORMAT_MOD_Y_TILED     , .since_gen = 9 },<br>
    { .modifier = I915_FORMAT_MOD_Y_TILED_CCS , .since_gen = 9 },<br>
 };<br>
<br>
-- <br>
2.7.4<br>
<br>
_______________________________________________<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/mailman/listinfo/mesa-dev</a><br>
</blockquote></div>