[Mesa-dev] [PATCH 29/45] st/nine: Test more presentation params

From: Patrick Rudolph <s...@das-labor.org> Return errors in case of invalid presentation parameters. Fixes failing WINE tests device.c test_swapchain_parameters(). Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/s

[Mesa-dev] [PATCH 38/45] st/nine: Add Render state validation layer

optimisation: (Value == previous Value) => return immediately, which affects D3D9 hacks too. 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 | 33 ++-- src/gallium/state_trackers/nine/nine

[Mesa-dev] [PATCH 27/45] st/nine: Print GUID instead of pointer

From: Patrick Rudolph <s...@das-labor.org> To ease debugging print the GUID instead of the pointer to it. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/guid.c | 18 +

[Mesa-dev] [PATCH 16/45] st/nine: Catch redundant SetStreamSourceFreq calls

Some apps do redundant SetStreamSourceFreq calls. Catch them to improve performance. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/device9.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a

[Mesa-dev] [PATCH 35/45] st/nine: Handle Window Occlusion

pchain we do care only of the device one. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- include/D3D9/d3d9types.h | 1 + include/d3dadapter/present.h | 3 ++ src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 20/45] st/nine: Fix shade mode flat

From: Patrick Rudolph <s...@das-labor.org> Shade mode flat is only working if pixelshaders have interpolate set to TGSI_INTERPOLATE_COLOR on color inputs. Fixes failing WINE tests visual.c test_shademode(). Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel

[Mesa-dev] [PATCH 19/45] st/nine: Clear rendertarget on creation

From: Patrick Rudolph <s...@das-labor.org> Clear every rendertarget on creation. Fixes https://github.com/iXit/Mesa-3D/issues/139 Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/surface9.c | 9

[Mesa-dev] [PATCH 30/45] st/nine: Fix possible crash on error

From: Patrick Rudolph <s...@das-labor.org> In case swapchain creation fails This->swapchains[i] might be NULL and causes a crash. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c |

[Mesa-dev] [PATCH 21/45] st/nine: Use cso for viewport

From: Patrick Rudolph <s...@das-labor.org> Use CSO to catch redundant viewport changes. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_state.c | 3 +-- 1 file changed, 1 insertion(+), 2

[Mesa-dev] [PATCH 26/45] st/nine: Fix use of uninitialized memory

From: Patrick Rudolph <s...@das-labor.org> The values of box.z and box.depth weren't set and lead to a crash. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/auxiliary/util/u_box.h | 12 +++

[Mesa-dev] [PATCH 34/45] st/nine: Store minor version num

From: Patrick Rudolph <s...@das-labor.org> To keep compatible with older ID3DPresent interfaces (used to talk with Wine), store the minor version num accessible to all statetracker functions (in the NineDevice9 structure). Signed-off-by: Patrick Rudolph <s...@das-labor.org> Revie

[Mesa-dev] [PATCH 41/45] st/nine: Use fast clears more often for MRTs

of 4. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/device9.c| 2 +- src/gallium/state_trackers/nine/nine_state.c | 18 +- src/gallium/state_trackers/nine/nine_state.h | 2 +-

[Mesa-dev] [PATCH 17/45] st/nine: Introduce STREAMFREQ state

te would happen. This patch solves the problem by adding a new NINE_STATE_STREAMFREQ state. Another way would be to add state->changed.stream_freq & ~1 check to the main test. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/sta

[Mesa-dev] [PATCH 22/45] st/nine: Support D3DFMT_R8G8B8

nd its name is likely wrong (reversed). Fixes a crash in TmNationsForever. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_pipe.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff -

[Mesa-dev] [PATCH 06/45] st/nine: Fix ff calculation of midVec

From: Patrick Rudolph <s...@das-labor.org> In case of non local viewer the value has to be subtracted. Fixes failing WINE tests in test_specular_lighting() (visual.c) Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gall

[Mesa-dev] [PATCH 28/45] st/nine: Fix resource9 private data

