Re: [Mesa-dev] [PATCH 08/22] i965/gs: Allocate URB space for use by GS.

2013-08-31 Thread Paul Berry
On 29 August 2013 19:37, Chad Versace wrote: > On 08/26/2013 03:12 PM, Paul Berry wrote: > > + /* VS always requires at least 32 URB entries */ >> > > Haswell GT2 and GT3 require at least 64 URB entries. Good catch. Thank you double checking me on that. I'll make a follow-up patch that add

[Mesa-dev] [PATCH] i965/gen7.5: Fix lower bound on number of VS URB entries.

2013-08-31 Thread Paul Berry
Haswell GT2 and GT3 require the number of vertex shader URB entries to be at least 64, not 32. At the moment, we always meet this requirement automatically, because in the absence of a geometry shader, we assign all available URB space to the vertex shader. But when we turn on support for geometr

[Mesa-dev] [PATCH 1/3] i965/vs: Move vs-specific code out of brw_vec4.h.

2013-08-31 Thread Paul Berry
Now brw_vec4.h contains only code that is shared between the vertex and geometry shaders. --- src/mesa/drivers/dri/i965/brw_vec4.h | 32 -- src/mesa/drivers/dri/i965/brw_vs.h | 38 2 files changed, 38 insertions(+), 32 deletions(-)

[Mesa-dev] [PATCH 2/3] i965/vec4: Make with_writemask() non-static.

2013-08-31 Thread Paul Berry
This will allow it to be shared between brw_vec4_visitor.cpp and brw_vec4_vs_visitor.cpp (which will be created in the next patch). --- src/mesa/drivers/dri/i965/brw_vec4.h | 3 +++ src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) di

[Mesa-dev] [PATCH 3/3] i965/vs: Move vs-specific code out of brw_vec4_visitor.cpp.

2013-08-31 Thread Paul Berry
This patch creates a new file brw_vec4_vs_visitor.cpp, to contain code that is specific to the vertex shader. Now the organization of vertex shader and geometry shader visitor code is symmetric: vs-specific code is in brw_vec4_vs_visitor.cpp, gs-specific code is in brw_vec4_gs_visitor.cpp, and cod

Re: [Mesa-dev] [PATCH 1/3] i965/vs: Move vs-specific code out of brw_vec4.h.

2013-08-31 Thread Kenneth Graunke
On 08/31/2013 09:08 PM, Paul Berry wrote: Now brw_vec4.h contains only code that is shared between the vertex and geometry shaders. Series is: Acked-by: Kenneth Graunke ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedeskto