[Mesa-dev] [PATCH 58/75] st/nine: Allocate surface resources in surface ctor

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Allocate resources in surface ctor. Allows to use statetracker internal memory accounting. Fix for issue #231. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_tra

[Mesa-dev] [PATCH 01/75] llvmpipe: Fix overflow for 32 bits available memory computation

2016-10-05 Thread Axel Davy
And cap to 2 GB on 32 bits. Fixes https://bugs.freedesktop.org/show_bug.cgi?id=94561 Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/auxiliary/os/os_misc.c | 2 +- src/gallium/drivers/llvmpipe/lp_screen.c | 5 ++

[Mesa-dev] [PATCH 47/75] st/nine: Fix specular w coordinate

2016-10-05 Thread Axel Davy
Found looking at Wine formulas. Fixes a few visual issues. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_tr

[Mesa-dev] [PATCH 45/75] st/nine: Fix condition for specular lightning

2016-10-05 Thread Axel Davy
Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 23 +++ 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index 374e92f..1

[Mesa-dev] [PATCH 62/75] st/nine: Remove volume9 {Set/Get/Free}PrivateData functions

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Remove {Set/Get/Free}PrivateData in volume9. Functionality has been implement in IUnknown interface. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 60/75] st/nine: Implement {Set/Get/Free}PrivateData in iunknown

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Implement {Set/Get/Free}PrivateData in iunknown to get rid of duplicated code in resource9 and volume9. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nin

[Mesa-dev] [PATCH 31/75] st/nine: Fix ff SPECULARENABLE

2016-10-05 Thread Axel Davy
. Remove the vs code, and use the ps code. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 13 +++-- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_tracker

[Mesa-dev] [PATCH 65/75] st/nine: Add additional error handling in CheckDeviceMultiSampleType

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Return one supported quality level in error cases. Return error on invalid multisample count. Fixes failing wine tests. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/s

[Mesa-dev] [PATCH 39/75] st/nine: Fix ff computation for inverse

2016-10-05 Thread Axel Davy
Thanks to wine tests. Apparently 4x4 inverse is to be used, and if the inverse can't be calculated, the input matrix is to be used. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 26 ++ src/gallium/state_trackers/nine/nin

[Mesa-dev] [PATCH 55/75] st/nine: Fix assert in NineUnknown_QueryInterface

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Tests showed that is allowed to call this method on object that have a zero refcount. Required for issue #230. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_tracke

[Mesa-dev] [PATCH 70/75] st/nine: Assert on buffer creation failure

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Add an assert to make sure buffer creation doesn't fail. Add error handling in calling functions. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 34/75] st/nine: Fix tweening factor computation

2016-10-05 Thread Axel Davy
The computation was reversed. Deduced by tests on windows. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_tr

[Mesa-dev] [PATCH 14/75] st/nine: Initial mixed vertex processing support

2016-10-05 Thread Axel Davy
In mixed vertex processing, the user can enable or disable software vertex processing. It is on hardware by default. This feature is not a state, and thus the setting doesn't need to be recorded by stateblocks. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_tracker

[Mesa-dev] [PATCH 07/75] st/nine: Use fixed size arrays for swapchain buffers

2016-10-05 Thread Axel Davy
Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/swapchain9.c | 43 +--- src/gallium/state_trackers/nine/swapchain9.h | 8 +++--- 2 files changed, 11 insertions(+), 40 deletions(-) diff --git a/src/gallium/state_trackers/nine/swapch

[Mesa-dev] [PATCH 00/75] Gallium Nine patches for Mesa 13

2016-10-05 Thread Axel Davy
Processing should now work, and The Sims 2 (which needs ProcessVertices with vs3 support) works (These are not yet supported by wine ogl, but this is likely to come as well on their side in some near future). Yours, Axel Axel Davy (50): llvmpipe: Fix overflow for 32 bits available memory

[Mesa-dev] [PATCH 08/75] st/nine: Wait for pending tasks to execute in swapchain

