On 12/11/2024 11:38, Thomas “illwieckz” Debesse wrote:
Le 18/10/2024 à 10:47, Michel Dänzer a écrit :
On 2024-10-18 01:36, Timur Kristóf wrote:
Or that you bisect between
very old very new versions?
I suspect the latter.
Yes, it happens that almost every time I have to bisect Mesa, both
b
We can deploy more widely Nine tests in CI, as Pavel Ondračka did for r300, so
we'll be sure basivs are still in good shape.
David
Original Message
From: Axel Davy
Sent: September 29, 2024 4:25:13 PM UTC
To: Martin Roukala , mesa-dev@lists.freedesktop.org
Subject: Re: Gallium
Ok, I'll do.
About MAX_PREDS, maybe it's time to bring it out again :)
On 02/06/2015 06:16 PM, Marek Olšák wrote:
You can update MAX_PREDS in this patch.
I proposed removing predicates a long time ago, but some people disagreed.
Marek
On Fri, Feb 6, 2015 at 5:19 PM, David Heidelb
shader stages. It has never been supported and never will be. With
that done, this patch is:
Reviewed-by: Marek Olšák
Marek
On Thu, Feb 5, 2015 at 11:56 PM, David Heidelberg wrote:
Signed-off-by: David Heidelberg
---
src/gallium/drivers/r300/r300_reg.h| 22 ++
src
On 02/06/2015 12:57 AM, Matt Turner wrote:
On Thu, Feb 5, 2015 at 2:56 PM, David Heidelberg wrote:
Signed-off-by: David Heidelberg
---
This is not a small clean up. This is removing a bunch of potentially
informative comments.
I checked sources & docs:
* all unsure comments has
wrote:
On Thu, Feb 5, 2015 at 2:56 PM, David Heidelberg wrote:
Rather not assert on known factors, just print warning and use ZERO.
... because this allows me to and I cannot fix the
actual bug because of ...?
___
mesa-dev mailing list
mes
Rather not assert on known factors, just print warning and use ZERO.
Signed-off-by: David Heidelberg
---
src/gallium/drivers/r300/r300_state_inlines.h | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/r300/r300_state_inlines.h
b/src/gallium
Signed-off-by: David Heidelberg
---
src/gallium/drivers/r300/r300_reg.h| 22 ++
src/gallium/drivers/r300/r300_screen.c | 4 +---
src/gallium/drivers/r300/r300_state.c | 1 -
3 files changed, 3 insertions(+), 24 deletions(-)
diff --git a/src/gallium/drivers/r300
(get_src_arg_mask(inst,j,NO_MASK) & mask)))
return READ;
}
@@ -944,7 +944,7 @@ update_interval(GLint intBegin[], GLint intEnd[],
struct loop_info *loopStack, GLuint loopStackDepth,
GLuint index, GLuint ic)
{
- int
I'd prefer not have to revert this patch on my personal r400 based RS690
(laptop AMD X1250). You're correct, that most of applications won't run.
Still, there are some earlier DX9 apps, which runs and LFS which shows
really visible performance boost from unplayable 10 fps to 20-25fps fps.
As
Signed-off-by: David Heidelberg
---
src/gallium/state_trackers/nine/device9.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/device9.c
b/src/gallium/state_trackers/nine/device9.c
index b0e42eb..1d97688 100644
--- a/src/gallium/state_trackers
Same as ARL, just has extra rounding.
Useful for st/nine.
Tested-by: Pavel Ondračka
Reviewed-by: Marek Olšák
Signed-off-by: David Heidelberg
---
src/gallium/drivers/r300/compiler/r3xx_vertprog.c | 7 ---
src/gallium/drivers/r300/compiler/radeon_opcodes.c | 7 +++
src/gallium/drivers
From: Axel Davy
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
Cc: "10.4"
Reviewed-by: David Heidelberg
Signed-of
From: Axel Davy
This is the behaviour that wine tests.
Cc: "10.4"
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/query9.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/query
From: Axel Davy
This is the behaviour that Wine tests
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
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/stat
From: Axel Davy
Issuing D3DISSUE_END should:
. reset previous queries if possible
. end the query
Previous behaviour wasn't calling end_query for
queries not needing D3DISSUE_BEGIN, no resetting
previous queries.
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel
From: Axel Davy
It is the same behaviour as wine has.
Cc: "10.4"
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/query9.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/query
From: Axel Davy
Nine was allowing that behaviour, but was not filling the result.
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/query9.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/s
From: Axel Davy
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/query9.c | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git a/src/gallium/state_trackers/nine/query9.c
b/src/gallium/state_tra
From: Axel Davy
Some queries need the driver to advertise a cap to be supported.
For example r300 doesn't support them.
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c | 2 +-
src/gallium/state_trackers/nine
different behaviour. This patch drops the dummy queries
support and matches wine behaviour.
Cc: "10.4"
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c | 5 +-
src/gallium/state_trackers/nine/query9.c | 101 +++--
From: Axel Davy
Cc: "10.4"
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/query9.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/nine/query9.c
b/src/gallium/state_trackers/nine/quer
From: Axel Davy
It is an sint_4, but it was stored in a uint_8...
The code using it was acting as if it was signed.
Problem found thanks to Coverity
Cc: "10.4"
Tested-by: David Heidelberg
Reviewed-by: Ilia Mirkin
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nin
From: Axel Davy
Error detected by Coverity (COPY_PASTE_ERROR)
Cc: "10.4"
Reviewed-by: Ilia Mirkin
Signed-off-by: Axel Davy
Signed-off-by: David Heidelberg
---
src/gallium/state_trackers/nine/nine_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/s
2efabd9f5a711a7f6cd1846630244b7814bf25b3 removed them as unused.
This caused random memory overwrites (reported by Coverity).
Cc: "10.4"
Reviewed-by: Ilia Mirkin
Reviewed-by: Marek Olšák
Reviewed-by: Axel Davy
Signed-off-by: David Heidelberg
---
src/gallium/state_trackers/nine/ni
Signed-off-by: David Heidelberg
---
src/gallium/state_trackers/nine/device9.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/device9.c
b/src/gallium/state_trackers/nine/device9.c
index 66d5e66..d48f47d 100644
--- a/src/gallium/state_tra
t
the design required to support the feature and wasn't used.
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/basetexture9.c | 4 +-
src/gallium/state_trackers/nine/cubetexture9.c | 2 +-
src/gallium/state_trackers/nine/
From: Axel Davy
D3DPOOL_SCRATCH is disallowed according to spec.
D3DPOOL_SYSTEMMEM should be allowed but we don't handle it right for now.
v2: Fixes segfault in SetTexture when unsetting the texture
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
-
From: Stanislaw Halik
Cc: "10.4"
Reviewed-by: Ilia Mirkin
Reviewed-by: David Heidelberg
Reviewed-by: Axel Davy
Signed-off-by: Stanislaw Halik
---
src/gallium/state_trackers/nine/basetexture9.c | 17
src/gallium/state_trackers/nine/cubetexture9.c | 17
s
From: Axel Davy
Fixes "Error : CONST[20]: Undeclared source register" when running
dx9_alpha_blending_material. Also artifacts on ilo.
v2: also remove unused MISC_CONST
Cc: "10.4"
Tested-by: David Heidelberg
Reviewed-by: Ilia Mirkin
Signed-off-by: Axel Davy
---
src/ga
From: Axel Davy
Instead of having parts of the structures initialised by the parents,
have them initialised by the children.
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/basetexture9.c | 19 +++
s
From: Axel Davy
Pass ex specific parameters as arguments to device9 ctor instead
of passing them by filling the structure.
Cc: "10.4"
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/adapter9.c | 2 +-
src/gallium/state_trackers/nine
From: Axel Davy
From this moment we should handle errors same way as Wine does.
Original patch from John Ettedgui
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/query9.c | 66 +---
1 file c
From: Axel Davy
Cc: "10.4"
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c | 2 +-
src/gallium/state_trackers/nine/query9.c | 52 ---
src/gallium/state_trackers/nine/query9.h | 2 +-
3 files changed, 42 insertions(+), 14 deletions(-)
diff
From: Axel Davy
It is the same behaviour as wine has.
Cc: "10.4"
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/query9.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/query9.c
b/s
2efabd9f5a711a7f6cd1846630244b7814bf25b3 removed them as unused.
This caused random memory overwrites (reported by Coverity).
Cc: "10.4"
Reviewed-by: Marek Olšák
Reviewed-by: Axel Davy
Signed-off-by: David Heidelberg
---
src/gallium/state_trackers/nine/nine_state.c | 2 --
1 file
From: Axel Davy
Error detected by Coverity (COPY_PAST_ERROR)
Cc: "10.4"
Signed-off-by: Axel Davy
Signed-off-by: David Heidelberg
---
src/gallium/state_trackers/nine/nine_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/ni
From: Axel Davy
It is an sint_4, but it was stored in a uint_8...
The code using it was acting as if it was signed.
Problem found thanks to Coverity
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 6 --
1 fi
, preventing Half-Life 2 Lost Coast from working
properly, is to remove the data field from Resource9, put it in Surface9,
which was the only user, and add flag to Surface9 to know if it has created
the data or not. Surface9 dtor frees the data only if it created it.
Cc: "10.4"
Tested-by: David
From: Stanislaw Halik
Cc: "10.4"
Reviewed-by: David Heidelberg
Reviewed-by: Axel Davy
Signed-off-by: Stanislaw Halik
---
src/gallium/state_trackers/nine/basetexture9.c | 17
src/gallium/state_trackers/nine/cubetexture9.c | 17
src/gallium/state_tra
Reviewed-by: David Heidelberg
On 11/22/2014 05:23 AM, Emil Velikov wrote:
The version provided by it should be the same as the one
provided/handled by the module. Add the missing tiny version.
Cc:
Signed-off-by: Emil Velikov
---
configure.ac| 2 ++
src
From: Axel Davy
D3DPOOL_SCRATCH is disallowed according to spec.
D3DPOOL_SYSTEMMEM should be allowed but we don't handle it right for now.
v2: Fixes segfault in SetTexture when unsetting the texture
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
-
PIPE_CAP_VIDEO_MEMORY returns the amount of video memory in megabytes,
so need to converted it to bytes.
Fixed Warframe memory detection.
v2: also prepare for cards with more than 4GB memory
Cc: "10.4"
Tested-by: Yaroslav Andrusyak
Reviewed-by: Axel Davy
Signed-off-by: David
From: Axel Davy
Fixes "Error : CONST[20]: Undeclared source register" when running
dx9_alpha_blending_material. Also artifacts on ilo.
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_ff.c | 4 ++--
1 file changed
Sorry, for now please without this patch.
This one needs " nine: Refactor Resource9 BaseTexture9 Surface9 and
Texture9 initialization", which isn't part of batch.
Thanks, David.
On 11/22/2014 04:14 AM, David Heidelberg wrote:
From: Stanislaw Halik
Cc: "10.4"
Revi
From: John Ettedgui
This patch fixes black screen with games based on the Unreal Engine 3.
It was tested that it fixed the issue in Tera Online, Borderlands 2 and
Homefront.
Cc: "10.4"
Reviewed-by: Axel Davy
Reviewed-by: David Heidelberg
Signed-off-by: John Ettedgui
---
s
From: Axel Davy
D3DPOOL_SCRATCH is disallowed according to spec.
D3DPOOL_SYSTEMMEM should be allowed but we don't handle it right for now.
v2: Fixes segfault in SetTexture when unsetting the texture
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
-
From: Stanislaw Halik
Cc: "10.4"
Reviewed-by: David Heidelberg
Reviewed-by: Axel Davy
Signed-off-by: Stanislaw Halik
---
src/gallium/state_trackers/nine/basetexture9.c | 17
src/gallium/state_trackers/nine/cubetexture9.c | 17
src/gallium/state_tra
From: Axel Davy
Fixes "Error : CONST[20]: Undeclared source register" when running
dx9_alpha_blending_material. Also artifacts on ilo.
Cc: "10.4"
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_ff.c | 4 ++--
1 file changed
Reviewed-by: David Heidelberg
for the series.
David
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> The patch attached should do it.
just small nitpick, please drop "/* for radeons */" comment from DP2ADD,
thanks
Reviewed-by: David Heidelberg
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mai
Apply for both patch 1/2 and 2/2.
Host: chroot on AMD A3870K
Target: AMD TK-55 (compiled with -mno-sse41)
I tested chroot cross-compilation and it passed correctly.
Tested-by: David Heidelberg
On 11/15/2014 06:04 PM, Emil Velikov wrote:
This reverts commit
On 11/14/2014 04:31 PM, Henri Verbeet wrote:
On 14 November 2014 16:21, Ilia Mirkin wrote:
To the best of my knowledge, wine has no intent on merging anything
related to nine/st.
That's a bit broader than I'd put it, but yes, in it's current form
this is not something we'd merge.
At this mome
.
Maintaining it out-of tree make us bisecting more difficult, so it would
be really blessing to have it in upstream repository.
David
On 11/14/2014 04:21 PM, Ilia Mirkin wrote:
On Fri, Nov 14, 2014 at 9:14 AM, Emil Velikov wrote:
On 02/11/14 18:27, David Heidelberg wrote:
Hello everyone!
First
This patch remove workaround related to LLVM 2.7 bug.
Original bug has been closed as fixed in 2011.
At this moment gallium requires LLVM 3.3 (2013).
Original LLVM bug: http://llvm.org/bugs/show_bug.cgi?id=6960
Signed-off-by: David Heidelberg
---
src/gallium/auxiliary/draw/draw_context.c
buffers to the server only when rendering has finished.
Since waiting the rendering has finished in the main thread has a
performance impact, this patch uses an additional thread to offload the
wait and the sending of the buffers to the server.
Signed-off-by: Axel Davy
Signed-off-by: David
Implements vblank_mode and throttling, which allows us change default ratio
between framerate and input lag.
Signed-off-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/adapter9.h | 1 +
src/gallium/state_trackers/nine/swapchain9.c| 5
src
This patch is too big for ML, please see it in
https://github.com/iXit/Mesa-3D/commits/for-upstream-3 .
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
v3: thanks to Brian, improved coding style, also glennk helped spot few
things (unsigned -> int, two constify)
Signed-off-by: David Heidelberg
---
src/gallium/auxiliary/util/u_box.h | 230
src/gallium/auxiliary/util/u_rect.h | 28 +
2 files chan
v2: rename and extend support with code for C11 and MSVC (thanks to Brian)
Signed-off-by: David Heidelberg
---
src/gallium/auxiliary/util/u_dump.h | 6 ++
src/gallium/auxiliary/util/u_dump_defines.c | 86
+
2 files changed, 92 insertions(+)
diff --git a
Implement pipe_loader_sw_probe_wrapped which allows to use the wrapped
software renderer backend when using the pipe loader.
Signed-off-by: David Heidelberg
---
src/gallium/auxiliary/pipe-loader/pipe_loader.h| 11 +++
src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 22
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
---
src/gallium/auxiliary/util/u_atomic.h | 78
+++
1 file changed, 78 insertions
Signed-off-by: David Heidelberg
---
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
b/src/gallium/winsys/sw/wrapper/wrapper_sw_winsys.c
index e552ac2..a6bf498 100644
--- a
v2: moved in in same order as in p_shader_tokens (thanks Brian)
Signed-off-by: David Heidelberg
---
src/gallium/auxiliary/tgsi/tgsi_opcode_tmp.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_opcode_tmp.h
b/src/gallium/auxiliary/tgsi/tgsi_opcode_tmp.h
Hello everyone!
First I'd like thank you for great feedback.
Sending third Gallium Nine merge request. We reduced number of commits
to necessary minimum. I hope all proposed changes are incorporated in v3.
Thank you
Axel Davy (2):
nine: Add drirc options (v2)
nine: Implement threadpool
C
mit 013ff2fae13da41c2f5619c4698b0a7b5aa6a06d
Original LLVM bug: http://llvm.org/bugs/show_bug.cgi?id=6960
Signed-off-by: David Heidelberg
---
src/gallium/auxiliary/draw/draw_context.c | 15 +--
1 file changed, 1 insertion(+), 14 deletions(-)
diff --git a/src/gallium/auxiliary/d
66 matches
Mail list logo
