Re: [Mesa-dev] [PATCH v2 00/20] i965 basic CS program generation support

2015-04-27 Thread Kenneth Graunke
On Monday, April 27, 2015 05:44:48 PM Kenneth Graunke wrote:
> On Friday, April 24, 2015 04:32:52 PM Jordan Justen wrote:
> > git://people.freedesktop.org/~jljusten/mesa i965-cs-prog-v2
> > 
> > These patches could use review:
> >  * [08/20] i965/fs: Add emit_cs_terminate to emit CS_OPCODE_CS_TERMINATE
> >  * [10/20] i965/cs: Add generator support for CS_OPCODE_CS_TERMINATE
> >  * [13/20] i965/fs: Support compute programs in fs_visitor
> >  * [14/20] i965/cs: Add max_cs_threads
> >  * [15/20] i965/cs: Emit compute shader code and upload programs
> >  * [16/20] i965: Add brw_setup_tex_for_precompile. Use in VS, GS & FS.
> >  * [17/20] i965/cs: Support CS program precompile
> >  * [18/20] i965: Remove comment about chv device numbers relating to hsw gt1
> >  * [19/20] i965/cs: Upload brw_cs_state
> >  * [20/20] i965/fs: Add CS shader time support
> 
> Patch 19 is Acked-by, the rest are Reviewed-by, assuming you incorporate
> my suggestions.  Nice work as usual!  Thanks Jordan!

Oh, sorry, I didn't actually review the thread count patch.  I'm not
completely following how that's used in order to verify the values.


signature.asc
Description: This is a digitally signed message part.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH v2 00/20] i965 basic CS program generation support

2015-04-27 Thread Kenneth Graunke
On Friday, April 24, 2015 04:32:52 PM Jordan Justen wrote:
> git://people.freedesktop.org/~jljusten/mesa i965-cs-prog-v2
> 
> These patches could use review:
>  * [08/20] i965/fs: Add emit_cs_terminate to emit CS_OPCODE_CS_TERMINATE
>  * [10/20] i965/cs: Add generator support for CS_OPCODE_CS_TERMINATE
>  * [13/20] i965/fs: Support compute programs in fs_visitor
>  * [14/20] i965/cs: Add max_cs_threads
>  * [15/20] i965/cs: Emit compute shader code and upload programs
>  * [16/20] i965: Add brw_setup_tex_for_precompile. Use in VS, GS & FS.
>  * [17/20] i965/cs: Support CS program precompile
>  * [18/20] i965: Remove comment about chv device numbers relating to hsw gt1
>  * [19/20] i965/cs: Upload brw_cs_state
>  * [20/20] i965/fs: Add CS shader time support

Patch 19 is Acked-by, the rest are Reviewed-by, assuming you incorporate
my suggestions.  Nice work as usual!  Thanks Jordan!


signature.asc
Description: This is a digitally signed message part.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [PATCH v2 00/20] i965 basic CS program generation support

2015-04-24 Thread Jordan Justen
git://people.freedesktop.org/~jljusten/mesa i965-cs-prog-v2

These patches could use review:
 * [08/20] i965/fs: Add emit_cs_terminate to emit CS_OPCODE_CS_TERMINATE
 * [10/20] i965/cs: Add generator support for CS_OPCODE_CS_TERMINATE
 * [13/20] i965/fs: Support compute programs in fs_visitor
 * [14/20] i965/cs: Add max_cs_threads
 * [15/20] i965/cs: Emit compute shader code and upload programs
 * [16/20] i965: Add brw_setup_tex_for_precompile. Use in VS, GS & FS.
 * [17/20] i965/cs: Support CS program precompile
 * [18/20] i965: Remove comment about chv device numbers relating to hsw gt1
 * [19/20] i965/cs: Upload brw_cs_state
 * [20/20] i965/fs: Add CS shader time support