https://github.com/iXit/Mesa-3D/issues/130 Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_pdata.h | 1 + src/gallium/state_trackers/nine/resource9.c | 3 ++- 2 files changed, 3 insertions(+), 1 delet

[Mesa-dev] [PATCH 32/45] st/nine: Fix remaining swapchain tests

Buffer, but not in NineSwapChain9_GetBackBuffer. This fixes all WINE's device.c test_swapchain() tests. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 3 +++ src/gallium/state_trackers/nine/swapchain9.c | 3 ++- 2 files cha

[Mesa-dev] [PATCH 24/45] st/nine: Fix Volumetexture9_LockBox

From: Patrick Rudolph <s...@das-labor.org> Check for valid locked box dimensions. Fixes failing wine tests device.c test_lockbox_invalid. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/volume

[Mesa-dev] [PATCH 40/45] st/nine: Use linear filtering for shadow mapping

Some docs say linear filtering is always used when app does shadow mapping. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_pipe.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[Mesa-dev] [PATCH 23/45] st/nine: Fix ATI2 pitch for non-square

Fixes crash for non-square textures. We were using the height instead of the width for some calculations. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/surface9.c | 4 ++-- 1 file changed, 2 inse

[Mesa-dev] [PATCH 18/45] st/nine: Allow ColorFill on D3DFMT_NULL surfaces

From: Patrick Rudolph <s...@das-labor.org> Report success instead of failing as there's no resource for those surfaces. Fixes a crash in Crysis: Warhead. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_track

[Mesa-dev] [PATCH 10/45] st/nine: Initialize lights in stateblocks

This fixes a crash. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/stateblock9.c | 4 1 file changed, 4 insertions(+) diff --git a/src/gallium/state_trackers/nine/stateblock9.c b/src/gallium/st

[Mesa-dev] [PATCH 33/45] st/nine: Call flush_resource before flush

flush_resource needs to be called before flush (for fast clear resolve, etc). Removes useless computation of resource (it is already set correctly). Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/

[Mesa-dev] [PATCH 44/45] st/nine: Disallow non-argb8888 cursors

Only argb cursors are allowed. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/device9.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/nine/device9.c b

[Mesa-dev] [PATCH 13/45] st/nine: Use ff vertex shader when position_t is used

. Up to know we were checking if device->state.vs is set to know whether to use programmable shader or not. With this commit we determine whether we use programmable shader or not when vertex shader/declaration are set, but stateblocks do complicate things a bit. Signed-off-by: Axel Davy <

[Mesa-dev] [PATCH 07/45] st/nine: Replace r[0] with tmp

From: Patrick Rudolph <s...@das-labor.org> Replace r[0] with tmp to ease code reading. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 28 ++-- 1 file chan

[Mesa-dev] [PATCH 09/45] st/nine: Fix fixed-function blendweights

From: Patrick Rudolph <s...@das-labor.org> The last weighted element is one minus the sum of all previous weights. Fixes WINE test visual.c test_vertex_blending. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/s

[Mesa-dev] [PATCH 25/45] st/nine: Fix clear for multisample mismatch depth-stencil

gt; Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index ecb33a8..325f5e8 100644

[Mesa-dev] [PATCH 14/45] st/nine: Unset vtxbuf on reset

We forgot to reset vtxbuf. This fixes some crashes. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_state.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/st

[Mesa-dev] [PATCH 36/45] st/nine: Implement NineDevice9_GetAvailableTextureMem

uot;Vampire: The Masquerade - Bloodlines" allocating resources until crash. Fixes "Age of Conan" allocating resources until crash. Fixes failing WINE test device.c test_vidmem_accounting(). Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.

[Mesa-dev] [PATCH 42/45] st/nine: Fix centroid flag

sem.reg.mod & NINED3DSPDM_CENTROID is worth 4 when centroid is requested, whereas TGSI_INTERPOLATE_LOC_CENTROID is worth 1. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_shader.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff -

[Mesa-dev] [PATCH 37/45] DRI_CONFIG: Add option to override vendor id

From: Patrick Rudolph <s...@das-labor.org> Add config option override_vendorid to report a fake card in d3dadapter9 drm. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/targets/d3dadapter9/dr

