Re: [Mesa-dev] [PATCH v03 14/38] i965: Move MOCS macros to brw_context.h.

2017-05-04 Thread Rafael Antognolli
On Wed, May 03, 2017 at 11:36:29PM -0700, Kenneth Graunke wrote: > On Wednesday, May 3, 2017 7:52:01 PM PDT Pohjolainen, Topi wrote: > > On Wed, May 03, 2017 at 05:11:45PM -0700, Rafael Antognolli wrote: > > > On Wed, May 03, 2017 at 08:28:24PM +0300, Pohjolainen, Topi wrote: &

[Mesa-dev] [PATCH] i965: Move MOCS macros to brw_state.h.

2017-05-04 Thread Rafael Antognolli
brw_state.h is a better place to keep them, instead of brw_context.h. Signed-off-by: Rafael Antognolli Reviewed-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_context.h | 42 - src/mesa/drivers/dri/i965/brw_state.h | 42

Re: [Mesa-dev] [PATCH 1/3] mesa: Simplify _mesa_primitive_restart_index().

2017-05-04 Thread Rafael Antognolli
Reviewed-by: Rafael Antognolli On Thu, May 04, 2017 at 08:13:05AM -0700, Kenneth Graunke wrote: > We can use a simple shift equation rather than a switch statement. > --- > src/mesa/main/varray.c | 12 ++-- > 1 file changed, 2 insertions(+), 10 deletions(-) > > dif

[Mesa-dev] [PATCH] genxml: Fix 3DSTATE_DEPTH_BUFFER length on gen5.

2017-05-03 Thread Rafael Antognolli
The hardware docs are wrong, but the length used in the xml is also wrong. Signed-off-by: Rafael Antognolli --- src/intel/genxml/gen5.xml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/intel/genxml/gen5.xml b/src/intel/genxml/gen5.xml index 0b84650..447499c 100644

Re: [Mesa-dev] [PATCH v03 14/38] i965: Move MOCS macros to brw_context.h.

2017-05-03 Thread Rafael Antognolli
On Wed, May 03, 2017 at 08:28:24PM +0300, Pohjolainen, Topi wrote: > On Mon, May 01, 2017 at 06:43:02PM -0700, Rafael Antognolli wrote: > > These macros are defined in brw_defines.h, which contains a lot of > > macros that conflict with autogenerated code from genxml. But we need

Re: [Mesa-dev] [PATCH v03 17/38] genxml: Add rules to build gen4, gen45 and ge5.

2017-05-03 Thread Rafael Antognolli
01, 2017 at 06:43:05PM -0700, Rafael Antognolli wrote: > > Signed-off-by: Rafael Antognolli > > --- > > src/mesa/drivers/dri/i965/Makefile.am | 12 > > src/mesa/drivers/dri/i965/Makefile.sources | 9 + > > src/mesa/drivers/dri/i965/b

Re: [Mesa-dev] i965: Use isl for hiz and stencil

2017-05-03 Thread Rafael Antognolli
gt; isl. That should allow me to start using isl state emitter for > depth-hiz-stencil. > > CC: Jason Ekstrand > CC: Nanley Chery > CC: Chad Versace > CC: Rafael Antognolli > > Topi Pohjolainen (39): > i965/dbg: Add means for forcing stencil sampling using y-tile

Re: [Mesa-dev] [PATCH] i965: Drop "Destination Element Offset" from Ironlake SGVs.

2017-05-03 Thread Rafael Antognolli
Reviewed-by: Rafael Antognolli On Fri, Apr 28, 2017 at 05:04:05PM -0700, Kenneth Graunke wrote: > The Ironlake documentation is terrible, so it's unclear whether or not > this field exists there. It definitely doesn't exist on Sandybridge > and later. It definitely does ex

Re: [Mesa-dev] [PATCH 08/11] intel/decoder: Fix indentation

2017-05-02 Thread Rafael Antognolli
Reviewed-by: Rafael Antognolli On Mon, May 01, 2017 at 01:54:52PM -0700, Matt Turner wrote: > --- > src/intel/common/gen_decoder.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/src/intel/common/gen_decoder.c b/src/intel/common/gen_decoder.c

Re: [Mesa-dev] [PATCH 05/11] genxml: Remove brackets from kernel start pointer names

2017-05-02 Thread Rafael Antognolli
Patch is Reviewed-by: Rafael Antognolli On Mon, May 01, 2017 at 01:54:49PM -0700, Matt Turner wrote: > Newer Gens' names don't have the brackets. Having common names will make > some later patches simpler. > --- > src/intel/genxml/gen4.xml | 2 +- > src/intel/genxm

[Mesa-dev] [PATCH] i965: Make the field computed_depth_mode an enum.

2017-05-02 Thread Rafael Antognolli
Since the enum is in the same header now, we can use it as the type of the field. Signed-off-by: Rafael Antognolli --- PS: We can merge this with the previous patch too if that's better. src/intel/compiler/brw_compiler.h | 2 +- src/intel/compiler/brw_fs.cpp | 2 +- 2 files chang

[Mesa-dev] [PATCH] i965: Move enums to brw_compiler.h.

2017-05-02 Thread Rafael Antognolli
These enums live inside struct brw_wm_prog_data, so it makes sense to keep them in the same header. It also allows to use them without including brw_eu_defines.h. Signed-off-by: Rafael Antognolli --- src/intel/compiler/brw_compiler.h | 21 + src/intel/compiler

Re: [Mesa-dev] [PATCH v03 12/38] i965: Split out enum from brw_eu_defines.h

