Re: [Mesa-dev] [PATCH 6/8] i965: Implement HiZ resolves on Broadwell.

2014-02-10 Thread Chris Forbes
Since we don't have to do nonzero miplevels for multisampling, can't you just use mt->logical_width0 and mt->logical_height0 if multisampling? On Tue, Feb 11, 2014 at 6:48 PM, Kenneth Graunke wrote: > On 02/07/2014 10:43 PM, Kenneth Graunke wrote: >> Broadwell's 3DSTATE_WM_HZ_OP packet makes this

Re: [Mesa-dev] [PATCH 6/8] i965: Implement HiZ resolves on Broadwell.

2014-02-10 Thread Kenneth Graunke
On 02/07/2014 10:43 PM, Kenneth Graunke wrote: > Broadwell's 3DSTATE_WM_HZ_OP packet makes this much easier. > > Instead of programming the whole pipeline, we simply have to emit the > depth/stencil packets, a state override, and a pipe control. Then > arrange for the state to be put back. This

[Mesa-dev] [PATCH] nv30,nvc0: only claim a single viewport

2014-02-10 Thread Ilia Mirkin
It should be possible to make this be 16 on nvc0. Signed-off-by: Ilia Mirkin --- Not touching nv50 since I have a patch that actually impelents support for multiple viewports there. src/gallium/drivers/nouveau/nv30/nv30_screen.c | 2 ++ src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 ++ 2

[Mesa-dev] [PATCH v2] mesa/st: hardcode the viewport bounds range

2014-02-10 Thread Ilia Mirkin
The bound range is disconnected from the viewport dimensions. This is the relevant bit from glViewportArray: """ The location of the viewport's bottom left corner, given by (x, y) is clamped to be within the implementaiton-dependent viewport bounds range. The viewport bounds range [min, max] can b

[Mesa-dev] [PATCH] mesa/st: use a new cap to determine the viewport bounds range

2014-02-10 Thread Ilia Mirkin
The bound range is disconnected from the viewport dimensions. This is the relevant bit from glViewportArray: """ The location of the viewport's bottom left corner, given by (x, y) is clamped to be within the implementaiton-dependent viewport bounds range. The viewport bounds range [min, max] can b

[Mesa-dev] [PATCH 12/13] targets/vdpau: introduce standalone vdpau library/backend

2014-02-10 Thread Emil Velikov
Only one library for all hardware and one for software targets is created. Drivers interact with the library using the pipe-loader scheme. Currently software vdpau library is created for testing and POC reasons. Note that it's in the same buggy state as to prior it's removal a while back. Signed-

[Mesa-dev] [PATCH 09/13] pipe-loader: introduce pipe_loader_sw_probe_dri helper

2014-02-10 Thread Emil Velikov
Will be used in the following commits. v2: Link gallium tests against the library. Signed-off-by: Emil Velikov --- src/gallium/auxiliary/pipe-loader/pipe_loader.h| 13 + src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 19 +++ src/gallium/tests/trivial/Makefi

[Mesa-dev] [PATCH 10/13] pipe-loader: introduce pipe_loader_sw_probe_null helper function

2014-02-10 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/auxiliary/pipe-loader/pipe_loader.h| 10 ++ src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 18 +- 2 files changed, 27 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/pipe-loader/pipe_loader.h b/src/gal

[Mesa-dev] [PATCH 11/13] targets/pipe-loader: sync up targets with their respective dri* equivalent

2014-02-10 Thread Emil Velikov
Currently only nouveau, r300, r600 and radeonsi are being targeted. Initial step towards restructuring the vdpau backends to be driven by the pipe-loader. Signed-off-by: Emil Velikov --- src/gallium/targets/pipe-loader/pipe_nouveau.c | 24 +-- src/gallium/targets/pipe-loader/pipe_r300.c

[Mesa-dev] [PATCH 03/13] pipe-loader: destroy sw_winsys on sw_release

2014-02-10 Thread Emil Velikov
The sw pipe-loader implicitly handles winsys_create, thus we it would make sense to implicitly destroy it upon releasing the loader. Currently we leak the sw_winsys when releasing the pipe-loader. Signed-off-by: Emil Velikov --- src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 3 +++ 1 file

[Mesa-dev] [PATCH 02/13] vl/winsys_dri: cleanup vl_screen_create error path

2014-02-10 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/auxiliary/vl/vl_winsys_dri.c | 32 +++- 1 file changed, 19 insertions(+), 13 deletions(-) diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri.c b/src/gallium/auxiliary/vl/vl_winsys_dri.c index fb4aaad..5d83e57 100644 --- a/

[Mesa-dev] [PATCH 08/13] pipe-loader: introduce pipe_loader_sw_probe_xlib helper

