[Mesa-dev] [PATCH 11/39] st/nine: Increase minor d3dadapter9drm ABI

Version 0.1 allows to assume that the second element of the IDirect3D* structures will be a pointer to the internal nine vtable. This is useful if the gallium nine user wants to wrap some interfaces. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- include/d3dadapter/drm.h

[Mesa-dev] [PATCH 21/39] st/nine: Implement BumpEnvMap for ff

Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 78 +-- 1 file changed, 54 insertions(+), 24 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index 5

[Mesa-dev] [PATCH 27/39] st/nine: Fix buffer locking flags handling

Our behaviour was not entirely similar to what the docs and our tests describe. Drop d3dlock_buffer_to_pipe_transfer_usage. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/buffer9.c | 24 ++-- src/gallium/state_trackers/nine/nine_pipe.

[Mesa-dev] [PATCH 31/39] st/nine: Fix output shift calculation

We were getting it wrong for negative values. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_shader.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_shader.c b/src/gallium/state_tracker

[Mesa-dev] [PATCH 15/39] gallium/util: Implement util_format_translate_3D

This is the equivalent of util_format_translate, but for volumes. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/auxiliary/util/u_format.c | 34 ++ src/gallium/auxiliary/util/u_format.h | 13 + 2 files changed, 47 insertions(+) diff

[Mesa-dev] [PATCH 14/39] st/nine: Fix Pointsize in programmable 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 | 52 +++-- src/gallium/state_trackers/nine/nine_state.c| 7 ++-- src/gallium/state_trackers/nine/nine_s

[Mesa-dev] [PATCH 33/39] st/nine: Add asserts for output/input packing

Nine doesn't support vs output/ps input packing. We haven't found any application requiring that, and implementing it properly is complex. Add asserts for now. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_shader.c | 4 1 file changed, 4 inse

[Mesa-dev] [PATCH 05/39] st/nine: Rework UpdateTexture Checks

Our code did match the user documentation of the function quite well (except for format check). However the DDI documentation and wine tests show that documentation was not correct. Thus adapt our code to fit the best possible to the -real- spec. Signed-off-by: Axel Davy <axel.d...@ens

[Mesa-dev] [PATCH 39/39] st/nine: Minor change to support musl libc

From: Wang He <xw897002...@gmail.com> A few changes to support musl libc as well. In particular fpu_control.h is glibc specific. fenv.h doesn't enable to do exactly what we want either, so instead use assembly directly. Signed-off-by: Wang He <xw897002...@gmail.com> Reviewed-b

[Mesa-dev] [PATCH 26/39] st/nine: Improve logging

From: Patrick Rudolph <s...@das-labor.org> Add missing DBG calls in dtors. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/buffer9.c| 2 ++ src/gallium/state_trackers/nine/query9

[Mesa-dev] [PATCH 00/39] Some nine patches for Mesa 12.0

it default. Yours, Axel Davy Axel Davy (32): d3dadapter9: Add ddebug support d3dadapter9: Add d3dadapter9.pc llvmpipe: Fix overflow for 32 bits available memory computation st/nine: Use bufs instead of Flags for Clear st/nine: Rework UpdateTexture Checks st/nine: Forbid POSITION0

[Mesa-dev] [PATCH 34/39] st/nine: Add debug string for ATOC

We were missing a debug string for this format. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_pipe.h | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/nine/nine_pipe.h b/src/gallium/state_trackers/nine/nine_pipe.h index 5

[Mesa-dev] [PATCH 38/39] st/nine: Enable D3DPMISCCAPS_PERSTAGECONSTANT

gt; Reviewed-by: Axel Davy <axel.d...@ens.fr> --- 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 b06528d..41140d4 100644 --- a/src/gallium/s

[Mesa-dev] [PATCH 37/39] st/nine: Turn on thread_submit by default when on different device

