[Mesa-dev] [Bug 74717] r600g: 'invalid read' linking geometry shader

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74717 Michel Dänzer mic...@daenzer.net changed: What|Removed |Added Assignee|dri-devel@lists.freedesktop

[Mesa-dev] [Bug 74727] Mesa 10.1: doesn't build with DRI3 enabled

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74727 Michel Dänzer mic...@daenzer.net changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop.

[Mesa-dev] [PATCH] i965: Don't call abort() on an unknown device.

2014-02-10 Thread Kenneth Graunke
If we don't recognize the PCI ID, we can't reasonably load the driver. However, calling abort() is quite rude - it means the application that tried to initialize us (possibly the X server) can't continue via fallback paths. We already have a more polite mechanism - failing to create the context.

[Mesa-dev] [PATCH 3/3] glx/apple: Fixed glx context memory leak in case of failure.

2014-02-10 Thread Siavash Eliasi
--- src/glx/apple/apple_glx_context.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/glx/apple/apple_glx_context.c b/src/glx/apple/apple_glx_context.c index 0bb25b4..8a6ded2 100644 --- a/src/glx/apple/apple_glx_context.c +++ b/src/glx/apple/apple_glx_context.c @@ -142,6 +142,7 @@

[Mesa-dev] [PATCH 2/3] gbm/dri: Fixed buffer object memory leak in case of failure.

2014-02-10 Thread Siavash Eliasi
--- src/gbm/backends/dri/gbm_dri.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/gbm/backends/dri/gbm_dri.c b/src/gbm/backends/dri/gbm_dri.c index e013980..db4e074 100644 --- a/src/gbm/backends/dri/gbm_dri.c +++ b/src/gbm/backends/dri/gbm_dri.c @@ -609,7 +609,7

[Mesa-dev] [PATCH 1/3] r300g/tests: Added missing fclose for FILE resource.

2014-02-10 Thread Siavash Eliasi
--- src/gallium/drivers/r300/compiler/tests/rc_test_helpers.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/r300/compiler/tests/rc_test_helpers.c b/src/gallium/drivers/r300/compiler/tests/rc_test_helpers.c index b4e30d8..239a762 100644 ---

[Mesa-dev] Texture from RenderBuffer for blitting

2014-02-10 Thread Rogovin, Kevin
Hi, I was looking over the _mesa_meta_BlitFramebuffer and in particular, blitframebuffer_texture(). A few idea (feedback welcome and wanted): Firstly, the check if src is a renderbuffer or not. I think it might be a good idea to add an internal Mesa API to allow a driver to specify that a

Re: [Mesa-dev] Texture from RenderBuffer for blitting

2014-02-10 Thread Rogovin, Kevin
Hi, Um, please ignore the 1st suggestion; it was already in Mesa and I was not awake of enough. Sighs. From: Rogovin, Kevin Sent: Monday, February 10, 2014 12:06 PM To: mesa-dev@lists.freedesktop.org Cc: Romanick, Ian D Subject: Texture from RenderBuffer for

[Mesa-dev] [PATCH] meta_blitframebuffer integer format corrections

2014-02-10 Thread Kevin Rogovin
Corrects the format for the temp texture used in for the case where source framebuffer is an integer texture when blitframebuffer_texture() cannot be used. Note: Does NOT correct the underling blits, rather only corrects the format of the temporary texture to be an integer format. ---

[Mesa-dev] [Bug 74760] Anno 1701 colors of textures are corrupted using Intel GM45 Express chipset

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74760 --- Comment #4 from chefche...@gmx.de --- Created attachment 93763 -- https://bugs.freedesktop.org/attachment.cgi?id=93763action=edit winedbg error log -- You are receiving this mail because: You are the assignee for the bug.

Re: [Mesa-dev] [PATCH 2/3] st/xa: use pipe-loader to get screen

2014-02-10 Thread Emil Velikov
Hi Rob On 08/02/14 22:41, Rob Clark wrote: From: Rob Clark robcl...@freedesktop.org This lets multiple gallium drivers use XA. Signed-off-by: Rob Clark robcl...@freedesktop.org --- configure.ac | 8 ++-- src/gallium/state_trackers/xa/Makefile.am |

[Mesa-dev] [Bug 74760] Anno 1701 colors of textures are corrupted using Intel GM45 Express chipset

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74760 --- Comment #6 from chefche...@gmx.de --- Created attachment 93768 -- https://bugs.freedesktop.org/attachment.cgi?id=93768action=edit intel bios dump -- You are receiving this mail because: You are the assignee for the bug.

[Mesa-dev] [Bug 74760] Anno 1701 colors of textures are corrupted using Intel GM45 Express chipset

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74760 --- Comment #5 from chefche...@gmx.de --- Created attachment 93767 -- https://bugs.freedesktop.org/attachment.cgi?id=93767action=edit intel reg dumper output -- You are receiving this mail because: You are the assignee for the bug.