2017-05-02 Thread Rafael Antognolli
On Tue, May 02, 2017 at 08:44:05AM -0700, Jason Ekstrand wrote: > On Tue, May 2, 2017 at 8:28 AM, Rafael Antognolli > > wrote: > > On Tue, May 02, 2017 at 07:26:53AM -0700, Jason Ekstrand wrote: > > On Mon, May 1, 2017 at 6:43 PM, Rafael Antognolli < >

Re: [Mesa-dev] [PATCH v03 12/38] i965: Split out enum from brw_eu_defines.h

2017-05-02 Thread Rafael Antognolli
On Tue, May 02, 2017 at 07:26:53AM -0700, Jason Ekstrand wrote: > On Mon, May 1, 2017 at 6:43 PM, Rafael Antognolli > > wrote: > > We need to use some enums inside genX_state_upload.c, but including the > whole header will cause several conflicts between thin

Re: [Mesa-dev] [PATCH v03 12/38] i965: Split out enum from brw_eu_defines.h

2017-05-02 Thread Rafael Antognolli
On Tue, May 02, 2017 at 09:38:53AM +0100, Emil Velikov wrote: > On 2 May 2017 at 09:32, Emil Velikov wrote: > > Hi Rafael, > > > > On 2 May 2017 at 02:43, Rafael Antognolli > > wrote: > >> We need to use some enums inside genX_state_upload.c, but includ

[Mesa-dev] [PATCH v03 37/38] i965: Port gen6+ 3DSTATE_CC_STATE_POINTERS state to genxml.

2017-05-01 Thread Rafael Antognolli
Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 1 +- src/mesa/drivers/dri/i965/brw_state.h | 1 +- src/mesa/drivers/dri/i965/gen6_cc.c | 90 + src/mesa/drivers/dri/i965/genX_state_upload.c | 53 +++- 4 files

[Mesa-dev] [PATCH v03 33/38] i965: Port gen6+ 3DSTATE_SCISSOR_STATE_POINTERS to use genxml.

2017-05-01 Thread Rafael Antognolli
Emit 3DSTATE_SCISSOR_STATE_POINTERS using brw_batch_emit, and pack the scissor states using GENX(SCISSOR_RECT_pack), generated from genxml. v3: - Remove old code (Ken) - Style fixes (Ken) Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH v03 14/38] i965: Move MOCS macros to brw_context.h.

2017-05-01 Thread Rafael Antognolli
nes.h. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_context.h | 41 +- src/mesa/drivers/dri/i965/brw_defines.h | 42 +-- 2 files changed, 41 insertions(+), 42 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_context.h b/src

[Mesa-dev] [PATCH v03 13/38] anv: Use BRW_BARYCENTRIC_NONPERSPECTIVE_BITS from common header.

2017-05-01 Thread Rafael Antognolli
In a previous patch some enums were split out from brw_eu_defines.h, so they could be used by genxml based code. anv can also benefit from this. Signed-off-by: Rafael Antognolli --- src/intel/vulkan/genX_pipeline.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/intel

[Mesa-dev] [PATCH v03 38/38] i965: Port gen4+ state emitting code to genxml.

2017-05-01 Thread Rafael Antognolli
On this patch, we port: - brw_polygon_stipple - brw_polygon_stipple_offset - brw_line_stipple - brw_drawing_rect v2: - Also emit states for gen4-5 with this code. v3: - Style fixes and remove excessive checks (Ken). Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri

[Mesa-dev] [PATCH v03 07/38] genxml: 3DSTATE_VS rename Function Enable to Enable.

2017-05-01 Thread Rafael Antognolli
Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/intel/blorp/blorp_genX_exec.h | 2 +- src/intel/genxml/gen6.xml | 2 +- src/intel/genxml/gen7.xml | 2 +- src/intel/genxml/gen75.xml| 2 +- src/intel/genxml/gen8.xml | 2 +- src/intel/genxml

[Mesa-dev] [PATCH v03 11/38] genxml: Normalize xml for 3DSTATE_CC_STATE_POINTERS.

2017-05-01 Thread Rafael Antognolli
- "COLOR_CALC_STATE Change" -> "Color Calc State Pointer Valid" - "Pointer to COLOR_CALC_STATE" -> "Color Calc State Pointer" - "BackFace" -> "Backface" Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --

[Mesa-dev] [PATCH v03 19/38] i965: Port Gen6+ 3DSTATE_CLIP state to genxml.

2017-05-01 Thread Rafael Antognolli
Emit clip state on Gen6+ using brw_batch_emit helper, using pack structs from genxml. v3: - Lots style fixes (Ken) - Do not set CullTestEnableBitMask on Gen8+ (Ken) Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_state.h | 1 +- src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH v03 31/38] i965: Port gen6+ blend state code to genxml.

2017-05-01 Thread Rafael Antognolli
Upload blend states using GENX(BLEND_STATE_ENTRY_pack), generated from genxml. v3: - style fixes (Ken) - cleanup to remove excessive #ifdef's (Ken) - remove memset (Ken) - disable blend.AlphaToCoverageDitherEnable on gen6 (Ken) Signed-off-by: Rafael Antognolli Reviewed-by: Ke

[Mesa-dev] [PATCH v03 21/38] i965: Add brw_get_line_width_float.

2017-05-01 Thread Rafael Antognolli
That helper function returns the line width as a float, and is then used by brw_get_line_width to return the fixed point width. Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_util.h | 25 ++--- 1 file changed, 14 insertions

[Mesa-dev] [PATCH v03 25/38] i965: Port gen7+ 3DSTATE_SOL to genxml.

