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

2015-08-16 Thread Axel Davy
Signed-off-by: Axel Davy axel.d...@ens.fr --- 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

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

2015-08-16 Thread Axel Davy
From: Patrick Rudolph s...@das-labor.org 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 axel.d...@ens.fr Signed-off-by: Patrick Rudolph s...@das-labor.org --- src/gallium/state_trackers/nine

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

2015-08-16 Thread Axel Davy
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 axel.d...@ens.fr --- src/gallium/state_trackers/nine/nine_state.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src

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

2015-08-16 Thread Axel Davy
We had red and green in the wrong channels for the ATI2 format (RGTC2). Found thanks to wine tests. Signed-off-by: Axel Davy axel.d...@ens.fr Reviewed-by: David Heidelberg da...@ixit.cz --- src/gallium/state_trackers/nine/basetexture9.c | 5 + 1 file changed, 5 insertions(+) diff --git

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

2015-08-16 Thread Axel Davy
Signed-off-by: Axel Davy axel.d...@ens.fr Reviewed-by: David Heidelberg da...@ixit.cz --- 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

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

2015-08-16 Thread Axel Davy
This is the expected behaviour. Fixes more than 1 wine tests failures. Signed-off-by: Axel Davy axel.d...@ens.fr Reviewed-by: David Heidelberg da...@ixit.cz --- src/gallium/state_trackers/nine/cubetexture9.c | 7 +++ src/gallium/state_trackers/nine/device9.c| 7 +++ src

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

2015-08-16 Thread Axel Davy
From: Patrick Rudolph s...@das-labor.org 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 axel.d...@ens.fr Signed-off-by: Patrick Rudolph s...@das

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

2015-08-16 Thread Axel Davy
Similar to what was done for Surface9, track the dirty region only in VolumeTexture9. Signed-off-by: Axel Davy axel.d...@ens.fr --- src/gallium/state_trackers/nine/basetexture9.c | 9 +-- src/gallium/state_trackers/nine/volume9.c | 82 ++ src/gallium/state_trackers

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

2015-08-16 Thread Axel Davy
Signed-off-by: Axel Davy axel.d...@ens.fr --- 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

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

2015-08-16 Thread Axel Davy
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 axel.d...@ens.fr Reviewed-by: David Heidelberg da...@ixit.cz --- src/gallium

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

2015-08-16 Thread Axel Davy
From: Patrick Rudolph s...@das-labor.org 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 axel.d...@ens.fr Signed-off-by: Patrick Rudolph s...@das-labor.org --- src

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

2015-08-16 Thread Axel Davy
Signed-off-by: Axel Davy axel.d...@ens.fr --- 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

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

2015-08-16 Thread Axel Davy
Signed-off-by: Axel Davy axel.d...@ens.fr Reviewed-by: David Heidelberg da...@ixit.cz --- 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

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

2015-08-16 Thread Axel Davy
From: Patrick Rudolph s...@das-labor.org 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 da...@ixit.cz Reviewed-by: Axel Davy axel.d...@ens.fr Signed-off-by: Patrick Rudolph s...@das-labor.org --- src

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

2015-08-16 Thread Axel Davy
Signed-off-by: Axel Davy axel.d...@ens.fr Reviewed-by: David Heidelberg da...@ixit.cz --- 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

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

2015-08-16 Thread Axel Davy
From: Patrick Rudolph s...@das-labor.org Set all values to 0 after allocation. Found using valgrind. Reviewed-by: Axel Davy axel.d...@ens.fr Signed-off-by: Patrick Rudolph s...@das-labor.org --- src/gallium/state_trackers/nine/device9.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion

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

2015-08-16 Thread Axel Davy
Signed-off-by: Axel Davy axel.d...@ens.fr Reviewed-by: David Heidelberg da...@ixit.cz --- 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

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

2015-08-16 Thread Axel Davy
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 axel.d...@ens.fr --- src/gallium/state_trackers/nine/cubetexture9.c | 4 +++- src/gallium/state_trackers/nine/texture9.c

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

2015-08-16 Thread Axel Davy
Signed-off-by: Axel Davy axel.d...@ens.fr --- 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

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

2015-08-16 Thread Axel Davy
From: Patrick Rudolph s...@das-labor.org Mimic WindowsXp behaviour and allow negative values in the rectangle passed. Add comment to point out behaviour used. Reviewed-by: Axel Davy axel.d...@ens.fr Signed-off-by: Patrick Rudolph s...@das-labor.org --- src/gallium/state_trackers/nine/surface9.c

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

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

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