2016-10-05 Thread Axel Davy
Fixes crash after Reset() when using thread_submit=true Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/swapchain9.c | 4 1 file changed, 4 insertions(+) diff --git a/src/gallium/state_trackers/nine/swapchain9.c b/src/gallium/state_trackers/nine/swap

[Mesa-dev] [PATCH 23/75] st/nine: Cleaning code for vs temporaries

2016-10-05 Thread Axel Davy
no behaviour change, actually solved a visual bug observed on a sample program. This was due to ureg_normalize3 polluting a temporary variable. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 233 +- 1 file change

[Mesa-dev] [PATCH 15/75] st/nine: Add support for swvp constants in shaders

2016-10-05 Thread Axel Davy
swvp has relaxed limits (more nested loops, etc). In particular it enables more constants. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_shader.c | 156 ++-- src/gallium/state_trackers/nine/nine_shader.h | 2 + src/g

[Mesa-dev] [PATCH 13/75] st/nine: Implement SetNPatchMode

2016-10-05 Thread Axel Davy
Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index f2317b6..e4ede6e 100644 --- a/src/g

[Mesa-dev] [PATCH 17/75] st/nine: Change dirty tracking for vs int and bool constants

2016-10-05 Thread Axel Davy
This change makes easier to introduce tracking for swvp constants. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 14 +++--- src/gallium/state_trackers/nine/nine_state.c | 18 + src/gallium/state_trackers/nine/nine_state.h

[Mesa-dev] [PATCH 24/75] st/nine: Small simplification for position_t and fog

2016-10-05 Thread Axel Davy
position_t disables fog computation. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c

[Mesa-dev] [PATCH 44/75] st/nine: Do always accumulate diffuse

2016-10-05 Thread Axel Davy
According to spec. Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_ff.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b

[Mesa-dev] [PATCH 16/75] st/nine: Drop unused constant upload path

2016-10-05 Thread Axel Davy
This path has been disabled for some time because of some bugs with it. It hasn't been updated to the new features, and is not faster. Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/device9.c|

[Mesa-dev] [PATCH 50/75] st/nine: Properly declare sampler states for ff

2016-10-05 Thread Axel Davy
Fixes a softpipe assertion failure with wine tests Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_ff.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/gallium/st

[Mesa-dev] [PATCH 21/75] st/nine: Handle D3DRS_NORMALIZENORMALS

2016-10-05 Thread Axel Davy
When this state is set, the normals computed in the vs ff shader should be normalized. Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph s...@das-labor.org --- src/gallium/state_trackers/nine/nine_ff.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-)

[Mesa-dev] [PATCH 06/75] st/nine: Fix buffer count check for Ex devices

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/swapchain9.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/gallium/s

[Mesa-dev] [PATCH 25/75] st/nine: Reorganize ff vtx processing

2016-10-05 Thread Axel Davy
The new order simplified the code a bit for next patches. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 124 +- 1 file changed, 72 insertions(+), 52 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_f

[Mesa-dev] [PATCH 32/75] st/nine: Use materials if source is not given.

2016-10-05 Thread Axel Davy
Deduced by test on windows. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index 4

[Mesa-dev] [PATCH 53/75] st/nine: Print interface id in NineSurface9_GetContainer

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> To ease debugging print interface id. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/surface9.c | 5 + 1 file changed, 5 insertions(+) diff -

[Mesa-dev] [PATCH 33/75] st/nine: Disable ff vertex blending if required inputs are missing

2016-10-05 Thread Axel Davy
This behaviour has been partially tested on windows. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/g

[Mesa-dev] [PATCH 72/75] st/nine: Set CLAMP_TO_EDGE on cubetextures

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Wine tests show that cubetextures always use PIPE_TEX_WRAP_CLAMP_TO_EDGE regardless of set sampler states. Fixes failing d3d9 wine test test_cube_wrap. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <a

[Mesa-dev] [PATCH 38/75] st/nine: Used normed Vtx for reflectionvector

2016-10-05 Thread Axel Davy
Fix deduced from the spec. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index c

