[Mesa-dev] [PATCH 00/31] i965: Separate blorp from the rest of the driver

2016-08-19 Thread Jason Ekstrand
This little patch series is even more blorp code churn. The end objective is in patch 31 which pulls blorp into its own directory completely separate from the i965 dri driver. Jason Ekstrand (31): i965/blorp: Add a blorp_context struct and init/finish funcs i965/blorp: Add an internal shader

[Mesa-dev] [PATCH 02/31] i965/blorp: Add an internal shader cache

2016-08-19 Thread Jason Ekstrand
The main shader cache in the i965 driver is just used as the backing storage and all of the actual caching is done internally to blorp. --- src/mesa/drivers/dri/i965/blorp.c | 123 src/mesa/drivers/dri/i965/blorp.h | 9 +++

[Mesa-dev] [PATCH 06/31] i965/blorp: Use gen6_upload_urb

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/genX_blorp_exec.c b/src/mesa/drivers/dri/i965/genX_blorp_exec.c index a6ac7b0..402ae3f 100644 --- a/src/mesa/drivers/dri/i965/genX_blorp_exec.c +++

[Mesa-dev] [PATCH 03/31] i965/blorp/genX: Add helpers for allocating various bits of state

2016-08-19 Thread Jason Ekstrand
This pulls most of the brw-specific bits into helpers with generic names. Later, those will become the driver hooks for generic code. --- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 85 ++--- 1 file changed, 53 insertions(+), 32 deletions(-) diff --git

[Mesa-dev] [PATCH 05/31] i965/gen6: Refactor gen6_upload_urb

2016-08-19 Thread Jason Ekstrand
This splits it into two functions very similar to gen7_upload_urb. --- src/mesa/drivers/dri/i965/brw_context.h | 3 ++ src/mesa/drivers/dri/i965/gen6_urb.c| 56 +++-- 2 files changed, 35 insertions(+), 24 deletions(-) diff --git

[Mesa-dev] [PATCH 08/31] i965/blorp: Add driver mocs settings to the context

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 11 +-- src/mesa/drivers/dri/i965/blorp.h | 6 ++ src/mesa/drivers/dri/i965/brw_blorp.c | 26 ++ src/mesa/drivers/dri/i965/genX_blorp_exec.c | 14 ++ 4 files changed, 39

[Mesa-dev] [PATCH 01/31] i965/blorp: Add a blorp_context struct and init/finish funcs

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 14 ++ src/mesa/drivers/dri/i965/blorp.h | 10 ++ src/mesa/drivers/dri/i965/brw_blorp.c | 6 ++ src/mesa/drivers/dri/i965/brw_blorp.h | 2 ++ src/mesa/drivers/dri/i965/brw_context.c | 7 +++

[Mesa-dev] [PATCH] program_resource: subroutine active uniforms should return NumSubroutineUniforms

2016-08-19 Thread Alejandro Piñeiro
Before this commit, GetProgramInterfaceiv for pname ACTIVE_RESOURCES and all the _SUBROUTINE_UNIFORM programInterface were returning the count of resources on the shader program using that interface, instead of the number of active uniform resources. This would get a wrong value (for example) if

Re: [Mesa-dev] XDC 2016 : Call for paper

2016-08-19 Thread Rob Clark
On Thu, May 12, 2016 at 6:56 PM, Martin Peres wrote: > Hello, > > I have the pleasure to announce that the X.org Developer Conference 2016 > will be held in Helsinki from September 21 to September 23. The venue is > located at Haaga-Helia university[0], next to the Pasila

[Mesa-dev] [PATCH 24/31] i965: Move the hiz_op enum to blorp

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 3 +-- src/mesa/drivers/dri/i965/blorp.h | 20 ++-- src/mesa/drivers/dri/i965/blorp_priv.h| 2 +- src/mesa/drivers/dri/i965/brw_blorp.c | 12 ++-- src/mesa/drivers/dri/i965/brw_blorp.h

[Mesa-dev] [PATCH 22/31] i965/blorp: Make blorp_addres::buffer a void*

2016-08-19 Thread Jason Ekstrand
The Vulkan driver doesn't use libdrm so we don't want to bake that in. --- src/mesa/drivers/dri/i965/blorp.h | 3 +-- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 5 +++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/blorp.h

[Mesa-dev] [PATCH 16/31] i965/blorp: Pull the guts of blorp_exec into a driver-agnostic header

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/Makefile.sources | 15 +- src/mesa/drivers/dri/i965/blorp_priv.h |2 +- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 1113 +- src/mesa/drivers/dri/i965/genX_blorp_exec.h | 1121 +++ 4 files changed, 1133

[Mesa-dev] [PATCH 11/31] i965/blorp/genX: Add a blorp_surface_reloc helper

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 38 - 1 file changed, 21 insertions(+), 17 deletions(-) diff --git a/src/mesa/drivers/dri/i965/genX_blorp_exec.c b/src/mesa/drivers/dri/i965/genX_blorp_exec.c index 32a7445..f226255 100644 ---