[Mesa-dev] [Bug 74760] Anno 1701 colors of textures are corrupted using Intel GM45 Express chipset

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74760 --- Comment #7 from chefche...@gmx.de --- lspci 00:00.0 Host bridge: Intel Corporation Mobile 4 Series Chipset Memory Controller Hub (rev 07) 00:02.0 VGA compatible controller: Intel Corporation Mobile 4 Series Chipset Integrated Graphics

Re: [Mesa-dev] [PATCH 3/3] pipe-loader: add pipe loader for freedreno/msm

2014-02-10 Thread Emil Velikov
On 08/02/14 22:41, Rob Clark wrote: From: Rob Clark robcl...@freedesktop.org Signed-off-by: Rob Clark robcl...@freedesktop.org --- src/gallium/targets/pipe-loader/Makefile.am | 16 src/gallium/targets/pipe-loader/pipe_msm.c | 20 2 files changed, 36

[Mesa-dev] [Bug 74717] r600g: 'invalid read' linking geometry shader

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74717 --- Comment #8 from Dieter Nützel die...@nuetzel-hh.de --- 3.13.2 + geom + UVD IRQ patch Mesa 10.2.0-devel (git-a487ef8) LLVM 3.4 stable RV730 AGP With R600_DEBUG=nollvm it sometimes runs without crash but empty (black) window and somtimes like

Re: [Mesa-dev] [PATCH 2/3] st/xa: use pipe-loader to get screen

2014-02-10 Thread Rob Clark
On Mon, Feb 10, 2014 at 7:37 AM, Emil Velikov emil.l.veli...@gmail.com wrote: Hi Rob On 08/02/14 22:41, Rob Clark wrote: From: Rob Clark robcl...@freedesktop.org This lets multiple gallium drivers use XA. Signed-off-by: Rob Clark robcl...@freedesktop.org --- configure.ac

[Mesa-dev] [Bug 74760] Anno 1701 colors of textures are corrupted using Intel GM45 Express chipset

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74760 --- Comment #8 from chefche...@gmx.de --- Created attachment 93772 -- https://bugs.freedesktop.org/attachment.cgi?id=93772action=edit strace extract -- You are receiving this mail because: You are the assignee for the bug.

Re: [Mesa-dev] [PATCH 1/3] pipe-loader: split out client version

2014-02-10 Thread Emil Velikov
On 08/02/14 22:41, Rob Clark wrote: From: Rob Clark robcl...@freedesktop.org Build two versions of pipe-loader, with only the client version linking in x11 client side dependencies. This will allow the XA state tracker to use pipe-loader. AFAICS it's rather messy story *

Re: [Mesa-dev] [PATCH 1/3] pipe-loader: split out client version

2014-02-10 Thread Emil Velikov
On 08/02/14 22:41, Rob Clark wrote: From: Rob Clark robcl...@freedesktop.org Build two versions of pipe-loader, with only the client version linking in x11 client side dependencies. This will allow the XA state tracker to use pipe-loader. Signed-off-by: Rob Clark robcl...@freedesktop.org

[Mesa-dev] [Bug 67676] Transparent windows no longer work

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=67676 --- Comment #6 from Sjoerd Simons sjo...@luon.net --- Created attachment 93777 -- https://bugs.freedesktop.org/attachment.cgi?id=93777action=edit Proposed patch egl_dri2: Allow both 24 and 32 bit X visuals for RGBA configs When using RGBA

Re: [Mesa-dev] [PATCH 3/3] glx/apple: Fixed glx context memory leak in case of failure.

2014-02-10 Thread Ian Romanick
Perhaps Jeremy can review this patch... On 02/10/2014 01:58 AM, Siavash Eliasi wrote: --- src/glx/apple/apple_glx_context.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/glx/apple/apple_glx_context.c b/src/glx/apple/apple_glx_context.c index 0bb25b4..8a6ded2 100644 ---

Re: [Mesa-dev] [PATCH 1/2] configure: error out when building radeonsi without gallium-llvm

2014-02-10 Thread Tom Stellard
On Sat, Feb 08, 2014 at 03:52:29PM +, Emil Velikov wrote: --enable-gallium-llvm is required by radeonsi. Currently we check only for LLVM_VERSION_INT which is 0, whenever gallium-llvm is disabled explicitly. ./configure --with-gallium-drivers=r600,radeonsi --disable-gallium-llvm I

Re: [Mesa-dev] [PATCH 02/22] targets/pipe-loader: automake: drop obsolete version-script

2014-02-10 Thread Tom Stellard
On Sat, Feb 08, 2014 at 08:13:48PM +, Emil Velikov wrote: On 08/02/14 17:53, Matt Turner wrote: The purpose of this version script looks to be related to static builds, but in your cover letter you said you were building with shared LLVM libraries. Does this affect static builds?

Re: [Mesa-dev] [PATCH 1/3] r300g/tests: Added missing fclose for FILE resource.

2014-02-10 Thread Tom Stellard
Reviewed-by: Tom Stellard thomas.stell...@amd.com On Mon, Feb 10, 2014 at 01:28:14PM +0330, Siavash Eliasi wrote: --- src/gallium/drivers/r300/compiler/tests/rc_test_helpers.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/r300/compiler/tests/rc_test_helpers.c