[Mesa-dev] [PATCH 61/75] st/nine: Switch {Set/Get/Free}PrivateData functions

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Switch {Set/Get/Free}PrivateData function to introduced IUnknown functions. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/cubetexture9.c |

[Mesa-dev] [PATCH 49/75] st/nine: Handle user clipping planes properly for ff

2016-10-05 Thread Axel Davy
Found reading msdn and checking Wine. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 21 + 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_tracker

[Mesa-dev] [PATCH 57/75] st/nine: Fix D3DFMT_NULL size

2016-10-05 Thread Axel Davy
D3DFMT_NULL is mapped to PIPE_FORMAT_NONE. Instead of relying on PIPE_FORMAT_NONE to return a size, pick one. The one picked is the same than Wine. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_pipe.h | 2 ++ 1 file changed, 2 insertions(+) diff

[Mesa-dev] [PATCH 74/75] st/nine: Add debug output for lost devices

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Add debug output to ease debugging. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/swapchain9.c | 2 ++ 1 file changed, 2 insertions(+) diff -

[Mesa-dev] [PATCH 26/75] st/nine: Fix vertex blending aVtx computation

2016-10-05 Thread Axel Davy
There was an multiplication by the world matrix 0 which had nothing to do there. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 27 ++- 1 file changed, 10 insertions(+), 17 deletions(-) diff --git a/src/gallium/state_tr

[Mesa-dev] [PATCH 12/75] st/nine: Implement D3DUSAGE_SOFTWAREPROCESSING

2016-10-05 Thread Axel Davy
Buffers with this flag must be usable with both software and hardware vertex processing. Use Staging for fast cpu access. Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/buffer9.c | 5 +++-- 1 fil

[Mesa-dev] [PATCH 09/75] st/nine: Implement relative addressing for ps inputs

2016-10-05 Thread Axel Davy
To implement the feature we copy the ps inputs to a temp array. This is not optimal for performance, but it is the simplest solution. This is a feature that is very very rarely used. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_shader.

[Mesa-dev] [PATCH 54/75] st/nine: Print interface id in NineVolume9_GetContainer

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> To ease debugging print interface id. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/volume9.c | 5 + 1 file changed, 5 insertions(+) diff -

[Mesa-dev] [PATCH 48/75] st/nine: Fix the calculation of the number of vs inputs

2016-10-05 Thread Axel Davy
Fixes hangs on radeonsi, and assert on llvmpipe. Signed-off-by: Axel Davy <axel.d...@ens.fr> Cc: "12.0" <mesa-...@lists.freedestkop.org> --- src/gallium/state_trackers/nine/nine_shader.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/galli

[Mesa-dev] [PATCH 51/75] st/nine: Move assert in NineSurface9_ctor

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Move assert to function entry. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/surface9.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[Mesa-dev] [PATCH 73/75] st/nine: Prevent crash in GetRenderTargetData

2016-10-05 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> Return error instead of crashing on source surfaces with format D3DFMT_NULL. Fix for issue #236. Tested on Windows 7. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/s

[Mesa-dev] [PATCH 27/75] st/nine: Compact ff vs constants a bit

2016-10-05 Thread Axel Davy
There are several holes. This patch reduces the holes a bit, which reduces the size of the constant buffer uploaded. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff

Re: [Mesa-dev] Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?)

2016-10-04 Thread Axel Davy
On 04/10/2016 12:32, Emil Velikov wrote: On 2 October 2016 at 14:17, Axel Davy <axel.d...@ens.fr> wrote: Hi, If I understand, there hasn't been yet statements on whether the freeze is for Oct 7 or Oct 14. Could there be one ? I'd prefer myself Oct 14, because we have a lot of p

[Mesa-dev] [PATCH] st/nine: Memset pipe_resource templates

2016-10-09 Thread Axel Davy
Fixes regression introduced by ecd6fce2611e88ff8468a354cff8eda39f260a31 and is more future proof than just clearing the next field. Other nine usages did already zero out the templates. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 6 +

Re: [Mesa-dev] [PATCH] vl/dri3: handle the case of different GPU(v4.1)