2017-05-01 Thread Rafael Antognolli
Emit 3DSTATE_SOL on Gen7+ using brw_batch_emit helper, that uses pack structs from genxml. v2: - Add helpers to assign struct brw_address (Kristian) v3: - Rename MOCS -> SOBufferMOCS - Do not re-declare MOCS macros (Ken). - Style and code reorganization (Ken). Signed-off-by: Raf

[Mesa-dev] [PATCH v03 36/38] i965: Port gen6+ multisample state emitting code to genxml.

2017-05-01 Thread Rafael Antognolli
Emit 3DSTATE_MULTISAMPLE using brw_batch_emit. v3: - Remove dead code (Ken) - Simplify #if/#endif (Ken) Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_context.h| 9 +- src/mesa/drivers/dri/i965/brw_state.h | 2

[Mesa-dev] [PATCH v03 20/38] i965: Port Gen8+ 3DSTATE_RASTER state to genxml.

2017-05-01 Thread Rafael Antognolli
Emits 3DSTATE_RASTER from genX_state_upload.c using pack structs from genxml. v3: - Style fixes (Ken) Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_state.h | 1 +- src/mesa/drivers/dri/i965/gen8_sf_state.c | 125

[Mesa-dev] [PATCH v03 12/38] i965: Split out enum from brw_eu_defines.h

2017-05-01 Thread Rafael Antognolli
-by: Rafael Antognolli Acked-by: Reviewed-by: Kenneth Graunke --- src/intel/Makefile.sources | 1 +- src/intel/compiler/brw_defines_common.h | 46 ++- src/intel/compiler/brw_eu_defines.h | 22 + 3 files changed, 48 insertions(+), 21 deletions

[Mesa-dev] [PATCH v03 17/38] genxml: Add rules to build gen4, gen45 and ge5.

2017-05-01 Thread Rafael Antognolli
Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.am | 12 src/mesa/drivers/dri/i965/Makefile.sources | 9 + src/mesa/drivers/dri/i965/brw_state.h | 1 + 3 files changed, 22 insertions(+) diff --git a/src/mesa/drivers/dri/i965/Makefile.am

[Mesa-dev] [PATCH v03 09/38] genxml: Rename "Function Enable" to "Enable".

2017-05-01 Thread Rafael Antognolli
Rename that field name on genxml for: - 3DSTATE_GS - gen6+ - 3DSTATE_DS - gen7+ - 3DSTATE_HS - gen7+ Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/intel/genxml/gen6.xml| 2 +- src/intel/genxml/gen7.xml| 6 +++--- src/intel/genxml/gen75.xml

[Mesa-dev] [PATCH v03 28/38] i965: Port gen8+ 3DSTATE_PS_EXTRA to genxml.

2017-05-01 Thread Rafael Antognolli
Emit 3DSTATE_PS_EXTRA on Gen8+ using brw_batch_emit helper, that uses pack structs from genxml. v3: - Style fixes and moving code around to be cleaner (Ken) Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/Makefile.sources| 1 +- src/mesa

[Mesa-dev] [PATCH v03 18/38] i965: Port Gen6+ DEPTH_STENCIL state to genxml.

2017-05-01 Thread Rafael Antognolli
From: Kenneth Graunke This emits 3DSTATE_WM_DEPTH_STENCIL on Gen8+ or DEPTH_STENCIL_STATE (and the relevant pointer packets) on Gen6-7.5 from a single function. v3: - Watch for BRW_NEW_BATCH too on gen8+ (Ken) Signed-off-by: Kenneth Graunke Signed-off-by: Rafael Antognolli Reviewed-by

[Mesa-dev] [PATCH v03 26/38] i965: Port gen7+ 3DSTATE_PS to genxml.

2017-05-01 Thread Rafael Antognolli
Emit 3DSTATE_PS on Gen7+ using brw_batch_emit helper, that uses pack structs from genxml. v2: - Use render_bo helper to setup brw_address (Kristian) v3: - Style fixes and code cleanup (Ken) Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH v03 23/38] i965: Port Gen7+ 3DSTATE_SBE state to genxml.

2017-05-01 Thread Rafael Antognolli
Emit 3DSTATE_SBE on Gen7+ using brw_batch_emit helper, that uses pack structs from genxml. v2: - Use ACTIVE_COMPONENT_XYZW from gen9.xml. v3: - Style fixes (Ken) Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/Makefile.sources| 2 +- src

[Mesa-dev] [PATCH v03 22/38] i965: Port gen6+ 3DSTATE_SF to genxml.

2017-05-01 Thread Rafael Antognolli
Emit sf state on Gen6+ using brw_batch_emit helper, using pack structs from genxml. v3: - Reorganize code and reduce #if/#endif's (Ken) - Style fixes (Ken) - Always set AALINEDISTANCE_TRUE (Ken) Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_state.h

[Mesa-dev] [PATCH v03 24/38] i965: Remove calculate_attr_overrides.

2017-05-01 Thread Rafael Antognolli
This function now lives inside genX_state_upload.c. Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/Makefile.sources | 1 +- src/mesa/drivers/dri/i965/brw_state.h | 8 +- src/mesa/drivers/dri/i965/gen6_sf_state.c | 265

[Mesa-dev] [PATCH v03 29/38] i965: Port gen6+ 3DSTATE_VS to genxml.

2017-05-01 Thread Rafael Antognolli
Emit 3DSTATE_VS on Gen6+ using brw_batch_emit helper, that uses pack structs from genxml. v2: - Use render_bo helper to setup brw_address (Kristian) v3: - Bring back some comments for gen6 and remove _NEW_TRANSFORM blocks from gen7+. Signed-off-by: Rafael Antognolli Reviewed-by

