Re: [Mesa-dev] [PATCH 0/4] drisw/glx: use XShm if possible

having a zero copy mechanism for wayland swrast. Yours, Axel Davy ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [RFC] DRI3 support for EGL

ed up the DRI2 implementation. It was misnamed. Thus the answer to your question is yes. Yours, Axel Davy ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH RFC] egl/dri2: Add dri3 support to x11 platform

ur device is not a render node, it is possible to get the render node name), and then you don't have to authenticate. In case render-nodes are not available, then it's ok not to advertise the extension. Yours, Axel Davy ___ mesa-dev maili

Re: [Mesa-dev] [RFC] loader: libudev vs sysfs vs libdrm

ed by udev with the device. Perhaps there is another way to compute that tag though. Axel Davy ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [RFC] loader: libudev vs sysfs vs libdrm

On 09/07/2015 15:33, Emil Velikov wrote : Yakes forgot about that one. From a quick look we can (I know it feels durty) use readlink( /sys/dev/char/$(major):$(minor)). Everything seems to be there - bus type and exact location of the device on the bus. It's limited to sysfs users, but they are

Re: [Mesa-dev] [PATCH v2 6/7] radeonsi: Add DCC decompress.

On 12/10/2015 17:05, Bas Nieuwenhuizen wrote: Hi Marek, Thanks for the quick review. I do not think I understand the sharing semantics. We currently have fast clear for scanout surfaces with the CMASK and eliminate it on flush resource. I would think we could do that similarly with DCC fast cle

Re: [Mesa-dev] [PATCH v2 6/7] radeonsi: Add DCC decompress.

On 13/10/2015 00:42, Bas Nieuwenhuizen wrote: Hi Axel, Using DCC for scanout surfaces is problematic because as far as I understand the display hardware does not support it. We could solve that partially by decompressing when displaying. However, the X server can also use these surfaces as a fr

Re: [Mesa-dev] [PATCH 37/40] targets/nine: add note about fd owndership

On 18/10/2015 12:25, Emil Velikov wrote: +/* XXX: Is the owner responsible for FD owndership or are we ? + * In the former we're screwing things hard here */ This will be updated to "Although the fd is provided from external source, mesa/nine takes ownership of it." based on input from

Re: [Mesa-dev] [PATCH v3 1/7] radeonsi: Allocate buffers for DCC.

fers. It seems however a bit weak to rely on that. I suggest rather to use the pipe shared bind flag. I noticed in some case of imported surfaces the bind flag is not advertised, I'm going to send a patch to fix that. Yours, Axel Davy ___ mes

[Mesa-dev] [PATCH 1/4] egl/wayland: Use scanout flag for backbuffers

The back buffers need to be scanout-able in case the compositor wants to use the buffer (once sent) as display framebuffer. Signed-off-by: Axel Davy --- src/egl/drivers/dri2/platform_wayland.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/egl/drivers/dri2

[Mesa-dev] [PATCH 4/4] pipe: Add new bind flag for shared resources with flush_resource call

call outside dri2 and nine state trackers. Signed-off-by: Axel Davy --- src/gallium/include/pipe/p_defines.h | 8 src/gallium/state_trackers/dri/dri2.c | 17 + src/gallium/state_trackers/dri/dri_drawable.c | 9 +++-- src/gallium/state_trackers/nine/swapc

[Mesa-dev] [PATCH 2/4] st/dri2: Add shared flag to missing locations

The PIPE_BIND_SHARED flag should be added whenever the resource may be shared with another process. In particular if the resource is imported, or may be exported, the flag should be used. Signed-off-by: Axel Davy --- src/gallium/state_trackers/dri/dri2.c | 9 +++-- src/gallium

[Mesa-dev] [PATCH 3/4] dri: Add backbuffer use flag

contrary to gbm buffers, front buffers and fake front buffers, which could be read after a flush. Signed-off-by: Axel Davy --- include/GL/internal/dri_interface.h | 1 + src/egl/drivers/dri2/platform_wayland.c | 3 ++- src/glx/dri3_glx.c | 6 -- 3 files changed, 7

