Re: [Mesa-dev] [PATCH] i965/cs: Initialize GPGPU Thread Count

2015-07-09 Thread Ben Widawsky
On Thu, Jul 02, 2015 at 11:32:03PM -0700, Jordan Justen wrote: > On 2015-06-25 11:34:59, Ben Widawsky wrote: > > On Thu, Jun 11, 2015 at 09:04:45PM -0700, Jordan Justen wrote: > > > + desc[dw++] = 0; > > > + const uint32_t media_threads = > > > + brw->gen >= 8 ? > > > + SET_FIELD(thre

Re: [Mesa-dev] [PATCH] i965/cs: Initialize GPGPU Thread Count

2015-07-02 Thread Jordan Justen
On 2015-06-25 11:34:59, Ben Widawsky wrote: > On Thu, Jun 11, 2015 at 09:04:45PM -0700, Jordan Justen wrote: > > + desc[dw++] = 0; > > + const uint32_t media_threads = > > + brw->gen >= 8 ? > > + SET_FIELD(threads, GEN8_MEDIA_GPGPU_THREAD_COUNT) : > > + SET_FIELD(threads, MEDIA_G

Re: [Mesa-dev] [PATCH] i965/cs: Initialize GPGPU Thread Count

2015-06-25 Thread Ben Widawsky
On Thu, Jun 11, 2015 at 09:04:45PM -0700, Jordan Justen wrote: > This field should always be set for gen8. In the bdw PRM, Volume 2d: > Command Reference: Structures under INTERFACE_DESCRIPTOR_DATA, DWORD > 6, Bits 9:0, Number of Threads in GPGPU Thread Group: > > "This field should not be set to

[Mesa-dev] [PATCH] i965/cs: Initialize GPGPU Thread Count

2015-06-11 Thread Jordan Justen
This field should always be set for gen8. In the bdw PRM, Volume 2d: Command Reference: Structures under INTERFACE_DESCRIPTOR_DATA, DWORD 6, Bits 9:0, Number of Threads in GPGPU Thread Group: "This field should not be set to 0 even if the barrier is disabled, since an accurate value is needed for