[Mesa-dev] [PATCH] radeonsi: disable the constant engine (CE) on Carrizo and Stoney

2016-12-07 Thread Marek Olšák
From: Marek Olšák It must be disabled until the kernel bug is fixed, and then we'll enable CE based on the DRM version. Cc: 12.0 13.0 --- src/gallium/drivers/radeonsi/si_pipe.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-)

[Mesa-dev] [PATCH 05/84] st/nine: Move texture setting to nine_context_*

2016-12-07 Thread Axel Davy
And move samplers_shadow to nine_context. Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 36

[Mesa-dev] [PATCH 18/84] st/nine: Put ff data in a separate structure

2016-12-07 Thread Axel Davy
And make nine_state_access_transform take this new structure as input. Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c

[Mesa-dev] [PATCH 09/84] st/nine: Move stream freq data to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 19 src/gallium/state_trackers/nine/nine_state.c

[Mesa-dev] [PATCH 13/84] st/nine: Back all shader constants to nine_context

2016-12-07 Thread Axel Davy
For device vs shader float constants and may_swvp, the same tips than for the other constant types is used. Also memset the constants properly. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 138 ++

[Mesa-dev] [PATCH 06/84] st/nine: Back textures into nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 1 + src/gallium/state_trackers/nine/device9.c | 2 ++

[Mesa-dev] [PATCH 04/84] st/nine: Track changed.texture only for stateblocks

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 4 +--- src/gallium/state_trackers/nine/device9.c | 3 ++-

[Mesa-dev] [PATCH 02/84] st/nine: Move core of device clear to nine_state

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 130 +

[Mesa-dev] [PATCH 07/84] st/nine: Move stream_usage_mask to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 6 +++--- src/gallium/state_trackers/nine/nine_state.h | 3 ++- 2

[Mesa-dev] [PATCH 19/84] st/nine: Refactor SetLight

2016-12-07 Thread Axel Davy
Call a helper function to set the light. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 24 src/gallium/state_trackers/nine/nine_state.c | 28 src/gallium/state_trackers/nine/nine_state.h | 3

[Mesa-dev] [PATCH 29/84] st/nine: Access pipe_context via NineDevice9_GetPipe

2016-12-07 Thread Axel Davy
Except for nine_ff and nine_state. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 13 +--- src/gallium/state_trackers/nine/basetexture9.h | 1 - src/gallium/state_trackers/nine/buffer9.c | 16 +

[Mesa-dev] [PATCH 00/84] Introduce gallium nine internal multithreading

2016-12-07 Thread Axel Davy
This patch adds internal multithreading to gallium nine. The goal is to offload almost all gallium nine calls (and some other work) to a worker thread. The patch serie does first a lot of refactoring, and introduces a new nine_context structure containing all the required internal states to do

[Mesa-dev] [PATCH 44/84] st/nine: Add struct nine_clipplane

2016-12-07 Thread Axel Davy
From: Patrick Rudolph Required to know the size exact size of the plane. Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c| 2 +- src/gallium/state_trackers/nine/nine_state.c | 4 ++--

[Mesa-dev] [PATCH 43/84] st/nine: Add nine_queue

2016-12-07 Thread Axel Davy
From: Patrick Rudolph This queue mechanism will be used for CSMT. Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/Makefile.sources | 2 + src/gallium/state_trackers/nine/nine_queue.c | 251 +++

[Mesa-dev] [PATCH 64/84] st/nine: Simplify ARG_BIND_REF

2016-12-07 Thread Axel Davy
Remove some noop operations. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_csmt_helper.h | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_csmt_helper.h

[Mesa-dev] [PATCH 50/84] st/nine: Use nine_context for resource_copy_region

2016-12-07 Thread Axel Davy
From: Patrick Rudolph Use nine_context wrapper for resource_copy_region. Enables to offload it with CSMT. Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c| 6 ++ src/gallium/state_trackers/nine/nine_state.c | 17

[Mesa-dev] [PATCH 75/84] st/nine: Fix leak with cubetexture dtor

2016-12-07 Thread Axel Davy
The last level was not released. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/cubetexture9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/cubetexture9.c b/src/gallium/state_trackers/nine/cubetexture9.c

[Mesa-dev] [PATCH 40/84] st/nine: Change the way nine_shader gets the pipe