[Mesa-dev] [PATCH] configure: error out when building radeonsi without gallium-llvm

2014-02-10 Thread Emil Velikov
--enable-gallium-llvm is required by radeonsi. Currently we check only for LLVM_VERSION_INT which is 0, whenever gallium-llvm is disabled explicitly. ./configure --with-gallium-drivers=r600,radeonsi --disable-gallium-llvm v2: Correct typo in error message. Spotted by Tom Stellard Signed-off-by:

Re: [Mesa-dev] [PATCH] configure: error out when building radeonsi without gallium-llvm

2014-02-10 Thread Tom Stellard
On Mon, Feb 10, 2014 at 04:37:31PM +, Emil Velikov wrote: --enable-gallium-llvm is required by radeonsi. Currently we check only for LLVM_VERSION_INT which is 0, whenever gallium-llvm is disabled explicitly. ./configure --with-gallium-drivers=r600,radeonsi --disable-gallium-llvm v2:

Re: [Mesa-dev] [PATCH 3/3] pipe-loader: add pipe loader for freedreno/msm

2014-02-10 Thread Johannes Obermayr
There should be one called pipe_freedreno and the pipe driver loader should support sth. like: if ((strcmp(dev-driver_name, kgsl) == 0) || (strcmp(dev-driver_name, msm) == 0)) dev-driver_name = freedreno; in pipe_loader_find_module ... ___

[Mesa-dev] [Bug 74803] New: Cogs shadows are broken

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74803 Priority: medium Bug ID: 74803 Assignee: mesa-dev@lists.freedesktop.org Summary: Cogs shadows are broken Severity: normal Classification: Unclassified OS: Linux (All)

[Mesa-dev] [Bug 74803] Cogs shadows are broken

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74803 funkydude funkydud...@hotmail.co.uk changed: What|Removed |Added Attachment #93791|0 |1 is

Re: [Mesa-dev] [PATCH] i965: Don't call abort() on an unknown device.

2014-02-10 Thread Ian Romanick
On 02/10/2014 01:54 AM, Kenneth Graunke wrote: If we don't recognize the PCI ID, we can't reasonably load the driver. However, calling abort() is quite rude - it means the application that tried to initialize us (possibly the X server) can't continue via fallback paths. We already have a more

Re: [Mesa-dev] [PATCH 3/3] pipe-loader: add pipe loader for freedreno/msm

2014-02-10 Thread Rob Clark
On Mon, Feb 10, 2014 at 12:30 PM, Johannes Obermayr johannesoberm...@gmx.de wrote: There should be one called pipe_freedreno and the pipe driver loader should support sth. like: if ((strcmp(dev-driver_name, kgsl) == 0) || (strcmp(dev-driver_name, msm) == 0)) dev-driver_name =

[Mesa-dev] [Bug 74803] Cogs shadows are broken

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74803 --- Comment #2 from Alex Deucher ag...@yahoo.com --- Is this a regression? If so, can you bisect? Also, please attach your xorg log and glxinfo and dmesg output. -- You are receiving this mail because: You are the assignee for the bug.

[Mesa-dev] [PATCH] R600/SI: Completely Disable TypeRewriter on compute

2014-02-10 Thread Tom Stellard
From: Tom Stellard thomas.stell...@amd.com --- lib/Target/R600/SITypeRewriter.cpp | 6 +++--- test/CodeGen/R600/bitcast.ll | 9 + 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/lib/Target/R600/SITypeRewriter.cpp b/lib/Target/R600/SITypeRewriter.cpp index

[Mesa-dev] [PATCHv2 3/4] st/xa: missing handle type

2014-02-10 Thread Rob Clark
From: Rob Clark robcl...@freedesktop.org DRM_API_HANDLE_TYPE_SHARED is zero, so doesn't actually fix anything. But we shouldn't rely on SHARED handle type being zero. Signed-off-by: Rob Clark robcl...@freedesktop.org --- src/gallium/state_trackers/xa/xa_tracker.c | 1 + 1 file changed, 1

[Mesa-dev] [PATCHv2 4/4] pipe-loader: add pipe loader for freedreno/msm

2014-02-10 Thread Rob Clark
From: Rob Clark robcl...@freedesktop.org Signed-off-by: Rob Clark robcl...@freedesktop.org --- src/gallium/targets/pipe-loader/Makefile.am | 16 src/gallium/targets/pipe-loader/pipe_msm.c | 21 + 2 files changed, 37 insertions(+) create mode 100644

[Mesa-dev] [PATCHv2 0/4] enable XA for freedreno

2014-02-10 Thread Rob Clark
From: Rob Clark robcl...@freedesktop.org Original patchset: http://lists.freedesktop.org/archives/mesa-dev/2014-February/053632.html v1: original v2: moves xa target into targets/xa, and fixes various issues spotted by Emil Rob Clark (4): pipe-loader: split out client version st/xa: use

[Mesa-dev] [PATCHv2 2/4] st/xa: use pipe-loader to get screen

