<div dir="ltr">Hi Marek,<br><br>I agree with Ken here. <div><br></div><div>For what it's worth, the list of our titles that use primitive restart here is likely the full list. DXMD was the first as far as I know - see <a href="https://cgit.freedesktop.org/mesa/mesa/commit/?id=e33f31d61f5e9019f8b0bac0378dfb8fd1147421" target="_blank">https://cgit.freedesktop.org/<wbr>mesa/mesa/commit/?id=<wbr>e33f31d61f5e9019f8b0bac0378dfb<wbr>8fd1147421</a>. It also appears to be an app side issue so we will patch as needed.</div><div><br></div><div>In the future let us know first, in pretty much all cases we'd prefer to make the change on our side instead of adding game specific hacks in Mesa.</div><div><div><br>Cheers,</div><div><br></div></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>-----</div><div><span style="font-size:12.8000001907349px">Marc Di Luzio</span><br></div><div><span style="font-size:12.8000001907349px">Linux Group Lead @ </span><span style="color:rgb(0,0,0);font-family:Helvetica;font-size:12.8px">Feral Interactive Ltd.</span></div></div></div></div></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On 24 April 2017 at 23:26, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</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 Monday, April 24, 2017 6:22:41 AM PDT Marek Olšák wrote:<br>
> From: Marek Olšák <<a href="mailto:marek.olsak@amd.com">marek.olsak@amd.com</a>><br>
><br>
> ---<br>
>  src/gallium/drivers/radeonsi/<wbr>si_pipe.c       | 20 +++++++++++++<br>
>  src/gallium/drivers/radeonsi/<wbr>si_pipe.h       |  1 +<br>
>  src/gallium/drivers/radeonsi/<wbr>si_state_draw.c | 45 ++++++++++++++++++++--------<br>
>  3 files changed, 54 insertions(+), 12 deletions(-)<br>
><br>
> diff --git a/src/gallium/drivers/<wbr>radeonsi/si_pipe.c b/src/gallium/drivers/<wbr>radeonsi/si_pipe.c<br>
> index 1a83564..53a8201 100644<br>
> --- a/src/gallium/drivers/<wbr>radeonsi/si_pipe.c<br>
> +++ b/src/gallium/drivers/<wbr>radeonsi/si_pipe.c<br>
> @@ -29,20 +29,29 @@<br>
>  #include "radeon/radeon_uvd.h"<br>
>  #include "util/u_memory.h"<br>
>  #include "util/u_suballoc.h"<br>
>  #include "util/u_tests.h"<br>
>  #include "vl/vl_decoder.h"<br>
>  #include "../ddebug/dd_util.h"<br>
><br>
>  #define SI_LLVM_DEFAULT_FEATURES \<br>
>       "+DumpCode,+vgpr-spilling,-<wbr>fp32-denormals,-xnack"<br>
><br>
> +/* DX10/11 apply primitive restart to strip primitive types only. */<br>
> +static const char *apps_with_prim_restart_dx_<wbr>behavior[] = {<br>
> +     "DeusExMD",<br>
> +     "DirtRally",<br>
> +     "HitmanPro",<br>
> +     "MadMax",<br>
> +     "TotalWarhammer",<br>
> +};<br>
> +<br>
<br>
</div></div>Hi Marek,<br>
<br>
You seem to be adding driver workarounds for an incomplete list of Feral<br>
Interactive's titles.  Presumably, if you're going to go this route, you<br>
may need to add more of them.  Or, perhaps this is something they can<br>
fix in their translator layer, so they only enable it when they want it?<br>
<br>
I've copied Marc and Alex from Feral in case they want to weigh in.<br>
<br>
--Ken<br>
</blockquote></div><br></div>