[Mesa-dev] [PATCH 11/45] st/nine: Change StretchRect check order

Textures in SYSTEMMEM don't have resources attached. Instead of returning an error for them, StretchRect was crashing. This changes the check order to fix that case. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_t

[Mesa-dev] [PATCH 31/45] st/nine: Fix crash NineDevice9_CreateAdditionalSwapChain

When no window is specified, we should revert to the focus window. This deserves more tests however (what if the device swapchain is already using the focus window ?) Fixes crash for FFXIV Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org&

[Mesa-dev] [PATCH 08/45] st/nine: Always normalize hitDir

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/nine_ff.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/gallium/s

[Mesa-dev] [PATCH 15/45] st/nine: Squash indexbuffer9 and vertexbuffer9

From: Patrick Rudolph <s...@das-labor.org> The indexbuffer9 codebase was lagging behind the one of vertexbuffer9. Add buffer9 as common code base for indexbuffer9 and vertexbuffer9. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr>

[Mesa-dev] [PATCH 39/45] st/nine: Respect block alignment on surface lock

From: Patrick Rudolph <s...@das-labor.org> Respect block alignment for ATI1/ATI2 format when trying to lock a surface using LockRect(). Fixes failing WINE tests device.c test_surface_blocks() tests. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <a

[Mesa-dev] [PATCH 00/45] New Gallium Nine fixes

. Some rendering issues are gone . A new drirc option to advertise a card from another vendor (for example NVidia owners should advertise AMD for Skyrim in order to get water working properly) Axel Davy (18): st/nine: Fix D3DPMISCCAPS_FOGANDSPECULARALPHA support st/nine: Fix rounding issue

[Mesa-dev] [PATCH 05/45] st/nine: Implement D3DRS_SPECULARENABLE

From: Patrick Rudolph <s...@das-labor.org> Implement fixed function D3DRS_SPECULARENABLE. Fixes failing WINE tests in test_specular_lighting() (visual.c) Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_track

[Mesa-dev] [PATCH 01/45] st/nine: Fix AlphaCmpCaps

From: Patrick Rudolph <s...@das-labor.org> AlphaCmpCaps should advertise D3DPCMPCAPS_NEVER as well. Fixes https://github.com/iXit/Mesa-3D/issues/142 Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/n

[Mesa-dev] [PATCH 04/45] st/nine: Fix D3DRS_LOCALVIEWER being ignored

From: Patrick Rudolph <s...@das-labor.org> Set key->localviewer to D3DRS_LOCALVIEWER. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 2 ++ 1 file changed, 2 insertions(+) dif

Re: [Mesa-dev] [PATCH] gallium/radeon: always unmap textures on 32-bit

mapped, you get the optimization, else for most resources, which get mapped once or two times, you don't keep the additionnal memory storage. Yours, Axel Davy On 01/02/2016 15:20, Marek Olšák wrote: From: Marek Olšák <marek.ol...@amd.com> This might fix mmap errors with Natural Selec

[Mesa-dev] [PATCH 2/2] st/nine: Unset vtxbuf on reset

We forgot to reset vtxbuf. This fixes some crashes. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_state.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/st

[Mesa-dev] [PATCH 1/2] st/nine: Use pipe_resource_reference for vtxbuf

This seems cleaner to actually reference the resources for vtxbuf, rather than relying on the fact the bound d3d streams do. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 3 ++- src/gallium/state_trackers/nine/stateblock9.c | 4 ++-- 2

Re: [Mesa-dev] [PATCH 00/45] New Gallium Nine fixes

Thanks a lot ! On 29/01/2016 17:59, Ilia Mirkin wrote: FWIW I looked over these quickly and sent a very small amount of feedback. I didn't really see anything too objectionable, but I'm also not an expert on any of this stuff. On Fri, Jan 29, 2016 at 11:08 AM, Axel Davy <axel.d...@ens

Re: [Mesa-dev] [PATCH 19/45] st/nine: Clear rendertarget on creation