2014-02-10 Thread Rob Clark
From: Rob Clark robcl...@freedesktop.org This lets multiple gallium drivers use XA. Signed-off-by: Rob Clark robcl...@freedesktop.org --- configure.ac | 9 ++-- src/gallium/state_trackers/xa/Makefile.am | 1 + src/gallium/state_trackers/xa/xa_priv.h

[Mesa-dev] [PATCHv2 1/4] pipe-loader: split out client version

2014-02-10 Thread Rob Clark
From: Rob Clark robcl...@freedesktop.org Build two versions of pipe-loader, with only the client version linking in x11 client side dependencies. This will allow the XA state tracker to use pipe-loader. Signed-off-by: Rob Clark robcl...@freedesktop.org --- configure.ac

[Mesa-dev] [Bug 74803] Cogs shadows are broken

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74803 --- Comment #3 from funkydude funkydud...@hotmail.co.uk --- No idea if it's a regression. The first time I played this game on Linux was ~2 weeks ago and the issue has been there since that time. Shadows are on by default, and this is a pretty

[Mesa-dev] [Bug 74803] Cogs shadows are broken

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74803 --- Comment #4 from Alex Deucher ag...@yahoo.com --- Just redirect the output to a file: glxinfo glxinfo.txt dmesg dmesg.txt The xorg log location is usually /var/log/Xorg.0.log. Don't worry about deleting it. It gets overwritten each time

[Mesa-dev] [PATCH 1/7] i965/blorp: refactor assertions for tile offset alignments

2014-02-10 Thread Topi Pohjolainen
There are only two callers for compute_tile_offsets() and both require the exact same constraints for the tile offsets. Signed-off-by: Topi Pohjolainen topi.pohjolai...@intel.com --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 6 ++ src/mesa/drivers/dri/i965/gen6_blorp.cpp | 5 -

[Mesa-dev] [PATCH 2/7] i965/blorp: split calculations of page and tile offsets

2014-02-10 Thread Topi Pohjolainen
These will be handled separately in the coming patches. Signed-off-by: Topi Pohjolainen topi.pohjolai...@intel.com --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 21 + src/mesa/drivers/dri/i965/brw_blorp.h| 3 ++- src/mesa/drivers/dri/i965/gen6_blorp.cpp | 4 ++--

[Mesa-dev] Blorp without surface state xy-offsets

2014-02-10 Thread Topi Pohjolainen
Surface state for blorp handles miplevel (and layer) offsets by adding full pages to the base address and the remaining offset within a tile (page) using special tile_xy-fields in the surface state config. This series moves the intra-tile offset handling to the blorp program. Vertices provided to

[Mesa-dev] [PATCH 3/7] i965/blorp: add intra-tile offset support without surface config

2014-02-10 Thread Topi Pohjolainen
In gen6/7 the surface state can be adjusted with these directly but starting from gen8 there isn't that option anymore. In this patch these are still just zero and have no effect. Signed-off-by: Topi Pohjolainen topi.pohjolai...@intel.com --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 42

[Mesa-dev] [PATCH 4/7] i965/blorp: handle destination tile offsets for w-tiled in the program

2014-02-10 Thread Topi Pohjolainen
instead of using the surface state x/y-offsets. These are not available in the gen8 anymore. No regressions on IVB and SNB. Signed-off-by: Topi Pohjolainen topi.pohjolai...@intel.com --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 7 +++ src/mesa/drivers/dri/i965/gen6_blorp.cpp | 22

[Mesa-dev] [PATCH 6/7] i965/blorp: handle tile offsets for all destinations in the program

2014-02-10 Thread Topi Pohjolainen
Note that tile offset calculation is unnecessary for hiz as it does not have a shader that would use them nor does the hiz execution update the surface state table for destination - it configures only the depth surface which is handled separately. No regressions on IVB and SNB. Signed-off-by:

[Mesa-dev] [PATCH 7/7] i965/blorp: handle tile offsets for all sources in the program

2014-02-10 Thread Topi Pohjolainen
Note that hiz and clear programs need no consideration here as they do not use any source surfaces. No regressions on IVB and SNB. Signed-off-by: Topi Pohjolainen topi.pohjolai...@intel.com --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 10 ++ src/mesa/drivers/dri/i965/gen6_blorp.cpp |

[Mesa-dev] [PATCH 5/7] i965/blorp: handle source tile offsets for w-tiled in the program

2014-02-10 Thread Topi Pohjolainen
instead of using the surface state x/y-offsets. These are not available in the gen8 anymore. No regressions on IVB and SNB. Signed-off-by: Topi Pohjolainen topi.pohjolai...@intel.com --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 7 +++ src/mesa/drivers/dri/i965/gen6_blorp.cpp | 4 ++--

[Mesa-dev] [Bug 74803] Cogs shadows are broken

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74803 --- Comment #5 from funkydude funkydud...@hotmail.co.uk --- Created attachment 93797 -- https://bugs.freedesktop.org/attachment.cgi?id=93797action=edit glxinfo -- You are receiving this mail because: You are the assignee for the bug.