2016-09-30 Thread Axel Davy
On 16/09/2016 14:51, Nayan Deshmukh wrote: bool flushed; + bool is_different_gpu; }; I get a warning when I build about incompatible pointer type at the line fd = loader_get_user_preferred_fd(fd, >is_different_gpu); When compiling on 32 bits. I get this should be int. Axel

Re: [Mesa-dev] Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?)

2016-10-02 Thread Axel Davy
Hi, If I understand, there hasn't been yet statements on whether the freeze is for Oct 7 or Oct 14. Could there be one ? I'd prefer myself Oct 14, because we have a lot of patches for nine, and they deserve more cleaning and testing, but if it's Oct 7, we'll try be on time. Axel On

Re: [Mesa-dev] [PATCH 11/16] travis: remove no longer needed liudev-dev dependency

2016-10-15 Thread Axel Davy
typo in the commit title: libudev-dev Axel On 11/10/2016 20:31, Emil Velikov wrote: From: Emil Velikov Signed-off-by: Emil Velikov --- .travis.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index

Re: [Mesa-dev] [PATCH v2 05/16] loader: reimplement loader_get_user_preferred_fd via libdrm

2016-10-15 Thread Axel Davy
On 14/10/2016 22:33, Emil Velikov wrote: On 14 October 2016 at 20:21, Axel Davy <axel.d...@ens.fr> wrote: The code looks good. With the minor nitpick fixed, this patch is: Reviewed-by: Axel Davy <axel.d...@ens.fr> Thanks. If you can skim through any of the other pat

[Mesa-dev] [PATCH 2/4] st/nine: Use align_calloc instead of align_malloc

2016-10-22 Thread Axel Davy
We are not sure exactly what needs to be 0 initialized, but we are missing some cases. 0 initialize all our current aligned allocation. Fixes Tree of Savior visual issues. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/buffer9.c | 2 +- src/g

[Mesa-dev] [PATCH 1/4] gallium/util: Add align_calloc

2016-10-22 Thread Axel Davy
Add implementation for align_calloc, which is align_malloc + memset. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/auxiliary/util/u_memory.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/src/gallium/auxiliary/util/u_memory.h b/src/gallium/auxiliary/util/u_me

[Mesa-dev] [PATCH 4/4] st/nine: Fix locking CubeTexture surfaces

2016-10-22 Thread Axel Davy
Only one Face of Cubetextures were locked when in DEFAULT Pool. Fixes: https://github.com/iXit/Mesa-3D/issues/129 CC: "12.0 13.0" <mesa-sta...@lists.freedesktop.org> Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/surface9.c | 1 + 1 fil

Re: [Mesa-dev] [PATCH 1/4] gallium/util: Add align_calloc

2016-10-22 Thread Axel Davy
On 22/10/2016 16:42, Brian Paul wrote: On Sat, Oct 22, 2016 at 4:28 AM, Axel Davy <axel.d...@ens.fr <mailto:axel.d...@ens.fr>> wrote: Add implementation for align_calloc, which is align_malloc + memset. Signed-off-by: Axel Davy <axel.d...@ens.fr <mailt

[Mesa-dev] [PATCH 3/4] st/nine: Fix mistake in Volume9 UnlockBox

2016-10-22 Thread Axel Davy
In the format fallback path, the height was used instead of the depth. CC: "12.0 13.0" <mesa-sta...@lists.freedesktop.org> Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/volume9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[Mesa-dev] [PATCH 1/4 v2] gallium/util: Add align_calloc

2016-10-23 Thread Axel Davy
Add implementation for align_calloc, which is align_malloc + memset. v2: add if (ptr) before memset. Fix indentation. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/auxiliary/util/u_memory.h | 8 1 file changed, 8 insertions(+) diff --git a/src/gallium/auxiliar

Re: [Mesa-dev] [PATCH 2/2] st/nine: Fix leak at device dtor

2016-10-20 Thread Axel Davy
For some reason, it introduces crashes. Not sure why. I also figured out this doesn't free anything. There seems to be missing code to free the 'nine_range_pool', and we leak at every device dtor. Patch 1/2 is still an important fix to avoid leaks. On 18/10/2016 23:11, Axel Davy wrote

