I don't have push access.
Could you push it ?
On 01/07/2014 11:22, Brian Paul wrote :
That works.
Tested-by: Brian Paul bri...@vmware.com
On 07/01/2014 09:19 AM, Axel Davy wrote:
Hi,
This path should resolve the problem.
Axel
On 01/07/2014 10:53, Axel Davy wrote :
In src/loader
section.
Yours,
Axel Davy
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Hi,
When reading si_dma.c code, it looks like the requested width of the
copy is ignored except for PIPE_BUFFER.
Perhaps that explains the bugs observed ?
Axel Davy
On 06/09/2014 19:14, Andy Furniss wrote :
Marek Olšák wrote:
From: Marek Olšák marek.ol...@amd.com
---
Turn this on, run
On 08/09/2014 20:21, Grigori Goronzy wrote :
On 08.09.2014 14:50, Axel Davy wrote:
Hi,
When reading si_dma.c code, it looks like the requested width of the
copy is ignored except for PIPE_BUFFER.
Perhaps that explains the bugs observed ?
It isn't ignored. Partial DMA copies (i.e. operations
Le 08/09/2014 21:11, Grigori Goronzy a écrit :
On 08.09.2014 21:07, Axel Davy wrote:
On 08/09/2014 20:21, Grigori Goronzy wrote :
On 08.09.2014 14:50, Axel Davy wrote:
Hi,
When reading si_dma.c code, it looks like the requested width of the
copy is ignored except for PIPE_BUFFER.
Perhaps
On 20/10/2014 04:32, Michel Dänzer wrote :
On 18.10.2014 20:55, David Heidelberger wrote:
From: Axel Davy axel.d...@ens.fr
[...]
+#define DRI_CONF_NINE_VSYNC(def) \
+DRI_CONF_OPT_BEGIN_V(vsync_force,enum,def,0:2) \
+DRI_CONF_DESC_BEGIN(en,gettext(Force the usage of vsync
On 20/10/2014 09:12, Michel Dänzer wrote :
On 20.10.2014 14:47, Axel Davy wrote:
On 20/10/2014 04:32, Michel Dänzer wrote :
On 18.10.2014 20:55, David Heidelberger wrote:
From: Axel Davy axel.d...@ens.fr
[...]
+#define DRI_CONF_NINE_VSYNC(def) \
+DRI_CONF_OPT_BEGIN_V(vsync_force,enum,def
to 0.0 instead of 1.0.
I don't know the code enough to know if it's a good change or not, but I
guess it's likely to be an error.
Discussing with David Heidelberger, the patch will likely be dropped.
Axel Davy
On 20/10/2014 17:37, Brian Paul wrote :
I don't understand the need
It's only used to blend the cursor.
If you check the second version of the series, the patch was dropped.
Axel Davy
On 20/10/2014 22:18, Jose Fonseca wrote :
I also can't see where this is being used in
https://github.com/iXit/Mesa-3D/commit/9671cfd9bdad5c310b34c57befcfd7c43034b925.patch
backends.
Thus I think it's ok the Mesa part builds, even if no non-DRI3 backend
exists yet and that DRI3 is not available on the system.
Axel Davy
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo
for a resize
and after swapbuffers.
Axel Davy
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
explicitly
enable/disable Nine support in wine, and if it is enabled and fails to
load, it will fallback to normal wine support. User gets a message in
console to tell him why it failed/ if Nine is activated.
I hope these arguments will help convince you,
Axel Davy
Le 14/11/2014 18:40, Henri Verbeet a écrit :
On 14 November 2014 17:52, Axel Davy axel.d...@ens.fr wrote:
Second d3d9 as gallium state tracker seems much easier than d3d9 on OpenGL.
As for me, I contributed only since a few months ago, and was able to
implement a lot of things quite easily
-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/Makefile.sources | 2 +
src/gallium/state_trackers/nine/adapter9.h | 1 +
src/gallium/state_trackers/nine/swapchain9.c | 86 +--
src/gallium/state_trackers/nine/swapchain9.h | 7 +
src/gallium
From: Christoph Bumiller christoph.bumil...@speed.at
v3: thanks to Brian, improved coding style, also glennk helped spot few
things (unsigned - int, two constify)
v4: thanks Ilia improved function, dropped u_box_clip_3d
v5: incorporated rest of Gregor proposed changes,clean ups
v6: u_box_clip_2d
Implements vblank_mode and throttling, which allows us change default ratio
between framerate and input lag.
Signed-off-by: David Heidelberg da...@ixit.cz
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/adapter9.h | 1 +
src/gallium/state_trackers/nine
Hi,
Here is last (4th) iteration of Gallium Nine patches.
We have integrated the new feedback we have got and hope
the status of the serie is good enough now for merge.
Thanks,
Axel Davy
Axel Davy (2):
nine: Add drirc options (v2)
nine: Implement threadpool
Christoph Bumiller (5
From: Christoph Bumiller christoph.bumil...@speed.at
At this moment we use only zero or positive values.
v2: Implement it for also for Solaris, MSVC assembly
and enable for other combinations.
Signed-off-by: David Heidelberg da...@ixit.cz
---
src/gallium/auxiliary/util/u_atomic.h | 78
From: Christoph Bumiller christoph.bumil...@speed.at
v2: moved in in same order as in p_shader_tokens (thanks Brian)
Reviewed-by: Marek Olšák marek.olsak at amd.com
Signed-off-by: David Heidelberg da...@ixit.cz
---
src/gallium/auxiliary/tgsi/tgsi_opcode_tmp.h | 1 +
1 file changed, 1
From: Christoph Bumiller christoph.bumil...@speed.at
Implement pipe_loader_sw_probe_wrapped which allows to use the wrapped
software renderer backend when using the pipe loader.
v2: - remove unneeded ifdef
- use GALLIUM_PIPE_LOADER_WINSYS_LIBS
- check for CALLOC_STRUCT
thanks to Emil
From: Christoph Bumiller christoph.bumil...@speed.at
Signed-off-by: David Heidelberg da...@ixit.cz
---
src/gallium/winsys/sw/wrapper/wrapper_sw_winsys.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/gallium/winsys/sw/wrapper/wrapper_sw_winsys.c
Sorry, patch 4 is too big the mailing list.
The entire serie can be found here:
https://github.com/iXit/Mesa-3D/commits/for-upstream-5
On 17/11/2014 16:58, Axel Davy wrote :
Hi,
Here is last (4th) iteration of Gallium Nine patches.
We have integrated the new feedback we have got and hope
From: Christoph Bumiller christoph.bumil...@speed.at
At this moment we use only zero or positive values.
v2: Implement it for also for Solaris, MSVC assembly
and enable for other combinations.
v3: Replace MSVC assembly by assert + warning during compilation
Signed-off-by: David Heidelberg
From: Christoph Bumiller christoph.bumil...@speed.at
At this moment we use only zero or positive values.
v2: Implement it for also for Solaris, MSVC assembly
and enable for other combinations.
v3: Replace MSVC assembly by assert + warning during compilation
v4: remove inc and dec with
Hi,
Series looks good. You can add my r-b.
Do you want also to remove the DP2A reference like
did Jose patch ?
Axel Davy
On 20/11/2014 23:31, Eric Anholt wrote :
---
src/gallium/state_trackers/nine/nine_shader.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src
On 24/11/2014 06:54, Ilia Mirkin wrote :
On Sun, Nov 23, 2014 at 5:40 PM, David Heidelberg da...@ixit.cz wrote:
From: Axel Davy axel.d...@ens.fr
Fixes Error : CONST[20]: Undeclared source register when running
dx9_alpha_blending_material. Also artifacts on ilo.
Cc: 10.4 mesa-sta
Hi,
This patch removes the tripple buffering behaviour that the GLX
implementation
has with DRI3. I understand your concern for Medical softwares,
but perhaps this would be better handled with an user option.
Axel Davy
On 25/11/2014 04:00, Mario Kleiner wrote :
Restores proper immediate
Thanks,
patch 1 is not needed for patch 2 and 3.
As for patch 2, I'm 99% sure it's not needed for patch 3.
Axel
On 26/11/2014 21:14, Emil Velikov wrote :
Hi guys,
I've picked up patches 1-10 13 for master. Afaict there is still a
comment on two on the topic of queries, so I'll leave these
mesa-sta...@lists.freedesktop.org
Tested-by: Yaroslav Andrusyak pontost...@gmail.com
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/nine_defines.h | 40 ++-
src/gallium/state_trackers/nine/nine_ff.c | 49 +++--
src/gallium/state_trackers
map all combinations.
Cc: 10.4 mesa-sta...@lists.freedesktop.org
Tested-by: Yaroslav Andrusyak pontost...@gmail.com
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/nine_shader.c | 112 --
1 file changed, 52 insertions(+), 60 deletions(-)
diff
On Fri, 28 Nov 2014, Marek Olšák wrote:
Does this have any effect on what data drivers get or is it just
internal to Nine?
Marek
No, this patch shouldn't change anything for the driver. It is only
internal to Nine.
Axel___
mesa-dev mailing
mesa-sta...@lists.freedesktop.org
Tested-by: Yaroslav Andrusyak pontost...@gmail.com
Acked-by: Ilia Mirkin imir...@alum.mit.edu
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/nine_defines.h | 40 ++-
src/gallium/state_trackers/nine/nine_ff.c | 49
map all combinations.
Cc: 10.4 mesa-sta...@lists.freedesktop.org
Tested-by: Yaroslav Andrusyak pontost...@gmail.com
Reviewed-by: Marek Olšák marek.ol...@amd.com
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/nine_shader.c | 112 --
1 file
This is the behaviour that Wine tests
Tested-by: David Heidelberg da...@ixit.cz
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/query9.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/state_trackers/nine/query9.c
b/src/gallium/state_trackers
Tested-by: David Heidelberg da...@ixit.cz
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/query9.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/src/gallium/state_trackers/nine/query9.c
b/src/gallium/state_trackers/nine/query9.c
index
. This patch drops the dummy queries
support and matches wine behaviour.
Reviewed-by: David Heidelberg da...@ixit.cz
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/device9.c | 5 +-
src/gallium/state_trackers/nine/query9.c | 101 +++---
2 files
Some queries need the driver to advertise a cap to be supported.
For example r300 doesn't support them.
v2 (David): check also for PIPE_CAP_QUERY_PIPELINE_STATISTICS, fix wine
tests on r300g
Reviewed-by: David Heidelberg da...@ixit.cz
Signed-off-by: Axel Davy axel.d...@ens.fr
get_query_result flushes automatically, we don't need to flush.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/query9.c | 13 +
src/gallium/state_trackers/nine/query9.h | 1 -
2 files changed, 5 insertions(+), 9 deletions(-)
diff --git a/src/gallium
It is the same behaviour as wine has.
Reviewed-by: David Heidelberg da...@ixit.cz
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/query9.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/query9.c
b/src/gallium
Nine was allowing that behaviour, but was not filling the result.
Tested-by: David Heidelberg da...@ixit.cz
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/query9.c | 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/src/gallium
This is the behaviour that wine tests.
Reviewed-by: David Heidelberg da...@ixit.cz
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/query9.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/query9.c
b/src
XCB_PRESENT_OPTION_ASYNC, but this should not be the default.
You can add my
Reviewed-by: Axel Davy axel.d...@ens.fr
Axel Davy
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
;
}
Looks good to me.
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
Le 04/12/2014 11:44, Axel Davy a écrit :
On 02/12/2014 20:53, Mario Kleiner wrote :
targetSBC == 0 is a special case, which asks the function
to block until all pending OpenGL bufferswap requests have
completed.
Currently the function just falls through for targetSBC == 0,
returning bogus
*/
uint32_t send_msc_serial;
uint32_t recv_msc_serial;
I find using 'vblank' here is ambiguous. Wouldn't 'notify' fit better,
since you want to say it's what you received with the present_notify_event ?
Could you fix names and the comment ?
Axel Davy
.
+ */
+ remainder = 0;
+ }
if (priv-swap_interval == 0)
options |= XCB_PRESENT_OPTION_ASYNC;
It seems a bit heavy, but I don't find a better way to handle the case
you describe.
Reviewed-by: Axel Davy axel.d...@ens.fr
___
mesa
presentation, target msc y.
Without patch 2/5, the calculus of y will be done on the basis that last
presentation was done at x+1 instead of x.
Axel Davy
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman
to be built.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
On 19/01/2015 23:23, Ilia Mirkin wrote :
On Mon, Jan 19, 2015 at 5:17 PM, Axel Davy axel.d...@ens.fr 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
it to the drivers. It is advertised for drivers
for which it is sure the ability is supported.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
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-dev@lists.freedesktop.org
http
Signed-off-by: Axel Davy axel.d...@ens.fr
Cc: 10.4 mesa-sta...@lists.freedesktop.org
---
Since last version, we have tested precisely Win behaviour for depth buffer
formats
src/gallium/state_trackers/nine/basetexture9.c | 45 ++
1 file changed, 32 insertions(+), 13
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 axel.d...@ens.fr
---
This feature corresponds to the RESZ d3d9 hack.
d3d9 hacks are equivalent to GL
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
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 depth buffer
.
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-by: David
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 axel.d...@ens.fr
---
src/gallium/state_trackers/nine
We weren't releasing hal and ref, causing some issues (threads not released,
etc)
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/targets/d3dadapter9/drm.c | 18 --
1 file changed, 12 insertions(+), 6 deletions(-)
diff --git a/src/gallium/targets/d3dadapter9/drm.c
b
From: Patrick Rudolph s...@das-labor.org
Reviewed-by: Axel Davy axel.d...@ens.fr
Signed-off-by: Patrick Rudolph s...@das-labor.org
---
src/gallium/state_trackers/nine/vertexdeclaration9.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
cap to
determine the vendor and device id from the driver.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
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 axel.d...@ens.fr
---
src/gallium/targets/d3dadapter9/drm.c | 18 ++
1 file changed, 10 insertions(+), 8 deletions
can always retrieve
that info by hand.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
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 axel.d...@ens.fr
---
src/gallium
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
From: Patrick Rudolph s...@das-labor.org
This-data was allocated by nine, but not freed.
Reviewed-by: Axel Davy axel.d...@ens.fr
Signed-off-by: Patrick Rudolph s...@das-labor.org
---
src/gallium/state_trackers/nine/volume9.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium
add support for DF24 yet, because it implies support
for FETCH4, which we don't support for now.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/basetexture9.c | 12 ++--
src/gallium/state_trackers/nine/nine_pipe.h| 6 --
2 files changed, 14 insertions
From: Patrick Rudolph s...@das-labor.org
Likely a rebase error
Reviewed-by: Axel Davy axel.d...@ens.fr
Signed-off-by: Patrick Rudolph s...@das-labor.org
---
src/gallium/state_trackers/nine/resource9.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/src/gallium/state_trackers/nine
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 axel.d...@ens.fr
---
src/gallium/state_trackers
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
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 axel.d...@ens.fr
---
include/D3D9/d3d9types.h | 1 +
src/gallium/state_trackers/nine/adapter9.c | 4
src/gallium
From: Patrick Rudolph s...@das-labor.org
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 axel.d...@ens.fr
Signed-off-by: Patrick Rudolph s...@das-labor.org
---
src/gallium/state_trackers
From: Patrick Rudolph s...@das-labor.org
Add returncode E_FAIL.
Return E_FAIL for any vertexdeclaration element with type unused.
Reviewed-by: Axel Davy axel.d...@ens.fr
Signed-off-by: Patrick Rudolph s...@das-labor.org
---
include/D3D9/d3d9types.h | 1 +
src/gallium
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
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 axel.d...@ens.fr
---
src/gallium/state_trackers/nine/nine_pipe.c | 123 ++--
1 file
Count explicitly the slots for float, int and bool constants,
and deduce the constbuf size in nine_shader.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/nine_shader.c | 17 ++---
src/gallium/state_trackers/nine/nine_shader.h | 22
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
From: Patrick Rudolph s...@das-labor.org
Fixes wine test test_vertex_declaration_alignment()
Reviewed-by: Axel Davy axel.d...@ens.fr
Signed-off-by: Patrick Rudolph s...@das-labor.org
---
src/gallium/state_trackers/nine/vertexdeclaration9.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion
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 axel.d...@ens.fr
---
src/gallium/state_trackers/nine/device9.c| 17 +++--
src/gallium/state_trackers/nine/nine_pipe.c | 2 +-
src/gallium
From: Patrick Rudolph s...@das-labor.org
Reviewed-by: Axel Davy axel.d...@ens.fr
Signed-off-by: Patrick Rudolph s...@das-labor.org
---
src/gallium/state_trackers/nine/surface9.c | 24
1 file changed, 24 insertions(+)
diff --git a/src/gallium/state_trackers/nine/surface9
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 axel.d...@ens.fr
---
src/gallium/state_trackers/nine/nine_shader.c | 38 +--
1 file changed, 30 insertions
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 axel.d...@ens.fr
---
src/gallium/state_trackers/nine/adapter9.c
reduce the amount of constants needed, and that it is
restricted to 256 constant slots.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/nine_shader.c | 38 ---
1 file changed, 11 insertions(+), 27 deletions(-)
diff --git a/src/gallium
will be 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 axel.d...@ens.fr
---
This patch requires gallium: Add MULTISAMPLE_Z_RESOLVE cap
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
These values are taken from wine.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
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
From: Tiziano Bacocco tizb...@gmail.com
Reviewed-by: Axel Davy axel.d...@ens.fr
Reviewed-by: David Heidelberg da...@ixit.cz
Signed-off-by: Tiziano Bacocco tizb...@gmail.com
---
include/D3D9/d3d9types.h| 1 +
src/gallium/state_trackers/nine/nine_pipe.h | 1 +
2 files changed
This seems to be the behaviour on Win. Previous behaviour led
to different issues depending on the driver.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/nine_state.c | 46 +---
1 file changed, 28 insertions(+), 18 deletions(-)
diff --git
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 axel.d...@ens.fr
---
src/gallium
-by: Axel Davy axel.d...@ens.fr
---
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
D3DSP_NOSWIZZLE already contains the shift.
Detected with Clang.
Reviewed-by: David Heidelberg da...@ixit.cz
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/nine_shader.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers
From: Tiziano Bacocco tizb...@gmail.com
Reviewed-by: Axel Davy axel.d...@ens.fr
Signed-off-by: Tiziano Bacocco tizb...@gmail.com
---
include/D3D9/d3d9types.h| 1 +
src/gallium/state_trackers/nine/nine_pipe.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/include/D3D9
From: Tiziano Bacocco tizb...@gmail.com
Signed-off-by: Tiziano Bacocco tizb...@gmail.com
---
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
integer and boolean constants in the shader.
Eventually when this is done, we can reintroduce support for older cards.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/gallium/state_trackers/nine/adapter9.c | 106 +
src/gallium/state_trackers/nine/device9.c | 9 +--
2
From: Patrick Rudolph s...@das-labor.org
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 axel.d...@ens.fr
Signed-off-by: Patrick Rudolph s...@das-labor.org
---
src/gallium
From: Tiziano Bacocco tizb...@gmail.com
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 axel.d...@ens.fr
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 axel.d...@ens.fr wrote:
Signed-off-by: Axel
for 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 investigate
101 - 200 of 1075 matches
Mail list logo