[Mesa-dev] [PATCH v03 16/38] i965: Get real per-gen atom lists

2017-05-01 Thread Rafael Antognolli
From: Kenneth Graunke Make atoms initalization compile conditionally based on the target platform. --- src/mesa/drivers/dri/i965/brw_state.h | 12 +- src/mesa/drivers/dri/i965/brw_state_upload.c | 385 +--- src/mesa/drivers/dri/i965/genX_state_upload.c | 340 +++

[Mesa-dev] [PATCH v03 35/38] i965: Port gen4+ emit vertices code to genxml.

2017-05-01 Thread Rafael Antognolli
) - Use macros for MOCS values. - Do not use #ifndef NDEBUG on code that is actually used (Ken) v3: - Style and code clenup (Ken) - Keep some of the common code inside brw_draw_upload.c (Ken) Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_draw_upload.c | 454

[Mesa-dev] [PATCH v03 34/38] i965: Port push constant code to genxml.

2017-05-01 Thread Rafael Antognolli
macro (Ken) Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 4 +- src/mesa/drivers/dri/i965/brw_state.h | 5 +- src/mesa/drivers/dri/i965/gen6_gs_state.c | 33 +--- src/mesa/drivers/dri/i965/gen6_vs_state.c | 70 +-- src/mesa/drivers

[Mesa-dev] [PATCH v03 03/38] genxml: Update xml for 3DSTATE_SF.

2017-05-01 Thread Rafael Antognolli
- Normalize "Anti-Aliasing Enable" - Add "Multisample Rasterization Mode" constants - Rename "Use Point Width on Vertex" to "Vertex" - Rename "Use Point Width from State" to "State" Signed-off-by: Rafael Antognolli Revie

[Mesa-dev] [PATCH v03 30/38] i965: Port gen6+ state emitting code to genxml.

2017-05-01 Thread Rafael Antognolli
Ported in this patch: - 3DSTATE_DS - 3DSTATE_GS - 3DSTATE_HS - 3DSTATE_VIEWPORT_STATE_POINTERS_SF_CL v3: - Remove NEW_TRANSFORM blocks (Ken) - Bring back some comments and workaround for Ivybridge (Ken) Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH v03 27/38] i965: Port gen6+ 3DSTATE_WM to genxml.

2017-05-01 Thread Rafael Antognolli
) Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/Makefile.sources| 1 +- src/mesa/drivers/dri/i965/brw_state.h | 14 +- src/mesa/drivers/dri/i965/gen6_wm_state.c | 221 +--- src/mesa/drivers/dri/i965/gen7_wm_state.c

[Mesa-dev] [PATCH v03 05/38] genxml: Add alias for MOCS.

2017-05-01 Thread Rafael Antognolli
Use an alias, so we can set the same value as the #define's. v3: - Call it "SO Buffer MOCS" to follow the most common naming scheme. - Add alias for gen7 and gen75 too (Ken). Signed-off-by: Rafael Antognolli --- src/intel/genxml/gen7.xml | 1 + src/intel/genxml/gen75

[Mesa-dev] [PATCH v03 15/38] i965: Add genxml related plumbing in a new genX_state_upload.c file.

2017-05-01 Thread Rafael Antognolli
From: Kenneth Graunke v3: - Drop aub parameter (Ken) Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/Makefile.sources| 15 ++- src/mesa/drivers/dri/i965/genX_state_upload.c | 109 +++- 2 files changed, 119 insertions(+), 5 deletions(-) create mode 100644 s

[Mesa-dev] [PATCH v03 02/38] genxml: Rename clip enable property.

2017-05-01 Thread Rafael Antognolli
There are two variants: - Clip Enable - CLIP Enable (on gen6) Rename everything to Clip Enable. Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/intel/genxml/gen4.xml | 2 +- src/intel/genxml/gen45.xml | 2 +- src/intel/genxml/gen5.xml | 2 +- src/intel/genxml

[Mesa-dev] [PATCH v03 08/38] genxml: Clip guardbands are float, not int.

2017-05-01 Thread Rafael Antognolli
This makes genxml create the right struct types, and generate the right batch commands. Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/intel/genxml/gen6.xml | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/intel/genxml/gen6.xml b/src/intel

[Mesa-dev] [PATCH v03 10/38] genxml: Normalize xml for 3DSTATE_MULTISAMPLE.

2017-05-01 Thread Rafael Antognolli
Name the options to "Pixel Location": - PIXLOC_CENTER -> CENTER - PIXLOC_UL_CORNER -> UL_CORNER Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/intel/blorp/blorp_genX_exec.h | 4 +--- src/intel/genxml/gen6.xml | 4 ++-- src/intel/genxml/gen7

[Mesa-dev] [PATCH v03 32/38] i965: Port gen7+ 3DSTATE_TE to genxml.

2017-05-01 Thread Rafael Antognolli
Emit 3DSTATE_TE on Gen7+ using brw_batch_emit helper. Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/Makefile.sources| 1 +- src/mesa/drivers/dri/i965/brw_state.h | 1 +- src/mesa/drivers/dri/i965/gen7_te_state.c | 67

[Mesa-dev] [PATCH v03 06/38] genxml: Make "Reorder Mode" fields consistent.