. -ilia On Fri, Jan 29, 2016 at 11:08 AM, Axel Davy <axel.d...@ens.fr> wrote: From: Patrick Rudolph <s...@das-labor.org> Clear every rendertarget on creation. Fixes https://github.com/iXit/Mesa-3D/issues/139 Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by:

Re: [Mesa-dev] [PATCH 14/45] st/nine: Unset vtxbuf on reset

On 29/01/2016 17:49, Ilia Mirkin wrote: On Fri, Jan 29, 2016 at 11:08 AM, Axel Davy <axel.d...@ens.fr> wrote: We forgot to reset vtxbuf. This fixes some crashes. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/s

[Mesa-dev] [PATCH 2/3] st/nine: Fix Multithreading issue with MANAGED buffers

of things to upload (We move it from Lock time to Unlock time). Cc: "11.2" <mesa-sta...@lists.freedesktop.org> Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/buffer9.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) di

[Mesa-dev] [PATCH 1/3] st/nine: Handle READONLY for buffer MANAGED pool

READONLY won't trigger an upload. Cc: "11.2" <mesa-sta...@lists.freedesktop.org> Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/buffer9.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/src/galli

[Mesa-dev] [PATCH 0/3] gallium nine MANAGED buffers fixed

ll needing the data, implementing the behaviour would also fix the problem. However as it needs a bit of work, I plan to merge for now these fixes, and implement later the correct behaviour. Axel Davy (3): st/nine: Handle READONLY for buffer MANAGED pool st/nine: Fix Multithreading issue wi

[Mesa-dev] [PATCH 3/3] st/nine: Fix second Multithreading issue with MANAGED buffers

will then upload the data and unset the dirty flag, and the Unlock won't trigger a second upload. Fixes regression introduced by cc0114f30b587a10766ec212afb3ad356099ef23: "st/nine: Implement Managed vertex/index buffers" Cc: "11.2" <mesa-sta...@lists.freedesktop.org> Signed-off-by: A

[Mesa-dev] [PATCH 3/3 v2] st/nine: Fix second Multithreading issue with MANAGED buffers

reedesktop.org> Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/buffer9.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/state_trackers/nine/buffer9.c b/src/gallium/state_trackers/nine/buffer9.c index 6d5d9d6..1103741 100644 --- a/src/gal

Re: [Mesa-dev] [PATCH 1/2] gallium: add usage flags to resource_get_handle

On 25/02/2016 00:10, Marek Olšák wrote: From: Marek Olšák This will help differentiate between window-system framebuffers and other buffers. PIPE_HANDLE_USAGE_EXPLICIT_FLUSH should yield higher performance (e.g. allows delayed clears), but PIPE_HANDLE_USAGE_COHERENT is

[Mesa-dev] [PATCH] radeonsi: Add option for SI scheduler

Add a debug option to select the LLVM SI Machine Scheduler. R600_DEBUG=sisched Signed-off-by: Axel Davy <axel.d...@ens.fr> --- The corresponding llvm patch is on llvm master, and should land soon for 3.8 branch src/gallium/drivers/radeon/r600_pipe_common.c | 1 + src/gallium/drivers/

Re: [Mesa-dev] [PATCH 6/7] radeonsi: use all SPI color formats

On 19/01/2016 17:11, Marek Olšák wrote: From: Marek Olšák because not using SPI_SHADER_32_ABGR doubles fill rate. We should also get optimal performance if alpha isn't needed or blending isn't enabled. --- src/gallium/drivers/radeon/r600_pipe_common.h | 6 +-

Re: [Mesa-dev] [PATCH] egl/wayland: Set __DRI_IMAGE_USE_SCANOUT for shared buffers

There was this other version http://lists.freedesktop.org/archives/mesa-dev/2015-October/097917.html Reviewed-by: Axel Davy <axel.d...@ens.fr> On 27/01/2016 10:34, Michel Dänzer wrote: From: Michel Dänzer <michel.daen...@amd.com> The compositor may have the hardware scan out d

Re: [Mesa-dev] [PATCH] egl/wayland: Set __DRI_IMAGE_USE_SCANOUT for shared buffers

