<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, 4 Jun 2018 at 21:36, Ilia Mirkin <<a href="mailto:imirkin@alum.mit.edu" target="_blank">imirkin@alum.mit.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Jun 4, 2018 at 4:30 PM, Plamena Manolova<br>
<<a href="mailto:plamena.n.manolova@gmail.com" target="_blank">plamena.n.manolova@gmail.com</a>> wrote:<br>
> Thank you for the review Ilia!<br>
><br>
> On Fri, 1 Jun 2018 at 23:44, Ilia Mirkin <<a href="mailto:imirkin@alum.mit.edu" target="_blank">imirkin@alum.mit.edu</a>> wrote:<br>
>><br>
>> On Fri, Jun 1, 2018 at 6:21 PM, Plamena Manolova<br>
>> <<a href="mailto:plamena.n.manolova@gmail.com" target="_blank">plamena.n.manolova@gmail.com</a>> wrote:<br>
>> > +      /*<br>
>> > +       * If the local work group size is variable we have to use a<br>
>> > dispatch<br>
>> > +       * width of 32 here, since at this point we don't know the actual<br>
>> > size of<br>
>> > +       * the workload.<br>
>> > +       */<br>
>> > +      min_dispatch_width = 32;<br>
>><br>
>> Is that a good idea? You are able to specify a different maximum when<br>
>> using a variable size (MAX_COMPUTE_VARIABLE_GROUP_INVOCATIONS_ARB)<br>
>> s.t. this is 16 (or even 8, although that may be too few for practical<br>
>> use) -- that way you would just set the max to 768 or whatever on<br>
>> gen8+.<br>
><br>
><br>
> That's a good point, MAX_COMPUTE_VARIABLE_GROUP_INVOCATIONS_ARB is the<br>
> same on all platforms, so it makes sense to use simd16 instead. Thank you<br>
> for noticing that.<br>
<br>
Well IIRC gen7/gen7.5 can do something like 1536 invocations with<br>
SIMD16, while gen8 are down to 768. Either way, not forcing SIMD32 may<br>
be nice -- but that's something for you Intel folk to decide. I just<br>
wanted to point out that you could have different max's for "regular"<br>
compute (where the min max is 1024) and variable-size groups.<br></blockquote><div><br></div><div>Ah, I see what you mean now. I took a closer look at </div><div>MAX_COMPUTE_VARIABLE_GROUP_INVOCATIONS_ARB and it's </div><div>related to the maximum number of CS threads (which of course varies).</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
  -ilia<br>
</blockquote></div></div>