[Mesa-dev] [PATCH 17/31] i965/blorp: Move the guts of brw_blorp_exec into genX_blorp_exec.c

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 66 - src/mesa/drivers/dri/i965/genX_blorp_exec.c | 66 + src/mesa/drivers/dri/i965/genX_blorp_exec.h | 8 ++-- 3 files changed, 70 insertions(+), 70 deletions(-) diff --git

[Mesa-dev] [PATCH 21/31] i965/blorp: Get rid of brw_context

2016-08-19 Thread Jason Ekstrand
This commit switches all of blorp from taking a brw_context to taking a blorp_context and, where useful, a void *batch. In the GL driver, we only have one active batch at a time so the brw_context *is* the batch but in Vulkan, batch will point to the anv_cmd_buffer in which we are building

[Mesa-dev] [PATCH 15/31] i965/blorp/exec: Refactor to use blorp_context and a void *batch

2016-08-19 Thread Jason Ekstrand
This gets rid of brw_context throughout the core of the state setup code. --- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 332 +++- 1 file changed, 182 insertions(+), 150 deletions(-) diff --git a/src/mesa/drivers/dri/i965/genX_blorp_exec.c

[Mesa-dev] [PATCH 13/31] i965/blorp: Use BT_INDEX enums for setting up the binding table

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/blorp.c b/src/mesa/drivers/dri/i965/blorp.c index 2a488c2..dba3441 100644 --- a/src/mesa/drivers/dri/i965/blorp.c +++ b/src/mesa/drivers/dri/i965/blorp.c @@

[Mesa-dev] [PATCH 09/31] i965/blorp: Pull emit_surface_state into genX_blorp_exec.c

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 75 - src/mesa/drivers/dri/i965/blorp_priv.h | 6 -- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 85 +++-- 3 files changed, 80 insertions(+), 86 deletions(-) diff --git

[Mesa-dev] [PATCH 10/31] i965/blorp: Use blorp_address in brw_blorp_surface instead of bo+offset

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 6 +-- src/mesa/drivers/dri/i965/blorp.h | 13 -- src/mesa/drivers/dri/i965/blorp_blit.c | 2 +- src/mesa/drivers/dri/i965/blorp_priv.h | 6 +-- src/mesa/drivers/dri/i965/brw_blorp.c | 39 +++--

[Mesa-dev] [PATCH 14/31] i965/blorp: Add a helper for allocating binding tables and surface states

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 64 +++-- 1 file changed, 42 insertions(+), 22 deletions(-) diff --git a/src/mesa/drivers/dri/i965/genX_blorp_exec.c b/src/mesa/drivers/dri/i965/genX_blorp_exec.c index 6461af7..288e384 100644 ---

[Mesa-dev] [PATCH 20/31] i965/blorp: Take a blorp_context in compile_nir_shader

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 7 --- src/mesa/drivers/dri/i965/blorp.h | 2 ++ src/mesa/drivers/dri/i965/blorp_blit.c | 2 +- src/mesa/drivers/dri/i965/blorp_clear.c | 2 +- src/mesa/drivers/dri/i965/blorp_priv.h | 2 +- src/mesa/drivers/dri/i965/brw_blorp.c | 2 ++

[Mesa-dev] [PATCH 23/31] i965/blorp: Add a fast_clear_op enum

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 1 - src/mesa/drivers/dri/i965/blorp_clear.c | 15 ++- src/mesa/drivers/dri/i965/blorp_priv.h | 12 src/mesa/drivers/dri/i965/genX_blorp_exec.h | 20 ++-- 4 files changed, 32 insertions(+), 16

[Mesa-dev] [PATCH 19/31] i965/meta_util: Take an isl_device in get_fast_clear_rect

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp_clear.c | 4 ++-- src/mesa/drivers/dri/i965/brw_meta_util.c | 4 ++-- src/mesa/drivers/dri/i965/brw_meta_util.h | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/mesa/drivers/dri/i965/blorp_clear.c

[Mesa-dev] [PATCH 25/31] i965/blorp: Get rid of most brw and mesa includes

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 3 --- src/mesa/drivers/dri/i965/blorp_blit.c | 7 --- src/mesa/drivers/dri/i965/blorp_clear.c | 11 +-- src/mesa/drivers/dri/i965/blorp_priv.h | 1 + 4 files changed, 2 insertions(+), 20 deletions(-) diff --git

[Mesa-dev] [PATCH 07/31] i965/blorp/genX: Move emit_urb_config into another helper

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 30 - 1 file changed, 17 insertions(+), 13 deletions(-) diff --git a/src/mesa/drivers/dri/i965/genX_blorp_exec.c b/src/mesa/drivers/dri/i965/genX_blorp_exec.c index 402ae3f..7e09299 100644 ---

[Mesa-dev] [PATCH 12/31] i965/blorp: Shorten binding table index enum names

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp_priv.h | 6 +++--- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/mesa/drivers/dri/i965/blorp_priv.h b/src/mesa/drivers/dri/i965/blorp_priv.h index 15e4335..977f54d 100644 ---