[Mesa-dev] [Bug 74803] Cogs shadows are broken

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74803 --- Comment #6 from funkydude funkydud...@hotmail.co.uk --- Created attachment 93798 -- https://bugs.freedesktop.org/attachment.cgi?id=93798action=edit xorg log -- You are receiving this mail because: You are the assignee for the bug.

[Mesa-dev] [Bug 74803] Cogs shadows are broken

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74803 --- Comment #7 from funkydude funkydud...@hotmail.co.uk --- I purged the PPA which downgraded me to mesa 9.2.1 (I think). The issue is still there. -- You are receiving this mail because: You are the assignee for the bug.

[Mesa-dev] [PATCH] glsl: Do not vectorize vector array dereferences.

2014-02-10 Thread Matt Turner
Array dereferences must have scalar indices, so we cannot vectorize them. Reported-by: Andrew Guertin li...@dolphinling.net --- src/glsl/opt_vectorize.cpp | 14 ++ 1 file changed, 14 insertions(+) diff --git a/src/glsl/opt_vectorize.cpp b/src/glsl/opt_vectorize.cpp index

Re: [Mesa-dev] glsl: Vectorize multiple scalar assignments (mesa 4bd6e0d) breaks League of Legends on wine

2014-02-10 Thread Matt Turner
On Mon, Feb 10, 2014 at 11:08 AM, Andrew Guertin li...@dolphinling.net wrote: On 02/10/2014 12:11 AM, Matt Turner wrote: On Sat, Feb 8, 2014 at 5:29 AM, Andrew Guertin li...@dolphinling.net wrote: On 02/08/2014 02:41 AM, Matt Turner wrote: On Fri, Feb 7, 2014 at 10:22 PM, Andrew Guertin

Re: [Mesa-dev] [PATCH 3/3] glx/apple: Fixed glx context memory leak in case of failure.

2014-02-10 Thread Jeremy Huddleston Sequoia
Yep, looks right to me. Reviewed-by: Jeremy Huddleston Sequoia: jerem...@apple.com On Feb 10, 2014, at 07:57, Ian Romanick i...@freedesktop.org wrote: Perhaps Jeremy can review this patch... On 02/10/2014 01:58 AM, Siavash Eliasi wrote: --- src/glx/apple/apple_glx_context.c | 1 + 1 file

[Mesa-dev] [PATCH] R600/SI: Add a pattern for i32 anyext

2014-02-10 Thread Tom Stellard
From: Tom Stellard thomas.stell...@amd.com --- lib/Target/R600/SIInstructions.td | 7 +-- test/CodeGen/R600/anyext.ll | 14 ++ 2 files changed, 19 insertions(+), 2 deletions(-) create mode 100644 test/CodeGen/R600/anyext.ll diff --git a/lib/Target/R600/SIInstructions.td

[Mesa-dev] [PATCH] R600/SI: Split global vector loads with more than 4 elements

2014-02-10 Thread Tom Stellard
From: Tom Stellard thomas.stell...@amd.com --- lib/Target/R600/SIISelLowering.cpp | 8 +- test/CodeGen/R600/load.ll | 178 +++-- 2 files changed, 98 insertions(+), 88 deletions(-) diff --git a/lib/Target/R600/SIISelLowering.cpp

Re: [Mesa-dev] Mesa (master): nv30: report 8 maximum inputs

2014-02-10 Thread Ian Romanick
On 02/08/2014 04:18 PM, Ilia Mirkin wrote: Module: Mesa Branch: master Commit: 356aff3a5c08be055d6befff99a72f5551b3ac2d URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=356aff3a5c08be055d6befff99a72f5551b3ac2d Author: Ilia Mirkin imir...@alum.mit.edu Date: Wed Jan 29 12:36:13

Re: [Mesa-dev] Mesa (master): nv30: report 8 maximum inputs

2014-02-10 Thread Ilia Mirkin
On Mon, Feb 10, 2014 at 4:43 PM, Ian Romanick i...@freedesktop.org wrote: On 02/08/2014 04:18 PM, Ilia Mirkin wrote: Module: Mesa Branch: master Commit: 356aff3a5c08be055d6befff99a72f5551b3ac2d URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=356aff3a5c08be055d6befff99a72f5551b3ac2d

[Mesa-dev] [Bug 74471] [swrast] piglit ext_texture_array-gen-mipmap regression

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74471 Ian Romanick i...@freedesktop.org changed: What|Removed |Added Status|NEW |RESOLVED

[Mesa-dev] [Bug 73946] scanout broken on radeon SI (OLAND)

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73946 --- Comment #11 from Alex Deucher ag...@yahoo.com --- Also for oland, make sure your kernel has this patch:

Re: [Mesa-dev] [PATCH] glsl: Do not vectorize vector array dereferences.

