Re: [Mesa-dev] [PATCH 3/3] i965/fs: Compute q-values for register allocation manually

2014-10-03 Thread Connor Abbott
On Fri, Oct 3, 2014 at 10:03 PM, Jason Ekstrand wrote: > Previously, we were allowing the register allocation code to do the > computation for us in ra_set_finalize. However, the runtime for this > computation is O(c^4 * g) where c is the number of classes and g is the > number of GRF registers.

[Mesa-dev] [PATCH 2/3] i965/fs: Add another use of MAX_VGRF_SIZE

2014-10-03 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp b/src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp index 32669f6..fd34941 100644 --- a/src/me

[Mesa-dev] [PATCH 3/3] i965/fs: Compute q-values for register allocation manually

2014-10-03 Thread Jason Ekstrand
Previously, we were allowing the register allocation code to do the computation for us in ra_set_finalize. However, the runtime for this computation is O(c^4 * g) where c is the number of classes and g is the number of GRF registers. However, these q-values are directly computable based on the wa

[Mesa-dev] [PATCH 1/3] util: Use reg_belongs_to_class instead of BITSET_TEST

2014-10-03 Thread Jason Ekstrand
This shouldn't be a functional change since reg_belongs_to_class is just a wrapper around BITSET_TEST. It just makes the code a little easier to read. Signed-off-by: Jason Ekstrand --- src/util/register_allocate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/regi

Re: [Mesa-dev] [PATCH 2/3] i965/blit: Add a color blit function and use it in set_alpha_to_one

2014-10-03 Thread Matt Turner
On Wed, Oct 1, 2014 at 2:07 PM, Jason Ekstrand wrote: > Signed-off-by: Jason Ekstrand > --- > src/mesa/drivers/dri/i965/intel_blit.c | 90 > -- > src/mesa/drivers/dri/i965/intel_blit.h | 10 > 2 files changed, 73 insertions(+), 27 deletions(-) > > diff --git

Re: [Mesa-dev] [PATCH 1/2] i965/compaction: Simplify jump target code.

2014-10-03 Thread Jordan Justen
On Sat, Sep 27, 2014 at 12:57 PM, Matt Turner wrote: > My attempts to clarify the code with _compacted/_uncompacted prefixed > variables apparently failed. Hopefully this is clearer. Heh. Maybe the 'shift' variables could use a better names? (compact_shift?) Regardless, series Reviewed-by: Jorda

[Mesa-dev] [Bug 84570] Borderlands 2: Constant frame rate drops while playing; really bad with additionl lighting

2014-10-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84570 Kai changed: What|Removed |Added CC||ibull...@aspyr.com Summary|Borderlands

[Mesa-dev] [Bug 84570] Borderlands 2: Framerate drops dramatically with dynamic lighting activated

2014-10-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84570 --- Comment #1 from Ian C. Bullard --- A little more information (I'm a developer at Aspyr Media): DynamicLights' name is misleading, when set to false *all* lighting except for ambient light is turned off. That's why everything looks so flat w

Re: [Mesa-dev] [PATCH] mesa: fix spurious wglGetProcAddress / GL_INVALID_OPERATION error

2014-10-03 Thread Sinclair Yeh
This looks reasonable to me. Acked-by: Sinclair Yeh On Thu, Oct 02, 2014 at 09:49:04AM -0600, Brian Paul wrote: > On Windows, the Piglit primitive-restart test was failing a > glGetError()==0 assertion when it was run w/out any command line > arguments. Piglit's all.py script only runs primiti

[Mesa-dev] [Bug 84242] FTBFS: libOpenCL.so.1.0.0: ld: .eh_frame_hdr table[5707] FDE at 0000000000c45b8c overlaps table[5708] FDE at 0000000000c45a88

2014-10-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84242 --- Comment #16 from Alan Modra --- It looks like the overlapping FDE error is caused by a bunch of zero address range FDEs in /usr/lib/llvm-3.5/lib/libclangCodeGen.a(CGStmtOpenMP.o). These do have the potential to cause an exception handling pr

Re: [Mesa-dev] [PATCH] glsl/glsl_parser_extras: Handle GLSL 4.50