[Mesa-dev] [PATCH 04/31] i965/blorp/genX: Pull emit_3dstate_multisample into a helper

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/genX_blorp_exec.c | 16 +++- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/src/mesa/drivers/dri/i965/genX_blorp_exec.c b/src/mesa/drivers/dri/i965/genX_blorp_exec.c index 13c10ab..a6ac7b0 100644 ---

[Mesa-dev] [PATCH 18/31] i965/blorp: Add an "exec" function pointer to blorp_context

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 27 +-- src/mesa/drivers/dri/i965/blorp.h | 4 src/mesa/drivers/dri/i965/blorp_blit.c | 2 +- src/mesa/drivers/dri/i965/blorp_clear.c | 6 +++--- src/mesa/drivers/dri/i965/blorp_priv.h | 21

Re: [Mesa-dev] XDC 2016 : Call for paper

2016-08-19 Thread Rob Clark
On Thu, May 12, 2016 at 6:56 PM, Martin Peres wrote: > Hello, > > I have the pleasure to announce that the X.org Developer Conference 2016 > will be held in Helsinki from September 21 to September 23. The venue is > located at Haaga-Helia university[0], next to the Pasila

[Mesa-dev] [PATCH 28/31] i965: Move the type_size function declartaions to brw_nir.h

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/brw_nir.h| 9 +++-- src/mesa/drivers/dri/i965/brw_shader.h | 6 -- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_nir.h b/src/mesa/drivers/dri/i965/brw_nir.h index 6185310..8e73c64 100644 ---

[Mesa-dev] [PATCH 29/31] i965/blorp: Use isl_format_get_depth_format for setting depth formats

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 15 +-- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/src/mesa/drivers/dri/i965/blorp.c b/src/mesa/drivers/dri/i965/blorp.c index ffc9c7c..1b00ccc 100644 --- a/src/mesa/drivers/dri/i965/blorp.c +++

[Mesa-dev] [PATCH 27/31] i965: Move get_fast_clear_rect to blorp_clear.c

2016-08-19 Thread Jason Ekstrand
This has been the only caller since we deleted the meta fast clear code. --- src/mesa/drivers/dri/i965/blorp_clear.c | 127 +- src/mesa/drivers/dri/i965/brw_meta_util.c | 122 src/mesa/drivers/dri/i965/brw_meta_util.h | 6 -- 3 files

[Mesa-dev] [PATCH 30/31] i965/blorp: Remove the remaining brw prefixes from the blorp.h API

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp.c | 10 ++--- src/mesa/drivers/dri/i965/blorp.h | 40 +- src/mesa/drivers/dri/i965/blorp_blit.c | 30 +++--- src/mesa/drivers/dri/i965/blorp_clear.c | 32 +++

[Mesa-dev] [PATCH 31/31] i965: Move blorp into src/intel/blorp

2016-08-19 Thread Jason Ekstrand
At this point, blorp is completely driver agnostic and can be safely moved into its own folder. Soon, we hope to start using it for doing blits in the Vulkan driver. --- configure.ac | 1 + src/intel/Makefile.am | 2 +-

[Mesa-dev] [PATCH 26/31] i965: Roll brw_get_ccs_resolve_rect into blorp_ccs_resolve

2016-08-19 Thread Jason Ekstrand
--- src/mesa/drivers/dri/i965/blorp_clear.c | 32 --- src/mesa/drivers/dri/i965/brw_meta_util.c | 36 --- src/mesa/drivers/dri/i965/brw_meta_util.h | 6 -- 3 files changed, 29 insertions(+), 45 deletions(-) diff --git

Re: [Mesa-dev] [PATCH] anv: Give the installed intel_icd.json file an absolute path

2016-08-19 Thread Eric Engestrom
On Fri, Aug 19, 2016 at 09:04:14AM -0700, Jason Ekstrand wrote: > Not providing a path allows the ICD to work on multi-arch systems but > breaks it if you install anywhere other than /usr/lib. Given that users > may be installing locally in .local or similar, we probably do want to > provide a

Re: [Mesa-dev] [PATCH] vbo: Correctly handle attribute offsets in dlist draw.

2016-08-19 Thread Erik Faye-Lund
On Thu, Aug 18, 2016 at 7:50 PM, wrote: > From: Mathias Fröhlich > > Hi, > > I found the below while fixing a similar problem lately in > the immediate mode glBegin/glEnd code path. > > Please review > Thanks > > Mathias > > > > When

[Mesa-dev] [PATCH] loader/dri3: Overhaul dri3_update_num_back

2016-08-19 Thread Michel Dänzer
From: Michel Dänzer Always use 3 buffers when flipping. With only 2 buffers, we have to wait for a flip to complete (which takes non-0 time even with asynchronous flips) before we can start working on the next frame. We were previously only using 2 buffers for flipping if

Re: [Mesa-dev] [PATCH] Return corresponding offset of EGLImage instead of 0.

2016-08-19 Thread Dave Airlie
On 19 August 2016 at 17:24, Weng, Chuanbo wrote: > Thanks Dave for your suggestion! > I'll refine the patch. One question: Is there any standard process to bump > version on this extension, or I just need > to add something to docs/specs/MESA_image_dma_buf_export.txt ?

