[Mesa-dev] [PATCH v2 04/52] st/nine: Add missing c++ declaration for IDirect3DVolumeTexture9

2015-01-10 Thread Axel Davy
From: Xavier Bouchoux Reviewed-by: Ilia Mirkin Reviewed-by: David Heidelberg Reviewed-by: Axel Davy Signed-off-by: Xavier Bouchoux Cc: "10.4" --- include/D3D9/d3d9.h | 10 ++ 1 file changed, 10 insertions(+) diff --git a/include/D3D9/d3d9.h b/include/D3D9/d3d9.h ind

[Mesa-dev] [PATCH v2 06/52] st/nine: Fix D3DRS_POINTSPRITE support

2015-01-10 Thread Axel Davy
From: Xavier Bouchoux It's done by testing the existence of the point sprite output register *after* parsing the vertex shader. Reviewed-by: Ilia Mirkin Reviewed-by: David Heidelberg Reviewed-by: Axel Davy Signed-off-by: Xavier Bouchoux Cc: "10.4" --- src/gallium/stat

[Mesa-dev] [PATCH v2 11/52] st/nine: Return D3DERR_INVALIDCALL when trying to create a texture of bad format

2015-01-10 Thread Axel Davy
Reviewed-by: David Heidelberg Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/cubetexture9.c | 10 +- src/gallium/state_trackers/nine/texture9.c | 9 - src/gallium/state_trackers/nine/volumetexture9.c | 10 +- 3 files changed, 26 insertions(+), 3

Re: [Mesa-dev] [PATCH 12/53] st/nine: NineBaseTexture9: update sampler view creation

2015-01-10 Thread Axel Davy
On 07/01/2015 18:16, Ilia Mirkin wrote : On Wed, Jan 7, 2015 at 11:36 AM, Axel Davy wrote: While previous code was having the correct behaviour in general, this new code is more readable (without checking all gallium formats manually) and has a more defined behaviour for depth stencil

[Mesa-dev] [PATCH v2 16/52] st/nine: Rework of boolean constants

2015-01-10 Thread Axel Davy
Convert them to shader booleans at earlier stage. Previous code is fine, but later patch will make integers being converted at earlier stage, so do the same for booleans Signed-off-by: Axel Davy Cc: "10.4" --- src/gallium/state_trackers/nine/device9.c| 35 +--

[Mesa-dev] [PATCH v2 18/52] st/nine: Remove some shader unused code

2015-01-10 Thread Axel Davy
Signed-off-by: Axel Davy Cc: "10.4" --- src/gallium/state_trackers/nine/nine_shader.c | 23 +-- 1 file changed, 1 insertion(+), 22 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_shader.c b/src/gallium/state_trackers/nine/nine_shader.c index fcc1c6

[Mesa-dev] [PATCH v2 27/52] st/nine: Rewrite LOOP implementation, and a0 aL handling

2015-01-10 Thread Axel Davy
crash while loading Bioshock - bug https://bugs.freedesktop.org/show_bug.cgi?id=85696 Tested-by: David Heidelberg Signed-off-by: Axel Davy Cc: "10.4" --- src/gallium/state_trackers/nine/nine_shader.c | 163 -- 1 file changed, 100 insertions(+), 63 deletions(-

[Mesa-dev] [PATCH v2 28/52] st/nine: Match REP implementation to LOOP

2015-01-10 Thread Axel Davy
Previous implementation was fine, just instead of having increasing counter, have a decreasing counter. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_shader.c | 49 --- 1 file changed, 30 insertions(+), 19 deletions(-) diff --git a/src/gallium

[Mesa-dev] [PATCH v2 29/52] st/nine: Fix CND implementation

2015-01-10 Thread Axel Davy
Signed-off-by: Axel Davy Signed-off-by: Tiziano Bacocco Cc: "10.4" --- src/gallium/state_trackers/nine/nine_shader.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_shader.c b/src/gallium/state_tra

[Mesa-dev] [PATCH v2 34/52] st/nine: Implement TEXCOORD special behaviours

2015-01-10 Thread Axel Davy
off-by: Axel Davy Cc: "10.4" --- src/gallium/state_trackers/nine/nine_shader.c | 33 +++ 1 file changed, 28 insertions(+), 5 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_shader.c b/src/gallium/state_trackers/nine/nine_shader.c index b7e0c81..0

Re: [Mesa-dev] [PATCH 44/53] st/nine: Implement ps3 advanced input definition feature