Hi, On 27/01/2016 13:43, Daniel Stone wrote: Hi, On 27 January 2016 at 09:34, Michel Dänzer wrote: The compositor may have the hardware scan out directly from the buffers sent by the client, so we must make sure the buffers we create are suitable for scanout. If the

[Mesa-dev] [PATCH] st/nine: Forbid POSITION0 and fix depth input for PS3.0

POSITION0 input is forbidden for PS3.0 apparently. Besides for depth input we weren't using the system value version if needed Signed-off-by: Axel Davy <axel.d...@ens.fr> --- Could be better having it separated into two patches perhaps. It should get into Mesa 11.2, to prevent regression

Re: [Mesa-dev] [PATCH 00/12] Some more Nine fixes

On 09/02/2016 14:58, Emil Velikov wrote: On 7 February 2016 at 23:13, Axel Davy <axel.d...@ens.fr> wrote: A few more patches I'd like to get in 11.2. There a few cleanup patches and some fixes. The last patch fixes build with llvm 32 bits when it isn't built with -mstackrealign. Bas

[Mesa-dev] [PATCH 2/2] st/nine: Use position input helper for ps3 declared inputs

When the semantic is position (which can happen with index 0 only), use the helper to get position input. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- The other changes of the initial patch (forbid POSITION0, etc) will be part of another serie. src/gallium/state_trackers/nine/nine_sh

[Mesa-dev] [PATCH 1/2] st/nine: Introduce helper for Position shader input

Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_shader.c | 31 +-- 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_shader.c b/src/gallium/state_trackers/nine/nine_shader.c

Re: [Mesa-dev] [PATCH 2/2] gallium: add external_usage flags to resource_from(get)_handle (v2)

;layer = 0; img->dri_format = format; img->dri_components = 0; + img->use = use; This looks ok to me (except the minor nitpick). You can add my Rb: Reviewed-by: Axel Davy <axel.d...@ens.fr> For the first patch, I think it would be better to document som

Re: [Mesa-dev] [RFC PATCH] GBM: Add map/unmap functions