2014-02-10 Thread Andrew Guertin
Tested-by: Andrew Guertin li...@dolphinling.net On 02/10/2014 04:09 PM, Matt Turner wrote: Array dereferences must have scalar indices, so we cannot vectorize them. Reported-by: Andrew Guertin li...@dolphinling.net --- src/glsl/opt_vectorize.cpp | 14 ++ 1 file changed, 14

Re: [Mesa-dev] glsl: Vectorize multiple scalar assignments (mesa 4bd6e0d) breaks League of Legends on wine

2014-02-10 Thread Andrew Guertin
On 02/10/2014 12:11 AM, Matt Turner wrote: On Sat, Feb 8, 2014 at 5:29 AM, Andrew Guertin li...@dolphinling.net wrote: On 02/08/2014 02:41 AM, Matt Turner wrote: On Fri, Feb 7, 2014 at 10:22 PM, Andrew Guertin li...@dolphinling.net wrote: Hi, I updated mesa and started getting some bad

Re: [Mesa-dev] [PATCH] R600/SI: Split global vector loads with more than 4 elements

2014-02-10 Thread Matt Arsenault
Why would you want to do this for the small types? You should be able to load those in fewer loads and then promote them. On 02/10/2014 01:32 PM, Tom Stellard wrote: From: Tom Stellard thomas.stell...@amd.com --- lib/Target/R600/SIISelLowering.cpp | 8 +- test/CodeGen/R600/load.ll

[Mesa-dev] [PATCH] mesa: assorted clean-ups in detach_shader()

2014-02-10 Thread Brian Paul
Fix formatting, add new comments, get rid of extraneous indentation. Suggested by Ian in bug 74723. --- src/mesa/main/shaderapi.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c index

[Mesa-dev] [PATCH 1/2] i965/fs: Simplify FS_OPCODE_SET_SAMPLE_ID stride mashing a bit.

2014-02-10 Thread Kenneth Graunke
stride(brw_vec1_reg(...) ...) takes some register, changes the strides, then changes the strides again. Let's do it once. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH 2/2] i965/fs: Simplify FS_OPCODE_SET_OMASK stride mashing a bit.

2014-02-10 Thread Kenneth Graunke
In the first case, we can simply call stride(mask, 16, 8, 2) rather than creating a new register with a different stride, then immediately changing it a second time. In the second case, the stride was already what we wanted, so we can just use mask without any changes at all. Signed-off-by:

Re: [Mesa-dev] Mesa (master): nv30: report 8 maximum inputs

2014-02-10 Thread Ian Romanick
On 02/10/2014 02:04 PM, Ilia Mirkin wrote: On Mon, Feb 10, 2014 at 4:43 PM, Ian Romanick i...@freedesktop.org wrote: On 02/08/2014 04:18 PM, Ilia Mirkin wrote: Module: Mesa Branch: master Commit: 356aff3a5c08be055d6befff99a72f5551b3ac2d URL:

Re: [Mesa-dev] [PATCH] mesa: assorted clean-ups in detach_shader()

2014-02-10 Thread Ian Romanick
On 02/10/2014 03:43 PM, Brian Paul wrote: Fix formatting, add new comments, get rid of extraneous indentation. Suggested by Ian in bug 74723. --- src/mesa/main/shaderapi.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git

Re: [Mesa-dev] [PATCH] glsl: Do not vectorize vector array dereferences.

2014-02-10 Thread Ian Romanick
On 02/10/2014 01:09 PM, Matt Turner wrote: Array dereferences must have scalar indices, so we cannot vectorize them. Reported-by: Andrew Guertin li...@dolphinling.net Seems like the easy fix. Reviewed-by: Ian Romanick ian.d.roman...@intel.com Candidate for 10.1? ---

Re: [Mesa-dev] [PATCH 1/2] i965/fs: Simplify FS_OPCODE_SET_SAMPLE_ID stride mashing a bit.

2014-02-10 Thread Ian Romanick
Series is Reviewed-by: Ian Romanick ian.d.roman...@intel.com On 02/10/2014 03:46 PM, Kenneth Graunke wrote: stride(brw_vec1_reg(...) ...) takes some register, changes the strides, then changes the strides again. Let's do it once. Signed-off-by: Kenneth Graunke kenn...@whitecape.org ---

Re: [Mesa-dev] [PATCH] glsl: Do not vectorize vector array dereferences.

2014-02-10 Thread Matt Turner
On Mon, Feb 10, 2014 at 4:32 PM, Ian Romanick i...@freedesktop.org wrote: On 02/10/2014 01:09 PM, Matt Turner wrote: Array dereferences must have scalar indices, so we cannot vectorize them. Reported-by: Andrew Guertin li...@dolphinling.net Seems like the easy fix. Reviewed-by: Ian

[Mesa-dev] [PATCH 1/3] glsl: Compile error if fs defines conflicting qualifiers for gl_FragCoord

2014-02-10 Thread Anuj Phogat
GLSL 1.50 spec says: If gl_FragCoord is redeclared in any fragment shader in a program, it must be redeclared in all the fragment shaders in that program that have a static use gl_FragCoord. All redeclarations of gl_FragCoord in all fragment shaders in a single program must have