2015-01-10 Thread Axel Davy
On 08/01/2015 05:01, Ilia Mirkin wrote : On Wed, Jan 7, 2015 at 11:36 AM, Axel Davy wrote: ps3 allows definitions of the inputs like: DCL_TEXCOORD0 v0.xy; Could this be .xw? I assume not... but if it can, that's not handled in this change. I think this patch should be improved, and

[Mesa-dev] [PATCH v2 50/52] st/nine: Explicit nine requirements

2015-01-10 Thread Axel Davy
upport for them. We are already having hard time supporting r500, which is the most flexible dx9-only card apparently. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c | 106 + src/gallium/state_trackers/nine/device9.c | 9 +-- 2 files c

[Mesa-dev] [PATCH v2 51/52] st/nine: Change comment relating to vertex shader inputs not matching declaration

2015-01-10 Thread Axel Davy
Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_state.c b/src/gallium/state_trackers/nine/nine_state.c index 1187a20..dad6651 100644 --- a/src/gallium

Re: [Mesa-dev] [PATCH v2 28/52] st/nine: Match REP implementation to LOOP

2015-01-11 Thread Axel Davy
the loop is stopped early. On Sat, Jan 10, 2015 at 5:42 AM, Axel Davy wrote: Previous implementation was fine, just instead of having increasing counter, have a decreasing counter. Didn't you have some other reason to do this? Why are there all these changes which are basically just like &

Re: [Mesa-dev] [PATCH 19/53] st/nine: Clamp color inputs for ps <= 2.0 at ps level instead of vs

2015-01-11 Thread Axel Davy
On 07/01/2015 17:36, Axel Davy wrote : Nine code was clamping color outputs for vs < 3, but msdn docs says it is done in the ps. Wine seems to clamp them at the vs level. It makes more sense to clamp at vs level for performance, but according to doc, ps 2.x shouldn't see clamping.

[Mesa-dev] [PATCH v3 33/51] st/nine: Implement TEXCOORD special behaviours

2015-01-11 Thread Axel Davy
ace DIV by RCP + MUL v3: Remove an useless MOV Signed-off-by: Axel Davy Cc: "10.4" --- src/gallium/state_trackers/nine/nine_shader.c | 31 ++- 1 file changed, 26 insertions(+), 5 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_shader.c

Re: [Mesa-dev] [PATCH v2 50/52] st/nine: Explicit nine requirements

2015-01-13 Thread Axel Davy
ble performance boost from unplayable 10 fps to 20-25fps fps. As in previous conversation we had, I'd prefer pushing this patch only at moment, when it's already required by some rework or something which completly disallow < r500 to work. I'm ok with droppi

[Mesa-dev] [PATCH] egl: Soften several HAVE_DRM_PLATFORM to HAVE_LIBDRM

2015-01-14 Thread Axel Davy
built. Signed-off-by: Axel Davy --- src/egl/drivers/dri2/egl_dri2.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c index 86e5f24..6306483 100644 --- a/src/egl/drivers/dri2/egl_dri2.c +++ b/src/egl

[Mesa-dev] [PATCH] gallium: add MULTISAMPLE_Z_RESOLVE cap

2015-01-17 Thread Axel Davy
add it to the drivers. It is advertised for drivers for which it is sure the ability is supported. A comment was added for drivers for which the feature is probably supported. Signed-off-by: Axel Davy --- This feature corresponds to the RESZ d3d9 hack. d3d9 hacks are equivalent to GL extensions. R

Re: [Mesa-dev] [PATCH] gallium: add MULTISAMPLE_Z_RESOLVE cap

2015-01-18 Thread Axel Davy
and nouveau the check for the support of the sample count and the bind flag are separated. On 17/01/2015 14:44, Axel Davy wrote : Resolving a multisampled depth texture into a single sampled texture is supported on >= SM4.1 hw. It is possible some previous hw support it. The ability was tested

Re: [Mesa-dev] [PATCH] gallium: add MULTISAMPLE_Z_RESOLVE cap

2015-01-18 Thread Axel Davy
etter would be to fix is_format_supported in drivers, and check if a multisampled buffer can be sampled in a shader. Axel Davy On 18/01/2015 20:23, Ilia Mirkin wrote : I'm confused by what this flag is allowing. Are we talking about a blit from MS -> non-MS? texelFetch(sampler2DMS) for a

Re: [Mesa-dev] [PATCH 12/53] st/nine: NineBaseTexture9: update sampler view creation