2014-10-03 Thread Olivier Galibert
Sorry for not replying earlier, I didn't see your answer. On Thu, Sep 4, 2014 at 12:33 AM, Matt Turner wrote: > Did you change the leading whitespace on purpose? Not really, I can un-change that. I have an emacs config that's supposedly what mesa wants, but it may be incorrect. >> - } suppo

Re: [Mesa-dev] [PATCH v3] radeonsi: Add CIK SDMA support

2014-10-03 Thread Marek Olšák
Looks great. Thanks. I haven't read SDMA docs though, so I can't tell if the SDMA patckets are correct. Reviewed-by: Marek Olšák Marek On Fri, Oct 3, 2014 at 11:55 AM, Michel Dänzer wrote: > From: Michel Dänzer > > Based on the corresponding SI support. > > Signed-off-by: Michel Dänzer > --

Re: [Mesa-dev] [PATCH] glsl: Fix memory leak in builtin_builder::_image_prototype.

2014-10-03 Thread Francisco Jerez
Kenneth Graunke writes: > On Thursday, October 02, 2014 11:28:35 AM Ian Romanick wrote: >> Since this is Curro's code, I'm CC'ing me. >> >> On 10/01/2014 03:12 AM, Iago Toral Quiroga wrote: >> > in_var calls the ir_variable constructor, which dups the variable name. >> > --- >> > src/glsl/built

Re: [Mesa-dev] [PATCH v2] radeonsi: Add CIK SDMA support

2014-10-03 Thread Michel Dänzer
On 02.10.2014 19:14, Marek Olšák wrote: 1) We should have our own copyright in the new file. Other than that: Reviewed-by: Marek Olšák The following things can be done in a separate patch. 2) I recommend removing the CMASK and FMASK checking and replacing it with nr_samples > 1 (for SI too).

[Mesa-dev] [PATCH v3] radeonsi: Add CIK SDMA support

2014-10-03 Thread Michel Dänzer
From: Michel Dänzer Based on the corresponding SI support. Signed-off-by: Michel Dänzer --- v3: * Add AMD copyright header * Add 009910_MICRO_TILE_MODE_NEW register field definitions * Call r600_need_dma_space before r600_context_bo_reloc in cik_sdma_do_copy_buffer * Drop x and z parameters t

Re: [Mesa-dev] [PATCH] glsl: Fix memory leak in builtin_builder::_image_prototype.

2014-10-03 Thread Kenneth Graunke
On Thursday, October 02, 2014 11:28:35 AM Ian Romanick wrote: > Since this is Curro's code, I'm CC'ing me. > > On 10/01/2014 03:12 AM, Iago Toral Quiroga wrote: > > in_var calls the ir_variable constructor, which dups the variable name. > > --- > > src/glsl/builtin_functions.cpp | 8 +--- > >

Re: [Mesa-dev] [PATCH] glsl: Fix memory leak in builtin_builder::_image_prototype.

2014-10-03 Thread Francisco Jerez
Ian Romanick writes: > Since this is Curro's code, I'm CC'ing me. > > On 10/01/2014 03:12 AM, Iago Toral Quiroga wrote: >> in_var calls the ir_variable constructor, which dups the variable name. >> --- >> src/glsl/builtin_functions.cpp | 8 +--- >> 1 file changed, 5 insertions(+), 3 deletion

Re: [Mesa-dev] [PATCH] st/mesa: use pipe_sampler_view_release for releasing sampler views

2014-10-03 Thread Christian König
Am 02.10.2014 um 22:37 schrieb Alex Deucher: On Thu, Oct 2, 2014 at 1:51 PM, Christian König wrote: Am 02.10.2014 um 19:34 schrieb Marek Olšák: From: Marek Olšák This fixes a crash when exiting Firefox. I have really no idea how Firefox does it. It seems to involve multiple contexts and mult

Re: [Mesa-dev] [PATCH] Revert "configure: ask vdpau.pc for the default location of the vdpau drivers"

2014-10-03 Thread Christian König
Am 03.10.2014 um 03:53 schrieb Ilia Mirkin: On Thu, Oct 2, 2014 at 7:59 PM, Emil Velikov wrote: On 02/10/14 06:41, Ilia Mirkin wrote: On Mon, Sep 29, 2014 at 8:33 PM, Emil Velikov wrote: On 29/09/14 17:24, Matt Turner wrote: On Mon, Sep 29, 2014 at 9:16 AM, Emil Velikov wrote: So all in a