[Mesa-dev] [PATCH 2/3] glsl: Link error if fs defines conflicting qualifiers for gl_FragCoord

2014-02-10 Thread Anuj Phogat
GLSL 1.50 spec says: If gl_FragCoord is redeclared in any fragment shader in a program, it must be redeclared in all the fragment shaders in that program that have a static use gl_FragCoord. All redeclarations of gl_FragCoord in all fragment shaders in a single program must have

[Mesa-dev] [PATCH 3/3] glsl: Apply the link error conditions to GL_ARB_fragment_coord_conventions

2014-02-10 Thread Anuj Phogat
Link error conditions added in previous patch are equally applicable to GL_ARB_fragment_coord_conventions implementation. Extension's spec says: If gl_FragCoord is redeclared in any fragment shader in a program, it must be redeclared in all the fragment shaders in that program that have

[Mesa-dev] [Bug 74803] Cogs shadows are broken

2014-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74803 --- Comment #8 from Marek Olšák mar...@gmail.com --- Try to set this environment variable: R600_DEBUG=nohyperz -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev

[Mesa-dev] [PATCH 3/3] st/mesa: add texture gather support. (v2)

2014-02-10 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This adds support for GL_ARB_texture_gather, and one step of support for GL_ARB_gpu_shader5. This adds support for passing the TG4 instruction, along with non-constant texture offsets, and tracking them for the optimisation passes. This doesn't support

[Mesa-dev] [PATCH 2/3] gallium: add texture gather support to gallium (v3)

2014-02-10 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This adds support to gallium for a TG4 instruction, and two CAPs. The first CAP is required for GL_ARB_texture_gather. The second CAP is required to expose GL_ARB_gpu_shader5. However so far we haven't found any hardware that natively exposes the

[Mesa-dev] gallium/st: add texture gather support

2014-02-10 Thread Dave Airlie
This adds support to gallium, and also to the state tracker for ARB_texture_gather and also the gather features of ARB_gpu_shader5. It adds a single CAP for the ARB_texture_gather max components query, then another CAP to denote the GPU is capable of SM5 gather. This doesn't add native support

[Mesa-dev] [PATCH 1/3] glsl/i965: move lower_offset_array up to GLSL compiler level.

2014-02-10 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This lowering pass will be useful for gallium drivers as well, in order to support the GL TG4 oddity that is textureGatherOffsets. Signed-off-by: Dave Airlie airl...@redhat.com --- src/glsl/Makefile.sources | 1 +

Re: [Mesa-dev] Mesa (master): nv30: report 8 maximum inputs

2014-02-10 Thread Ilia Mirkin
On Mon, Feb 10, 2014 at 7:12 PM, Ian Romanick i...@freedesktop.org wrote: On 02/10/2014 02:04 PM, Ilia Mirkin wrote: On Mon, Feb 10, 2014 at 4:43 PM, Ian Romanick i...@freedesktop.org wrote: On 02/08/2014 04:18 PM, Ilia Mirkin wrote: Module: Mesa Branch: master Commit:

[Mesa-dev] [PATCH 02/10] r600g,radeonsi: share r600_surface

2014-02-10 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com I'm gonna use this in radeonsi. --- src/gallium/drivers/r600/r600_resource.h | 44 -- src/gallium/drivers/radeon/r600_pipe_common.h | 45 +++ src/gallium/drivers/radeonsi/si_pipe.h| 4 ---

[Mesa-dev] [PATCH 01/10] radeonsi: move PA_SU_POLY_OFFSET_DB_FMT_CNTL to framebuffer state

2014-02-10 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com It doesn't depend on anything else. --- src/gallium/drivers/radeonsi/si_state.c | 29 + 1 file changed, 21 insertions(+), 8 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state.c

[Mesa-dev] [PATCH 07/10] radeonsi: compute depth surface registers only once

2014-02-10 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/radeonsi/si_state.c | 98 ++--- 1 file changed, 54 insertions(+), 44 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c index 89b4242..6642c7b

[Mesa-dev] [PATCH 05/10] r600g: remove r600_resource.h

2014-02-10 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/r600/compute_memory_pool.c | 1 - src/gallium/drivers/r600/evergreen_compute.c | 1 - src/gallium/drivers/r600/evergreen_compute.h | 7 src/gallium/drivers/r600/r600_pipe.h | 9 +-

[Mesa-dev] [PATCH 03/10] r600g: use r600_surface::db_z_info

2014-02-10 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com db_z_info was unused. This just renames the variable to match the register name. Now, db_depth_info is unused on Evergreen. Both variables will be needed on SI though. --- src/gallium/drivers/r600/evergreen_state.c | 20 ++-- 1 file changed,

[Mesa-dev] [PATCH 04/10] r600g: remove r600_surface::htile_enabled

2014-02-10 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com We can just use the base address register instead. --- src/gallium/drivers/r600/evergreen_state.c| 6 ++ src/gallium/drivers/r600/r600_state.c | 6 ++ src/gallium/drivers/radeon/r600_pipe_common.h | 2 -- 3 files changed, 4 insertions(+),