Re: [Mesa-dev] [PATCH] loader/dri3: Overhaul dri3_update_num_back

2016-08-19 Thread Frank Binns
On 19/08/16 08:07, Michel Dänzer wrote: From: Michel Dänzer Always use 3 buffers when flipping. With only 2 buffers, we have to wait for a flip to complete (which takes non-0 time even with asynchronous flips) before we can start working on the next frame. We were

Re: [Mesa-dev] [PATCH] vbo: Correctly handle attribute offsets in dlist draw.

2016-08-19 Thread Mathias Fröhlich
On Friday, 19 August 2016 09:20:41 CEST Erik Faye-Lund wrote: > On Thu, Aug 18, 2016 at 7:50 PM, wrote: > > From: Mathias Fröhlich > > > > Hi, > > > > I found the below while fixing a similar problem lately in > > the immediate mode

Re: [Mesa-dev] [PATCH 33/95] i965/vec4: implement d2b

2016-08-19 Thread Iago Toral
On Fri, 2016-08-19 at 08:50 +0200, Iago Toral wrote: > On Thu, 2016-08-18 at 12:08 -0700, Francisco Jerez wrote: > > > > Iago Toral writes: > > > > > > > > > > > On Wed, 2016-08-17 at 15:15 -0700, Francisco Jerez wrote: > > > > > > > > > > > > Iago Toral

Re: [Mesa-dev] [PATCH] Return corresponding offset of EGLImage instead of 0.

2016-08-19 Thread Weng, Chuanbo
Thanks Dave for your suggestion! I'll refine the patch. One question: Is there any standard process to bump version on this extension, or I just need to add something to docs/specs/MESA_image_dma_buf_export.txt ? -Original Message- From: Dave Airlie [mailto:airl...@gmail.com] Sent:

Re: [Mesa-dev] [PATCH] Return corresponding offset of EGLImage instead of 0.

2016-08-19 Thread Weng, Chuanbo
I think I have understood what you mean. Thanks! I will send out patch v2 based on your suggestion later. Please help to review it. Thanks. -Original Message- From: Dave Airlie [mailto:airl...@gmail.com] Sent: Friday, August 19, 2016 3:31 PM To: Weng, Chuanbo

Re: [Mesa-dev] [PATCH] i965/vec4: Ignore swizzle of VGRF for use by var_range_end().

2016-08-19 Thread Kenneth Graunke
On Thursday, August 18, 2016 10:23:01 PM PDT Matt Turner wrote: > var_range_end(v, n) loops over the n components of variable number v and > finds the maximum value, giving the last use of any component of v. > Therefore it expects v to correspond to the variable associated with the > .x channel

[Mesa-dev] [PATCH] i965: fix compilation warnings in blorp

2016-08-19 Thread Tapani Pälli
(warning: ‘surftype’ may be used uninitialized ...) Signed-off-by: Tapani Pälli --- src/mesa/drivers/dri/i965/gen6_blorp.c | 2 ++ src/mesa/drivers/dri/i965/gen7_blorp.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/src/mesa/drivers/dri/i965/gen6_blorp.c

Re: [Mesa-dev] [PATCH] i965: fix compilation warnings in blorp

2016-08-19 Thread Jason Ekstrand
On Fri, Aug 19, 2016 at 12:51 AM, Iago Toral wrote: > Reviewed-by: Iago Toral Quiroga > > On Fri, 2016-08-19 at 10:39 +0300, Tapani Pälli wrote: > > (warning: ‘surftype’ may be used uninitialized ...) > > > > Signed-off-by: Tapani Pälli

Re: [Mesa-dev] [PATCH] vbo: Correctly handle attribute offsets in dlist draw.

2016-08-19 Thread Erik Faye-Lund
On Fri, Aug 19, 2016 at 10:38 AM, Mathias Fröhlich wrote: > On Friday, 19 August 2016 09:20:41 CEST Erik Faye-Lund wrote: >> On Thu, Aug 18, 2016 at 7:50 PM, wrote: >> > From: Mathias Fröhlich >> > >> > Hi, >> > >>

[Mesa-dev] [PATCH 3/4] winsys/amdgpu: disable IB chaining on SI

2016-08-19 Thread Marek Olšák
From: Marek Olšák --- src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c index bf3e306..e246f81 100644 ---

[Mesa-dev] [PATCH 4/4] radeonsi: disable CE on SI + AMDGPU

2016-08-19 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_pipe.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_pipe.c b/src/gallium/drivers/radeonsi/si_pipe.c index 2669c7b..ff1a0bc 100644 ---

[Mesa-dev] [PATCH 1/4] winsys/amdgpu: initial SI support

2016-08-19 Thread Marek Olšák
From: Ronie Salgado --- src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 18 ++ src/gallium/winsys/amdgpu/drm/amdgpu_surface.c | 9 +++-- src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 25 + 3 files changed, 46 insertions(+), 6

[Mesa-dev] [PATCH 2/4] winsys/amdgpu: finish up SI addrlib integration

