<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Dec 12, 2017 at 5:42 AM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@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 11 December 2017 at 22:03, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> On Mon, Dec 11, 2017 at 12:08 PM, Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
> wrote:<br>
>><br>
>> On 21 November 2017 at 00:13, Andres Gomez <<a href="mailto:agomez@igalia.com">agomez@igalia.com</a>> wrote:<br>
>> > Jason, this nominated series landed without mentioning any specific<br>
>> > stable queue.<br>
>> ><br>
>> > From what I'm seeing, both depend on 2c4097aff1b which didn't make it<br>
>> > for 17.2 so I'm dropping them for that queue.<br>
>> ><br>
>> This is a preparatory patch for the follow-up commit.<br>
>> With the latter addressing issue caused by<br>
>> 4b1e70cc57d7ff5f465544644b2180<wbr>dee1490cee - only available in<br>
>> 17.4.0-dev.<br>
>><br>
>> I believe the 17.2 and 17.3 series should be safe ;-)<br>
><br>
><br>
> This bug goes all the way back.  It interacts with the patch mentioned but<br>
> is not the same thing.  This patch (and the subsequent one) should probably<br>
> go back as far as brw_bufmgr does.<br>
</div></div>Having a closer look at the patches related to external buffers, shows<br>
the following:<br>
<br>
2c4097aff1b i965: Only put external handles into the handle ht<br>
17.3 only<br>
<br>
d7a19d69ebc i965: Use PTE MOCS for all external buffers<br>
17.4 + 17.3<br>
4b1e70cc57d i965: Switch over to fully external-or-not MOCS scheme<br>
17.4 only<br>
<br>
344252a27f8 i965/bufmgr: Add a helper to mark a BO as external<br>
0a6a137eb27 i965: Mark BOs as external when we export their handle<br>
These fixes<br>
<br>
With the above in mind, it seems that 4b1e70cc57d should have been<br>
tagged for 17.3?<br></blockquote><div><br></div><div>It's not strictly necessary but probably makes back-porting easier.  I'm fine with pulling it into 17.3.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On the 17.2 front: Andres is right - commit 2c4097aff1b seems like a<br>
bare minimum for this series.<br>
Considering how well into the 17.2 series we are, I'm a bit worried<br>
about picking if :-\<br></blockquote><div><br></div><div>Agreed. <br></div></div></div></div>