[Mesa-dev] [PATCH 10/10] r600g, radeonsi: consolidate create_surface and surface_destroy

2014-02-10 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/r600/r600_state_common.c | 44 --- src/gallium/drivers/radeon/r600_pipe_common.c | 3 +- src/gallium/drivers/radeon/r600_pipe_common.h | 7 +++- src/gallium/drivers/radeon/r600_texture.c | 52

[Mesa-dev] [PATCH 09/10] radeonsi: inline util_blitter_copy_texture

2014-02-10 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com This will be used for changing texture properties without modifying pipe_resource like r600g, but not in this series. For now, this change allows consolidation of pipe_surface functions. --- src/gallium/drivers/radeonsi/si_blit.c | 24

[Mesa-dev] [PATCH 06/10] radeonsi: compute color surface registers only once

2014-02-10 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com Same as r600g. --- src/gallium/drivers/radeonsi/si_state.c | 99 ++--- 1 file changed, 55 insertions(+), 44 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c index

[Mesa-dev] [PATCH 08/10] radeonsi: remove useless psbox variable from resource_copy_region

2014-02-10 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/radeonsi/si_blit.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_blit.c b/src/gallium/drivers/radeonsi/si_blit.c index bf76ecf..8bfa6ed 100644 ---

Re: [Mesa-dev] [PATCH 01/10] radeonsi: move PA_SU_POLY_OFFSET_DB_FMT_CNTL to framebuffer state

2014-02-10 Thread Chris Forbes
FORMAT_X8Z24_UNORM: + case PIPE_FORMAT_Z24X8_UNORM: + case PIPE_FORMAT_Z24_UNORM_S8_UINT: + pa_su_poly_offset_db_fmt_cntl = S_028B78_POLY_OFFSET_NEG_NUM_DB_BITS(-24); + break; + case PIPE_FORMAT_Z32_FLOAT: + case

Re: [Mesa-dev] [PATCH 01/10] radeonsi: move PA_SU_POLY_OFFSET_DB_FMT_CNTL to framebuffer state

2014-02-10 Thread Chris Forbes
[Possibly ignore that, I guess it's the negated number of mantissa bits] On Tue, Feb 11, 2014 at 3:39 PM, Chris Forbes chr...@ijw.co.nz wrote: FORMAT_X8Z24_UNORM: + case PIPE_FORMAT_Z24X8_UNORM: + case PIPE_FORMAT_Z24_UNORM_S8_UINT: + pa_su_poly_offset_db_fmt_cntl =

Re: [Mesa-dev] [PATCH] R600/SI: Split global vector loads with more than 4 elements

2014-02-10 Thread Aaron Watry
Hi Tom, This definitely fixes some issues that I've been seeing with int8/16 vload8() and vload16() in CL. vstore8/vstore16 are still broken, but at least the loads are working now (I've only tested int, but I can give a full test run if you want/need). For reference, the tests that failed

Re: [Mesa-dev] [PATCH 3/3] st/mesa: add texture gather support. (v2)

2014-02-10 Thread Ilia Mirkin
Just one (probably) trivial comment below: On Mon, Feb 10, 2014 at 8:43 PM, Dave Airlie airl...@gmail.com wrote: From: Dave Airlie airl...@redhat.com This adds support for GL_ARB_texture_gather, and one step of support for GL_ARB_gpu_shader5. This adds support for passing the TG4

Re: [Mesa-dev] [PATCH 3/3] st/mesa: add texture gather support. (v2)

2014-02-10 Thread Dave Airlie
const GLuint *inputMapping; @@ -4379,22 +4428,34 @@ translate_src(struct st_translate *t, const st_src_reg *src_reg) static struct tgsi_texture_offset translate_tex_offset(struct st_translate *t, - const struct tgsi_texture_offset *in_offset) +

[Mesa-dev] [PATCH 01/13] targets/pipe-loader: link pipe-nouveau against libdrm

2014-02-10 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/targets/pipe-loader/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/targets/pipe-loader/Makefile.am b/src/gallium/targets/pipe-loader/Makefile.am index 97733c1..cb2eff7 100644 ---

[Mesa-dev] Convert vdpau and dri targets to the pipe-loader

2014-02-10 Thread Emil Velikov
Hello list, The recent patches from Rob gave me a nice kick to give another stab at integrating the pipe-loader into the vdpau/dri targets. What: - With these patches one library will be created for hardware and one for software driven backends - eg. libvdpau_gallium_dri, libvdpau_gallium_sw -

[Mesa-dev] [PATCH 05/13] pipe-loader: handle memory allocation failure

2014-02-10 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c | 3 +++ src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 1 + 2 files changed, 4 insertions(+) diff --git a/src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c

[Mesa-dev] [PATCH 04/13] pipe-loader: build pipe_loader_drm_x_auth whenever HAVE_PIPE_LOADER_XCB is defined

2014-02-10 Thread Emil Velikov
Currently HAVE_PIPE_LOADER_XCB is defined, rather than being set to 1/0. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

  1   2   >