2017-05-01 Thread Rafael Antognolli
From: Kenneth Graunke Both GS and SOL have these fields. Some were ReorderEnable = true, some were ReorderMode = REORDER_TRAILING, and some were just TRAILING. Signed-off-by: Kenneth Graunke Reviewed-by: Rafael Antognolli --- src/intel/genxml/gen6.xml| 5 - src/intel/genxml/gen7

[Mesa-dev] [PATCH v03 04/38] genxml: Add missing field values to 3DSTATE_SBE.

2017-05-01 Thread Rafael Antognolli
Fill out "Attribute Active Component Format" possible values. Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/intel/genxml/gen9.xml | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/intel/genxml/gen9.xml b/src/intel/genxml/gen9

[Mesa-dev] [PATCH v03 00/38] Rebased and reviewed series to convert state emitting code to genxml.

2017-05-01 Thread Rafael Antognolli
g in a new genX_state_upload.c file. i965: Get real per-gen atom lists i965: Port Gen6+ DEPTH_STENCIL state to genxml. Louis-Francis Ratté-Boulianne (1): genxml: Fill out Gen4, Gen45 and Gen5 XML Rafael Antognolli (33): genxml: Rename clip enable property. genxml: Update xml for 3DSTATE_SF

Re: [Mesa-dev] [PATCH v02 26/37] i965: Port gen6+ 3DSTATE_WM to genxml.

2017-04-27 Thread Rafael Antognolli
On Wed, Apr 26, 2017 at 11:15:45PM -0700, Kenneth Graunke wrote: > On Monday, April 24, 2017 3:19:21 PM PDT Rafael Antognolli wrote: > [snip] > > diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c > > b/src/mesa/drivers/dri/i965/genX_state_upload.c > > index

Re: [Mesa-dev] [PATCH] i965: Set point rasterization rule to UPPER_RIGHT on Gen6-7.5.

2017-04-27 Thread Rafael Antognolli
Makes sense to me. Reviewed-by: Rafael Antognolli On Wed, Apr 26, 2017 at 11:14:47PM -0700, Kenneth Graunke wrote: > Gen4-5 and Gen8+ already set this, but Gen6-7.5 did not. We ought to > be consistent - the answer depends on the API, not the hardware generation. > > The Sandybri

[Mesa-dev] [PATCH v3 06/37] genxml: Add alias for MOCS.

2017-04-25 Thread Rafael Antognolli
Use an alias, so we can set the same value as the #define's. v3: - Call it "SO Buffer MOCS" to follow the most common naming scheme. - Add alias for gen7 and gen75 too (Ken). Signed-off-by: Rafael Antognolli --- src/intel/genxml/gen7.xml | 1 + src/intel/genxml/gen75

Re: [Mesa-dev] [PATCH v02 06/37] genxml: Add alias for MOCS.

2017-04-24 Thread Rafael Antognolli
On Mon, Apr 24, 2017 at 03:59:07PM -0700, Kenneth Graunke wrote: > On Monday, April 24, 2017 3:19:01 PM PDT Rafael Antognolli wrote: > > Use an alias, so we can set the same value as the #define's. > > > > Signed-off-by: Rafael Antognolli > > --- > > sr

[Mesa-dev] [PATCH v02 00/37] Updated series to convert part of the state emitting code to genxml.

2017-04-24 Thread Rafael Antognolli
ml: Make "Reorder Mode" fields consistent. i965: Add genxml related plumbing in a new genX_state_upload.c file. i965: Get real per-gen atom lists i965: Port Gen6+ DEPTH_STENCIL state to genxml. Louis-Francis Ratté-Boulianne (1): genxml: Fill out Gen4, Gen45 and Gen5 XML Rafael Antognoll

[Mesa-dev] [PATCH v02 18/37] i965: Port Gen6+ DEPTH_STENCIL state to genxml.

2017-04-24 Thread Rafael Antognolli
From: Kenneth Graunke This emits 3DSTATE_WM_DEPTH_STENCIL on Gen8+ or DEPTH_STENCIL_STATE (and the relevant pointer packets) on Gen6-7.5 from a single function. Signed-off-by: Kenneth Graunke Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 2 +- src

[Mesa-dev] [PATCH v02 20/37] i965: Port Gen8+ 3DSTATE_RASTER state to genxml.

2017-04-24 Thread Rafael Antognolli
Emits 3DSTATE_RASTER from genX_state_upload.c using pack structs from genxml. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_state.h | 1 +- src/mesa/drivers/dri/i965/gen8_sf_state.c | 125 +--- src/mesa/drivers/dri/i965/genX_state_upload.c

[Mesa-dev] [PATCH v02 02/37] genxml: Fix gen4-5 xml to make it compile correctly.

2017-04-24 Thread Rafael Antognolli
Set the type of some fields, instead of prefix. Also fix the SAMPLER_BORDER_COLOR_STATE fields of gen5.xml. Signed-off-by: Rafael Antognolli --- src/intel/genxml/gen4.xml | 13 +- src/intel/genxml/gen45.xml | 12 - src/intel/genxml/gen5.xml | 52

[Mesa-dev] [PATCH v02 13/37] i965: Split out enum from brw_eu_defines.h

2017-04-24 Thread Rafael Antognolli
-by: Rafael Antognolli --- src/intel/Makefile.sources | 1 +- src/intel/compiler/brw_defines_common.h | 46 ++- src/intel/compiler/brw_eu_defines.h | 22 + 3 files changed, 48 insertions(+), 21 deletions(-) create mode 100644 src/intel

[Mesa-dev] [PATCH v02 17/37] genxml: Add rules to build gen4, gen45 and ge5.

