On 29/05/2014 13:18, Marek Olšák wrote :
Hi Axel,
flush_resource should be called before ctx->st->flush.
Marek
Thanks, I'll update for the next version of the patch.
Axel Davy
___
mesa-dev mailing list
mesa-dev@lists.freedeskt
configure flags?
I tried the patches on top of febec731475a5b288cc50660b2d32780e395b950
and I didn't have any compiling issues, even when building gallium-egl.
Axel Davy
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedes
On 29/05/2014 23:31, Michel Dänzer a wrote :
On 30.05.2014 12:02, Axel Davy wrote:
Could you tell me what are your configure flags?
I tried the patches on top of febec731475a5b288cc50660b2d32780e395b950
I can still reproduce it on top of that.
and I didn't have any compiling issues,
On 30/05/2014 06:05, Michel Dänzer wrote :
On 30.05.2014 12:49, Axel Davy wrote:
On 29/05/2014 23:31, Michel Dänzer a wrote :
I tried with your compile options (except for the opencl flags, didn't
have the required libs), and it worked.
Did you try adding -lm to libloader_la_LIBADD ?
ateUsableExtensions should not get
called anymore with "display_is_direct_capable = False" for
GLX_USE_APPLEGL ?
Axel Davy
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 31/01/2014, Michel Dänzer wrote :
On Don, 2014-01-30 at 16:10 +0100, Axel Davy wrote:
dri2_dup_image was not copying the dri_format field.
This was causing some bugs, for example:
. we create an gbm_bo.
. we get an EGLImage from the gbm_bo.
. Bug: impossible to get again the gbm_bo from the
a
buffer with an handle, a Gem name or a prime fd.
Given the imported buffer can be shared with another card (in the prime fd
case), I imagine there are some cases where the buffer must stay in GTT.
What would happen in this case with your patch?
Axel Davy
_
ested your patch as is with my Wayland Prime
patches, and as expected it doesn't work well anymore.
Axel Davy
On 03/02/2014, Marek Olšák wrote :
Well, it's kinda obvious what would happen. The buffer would be
relocated to VRAM.
Do you have any suggestion how to find out if the buf
pixmaps to be argb, whereas by
creating the texture directly you can't assert this.
The bug you hit comes from this code.
I already corrected a bug recently in gallium code when it was importing
the gbm bo as EGLImage. A field wasn't copied.
I can't say from where the bug you
7;s in the specification.
v2: some cleanup and fixes
Signed-off-by: Axel Davy
---
src/egl/drivers/dri2/Makefile.am| 3 +-
src/egl/drivers/dri2/egl_dri2.h | 1 +
src/egl/drivers/dri2/platform_wayland.c | 260
3 files changed, 235 insertions(+), 29
will do)
. patch 7 introduces a new mode using blitImage. We render to
a buffer (with tiling) which is blitted to a shared buffer (a linear buffer)
Axel Davy (7):
wayland: Fix the logic in disabling the prime capability
EGL/Wayland: DRI_PRIME support v3
drirc: Add string support
EGL/Wayla
It allows to blit two __DRIimages.
Signed-off-by: Axel Davy
---
include/GL/internal/dri_interface.h | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/include/GL/internal/dri_interface.h
b/include/GL/internal/dri_interface.h
index d028d05..91f2fe3 100644
--- a
Add to drirc a new parameter: "wanted_device_id_path_tag"
to indicate the gpu we would like to use.
The EGL Wayland platform checks this parameter under
the driver name "init". It should be possible to get GLX
to check this parameter too, and use it for DRI3
GPU offloading.
What we want is disabling the path using primes fds if
the client can't create primes fds.
Signed-off-by: Axel Davy
Signed-off-by: Neil Roberts
---
Neil and me did the same correction independantly.
src/egl/drivers/dri2/platform_wayland.c | 2 +-
1 file changed, 1 insertion(+), 1 del
Signed-off-by: Axel Davy
---
src/mesa/drivers/dri/common/xmlconfig.c | 29 +
src/mesa/drivers/dri/common/xmlconfig.h | 7 ++-
2 files changed, 35 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/common/xmlconfig.c
b/src/mesa/drivers/dri/common
When the client use a different graphic device than the server,
we want for better performance to render to a tiled buffer,
and copy to a linear buffer which we share with the server.
Signed-off-by: Axel Davy
---
src/egl/drivers/dri2/egl_dri2.h | 2 +
src/egl/drivers/dri2
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/dri/drm/dri2.c | 44 ---
1 file changed, 41 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/dri/drm/dri2.c
b/src/gallium/state_trackers/dri/drm/dri2.c
index 3d8d569..4ddc81a 100644
--- a
I should have added that I choosed to be consistent with the code style
in the file
and then used GLuint as everywhere in the file.
Another patch should probably remove all occurences of GLuint in the
file to replace it with unsigned.
Axel Davy
Signed-off-by: Axel Davy
---
src/mesa
/?h=dri3%2Bgallium>
Without the DRIimage driver extension, egl won't launch on the requested
card using gallium, saying "wayland-egl: display is not render-node
capable", because
DRIimage driver extension is required to work with render-nodes.
Axel Davy
This is the updated
sn't have that yet).
The trick I had to use when writing the code is to convert to gbm_bo the
EGLImage,
then convert back the gbm_bo to EGLImage. The new EGLImage has no restrictions.
Axel Davy
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
dex)
Cc: "10.4"
Tested-by: Yaroslav Andrusyak
Acked-by: Ilia Mirkin
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_defines.h | 40 ++-
src/gallium/state_trackers/nine/nine_ff.c | 49 +++--
src/gallium/state_trackers/nine/nine_shader.h
TE, we can map all combinations.
Cc: "10.4"
Tested-by: Yaroslav Andrusyak
Reviewed-by: Marek Olšák
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 112 --
1 file changed, 52 insertions(+), 60 deletions(-)
diff --git a/src/gallium
This is the behaviour that Wine tests
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/state_trackers/nine/query9.c
index f22e821
avid Heidelberg
Signed-off-by: Axel Davy
---
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 a3184d9..dfb17b9 100644
--- a/src/gallium
. This patch drops the dummy queries
support and matches wine behaviour.
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 +++---
2 files changed, 12 insertions(+), 94
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
Signed-off-by: Axel Davy
---
src/gallium/state_trackers
get_query_result flushes automatically, we don't need to flush.
Signed-off-by: Axel Davy
---
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/state_tra
It is the same behaviour as wine has.
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/query9.c
b/src/gallium/state_trackers/nine/query9.c
Nine was allowing that behaviour, but was not filling the result.
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/query9.c | 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/src/gallium/state_trackers/nine/query9.c
b/src
This is the behaviour that wine tests.
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/query9.c
b/src/gallium/state_trackers/nine
y not using
XCB_PRESENT_OPTION_ASYNC, but this should not be the default.
You can add my
Reviewed-by: Axel Davy
Axel Davy
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
3_drawable *) pdraw;
- while (priv->recv_sbc < target_sbc) {
+ while ((target_sbc != 0 && priv->recv_sbc < target_sbc) ||
+ (target_sbc == 0 && priv->recv_sbc < priv->send_sbc)) {
if (!dri3_wait_for_event(pdraw))
return 0;
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
t vblank_ust, vblank_msc;
+
/* Serial numbers for tracking wait_for_msc events */
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 y
== 0 for divisor == 0.
+ */
+ 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
rrent msc
we make new 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.fr
ution, instead of switching to the new one ?
I know DRI3 is having a few issues to get support because of a few bugs
in the stack, but if what you want is just small improvement to reduce
overhead, then I woud think the answer is more implement that feature
with DRI3.
Axel
This->surfaces contains the surfaces associated to the levels
and faces. This->surfaces[6*Level] is what we want here,
since it gives us a face descriptor for the level 'Level'.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Signed-off-by: Xavier Bouchoux
Cc: "10
Use same similar settings as u_sampler_view_default_template
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/basetexture9.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/state_tra
ick Sarnie
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/swapchain9.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/swapchain9.c
b/src/gallium/state_trackers/nine/swapchain9.c
index bf87aaf..24ff
From: David Heidelberg
Signed-off-by: David Heidelberg
Reviewed-by: Axel Davy
---
src/gallium/state_trackers/nine/query9.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/query9.c
b/src/gallium/state_trackers/nine/query9.c
index 6df4ead..466b4ba 100644
From: xavier
It's done by testing the existence of the point sprite output register *after*
parsing the vertex shader.
Reviewed-by: David Heidelberg
Reviewed-by: Axel Davy
Signed-off-by: Xavier Bouchoux
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 6
e hit this case, which
prevents
crashing, but doesn't help with the resource content.
This fixes several games crashing at launch.
Acked-by: Axel Davy
Acked-by: David Heidelberg
Signed-off-by: Stanislaw Halik
Cc: "10.4"
---
src/gallium/state_trackers/nine/basetexture9.c | 3
Convert them to shader booleans at earlier stage
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/device9.c| 35 +---
src/gallium/state_trackers/nine/device9.h| 6 ++---
src/gallium/state_trackers/nine/nine_state.c | 13 +++-
The cap means D3DFVF_XYZRHW vertices will see clipping.
This is not the case when
PIPE_CAP_TGSI_VS_WINDOW_SPACE_POSITION is supported, since
it'll disable clipping.
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/adapter9.c | 4 +++-
1 file changed, 3 i
From: xavier
Reviewed-by: David Heidelberg
Reviewed-by: Axel Davy
Signed-off-by: Xavier Bouchoux
Cc: "10.4"
---
include/D3D9/d3d9types.h | 10 ++
1 file changed, 10 insertions(+)
diff --git a/include/D3D9/d3d9types.h b/include/D3D9/d3d9types.h
index 0a8f9e5..e53e389 10
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/cubetexture9.c | 8
src/gallium/state_trackers/nine/texture9.c | 9 -
src/gallium/state_trackers/nine/volumetexture9.c | 10 +-
3 files changed, 25 insertions(+), 2
ean there is
no mistake in these patches, so please review :-)
Axel Davy (48):
st/nine: Fix clip state logic
st/nine: Add new texture format strings
st/nine: Correctly advertise D3DPMISCCAPS_CLIPTLVERTS
st/nine: NineBaseTexture9: fix setting of last_layer
st/nine: CubeTexture: fix GetLeve
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
include/D3D9/d3d9types.h| 3 +++
src/gallium/state_trackers/nine/nine_pipe.h | 3 +++
2 files changed, 6 insertions(+)
diff --git a/include/D3D9/d3d9types.h b/include/D3D9/d3d9types.h
ind
From: xavier
Reviewed-by: David Heidelberg
Reviewed-by: Axel Davy
Signed-off-by: Xavier Bouchoux
Cc: "10.4"
---
include/D3D9/d3d9.h | 10 ++
1 file changed, 10 insertions(+)
diff --git a/include/D3D9/d3d9.h b/include/D3D9/d3d9.h
index f872be7..e8b5214 100644
--- a/in
The clip state was reset everytime, incurring an overhead.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_state.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_state.c
b/src/gallium
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 Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
s
Previous implementation was fine,
just instead of having increasing counter,
have a decreasing counter.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 41 +++
1 file changed, 23 insertions(+), 18 deletions(-)
diff --git a/src/gallium
Not filling them correctly results in bad padding and later crash.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 46 +--
1 file changed, 23 insertions(+), 23 deletions(-)
diff --git a/s
Relative addressing needs the constant buffer to get all
the correct constants, even those defined by the shader.
The code to copy the shader constants to the constant buffer
was enabled only for debug build. Enable it always.
Cc: "10.4"
Reviewed-by: David Heidelberg
Signed-off-by:
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
index c484e7c..02fb
According to msdn, we must act as if user didn't ask srgb if we don't
support it.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/basetexture9.c | 11 ++-
src/gallium/state_trackers/nine/surface9.c | 10
Signed-off-by: Axel Davy
Cc: "10.4"
---
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_trackers/nine/nine_ff.c
index a6bd360..d2b30f8 100644
--- a/s
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/device9.c | 10 ++
src/gallium/state_trackers/nine/device9.h | 2 ++
src/gallium/state_trackers/nine/stateblock9.c | 4 ++--
3 files changed, 10 insertions(+), 6 deletions(-)
diff --git a/s
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 27 ++-
1 file changed, 26 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
ind
The shader code is already behaving as if they are floats when the the card
doesn't support integers
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/device9.c | 65 ---
1 file changed, 52 insertions(+), 13 deletions(-)
d
POW doesn't match directly TGSI, since we should
take the absolute value of src0.
Fixes black textures in some games
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 13 -
1 file changed, 12 i
The fix is that this line:
"src[s] = tx->regs.vT[s];" is wrong if s doesn't start from 0.
Instead access tx->regs.vT directly when needed.
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 53 ++-
This is wine (and windows) behaviour.
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
ind
Nine code was clamping color outputs for vs < 3,
but msdn docs says it is done in the ps.
Wine seems to clamp them at the vs level.
It makes more sense to clamp at vs level for performance,
but according to doc, ps 2.x shouldn't see clamping.
Reviewed-by: David Heidelberg
Signed-off-
Signed-off-by: Axel Davy
Signed-off-by: Tiziano Bacocco
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 16 +++-
1 file changed, 7 insertions(+), 9 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_tra
ssing on the ps inputs.
Since here the inputs are not contiguous (we allocate temps)
it cannot be implemented (we have an assert for that currently
in the code, and at least one app was reported to need this to
work)
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c
Let's say we have c1 and c2 declared in the shader and c0 given by the app
Then here we would have read c0, c1 and c2 given by the app, instead
of the correct c0, c1, c2.
This correction fixes several issues in some games.
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium
relative adressing for constants is possible only for vs float
constants.
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_s
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 23 ++-
1 file changed, 22 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_tra
Since constant indirect adressing is not allowed for ps,
we can remove our code to handle that.
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_state.c | 43 +++---
src/gallium/state_trackers/nine/pixelshader9.c | 10 +++---
s
We should use the absolute value of the input as input to ureg_RSQ.
Moreover, an input of 0.0 should return FLT_MAX.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 13 -
1 file changed, 12 insert
needs a texture coord input and ff vs doesn't output
it is not handled, and should probably be a runtime error.
This fixes 3Dmark05, which uses ff vs with programmable ps.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_ff.c | 31 ---
1 file ch
Cc: "10.4"
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
ind
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 20 +++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_tra
crash while loading Bioshock -
bug https://bugs.freedesktop.org/show_bug.cgi?id=85696
Tested-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 156 +++---
1 file changed, 92 insertions(+), 64 deletions(-
upport for them. We are already having hard time supporting r500,
which is the most flexible dx9-only card apparently.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/adapter9.c | 106 +
src/gallium/state_trackers/nine/device9.c | 9 +--
2 files c
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 39 ++-
1 file changed, 38 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
ind
We should take the absolute value of the input.
Also return -FLT_MAX instead of -Inf for an input of 0.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 15 +--
1 file changed, 13 insertions(+), 2
When the shader does indirect addressing on the constants,
we allocate a temporary constant buffer to which we copy
the constants from the app given user constants and
the constants filled in the shader.
This patch makes this buffer be allocated once.
Signed-off-by: Axel Davy
Signed-off-by
off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 29 ++-
1 file changed, 24 insertions(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
index 899aee8..c
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_state.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_state.c
b/src/gallium/state_trackers/nine/nine_state.c
index 09b4401..99173fa 100644
--- a/src/gallium
to
0 or 0x.
Given 0x is NaN if it was a float, better use
UIF than IF.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 13 -
1 file changed, 4 insertions(+), 9 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
index 8bcf67b
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 39 ---
1 file changed, 36 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
ind
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 23 +--
1 file changed, 1 insertion(+), 22 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
index fcc1c6
According to docs and Wine, these two vs outputs have
to be saturated.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/state_tra
Adds ATI1 and ATI2 support to nine.
They map to PIPE_FORMAT_RGTC1_UNORM and PIPE_FORMAT_RGTC2_UNORM,
but need special handling.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Signed-off-by: Xavier Bouchoux
Cc: "10.4"
---
src/gallium/state_trackers/nine/adapter9.c
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/nine_shader.c | 20 +++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
index 02fb69
When the input's xyz are 0.0, the output
should be 0.0. This is due to the fact that
Inf * 0 = 0 for dx9. To handle this case,
cap the result of RSQ to FLT_MAX. We have
FLT_MAX * 0 = 0.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/stat
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/device9.c| 6 +++---
src/gallium/state_trackers/nine/nine_state.c | 7 ---
src/gallium/state_trackers/nine/nine_state.h | 2 +-
3 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/s
On Wed, 7 Jan 2015, Ilia Mirkin wrote:
On Wed, Jan 7, 2015 at 11:36 AM, Axel Davy wrote:
The cap means D3DFVF_XYZRHW vertices will see clipping.
This is not the case when
PIPE_CAP_TGSI_VS_WINDOW_SPACE_POSITION is supported, since
it'll disable clipping.
Signed-off-by: Axel Davy
Cc:
On 07/01/2015 18:42, Ilia Mirkin wrote :
On Wed, Jan 7, 2015 at 11:36 AM, Axel Davy wrote:
Adds ATI1 and ATI2 support to nine.
They map to PIPE_FORMAT_RGTC1_UNORM and PIPE_FORMAT_RGTC2_UNORM,
but need special handling.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Signed-off-by
On 07/01/2015 18:23, Ilia Mirkin wrote :
On Wed, Jan 7, 2015 at 11:36 AM, Axel Davy wrote:
Convert them to shader booleans at earlier stage
Why? What's wrong with the conversion as it is now?
The conversion is fine for booleans, but not for integers.
I think it is cleaner to have
Le 07/01/2015 19:09, Ilia Mirkin a écrit :
On Wed, Jan 7, 2015 at 11:36 AM, Axel Davy wrote:
We should use the absolute value of the input as input to ureg_RSQ.
Moreover, an input of 0.0 should return FLT_MAX.
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
Cc: "10.4"
Le 07/01/2015 21:13, Ilia Mirkin a écrit :
On Wed, Jan 7, 2015 at 11:36 AM, Axel Davy wrote:
Previous implementation was fine,
just instead of having increasing counter,
have a decreasing counter.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 41
On Wed, 7 Jan 2015, Ilia Mirkin wrote:
On Wed, Jan 7, 2015 at 11:36 AM, Axel Davy wrote:
Reviewed-by: David Heidelberg
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/cubetexture9.c | 8
src/gallium/state_trackers/nine/texture9.c | 9 -
src/gallium
On 07/01/2015 23:18, Axel Davy wrote :
On 07/01/2015 18:42, Ilia Mirkin wrote :
On Wed, Jan 7, 2015 at 11:36 AM, Axel Davy wrote:
Adds ATI1 and ATI2 support to nine.
They map to PIPE_FORMAT_RGTC1_UNORM and PIPE_FORMAT_RGTC2_UNORM,
but need special handling.
Reviewed-by: David Heidelberg
On 08/01/2015 05:08, Ilia Mirkin wrote :
On Wed, Jan 7, 2015 at 11:36 AM, Axel Davy wrote:
Signed-off-by: Axel Davy
Cc: "10.4"
---
src/gallium/state_trackers/nine/device9.c | 10 ++
src/gallium/state_trackers/nine/device9.h | 2 ++
src/gallium/state_tra
From: Xavier Bouchoux
Reviewed-by: Ilia Mirkin
Reviewed-by: David Heidelberg
Reviewed-by: Axel Davy
Signed-off-by: Xavier Bouchoux
Cc: "10.4"
---
include/D3D9/d3d9types.h | 10 ++
1 file changed, 10 insertions(+)
diff --git a/include/D3D9/d3d9types.h b/include/D3D9/d
501 - 600 of 1092 matches
Mail list logo