2016-12-07 Thread Axel Davy
The change is required with csmt, where depending on the thread you don't access the pipe the same way. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_shader.c | 3 +-- src/gallium/state_trackers/nine/nine_shader.h | 4 +++-

[Mesa-dev] [PATCH 52/84] st/nine: Fix BASETEX_REGISTER_UPDATE

2016-12-07 Thread Axel Davy
BASETEX_REGISTER_UPDATE was adding the texture to the list of textures to upload in too many cases. tex->base.base.bind will be set to true if the texture is in a stateblock, whereas we want to upload only if bound to the device, which is what bind_count is for. Signed-off-by: Axel Davy

[Mesa-dev] [PATCH 59/84] st/nine: Simplify ColorFill

2016-12-07 Thread Axel Davy
For render targets, NineSurface9_GetSurface is not expected to fail. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c

[Mesa-dev] [PATCH 38/84] st/nine: Decompose nine_context_set_texture

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 73 ++-- 1 file changed, 48 insertions(+),

[Mesa-dev] [PATCH 72/84] st/nine: Use nine_context_gen_mipmap in BaseTexture9

2016-12-07 Thread Axel Davy
Generate mipmaps in the worker thread. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/src/gallium/state_trackers/nine/basetexture9.c

[Mesa-dev] [PATCH 01/36] st/nine: Improve doc of D3DPMISCCAPS_POSTBLENDSRGBCONVERT

2016-12-07 Thread Axel Davy
The cap should be advertised for d3d10 able cards, but only for Ex contexts. Unfortunately at this point Mesa has no way to know if Ex is used or not (the info is got later). Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c | 2 +- 1 file changed, 1

[Mesa-dev] [PATCH 17/36] st/nine: Do not generate gallium NOP on d3d NOP

2016-12-07 Thread Axel Davy
Some drivers crash if NOP is generated. Besides there is no point to generate NOP. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_shader.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 31/36] st/nine: Fix ff cases when stages should be disabled

2016-12-07 Thread Axel Davy
When a texture is read by a stage for colorop, it should be disabled, and disable following stages. When a texture is read for alphaop, 1.0f is read for the input, which is the behaviour for a dummy texture. Signed-off-by: Axel Davy ---

[Mesa-dev] [PATCH 30/36] st/nine: Always initialize current in ff ps

2016-12-07 Thread Axel Davy
The check was not catching all possible cases. NVE4 should be fine. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 25 + 1 file changed, 1 insertion(+), 24 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c

[Mesa-dev] [PATCH 19/36] d3dadapter/present: Add new API to ID3DPresent

2016-12-07 Thread Axel Davy
The API will enable better support for the commonly used DISCARD swapchain parameter. Signed-off-by: Axel Davy --- include/d3dadapter/present.h | 24 1 file changed, 24 insertions(+) diff --git a/include/d3dadapter/present.h

[Mesa-dev] [PATCH 14/36] st/nine: Handle when cursor stride is not what is expected

2016-12-07 Thread Axel Davy
SetCursor assumes for now a 32x32 argb cursor with pitch 128. 32x32 argb doesn't have pitch 128 on all hw, thus use a temporary surface with the correct pitch when needed. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 20 ++--

[Mesa-dev] [PATCH 00/36] Some gallium nine fixes

2016-12-07 Thread Axel Davy
This serie fixes several gallium nine bugs. There is also a few cleanups. It also has a new feature: new options are added to choose the behaviour of the swapchain DISCARD mode (which is the most common mode). Basically the default is that when vsync is off, the frames are presented as soon as

[Mesa-dev] [PATCH 02/36] st/nine: Add validation to SetSamplerState

2016-12-07 Thread Axel Davy
Check value validity and mimick Win behaviour. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 9 --- src/gallium/state_trackers/nine/nine_limits.h | 36 +++ src/gallium/state_trackers/nine/nine_state.c | 2 ++

[Mesa-dev] [PATCH 04/36] st/nine: Don't update stream_usage_mask in sw path

2016-12-07 Thread Axel Davy
The variable is used only in the hw path. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_state.c b/src/gallium/state_trackers/nine/nine_state.c index

[Mesa-dev] [PATCH 28/36] st/nine: Do not saturate illumination coefficients in ff