[Mesa-dev] [PATCH 1/2] st/nine: Fix leak with integer and boolean constants

2016-10-18 Thread Axel Davy
. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_state.c | 39 +--- 1 file changed, 18 insertions(+), 21 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_state.c b/src/gallium/state_trackers/nine/nine_state.c index f

[Mesa-dev] [PATCH 2/2] st/nine: Fix leak at device dtor

2016-10-18 Thread Axel Davy
The datastructures to track dirty constants weren't freed. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 19 +++ 1 file changed, 19 insertions(+) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_tr

[Mesa-dev] [PATCH] gallium/os: Use unsigned integers for size computation

2016-10-11 Thread Axel Davy
Use uint64_t instead of int64_t in the calculation, as the result is uint64_t. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- This corrects the small mistake from 218459771a1801d7ad20dd340ac35a50f2b5b81a as reported by Emil. src/gallium/auxiliary/os/os_misc.c | 4 ++-- 1 file chan

Re: [Mesa-dev] [PATCH] gallium/os: Use unsigned integers for size computation

2016-10-11 Thread Axel Davy
On Tue, Oct 11, 2016 at 12:59 PM, Axel Davy <axel.d...@ens.fr <mailto:axel.d...@ens.fr>> wrote: Use uint64_t instead of int64_t in the calculation, as the result is uint64_t. Signed-off-by: Axel Davy <axel.d...@ens.fr <mailto:axel.d...@ens.fr>> --- This co

Re: [Mesa-dev] [PATCH 05/16] loader: reimplement loader_get_user_preferred_fd via libdrm

2016-10-11 Thread Axel Davy
On 11/10/2016 20:31, Emil Velikov wrote: - udev = udev_new(); - if (!udev) - goto prime_clean; + if (drmGetDevice(default_fd, ) != 0) + goto err; - default_device_id_path_tag = get_id_path_tag_from_fd(udev, default_fd); - if (!default_device_id_path_tag) - goto

Re: [Mesa-dev] [PATCH 1/3] gallium/os: Fix overflow on 32 bits

2016-10-10 Thread Axel Davy
On 10/10/2016 12:44, Emil Velikov wrote: On 6 October 2016 at 18:51, Axel Davy <axel.d...@ens.fr> wrote: On systems with more than 4GB of ram, os_get_total_physical_memory was triggering an integer overflow for the linux and haiku path, when on 32 bits. Bugzilla: https://bugs.freedeskt

[Mesa-dev] [PATCH] st/nine: Fix multisample limit check

2016-10-16 Thread Axel Davy
Fixes regression introduced by b5603056872708fdd82f1224854097805a01d4c0 The regression prevents some apps to start. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/adapter9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/g

Re: [Mesa-dev] [PATCH] st/nine: Fix multisample limit check

2016-10-16 Thread Axel Davy
I've pushed this just now to be sure it makes it for mesa 13. On 16/10/2016 11:47, Axel Davy wrote: Fixes regression introduced by b5603056872708fdd82f1224854097805a01d4c0 The regression prevents some apps to start. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_tr

Re: [Mesa-dev] [PATCH 05/16] loader: reimplement loader_get_user_preferred_fd via libdrm

2016-10-12 Thread Axel Davy
On 12/10/2016 19:15, Emil Velikov wrote: On 11 October 2016 at 19:54, Axel Davy <axel.d...@ens.fr> wrote: On 11/10/2016 20:31, Emil Velikov wrote: - udev = udev_new(); - if (!udev) - goto prime_clean; + if (drmGetDevice(default_fd, ) != 0) + go

[Mesa-dev] [PATCH 2/2] st/nine: Fix possible segfault in surface ctor

2016-10-12 Thread Axel Davy
our of previous code. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/surface9.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/surface9.c b/src/gallium/state_trackers/nine/surface9.c index dc31bb9..664b7

[Mesa-dev] [PATCH 1/2] st/nine: Remove useless code in nine_shader