2015-08-16 Thread Axel Davy
Signed-off-by: Axel Davy axel.d...@ens.fr --- 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

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

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

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

2015-08-16 Thread Axel Davy
Signed-off-by: Axel Davy axel.d...@ens.fr --- 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

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

2015-08-16 Thread Axel Davy
From: Patrick Rudolph s...@das-labor.org 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 axel.d...@ens.fr Signed-off-by: Patrick Rudolph s...@das-labor.org --- src/gallium

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

2015-08-16 Thread Axel Davy
From: David Heidelberg da...@ixit.cz Fixes bug: fdo #89978 Signed-off-by: David Heidelberg da...@ixit.cz Cc: 10.4 10.5 10.6 mesa-sta...@lists.freedesktop.org --- configure.ac | 4 1 file changed, 4 insertions(+) diff --git a/configure.ac b/configure.ac index 4e751e3..c355092 100644 ---

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

2015-08-16 Thread Axel Davy
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 to 1 st/nine: fix

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

2015-08-21 Thread Axel Davy
vendor, or advertise another vendor (some games do disable some features for some cards). Yours, Axel Davy On 17/08/2015 19:30, Benjamin Bellec write : Hello, Where does this mapping list come from ? Is this an official map (Vendor or Windows drivers) ? Is that used just for information ? Because

Re: [Mesa-dev] [PATCH] st/nine: Disable NV path, it's broken right now.

2015-08-21 Thread Axel Davy
. User constant buffers are expected to be slower for Nvidia cards, so ideally this patch should be reverted when the path is fixed. I add this patch to the list of patches I will merge today. Axel DAvy On 21/08/2015 13:05, Krzysztof A. Sobiecki wrote : st/nine: Disable NV path, it's broken

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

2015-11-10 Thread Axel Davy
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 some time

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

2015-10-21 Thread Axel Davy
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 <axel.d...@ens.fr> --- src/gallium/state_trackers/dri/dri2.c

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

2015-10-21 Thread Axel Davy
, in contrary to gbm buffers, front buffers and fake front buffers, which could be read after a flush. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- include/GL/internal/dri_interface.h | 1 + src/egl/drivers/dri2/platform_wayland.c | 3 ++- src/glx/dri3_glx.c | 6 +++

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

2015-10-21 Thread Axel Davy
outside dri2 and nine state trackers. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- 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/g

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

2015-10-21 Thread Axel Davy
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 <axel.d...@ens.fr> --- src/egl/drivers/dri2/platform_wayland.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/s

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

2015-10-21 Thread Axel Davy
On 21/10/2015 13:16, Bas Nieuwenhuizen wrote: On Wed, Oct 21, 2015 at 12:28 PM, Axel Davy <axel.d...@ens.fr> 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 + * thing

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

2015-10-21 Thread Axel Davy
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

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

2015-10-18 Thread Axel Davy
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

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

2015-10-21 Thread Axel Davy
buffers. 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 ___ mesa-dev

Re: [Mesa-dev] [PATCH 3/8] radeonsi: Enable DCC.

2015-10-10 Thread Axel Davy
change in the future: For example, perhaps some day wayland will tell egl the app is not fullscreen and that a non-scanoutable buffer can be used. Axel Davy ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinf

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

2015-10-12 Thread Axel Davy
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

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

2015-10-12 Thread Axel Davy
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

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

2015-07-08 Thread Axel Davy
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

2015-07-09 Thread Axel Davy
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] gallium: remove support for predicates from TGSI

2015-09-01 Thread Axel Davy
it. The variable isn't used. Axel Davy On 01/09/2015 23:19, Marek Olšák wrote : From: Marek Olšák <marek.ol...@amd.com> Neved used. --- ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 0/8] Add DCC support.

2015-09-04 Thread Axel Davy
On 04/09/2015 23:49, Bas Nieuwenhuizen wrote : With DRI2, my system always gives me SCANOUT framebuffers, even when the application is not running fullescreen, which means that decompression will not happen in this common case. Although I realize now that I am not sure that flag is always set

Re: [Mesa-dev] [PATCH 1/5] svga: Ignore pipe_rasterizer_state::line_last_pixel.