2014-02-10 Thread Emil Velikov
Will be used in the upcoming patches. Signed-off-by: Emil Velikov --- src/gallium/auxiliary/pipe-loader/Makefile.am | 4 src/gallium/auxiliary/pipe-loader/pipe_loader.h| 19 +++ src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 22 +++--- 3 files

[Mesa-dev] [PATCH 07/13] pipe-loader: use bool type for pipe_loader_drm_probe_fd()

2014-02-10 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/auxiliary/pipe-loader/pipe_loader.h | 2 +- src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c | 8 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/gallium/auxiliary/pipe-loader/pipe_loader.h b/src/gallium/auxiliary/pip

[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 --- 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 b/src/gallium/auxiliary/pipe-loader/pipe_load

[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 --- src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c b/src/gallium/auxi

[Mesa-dev] [PATCH 06/13] winsys/xlib: move xlib_create_sw_winsys within the winsys

2014-02-10 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 2 +- src/gallium/auxiliary/vl/vl_winsys_xsp.c | 4 ++- src/gallium/include/state_tracker/xlib_sw_winsys.h | 29 -- src/gallium/include/state_tracker/xlibsw_api.h | 19

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

2014-02-10 Thread Emil Velikov
Signed-off-by: Emil Velikov --- 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 --- a/src/gallium/targets/pipe-loader/Makefile.a

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) >> +

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 wrote: > From: Dave Airlie > > 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 t

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 befor

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 wrote: > FORMAT_X8Z24_UNORM: >> + case PIPE_FORMAT_Z24X8_UNORM: >> + case PIPE_FORMAT_Z24_UNORM_S8_UINT: >> + pa_su_poly_offset_db_fmt_cntl = >> S_028B

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 PIPE_FORMAT_Z32_FLO

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

2014-02-10 Thread Marek Olšák
From: Marek Olšák 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 0154e58..89b4242 100644 --

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

2014-02-10 Thread Marek Olšák
From: Marek Olšák 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 +--- 1 file changed, 2

[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 --- 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 --- a/src/gallium/drivers/radeonsi/si_blit.c +++ b/

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

2014-02-10 Thread Marek Olšák
From: Marek Olšák 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(+), 10 deletions(-) di

[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 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 b/src/gallium/drivers/radeonsi/si_state.c index 5020547..f

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

2014-02-10 Thread Marek Olšák
From: Marek Olšák --- 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 100644 --- a/src/gallium

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

2014-02-10 Thread Marek Olšák
From: Marek Olšák --- 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 +- src/gallium/drivers/r600/r600_resource.h

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

2014-02-10 Thread Marek Olšák
From: Marek Olšák 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, 10 insertions(+), 1

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

2014-02-10 Thread Marek Olšák
From: Marek Olšák --- 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 ++- src/ga

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

2014-02-10 Thread Marek Olšák
From: Marek Olšák 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 --- src/gallium/drivers/radeons

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 wrote: > On 02/10/2014 02:04 PM, Ilia Mirkin wrote: >> On Mon, Feb 10, 2014 at 4:43 PM, Ian Romanick wrote: >>> On 02/08/2014 04:18 PM, Ilia Mirkin wrote: Module: Mesa Branch: master Commit: 356aff3a5c08be055d6befff99a72f5551b3ac2d

[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 fo

[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 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 --- src/glsl/Makefile.sources | 1 + src/glsl/ir_optimization.h |

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

2014-02-10 Thread Dave Airlie
From: Dave Airlie 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 native textureGatherOffs

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

2014-02-10 Thread Dave Airlie
From: Dave Airlie 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 textureGatherOffsets feature f

[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 --- Try to set this environment variable: R600_DEBUG=nohyperz -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev mailing list mesa-

[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

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 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 > > Seems like the easy fix. > > Reviewed-by: Ian Romanick > > 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 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 > --- > src/mesa/drivers/dri/i965/brw_fs_generat

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 Seems like the easy fix. Reviewed-by: Ian Romanick Candidate for 10.1? > --- > src/glsl/opt_vectorize.cpp | 14 ++ > 1 file ch

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 a/src/mesa/main/s

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 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

[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: Kenne

[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 --- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/

[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 97a57a4..44

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 wrote: On 02/08/2014 02:41 AM, Matt Turner wrote: On Fri, Feb 7, 2014 at 10:22 PM, Andrew Guertin wrote: Hi, I updated mesa and started getting some bad behavior in League of Legends (played through

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 --- lib/Target/R600/SIISelLowering.cpp | 8 +- test/CodeGen/R600/load.ll | 178 +++

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

2014-02-10 Thread Andrew Guertin
Tested-by: Andrew Guertin 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 --- src/glsl/opt_vectorize.cpp | 14 ++ 1 file changed, 14 insertions(+) diff --git a/src/glsl/opt_vectori

[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 changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[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 --- Also for oland, make sure your kernel has this patch: http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drivers/gpu/drm/radeon/atombios_crtc.c?id=227ae10f17a5f2fd1307b7e582b603ef7bbb7e97

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 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 >> >> Aut

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 > Date: Wed Jan 29 12:36:13 2014 -0500 >

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

2014-02-10 Thread Tom Stellard
From: Tom Stellard --- 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 b/lib/Target/R600/SIISelLowering.cpp index 9537

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

2014-02-10 Thread Tom Stellard
From: Tom Stellard --- 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 b/lib/Target/R600/SIIns

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: On Feb 10, 2014, at 07:57, Ian Romanick 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 changed, 1 insertion(+) >> >>

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 wrote: > On 02/10/2014 12:11 AM, Matt Turner wrote: >> >> On Sat, Feb 8, 2014 at 5:29 AM, Andrew Guertin >> wrote: >>> >>> On 02/08/2014 02:41 AM, Matt Turner wrote: On Fri, Feb 7, 2014 at 10:22 PM, Andrew Guertin wrote: >

[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 --- 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 8ee81f1..dba303d 100644 --- a/src

[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 --- 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] [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 --- Created attachment 93798 --> https://bugs.freedesktop.org/attachment.cgi?id=93798&action=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 #5 from funkydude --- Created attachment 93797 --> https://bugs.freedesktop.org/attachment.cgi?id=93797&action=edit glxinfo -- You are receiving this mail because: You are the assignee for the bug.

[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 --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 10 ++ src/mesa/drivers/dri/i965/gen6_blorp.cpp | 18 -- src/

[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 --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 7 +++ src/mesa/drivers/dri/i965/gen6_blorp.cpp | 4 ++-- src/mesa/drivers/dri/i965/gen7

[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: Top

[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 --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 7 +++ src/mesa/drivers/dri/i965/gen6_blorp.cpp | 22 -- src/mesa/d

[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 t

[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 --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 42 +++-

[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 --- src/mesa/drivers/dri/i965/brw_blorp.cpp | 6 ++ src/mesa/drivers/dri/i965/gen6_blorp.cpp | 5 - src/mesa/drivers/dri/i965/gen7_blor

[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 --- 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 ++-- src/mesa/drivers/dri/i965/gen7_b

[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 --- 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 you start X. G

[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 --- 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 old game. My guess is it is

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

2014-02-10 Thread Rob Clark
From: Rob Clark 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 --- configure.ac | 14 +++-- src/gallium/aux

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

2014-02-10 Thread Rob Clark
From: Rob Clark 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 pipe-loader to get scr

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

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

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

2014-02-10 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- 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 src/gallium/targets/pipe-loader/pipe_msm.c diff --git a/

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

2014-02-10 Thread Rob Clark
From: Rob Clark 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 --- src/gallium/state_trackers/xa/xa_tracker.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/x

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

2014-02-10 Thread Tom Stellard
From: Tom Stellard --- 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 b01a443..7de7b1f 100644 --- a/lib

[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 --- 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. __

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 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 = "freedreno"; >

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 m

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

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

[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)

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 ... _

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 > >

[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 1/3] r300g/tests: Added missing fclose for FILE resource.

2014-02-10 Thread Tom Stellard
Reviewed-by: Tom Stellard 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 > b/src/gallium/driv

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/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 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 > ---

[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 --- Created attachment 93777 --> https://bugs.freedesktop.org/attachment.cgi?id=93777&action=edit Proposed patch egl_dri2: Allow both 24 and 32 bit X visuals for RGBA configs When using RGBA EGLConfigs allow

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 > > 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 > --- > configure.ac

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 > > 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 * s/HAVE_WINSYS_XLIB/NEED_WINSYS_XLIB

[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=93772&action=edit strace extract -- 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 Rob Clark
On Mon, Feb 10, 2014 at 7:37 AM, Emil Velikov wrote: > Hi Rob > > On 08/02/14 22:41, Rob Clark wrote: >> From: Rob Clark >> >> This lets multiple gallium drivers use XA. >> >> Signed-off-by: Rob Clark >> --- >> configure.ac | 8 ++-- >> src/gallium/state_tracke

[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 --- 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 this: /opt/ogl-sampl

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 > > Signed-off-by: Rob Clark > --- > src/gallium/targets/pipe-loader/Makefile.am | 16 > src/gallium/targets/pipe-loader/pipe_msm.c | 20 > 2 files changed, 36 insertions(+) > create mode 100644 src/ga

  1   2   >