2016-10-12 Thread Axel Davy
Since 1604efa6fda9b780e8537a131ad77f3e83e5a67a, lconsti and lconstb don't need to be initialized. Remove some leftovers from the previous code (which has now invalid use of ARRAY_SIZE on a pointer instead of an array). Reported by Coverity. Signed-off-by: Axel Davy <axel.d...@ens.fr> --

Re: [Mesa-dev] [PATCH v2 05/16] loader: reimplement loader_get_user_preferred_fd via libdrm

2016-10-14 Thread Axel Davy
eserve the current functionality for libudev users and allow others to benefit from it as well. v2: Drop ranty comments, pick the correct device Cc: Axel Davy <axel.d...@ens.fr> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- src/loa

[Mesa-dev] [PATCH 24/84] st/nine: Back User Clip Planes 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 5 - src/gallium/state_trackers/nine/nine_state.

[Mesa-dev] [PATCH 20/84] st/nine: Refactor LightEnable

2016-12-07 Thread Axel Davy
Call a helper function. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 25 +--- src/gallium/state_trackers/nine/nine_state.c | 35 src/gallium/state_trackers/nine/nine_state.h | 4 3 files c

[Mesa-dev] [PATCH 08/84] st/nine: Move vtxbuf 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 33 -- src/gallium/state_trackers/nine/nine_s

[Mesa-dev] [PATCH 25/84] st/nine: Track dirty state groups in 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/basetexture9.c | 3 +- src/gallium/state_trackers/nine/device9.c

[Mesa-dev] [PATCH 22/84] st/nine: Back ds 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 7 +++--- src/gallium/state_trackers/nine/nine_state.

[Mesa-dev] [PATCH 12/84] st/nine: Back sampler states 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 18 +++--- src/gallium/state_trackers/nine/nine_state.c

[Mesa-dev] [PATCH 17/84] st/nine: Back viewport 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 4 ++- src/gallium/state_trackers/nine/nine_ff.c| 2 +

[Mesa-dev] [PATCH 01/84] st/nine: Introduce nine_context

2016-12-07 Thread Axel Davy
nine_context is a new structure which goal will be to contain all internal states. It will be the states of the second thread in the to-be-introduced CSMT mode. This patch moves several internal states to nine_context, while the next patches add the other fields. Signed-off-by: Axel Davy <axe

[Mesa-dev] [PATCH 30/84] st/nine: Rename cso in nine_context to cso_shader

2016-12-07 Thread Axel Davy
This patch it to avoid name conflict when device->cso is moved to nine_context. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_state.c| 12 ++-- src/gallium/state_trackers/nine/nine_state.h| 2 +- src/gallium/state_trackers/nine/pix

[Mesa-dev] [PATCH 03/84] st/nine: Move draw calls 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 209 +++-- src/gallium/state_tracker

[Mesa-dev] [PATCH 21/84] st/nine: Back all ff states in 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 60 -- src/gallium/state_trackers/nine/nin

[Mesa-dev] [PATCH 37/84] st/nine: Decompose nine_context_set_indices

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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_state.c | 32 +--- 1 file changed, 24 inse

[Mesa-dev] [PATCH 62/84] st/nine: Add CSMT_NO_WAIT_WITH_COUNTER

2016-12-07 Thread Axel Davy
From: Patrick Rudolph Similar to the other macros, but introduces a counter, which enables to know when the instructions has been executed. Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/nine_csmt_helper.h | 54 ++

[Mesa-dev] [PATCH 60/84] st/nine: Optimize ColorFill

2016-12-07 Thread Axel Davy
When we lock the whole surface to overwrite it, we can use DISCARD. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/g

[Mesa-dev] [PATCH 26/84] st/nine: Use atomics for nine_bind

2016-12-07 Thread Axel Davy
nine_bind didn't need atomics up to now, because it's use what always within a protected mutex. We need to use atomics because with the next patches several threads may use nine_bind. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/iunknown.h | 5 +++--

[Mesa-dev] [PATCH 65/84] st/nine: Fix NineUnknown_Detach