2015-09-14 Thread Axel Davy
, Axel Davy On 14/09/2015 13:09, Jose Fonseca wrote : Altough SVGA3D_RS_LASTPIXEL renderstate exists, most backends ignore it. --- src/gallium/drivers/svga/svga_context.h | 1 - src/gallium/drivers/svga/svga_pipe_rasterizer.c | 1 - src/gallium/drivers/svga/svga_state_rss.c | 2

Re: [Mesa-dev] [PATCH 1/5] svga: Ignore pipe_rasterizer_state::line_last_pixel.

2015-09-14 Thread Axel Davy
ented more easily), and I can add myself the comment in nine code. Thus the series is: Reviewed-by: Axel Davy <axel.d...@ens.fr> ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] gallium/dri2: Avoid uneeded stride to pitch conversion

2016-01-05 Thread Axel Davy
Hi Nicolas, I believe this patch doesn't fully solve the problem and could introduce some bugs. For example you change dri2_create_image_from_name pitch argument to stride, whereas this function is used as is to implement createImageFromName from __DRIimageExtensionRec, which takes a pitch.

Re: [Mesa-dev] [PATCH 8/8] nine: allow fragment shader POSITION and FACE to be system values

2016-01-08 Thread Axel Davy
On 08/01/2016 02:29, Marek Olšák wrote: From: Marek Olšák --- src/gallium/state_trackers/nine/nine_ff.c | 10 +- src/gallium/state_trackers/nine/nine_shader.c | 48 +-- 2 files changed, 46 insertions(+), 12 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 1/2] radeonsi: simplify gl_FragCoord behavior

2016-01-08 Thread Axel Davy
On 08/01/2016 02:30, Marek Olšák wrote: From: Marek Olšák It will become a system value, not an input. --- src/gallium/drivers/radeonsi/si_state_shaders.c | 45 - 1 file changed, 22 insertions(+), 23 deletions(-) diff --git

Re: [Mesa-dev] [PATCH] radeon: only suspend queries on flush if they haven't been suspended yet

2015-11-27 Thread Axel Davy
Tested-by: Axel Davy <axel.d...@ens.fr> On 28/11/2015 00:24, Nicolai Hähnle wrote: From: Nicolai Hähnle <nicolai.haeh...@amd.com> Non-timer queries are suspended during blits. When the blits end, the queries are resumed, but this resume operation itself might run out of CS space

Re: [Mesa-dev] radeon cmask sync with EGL/GBM

2016-01-11 Thread Axel Davy
On 10/01/2016 19:38, Jay Cornwall wrote: Hi, Using EGL with the GBM platform it is possible to bind a GBM surface to a GL renderbuffer. I believe it's also valid to use the GBM BO as a framebuffer target with drmModeAddFB/drmModePageFlip. I haven't been able to locate a clear statement on

Re: [Mesa-dev] [PATCH 1/3] gallium/st: add pipe_context::generate_mipmap()