Re: [Mesa-dev] [PATCH 4/4] pipe: Add new bind flag for shared resources with flush_resource call

On 21/10/2015 13:16, Bas Nieuwenhuizen wrote: On Wed, Oct 21, 2015 at 12:28 PM, Axel Davy wrote: +/* This flag indicates that in addition to being shared, the resource won't be + * read by any external process before we call flush_resource. This allows + * things like compressing the b

Re: [Mesa-dev] [PATCH 4/4] pipe: Add new bind flag for shared resources with flush_resource call

On 21/10/2015 13:36, Bas Nieuwenhuizen wrote: My apologies, wrong term. I meant the front buffer of the X server in the non-compositing case. - Bas I think only glamor uses mesa for X rendering. Depending on the DDX, the front buffer will either be created with gbm, or imported as an EGLIma

Re: [Mesa-dev] [PATCH 0/7] DRI3 support for EGL (v4)

bled, just like GLX_EXT_texture_from_pixmap is for GLX. Yours, Axel Davy On 10/11/2015 18:30, Martin Peres wrote: Here is an update to the v3, addressing almost all the comments I got during the previous round. The one item that is left to do is the handling of EGL_BUFFER_PRESERVED which will take

Re: [Mesa-dev] killing off the address reg in tgsi

(either to a integer directly if card supports it, or to float). Axel Davy On 29/01/2015 22:20, Roland Scheidegger wrote : Hi, the address reg in tgsi is quite a nuisance. glsl-to-tgsi code assumes that indirections can only be done through the address reg and has quite some extra code to

[Mesa-dev] [PATCH 01/43] st/nine: Explicit nine requirements

ards to include integer and boolean constants in the shader. Eventually when this is done, we can reintroduce support for older cards. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c | 106 + src/gallium/state_trackers/nine/device9.c | 9 +--

[Mesa-dev] [PATCH 25/43] st/nine: Set [out] argument to NULL for some functions

From: Patrick Rudolph Wine tests, and probably some apps, check for errors by checking for NULL instead of error codes. Fixes wine test test_surface_blocks() Reviewed-by: Axel davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c | 6 ++ 1 file changed, 6

[Mesa-dev] [PATCH 11/43] st/nine: fix early basetexture destruction

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/basetexture9.c b/src/gallium/state_trackers/nine/basetexture9.c index 5134de2..8d183e6 100644 --- a/src/gallium

[Mesa-dev] [PATCH 27/43] st/nine: Check for aligned offset in each vertex element

From: Patrick Rudolph Fixes wine test test_vertex_declaration_alignment() Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/vertexdeclaration9.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 20/43] st/nine: Implement AMD alpha to coverage

This D3D hack is supposed to be supported by all AMD SM2+ cards. Apps use it without checking if they are on AMD. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 17 +++-- src/gallium/state_trackers/nine/nine_pipe.c | 2 +- src/gallium/state_trackers

[Mesa-dev] [PATCH 09/43] st/nine: Check block alignment for compressed textures in NineSurface9_CopySurface

From: Patrick Rudolph Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/surface9.c | 24 1 file changed, 24 insertions(+) diff --git a/src/gallium/state_trackers/nine/surface9.c b/src/gallium/state_trackers/nine/surface9.c

[Mesa-dev] [PATCH 05/43] st/nine: Introduce failure handling for shader parsing.

Instead of crashing on buggy shaders, we should return an error. This patch introduces this behaviour in the case of invalid constant access Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_shader.c | 38 +-- 1 file changed, 30 insertions(+), 8 deletions

[Mesa-dev] [PATCH 15/43] st/nine: Refactor format d3d9 to pipe conversion

Move the checks of whether the format is supported into a common place. The advantage is that allows to handle when a d3d9 format can be mapped to several formats, and that cards don't support all of them. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c

[Mesa-dev] [PATCH 03/43] st/nine: Declare constants only up to the maximum needed.

reduce the amount of constants needed, and that it is restricted to 256 constant slots. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_shader.c | 38 --- 1 file changed, 11 insertions(+), 27 deletions(-) diff --git a/src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 12/43] st/nine: Implement RESZ hack

reworked. For now the current behaviour is ok, since apps use the D3DCREATE_PUREDEVICE flag when creating the device, which means they won't read states and in exchange get better performance. Signed-off-by: Axel Davy --- This patch requires gallium: Add MULTISAMPLE_Z_RESOLVE cap include

[Mesa-dev] [PATCH 17/43] st/nine: NineDevice9_SetClipPlane: pPlane must be non-NULL

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index e4b2c90..a446c8f 100644 --- a/src/gallium/state_trackers

[Mesa-dev] [PATCH 18/43] st/nine: Change the value of some advertised caps

These values are taken from wine. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/adapter9.c b/src/gallium/state_trackers/nine/adapter9.c index 84469b1..84578ac

[Mesa-dev] [PATCH 31/43] st/nine: D3DRS_FILLMODE set to 0 is D3DFILL_SOLID

From: Tiziano Bacocco Reviewed-by: Axel Davy Reviewed-by: David Heidelberg Signed-off-by: Tiziano Bacocco --- include/D3D9/d3d9types.h| 1 + src/gallium/state_trackers/nine/nine_pipe.h | 1 + 2 files changed, 2 insertions(+) diff --git a/include/D3D9/d3d9types.h b

[Mesa-dev] [PATCH 29/43] st/nine: Implement fallback behaviour when rts and ds don't match

This seems to be the behaviour on Win. Previous behaviour led to different issues depending on the driver. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 46 +--- 1 file changed, 28 insertions(+), 18 deletions(-) diff --git a/src/gallium

[Mesa-dev] [PATCH 04/43] st/nine: Print warnings for r500 when shader is likely to go wrong

r500 hasn't enough float constants for vs to fill all needs. Overlapping issues can happen with complex shaders. The fix would be to recompile shaders to include the integer and boolean constants, instead of reserving slots for them. Signed-off-by: Axel Davy --- src/gallium/state_trackers

[Mesa-dev] [PATCH 08/43] st/nine: Commit sampler views again if srgb state changed.

: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 3 +++ src/gallium/state_trackers/nine/nine_state.c | 5 +++-- src/gallium/state_trackers/nine/nine_state.h | 1 + 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium

[Mesa-dev] [PATCH 06/43] st/nine: Check for the correct number of constants.

tions for old vs and ps versions. The restrictions were fixed, unlike for the number of vertex shader constants for later versions. Likely apps use the correct number, and it's not a problem for us if it wants use more. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_sh

[Mesa-dev] [PATCH 41/43] gallium/targets/d3dadapter9: Release the pipe_screen at destruction.

We weren't releasing hal and ref, causing some issues (threads not released, etc) Signed-off-by: Axel Davy --- src/gallium/targets/d3dadapter9/drm.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/src/gallium/targets/d3dadapter9/drm.c b/src/ga

[Mesa-dev] [PATCH 22/43] st/nine: Missing sanity check for CALLOC return E_OUTOFMEMORY if allocation of usage_map fails

From: Patrick Rudolph Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/vertexdeclaration9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/vertexdeclaration9.c b/src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 36/43] st/nine: Fix multisampling support detection

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c | 12 +--- src/gallium/state_trackers/nine/nine_pipe.h | 25 - 2 files changed, 9 insertions(+), 28 deletions(-) diff --git a/src/gallium/state_trackers/nine/adapter9.c b/src/gallium

[Mesa-dev] [PATCH 40/43] gallium/targets/d3dadapter9: Fix device detection for render-nodes

d the pipe cap to determine the vendor and device id from the driver. Signed-off-by: Axel Davy --- src/gallium/targets/d3dadapter9/drm.c | 59 +-- 1 file changed, 14 insertions(+), 45 deletions(-) diff --git a/src/gallium/targets/d3dadapter9/drm.c b/src/galli

[Mesa-dev] [PATCH 42/43] gallium/targets/d3dadapter9: Free card device

The drm fd wasn't released, causing a crash for wine tests on nouveau, which seems to have a bug when a lot of device descriptors are open. Signed-off-by: Axel Davy --- src/gallium/targets/d3dadapter9/drm.c | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --

[Mesa-dev] [PATCH 13/43] st/nine: Improve CheckDeviceFormat debug output

ts, but we can always retrieve that info by hand. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/src/gallium/state_trackers/nine/adapter9.c b/src/gallium/state_trackers/nine/adapter9.c ind

[Mesa-dev] [PATCH 39/43] st/nine: Dummy sampler should have a=1

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 39 +++ src/gallium/state_trackers/nine/device9.h| 3 + src/gallium/state_trackers/nine/nine_ff.c| 7 ++ src/gallium/state_trackers/nine/nine_state.c | 155 +++ src/gallium

[Mesa-dev] [PATCH 34/43] st/nine: Fix depth stencil formats bindings flags.

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c | 26 ++ src/gallium/state_trackers/nine/device9.c| 4 ++- src/gallium/state_trackers/nine/nine_pipe.h | 51 src/gallium/state_trackers/nine/swapchain9.c | 20 +-- 4

[Mesa-dev] [PATCH 28/43] st/nine: Fix present_buffers allocation

If has_present_buffers was false at first, but after a device reset, it turns true (for example if we begin to render to a multisampled back buffer), there was a crash due to present_buffers being uninitialised. This patch fixes it. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 32/43] st/nine: SetResourceResize should track nr_samples too

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/surface9.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/nine/surface9.c b/src/gallium/state_trackers/nine/surface9.c index 21a4707..f1b4aa9 100644 --- a/src/gallium/state_trackers/nine/surface9.c

[Mesa-dev] [PATCH 10/43] st/nine: Do not leak private data in volume9.

From: Patrick Rudolph This->data was allocated by nine, but not freed. Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/volume9.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/state_trackers/nine/volume9.c b/src/gall

[Mesa-dev] [PATCH 19/43] st/nine: Add D3DFMT_DF16 support

don't add support for DF24 yet, because it implies support for FETCH4, which we don't support for now. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 12 ++-- src/gallium/state_trackers/nine/nine_pipe.h| 6 -- 2 files changed, 14 insertions(

[Mesa-dev] [PATCH 33/43] st/nine: Fix gpu memory leak in swapchain

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/swapchain9.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/swapchain9.c b/src/gallium/state_trackers/nine/swapchain9.c index a6e9c57..9db6ff4 100644 --- a/src/gallium

[Mesa-dev] [PATCH 24/43] st/nine: Remove duplicated debug message

From: Patrick Rudolph Likely a rebase error Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/resource9.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/resource9.c b/src/gallium/state_trackers/nine/resource9.c

[Mesa-dev] [PATCH 16/43] st/nine: Implement fallback for D3DFMT_D24S8, D3DFMT_D24X8 and D3DFMT_INTZ

Some drivers support PIPE_FORMAT_S8_UINT_Z24_UNORM, some others PIPE_FORMAT_Z24_UNORM_S8_UINT, some both. It doesn't matter which one we use, since the d3d formats they map to aren't lockable (app can read it directly). Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/n

[Mesa-dev] [PATCH 38/43] st/nine: Fix update_framebuffer binding cbufs the pixel shader wouldn't render to

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 8 src/gallium/state_trackers/nine/nine_state.c | 16 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers

[Mesa-dev] [PATCH 21/43] st/nine: Implement ATOC hack

ATOC is an hack for Alpha to coverage that is supported by NV and Intel. You need to check the support for it with CheckDeviceFormat. Signed-off-by: Axel Davy --- include/D3D9/d3d9types.h | 1 + src/gallium/state_trackers/nine/adapter9.c | 4 src/gallium/state_trackers

[Mesa-dev] [PATCH 26/43] st/nine: Fix bufferoverflow in {Get|Set}PixelShaderConstantF

From: Patrick Rudolph Previous code wasn't checking against the correct limit: 224 for sm3 hardware, but 256. Fixes wine test test_pixel_shader_constant() Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c | 10 +- 1 file chang

[Mesa-dev] [PATCH 23/43] st/nine: Return E_FAIL for unused vertexdeclaration type

From: Patrick Rudolph Add returncode E_FAIL. Return E_FAIL for any vertexdeclaration element with type unused. Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- include/D3D9/d3d9types.h | 1 + src/gallium/state_trackers/nine/vertexdeclaration9.c | 7

[Mesa-dev] [PATCH 37/43] st/nine: Clear: better behave if rt_mask is different to the one of the framebuffer bound

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 40 +-- 1 file changed, 27 insertions(+), 13 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index c6bd052..5eb9d54 100644 --- a

[Mesa-dev] [PATCH 14/43] st/nine: Refactor nine_d3d9_to_pipe_format_map

The order of the format is changed to have an increasing ordering of the d3d9 format values. Some missing formats are added and matched to PIPE_FORMAT_NONE Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_pipe.c | 123 ++-- 1 file changed, 44 insertions

[Mesa-dev] [PATCH 02/43] st/nine: Refactor how user constbufs sizes are calculated

Count explicitly the slots for float, int and bool constants, and deduce the constbuf size in nine_shader. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_shader.c | 17 ++--- src/gallium/state_trackers/nine/nine_shader.h | 22 ++ src

[Mesa-dev] [PATCH 07/43] st/nine: Fix use of D3DSP_NOSWIZZLE

D3DSP_NOSWIZZLE already contains the shift. Detected with Clang. Reviewed-by: David Heidelberg Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_shader.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_shader.c b/src

[Mesa-dev] [PATCH 30/43] st/nine: Setting D3DRS_ALPHAFUNC to 0 means D3DCMP_NEVER

From: Tiziano Bacocco Reviewed-by: Axel Davy Signed-off-by: Tiziano Bacocco --- include/D3D9/d3d9types.h| 1 + src/gallium/state_trackers/nine/nine_pipe.h | 1 + 2 files changed, 2 insertions(+) diff --git a/include/D3D9/d3d9types.h b/include/D3D9/d3d9types.h index

[Mesa-dev] [PATCH 35/43] st/nine: Fix enabled lights in stateblocks

From: Tiziano Bacocco Signed-off-by: Tiziano Bacocco --- src/gallium/state_trackers/nine/device9.c | 4 +++- src/gallium/state_trackers/nine/stateblock9.c | 6 -- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/stat

[Mesa-dev] [PATCH 43/43] st/nine: Implement dummy vbo behaviour when vs is missing inputs

From: Tiziano Bacocco Use a dummy vertex buffer object when vs inputs have no corresponding entries in the vertex declaration. This dummy buffer will give to the shader float4(0,0,0,0). This fixes several artifacts on some games. Signed-off-by: Axel Davy Signed-off-by: Tiziano Bacocco

Re: [Mesa-dev] [PATCH 36/43] st/nine: Fix multisampling support detection

: I recommend not using PIPE_BIND_TRANSFER_READ/WRITE. Transfers with multisample textures should be supported, but the mapped texture is always a resolved single-sample copy, so it's not really an MSAA transfer. Marek On Fri, Jan 30, 2015 at 9:34 PM, Axel Davy wrote: Signed-off-by: Axel Davy

Re: [Mesa-dev] [PATCH 36/43] st/nine: Fix multisampling support detection

some drivers ? Axel On 31/01/2015 07:04, Axel Davy wrote : The patch modifies the code used to check for format support. It needs to test for these flags for render targets because we create the back buffers with these. Probably these flags are not required for back buffers, I'll invest

Re: [Mesa-dev] [PATCH] radeonsi: fix a crash if a stencil ref state is set before a DSA state

It works fine. Since no state tracker need it currently, why do you want to backport it to older mesa versions ? In all cases, Reviewed-by: Axel Davy (btw: missing signoff) Axel Davy On 15/02/2015 18:21, Marek Olšák wrote : From: Marek Olšák + minor indentation fixes Discovered by Axel

Re: [Mesa-dev] Gallium: Removal of set_index_buffer (discussion)

On 20/02/2017 20:11, Ilia Mirkin wrote: On Mon, Feb 20, 2017 at 2:01 PM, Marek Olšák wrote: Hi, I'd like to remove pipe_context::set_index_buffer. It's not useful to most drivers and the interface is inconvenient for Mesa/OpenGL, because it's a draw state that is set with a separate driver cal

Re: [Mesa-dev] [PATCH] st/nine: make use of common uploaders v2

On 20/02/2017 20:22, Constantine Charlamov wrote: Make use of common uploaders that landed recently to Mesa v2: fixed formatting, broken due to thunderbird configuration --- src/gallium/state_trackers/nine/device9.c| 48 src/gallium/state_trackers/nine/device

Re: [Mesa-dev] [PATCH] st/nine: make use of common uploaders v3

On 20/02/2017 21:30, Constantine Charlamov wrote: Make use of common uploaders that landed recently to Mesa v2: fixed formatting, broken due to thunderbird configuration v3: per Axel comment: added a comment into NineDevice9_DrawPrimitiveUP --- src/gallium/state_trackers/nine/device9.c|

Re: [Mesa-dev] Gallium: Removal of set_index_buffer (discussion)

Olšák: On Mon, Feb 20, 2017 at 8:29 PM, Axel Davy wrote: On 20/02/2017 20:11, Ilia Mirkin wrote: On Mon, Feb 20, 2017 at 2:01 PM, Marek Olšák wrote: Hi, I'd like to remove pipe_context::set_index_buffer. It's not useful to most drivers and the interface is inconvenient for M

Re: [Mesa-dev] [PATCH] st/nine: make use of common uploaders v4

This looks fine to me. Reviewed-by: Axel Davy I think the patch requires your Signed-off-by though. Axel On 21/02/2017 05:31, Constantine Charlamov wrote: Make use of common uploaders that landed recently to Mesa v2: fixed formatting, broken due to thunderbird configuration v3: per Axel

Re: [Mesa-dev] [PATCH] st/nine: Remove code for no USER_INDEX_BUFFERS as these are always on

Hi Mike, We really want not to use user index buffers when csmt is active (thus the !This->csmt_active). This should be a one line patch to just remove the part GET_PCAP(USER_INDEX_BUFFERS) Yours, Axel Davy On 25/02/2017 06:23, Mike Lothian wrote: This fi

[Mesa-dev] [PATCH 00/71] New Gallium Nine Fixes and Improvements

refactoring. I hesitated to send them, but I'd still like have them in. Patches 61-67 are some more fixes or improvements Patches 68-71 do silent some compilation warnings Axel Davy (55): st/nine: Fix Swizzle for ATI2 format st/nine: Always set point_quad_rasterization

[Mesa-dev] [PATCH 01/71] st/nine: Require gcc >= 4.6

From: David Heidelberg Fixes bug: fdo #89978 Signed-off-by: David Heidelberg Cc: "10.4 10.5 10.6" --- configure.ac | 4 1 file changed, 4 insertions(+) diff --git a/configure.ac b/configure.ac index 4e751e3..c355092 100644 --- a/configure.ac +++ b/configure.ac @@ -1632,6 +1632,10 @@ if

[Mesa-dev] [PATCH 14/71] st/nine: Simplify Volume9 dirty region tracking

Similar to what was done for Surface9, track the dirty region only in VolumeTexture9. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 9 +-- src/gallium/state_trackers/nine/volume9.c | 82 ++ src/gallium/state_trackers/nine/volume9.h

[Mesa-dev] [PATCH 17/71] st/nine: SetAutoGenFilterType should regenerate the sublevels

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/state_trackers/nine/basetexture9.c b/src/gallium/state_trackers/nine/basetexture9.c index cc74cc9..728aafd 100644 --- a/src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 10/71] st/nine: Do not call ID3DPresent_GetCursorPos for sw cursor

For sw cursor we do not tell wine the cursor position (the app tells us directly). We shouldn't use ID3DPresent_GetCursorPos. device->cursor.pos already contains the coordinates the app gave us. Signed-off-by: Axel Davy Reviewed-by: David Heidelberg --- src/gallium/state_track

[Mesa-dev] [PATCH 02/71] target/d3dadapter9: Return Windows like card names

From: Patrick Rudolph Add support for multiple cards and fill in Windows like card name, driver name and version info. Use fallback for unknown vendors and unknown card names. Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/targets/d3dadapter9/Makefile.am | 1

[Mesa-dev] [PATCH 19/71] st/nine: Track dirty region for SYSTEMMEM too

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/cubetexture9.c | 6 -- src/gallium/state_trackers/nine/surface9.c | 4 ++-- src/gallium/state_trackers/nine/texture9.c | 10 ++ src/gallium/state_trackers/nine/volumetexture9.c | 8 +--- 4 files changed

[Mesa-dev] [PATCH 11/71] st/nine: Revert to sw cursor in case of failure to set hw cursor

Signed-off-by: Axel Davy Reviewed-by: David Heidelberg --- src/gallium/state_trackers/nine/device9.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index 9f6c90e..38af8e3 100644 --- a

[Mesa-dev] [PATCH 04/71] st/nine: Always set point_quad_rasterization to 1

Both Points and Point Sprites are rasterized like quads, according to d3d9 doc and gallium rasterizer doc. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_pipe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_pipe.c b

[Mesa-dev] [PATCH 20/71] st/nine: Textures start dirty

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/cubetexture9.c | 6 +- src/gallium/state_trackers/nine/texture9.c | 3 +++ src/gallium/state_trackers/nine/volumetexture9.c | 3 +++ 3 files changed, 11 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers

[Mesa-dev] [PATCH 25/71] st/nine: Fix FillColor Flag check

IT is better check if the surface was created with RT flag, instead of checking capability (llvmpipe was complaining) Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 23/71] st/nine: Implement EvictManagedResources

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 15 +++ src/gallium/state_trackers/nine/basetexture9.h | 3 +++ src/gallium/state_trackers/nine/device9.c | 12 3 files changed, 26 insertions(+), 4 deletions(-) diff --git a/src/gallium

[Mesa-dev] [PATCH 30/71] st/nine: Return NULL pointer in lock error cases

From: Patrick Rudolph Tests showed, that in case of errors, the pBits pointer is set to NULL. The pBits field isn't set to NULL in case of an already locked object. Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/surface9.c | 11 +++

[Mesa-dev] [PATCH 16/71] st/nine: Simplify NineVolume9_CopyVolume

We had only one usage for this function. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 4 +- src/gallium/state_trackers/nine/volume9.c | 90 ++- src/gallium/state_trackers/nine/volume9.h | 8 +-- 3 files changed, 21 insertions(+), 81

[Mesa-dev] [PATCH 15/71] st/nine: Split NineSurface9_CopySurface

NineDevice9_UpdateSurface Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 81 +++- src/gallium/state_trackers/nine/nine_pipe.h | 17 +++ src/gallium/state_trackers/nine/surface9.c | 176 ++- src/gallium/state_trackers/nine/surface9.h | 14

[Mesa-dev] [PATCH 21/71] st/nine: Only update dirty rect for UpdateTexture

UpdateTexture is supposed to optimise by uploading only for the dirty region of the source (d3d9 doc, wine tests). This patch adds the behaviour for surfaces, but not entirely for volumes. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 47

[Mesa-dev] [PATCH 13/71] util/u_blitter: implement alpha blending for pipe->blit

From: Marek Olšák --- src/gallium/auxiliary/util/u_blitter.c | 54 ++--- src/gallium/auxiliary/util/u_blitter.h | 3 +- src/gallium/auxiliary/util/u_surface.c | 3 ++ src/gallium/drivers/i915/i915_surface.c | 3 +- src/gallium/drivers/r300/r300_blit.c| 3 +-

[Mesa-dev] [PATCH 22/71] st/nine: Track managed textures

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 7 ++- src/gallium/state_trackers/nine/basetexture9.h | 3 ++- src/gallium/state_trackers/nine/device9.c | 1 + src/gallium/state_trackers/nine/device9.h | 1 + 4 files changed, 10 insertions(+), 2

[Mesa-dev] [PATCH 06/71] st/nine: Account POINTSIZE_MIN and POINTSIZE_MAX for point size

From: Patrick Rudolph When using D3DRS_POINTSIZE make sure the value is at least D3DRS_POINTSIZE_MIN but not greater than D3DRS_POINTSIZE_MAX. Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/nine_pipe.c | 8 +++- src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 07/71] st/nine: fix D3DRS_DITHERENABLE wrong state group

D3DRS_DITHERENABLE was assigned to the rasterizer state group, but it was used for the blend group. Assign it to the blend group. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium

[Mesa-dev] [PATCH 03/71] st/nine: Fix Swizzle for ATI2 format

We had red and green in the wrong channels for the ATI2 format (RGTC2). Found thanks to wine tests. Signed-off-by: Axel Davy Reviewed-by: David Heidelberg --- src/gallium/state_trackers/nine/basetexture9.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/state_trackers

[Mesa-dev] [PATCH 08/71] st/nine: Hide hardware cursor when we don't use it

Signed-off-by: Axel Davy Reviewed-by: David Heidelberg --- src/gallium/state_trackers/nine/device9.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index 55948cb

[Mesa-dev] [PATCH 31/71] st/nine: Impose restrictions on DXTN texture sizes

This is the expected behaviour. Fixes more than 1 wine tests failures. Signed-off-by: Axel Davy Reviewed-by: David Heidelberg --- src/gallium/state_trackers/nine/cubetexture9.c | 7 +++ src/gallium/state_trackers/nine/device9.c| 7 +++ src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 28/71] st/nine: Clean GetPrivateData

From: Patrick Rudolph Move the assert to return error codes in the correct order. Always set the pSizeOfData to the required buffer size. Fixes failing wine test device.c test_private_data() Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/resource9.c

[Mesa-dev] [PATCH 05/71] st/nine: Align texture memory

From: Patrick Rudolph Align texture memory on 32 byte boundry to allow SSE/AVX memcpy to work on locked rects. This fixes issue #88. Reviewed-by: David Heidelberg Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/cubetexture9.c | 2 +- src/gallium

[Mesa-dev] [PATCH 09/71] st/nine: Force hw cursor for Windowed mode

Signed-off-by: Axel Davy Reviewed-by: David Heidelberg --- src/gallium/state_trackers/nine/device9.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index fce19b2..9f6c90e

[Mesa-dev] [PATCH 36/71] st/nine: Fix use of uninitialized values

From: Patrick Rudolph Set all values to 0 after allocation. Found using valgrind. Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/device9

[Mesa-dev] [PATCH 42/71] st/nine: Avoid useless updates in SetSamplerState

Signed-off-by: Axel Davy Reviewed-by: David Heidelberg --- 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 4aa5892..a327bf8

[Mesa-dev] [PATCH 18/71] st/nine: Add missing BASETEX_REGISTER_UPDATE calls

If the texture is bound and dirty_mip is true, BASETEX_REGISTER_UPDATE adds the texture to the list of things to update before the next draw call. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/cubetexture9.c | 4 +++- src/gallium/state_trackers/nine/texture9.c | 4 +++- 2

[Mesa-dev] [PATCH 32/71] st/nine: Fix Lock Checks for Compressed textures

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/surface9.c | 10 ++ src/gallium/state_trackers/nine/volume9.c | 9 + 2 files changed, 11 insertions(+), 8 deletions(-) diff --git a/src/gallium/state_trackers/nine/surface9.c b/src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 27/71] st/nine: fix failing wine test device.c test_lockrect_invalid()

From: Patrick Rudolph Mimic WindowsXp behaviour and allow negative values in the rectangle passed. Add comment to point out behaviour used. Reviewed-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/surface9.c | 8 ++-- 1 file changed, 2 insertions(+), 6

  1   2   3   4   5   6   7   8   9   10   >