2016-12-07 Thread Axel Davy
We don't bind the container in AddRef. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/iunknown.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/iunknown.h b/src/gallium/state_trackers/nine/iunknown.h

[Mesa-dev] [PATCH 10/84] st/nine: Back vdecl 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 21 +-- src/gallium/state_tracker

[Mesa-dev] [PATCH 34/84] st/nine: Move Managed Pool handling out of 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 57 src/gallium/state_tracker

[Mesa-dev] [PATCH 28/84] st/nine: Remove NineDevice9_GetCSO

2016-12-07 Thread Axel Davy
Was useless. Remove useless usage in swapchain9. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 6 -- src/gallium/state_trackers/nine/device9.h| 3 --- src/gallium/state_trackers/nine/swapchain9.c | 1 - src/gallium/state_tracker

[Mesa-dev] [PATCH 15/84] st/nine: Back RT 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 16 +++- src/gallium/state_trackers/nine/nine_state.c

[Mesa-dev] [PATCH 09/36] st/nine: Remove useless check in surface9 ctor

2016-12-07 Thread Axel Davy
Textures already have the check in BaseTexture9. Non-Textures cannot be in the MANAGED Pool. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/surface9.c | 4 1 file changed, 4 deletions(-) diff --git a/src/gallium/state_trackers/nine/surface9.c b/src/g

[Mesa-dev] [PATCH 24/36] driconf: Fix missing gettext

2016-12-07 Thread Axel Davy
DRI_CONF_NINE_OVERRIDEVENDOR was missing gettext for the description. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/mesa/drivers/dri/common/xmlpool/t_options.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/common/xmlpool/t_options.h b/sr

[Mesa-dev] [PATCH 06/36] st/nine: Fix ps multisample check

2016-12-07 Thread Axel Davy
We want to use centroid for nonmaskable multisampling as well. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/pixelshader9.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/pixelshader9.h b/src/g

[Mesa-dev] [PATCH 15/36] st/nine: Correctly release sw cursor image

2016-12-07 Thread Axel Davy
cursor.image is used for software cursor emulation. It wasn't released. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_tracker

[Mesa-dev] [PATCH 13/36] st/nine: Avoid crash on empty Draw*Up

2016-12-07 Thread Axel Davy
Ignore empty draw calls. Avoid assertion fault when such draw calls happen in u_upload_mgr. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/g

[Mesa-dev] [PATCH 03/36] st/nine: Remove useless call to nine_update_state

2016-12-07 Thread Axel Davy
The call was not needed. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index 055e71d..7bdf19f 100644 ---

[Mesa-dev] [PATCH 67/84] st/nine: Comment and simplify iunknown

2016-12-07 Thread Axel Davy
The behaviour is a bit less obscure now. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/iunknown.c | 9 ++--- src/gallium/state_trackers/nine/iunknown.h | 30 -- 2 files changed, 18 insertions(+), 21 deletions(-) diff --git

[Mesa-dev] [PATCH 63/84] st/nine: Avoid flushing the queue for queries GetData

2016-12-07 Thread Axel Davy
Use the newly introduced counter to know when we don't need synchronization. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_state.c | 33 +++- src/gallium/state_trackers/nine/nine_state.h | 6 ++--- src/gallium/state_tracker

[Mesa-dev] [PATCH 33/84] st/nine: Integrate nine_pipe_context_clear to nine_context_clear

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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 7 ++- src/gallium/state_trackers/nine/device9ex.c

[Mesa-dev] [PATCH 55/84] st/nine: Add nine_context_get_pipe_acquire/release

2016-12-07 Thread Axel Davy
See commit for description. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_queue.c | 9 src/gallium/state_trackers/nine/nine_queue.h | 3 ++ src/gallium/state_trackers/nine/nine_state.c | 67 src/gallium/state_tr

[Mesa-dev] [PATCH 41/84] st/nine: Back swvp in 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 <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 3 ++- src/gallium/state_trackers/nine/nine_state.c

<    3   4   5   6   7   8   9   10   11   >