2016-08-19 Thread Marek Olšák
From: Marek Olšák --- src/gallium/winsys/amdgpu/drm/amdgpu_surface.c | 30 +- 1 file changed, 25 insertions(+), 5 deletions(-) diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_surface.c b/src/gallium/winsys/amdgpu/drm/amdgpu_surface.c index

[Mesa-dev] [PATCH 0/4] SI amdgpu Mesa support

2016-08-19 Thread Marek Olšák
Hi, These patches add SI amdgpu support to Mesa. I intend to merge them before the kernel support lands. I hope that's OK. I've not tested piglit, but Ubuntu desktop and some 3D apps seem to work well. Thanks to Ronie Salgado for starting this and other people who contributed to the SI amdgpu

Re: [Mesa-dev] [PATCH 43/95] i965/disasm: print NibCtrl for instructions with execsize 4

2016-08-19 Thread Iago Toral
On Thu, 2016-08-18 at 16:16 -0700, Francisco Jerez wrote: > Iago Toral writes: > > > > > On Mon, 2016-08-08 at 15:58 -0700, Francisco Jerez wrote: > > > > > > Iago Toral Quiroga writes: > > > > > > > > > > > > > > > --- > > > >  

Re: [Mesa-dev] [PATCH 33/95] i965/vec4: implement d2b

2016-08-19 Thread Iago Toral
On Thu, 2016-08-18 at 12:08 -0700, Francisco Jerez wrote: > Iago Toral writes: > > > > > On Wed, 2016-08-17 at 15:15 -0700, Francisco Jerez wrote: > > > > > > Iago Toral writes: > > > > > > > > > > > > > > > On Tue, 2016-08-02 at 18:27 -0700, Francisco

Re: [Mesa-dev] [PATCH] i965: fix compilation warnings in blorp

2016-08-19 Thread Iago Toral
Reviewed-by: Iago Toral Quiroga On Fri, 2016-08-19 at 10:39 +0300, Tapani Pälli wrote: > (warning: ‘surftype’ may be used uninitialized ...) > > Signed-off-by: Tapani Pälli > --- >  src/mesa/drivers/dri/i965/gen6_blorp.c | 2 ++ >  

Re: [Mesa-dev] [PATCH 24/32] i965/blorp/gen6: Use genxml packing structs for state setup

2016-08-19 Thread Jason Ekstrand
On Thu, Aug 18, 2016 at 6:27 AM, Pohjolainen, Topi < topi.pohjolai...@intel.com> wrote: > On Thu, Aug 11, 2016 at 02:15:21PM -0700, Jason Ekstrand wrote: > > --- > > src/mesa/drivers/dri/i965/Makefile.am | 1 + > > src/mesa/drivers/dri/i965/gen6_blorp.c | 685 >

Re: [Mesa-dev] [PATCH] i965/sched: Simplify work done by add_barrier_deps().

2016-08-19 Thread Kenneth Graunke
On Thursday, August 18, 2016 5:17:43 PM PDT Matt Turner wrote: > Scheduling barriers are implemented by placing a dependence on every > node before and after the barrier. This is unnecessary as we can limit > the number of nodes we place dependencies on to those between us and the > next barrier

[Mesa-dev] [Bug 97260] [bisected] R9 290 low performance in Linux 4.7

2016-08-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97260 Michel Dänzer changed: What|Removed |Added Version|unspecified |git

[Mesa-dev] [PATCH v2 1/2] glsl: fix key used for hashing switch statement cases

2016-08-19 Thread Tapani Pälli
Implementation previously used value itself as the key, however after hash implementation change by ee02a5e we cannot use 0 as key. v2: use constant pointer as the key and implement comparison for contents (Eric Anholt) Signed-off-by: Tapani Pälli Bugzilla:

[Mesa-dev] [PATCH v2 2/2] util: add assert that key cannot be NULL on insertion

2016-08-19 Thread Tapani Pälli
Signed-off-by: Tapani Pälli --- src/util/hash_table.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/util/hash_table.c b/src/util/hash_table.c index 4cfe3d9..9e643af 100644 --- a/src/util/hash_table.c +++ b/src/util/hash_table.c @@ -295,6 +295,8 @@

Re: [Mesa-dev] [PATCH] i965: fix compilation warnings in blorp

2016-08-19 Thread Tapani Pälli
On 08/19/2016 11:34 AM, Jason Ekstrand wrote: On Fri, Aug 19, 2016 at 12:51 AM, Iago Toral > wrote: Reviewed-by: Iago Toral Quiroga > On Fri, 2016-08-19 at 10:39 +0300, Tapani Pälli wrote:

Re: [Mesa-dev] [PATCH v3] st/vdpau: change the order in which filters are applied(v3)

2016-08-19 Thread Christian König
Hi Nayan, double check the width/height of the surface and also how many you got with a debugger. Actually thinking more about it it could be that top/bottom fields are separated and so you got width/2 and width/2/4 compared to the video mixer. Maybe try your idea instead of caching the

Re: [Mesa-dev] [PATCH v2 2/2] util: add assert that key cannot be NULL on insertion