2017-04-24 Thread Rafael Antognolli
Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.am | 12 src/mesa/drivers/dri/i965/Makefile.sources | 9 + src/mesa/drivers/dri/i965/brw_state.h | 1 + 3 files changed, 22 insertions(+) diff --git a/src/mesa/drivers/dri/i965/Makefile.am

[Mesa-dev] [PATCH v02 27/37] i965: Port gen8+ 3DSTATE_PS_EXTRA to genxml.

2017-04-24 Thread Rafael Antognolli
Emit 3DSTATE_PS_EXTRA on Gen8+ using brw_batch_emit helper, that uses pack structs from genxml. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 1 +- src/mesa/drivers/dri/i965/brw_state.h | 10 +- src/mesa/drivers/dri/i965/gen8_ps_state.c

[Mesa-dev] [PATCH v02 21/37] i965: Port gen6+ 3DSTATE_SF to genxml.

2017-04-24 Thread Rafael Antognolli
Emit sf state on Gen6+ using brw_batch_emit helper, using pack structs from genxml. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_state.h | 3 +- src/mesa/drivers/dri/i965/brw_util.h | 25 +- src/mesa/drivers/dri/i965/gen6_sf_state.c | 190

[Mesa-dev] [PATCH v02 34/37] i965: Port gen4+ emit vertices code to genxml.

2017-04-24 Thread Rafael Antognolli
) - Use macros for MOCS values. - Do not use #ifndef NDEBUG on code that is actually used (Ken) Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_context.h |6 +- src/mesa/drivers/dri/i965/brw_draw.h |2 +- src/mesa/drivers/dri/i965/brw_draw_upload.c

[Mesa-dev] [PATCH v02 19/37] i965: Port Gen6+ 3DSTATE_CLIP state to genxml.

2017-04-24 Thread Rafael Antognolli
Emit clip state on Gen6+ using brw_batch_emit helper, using pack structs from genxml. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_state.h | 1 +- src/mesa/drivers/dri/i965/gen6_clip_state.c | 139 +-- src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH v02 37/37] i965: Port gen4+ state emitting code to genxml.

2017-04-24 Thread Rafael Antognolli
On this patch, we port: - brw_polygon_stipple - brw_polygon_stipple_offset - brw_line_stipple - brw_drawing_rect v2: - Also emit states for gen4-5 with this code. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources | 1 +- src/mesa/drivers/dri

[Mesa-dev] [PATCH v02 36/37] i965: Port gen6+ 3DSTATE_CC_STATE_POINTERS state to genxml.

2017-04-24 Thread Rafael Antognolli
Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 1 +- src/mesa/drivers/dri/i965/brw_state.h | 1 +- src/mesa/drivers/dri/i965/gen6_cc.c | 90 + src/mesa/drivers/dri/i965/genX_state_upload.c | 53 +++- 4 files

[Mesa-dev] [PATCH v02 03/37] genxml: Rename clip enable property.

2017-04-24 Thread Rafael Antognolli
There are two variants: - Clip Enable - CLIP Enable (on gen6) Rename everything to Clip Enable. Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/intel/genxml/gen4.xml | 2 +- src/intel/genxml/gen45.xml | 2 +- src/intel/genxml/gen5.xml | 2 +- src/intel/genxml

[Mesa-dev] [PATCH v02 28/37] i965: Port gen6+ 3DSTATE_VS to genxml.

2017-04-24 Thread Rafael Antognolli
Emit 3DSTATE_VS on Gen6+ using brw_batch_emit helper, that uses pack structs from genxml. v2: - Use render_bo helper to setup brw_address (Kristian) Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 2 +- src/mesa/drivers/dri/i965/brw_state.h

[Mesa-dev] [PATCH v02 14/37] anv: Use BRW_BARYCENTRIC_NONPERSPECTIVE_BITS from common header.

2017-04-24 Thread Rafael Antognolli
In a previous patch some enums were split out from brw_eu_defines.h, so they could be used by genxml based code. anv can also benefit from this. Signed-off-by: Rafael Antognolli --- src/intel/vulkan/genX_pipeline.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/intel

[Mesa-dev] [PATCH v02 15/37] i965: Add genxml related plumbing in a new genX_state_upload.c file.

2017-04-24 Thread Rafael Antognolli
From: Kenneth Graunke Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/Makefile.sources| 15 ++- src/mesa/drivers/dri/i965/genX_state_upload.c | 109 +++- 2 files changed, 119 insertions(+), 5 deletions(-) create mode 100644 src/mesa/drivers/dri/i965/genX_state

[Mesa-dev] [PATCH v02 05/37] genxml: Add missing field values to 3DSTATE_SBE.

2017-04-24 Thread Rafael Antognolli
Fill out "Attribute Active Component Format" possible values. Signed-off-by: Rafael Antognolli --- src/intel/genxml/gen9.xml | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/intel/genxml/gen9.xml b/src/intel/genxml/gen9.xml index ee7056b..59daa31 100644

[Mesa-dev] [PATCH v02 06/37] genxml: Add alias for MOCS.

2017-04-24 Thread Rafael Antognolli
Use an alias, so we can set the same value as the #define's. Signed-off-by: Rafael Antognolli --- src/intel/genxml/gen8.xml | 1 + src/intel/genxml/gen9.xml | 1 + 2 files changed, 2 insertions(+) diff --git a/src/intel/genxml/gen8.xml b/src/intel/genxml/gen8.xml index 408d241..2908082 1

[Mesa-dev] [PATCH v02 25/37] i965: Port gen7+ 3DSTATE_PS to genxml.