2015-01-18 Thread Axel Davy
n Win. I'm going to update the patch. On 07/01/2015 17:36, Axel Davy wrote : While previous code was having the correct behaviour in general, this new code is more readable (without checking all gallium formats manually) and has a more defined behaviour for depth stencil resources. Reviewed

Re: [Mesa-dev] [PATCH] gallium: add MULTISAMPLE_Z_RESOLVE cap

2015-01-19 Thread Axel Davy
akes sense to add the cap just to say this behaviour is what happens with a pipe_blit, since it is different than the ability to read the texture in a shader, like the is_format_supported would indicate. Axel Davy ___ mesa-dev mailing list mesa-

[Mesa-dev] [PATCH v3 13/51] st/nine: NineBaseTexture9: update sampler view creation

2015-01-19 Thread Axel Davy
While previous code was having the correct behaviour in general, this new code is more readable (without checking all gallium formats manually) and has a more defined behaviour for depth stencil resources. Reviewed-by: Tiziano Bacocco Reviewed-by: David Heidelberg Signed-off-by: Axel Davy Cc

[Mesa-dev] [PATCH v2] gallium: Add MULTISAMPLE_Z_RESOLVE cap

2015-01-19 Thread Axel Davy
dds it to the drivers. It is advertised for drivers for which it is sure the ability is supported. Signed-off-by: Axel Davy --- v2: Add mention of the first sample to the definition. Remove comment about possible support for drivers that don't support MSAA yet. src/gallium/docs/source

Re: [Mesa-dev] [PATCH v2] gallium: Add MULTISAMPLE_Z_RESOLVE cap

2015-01-19 Thread Axel Davy
On 19/01/2015 23:23, Ilia Mirkin wrote : On Mon, Jan 19, 2015 at 5:17 PM, Axel Davy wrote: Resolving a multisampled depth texture into a single sampled texture is supported on >= SM4.1 hw. It is possible some previous hw support it. The ability was tested on radeonsi and nvc0. I'm n

Re: [Mesa-dev] Thoughts after hitting 100 merge requests?

