<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 28 March 2017 at 17:09, 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">On 22 March 2017 at 10:06, Bas Nieuwenhuizen <<a href="mailto:bas@basnieuwenhuizen.nl">bas@basnieuwenhuizen.nl</a>> wrote:<br>
> On Tue, Mar 21, 2017 at 1:02 PM, Alex Smith <<a href="mailto:asmith@feralinteractive.com">asmith@feralinteractive.com</a>> wrote:<br>
>> CP DMA and PKT3_WRITE_DATA (in CmdUpdateBuffer) don't (currently) write<br>
>> through L2. Therefore, to make these writes visible to later accesses<br>
>> we must invalidate L2 rather than just writing it back, to avoid the<br>
>> possibility that stale data is read through L2.<br>
>><br>
>> Cc: "17.0" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.<wbr>freedesktop.org</a>><br>
>> Signed-off-by: Alex Smith <<a href="mailto:asmith@feralinteractive.com">asmith@feralinteractive.com</a>><br>
>> ---<br>
>> It's possible for both CP DMA and PKT3_WRITE_DATA to write through L2<br>
>> as far as I can see, and changing things so that they do also solves<br>
>> the problems that this patch fixes.<br>
>><br>
>> However, I don't know what the exact consequences of doing so are, or<br>
>> whether there are any situations where that shouldn't be done, so I've<br>
>> gone with this fix instead as it seems like a safer option for now.<br>
><br>
> Yeah we should be able to. I'm more comfortable sending this patch to<br>
> stable though, so this patch is<br>
><br>
</span>Bas, others,<br>
<br>
Patch addresses radv_{src,dst}_access_flush() which landed with commit<br>
6dbb0eaccc3, after the 17.0 branchpoint.<br></blockquote><div><br></div><div>Oops, my mistake.</div><div><br></div><div>I think <span style="color:rgb(0,0,0);font-size:13.3333px">radv_CmdPipelineBarrier on the 17.0 branch still needs </span><span style="color:rgb(0,0,0);font-size:13.3333px">RADV_CMD_FLAG_INV_GLOBAL_L2 added for TRANSFER_WRITE barriers at least. Bas, do you think that should be added in a separate patch just for stable, or would you prefer to push those later changes to stable as well? Looks like there's some fixes in those as well.</span></div><div><span style="color:rgb(0,0,0);font-size:13.3333px"><br></span></div><div><span style="color:rgb(0,0,0);font-size:13.3333px">Alex</span></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
>From a quick look, the functions has been fixed/enhanced with the<br>
following 7 commits. Some of which require additional patches.<br>
If we want this in stable, please provide a backport or branch based<br>
on top of mesa-17.0.2.<br>
<br>
Thanks<br>
Emil<br>
<br>
40e0dbf96c4d812be940561f5732b1<wbr>b0e44b5e1d<br>
f3dc318464b786c2696e650e7c6998<wbr>4b5453624b<br>
b075eb7d476eb750092f72e7ec65bc<wbr>41003fa658<br>
dd094e4ff9ff0967b515a4330e40fe<wbr>ca55247e25<br>
7a600bbc8186fef9475bedfe6d58a5<wbr>4011a8022b<br>
0567ab0407e4058c108443b90b7c23<wbr>a40c391c3b<br>
f92a118434452df201cda6d9ec2405<wbr>aca669b104<br>
</blockquote></div><br></div></div>