2016-08-19 Thread Jason Ekstrand
Seems reasonable, Reviewed-by: Jason Ekstrand On Fri, Aug 19, 2016 at 5:02 AM, Tapani Pälli wrote: > Signed-off-by: Tapani Pälli > --- > src/util/hash_table.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git

Re: [Mesa-dev] [PATCH] vbo: Correctly handle attribute offsets in dlist draw.

2016-08-19 Thread Mathias Fröhlich
Hi, On Friday, 19 August 2016 19:47:13 CEST Timothy Arceri wrote: > On Fri, 2016-08-19 at 10:52 +0200, Erik Faye-Lund wrote: > > On Fri, Aug 19, 2016 at 10:38 AM, Mathias Fröhlich > > wrote: > > > > > > On Friday, 19 August 2016 09:20:41 CEST Erik Faye-Lund wrote: > >

Re: [Mesa-dev] How to build glu lib

2016-08-19 Thread Brian Paul
On 08/18/2016 08:17 AM, Jian wrote: Dear Gurus, I have downloaded the latest source code of glu lib. I have no idea how to build it in VS 2015. There is no make file in the folder. Could you please give me some advice? BTW, I have already build latest mesa lib. The demo example works fine

Re: [Mesa-dev] [PATCH] i965: fix compilation warnings in blorp

2016-08-19 Thread Jason Ekstrand
On Fri, Aug 19, 2016 at 3:21 AM, Tapani Pälli wrote: > > > On 08/19/2016 11:34 AM, Jason Ekstrand wrote: > >> >> >> On Fri, Aug 19, 2016 at 12:51 AM, Iago Toral > > wrote: >> >> Reviewed-by: Iago Toral Quiroga

Re: [Mesa-dev] [PATCH] vbo: Correctly handle attribute offsets in dlist draw.

2016-08-19 Thread Brian Paul
On 08/19/2016 02:52 AM, Erik Faye-Lund wrote: On Fri, Aug 19, 2016 at 10:38 AM, Mathias Fröhlich wrote: On Friday, 19 August 2016 09:20:41 CEST Erik Faye-Lund wrote: On Thu, Aug 18, 2016 at 7:50 PM, wrote: From: Mathias Fröhlich

Re: [Mesa-dev] [PATCH] vbo: Correctly handle attribute offsets in dlist draw.

2016-08-19 Thread Timothy Arceri
On Fri, 2016-08-19 at 10:52 +0200, Erik Faye-Lund wrote: > On Fri, Aug 19, 2016 at 10:38 AM, Mathias Fröhlich > wrote: > > > > On Friday, 19 August 2016 09:20:41 CEST Erik Faye-Lund wrote: > > > > > > On Thu, Aug 18, 2016 at 7:50 PM,   > > >

Re: [Mesa-dev] [PATCH] loader/dri3: Overhaul dri3_update_num_back

2016-08-19 Thread Keith Packard
Michel Dänzer writes: > From: Michel Dänzer > > Always use 3 buffers when flipping. With only 2 buffers, we have to wait > for a flip to complete (which takes non-0 time even with asynchronous > flips) before we can start working on the next frame. We

[Mesa-dev] [PATCH] anv: Give the installed intel_icd.json file an absolute path

2016-08-19 Thread Jason Ekstrand
Not providing a path allows the ICD to work on multi-arch systems but breaks it if you install anywhere other than /usr/lib. Given that users may be installing locally in .local or similar, we probably do want to provide a filename. Distros can carry a revert of this commit if they want an

Re: [Mesa-dev] [PATCH 1/2] nir: Fix a comment typo in nir_lower_drawpixels.

2016-08-19 Thread Rob Clark
On Fri, Aug 19, 2016 at 7:11 PM, Eric Anholt wrote: > opps, r-b > --- > src/compiler/nir/nir_lower_drawpixels.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/compiler/nir/nir_lower_drawpixels.c > b/src/compiler/nir/nir_lower_drawpixels.c > index

Re: [Mesa-dev] [PATCH v2 1/2] glsl: fix key used for hashing switch statement cases

2016-08-19 Thread Eric Anholt
Tapani Pälli writes: > Implementation previously used value itself as the key, however after > hash implementation change by ee02a5e we cannot use 0 as key. > > v2: use constant pointer as the key and implement comparison > for contents (Eric Anholt) > >

Re: [Mesa-dev] [PATCH 25/32] i965: Move gen6_blorp.c to a file that gets recompiled per-gen

2016-08-19 Thread Mauro Rossi
2016-08-15 18:43 GMT+02:00 Jason Ekstrand : >> >> > --- >> > src/mesa/drivers/dri/i965/Makefile.am | 14 +- >> > src/mesa/drivers/dri/i965/Makefile.sources | 5 +- >> > src/mesa/drivers/dri/i965/gen6_blorp.c | 400 >> > >> >

Re: [Mesa-dev] [PATCH 25/32] i965: Move gen6_blorp.c to a file that gets recompiled per-gen