2019-01-12 Thread Axel Davy
So far I only went check the list of gitlab merge requests 3 times, whereas I go through my mails several times a day. Yours, Axel Davy On 11/01/2019 17:57, Jason Ekstrand wrote: All, The mesa project has now hit 100 merge requests (36 are still open).  I (and I'm sure others) wou

[Mesa-dev] [PATCH v2] st/nine: Enable debug info if NDEBUG is not set

2019-01-13 Thread Axel Davy
We want to have debug info as well if using meson's debugoptimized when ndebug is off. v2: use u_debug functions that do something even if DEBUG is not set. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 6 +++--- src/gallium/state_trackers/nine/basetextu

Re: [Mesa-dev] [PATCH] radeonsi: use compute for resource_copy_region when possible

2019-01-15 Thread Axel Davy
with dma_copy if no conversion is needed or with a shader using buffers (no images), which would avoid using the image sampling hw which I believe can be more limiting than sampling a buffer when there is a lot of wavefronts. The data conversion can be done for no cost in the shader as it shou

[Mesa-dev] [PATCH] st/nine: Immediately upload user provided textures

2019-01-22 Thread Axel Davy
s the csmt queue right away after queuing the upload for this type of textures. This is more future-proof, as we may want to bind the surface for other reasons in the future. Signed-off-by: Axel Davy Cc: 18.3 --- The regression affects Mesa 18.3.2. At least HL2 lost coast is affected, has artifacts a

Re: [Mesa-dev] [PATCH 4/4] radeonsi: use SDMA for uploading data through const_uploader

2019-02-07 Thread Axel Davy
On 07/02/2019 02:22, Marek Olšák wrote: + bool use_sdma_upload = sscreen->info.has_dedicated_vram && sctx->dma_cs && debug_get_bool_option("SDMA", true); + sctx->b.const_uploader = u_upload_create(&sctx->b, 256 * 1024, +0, PIPE_USAGE_DEFAUL

Re: [Mesa-dev] [PATCH 00/26] RadeonSI: Primitive culling with async compute

2019-02-12 Thread Axel Davy
On 13/02/2019 06:15, Marek Olšák wrote: I decided to enable this optimization on all Pro graphics cards. The reason is that I haven't had time to benchmark games. This decision may be changed based on community feedback, etc. Could the decision to run the optimization be based on some perf co

Re: [Mesa-dev] [PATCH 00/26] RadeonSI: Primitive culling with async compute

2019-02-13 Thread Axel Davy
On 13/02/2019 17:42, Marek Olšák wrote: On Wed, Feb 13, 2019 at 2:28 AM Axel Davy <mailto:davyax...@gmail.com>> wrote: On 13/02/2019 06:15, Marek Olšák wrote: > I decided to enable this optimization on all Pro graphics cards. > The reason is that I haven't h

[Mesa-dev] [PATCH 1/2] st/nine: Ignore window size if error

2019-02-27 Thread Axel Davy
github.com/iXit/Mesa-3D/issues/331 https://github.com/iXit/Mesa-3D/issues/332 Fixes also crash at window destruction. Cc: mesa-sta...@lists.freedesktop.org Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/swapchain9.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) di

[Mesa-dev] [PATCH 2/2] st/nine: Ignore multisample quality level if no ms

2019-02-27 Thread Axel Davy
Apparently instead of returning error when passing a quality level different than 0 for D3DMULTISAMPLE_NONE, we should pass. Fixes: https://github.com/iXit/Mesa-3D/issues/340 Cc: mesa-sta...@lists.freedesktop.org Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_pipe.h | 4

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/2] st/nine: Ignore window size if error

2019-02-28 Thread Axel Davy
On 28/02/2019 12:54, Emil Velikov wrote: On Wed, 27 Feb 2019 at 22:49, Axel Davy wrote: Check GetWindowInfo and ignore the computed sizes if there is an error. Fixes the regression caused by: commit 2318ca68bbeb4fa6e21a4d8c650cec3f64246596 "st/nine: Handle window resize when a present

Re: [Mesa-dev] [Mesa-stable] [PATCH] st/nine: Immediately upload user provided textures

2019-03-06 Thread Axel Davy
On 06/03/2019 20:13, Dylan Baker wrote: Quoting Axel Davy (2019-01-22 12:08:05) Fixes regression caused by 42d672fa6a766363e5703f119607f7c7975918aa st/nine: Bind src not dst in nine_context_box_upload Before that patch, for user provided textures, when the texture was destroyed, the safety

[Mesa-dev] [PATCH 3/5] st/nine: Do not advertise support for D15S1 and D24X4S4

2019-03-07 Thread Axel Davy
The former is supported on Matrox cards but no other hw. The latter isn't supported anywhere. It is fine to not advertise them as supported, and it could prevent apps to trigger weird rendering paths. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_pipe.c | 4 ++-- 1

[Mesa-dev] [PATCH 1/5] st/nine: Disable depth write when nothing gets updated

2019-03-07 Thread Axel Davy
I do not see any perf impact on radeonsi, but it seems iris needs this. It seems something sensible to do. Signed-off-by: Axel Davy Reviewed-by: Timur Kristóf Tested-by: Andre Heider --- It may be argued this kind of stuff should be done in the driver. I don't mind either way. The ogl

[Mesa-dev] [PATCH 2/5] d3dadapter9: Support software renderer on any DRI device

2019-03-07 Thread Axel Davy
From: Patrick Rudolph If D3D_ALWAYS_SOFTWARE is set for debugging purposes, run on any DRI enabled platform. Instead of probing for a compatible gallium driver (which might fail if there's none) always use the KMS DRI software renderer. Allows to run nine on i915 when D3D_ALWAYS_SOFTWARE=1. Sig

[Mesa-dev] [PATCH 4/5] st/nine: Do not advertise CANMANAGERESOURCE

2019-03-07 Thread Axel Davy
It doesn't seem the main vendors advertise it. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/adapter9.c b/src/gallium/state_trackers/nine/adapter9.c index 0634d5

[Mesa-dev] [PATCH 5/5] st/nine: Change a few advertised caps

2019-03-07 Thread Axel Davy
ertise the cap. D3DCURSORCAPS_LOWRES: I don't know what is the status of this on X11, but I don't know of any dx9 game running at height < 400 either. D3DPTEXTURECAPS_TEXREPEATNOTSCALEDBYSIZE: The cap should correspond to what the current generation of hw is doing. Signed-off-by: Axel

Re: [Mesa-dev] [PATCH 1/5] st/nine: Disable depth write when nothing gets updated

2019-03-09 Thread Axel Davy
On 08/03/2019 07:26, Kenneth Graunke wrote: On Thursday, March 7, 2019 2:23:53 PM PST Axel Davy wrote: I don't think we actually need the NEVER check, but it seems fine. Patches 1 and 3 are: Reviewed-by: Kenneth Graunke I'm not really up to speed to review the others. Thanks, I go

Re: [Mesa-dev] [PATCH 2/5] d3dadapter9: Support software renderer on any DRI device

2019-03-09 Thread Axel Davy
After pushing this (with my r-b), gitlab's travis complained. It seems there needs to be some ifdefs to check if kms was built. I reverted the patch until it is fixed. Axel On 07/03/2019 23:23, Axel Davy wrote: From: Patrick Rudolph If D3D_ALWAYS_SOFTWARE is set for debugging purposes

Re: [Mesa-dev] [PATCH] gallium/docs: clarify set_sampler_views

2019-03-15 Thread Axel Davy
On 15/03/2019 03:12, Rob Clark wrote: On Thu, Mar 14, 2019 at 9:58 PM Roland Scheidegger wrote: Am 15.03.19 um 02:18 schrieb Rob Clark: On Thu, Mar 14, 2019 at 8:28 PM Roland Scheidegger wrote: Am 14.03.19 um 22:06 schrieb Rob Clark: On Thu, Mar 14, 2019 at 3:58 PM Roland Scheidegger wrote

Re: [Mesa-dev] [PATCH] gallium/docs: clarify set_sampler_views

2019-03-15 Thread Axel Davy
On 15/03/2019 13:12, Rob Clark wrote: On Fri, Mar 15, 2019 at 3:49 AM Axel Davy wrote: On 15/03/2019 03:12, Rob Clark wrote: On Thu, Mar 14, 2019 at 9:58 PM Roland Scheidegger wrote: Am 15.03.19 um 02:18 schrieb Rob Clark: On Thu, Mar 14, 2019 at 8:28 PM Roland Scheidegger wrote: Am

Re: [Mesa-dev] [PATCH] st/nine: enable csmt per default on iris

2019-03-20 Thread Axel Davy
/* We rely on u_upload_mgr using persistent coherent buffers (which don't * require flush to work in multi-pipe_context scenario) for vertex and Could have been an || inside the same if, but maybe it is easier to read that way. Reviewed-by: Axel Davy Axel _

Re: [Mesa-dev] [PATCH] gallium: remove support for predicates from TGSI

2015-09-01 Thread Axel Davy
just remove it. The variable isn't used. Axel Davy On 01/09/2015 23:19, Marek Olšák wrote : From: Marek Olšák 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 wh

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

2015-09-14 Thread Axel Davy
Yours, 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

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

2015-09-14 Thread Axel Davy
keeping the logic in patch 2 (means the feature can be reimplemented more easily), and I can add myself the comment in nine code. Thus the series is: Reviewed-by: Axel Davy ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

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

2015-10-10 Thread Axel Davy
or 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/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. T

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 a/src/gallium/state_tracker

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 a/src/gallium/drivers/radeonsi/si_state_sha

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 wrote: 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

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 sy

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 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 GBM BO as a frameb

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] 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

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

2016-01-19 Thread Axel Davy
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 +- src/gallium/drivers/radeons

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

2016-01-21 Thread Axel Davy
Add a debug option to select the LLVM SI Machine Scheduler. R600_DEBUG=sisched Signed-off-by: Axel Davy --- 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/radeon/r600_pipe_common.h | 1

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

2016-01-27 Thread Axel Davy
There was this other version http://lists.freedesktop.org/archives/mesa-dev/2015-October/097917.html Reviewed-by: Axel Davy On 27/01/2016 10:34, Michel Dänzer wrote: From: Michel Dänzer The compositor may have the hardware scan out directly from the buffers sent by the client, so we must

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

2016-01-27 Thread Axel Davy
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 compositor wants to scan

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph AlphaCmpCaps should advertise D3DPCMPCAPS_NEVER as well. Fixes https://github.com/iXit/Mesa-3D/issues/142 Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph Implement fixed function D3DRS_SPECULARENABLE. Fixes failing WINE tests in test_specular_lighting() (visual.c) Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 13 +++-- 1 file changed, 11 insertions(+), 2

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

2016-01-29 Thread Axel Davy
ow do . 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 12/45] st/nine: Don't increment refcount on VertexDeclaration creation failure

2016-01-29 Thread Axel Davy
From: Patrick Rudolph 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 Reviewed-by: Axel Davy --- .../state_trackers/nine/vertexdeclaration9.c | 24

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph 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 Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/surface9.c | 2 +- 1

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

2016-01-29 Thread Axel Davy
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 Reviewed-by: Patrick Rudolph --- src/gallium/state_trackers/nine/swapchain9.c | 10 -- 1 file changed, 4

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

2016-01-29 Thread Axel Davy
Only argb cursors are allowed. Signed-off-by: Axel Davy Reviewed-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index 593f2b5

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

2016-01-29 Thread Axel Davy
. 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 Revie

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph Replace r[0] with tmp to ease code reading. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/src/gallium/state_trackers

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph 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 Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 17 + 1 file changed, 13

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph Tests show in case of multisample mismatch between the depth-stencil buffer and the render target, then it is not cleared. Fixes failing WINE test visual.c test_multisample_mismatch(). Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers

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

2016-01-29 Thread Axel Davy
We forgot to reset vtxbuf. This fixes some crashes. Signed-off-by: Axel Davy Reviewed-by: Patrick Rudolph --- src/gallium/state_trackers/nine/nine_state.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_state.c b/src/gallium

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph 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 Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/Makefile.sources | 2 + src/gallium

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph Return errors in case of invalid presentation parameters. Fixes failing WINE tests device.c test_swapchain_parameters(). Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/swapchain9.c | 13 - 1 file changed, 8

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

2016-01-29 Thread Axel Davy
o match the expected optimisation: (Value == previous Value) => return immediately, which affects D3D9 hacks too. Signed-off-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c | 33 ++-- src/gallium/state_trackers/nine/nine_limits.h | 211

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph To ease debugging print the GUID instead of the pointer to it. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/guid.c | 18 ++ src/gallium/state_trackers/nine/guid.h | 4 src/gallium

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

2016-01-29 Thread Axel Davy
Some apps do redundant SetStreamSourceFreq calls. Catch them to improve performance. Signed-off-by: Axel Davy Reviewed-by: Patrick Rudolph --- 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

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

2016-01-29 Thread Axel Davy
the device one. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- include/D3D9/d3d9types.h | 1 + include/d3dadapter/present.h | 3 ++ src/gallium/state_trackers/nine/device9.c| 19 +-- src/gallium/state_trackers/nine/device9.h| 3

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph 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 Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph Clear every rendertarget on creation. Fixes https://github.com/iXit/Mesa-3D/issues/139 Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/surface9.c | 9 + 1 file changed, 9 insertions(+) diff --git a/src/gallium

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph In case swapchain creation fails This->swapchains[i] might be NULL and causes a crash. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gall

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph Use CSO to catch redundant viewport changes. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_state.c b/src

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph The values of box.z and box.depth weren't set and lead to a crash. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/auxiliary/util/u_box.h | 12 src/gallium/state_trackers/nine/basetexture9.c | 2 +- 2 files change

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

2016-01-29 Thread Axel Davy
erade - 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 Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 19 ++

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

2016-01-29 Thread Axel Davy
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 --- src/gallium/state_trackers/nine/nine_shader.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/gal

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph Add config option override_vendorid to report a fake card in d3dadapter9 drm. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/targets/d3dadapter9/drm.c | 61 +++-- src/mesa/drivers/dri/common/xmlpool/t_options.h | 5

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

2016-01-29 Thread Axel Davy
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 Reviewed-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c | 7 --- 1 file ch

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

2016-01-29 Thread Axel Davy
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 Reviewed-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c | 4

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph 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 Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine

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

2016-01-29 Thread Axel Davy
stead of 4. Signed-off-by: Axel Davy Reviewed-by: Patrick Rudolph --- 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 +- src/gallium/state_trackers/nine/swapchain9.c | 1

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

2016-01-29 Thread Axel Davy
ency, no update 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 Reviewed-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c

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

2016-01-29 Thread Axel Davy
(reversed). Fixes a crash in TmNationsForever. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/nine_pipe.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_pipe.c b/src/gallium

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

2016-01-29 Thread Axel Davy
From: Patrick Rudolph 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 Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 4 ++-- 1 file changed, 2 insertions(+), 2

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

2016-01-29 Thread Axel Davy
/issues/130 Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/nine_pdata.h | 1 + src/gallium/state_trackers/nine/resource9.c | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_pdata.h b/src/gallium

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

2016-01-29 Thread Axel Davy
WINE's device.c test_swapchain() tests. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 3 +++ src/gallium/state_trackers/nine/swapchain9.c | 3 ++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_tra

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