The last remaining issues with thread_submit have been resolved, thus turn it when on a different device (the case where is is beneficial). Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/targets/d3dadapter9/drm.c | 22 -- 1 file changed, 12 insertions(

[Mesa-dev] [PATCH 01/39] d3dadapter9: Add ddebug support

Add support for ddebug, noop, rbug and trace Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/targets/d3dadapter9/Makefile.am | 11 +++ 1 file changed, 11 insertions(+) diff --git a/src/gallium/targets/d3dadapter9/Makefile.am b/src/gallium/targets/d3dadapter9/Makef

[Mesa-dev] [PATCH 07/39] st/nine: Fix some shader failures not triggering error

Some failures during shader translation would not raise errors before this patch. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_shader.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_sh

[Mesa-dev] [PATCH 10/39] st/nine: Fix leak after ctor failures

Previously ctor failures would not unreference the device. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/iunknown.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/state_trackers/nine/iunknown.c b/src/gallium/state_trackers/nine/iunk

[Mesa-dev] [PATCH 04/39] st/nine: Use bufs instead of Flags for Clear

bufs doesn't contain depthstencil if there is z buffer mismatch. This is the behaviour we want. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_tracker

[Mesa-dev] [PATCH 09/39] st/nine: Add ColorFill test for compressed textures

ColorFill should contain alignment checks for compressed textures. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 8 1 file changed, 8 insertions(+) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_tracker

[Mesa-dev] [PATCH 19/39] st/nine: Remove one useless function output

Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/volume9.c | 6 +++--- src/gallium/state_trackers/nine/volume9.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/nine/volume9.c b/src/gallium/state_tracker

[Mesa-dev] [PATCH 12/39] st/nine: Fix header of GetIndices

There is a mistake in the online documentation, the function only has 2 arguments. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- include/D3D9/d3d9.h | 10 +- src/gallium/state_trackers/nine/device9.c | 3 +-- 2 files changed, 6 insertions(+), 7 deletions(-)

[Mesa-dev] [PATCH 20/39] st/nine: Format conversion for volumes in UpdateTexture

We were doing the conversion for surfaces, but not yet volumes. Now that volumes can do conversion, use it. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 8 1 file changed, 8 deletions(-) diff --git a/src/gallium/state_trackers/nine/d

[Mesa-dev] [PATCH 28/39] st/nine: Fix buffer bind flags

Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/buffer9.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/buffer9.c b/src/gallium/state_trackers/nine/buffer9.c index ed0c4f1..f6bad6a 100644 ---

[Mesa-dev] [PATCH 13/39] st/nine: Fix ff pointscale computation

Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 27 ++- 1 file changed, 18 insertions(+), 9 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index 7b26c95..5

[Mesa-dev] [PATCH 32/39] st/nine: Use correct PIPE_HANDLE_USAGE flag for frontbuffer copy

When taking screenshots we do a copy from the frontbuffer to an allocated buffer (which we then copy to a ram buffer). Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/swapchain9.c | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff

[Mesa-dev] [PATCH 08/39] st/nine: PositionT and Tessfactor are forbidden as PS input

According to wine tests, they are forbidden as PS input, which makes sense. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_shader.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_shade

[Mesa-dev] [PATCH 18/39] st/nine: Add support for X8L8V8U8

. Since some recent cards do not support it, do not support it either. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_pipe.c | 4 ++-- src/gallium/state_trackers/nine/nine_pipe.h | 12 +++- 2 files changed, 13 insertions(+), 3 deletions(-) diff

[Mesa-dev] [PATCH 03/39] llvmpipe: Fix overflow for 32 bits available memory computation

And cap to 2 GB on 32 bits. Fixes https://bugs.freedesktop.org/show_bug.cgi?id=94561 Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/auxiliary/os/os_misc.c | 2 +- src/gallium/drivers/llvmpipe/lp_screen.c | 5 + 2 files changed, 6 insertions(+), 1 deletion(-) diff

[Mesa-dev] [PATCH 35/39] st/nine: ATOC has effect only with ALPHATESTENABLE

ATOC extension does something only when alpha test is enabled. Use a second bit to encode the difference with ATIATOC. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 9 - src/gallium/state_trackers/nine/nine_pipe.c | 2 +- src/g

[Mesa-dev] [PATCH 06/39] st/nine: Forbid POSITION0 for PS3.0

POSITION0 input is forbidden for PS3.0 apparently. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_shader.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/state_trackers/nine/nine_shader.c b/src/gallium/state_trackers/nine/nine_sh

[Mesa-dev] [PATCH 23/39] st/nine: Use new function to query for resolution mismatch

From: Patrick Rudolph <s...@das-labor.org> Any third party app might change the current screen resolution. Poll for resolution mismatch to force a device reset. Required for non ex devices only. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <a

[Mesa-dev] [PATCH 02/39] d3dadapter9: Add d3dadapter9.pc

Export d3dadapter9.pc It is essentially the same than d3d.pc, except we want to drop d3d.pc in the future (we prefer the name d3dadapter9). Signed-off-by: Axel Davy <axel.d...@ens.fr> --- configure.ac| 1 + src/gallium/targets/d3dadapter9/Makefile.am | 2

[Mesa-dev] [PATCH 02/39 v2] d3dadapter9: Add d3dadapter9.pc

Export d3dadapter9.pc It is essentially the same than d3d.pc, except we want to drop d3d.pc in the future (we prefer the name d3dadapter9). v2: Resend because one file was missing. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- configure.ac | 1

[Mesa-dev] [PATCH 22/39] st/nine: Implement IPresent version 1.2

;s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- include/d3dadapter/present.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/d3dadapter/present.h b/include/d3dadapter/present.h index 162f703..187843f 100644 --- a/include/d3dadapter/present.h +++ b/include/d3

[Mesa-dev] [PATCH 36/39] st/nine: Fix usage of rasterizer multisample bit.

multisampling is same as not multisampled. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c| 2 +- src/gallium/state_trackers/nine/nine_pipe.c | 2 +- src/gallium/state_trackers/nine/nine_state.c | 30 +++- src/g

[Mesa-dev] [PATCH 30/39] st/nine: Fix CheckDeviceFormat advertising for surfaces

Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/adapter9.c | 12 1 file changed, 12 insertions(+) diff --git a/src/gallium/state_trackers/nine/adapter9.c b/src/gallium/state_trackers/nine/adapter9.c index 48e1e44..b06528d 100644 --- a/src/g

[Mesa-dev] [PATCH 29/39] st/nine: Improve buffer placement

Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/buffer9.c | 34 +++ 1 file changed, 30 insertions(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/nine/buffer9.c b/src/gallium/state_trackers/nine/buffer9.c index f

Re: [Mesa-dev] [PATCH 00/39] Some nine patches for Mesa 12.0

On 15/05/2016 12:45, Axel Davy wrote: . Steam overlay now working . Last remaining issue with thread_submit option solved (thread_submit allows DRI_PRIME support without any glitch or tearing), thus turn it default. Just a small clarification: The steam overlay needs patch "st/nine: Inc

Re: [Mesa-dev] [PATCH] st/nine: clean up WINAPI definition

Do we need the #ifndef WINAPI part ? Axel On 29/04/2016 20:53, Christian Schmidbauer wrote: As Emil pointed out, only gcc, clang and MSVC compatibility is required. Hence the check for GNUC can be skipped, as __i386__ and __x86_64__ are only defined for gcc/clang, not for MSVC. Remove the

Re: [Mesa-dev] [PATCH] st/nine: correctly fold guards around define WINAPI

I'm not competent to decide on the matter, but the patch seems ok. Acked-by: Axel Davy <axel.d...@ens.fr> On 15/04/2016 17:06, Emil Velikov wrote : From: Emil Velikov <emil.veli...@collabora.com> The __i386__ and __x86-64__ macros are gcc/clang specific, thus one does not need

Re: [Mesa-dev] [PATCH] gallium/radeon: fix Nine with its slightly shifted viewports

Reviewed-by: Axel Davy <axel.d...@ens.fr> On 16/04/2016 00:44, Marek Olšák wrote: From: Marek Olšák <marek.ol...@amd.com> just need to do the calculation in floating-point and then round things properly --- src/gallium/drivers/radeon/r600_viewport.c | 36 +-

Re: [Mesa-dev] [PATCH] gallium/radeon: handle vertex shaders that disable clipping & viewport

The patch seems unneeded, as a lot of nine apps with the disabled clipping behaviour are working fine. It seems the bug is with the scissor bound computation: for power of two RT, nine is removing a small offset to the viewport for radeon to workaround a raterizer precision bug. The scissor

Re: [Mesa-dev] [PATCH] st/nine: add missing copyright headers

d6dd698 100644 --- a/src/gallium/state_trackers/nine/nine_flags.h +++ b/src/gallium/state_trackers/nine/nine_flags.h @@ -1,4 +1,24 @@ - +/* + * Copyright 2016 Axel Davy <axel.d...@ens.fr> + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this sof

Re: [Mesa-dev] [PATCH 1/6] gallium/radeon: change the vendor string to Advanced Micro Devices, Inc.

Hi, This looks like it will perturb driver detection of already written games. For example from the dolphin sources, this change would make them detect catalyst as driver. As it is known workarounds are applied depending on the driver detected, or some features are disabled, I believe it

Re: [Mesa-dev] [PATCH 1/2] r600g: use ieee variants of multiplication instructions

sa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev Bad apps that depend on the behaviour could be listed in drirc with a workaround to force them use the gl extension associated with the feature. Yours, Axel Davy _

Re: [Mesa-dev] [PATCH v2 3/5] st/nine: set the MUL_ZERO_WINS flag when supported

With this patch, there is one case where we hit the "undefined case" of the spec (mismatch between different shaders): With streamout we use a dummy pixel created by utils instead of nine_shader. As we use software renderers for the feature doing the streamout, I guess that's ok for now.

Re: [Mesa-dev] Gallium: Removal of set_index_buffer (discussion)

om> wrote: Am 20.02.2017 um 20:56 schrieb Marek Olšák: On Mon, Feb 20, 2017 at 8:29 PM, Axel Davy <axel.d...@ens.fr> wrote: On 20/02/2017 20:11, Ilia Mirkin wrote: On Mon, Feb 20, 2017 at 2:01 PM, Marek Olšák <mar...@gmail.com> wrote: Hi, I'd like to remove pipe_context

Re: [Mesa-dev] [PATCH] st/nine: make use of common uploaders v4

This looks fine to me. Reviewed-by: Axel Davy <axel.d...@ens.fr> I think the patch requires your Signed-off-by though. Axel On 21/02/2017 05:31, Constantine Charlamov wrote: Make use of common uploaders that landed recently to Mesa v2: fixed formatting, broken due to thund

Re: [Mesa-dev] Gallium: Removal of set_index_buffer (discussion)

On 20/02/2017 20:11, Ilia Mirkin wrote: On Mon, Feb 20, 2017 at 2:01 PM, Marek Olšák wrote: Hi, I'd like to remove pipe_context::set_index_buffer. It's not useful to most drivers and the interface is inconvenient for Mesa/OpenGL, because it's a draw state that is set with a

Re: [Mesa-dev] [PATCH] st/nine: make use of common uploaders v2

On 20/02/2017 20:22, Constantine Charlamov wrote: Make use of common uploaders that landed recently to Mesa v2: fixed formatting, broken due to thunderbird configuration --- src/gallium/state_trackers/nine/device9.c| 48

Re: [Mesa-dev] [PATCH] st/nine: make use of common uploaders v3

On 20/02/2017 21:30, Constantine Charlamov wrote: Make use of common uploaders that landed recently to Mesa v2: fixed formatting, broken due to thunderbird configuration v3: per Axel comment: added a comment into NineDevice9_DrawPrimitiveUP --- src/gallium/state_trackers/nine/device9.c|

Re: [Mesa-dev] [PATCH] st/nine: Remove code for no USER_INDEX_BUFFERS as these are always on

Hi Mike, We really want not to use user index buffers when csmt is active (thus the !This->csmt_active). This should be a one line patch to just remove the part GET_PCAP(USER_INDEX_BUFFERS) Yours, Axel Davy On 25/02/2017 06:23, Mike Lothian wrote: This fi

Re: [Mesa-dev] [PATCH] nv50, nvc0: disable depth offsets when there is no depth buffer

Shouldn't this be fixed in wine, rather than in the driver ? Yours, Axel On 15/01/2017 21:57, Ilia Mirkin wrote: While I can find no support for this in the GL spec, this is apparently what D3D9 wants. Also appears to fix a very long-standing bug in Tomb Raider: Underworld and Deus Ex: Human

Re: [Mesa-dev] [PATCH 3/4] radeonsi: add support for an on-disk shader cache

On 02/03/2017 00:27, Timothy Arceri wrote: On 01/03/17 18:36, Axel Davy wrote: My understanding of the code is that the disk cache just depends on the sscreen->b.disk_shader_cache being initialized, and it looks like this is always initialized. Thus the disk cache is always enabled. Co

Re: [Mesa-dev] [PATCH 3/4] radeonsi: add support for an on-disk shader cache

My understanding of the code is that the disk cache just depends on the sscreen->b.disk_shader_cache being initialized, and it looks like this is always initialized. Thus the disk cache is always enabled. Could you add a RADEON_DEBUG env var to disable it, please ? Thanks, Axel Davy On

Re: [Mesa-dev] [PATCH] loader/dri3: Overhaul dri3_update_num_back

Le 02/09/2016 à 03:06, Michel Dänzer a écrit : On 02/09/16 12:37 AM, Alex Deucher wrote: On Thu, Sep 1, 2016 at 11:28 AM, Jason Ekstrand wrote: On Aug 31, 2016 11:39 PM, "Michel Dänzer" wrote: On 01/09/16 02:05 PM, Jason Ekstrand wrote: On Wed, Aug

Re: [Mesa-dev] [PATCH] loader/dri3: Overhaul dri3_update_num_back

Le 02/09/2016 à 16:41, Axel Davy a écrit : Le 02/09/2016 à 03:06, Michel Dänzer a écrit : On 02/09/16 12:37 AM, Alex Deucher wrote: On Thu, Sep 1, 2016 at 11:28 AM, Jason Ekstrand <ja...@jlekstrand.net> wrote: On Aug 31, 2016 11:39 PM, "Michel Dänzer" <mic...@daenzer.net&g

Re: [Mesa-dev] [PATCH 2/3] egl: return corresponding offset of EGLImage instead of 0.

interface version implementation check. (another nitpick: queryImage return true or false, not EGL_TRUE/EGL_FALSE. It's better to convert, instead of assuming they're the same) Yours, Axel Davy On 06/09/2016 11:06, Chuanbo Weng wrote: The offset should not always be 0. For example, if EGLImage

Re: [Mesa-dev] [PATCH 2/3] egl: return corresponding offset of EGLImage instead of 0.

That doesn't seem good to me. With that patch, that means that since no one is implementing __DRI_IMAGE_ATTRIB_OFFSET (yes I know in a later patch you implement it for i965), then what used to work will stop working (as the queryImage will return false). You need to introduce some

Re: [Mesa-dev] [PATCH v2 2/3] egl: return corresponding offset of EGLImage instead of 0.

lt;chuanbo.w...@intel.com> Cc: ML mesa-dev <mesa-dev@lists.freedesktop.org>; Axel Davy <axel.d...@ens.fr> Subject: Re: [PATCH v2 2/3] egl: return corresponding offset of EGLImage instead of 0. On 9 September 2016 at 08:58, Chuanbo Weng <chuanbo.w...@intel.com> wrote: The of

[Mesa-dev] [PATCH 02/75] gallium/util: Really allow aliasing for u_box_union_*

Gallium nine relies on aliasing to work with this function. Without this patch, dirty region tracking was incorrect, which could lead to incorrect textures or vertex buffers. Fixes several game bugs with nine. Fixes https://github.com/iXit/Mesa-3D/issues/234 Signed-off-by: Axel Davy <axe

[Mesa-dev] [PATCH 02/75] gallium/util: Really allow aliasing of dst for u_box_union_*

Gallium nine relies on aliasing to work with this function. Without this patch, dirty region tracking was incorrect, which could lead to incorrect textures or vertex buffers. Fixes several game bugs with nine. Fixes https://github.com/iXit/Mesa-3D/issues/234 Signed-off-by: Axel Davy <axe

Re: [Mesa-dev] [PATCH 04/75] st/nine: Fix some check flags

On 06/10/2016 11:34, Emil Velikov wrote: On 5 October 2016 at 21:08, Axel Davy <axel.d...@ens.fr> wrote: Uses the new defines introduced in previous commit. Please describe why the newly introduced flags are used over the present ones. Worth copying some of the in-source c

Re: [Mesa-dev] [PATCH 01/75] llvmpipe: Fix overflow for 32 bits available memory computation

On 06/10/2016 11:44, Emil Velikov wrote: Hi Axel, You seem to have forgotten/ignored virtually every suggestion for this patch from last time around. Did you send the wrong patch or ? Emil Well it was sitting on our repo, and I got persuaded I had addressed the issues. Looks like not.

Re: [Mesa-dev] [PATCH 20/75] st/nine: Initial ProcessVertices support

On 05/10/2016 22:08, Axel Davy wrote: HRESULT NINE_WINAPI NineDevice9_ProcessVertices( struct NineDevice9 *This, UINT SrcStartIndex, @@ -3174,33 +3188,69 @@ NineDevice9_ProcessVertices( struct NineDevice9

[Mesa-dev] [PATCH 3/3] softpipe: Cap to 2 GB on 32 bits

when 80% of the reported memory limit is used. This change helps some apps to start. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/softpipe/sp_screen.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/gallium/drivers/softpipe/sp_screen.c b/src/gallium/d

[Mesa-dev] [PATCH 1/3] gallium/os: Fix overflow on 32 bits

On systems with more than 4GB of ram, os_get_total_physical_memory was triggering an integer overflow for the linux and haiku path, when on 32 bits. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94561 Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/auxiliary/os/os_

[Mesa-dev] [PATCH 2/3] llvmpipe: Cap to 2 GB on 32 bits

when 80% of the reported memory limit is used. This change helps some apps to start. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/drivers/llvmpipe/lp_screen.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/gallium/drivers/llvmpipe/lp_screen.c b/src/gallium/d

Re: [Mesa-dev] [PATCH 02/75] gallium/util: Really allow aliasing for u_box_union_*

Sorry, ignore this, I sent a new one with the corrected title. On 06/10/2016 19:57, Axel Davy wrote: Gallium nine relies on aliasing to work with this function. Without this patch, dirty region tracking was incorrect, which could lead to incorrect textures or vertex buffers. Fixes several game

Re: [Mesa-dev] [PATCH 20/75] st/nine: Initial ProcessVertices support

On 06/10/2016 22:28, Ilia Mirkin wrote: On Thu, Oct 6, 2016 at 4:24 PM, Axel Davy <axel.d...@ens.fr> wrote: On 05/10/2016 22:08, Axel Davy wrote: HRESULT NINE_WINAPI NineDevice9_ProcessVertices( struct NineDevice9 *This, UINT SrcStartIndex, @@ -3

Re: [Mesa-dev] [PATCH] gallium/drivers: initialize pipe_resource::next to NULL

Hi, as of writing, there doesn't seem to be a concensus on the fix. Could one be found for Mesa 13 ? Gallium nine is apparently broken except on radeonsi which zeros out the next field... It'd need either the proposed patch of this thread merged, or to zero the next field everywhere. I guess

[Mesa-dev] [PATCH 04/75] st/nine: Fix some check flags

Uses the new defines introduced in previous commit. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/adapter9.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/src/gallium/state_trackers/nine/adapter9.c b/src/g

[Mesa-dev] [PATCH 18/75] st/nine: Rework vs int and bool constants buffer

This will help to support swvp constants. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/device9.c | 35 +++ src/gallium/state_trackers/nine/nine_state.c | 4 +-- src/gallium/state_trackers/nine/nine_state.h | 4 +-- src/g

[Mesa-dev] [PATCH 03/75] st/nine: Unify some check flags

The new defines will be reused in a later patch. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_defines.h | 8 src/gallium/state_trackers/nine/swapchain9.c | 8 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/src/g

[Mesa-dev] [PATCH 02/75] gallium/util: Really allow aliasing for u_box_union_*

Fixes several game bugs with nine. Fixes https://github.com/iXit/Mesa-3D/issues/234 Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> Cc: "12.0" <mesa-...@lists.freedestkop.org> --- src/gallium

[Mesa-dev] [PATCH 66/75] st/nine: Prepare update_framebuffer for MS quality levels

From: Patrick Rudolph <s...@das-labor.org> Compare resource's nr_samples instead of D3D multisample level. Required for multisample quality levels to work correct. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/s

[Mesa-dev] [PATCH 75/75] st/nine: More checks for GetRenderTargetData

Fixes a wine test crash Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/device9.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_t

[Mesa-dev] [PATCH 30/75] st/nine: Undefined specular should be full of zeros

Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_ff.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/st

[Mesa-dev] [PATCH 69/75] st/nine: Use NineDevice9_CreateDepthStencilSurface in swapchain9

From: Patrick Rudolph Replace custom code with NineDevice9_CreateDepthStencilSurface. All functionality is given now. --- src/gallium/state_trackers/nine/swapchain9.c | 30 1 file changed, 17 insertions(+), 13 deletions(-) diff --git

[Mesa-dev] [PATCH 56/75] st/nine: Add debugging output

From: Patrick Rudolph <s...@das-labor.org> Add DBG calls to NineTexture9_GetLevelDesc and NineTexture9_GetSurfaceLevel to ease debugging. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/texture9.

[Mesa-dev] [PATCH 42/75] st/nine: Do not pollute rTmp in ff ps

Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_ff.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_t

[Mesa-dev] [PATCH 41/75] st/nine: Allocate temporaries on demand for ps ff

Same change than for vs ff. This makes it easier to not introduce mistakes reusing temporaries whose result shouldn't be erased. Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/ni

[Mesa-dev] [PATCH 19/75] st/nine: Partial software vertex processing support

the start of the application, but never use more than the normal limit. When the apps do not need the software vertex processing features, they do not seem to turn it on. This means it is ok if that path is slow. Thus no care has been made to make the path optimized. Signed-off-by: Axel Davy <axel.d...@ens

[Mesa-dev] [PATCH 46/75] st/nine: Disable parts of lighting calculation if no normal provided

Behaviour found in Wine sources, and checked with some test apps. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 61 ++- 1 file changed, 35 insertions(+), 26 deletions(-) diff --git a/src/gallium/state_tracker

[Mesa-dev] [PATCH 40/75] st/nine: Fix texbem

Error found with wine tests. nine_shader was expecting another order than the one device9 was using. Signed-off-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, 2 inse

[Mesa-dev] [PATCH 10/75] st/nine: Add missing break

From: Patrick Rudolph <s...@das-labor.org> Add missing break instruction. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_pipe.h | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gall

[Mesa-dev] [PATCH 52/75] st/nine: Print interface id in NineUnknown_QueryInterface

From: Patrick Rudolph <s...@das-labor.org> To ease debugging print interface id. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/iunknown.c | 4 +++- 1 file changed, 3 insertions(+), 1 delet

[Mesa-dev] [PATCH 63/75] st/nine: Remove resource9 {Set/Get/Free}PrivateData functions

From: Patrick Rudolph <s...@das-labor.org> Remove {Set/Get/Free}PrivateData in resource9. Functionality has been implement in IUnknown interface. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 36/75] st/nine: Enable passthrough only if positiont is used

Wine tests for the passthrough feature are for positiont. Nothing seems to indicate passthrough happens when positiont it not used. However having passthrough with positiont makes sense (to be used with ProcessVertices outputs). Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/g

[Mesa-dev] [PATCH 29/75] st/nine: Implement normal transformation with vertex blending

The formula is different from the one of the spec, but otherwise nothing particular. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 26 +- 1 file changed, 21 insertions(+), 5 deletions(-) diff --git a/src/g

[Mesa-dev] [PATCH 71/75] st/nine: handle possible failure of D3DWindowBuffer_create

From: Patrick Rudolph <s...@das-labor.org> Check for errors and pass them to the callers. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/swapchain9.c | 8 1 file changed, 8 inserti

[Mesa-dev] [PATCH 43/75] st/nine: Initialize ps ff registers

Found with wine tests for the rTmp register. Not sure for the other ones. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/nine_ff.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_tr

[Mesa-dev] [PATCH 67/75] st/nine: Implement MSAA quality levels

level missmatch. Fixes several old games not having multisample support until now. Fix for issue #73. Signed-off-by: Patrick Rudolph <s...@das-labor.org> Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/adapter9.c | 20 -- src/gallium/s

[Mesa-dev] [PATCH 68/75] st/nine: Fix check and remove useless code in swapchain9

the resource_copy_region path, which should be fine without PIPE_BIND_SAMPLER_VIEW. Thus fix the check for 1), and remove the code. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/adapter9.c | 6 +- src/gallium/state_trackers/nine/swapchain9.c | 16 2

[Mesa-dev] [PATCH 37/75] st/nine: Implement SPHEREMAP

Behaviour checked with a test app. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/adapter9.c | 2 +- src/gallium/state_trackers/nine/nine_ff.c | 32 +++--- 2 files changed, 30 insertions(+), 4 deletions(-) diff --git a/src/g

[Mesa-dev] [PATCH 35/75] st/nine: Fix wrong mask in ff vs

Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_ff.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/ni

[Mesa-dev] [PATCH 59/75] st/nine: Return device in NineSurface9_GetContainer

;s...@das-labor.org> Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/surface9.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/surface9.c b/src/gallium/state_trackers/nine/surface9.c index e6a48

[Mesa-dev] [PATCH 28/75] st/nine: Increase MaxVertexBlendMatrixIndex

Modern cards do advertise 8. Signed-off-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/adapter9.c | 2 +- src/gallium/state_trackers/nine/nine_ff.c | 8 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/gallium/state_trackers/nine/adapter

[Mesa-dev] [PATCH 22/75] st/nine: No need for the local flag for temporaries in ff

Signed-off-by: Axel Davy <axel.d...@ens.fr> Signed-off-by: Patrick Rudolph <s...@das-labor.org> --- src/gallium/state_trackers/nine/nine_ff.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_t

[Mesa-dev] [PATCH 64/75] st/nine: Fix compiler warning

gt; Reviewed-by: Axel Davy <axel.d...@ens.fr> --- src/gallium/state_trackers/nine/iunknown.c | 14 +- src/gallium/state_trackers/nine/nine_pdata.h | 4 ++-- 2 files changed, 11 insertions(+), 7 deletions(-) diff --git a/src/gallium/state_trackers/nine/iunknown.c b/src/gallium/s

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