2017-04-24 Thread Rafael Antognolli
Emit 3DSTATE_PS on Gen7+ using brw_batch_emit helper, that uses pack structs from genxml. v2: - Use render_bo helper to setup brw_address (Kristian) Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_state.h | 2 +- src/mesa/drivers/dri/i965/gen7_wm_state.c

[Mesa-dev] [PATCH v02 22/37] i965: Port Gen7+ 3DSTATE_SBE state to genxml.

2017-04-24 Thread Rafael Antognolli
Emit 3DSTATE_SBE on Gen7+ using brw_batch_emit helper, that uses pack structs from genxml. v2: - Use ACTIVE_COMPONENT_XYZW from gen9.xml. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 2 +- src/mesa/drivers/dri/i965/brw_state.h | 2 +- src

[Mesa-dev] [PATCH v02 26/37] i965: Port gen6+ 3DSTATE_WM to genxml.

2017-04-24 Thread Rafael Antognolli
Emit 3DSTATE_WM on Gen6+ using brw_batch_emit helper, that uses pack structs from genxml. v2: - Use render_bo helper to setup brw_address (Kristian) - Remove TODO and use BRW_PSCDEPTH_OFF. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 1 +- src

[Mesa-dev] [PATCH v02 11/37] genxml: Normalize xml for 3DSTATE_MULTISAMPLE.

2017-04-24 Thread Rafael Antognolli
Name the options to "Pixel Location": - PIXLOC_CENTER -> CENTER - PIXLOC_UL_CORNER -> UL_CORNER Signed-off-by: Rafael Antognolli --- src/intel/blorp/blorp_genX_exec.h | 4 +--- src/intel/genxml/gen6.xml | 4 ++-- src/intel/genxml/gen7.xml | 4 ++-- src/in

[Mesa-dev] [PATCH v02 30/37] i965: Port gen6+ blend state code to genxml.

2017-04-24 Thread Rafael Antognolli
Upload blend states using GENX(BLEND_STATE_ENTRY_pack), generated from genxml. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 1 +- src/mesa/drivers/dri/i965/brw_state.h | 3 +- src/mesa/drivers/dri/i965/gen6_cc.c | 216

[Mesa-dev] [PATCH v02 32/37] i965: Port gen6+ 3DSTATE_SCISSOR_STATE_POINTERS to use genxml.

2017-04-24 Thread Rafael Antognolli
Emit 3DSTATE_SCISSOR_STATE_POINTERS using brw_batch_emit, and pack the scissor states using GENX(SCISSOR_RECT_pack), generated from genxml. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_state.h | 1 +- src/mesa/drivers/dri/i965/genX_state_upload.c | 89

[Mesa-dev] [PATCH v02 04/37] genxml: Update xml for 3DSTATE_SF.

2017-04-24 Thread Rafael Antognolli
- Normalize "Anti-Aliasing Enable" - Add "Multisample Rasterization Mode" constants - Rename "Use Point Width on Vertex" to "Vertex" - Rename "Use Point Width from State" to "State" Signed-off-by: Rafael Antognolli --- src/intel/gen

[Mesa-dev] [PATCH v02 29/37] i965: Port gen6+ state emitting code to genxml.

2017-04-24 Thread Rafael Antognolli
Ported in this patch: - 3DSTATE_DS - 3DSTATE_GS - 3DSTATE_HS - 3DSTATE_VIEWPORT_STATE_POINTERS_SF_CL Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources | 6 +- src/mesa/drivers/dri/i965/brw_state.h | 18 +- src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH v02 33/37] i965: Port push constant code to genxml.

2017-04-24 Thread Rafael Antognolli
The following states are ported on this patch: - gen6_gs_push_constants - gen6_vs_push_constants - gen6_wm_push_constants - gen7_tes_push_constants v2: - Use helper to setup brw_address (Kristian) Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources

[Mesa-dev] [PATCH v02 16/37] i965: Get real per-gen atom lists

2017-04-24 Thread Rafael Antognolli
From: Kenneth Graunke Make atoms initalization compile conditionally based on the target platform. --- src/mesa/drivers/dri/i965/brw_state.h | 12 +- src/mesa/drivers/dri/i965/brw_state_upload.c | 385 +--- src/mesa/drivers/dri/i965/genX_state_upload.c | 340 +++

[Mesa-dev] [PATCH v02 12/37] genxml: Normalize xml for 3DSTATE_CC_STATE_POINTERS.

2017-04-24 Thread Rafael Antognolli
- "COLOR_CALC_STATE Change" -> "Color Calc State Pointer Valid" - "Pointer to COLOR_CALC_STATE" -> "Color Calc State Pointer" - "BackFace" -> "Backface" Signed-off-by: Rafael Antognolli --- src/intel/blorp/blorp_genX_exe

[Mesa-dev] [PATCH v02 10/37] genxml: Rename "Function Enable" to "Enable".

2017-04-24 Thread Rafael Antognolli
Rename that field name on genxml for: - 3DSTATE_GS - gen6+ - 3DSTATE_DS - gen7+ - 3DSTATE_HS - gen7+ Signed-off-by: Rafael Antognolli --- src/intel/genxml/gen6.xml| 2 +- src/intel/genxml/gen7.xml| 6 +++--- src/intel/genxml/gen75.xml | 6 +++--- src/intel/genxml

[Mesa-dev] [PATCH v02 08/37] genxml: 3DSTATE_VS rename Function Enable to Enable.