2016-08-19 Thread Mauro Rossi
From 4f105a68918b6ee25f1d0d71bc26dc39abed61d7 Mon Sep 17 00:00:00 2001 From: Mauro Rossi Date: Fri, 19 Aug 2016 23:36:11 +0200 Subject: [PATCH 1/3] android: i965: add per-gen libmesa_i965_gen6 static library Needed to fix android build after commit c8bc1ae where new

[Mesa-dev] [PATCH] isl: round format alignment to nearest power of 2

2016-08-19 Thread Lionel Landwerlin
A few inline asserts in anv assume alignments are power of 2, but with formats like R8G8B8 we have odd alignments. Signed-off-by: Lionel Landwerlin --- src/intel/isl/isl.c | 1 + src/intel/isl/isl_priv.h | 6 ++ 2 files changed, 7 insertions(+) diff

Re: [Mesa-dev] [PATCH 2/2] nir: Fix crash in nir_lower_drawpixels.

2016-08-19 Thread Rob Clark
On Fri, Aug 19, 2016 at 7:12 PM, Eric Anholt wrote: > Generally you'd see the gl_Color reference first and get some cursor set. > However, in piglit draw-pixel-with-texture we're now seeing the TexCoord > dereferenced first. yeah, that makes sense, r-b > --- >

[Mesa-dev] [PATCH 2/2] nir: Fix crash in nir_lower_drawpixels.

2016-08-19 Thread Eric Anholt
Generally you'd see the gl_Color reference first and get some cursor set. However, in piglit draw-pixel-with-texture we're now seeing the TexCoord dereferenced first. --- src/compiler/nir/nir_lower_drawpixels.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[Mesa-dev] [PATCH 1/2] nir: Fix a comment typo in nir_lower_drawpixels.

2016-08-19 Thread Eric Anholt
--- src/compiler/nir/nir_lower_drawpixels.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compiler/nir/nir_lower_drawpixels.c b/src/compiler/nir/nir_lower_drawpixels.c index 7ffaa525b5a4..a71bccddfb07 100644 --- a/src/compiler/nir/nir_lower_drawpixels.c +++

Re: [Mesa-dev] [PATCH 0/6] mesa: support for EGL_ANDROID_native_fence_sync

2016-08-19 Thread Rob Clark
Ok, quick update on state of explicit fence fd's / EGL_ANDROID_native_fence_sync: For folks working on other drivers wishing to add support, we don't have a "real" userspace yet. But Gustavo has added atomic support to kmscube (as well as all of his work for in/out fences in atomic ioctl, plus

Re: [Mesa-dev] [PATCH 0/6] mesa: support for EGL_ANDROID_native_fence_sync

2016-08-19 Thread Rob Clark
On Fri, Aug 19, 2016 at 5:57 PM, Rob Clark wrote: > Ok, quick update on state of explicit fence fd's / > EGL_ANDROID_native_fence_sync: > > For folks working on other drivers wishing to add support, we don't > have a "real" userspace yet. But Gustavo has added atomic support

Re: [Mesa-dev] [PATCH 25/32] i965: Move gen6_blorp.c to a file that gets recompiled per-gen

2016-08-19 Thread Mauro Rossi
From 6a20bec0a5947cd9c33024db3af51546eeac3062 Mon Sep 17 00:00:00 2001 From: Mauro Rossi Date: Fri, 19 Aug 2016 23:55:54 +0200 Subject: [PATCH 2/3] android: i965: add per-gen libmesa_i965_gen{7,75} static libraries Needed to fix android build after commit e198983 which

Re: [Mesa-dev] [PATCH 25/32] i965: Move gen6_blorp.c to a file that gets recompiled per-gen

2016-08-19 Thread Mauro Rossi
From 88e08ee520b7c8a14b17160a9e7102d72fed8ecb Mon Sep 17 00:00:00 2001 From: Mauro Rossi Date: Sat, 20 Aug 2016 00:04:29 +0200 Subject: [PATCH 3/3] android: i965: add per-gen libmesa_i965_gen{8,9} static Needed to fix android build after commit 16a9fcb which enabled genxml

[Mesa-dev] [PATCH v2 2/2] aubinator: Fix the tool to correctly decode the DWords

2016-08-19 Thread Sirisha Gandikota
From: Sirisha Gandikota Several fixes have been added as part of this as listed below: 1) Fix the mask and add disassembler handling for STATE_DS, STATE_HS as the mask returned wrong values of the fields. 2) Fix the GEN_TYPE_ADDRESS/GEN_TYPE_OFFSET decoding - the

[Mesa-dev] [PATCH v2 0/2] *** Aubinator tool for Intel Gen platforms ***

2016-08-19 Thread Sirisha Gandikota
From: Sirisha Gandikota This is a patch series for adding the aubinator tool to the codebase. The aubinator tool is designed to help the driver developers to debug the driver functionality by decoding the data in the .aub files. This tool is for Intel Gen platforms

Re: [Mesa-dev] [PATCH] loader/dri3: Overhaul dri3_update_num_back

2016-08-19 Thread Eric Anholt
Michel Dänzer writes: > From: Michel Dänzer > > Always use 3 buffers when flipping. With only 2 buffers, we have to wait > for a flip to complete (which takes non-0 time even with asynchronous > flips) before we can start working on the next frame. We

Re: [Mesa-dev] [PATCH] anv: Give the installed intel_icd.json file an absolute path

2016-08-19 Thread Chad Versace
On Fri 19 Aug 2016, Jason Ekstrand wrote: > Not providing a path allows the ICD to work on multi-arch systems but > breaks it if you install anywhere other than /usr/lib. Given that users > may be installing locally in .local or similar, we probably do want to > provide a filename. Distros can

[Mesa-dev] [PATCH v2 1/2] aubinator: Add a new tool called Aubinator to the src/intel/tools folder.

2016-08-19 Thread Sirisha Gandikota
From: Kristian Høgsberg Kristensen The Aubinator tool is designed to help the driver developers in debugging the driver functionality by decoding the data in the .aub files. Primary Authors of this tool are Damien Lespiau and Kristian Høgsberg Kristensen . v2: Review

[Mesa-dev] [PATCH] gtn: Trim out unused VS input variables.

2016-08-19 Thread Eric Anholt
If we're going to skip setting up vertex input data in them, we should probably not leave them as vertex inputs with a driver_location that happens to alias to something else. Fixes a regression in glsl-mat-attribute on vc4 when enabling GTN. --- src/mesa/state_tracker/st_glsl_to_nir.cpp | 27

Re: [Mesa-dev] [PATCH 3/3] nir: Define system values for vc4's blending-lowering arguments.

2016-08-19 Thread Kenneth Graunke
On Saturday, August 6, 2016 12:28:38 AM PDT Eric Anholt wrote: > In the GLSL-to-NIR conversion of VC4, I had a bit of trouble with what I > was calling the "state uniforms" that I was putting into the NIR fighting > with its other lowering passes. Instead of using magic uniform base > numbers in

Re: [Mesa-dev] [PATCH] mesa: Fix fixed function spot lighting on newer hardware (again)

2016-08-19 Thread Daniel Scharrer
On 2016-08-14 06:38, Matt Turner wrote: > On Sat, Aug 13, 2016 at 5:07 PM, Daniel Scharrer wrote: >> This was first fixed in commit b3f9c5c and then broken again in commit >> fe2d2c7, which removed the abs modifier from input registers. >> >> Bugzilla:

[Mesa-dev] [PATCH v2] mesa: Fix fixed function spot lighting on newer hardware (again)

2016-08-19 Thread Daniel Scharrer
This was first fixed in commit b3f9c5c and then broken again in commit fe2d2c7, which removed the abs modifier from input registers. v2: Don't change the size of struct ureg. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91342 Signed-off-by: Daniel Scharrer Cc:

[Mesa-dev] [Bug 97231] GL_DEPTH_CLAMP doesn't clamp to the far plane

2016-08-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97231 --- Comment #23 from Roland Scheidegger --- For llvmpipe, this is fixed by 0849621891041498c7438080338ccea562440a9a. -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the

Re: [Mesa-dev] [PATCH] gtn: Trim out unused VS input variables.

2016-08-19 Thread Kenneth Graunke
On Friday, August 19, 2016 6:38:57 PM PDT Eric Anholt wrote: > If we're going to skip setting up vertex input data in them, we should > probably not leave them as vertex inputs with a driver_location that > happens to alias to something else. > > Fixes a regression in glsl-mat-attribute on vc4

Re: [Mesa-dev] [PATCH] gtn: Trim out unused VS input variables.

2016-08-19 Thread Kenneth Graunke
On Friday, August 19, 2016 6:38:57 PM PDT Eric Anholt wrote: > If we're going to skip setting up vertex input data in them, we should > probably not leave them as vertex inputs with a driver_location that > happens to alias to something else. > > Fixes a regression in glsl-mat-attribute on vc4

Re: [Mesa-dev] [PATCH v2] mesa: Fix fixed function spot lighting on newer hardware (again)

2016-08-19 Thread Matt Turner
On Fri, Aug 19, 2016 at 7:23 PM, Daniel Scharrer wrote: > This was first fixed in commit b3f9c5c and then broken again in commit > fe2d2c7, which removed the abs modifier from input registers. > > v2: Don't change the size of struct ureg. > > Bugzilla:

Re: [Mesa-dev] [PATCH v2] mesa: Fix fixed function spot lighting on newer hardware (again)

2016-08-19 Thread Matt Turner
On Fri, Aug 19, 2016 at 8:44 PM, Matt Turner wrote: > On Fri, Aug 19, 2016 at 7:23 PM, Daniel Scharrer wrote: >> This was first fixed in commit b3f9c5c and then broken again in commit >> fe2d2c7, which removed the abs modifier from input registers. >> >>

[Mesa-dev] [PATCH] a3xx: make use of software clipping when hw can't handle it

2016-08-19 Thread Ilia Mirkin
The hw clipper only handles up to 6 UCPs. If there are more than 6 UCPs, or a clip vertex, or clip distances are in use, then we must use the fallback discard-based clipping from the frag shader. Signed-off-by: Ilia Mirkin --- src/gallium/drivers/freedreno/a3xx/fd3_draw.c