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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
---
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
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
.
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
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
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
, 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 +++
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
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
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
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
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
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
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
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
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
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
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
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
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
,
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
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
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.
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
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
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
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
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
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:
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
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
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
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
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
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
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
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
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
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
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
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/
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 +
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
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.
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.
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
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
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
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
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
>
> 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
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
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
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.
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
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
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(
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
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
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
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
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
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
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 --
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
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
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
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
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
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
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
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
-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
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
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
401 - 500 of 1075 matches
Mail list logo