<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jan 19, 2017 at 11:48 PM, Pohjolainen, Topi <span dir="ltr"><<a href="mailto:topi.pohjolainen@gmail.com" target="_blank">topi.pohjolainen@gmail.com</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 Thu, Jan 19, 2017 at 01:39:49PM -0800, Jason Ekstrand wrote:<br>
>    On Thu, Jan 19, 2017 at 12:40 PM, Francisco Jerez<br>
</span>>    <[1]<a href="mailto:currojerez@riseup.net">currojerez@riseup.net</a>> wrote:<br>
<span class="">><br>
>      "Pohjolainen, Topi" <[2]<a href="mailto:topi.pohjolainen@gmail.com">topi.pohjolainen@gmail.com</a><wbr>> writes:<br>
>      > On Thu, Jan 19, 2017 at 12:10:02PM -0800, Francisco Jerez wrote:<br>
</span>>      >> Topi Pohjolainen <[3]<a href="mailto:topi.pohjolainen@gmail.com">topi.pohjolainen@gmail.com</a><wbr>> writes:<br>
>      >><br>
>      >> > CC: Francisco Jerez <[4]<a href="mailto:currojerez@riseup.net">currojerez@riseup.net</a>><br>
>      >> > CC: Kenneth Graunke <[5]<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
>      >> > CC: Jason Ekstrand <[6]<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
>      >> > Signed-off-by: Topi Pohjolainen <[7]<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a><wbr>><br>
<div><div class="h5">>      >> > ---<br>
>      >> >  src/mesa/drivers/dri/i965/<wbr>genX_blorp_exec.c | 3 +++<br>
>      >> >  1 file changed, 3 insertions(+)<br>
>      >> ><br>
>      >> > diff --git a/src/mesa/drivers/dri/i965/<wbr>genX_blorp_exec.c<br>
>      b/src/mesa/drivers/dri/i965/<wbr>genX_blorp_exec.c<br>
>      >> > index 647a362..594bd5a 100644<br>
>      >> > --- a/src/mesa/drivers/dri/i965/<wbr>genX_blorp_exec.c<br>
>      >> > +++ b/src/mesa/drivers/dri/i965/<wbr>genX_blorp_exec.c<br>
>      >> > @@ -261,4 +261,7 @@ retry:<br>
>      >> ><br>
>      >> >     if (params->dst.enabled)<br>
>      >> >        brw_render_cache_set_add_bo(<wbr>brw,<br>
>      params->dst.addr.buffer);<br>
>      >> > +<br>
>      >> > +   if (params->depth.enabled)<br>
>      >> > +      brw_render_cache_set_add_bo(<wbr>brw,<br>
>      params->depth.addr.buffer);<br>
>      >><br>
>      >> What about the stencil buffer?  Stencil texturing is likely to be<br>
>      >> unhappy unless you mark it as pending flush as well...<br>
>      ><br>
>      > As far as I know i965 only clears depth and color using blorp,<br>
>      stencil gets<br>
>      > cleared using meta. Blits in turn have it as destination.<br>
>      ><br>
>      That doesn't sound like a safe assumption to rely on looking forward<br>
>      if<br>
>      the blorp api already exposes support for stencil writes -- Tracking<br>
>      down the ultimate cause of a memory coherency bugs can be really<br>
>      hard,<br>
>      why make our future lives more intentionally difficult by<br>
>      introducing<br>
>      buggy corner cases like this?  The extra check is not going to hurt<br>
>      performance or cause any other harmful side effects unless stencil<br>
>      writes are used...<br>
><br>
>    Agreed.  Let's stick it in there for stencil too.  I've got patches to<br>
>    switch i965 over to blorp for depth/stencil blits.  I never landed them<br>
>    because of what was most likely flushing bugs.  I'm hoping that you've<br>
>    fixed those and I'll revive the patches.<br>
>    Also, please make sure these fixes hit stable.<br>
<br>
</div></div>This sits on top the four earlier patches. Rebasing this alone against stable<br>
requires manual work but can be done. How do you want to handle that?<br></blockquote><div><br></div><div>Ken, Curro, and I had a little chat about this in the office.  I think the conclusion we came to was the following:<br><br></div><div>1) The patches to add flushing around HiZ ops and fast clear ops should get back-ported all the way to 13.0.  They fix potentially serious bugs that could cause problems.<br></div><div>2) The patches that switch us over to the render cache should get backported to 17.0.  They aren't so much a bug fix as an enhancement but keeping 17.0 consistent with future will help in backporting other fixes.  For the record, this was me and Ken; Curro preferred to not backport these. <br></div></div><br></div></div>