2016-12-07 Thread Axel Davy
Fixes bad rendering of a test app. Wine has the same behaviour. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 4 1 file changed, 4 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c

[Mesa-dev] [PATCH 33/36] st/nine: Fix two special cases in ff ps

2016-12-07 Thread Axel Davy
if first alpha stage is disabled and writes to temp, diffuse alpha is written to temp. Last stage always writes to current. Behaviour was deduced by tests with a test app. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 20 1 file

[Mesa-dev] [PATCH 05/36] st/nine: Fix useless swapchain init checks

2016-12-07 Thread Axel Davy
In NineDevice9_SetDefaultState we can assume the implicit swapchain is properly initialized. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 21 +++-- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 1/3] cso: don't release sampler states that are bound

2016-12-07 Thread Marek Olšák
On Wed, Dec 7, 2016 at 10:06 PM, Roland Scheidegger wrote: > Am 07.12.2016 um 21:46 schrieb Marek Olšák: >> On Wed, Dec 7, 2016 at 6:00 PM, Roland Scheidegger >> wrote: >>> Am 07.12.2016 um 17:26 schrieb Marek Olšák: Optimizing the CSO cache isn't

Re: [Mesa-dev] [PATCH] automake: make the build less chatty

2016-12-07 Thread Eric Anholt
Emil Velikov writes: > From: Emil Velikov > > Having the "Entering|Leaving directory X" messages it not required nor > useful in vast majority of the cases. > > One can always have them printed by `make -w' or by overriding the >

Re: [Mesa-dev] [PATCH 1/3] cso: don't release sampler states that are bound

2016-12-07 Thread Ilia Mirkin
On Wed, Dec 7, 2016 at 3:46 PM, Marek Olšák wrote: > On Wed, Dec 7, 2016 at 6:00 PM, Roland Scheidegger wrote: >> Am 07.12.2016 um 17:26 schrieb Marek Olšák: >>> Optimizing the CSO cache isn't exactly on the top of my list, so I >>> can't really do that

Re: [Mesa-dev] [PATCH 4/6] nir/lower_tex: add lowering for texture gradient on shadow samplers

2016-12-07 Thread Iago Toral
On Mon, 2016-12-05 at 20:39 -0800, Kenneth Graunke wrote: > On Thursday, December 1, 2016 8:53:19 AM PST Iago Toral Quiroga > wrote: > > > > This is ported from the Intel lowering pass that we use with GLSL > > IR. > > This takes care of lowering texture gradients on shadow samplers > > other > >

Re: [Mesa-dev] [PATCH 1/3] cso: don't release sampler states that are bound

2016-12-07 Thread Nicolai Hähnle
On 07.12.2016 08:50, Michel Dänzer wrote: On 06/12/16 10:24 PM, Marek Olšák wrote: On Mon, Dec 5, 2016 at 10:05 AM, Michel Dänzer wrote: On 03/12/16 05:38 AM, Marek Olšák wrote: From: Marek Olšák This fixes random radeonsi GPU hangs in Batman

Re: [Mesa-dev] [PATCH v2 000/103] i965 Haswell ARB_gpu_shader_fp64 / OpenGL 4.0

2016-12-07 Thread Iago Toral
On Mon, 2016-12-05 at 15:21 -0800, Matt Turner wrote: > On 10/11, Iago Toral Quiroga wrote: > > > > It's been some time since > ... anyone has reviewed your patches. Sorry. :( > > I'm going to review from your rebased i965-fp64-gen7-scalar-vec4-rc2 > branch. There have probably been some

Re: [Mesa-dev] [PATCH] radeonsi: fix an off-by-one error in the bounds check for max_vertices

2016-12-07 Thread Nicolai Hähnle
On 07.12.2016 04:47, Michel Dänzer wrote: On 07/12/16 05:06 AM, Nicolai Hähnle wrote: From: Nicolai Hähnle The spec actually says that calling EmitStreamVertex is undefined when you exceed max_vertices. But we do need to avoid trampling over memory outside the GSVS

Re: [Mesa-dev] [PATCH v2 02/25] tgsi: add Stream{X, Y, Z, W} fields to tgsi_declaration_semantic

2016-12-07 Thread Nicolai Hähnle
On 07.12.2016 02:07, Marek Olšák wrote: On Tue, Dec 6, 2016 at 11:48 AM, Nicolai Hähnle wrote: From: Nicolai Hähnle This is for geometry shader outputs. Without it, drivers have no way of knowing which stream each output is intended for, and have

Re: [Mesa-dev] [PATCH 1/6] nir/lower_tex: generalize get_texture_size()

2016-12-07 Thread Iago Toral
On Mon, 2016-12-05 at 20:38 -0800, Kenneth Graunke wrote: > On Thursday, December 1, 2016 8:53:16 AM PST Iago Toral Quiroga > wrote: > > > > This was written specifically for RECT samplers. Make it more > > generic so > > we can call this from the gradient lowerings too. > > --- > >  

[Mesa-dev] [PATCH 2/5] configure: remove HAVE_EGL_DRIVER_DRI[23]

2016-12-07 Thread Emil Velikov
From: Emil Velikov We have them for local purposes in configure, where we can use their direct dependency. With the only remaining instance in the makefile(s) being always true, as it can be seen in the configure snippet. Signed-off-by: Emil Velikov

[Mesa-dev] [PATCH 5/5] configure: error out when building static XOR shared

2016-12-07 Thread Emil Velikov
From: Emil Velikov Current code warns out in such cases and falls-back to either static or shared. That can be easily missed amongst the volume produced by our configure script. Replace the warning with an error such that one gets direct feedback when they're doing

[Mesa-dev] [PATCH 3/5] configure: factor out commom egl/gbm checks

2016-12-07 Thread Emil Velikov
From: Emil Velikov Signed-off-by: Emil Velikov --- configure.ac | 47 ++- 1 file changed, 18 insertions(+), 29 deletions(-) diff --git a/configure.ac b/configure.ac index bea6975..162c31a

[Mesa-dev] [PATCH 1/5] configure: forbid static EGL/GBM

2016-12-07 Thread Emil Velikov
From: Emil Velikov Both libraries implicitly require shared GLAPI which in itself mandates shared libraries. Stop pretending that one can use it and error out at configure stage. Signed-off-by: Emil Velikov --- configure.ac | 36

[Mesa-dev] [PATCH 4/5] configure: explicitly require shared glapi for enable-dri

2016-12-07 Thread Emil Velikov
From: Emil Velikov We've been using and depending on it for at least a couple of years. Make it obvious and error out, should one opt for it. Signed-off-by: Emil Velikov --- configure.ac | 4 +--- 1 file changed, 1 insertion(+), 3

[Mesa-dev] [PATCH] automake: make the build less chatty

2016-12-07 Thread Emil Velikov
From: Emil Velikov Having the "Entering|Leaving directory X" messages it not required nor useful in vast majority of the cases. One can always have them printed by `make -w' or by overriding the AM_MAKEFLAGS variable. Signed-off-by: Emil Velikov

Re: [Mesa-dev] [PATCH] automake: make the build less chatty

2016-12-07 Thread Edward O'Callaghan
Reviewed-by: Edward O'Callaghan On 12/08/2016 12:22 AM, Emil Velikov wrote: > From: Emil Velikov > > Having the "Entering|Leaving directory X" messages it not required nor > useful in vast majority of the cases. > > One can always have

Re: [Mesa-dev] [PATCH] egl: wayland: bufferAge: only request a back buffer if there isn't one

2016-12-07 Thread Eric Engestrom
On Tuesday, 2016-12-06 12:13:32 +, Lionel Landwerlin wrote: > On 06/12/16 11:36, Eric Engestrom wrote: > > On Saturday, 2016-12-03 22:47:17 +, Lionel Landwerlin wrote: > > > Seeing gtk+ application lockup when they query the buffer age of a > > > surface. > > > > > > Since we update the

[Mesa-dev] [Bug 99014] clover is broken in 12.0.4

2016-12-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99014 --- Comment #4 from Vedran Miletić --- (In reply to Igor Gnatenko from comment #3) > (In reply to Michel Dänzer from comment #2) > > Can you bisect which commit broke it? > Unfortunately it's not my bugreport. In theory, I

Re: [Mesa-dev] [PATCH] i965/blit: Remove Yf/Ys tiled check under a FIXME at can_fast_copy_blit

2016-12-07 Thread Alejandro Piñeiro
On 07/12/16 09:36, Alejandro Piñeiro wrote: > On 06/12/16 22:26, Anuj Phogat wrote: >> On Tue, Dec 6, 2016 at 10:58 AM, Alejandro Piñeiro >> wrote: >>> The FIXME suggest that the check should be removed. >>> >> Only if we see any performance or feature benefits in doing

Re: [Mesa-dev] [PATCH] i965/blit: Remove Yf/Ys tiled check under a FIXME at can_fast_copy_blit

2016-12-07 Thread Alejandro Piñeiro
On 06/12/16 22:26, Anuj Phogat wrote: > On Tue, Dec 6, 2016 at 10:58 AM, Alejandro Piñeiro > wrote: >> The FIXME suggest that the check should be removed. >> > Only if we see any performance or feature benefits in doing that. > Last I checked I didn't see any performance

Re: [Mesa-dev] [PATCH] egl: wayland: bufferAge: only request a back buffer if there isn't one

2016-12-07 Thread Lionel Landwerlin
On 07/12/16 10:55, Eric Engestrom wrote: On Tuesday, 2016-12-06 12:13:32 +, Lionel Landwerlin wrote: On 06/12/16 11:36, Eric Engestrom wrote: On Saturday, 2016-12-03 22:47:17 +, Lionel Landwerlin wrote: Seeing gtk+ application lockup when they query the buffer age of a surface. Since

[Mesa-dev] [Bug 99014] clover is broken in 12.0.4

2016-12-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99014 Vedran Miletić changed: What|Removed |Added CC||ved...@miletic.net

[Mesa-dev] [Bug 99014] clover is broken in 12.0.4

2016-12-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99014 --- Comment #5 from Igor Gnatenko --- (In reply to Vedran Miletić from comment #4) > (In reply to Igor Gnatenko from comment #3) > > (In reply to Michel Dänzer from comment #2) > > > Can you bisect which commit broke

[Mesa-dev] [PATCH 1/2] egl: wayland: set buffer age value to 0 when image is destroyed

2016-12-07 Thread Lionel Landwerlin
We usually set that value up when creating the buffer (NULL to non-NULL transition). It doesn't hurt to set that value to 0 up front when we destroy the buffer (non-NULL to NULL transition). Signed-off-by: Lionel Landwerlin ---

[Mesa-dev] [PATCH 2/2] egl: wayland: bufferAge: only request a back buffer if there isn't one

2016-12-07 Thread Lionel Landwerlin
Seeing gtk+ application lockup when they query the buffer age of a surface. Since we update the buffer age field only when creating buffers & swaping them on the client side, there shouldn't be any need for requesting a new back buffer if there is already one available. This change depends on a

[Mesa-dev] [Bug 98563] Xorg segfaults with displaylink attached and mesa version >= 13.0

2016-12-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98563 --- Comment #14 from Emil Velikov --- (In reply to David Rosenstrauch from comment #11) > Can you please clarify? If this is resolved, then what new package release > provides the fixed behavior? I'm still seeing this

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-07 Thread Lionel Landwerlin
On 07/12/16 14:55, Ilia Mirkin wrote: On Thu, Dec 1, 2016 at 10:56 AM, Lionel Landwerlin wrote: Signed-off-by: Lionel Landwerlin --- src/compiler/glsl/ast.h | 5 src/compiler/glsl/ast_to_hir.cpp |

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-07 Thread Ilia Mirkin
On Wed, Dec 7, 2016 at 10:25 AM, Lionel Landwerlin wrote: > On 07/12/16 14:55, Ilia Mirkin wrote: >> >> On Thu, Dec 1, 2016 at 10:56 AM, Lionel Landwerlin >> wrote: >>> >>> Signed-off-by: Lionel Landwerlin

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-07 Thread Ilia Mirkin
On Thu, Dec 1, 2016 at 10:56 AM, Lionel Landwerlin wrote: > Signed-off-by: Lionel Landwerlin > --- > src/compiler/glsl/ast.h | 5 > src/compiler/glsl/ast_to_hir.cpp | 5 >

Re: [Mesa-dev] [PATCH] gallivm: use getHostCPUFeatures on x86/llvm-4.0+.

2016-12-07 Thread Roland Scheidegger
The bug in llvm has been fixed, can you confirm lp_test_format passes again? Roland Am 06.12.2016 um 19:00 schrieb Roland Scheidegger: > Ok, here is the bug: > https://llvm.org/bugs/show_bug.cgi?id=31296 > > Roland > > Am 06.12.2016 um 18:47 schrieb Roland Scheidegger: >> Actually I've

[Mesa-dev] [PATCH 23/36] st/nine: Add new driconf options to control DISCARD behaviour

2016-12-07 Thread Axel Davy
See the patch for the new controls added. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.h | 2 ++ src/gallium/state_trackers/nine/swapchain9.c| 4 ++-- src/gallium/targets/d3dadapter9/drm.c | 17 +

[Mesa-dev] [PATCH 10/36] st/nine: Add some debug info in stateblocks

2016-12-07 Thread Axel Davy
This is useful to check what is exactly recorded. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/stateblock9.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/state_trackers/nine/stateblock9.c b/src/gallium/state_trackers/nine/stateblock9.c

[Mesa-dev] [PATCH 27/36] st/nine: Fix ff COLOR0 w component computation

2016-12-07 Thread Axel Davy
The computation was wrong. COLOR0's last component should be equal to the material diffuse w component. The behaviour was checked with a test app on Windows. Wine has the same behaviour. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 31

[Mesa-dev] [PATCH 34/36] st/nine: Convert redundant check to assert in ff ps

2016-12-07 Thread Axel Davy
We disable the alpha stage if the color stage is disabled. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c

[Mesa-dev] [PATCH 16/36] st/nine: Fix leak in user constant upload path

2016-12-07 Thread Axel Davy
The new code properly releases the previous buffers allocated. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c| 22 +++-- src/gallium/state_trackers/nine/nine_state.c | 46 2 files changed, 39 insertions(+), 29

[Mesa-dev] [PATCH 11/36] st/nine: Add missing changed states to pixel stateblocks

2016-12-07 Thread Axel Davy
Some states were not properly recorded in pixel stateblocks. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/device9.c

[Mesa-dev] [PATCH 32/36] st/nine: Remove useless code in ff ps

2016-12-07 Thread Axel Davy
Current is already initialized to Diffuse. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 5 - 1 file changed, 5 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index

[Mesa-dev] [PATCH 08/36] st/nine: Fix bad light initialization in stateblocks

2016-12-07 Thread Axel Davy
src was initialized instead of dst. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/stateblock9.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/stateblock9.c b/src/gallium/state_trackers/nine/stateblock9.c

[Mesa-dev] [PATCH 35/36] st/nine: Fix ff texture coordinate selection

2016-12-07 Thread Axel Davy
The code was wrongly detecting which texture coordinates to generate when the coordinate index was different to the stage index. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git

[Mesa-dev] [PATCH 20/36] d3dadapter/present: Add precision for WaitBufferReleased

2016-12-07 Thread Axel Davy
Add precision on the behaviour of WaitBufferReleased. All implementers and users of the API were expecting that behaviour. Signed-off-by: Axel Davy --- include/d3dadapter/present.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 25/36] st/nine: Ignore MULTISAMPLEMASK when RT is not multisampled

2016-12-07 Thread Axel Davy
We were ignoring MULTISAMPLEMASK for non-maskable multisample modes, but we were missing the non-multisampled case. Fixes a crash in Halo. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 4 ++-- src/gallium/state_trackers/nine/nine_state.c | 2 +-

Re: [Mesa-dev] [PATCH 00/84] Introduce gallium nine internal multithreading

2016-12-07 Thread Dieter Nützel
Am 07.12.2016 23:54, schrieb Axel Davy: This patch adds internal multithreading to gallium nine. The goal is to offload almost all gallium nine calls (and some other work) to a worker thread. Hello Axel, really nice job! 'But' show us the 'numbers'... ...do you have some numbers of hand?

Re: [Mesa-dev] [PATCH v2 1/2] virgl: Fix a strict-aliasing violation in the encoder

2016-12-07 Thread Edward O'Callaghan
On 12/08/2016 02:53 AM, Emil Velikov wrote: > On 6 December 2016 at 22:34, Edward O'Callaghan > wrote: >> On 12/07/2016 12:36 AM, Emil Velikov wrote: >>> On 6 December 2016 at 11:30, Edward O'Callaghan >>> wrote: As per the C spec,

Re: [Mesa-dev] [PATCH 00/84] Introduce gallium nine internal multithreading

2016-12-07 Thread Axel Davy
On 08/12/2016 00:38, Dieter Nützel wrote: Am 07.12.2016 23:54, schrieb Axel Davy: This patch adds internal multithreading to gallium nine. The goal is to offload almost all gallium nine calls (and some other work) to a worker thread. Hello Axel, really nice job! 'But' show us the

Re: [Mesa-dev] radv cleanup series

2016-12-07 Thread Edward O'Callaghan
This series is, Reviewed-by: Edward O'Callaghan On 12/08/2016 11:18 AM, Dave Airlie wrote: > While working on spilling, geom shaders and compute queues, I've built > up a bunch of cleanups, so best to dequeue them early. > > Dave. > >

Re: [Mesa-dev] [PATCH 05/10] radeonsi: check for sampler state CSO corruption

2016-12-07 Thread Marek Olšák
On Fri, Dec 2, 2016 at 9:52 PM, Gustaw Smolarczyk wrote: > 2016-12-02 21:39 GMT+01:00 Marek Olšák : >> >> From: Marek Olšák >> >> It really happens. >> --- >> src/gallium/drivers/radeonsi/si_descriptors.c | 1 + >>

Re: [Mesa-dev] [PATCH v2] swr: [rasterizer common/core/jitter] fetch support for GL_FIXED

2016-12-07 Thread Cherniak, Bruce
Reviewed-by: Bruce Cherniak > On Dec 7, 2016, at 7:16 PM, Tim Rowley wrote: > > v2: use fmul(1/65536) instead of fdiv(65535) > --- > .../drivers/swr/rasterizer/common/formats.cpp | 104 ++--- >

Re: [Mesa-dev] [PATCH] swr: [rasterizer core/memory] Finish R24_UNORM_X8_TYPELESS for AVX512

2016-12-07 Thread Cherniak, Bruce
Reviewed-by: Bruce Cherniak > On Dec 7, 2016, at 5:55 PM, Rowley, Timothy O > wrote: > > This one-off specialization was missed. > --- > src/gallium/drivers/swr/rasterizer/core/backend.h | 4 +- >

Re: [Mesa-dev] Potentially EOL ilo gallium driver

2016-12-07 Thread Pierre-Loup A. Griffais
On 12/07/2016 06:13 PM, Michel Dänzer wrote: On 08/12/16 09:59 AM, Rob Clark wrote: +author again.. no idea why list keeps loosing extra cc's.. Mailman removes addresses from Cc which are subscribed to the list and have "Avoid duplicate copies of messages?" enabled in the list Subscription

Re: [Mesa-dev] Potentially EOL ilo gallium driver

2016-12-07 Thread Michel Dänzer
On 08/12/16 11:53 AM, Pierre-Loup A. Griffais wrote: > On 12/07/2016 06:13 PM, Michel Dänzer wrote: >> On 08/12/16 09:59 AM, Rob Clark wrote: >>> +author again.. no idea why list keeps loosing extra cc's.. >> >> Mailman removes addresses from Cc which are subscribed to the list and >> have "Avoid

[Mesa-dev] [PATCH 07/36] st/nine: Remove unused ff.changed.group

2016-12-07 Thread Axel Davy
It was unused. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.h | 1 - src/gallium/state_trackers/nine/stateblock9.c | 5 - 2 files changed, 6 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_state.h

[Mesa-dev] [PATCH 12/36] st/nine: Capture texturestage states in pixel stateblocks

2016-12-07 Thread Axel Davy
pixels stateblocks need to capture these. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index

[Mesa-dev] [PATCH 26/36] st/nine: Fix specular enable for alpha

2016-12-07 Thread Axel Davy
Apparently specular enable doesn't affect the alpha channel. Fixes https://github.com/iXit/Mesa-3D/issues/253 Behaviour comfirmed looking in wine sources. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 2 +- 1 file changed, 1 insertion(+), 1

[Mesa-dev] [PATCH] swr: [rasterizer common/core/jitter] fetch support for GL_FIXED

2016-12-07 Thread Tim Rowley
--- .../drivers/swr/rasterizer/common/formats.cpp | 104 ++--- .../drivers/swr/rasterizer/common/formats.h| 7 +- .../drivers/swr/rasterizer/core/format_traits.h| 90 +- .../drivers/swr/rasterizer/jitter/fetch_jit.cpp| 12 +++

[Mesa-dev] [PATCH 04/10] radv/meta: clean up buffer->image code.

2016-12-07 Thread Dave Airlie
From: Dave Airlie Removes some unnecessary functions and pull some stuff out of the loop. Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_meta_bufimage.c | 42 +++-- 1 file changed, 7 insertions(+), 35 deletions(-)

[Mesa-dev] [PATCH 06/10] radv/meta: cleanup image info setup.

2016-12-07 Thread Dave Airlie
From: Dave Airlie This just passes the subresource info in and uses it. Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_meta_copy.c | 32 +--- 1 file changed, 13 insertions(+), 19 deletions(-) diff --git

[Mesa-dev] [PATCH 02/10] radv: add missing license file to radv_meta_bufimage.

2016-12-07 Thread Dave Airlie
From: Dave Airlie Just noticed this file was missing license and any explaination of what is in it. Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_meta_bufimage.c | 27 +++ 1 file changed, 27 insertions(+) diff --git

[Mesa-dev] [PATCH 09/10] radv/ac: pass a mask of array params not a number.

2016-12-07 Thread Dave Airlie
From: Dave Airlie This makes it easier to add new params before the array ones. Signed-off-by: Dave Airlie --- src/amd/common/ac_nir_to_llvm.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git

[Mesa-dev] [PATCH 03/10] radv/ac: don't pass nir to create_function

2016-12-07 Thread Dave Airlie
From: Dave Airlie This isn't needed for later things like geom shader copy shaders, we won't have NIR. Signed-off-by: Dave Airlie --- src/amd/common/ac_nir_to_llvm.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git

[Mesa-dev] [PATCH 07/10] radv/meta: don't pass rect into blit2d src function.

2016-12-07 Thread Dave Airlie
From: Dave Airlie Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_meta_blit2d.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/amd/vulkan/radv_meta_blit2d.c b/src/amd/vulkan/radv_meta_blit2d.c index 6e92f80..5a42fd8 100644

[Mesa-dev] [PATCH 08/10] radv: split out a chunk of variant filling code.

2016-12-07 Thread Dave Airlie
From: Dave Airlie This code will have use for copy shaders etc. Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_pipeline.c | 76 -- 1 file changed, 43 insertions(+), 33 deletions(-) diff --git

[Mesa-dev] [PATCH 10/10] radv: fix warnings in ubo load code.

2016-12-07 Thread Dave Airlie
From: Dave Airlie Signed-off-by: Dave Airlie --- src/amd/common/ac_nir_to_llvm.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c index 5820e62..c2a74d4 100644 ---

[Mesa-dev] [PATCH 05/10] radv/meta: split copyimage api into api and meta function

2016-12-07 Thread Dave Airlie
From: Dave Airlie This make it easier to add multiple queues later. Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_meta_copy.c | 34 +++--- 1 file changed, 23 insertions(+), 11 deletions(-) diff --git

Re: [Mesa-dev] Potentially EOL ilo gallium driver

2016-12-07 Thread Edward O'Callaghan
On 12/08/2016 11:28 AM, Roland Scheidegger wrote: > I haven't seen the driver author's opinion on this yet, so it's probably > fair to give him some more time to answer. It's not like this is really > urgent... Absolutely! > > Roland > > Am 08.12.2016 um 01:11 schrieb Edward O'Callaghan: >>

[Mesa-dev] [PATCH mesa] egl/x11: cleanup init code

2016-12-07 Thread Eric Engestrom
No functional change, just rewriting it in an easier-to-understand way. Signed-off-by: Eric Engestrom --- src/egl/drivers/dri2/platform_x11.c | 24 ++-- 1 file changed, 10 insertions(+), 14 deletions(-) diff --git a/src/egl/drivers/dri2/platform_x11.c

[Mesa-dev] [PATCH mesa 1/2] egl: rename static functions to match convention

2016-12-07 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- src/egl/main/eglapi.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c index 9950a72..925e881 100644 --- a/src/egl/main/eglapi.c +++ b/src/egl/main/eglapi.c @@

<    1   2   3   >