v2:
 * Incorporate feedback from v1
 * Dropped "i965/cs: Emit gen code for CS_OPCODE_CS_TERMINATE" and
   instead move into use "i965/fs: Add emit_cs_terminate to emit
   CS_OPCODE_CS_TERMINATE" and "i965/cs: Add generator support for
   CS_OPCODE_CS_TERMINATE"
 * Fixed/verified scratch memory
 * Implemented/verified shader time

Jordan Justen (14):
  i965/cs: Add BRW_NEW_CS_PROG_DATA and BRW_CACHE_CS_PROG
  i965/cs: Add CS_OPCODE_CS_TERMINATE
  i965/fs: Add emit_cs_terminate to emit CS_OPCODE_CS_TERMINATE
  i965/cs: Mark g0 as used by CS_OPCODE_CS_TERMINATE
  i965/cs: Add generator support for CS_OPCODE_CS_TERMINATE
  i965/cache: Add support for CS in program state cache
  i965/fs: Support compute programs in fs_visitor
  i965/cs: Add max_cs_threads
  i965/cs: Emit compute shader code and upload programs
  i965: Add brw_setup_tex_for_precompile. Use in VS, GS & FS.
  i965/cs: Support CS program precompile
  i965: Remove comment about chv device numbers relating to hsw gt1
  i965/cs: Upload brw_cs_state
  i965/fs: Add CS shader time support

Paul Berry (6):
  i965/cs: Add BRW_NEW_COMPUTE_PROGRAM state flag.
  mesa/cs: Add inline functions for dealing with compute shaders.
  mesa/cs: Update program.c for compute shaders.
  mesa/cs: Add compute support to update_program().
  i965: Add an INTEL_DEBUG=cs option.
  i965/cs: Add brw_cs_prog_data, brw_cs_prog_key and brw_context::cs.

 src/mesa/drivers/dri/i965/Makefile.sources|   1 +
 src/mesa/drivers/dri/i965/brw_context.c   |   3 +
 src/mesa/drivers/dri/i965/brw_context.h   |  33 ++
 src/mesa/drivers/dri/i965/brw_cs.cpp  | 388 ++
 src/mesa/drivers/dri/i965/brw_cs.h|  48 +++
 src/mesa/drivers/dri/i965/brw_defines.h   |   8 +
 src/mesa/drivers/dri/i965/brw_device_info.c   |  16 +-
 src/mesa/drivers/dri/i965/brw_device_info.h   |   1 +
 src/mesa/drivers/dri/i965/brw_fs.cpp  | 104 +-
 src/mesa/drivers/dri/i965/brw_fs.h|  15 +
 src/mesa/drivers/dri/i965/brw_fs_generator.cpp|  35 ++
 src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp |   4 +
 src/mesa/drivers/dri/i965/brw_fs_visitor.cpp  |  42 +++
 src/mesa/drivers/dri/i965/brw_program.c   |  13 +-
 src/mesa/drivers/dri/i965/brw_shader.cpp  |   6 +
 src/mesa/drivers/dri/i965/brw_shader.h|   3 +
 src/mesa/drivers/dri/i965/brw_state.h |   1 +
 src/mesa/drivers/dri/i965/brw_state_cache.c   |   3 +
 src/mesa/drivers/dri/i965/brw_state_dump.c|   3 +
 src/mesa/drivers/dri/i965/brw_state_upload.c  |  12 +
 src/mesa/drivers/dri/i965/brw_vec4.cpp|  13 +-
 src/mesa/drivers/dri/i965/intel_debug.c   |   3 +-
 src/mesa/drivers/dri/i965/intel_debug.h   |   1 +
 src/mesa/main/state.c |  21 ++
 src/mesa/program/program.c|   3 +
 src/mesa/program/program.h|  22 ++
 26 files changed, 768 insertions(+), 34 deletions(-)
 create mode 100644 src/mesa/drivers/dri/i965/brw_cs.cpp
 create mode 100644 src/mesa/drivers/dri/i965/brw_cs.h

-- 
2.1.4

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev