<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Dec 5, 2017 at 8:23 AM, Kristian Høgsberg <span dir="ltr"><<a href="mailto:hoegsberg@gmail.com" target="_blank">hoegsberg@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Dec 5, 2017 at 7:57 AM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> On Tue, Dec 5, 2017 at 1:22 AM, Daniel Stone <<a href="mailto:daniel@fooishbar.org">daniel@fooishbar.org</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> On 18 November 2017 at 00:10, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
>> > This fixes a bug where we were taking the tiling from the BO regardless<br>
>> > of what the modifier said.  When we got images in from Vulkan where it<br>
>> > doesn't set the tiling on the BO, we would treat them as linear even<br>
>> > though the modifier expressly said to treat it as Y-tiled.<br>
>><br>
>> For some reason I thought Ken had already reviewed this and it landed,<br>
>> until Kristian mentioned last night.<br>
><br>
><br>
> There's been some discussion about what the right thing to do is here.  I've<br>
> got a patch (which is now landed) which will make us set the tiling from<br>
> Vulkan just like GL does.  It's rather annoying but I think that's<br>
> marginally better.<br>
<br>
</div></div>I don't know that it's better - it reinforces the notion that you have<br>
to make the kernel-side tiling match for the dma-buf import extension<br>
to work. I think it'd be better to land these patches (btw, Rb: me<br>
(can you even do parenthetical Rbs?))</blockquote><div><br></div><div>I'll allow it. :)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> and call set-tiling in mesa.</blockquote><div><br></div><div>Yeah, I think that's reasonable.  Really, this should only be a problem if we have a bunch of users trying to use the same BO with different modifiers.  It can happen in theory (immagine two images in the same BO, one X and one Y) but it's a pretty crazy case.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> The<br>
assumption being that if the tiling doesn't match the modifier, then<br>
maybe the producer didn't care about kernel tiling? Alternatively,<br>
could we set bo->tiling = INCONSISTENT or such in mesa and then use<br>
that to avoid the gtt map paths?<br></blockquote></div></div></div>