2016-01-12 Thread Axel Davy
On 12/01/2016 07:18, Charmaine Lee wrote: --- a/src/mesa/state_tracker/st_gen_mipmap.c +++ b/src/mesa/state_tracker/st_gen_mipmap.c @@ -149,12 +149,19 @@ st_generate_mipmap(struct gl_context *ctx, GLenum target, last_layer = util_max_layer(pt, baseLevel); } - /* Try to generate

Re: [Mesa-dev] [PATCH] st/mesa: change SQRT lowering to fix the game Risen

2016-06-05 Thread Axel Davy
if abs is not inserted in the source glsl, that's a wine bug, not a mesa bug, Thus I don't think doing this is a good idea. On 05/06/2016 17:17, Ilia Mirkin wrote: On Mon, May 30, 2016 at 7:19 PM, Marek Olšák wrote: From: Marek Olšák Bugzilla:

Re: [Mesa-dev] [PATCH 4/4] radeonsi: force non-displayable tiling based on surface dimensions

2016-06-06 Thread Axel Davy
On 06/06/2016 11:37, Michel Dänzer wrote : With DRI3, st/dri could (re-)allocate buffers with the scanout flag first and after any window geometry changes, then re-allocate without the flag if the present complete event indicates that page flipping couldn't be used. That sounds like a bad

Re: [Mesa-dev] [PATCH 4/4] radeonsi: force non-displayable tiling based on surface dimensions

2016-06-06 Thread Axel Davy
Le 06/06/2016 12:28, Marek Olšák a écrit : On Mon, Jun 6, 2016 at 12:24 PM, Marek Olšák <mar...@gmail.com> wrote: On Mon, Jun 6, 2016 at 11:47 AM, Michel Dänzer <mic...@daenzer.net> wrote: On 06.06.2016 18:44, Axel Davy wrote: On 06/06/2016 11:37, Michel Dänzer wrote : With

Re: [Mesa-dev] [PATCH v2] egl/x11_dri3: stop restricting support to render nodes unnecessarily

2016-06-10 Thread Axel Davy
more adapted than a hack based on whether wayland support is built. I favor however just not advertising the extension when render-nodes aren't available (+ add a noop authenticate with warning to have that extension properly implemented). Yours, Axel Davy On 10/06/2016 18:35, Frank Binns wrote

Re: [Mesa-dev] [PATCH 1/8] gallium: Add a cap for offset_units_unscaled

2016-06-15 Thread Axel Davy
On 15/06/2016 03:04, Roland Scheidegger wrote: Am 15.06.2016 um 01:08 schrieb Axel Davy: On 15/06/2016 00:21, Roland Scheidegger wrote: Am 14.06.2016 um 23:33 schrieb Axel Davy: diff --git a/src/gallium/include/pipe/p_state.h b/src/gallium/include/pipe/p_state.h index 396f563..7dce80a 100644

Re: [Mesa-dev] [PATCH] target/d3dadapter9: make sure strings are null-terminated

2016-05-31 Thread Axel Davy
The drvid structure seems to be memset in read_descriptor, thus I don't think we need to set 0 for the last table slot. Axel Davy On 31/05/2016 03:21, Eric Engestrom wrote: From strncpy's man page: Warning: If there is no null byte among the first n bytes of src, the string placed

[Mesa-dev] [PATCH 2/8] radeonsi: Emit poly_offset states together

2016-06-14 Thread Axel Davy
Emit PA_SU_POLY_OFFSET_DB_FMT_CNTL with rasterizer poly_offset states. This will be useful to implement PIPE_CAP_POLYGON_OFFSET_UNITS_UNSCALED. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/radeonsi/si_state.c | 31 --- 1 file chan

[Mesa-dev] [PATCH 1/8] gallium: Add a cap for offset_units_unscaled

2016-06-14 Thread Axel Davy
D3D9 has a different behaviour for depth bias. For OGL/D3D1X, the depth bias unit is the minimal resolvable value for the depth buffer, which depends on the format (and has different behaviour for float depth buffers). For D3D9, the depth bias unit is 1.0f. Signed-off-by: Axel Davy <axe

[Mesa-dev] [PATCH 4/8] r600g: Emit poly_offset states together

2016-06-14 Thread Axel Davy
Emit PA_SU_POLY_OFFSET_DB_FMT_CNTL with the other poly_offset states. This will be useful to implement PIPE_CAP_POLYGON_OFFSET_UNITS_UNSCALED. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/r600/evergreen_state.c | 36 ++ 1 file chang

[Mesa-dev] [PATCH 3/8] r600: Emit poly_offset states together

2016-06-14 Thread Axel Davy
Emit PA_SU_POLY_OFFSET_DB_FMT_CNTL with the other poly_offset states. This will be useful to implement PIPE_CAP_POLYGON_OFFSET_UNITS_UNSCALED. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/r600/r600_state.c | 35 --- 1 file chang

[Mesa-dev] [PATCH 8/8] st/nine: Use offset_units_unscaled

2016-06-14 Thread Axel Davy
offset_units_unscaled enables proper support for depth bias for gallium nine. Use it if available. Solves issues with some games using depth bias. For example: https://github.com/iXit/Mesa-3D/issues/220 Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/dev

[Mesa-dev] [PATCH 6/8] radeonsi: Implement POLYGON_OFFSET_UNITS_UNSCALED

2016-06-14 Thread Axel Davy
for PA_SU_POLY_OFFSET_DB_FMT_CNTL (ie setting the unorm depth bias behaviour with a scale of 2^0 = 1.0f) gives the unscaled behaviour. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/radeonsi/si_pipe.c | 2 +- src/gallium/drivers/radeonsi/si_state.c | 32 ++-- 2 files c

[Mesa-dev] [PATCH 5/8] radeon: Remove useless pa_su_poly_offset_db_fmt_cntl

2016-06-14 Thread Axel Davy
pa_su_poly_offset_db_fmt_cntl usages were removed in previous patches. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/radeon/r600_pipe_common.h | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/drivers/radeon/r600_pipe_common.h b/src/gallium/drivers/

[Mesa-dev] [PATCH 7/8] r600, r600g: Implement POLYGON_OFFSET_UNITS_UNSCALED

2016-06-14 Thread Axel Davy
for PA_SU_POLY_OFFSET_DB_FMT_CNTL (ie setting the unorm depth bias behaviour with a scale of 2^0 = 1.0f) gives the unscaled behaviour. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/r600/evergreen_state.c | 39 +++- src/gallium/drivers/r600/r600_pipe.c | 2 +

Re: [Mesa-dev] [PATCH 1/8] gallium: Add a cap for offset_units_unscaled

2016-06-14 Thread Axel Davy
On 15/06/2016 00:21, Roland Scheidegger wrote: Am 14.06.2016 um 23:33 schrieb Axel Davy: diff --git a/src/gallium/include/pipe/p_state.h b/src/gallium/include/pipe/p_state.h index 396f563..7dce80a 100644 --- a/src/gallium/include/pipe/p_state.h +++ b/src/gallium/include/pipe/p_state.h

[Mesa-dev] [PATCH v2] r600g: move PA_SU_POLY_OFFSET_DB_FMT_CNTL to poly offset states for r600

2016-06-22 Thread Axel Davy
Emit PA_SU_POLY_OFFSET_DB_FMT_CNTL with the other poly_offset states. This will be useful to implement PIPE_CAP_POLYGON_OFFSET_UNITS_UNSCALED. v2: Increase the num_dw field for the poly offset atom Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/r600/r600_state.

[Mesa-dev] [PATCH v2] r600g: move PA_SU_POLY_OFFSET_DB_FMT_CNTL to poly offset states for evergreen

2016-06-22 Thread Axel Davy
Emit PA_SU_POLY_OFFSET_DB_FMT_CNTL with the other poly_offset states. This will be useful to implement PIPE_CAP_POLYGON_OFFSET_UNITS_UNSCALED. v2: Increase the num_dw field for the poly offset atom Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/r600/evergreen_state.

Re: [Mesa-dev] [PATCH 1/3] egl/x11_dri3: disable WL_bind_wayland_display for devices without render nodes

2016-06-20 Thread Axel Davy
Hi, The three patches make sense to me. Reviewed-by: Axel Davy <axel.d...@ens.fr> On 17/06/2016 19:41, Frank Binns wrote : Up until now, DRI3 was only used for devices that have render nodes, unless overridden via an environment variable, with it falling back to DRI2 oth

Re: [Mesa-dev] [PATCH 1/3] egl/x11_dri3: disable WL_bind_wayland_display for devices without render nodes

2016-06-20 Thread Axel Davy
Le 20/06/2016 11:48, Michel Dänzer a écrit : On 18.06.2016 02:41, Frank Binns wrote: Up until now, DRI3 was only used for devices that have render nodes, unless overridden via an environment variable, with it falling back to DRI2 otherwise. This limitation was there in order to support

Re: [Mesa-dev] [PATCH 1/3] egl/x11_dri3: disable WL_bind_wayland_display for devices without render nodes

2016-06-21 Thread Axel Davy
On 21/06/2016 01:26, Michel Dänzer wrote: On 20.06.2016 20:06, Frank Binns wrote: On 20/06/16 10:48, Michel Dänzer wrote: On 18.06.2016 02:41, Frank Binns wrote: Up until now, DRI3 was only used for devices that have render nodes, unless overridden via an environment variable, with it falling

Re: [Mesa-dev] [PATCH 8/8] nine: allow fragment shader POSITION and FACE to be system values

2016-01-08 Thread Axel Davy
On 08/01/2016 17:30, Marek Olšák wrote: On Fri, Jan 8, 2016 at 12:25 PM, Axel Davy <axel.d...@ens.fr> wrote: On 08/01/2016 02:29, Marek Olšák wrote: From: Marek Olšák <marek.ol...@amd.com> --- src/gallium/state_trackers/nine/nine_ff.c | 10 +- src/gallium/state_t

Re: [Mesa-dev] radeon cmask sync with EGL/GBM

2016-01-11 Thread Axel Davy
On 11/01/2016 15:39, Marek Olšák wrote: On Mon, Jan 11, 2016 at 9:42 AM, Axel Davy <axel.d...@ens.fr> wrote: On 10/01/2016 19:38, Jay Cornwall wrote: Hi, Using EGL with the GBM platform it is possible to bind a GBM surface to a GL renderbuffer. I believe it's also valid to use the

Re: [Mesa-dev] [PATCH] radeonsi: enable late VS export memory allocation

2016-01-13 Thread Axel Davy
> > Axel Davy benchmarked this briefly. We may need more benchmarks though. > > Marek > I confirm setting this register helps get a few % with heaven. There was also another register to kill color exports early when doing depth only pass that helped a few % (b

[Mesa-dev] Are gallium unpack_rgba_8unorm/pack_rgba_8unorm safe for in-place conversion ?

2016-02-09 Thread Axel Davy
to be safe for in-place conversion ? Could we add that somewhere as requirement ? CC-ing vmware guys, as they probably know. Yours, Axel Davy ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] Are gallium unpack_rgba_8unorm/pack_rgba_8unorm safe for in-place conversion ?

2016-02-09 Thread Axel Davy
Hi again, Actually, "util_format_translate" seems to fit our needs. It could probably be optimised better (8unorm fitting format->ARGB could avoid using tmp buffer for example), but that should be ok for our needs. Yours, Axel Davy On 09/02/2016 20:17, Axel Davy wrote: Hi, We

[Mesa-dev] [PATCH v2] st/nine: Implement Managed vertex/index buffers

2016-02-06 Thread Axel Davy
to update_managed_buffers Initialize NineBuffer9 managed fields after the resource is allocated. In case of allocation failure, when the dtor is executed, This->base.pool is then rightfully set. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/buffer9.

[Mesa-dev] [PATCH] st/nine: Implement Managed vertex/index buffers

2016-02-06 Thread Axel Davy
We were implementing those the same way than the default pool, which is sub-optimal. The buffer is supposed to return pointer to a ram copy when user locks, and automatically update the vram copy when needed. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- This fixes Skyrim npcs causin

[Mesa-dev] [PATCH 06/12] st/nine: Support ATI1/ATI2 for CubeTexture

2016-02-07 Thread Axel Davy
Texture and CubeTexture use common code, and thus ATI1/ATI2 is already implemented for CubeTexture. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/adapter9.c | 5 +++-- src/gallium/state_trackers/nine/cubetexture9.c | 4 src/gallium/state_tr

[Mesa-dev] [PATCH 05/12] st/nine: Clean pSharedHandle Texture ctors checks

2016-02-07 Thread Axel Davy
Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/cubetexture9.c | 7 --- src/gallium/state_trackers/nine/texture9.c | 25 src/gallium/state_trackers/nine/volumetexture9.c | 7 --- 3 files changed, 21 insertions(

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

2016-02-07 Thread Axel Davy
The last patch awaits moderation because of its size, you can find it here: https://github.com/iXit/Mesa-3D/commit/29e2ccf64273814071655d84aca69b6496fbb4bd On 08/02/2016 00:13, Axel Davy wrote: A few more patches I'd like to get in 11.2. There a few cleanup patches and some fixes. The last

[Mesa-dev] [PATCH 08/12] st/nine: SCRATCH does support all formats

2016-02-07 Thread Axel Davy
Add new argument to d3d9_to_pipe_format_checked to be able to bypass format support checks. This argument is set to TRUE when the requested Pool is SCRATCH. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/adapter9.c | 21 +++-- src/g

[Mesa-dev] [PATCH 07/12] st/nine: Add format checks to create_zs_or_rt_surface

2016-02-07 Thread Axel Davy
supported. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index b6e75b4..3ebff3a 100644 --- a/src/g

[Mesa-dev] [PATCH 10/12] st/nine: Remove usage of SQRT in ff code

2016-02-07 Thread Axel Davy
SQRT is not supported everywhere, so replace it by RSQ + RCP Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_tr

[Mesa-dev] [PATCH 09/12] st/nine: Fix stateblocks crashes with lights

2016-02-07 Thread Axel Davy
We had several issues of crashes with it. This should fix it. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/stateblock9.c | 40 +-- 1 file changed, 32 insertions(+), 8 deletions(-) diff --git a/src/gallium/state_trackers/nine/state

[Mesa-dev] [PATCH 04/12] st/nine: Move texture creation checks

2016-02-07 Thread Axel Davy
We were having checks at both Create*Texture functions and in ctors. Move all Create*Texture checks to ctors. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/cubetexture9.c | 2 ++ src/gallium/state_trackers/nine/device9.c| 13

[Mesa-dev] [PATCH 01/12] st/nine: Do not set resource usage for SYSTEMMEM

2016-02-07 Thread Axel Davy
We do not create a resource for SYSTEMMEM textures, thus we do not need to set resource usage. The only exception is vertexbuffer SYSTEMMEM, since we do use a pipe resource for them. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/surface9.c | 11 --

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

2016-02-07 Thread Axel Davy
SSE code and llvm work correctly. I think the better is to just realign at d3d entry points. Any suggestion whether that last patch should be sent to mesa stable or not ? Yours. Axel Davy (12): st/nine: Do not set resource usage for SYSTEMMEM st/nine: Do not set SHARED flag for shared textures

[Mesa-dev] [PATCH 02/12] st/nine: Do not set SHARED flag for shared textures.

2016-02-07 Thread Axel Davy
We do not support shared textures, thus no need to set the shared flag. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/texture9.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/texture9.c b/src/gallium/state_tracker

[Mesa-dev] [PATCH 11/12] st/nine: Drop path for ureg_NRM and ureg_CLAMP

2016-02-07 Thread Axel Davy
using MIN/MAX is fine instead of CLAMP. NRM doesn't exist anymore. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 28 1 file changed, 4 insertions(+), 24 deletions(-) diff --git a/src/gallium/state_trackers/nine/nin

[Mesa-dev] [PATCH 03/12] st/nine: Clean useless code in texture9.c

2016-02-07 Thread Axel Davy
This->base.base.resource is worth NULL for SYSTEMMEM textures. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/texture9.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/gallium/state_trackers/nine/texture9.c b/sr

Re: [Mesa-dev] [PATCH 10/12] st/nine: Remove usage of SQRT in ff code

2016-02-07 Thread Axel Davy
On 08/02/2016 00:21, Ilia Mirkin wrote: On Sun, Feb 7, 2016 at 6:13 PM, Axel Davy <axel.d...@ens.fr> wrote: SQRT is not supported everywhere, so replace it by RSQ + RCP Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 3 ++- 1 fil

Re: [Mesa-dev] Are gallium unpack_rgba_8unorm/pack_rgba_8unorm safe for in-place conversion ?

2016-02-09 Thread Axel Davy
On 09/02/2016 20:30, Jose Fonseca wrote: On 09/02/16 19:17, Axel Davy wrote: Hi, We'd need to do some formats conversion in gallium nine, and if possible we would like to do them in-place. unpack_rgba_8unorm/pack_rgba_8unorm doesn't seem to explicitly allow in-place conversion

[Mesa-dev] [PATCH 12/45] st/nine: Don't increment refcount on VertexDeclaration creation failure

2016-01-29 Thread Axel Davy
From: Patrick Rudolph <s...@das-labor.org> NineUnknown_ctor increments the refcount even in case of an error. Restructure the code to prevent refcount increments. Fixes a couple of wine tests. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <a

[Mesa-dev] [PATCH 02/45] st/nine: Fix D3DPMISCCAPS_FOGANDSPECULARALPHA support

2016-01-29 Thread Axel Davy
than the doc. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/adapter9.c| 2 +- src/gallium/state_trackers/nine/nine_shader.c | 12 +--- 2 files changed, 2 insertions(+), 12 deletions(-) diff

[Mesa-dev] [PATCH 03/45] st/nine: Fix rounding issue with vs1.1 a0 reg

2016-01-29 Thread Axel Davy
-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_shader.c | 29 +++ 1 file changed, 7 insertions(+), 22 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_shader.c b

[Mesa-dev] [PATCH 43/45] st/nine: Enforce centroid for color input when multisampling is on

2016-01-29 Thread Axel Davy
The color inputs must automatically use centroid whether multisampling is used or not. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_shader.c | 12 src/gallium/state_trackers/nine/n

[Mesa-dev] [PATCH 45/45] st/nine: Use align_free when needed

2016-01-29 Thread Axel Davy
Use align_free to free memory allocated with align_malloc. Signed-off-by: Axel Davy <axel.d...@ens.fr> Reviewed-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/cubetexture9.c | 2 +- src/gallium/state_trackers/nine/surface9.c | 2 +- src/gallium/st

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