On 31/03/2016 05:21, Rob Herring wrote : int (*getCapabilities)(__DRIscreen *screen); + + /** +* Lock a part of a __DRIimage for specified usage +* +* flush_flag: +*0: no flush +*__BLIT_FLAG_FLUSH: flush after the blit operation +*

Re: [Mesa-dev] [PATCH] st/nine: specify WINAPI only for i386 and amd64

Hi, This looks good to me, but I'm not expert it compilers at all, could someone with more experience look at it ? Axel On 17/03/2016 20:12, Christian Schmidbauer wrote : Currently mesa fails building with the x32 abi as ms_abi is not defined in such a case. The patch uses ms_abi only for

[Mesa-dev] [PATCH 1/3] r600: Change default behaviour for undefined COLOR0

d3d 9 needs COLOR0 to be 1.0 on all channels when undefined. 0.0 for the others is fine. GL behaviour is undefined. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/r600/r600_state.c | 4 1 file changed, 4 insertions(+) diff --git a/src/gallium/drivers/r600/r600_s

[Mesa-dev] [PATCH 2/3] radeonsi: Change default behaviour for undefined COLOR0

d3d 9 needs COLOR0 to be 1.0 on all channels when undefined. 0.0 for the others is fine. GL behaviour is undefined. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/radeonsi/si_state_shaders.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/ra

[Mesa-dev] [PATCH 3/3] radeonsi: Mixed colorbuffer formats are unsupported

Besides depth/stencil, the hardware doesn't support mixed formats. The GL state tracker doesn't make use of them. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/radeonsi/si_state.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/src/gallium/drivers/ra

Re: [Mesa-dev] [PATCH 1/3] r600: Change default behaviour for undefined COLOR0

On 03/04/2016 19:37, Marek Olšák wrote: r600_state.c only applies to r600-r700. evergreen_state.c is for EG and NI. Did you want to update that as well? Marek Thanks for pointing that out, I'll write a patch for EG and NI as well. Axem ___ mesa-dev

[Mesa-dev] [PATCH] r600g: Change default behaviour for undefined COLOR0

d3d 9 needs COLOR0 to be 1.0 on all channels when undefined. 0.0 for the others is fine. GL behaviour is undefined. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/r600/evergreen_state.c | 4 1 file changed, 4 insertions(+) diff --git a/src/gallium/driver

[Mesa-dev] [PATCH] radeon: Change AA sample locations for EG+

This sets the AA location to the d3d11 spec. EG/NI 8X MSAA is left as is. Not sure why it was set different to Cayman, so let it as is. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/radeon/cayman_msaa.c | 68 1 file changed, 34 inse

Re: [Mesa-dev] [PATCH 1/3] r600: Change default behaviour for undefined COLOR0

On 03/04/2016 20:41, Roland Scheidegger wrote : Am 03.04.2016 um 10:11 schrieb Axel Davy: d3d 9 needs COLOR0 to be 1.0 on all channels when undefined. 0.0 for the others is fine. GL behaviour is undefined. Not that I care what drivers use by default for undefined values, but it probably would

Re: [Mesa-dev] [PATCH 1/3] r600: Change default behaviour for undefined COLOR0

On 04/04/2016 18:48, Roland Scheidegger wrote: Am 04.04.2016 um 17:27 schrieb Axel Davy: So is that ok to you for now to update the radeon behaviour ? As I said, you can do what you want there as far as I'm concerned. Just saying it's going to be a constant battle to fix other drivers

Re: [Mesa-dev] [PATCH 1/3] r600: Change default behaviour for undefined COLOR0

Le 04/04/2016 15:27, Roland Scheidegger a écrit : Am 04.04.2016 um 14:04 schrieb Axel Davy: On 03/04/2016 20:41, Roland Scheidegger wrote : Am 03.04.2016 um 10:11 schrieb Axel Davy: d3d 9 needs COLOR0 to be 1.0 on all channels when undefined. 0.0 for the others is fine. GL behaviour

Re: [Mesa-dev] [PATCH] gallium/radeon: handle vertex shaders that disable clipping & viewport

Unfortunately, that doesn't fix the issue. On 13/04/2016 17:37, Marek Olšák wrote: From: Marek Olšák This should fix Nine. --- src/gallium/drivers/radeon/r600_pipe_common.h | 1 + src/gallium/drivers/radeon/r600_viewport.c| 20 +++- 2 files

Re: [Mesa-dev] [PATCH 03/39] llvmpipe: Fix overflow for 32 bits available memory computation

On 16/05/2016 20:08, Emil Velikov wrote: Then there's the question if other drivers want to apply, and thus move, it elsewhere. -Emil Has anyone an opinion about that particular point ? Axel ___ mesa-dev mailing list

[Mesa-dev] [PATCH] d3dadapter9: Add ddebug, rbug and trace support

Add support for ddebug, rbug and trace Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/targets/d3dadapter9/Makefile.am | 9 + 1 file changed, 9 insertions(+) diff --git a/src/gallium/targets/d3dadapter9/Makefile.am b/src/gallium/targets/d3dadapter9/Makefile.am index 8

[Mesa-dev] [PATCH] gallium/util: Implement util_format_translate_3D

This is the equivalent of util_format_translate, but for volumes. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- Emil, I hope I understood what you wanted correctly. src/gallium/auxiliary/util/u_format.c | 34 ++ src/gallium/auxiliary/util/u_format.

Re: [Mesa-dev] [PATCH 00/39] Some nine patches for Mesa 12.0

Hi, I plan to merge the patches when freedesktop comes back, except d3dadapter9: Add d3dadapter9.pc which needs more work and llvmpipe: Fix overflow for 32 bits available memory computation for which discussions aren't finished. Axel On 15/05/2016 12:45, Axel Davy wrote: Hi, Here

Re: [Mesa-dev] [PATCH] gallium/util: Implement util_format_translate_3D

On 18/05/2016 22:49, Emil Velikov wrote: On 18 May 2016 at 21:33, Axel Davy <axel.d...@ens.fr> wrote: This is the equivalent of util_format_translate, but for volumes. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- Emil, I hope I understood what you wanted correctly. Imho the f

Re: [Mesa-dev] [PATCH 00/14] vl dri3 support for vaapi and vdpau

Hi, Do you have some local branch to review all at once (it is a bit hard to follow with the patches) ? From a quick looks, it seems you inspired from the loader dri3 code. There is also another implementation you can inspire from:

Re: [Mesa-dev] [PATCH 06/14] vl/dri3: add back buffers support

On 11/05/2016 17:06, Leo Liu wrote: This implements DRI3 PixmapFromBuffer. Create buffer objects, and associate it to a dma-buf fd, and then pass this fd with a pixmap ID to X server for creating pixmap object; also add a function for wait events. Signed-off-by: Leo Liu ---

Re: [Mesa-dev] [PATCH 13/14] vl/dri3: implement functions for get and set timestamp

On 11/05/2016 23:08, Leo Liu wrote: scrn->next_msc = ((int64_t)stamp - scrn->last_ust + scrn->ns_frame/2) / + scrn->ns_frame + scrn->last_msc; Could you explain this calculation ? I think it may get issues if ns_frame is wrong. For example for some reason (app hidden

Re: [Mesa-dev] [PATCH 13/14] vl/dri3: implement functions for get and set timestamp

On 11/05/2016 23:31, Leo Liu wrote: On 05/11/2016 05:18 PM, Axel Davy wrote: On 11/05/2016 23:08, Leo Liu wrote: scrn->next_msc = ((int64_t)stamp - scrn->last_ust + scrn->ns_frame/2) / + scrn->ns_frame + scrn->last_msc; Could you explain this calculat

Re: [Mesa-dev] [PATCH 02/14] vl/dri3: implement dri3 screen create and destroy

On 11/05/2016 17:06, Leo Liu wrote: Screen created with device fd returned from X server, also will bail out to DRI2 with certain conditions. Signed-off-by: Leo Liu --- configure.ac | 7 ++- src/gallium/auxiliary/vl/vl_winsys_dri3.c | 88

Re: [Mesa-dev] [PATCH 00/14] vl dri3 support for vaapi and vdpau

of the kernel details, but as I understand radeon and amdgpu support the kernel interface for dma-buf fences, but intel not. You should be looking to the intel team to implement the feature. Axel On 12/05/2016 07:46, Axel Davy wrote: On 12/05/2016 04:41, Mike Lothian wrote: Hi Axel

Re: [Mesa-dev] GBM backend dynamic dispatch method

Hi, I'm not sure if that answers your need, but to me it looks like it would be simpler if gbm was trying different backends for the given device, until it succeeds. You'd pass several backends to GBM_BACKEND (perhaps rename to GBM_BACKENDS), and in your example it would put amdgpu pro first.

Re: [Mesa-dev] GBM backend dynamic dispatch method

backend. So I just have to make amdgpu-pro backend create_device return NULL for Intel GPU. Many thanks, Qiang *From:* mesa-dev <mesa-dev-boun...@lists.freedesktop.org> on behalf of Axel Davy <axel.d...@ens.f

Re: [Mesa-dev] [PATCH 03/39] llvmpipe: Fix overflow for 32 bits available memory computation

On 16/05/2016 20:08, Emil Velikov wrote: On 15 May 2016 at 11:45, Axel Davy <axel.d...@ens.fr> wrote: And cap to 2 GB on 32 bits. Fixes https://bugs.freedesktop.org/show_bug.cgi?id=94561 Nitpick: please use "Bugzilla: htt.." Signed-off-by: Axel Davy <axel.d...@ens.fr

Re: [Mesa-dev] [PATCH 03/39] llvmpipe: Fix overflow for 32 bits available memory computation

On 16/05/2016 23:28, Rob Herring wrote: On Sun, May 15, 2016 at 5:45 AM, Axel Davy <axel.d...@ens.fr> wrote: 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> --- src/gallium/auxiliary/os/os_misc

Re: [Mesa-dev] [PATCH 11/39] st/nine: Increase minor d3dadapter9drm ABI

On 16/05/2016 20:26, Emil Velikov wrote: On 15 May 2016 at 11:45, Axel Davy <axel.d...@ens.fr> wrote: Version 0.1 allows to assume that the second element of the IDirect3D* structures will be a pointer to the internal nine vtable. This is useful if the gallium nine user wants to wra

Re: [Mesa-dev] [PATCH 03/39] llvmpipe: Fix overflow for 32 bits available memory computation

On 17/05/2016 20:16, Rob Herring wrote: On Tue, May 17, 2016 at 11:35 AM, Axel Davy <axel.d...@ens.fr> wrote: On 16/05/2016 23:28, Rob Herring wrote: On Sun, May 15, 2016 at 5:45 AM, Axel Davy <axel.d...@ens.fr> wrote: And cap to 2 GB on 32 bits. Fixes https://bugs.fre

Re: [Mesa-dev] [PATCH 28/39] st/nine: Fix buffer bind flags

On 16/05/2016 07:25, Ilia Mirkin wrote: On May 15, 2016 6:46 AM, "Axel Davy" <axel.d...@ens.fr <mailto:axel.d...@ens.fr>> wrote: > > Signed-off-by: Axel Davy <axel.d...@ens.fr <mailto:axel.d...@ens.fr>> > --- > src/gallium/state_trackers/

Re: [Mesa-dev] [PATCH 06/14] vl/dri3: add back buffers support

need the thread safety part for you. Basically the idea is that when only one pixmap hasn't been released, you send it again with the copy flag, which garantees it will get released. Axel On 11/05/2016 20:29, Axel Davy wrote: On 11/05/2016 17:06, Leo Liu wrote: This implements DRI3

Re: [Mesa-dev] [PATCH 06/14] vl/dri3: add back buffers support

vl_dri3_screen_texture_from_drawable return a texture on the last back buffer sent instead ? I don't know what vl_dri3_screen_texture_from_drawable is supposed to do, so perhaps I'm wrong. On 11/05/2016 20:42, Axel Davy wrote: Another thing is that I think there is no guarantee the Xserver releases all the pixmaps

Re: [Mesa-dev] [PATCH 00/14] vl dri3 support for vaapi and vdpau

On 12/05/2016 04:41, Mike Lothian wrote: Hi Axel Is the thread_submit=true only for nine or does it work with all of DRI3? I'm keen to get rid of the tearing on my Skylake/Tonga setup Thanks Mike It is gallium nine only for now. In the case of gallium nine, it as a reasonnable assumption

Re: [Mesa-dev] [PATCH 13/14] vl/dri3: implement functions for get and set timestamp

Hi, The present extension has something exactly to set the target ust for the presentation: PresentOptionUST Unfortunately, while it is in the spec it looks like the option is totally ignored, and thus it will be totally buggy (you are supposed to pass ust instead of msc...). However

[Mesa-dev] [PATCH 16/39] st/nine: Add format fallback with conversion to surfaces

Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/surface9.c | 70 +- src/gallium/state_trackers/nine/surface9.h | 3 ++ 2 files changed, 72 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/surface9.c

[Mesa-dev] [PATCH 25/39] st/nine: Use WINE thread for threadpool

From: Patrick Rudolph <s...@das-labor.org> Use present interface 1.2 function ID3DPresent_CreateThread to create the thread for threadpool. Creating the thread with WINE prevents some rarely occuring crashes. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel

[Mesa-dev] [PATCH 24/39] st/nine: Don't present if window is occluded

after occlusion check to return and prevent Xpixmaps errors. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/swapchain9.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff -

[Mesa-dev] [PATCH 17/39] st/nine: Add format fallback with conversion to volumes

Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/volume9.c | 115 +++--- src/gallium/state_trackers/nine/volume9.h | 4 ++ 2 files changed, 110 insertions(+), 9 deletions(-) diff --git a/src/gallium/state_trackers/nine/volume9.c

<    1   2   3   4   5   6   7   8   9   10   >