2017-04-24 Thread Rafael Antognolli
Signed-off-by: Rafael Antognolli --- src/intel/blorp/blorp_genX_exec.h | 2 +- src/intel/genxml/gen6.xml | 2 +- src/intel/genxml/gen7.xml | 2 +- src/intel/genxml/gen75.xml| 2 +- src/intel/genxml/gen8.xml | 2 +- src/intel/genxml/gen9.xml | 2 +- src

[Mesa-dev] [PATCH v02 09/37] genxml: Clip guardbands are float, not int.

2017-04-24 Thread Rafael Antognolli
This makes genxml create the right struct types, and generate the right batch commands. Signed-off-by: Rafael Antognolli Reviewed-by: Kenneth Graunke --- src/intel/genxml/gen6.xml | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/intel/genxml/gen6.xml b/src/intel

[Mesa-dev] [PATCH v02 31/37] i965: Port gen7+ 3DSTATE_TE to genxml.

2017-04-24 Thread Rafael Antognolli
Emit 3DSTATE_TE on Gen7+ using brw_batch_emit helper. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 1 +- src/mesa/drivers/dri/i965/brw_state.h | 1 +- src/mesa/drivers/dri/i965/gen7_te_state.c | 67 + src/mesa/drivers/dri

[Mesa-dev] [PATCH v02 35/37] i965: Port gen6+ multisample state emitting code to genxml.

2017-04-24 Thread Rafael Antognolli
Emit 3DSTATE_MULTISAMPLE using brw_batch_emit. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_context.h| 9 +- src/mesa/drivers/dri/i965/brw_state.h | 2 +- src/mesa/drivers/dri/i965/gen6_multisample_state.c | 6 +- src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH v02 07/37] genxml: Make "Reorder Mode" fields consistent.

2017-04-24 Thread Rafael Antognolli
From: Kenneth Graunke Both GS and SOL have these fields. Some were ReorderEnable = true, some were ReorderMode = REORDER_TRAILING, and some were just TRAILING. Signed-off-by: Kenneth Graunke --- src/intel/genxml/gen6.xml| 5 - src/intel/genxml/gen7.xml| 5 - src/intel/

[Mesa-dev] [PATCH v02 23/37] i965: Remove calculate_attr_overrides.

2017-04-24 Thread Rafael Antognolli
This function now lives inside genX_state_upload.c. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources | 1 +- src/mesa/drivers/dri/i965/brw_state.h | 8 +- src/mesa/drivers/dri/i965/gen6_sf_state.c | 265 +-- 3 files changed, 274

[Mesa-dev] [PATCH v02 24/37] i965: Port gen7+ 3DSTATE_SOL to genxml.

2017-04-24 Thread Rafael Antognolli
Emit 3DSTATE_SOL on Gen7+ using brw_batch_emit helper, that uses pack structs from genxml. v2: - Add helpers to assign struct brw_address (Kristian) Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources| 1 +- src/mesa/drivers/dri/i965/brw_state.h

Re: [Mesa-dev] [PATCH 20/35] genxml: Make "Reorder Mode" fields consistent.

2017-04-24 Thread Rafael Antognolli
On Mon, Apr 24, 2017 at 03:03:56PM -0700, Kenneth Graunke wrote: > On Wednesday, April 19, 2017 4:56:13 PM PDT Rafael Antognolli wrote: > > From: Kenneth Graunke > > > > Both GS and SOL have these fields. Some were ReorderEnable = true, > > some were ReorderMode =

Re: [Mesa-dev] [PATCH 22/35] i965: Port gen6+ 3DSTATE_VS to genxml.

2017-04-21 Thread Rafael Antognolli
On Thu, Apr 20, 2017 at 09:55:56AM -0700, Kristian H. Kristensen wrote: > Rafael Antognolli writes: > > > Emit 3DSTATE_VS on Gen6+ using brw_batch_emit helper, that uses pack > > structs from genxml. > > > > Signed-off-by: Rafael Antognolli > >

[Mesa-dev] [PATCH 32/35] i965: Port gen6+ multisample state emitting code to genxml.

2017-04-19 Thread Rafael Antognolli
Emit 3DSTATE_MULTISAMPLE using brw_batch_emit. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_context.h| 9 +- src/mesa/drivers/dri/i965/brw_state.h | 2 +- src/mesa/drivers/dri/i965/gen6_multisample_state.c | 6 +- src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH 35/35] i965: Port gen6+ state emitting code to genxml.

2017-04-19 Thread Rafael Antognolli
On this patch, we port: - brw_polygon_stipple - brw_polygon_stipple_offset - brw_line_stipple - brw_drawing_rect The original code is still left behind because it is being used by gen4-5. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/Makefile.sources | 1

[Mesa-dev] [PATCH 21/35] genxml: 3DSTATE_VS rename Function Enable to Enable.

2017-04-19 Thread Rafael Antognolli
Signed-off-by: Rafael Antognolli --- src/intel/blorp/blorp_genX_exec.h | 2 +- src/intel/genxml/gen6.xml | 2 +- src/intel/genxml/gen7.xml | 2 +- src/intel/genxml/gen75.xml| 2 +- src/intel/genxml/gen8.xml | 2 +- src/intel/genxml/gen9.xml | 2 +- src

[Mesa-dev] [PATCH 10/35] i965: Port Gen8+ 3DSTATE_RASTER state to genxml.

2017-04-19 Thread Rafael Antognolli
Emits 3DSTATE_RASTER from genX_state_upload.c using pack structs from genxml. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_state.h | 1 +- src/mesa/drivers/dri/i965/gen8_sf_state.c | 125 +--- src/mesa/drivers/dri/i965/genX_state_upload.c

<    1   2   3   4   5   6   >