[Mesa-dev] [PATCH] glsl: Dont call mark_whole_array_access() unless we are dealing with arrays

2013-12-17 Thread Timothy Arceri
Signed-off-by: Timothy Arceri --- src/glsl/ast_to_hir.cpp | 45 - 1 file changed, 24 insertions(+), 21 deletions(-) diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp index 3bc181e..a7aa4c7 100644 --- a/src/glsl/ast_to_hir.cpp +++ b/src/gls

Re: [Mesa-dev] [PATCH 3/8] i965: Add a new representation for Broadwell shader instructions.

2013-12-17 Thread Kenneth Graunke
On 12/10/2013 11:25 PM, Kenneth Graunke wrote: [snip] > +static inline void > +gen8_set_src1_3src_subreg_nr(struct gen8_instruction *inst, unsigned v) > +{ > + assert((v & ~0x7) == 0); > + > + gen8_instruction_set_bits(inst, 95, 94, v & 0x3f); > + gen8_instruction_set_bits(inst, 96, 96, v >>

Re: [Mesa-dev] [PATCH 5/8] i965: Add a new infrastructure for generating Broadwell shader assembly.

2013-12-17 Thread Kenneth Graunke
On 12/10/2013 11:25 PM, Kenneth Graunke wrote: > This replaces the brw_eu_emit.c layer for Broadwell. It will be > used by both the vector and scalar shader backends. > > v2: Port to use the C-based instruction representation. > > Signed-off-by: Kenneth Graunke > --- > src/mesa/drivers/dri/i96

Re: [Mesa-dev] [PATCH 1/2] mesa: Add gl_formats to cover all GLUser provided format/type combinations

2013-12-17 Thread Kenneth Graunke
On 12/17/2013 06:50 PM, Mark Mueller wrote: > > > > On Tue, Dec 17, 2013 at 12:19 PM, Mark Mueller > wrote: > > > > > On Sat, Nov 23, 2013 at 4:10 PM, Marek Olšák > wrote: > > On Sat, Nov 23, 2013 at 10:23 PM, Mark Mu

Re: [Mesa-dev] [PATCH 1/2] mesa: Add gl_formats to cover all GLUser provided format/type combinations

2013-12-17 Thread Mark Mueller
On Tue, Dec 17, 2013 at 12:19 PM, Mark Mueller wrote: > > > > On Sat, Nov 23, 2013 at 4:10 PM, Marek Olšák wrote: > >> On Sat, Nov 23, 2013 at 10:23 PM, Mark Mueller >> wrote: >> > >> > >> > >> > On Sat, Nov 23, 2013 at 2:26 AM, Marek Olšák wrote: >> >> >> > > [...] > > >> >> >> >> >MESA_F

Re: [Mesa-dev] [PATCH 1/9] st/mesa: always prefer pipe->clear over clear_with_quad

2013-12-17 Thread Marek Olšák
José, is it really worth adding a new cap? The only way to hit both pipe->clear and clear_with_quad for depth and stencil, respectively, is to have a partial stencil writemask. Marek On Fri, Dec 13, 2013 at 5:46 PM, Jose Fonseca wrote: > > So, if this provides a significant performance differenc

[Mesa-dev] [PATCH 1/2] st/dri: move fourcc->format conversion to a common place

2013-12-17 Thread Dave Airlie
From: Dave Airlie Before I cut-n-paste this a 3rd time lets consolidate it. Signed-off-by: Dave Airlie --- src/gallium/state_trackers/dri/drm/dri2.c | 81 +-- 1 file changed, 35 insertions(+), 46 deletions(-) diff --git a/src/gallium/state_trackers/dri/drm/dri2.c

[Mesa-dev] [PATCH 2/2] st/dri: add support for dma-buf importer (DRIimage v8)

2013-12-17 Thread Dave Airlie
From: Dave Airlie This is just a simple implementation that stores the extra values into the DRIimage struct and just uses the fd importer. I haven't looked into what is required to import YUV or deal with the extra parameters. Signed-off-by: Dave Airlie --- src/gallium/state_trackers/dri/com

[Mesa-dev] [PATCH] i965: Fix the region's pitch condition to use blitter

2013-12-17 Thread Anuj Phogat
intelEmitCopyBlit uses a signed 16-bit integer to represent buffer pitch, so it can only handle buffer pitches < 32k. This patch fixes assertion failure in depth_texture_mipmap.test in Khronos' OpenGL CTS. But, the test still fails due to GL_OUT_OF_MEMORY error in glTexImage2D(). Cc: mesa-sta...@

[Mesa-dev] [PATCH] mesa: minor simplification in _mesa_es3_error_check_format_and_type()

2013-12-17 Thread Brian Paul
The type_valid local was set to true and never changed. --- src/mesa/main/glformats.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/mesa/main/glformats.c b/src/mesa/main/glformats.c index 740faa8..1ab8b23 100644 --- a/src/mesa/main/glformats.c +++ b/src/mesa/main/gl

Re: [Mesa-dev] [PATCH] Rename overloads of _mesa_glsl_shader_target_name().

2013-12-17 Thread Brian Paul
On 12/17/2013 02:32 PM, Paul Berry wrote: Previously, _mesa_glsl_shader_target_name() had an overload for GLenum and an overload for the gl_shader_type enum, each of which behaved differently. However, since GLenum is a synonym for unsigned int, and unsigned ints are often used in place of gl_sh

[Mesa-dev] [PATCH 6/8] radeon/llvm: Free target data at end of optimization

2013-12-17 Thread Aaron Watry
Reviewed-by: Tom Stellard CC: "10.0" --- src/gallium/drivers/radeon/radeon_llvm_util.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/radeon/radeon_llvm_util.c b/src/gallium/drivers/radeon/radeon_llvm_util.c index cf6d21e..2ace91f 100644 --- a/src/gallium/drivers/radeo

[Mesa-dev] [PATCH 7/8] st/vdpau: Destroy context when initialization fails

2013-12-17 Thread Aaron Watry
Prevents a potential memory leak found when tracking down something else. Reviewed-by: Christian König Reviewed-by: Tom Stellard CC: "10.0" --- src/gallium/state_trackers/vdpau/device.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/vdpau/device.c b/src/galliu

[Mesa-dev] [PATCH 5/8] r600/compute: Use the correct FREE macro when deleting compute state

2013-12-17 Thread Aaron Watry
Reviewed-by: Tom Stellard CC: "10.0" --- src/gallium/drivers/r600/evergreen_compute.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/r600/evergreen_compute.c b/src/gallium/drivers/r600/evergreen_compute.c index caac599..251021c 100644 --- a/src/gallium/

[Mesa-dev] [PATCH 2/8] pipe_loader/sw: close dev->lib when initialization fails

2013-12-17 Thread Aaron Watry
Prevents a memory leak. Reviewed-by: Tom Stellard CC: "10.0" --- src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c b/src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c i

[Mesa-dev] [PATCH 3/8] radeon/compute: Stop leaking LLVMContexts in radeon_llvm_parse_bitcode

2013-12-17 Thread Aaron Watry
Previously we were creating a new LLVMContext every time that we called radeon_llvm_parse_bitcode, which caused us to leak the context every time that we compiled a CL program. Sadly, we can't dispose of the LLVMContext at the point that it was being created because evergreen_launch_grid (and poss

[Mesa-dev] [PATCH 1/8] clover: Remove unused variable

2013-12-17 Thread Aaron Watry
Reviewed-by: Tom Stellard CC: "10.0" --- src/gallium/state_trackers/clover/llvm/invocation.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp b/src/gallium/state_trackers/clover/llvm/invocation.cpp index 3f50317..e826669 100644 --- a/sr

[Mesa-dev] [PATCH 8/8] r600/pipe: Stop leaking context->start_compute_cs_cmd.buf on EG/CM

2013-12-17 Thread Aaron Watry
Found while tracking down memory leaks in VDPAU playback Reviewed-by: Tom Stellard CC: "10.0" --- src/gallium/drivers/r600/r600_pipe.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/r600/r600_pipe.c b/src/gallium/drivers/r600/r600_pipe.c index 4016bbe..74e007b 10064

[Mesa-dev] [PATCH 0/8 v2] Fix multiple memory leaks

2013-12-17 Thread Aaron Watry
Most of these fixes target radeon (both EG and SI), but some also help generic clover and also vdpau playback. v2: Remove an unnecessary null check in patch 4 of 8 CC: "10.0" Aaron Watry (8): clover: Remove unused variable pipe_loader/sw: close dev->lib when initialization fails. radeon/c

[Mesa-dev] [PATCH 4/8] r600/compute: Free compiled kernels when deleting compute state

2013-12-17 Thread Aaron Watry
v2: Remove unnecessary null pointer check CC: "10.0" --- src/gallium/drivers/r600/evergreen_compute.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/r600/evergreen_compute.c b/src/gallium/drivers/r600/evergreen_compute.c index f0f537c..caac599 100644 --- a/src/gallium

Re: [Mesa-dev] [PATCH 04/23] i965: Move up duplicated fields from stage-specific prog_data to brw_stage_prog_data.

2013-12-17 Thread Paul Berry
On 2 December 2013 11:31, Francisco Jerez wrote: > diff --git a/src/mesa/drivers/dri/i965/brw_program.c > b/src/mesa/drivers/dri/i965/brw_program.c > index 9a517be..a494bc2 100644 > --- a/src/mesa/drivers/dri/i965/brw_program.c > +++ b/src/mesa/drivers/dri/i965/brw_program.c > @@ -525,3 +525,31 @

[Mesa-dev] [PATCH 5/5] mesa: Add an assertion to _mesa_program_index_to_target().

2013-12-17 Thread Paul Berry
Only a Mesa bug could cause this function to be called with an out-of-range index, so raise an assertion if that ever happens. --- src/mesa/program/program.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/mesa/program/program.h b/src/mesa/program/program.h index 13527

[Mesa-dev] [PATCH 2/5] glsl: Get rid of hardcoded arrays of shader target names.

2013-12-17 Thread Paul Berry
We already have a function for converting a shader type index to a string: _mesa_glsl_shader_target_name(). --- src/glsl/link_atomics.cpp | 8 +++- src/glsl/linker.cpp | 16 ++-- 2 files changed, 9 insertions(+), 15 deletions(-) diff --git a/src/glsl/link_atomics.cpp b/src/

Re: [Mesa-dev] [PATCH 02/23] i965/fs: Allocate the param_size array dynamically.

2013-12-17 Thread Paul Berry
On 2 December 2013 11:31, Francisco Jerez wrote: > diff --git a/src/mesa/drivers/dri/i965/brw_wm.c > b/src/mesa/drivers/dri/i965/brw_wm.c > index bc1480c..b745d8f 100644 > --- a/src/mesa/drivers/dri/i965/brw_wm.c > +++ b/src/mesa/drivers/dri/i965/brw_wm.c > @@ -165,6 +165,7 @@ bool do_wm_prog(str

Re: [Mesa-dev] [PATCH] i965: Store QPitch in intel_mipmap_tree.

2013-12-17 Thread Paul Berry
On 17 December 2013 13:05, Kenneth Graunke wrote: > Broadwell allows us to specify an arbitrary value for QPitch, rather > than baking a specific formula into the hardware and requiring software > to lay things out to match. The only restriction is that the software > provided QPitch needs to be

[Mesa-dev] [PATCH] Rename overloads of _mesa_glsl_shader_target_name().

2013-12-17 Thread Paul Berry
Previously, _mesa_glsl_shader_target_name() had an overload for GLenum and an overload for the gl_shader_type enum, each of which behaved differently. However, since GLenum is a synonym for unsigned int, and unsigned ints are often used in place of gl_shader_type (e.g. in loop indices), there was

Re: [Mesa-dev] Testing optimizer

2013-12-17 Thread Patrick Baggett
On Tue, Dec 17, 2013 at 10:59 AM, Paul Berry wrote: > On 17 December 2013 08:46, Tom Stellard wrote: > >> On Tue, Dec 17, 2013 at 09:57:31AM -0600, Patrick Baggett wrote: >> > Hi all, >> > >> > Is there a way to see the machine code that is generated by the GLSL >> > compiler for all GPU instruct

Re: [Mesa-dev] [PATCH 1/5] main: Remove unused function _mesa_shader_index_to_type().

2013-12-17 Thread Brian Paul
On 12/17/2013 11:23 AM, Paul Berry wrote: --- src/mesa/main/shaderobj.h | 15 --- 1 file changed, 15 deletions(-) diff --git a/src/mesa/main/shaderobj.h b/src/mesa/main/shaderobj.h index de1c9fc..7245c5e 100644 --- a/src/mesa/main/shaderobj.h +++ b/src/mesa/main/shaderobj.h @@ -11

[Mesa-dev] [PATCH] i965: Store QPitch in intel_mipmap_tree.

2013-12-17 Thread Kenneth Graunke
Broadwell allows us to specify an arbitrary value for QPitch, rather than baking a specific formula into the hardware and requiring software to lay things out to match. The only restriction is that the software provided QPitch needs to be large enough so successive array slices do not overlap. In

Re: [Mesa-dev] [PATCH 2/5] glsl: Get rid of hardcoded arrays of shader target names.

2013-12-17 Thread Brian Paul
On 12/17/2013 11:23 AM, Paul Berry wrote: We already have a function for converting a shader type index to a string: _mesa_glsl_shader_target_name(). --- src/glsl/link_atomics.cpp | 8 +++- src/glsl/linker.cpp | 16 ++-- 2 files changed, 9 insertions(+), 15 deletions(-)

Re: [Mesa-dev] [PATCH 1/2] mesa: Add gl_formats to cover all GLUser provided format/type combinations

2013-12-17 Thread Mark Mueller
On Sat, Nov 23, 2013 at 4:10 PM, Marek Olšák wrote: > On Sat, Nov 23, 2013 at 10:23 PM, Mark Mueller > wrote: > > > > > > > > On Sat, Nov 23, 2013 at 2:26 AM, Marek Olšák wrote: > >> > [...] > >> > >> >MESA_FORMAT_RGBA1555_REV, > >> > >> I don't think you can have R with 1 bit. > >> > >

Re: [Mesa-dev] [PATCH 2/5] glsl: Get rid of hardcoded arrays of shader target names.

2013-12-17 Thread Brian Paul
On 12/17/2013 12:28 PM, Paul Berry wrote: On 17 December 2013 10:51, Brian Paul mailto:bri...@vmware.com>> wrote: On 12/17/2013 11:23 AM, Paul Berry wrote: We already have a function for converting a shader type index to a string: _mesa_glsl_shader_target_name(__). -

Re: [Mesa-dev] [PATCH 2/5] glsl: Get rid of hardcoded arrays of shader target names.

2013-12-17 Thread Paul Berry
On 17 December 2013 10:51, Brian Paul wrote: > On 12/17/2013 11:23 AM, Paul Berry wrote: > >> We already have a function for converting a shader type index to a >> string: _mesa_glsl_shader_target_name(). >> --- >> src/glsl/link_atomics.cpp | 8 +++- >> src/glsl/linker.cpp | 16

Re: [Mesa-dev] [PATCH 2/5] glsl: Get rid of hardcoded arrays of shader target names.

2013-12-17 Thread Kenneth Graunke
On 12/17/2013 11:50 AM, Brian Paul wrote: > On 12/17/2013 12:28 PM, Paul Berry wrote: [snip] >> How about this idea instead: rather than use function overloading to >> distinguish between the two meanings of _mesa_glsl_shader_target_name(), >> have two functions with separate names, e.g.: >> >> con

Re: [Mesa-dev] Testing optimizer

2013-12-17 Thread Paul Berry
On 17 December 2013 11:07, Patrick Baggett wrote: > > > > On Tue, Dec 17, 2013 at 10:59 AM, Paul Berry wrote: > >> On 17 December 2013 08:46, Tom Stellard wrote: >> >>> On Tue, Dec 17, 2013 at 09:57:31AM -0600, Patrick Baggett wrote: >>> > Hi all, >>> > >>> > Is there a way to see the machine cod

[Mesa-dev] [PATCH 4/5] mesa: Improve static error checking of arrays sized by MESA_SHADER_TYPES.

2013-12-17 Thread Paul Berry
This patch replaces the following pattern: foo bar[MESA_SHADER_TYPES] = { ... }; With: foo bar[] = { ... }; STATIC_ASSERT(Elements(bar) == MESA_SHADER_TYPES); This way, when a new shader type is added in a future version of Mesa, we will get a compile error to

[Mesa-dev] [PATCH 1/5] main: Remove unused function _mesa_shader_index_to_type().

2013-12-17 Thread Paul Berry
--- src/mesa/main/shaderobj.h | 15 --- 1 file changed, 15 deletions(-) diff --git a/src/mesa/main/shaderobj.h b/src/mesa/main/shaderobj.h index de1c9fc..7245c5e 100644 --- a/src/mesa/main/shaderobj.h +++ b/src/mesa/main/shaderobj.h @@ -118,21 +118,6 @@ _mesa_shader_type_to_index(GLen

[Mesa-dev] [PATCH 3/5] glsl: Remove extraneous shader_type argument from analyze_clip_usage().

2013-12-17 Thread Paul Berry
This argument was carrying the name of the shader target (as a string). We can get this just as easily by calling _mesa_glsl_shader_target_name(). --- src/glsl/linker.cpp | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp index 1

Re: [Mesa-dev] [PATCH 0/2] code de-duplication and non-pci support

2013-12-17 Thread Erik Faye-Lund
On Sat, Dec 14, 2013 at 8:28 PM, Rob Clark wrote: > From: Rob Clark > > It seems that over time, code related to finding driver name, dealing > with pci-id table, etc, has been copy/pasted everywhere it was needed. > Which is lame. And annoying if you have a device which is not pci. > > This pat

Re: [Mesa-dev] Testing optimizer

2013-12-17 Thread Marek Olšák
ST_DEBUG=tgsi ... is also useful. It dumps all shaders generated by mesa_to_tgsi and glsl_to_tgsi and it's easier to read than the GLSL IR. Marek On Tue, Dec 17, 2013 at 5:46 PM, Tom Stellard wrote: > On Tue, Dec 17, 2013 at 09:57:31AM -0600, Patrick Baggett wrote: >> Hi all, >> >> Is there a wa

Re: [Mesa-dev] [PATCH] radeonsi: set CB_DISABLE if the color mask is 0

2013-12-17 Thread Marek Olšák
I don't think so. Marek On Tue, Dec 17, 2013 at 5:10 PM, Sylvain BERTRAND wrote: >>> From: Marek Olšák >>> >>> Also needed for the DB in-place decompression according to hw docs. > > Published hw docs? > > regards, > > -- > Sylvain ___ mesa-dev mailin

Re: [Mesa-dev] Testing optimizer

2013-12-17 Thread Paul Berry
On 17 December 2013 08:46, Tom Stellard wrote: > On Tue, Dec 17, 2013 at 09:57:31AM -0600, Patrick Baggett wrote: > > Hi all, > > > > Is there a way to see the machine code that is generated by the GLSL > > compiler for all GPU instruction sets? For example, I would like to know > if > > the opti

Re: [Mesa-dev] Testing optimizer

2013-12-17 Thread Tom Stellard
On Tue, Dec 17, 2013 at 09:57:31AM -0600, Patrick Baggett wrote: > Hi all, > > Is there a way to see the machine code that is generated by the GLSL > compiler for all GPU instruction sets? For example, I would like to know if > the optimizer optimizes certain (equivalent) constructs (or not), and

Re: [Mesa-dev] [PATCH v3 19/23] glsl: Add image built-in function generator.

2013-12-17 Thread Paul Berry
On 17 December 2013 02:10, Francisco Jerez wrote: > Because of the combinatorial explosion of different image built-ins > with different image dimensionalities and base data types, enumerating > all the 242 possibilities would be annoying and a waste of .text > space. Instead use a special path

Re: [Mesa-dev] [PATCH] radeonsi: set CB_DISABLE if the color mask is 0

2013-12-17 Thread Sylvain BERTRAND
>> From: Marek Olšák >> >> Also needed for the DB in-place decompression according to hw docs. Published hw docs? regards, -- Sylvain ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH v2 1/1] clover: Add parameter checks to clCreateBuffer.

2013-12-17 Thread Jan Vesely
v2: use fewer if statements and functional tricks instead of single-use method, suggested by Francisco Jerez squash two small patches into one Signed-off-by: Jan Vesely --- Hi, this is v2 of the first two patches incorporating Francisco's comments, i.e. it's the code from his comments :)

Re: [Mesa-dev] [PATCH] radeonsi: set CB_DISABLE if the color mask is 0

2013-12-17 Thread Sylvain BERTRAND
> On Tue, Dec 17, 2013 at 7:54 AM, Marek Olšák wrote: >> From: Marek Olšák >> >> Also needed for the DB in-place decompression according to hw docs. Published hw docs? regards, -- Sylvain ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http

[Mesa-dev] Testing optimizer

2013-12-17 Thread Patrick Baggett
Hi all, Is there a way to see the machine code that is generated by the GLSL compiler for all GPU instruction sets? For example, I would like to know if the optimizer optimizes certain (equivalent) constructs (or not), and avoid them if possible. I know there is a lot to optimization on GPUs that

[Mesa-dev] [PATCH v2] glcpp: error on multiple #else/#elif directives

2013-12-17 Thread Erik Faye-Lund
The preprocessor currently accepts multiple else/elif-groups per if-section. The GLSL-preprocessor is defined by the C++ specification, which defines the following parse-rule: if-section: if-group elif-groups(opt) else-group(opt) endif-line This clearly only allows a single else-group, th

[Mesa-dev] [PATCH] mesa: Fix build to properly check for supported compiler flags

2013-12-17 Thread Lauri Kasanen
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=72708 It seems to me that the Intel code that uses this SSE4.1 function is still buggy, as it has no runtime check - would it not crash if built on a SSE4-capable system but used with a lower-class cpu? i965 systems = LGA775 = Pentium4 without S

Re: [Mesa-dev] [PATCH] radeonsi: set CB_DISABLE if the color mask is 0

2013-12-17 Thread Alex Deucher
On Tue, Dec 17, 2013 at 7:54 AM, Marek Olšák wrote: > From: Marek Olšák > > Also needed for the DB in-place decompression according to hw docs. Makes sense. Reviewed-by: Alex Deucher > --- > src/gallium/drivers/radeonsi/si_state.c | 11 --- > 1 file changed, 8 insertions(+), 3 deleti

Re: [Mesa-dev] [PATCH] radeonsi: add the htile buffer to the CS ioctl buffer list

2013-12-17 Thread Alex Deucher
On Tue, Dec 17, 2013 at 6:49 AM, Marek Olšák wrote: > From: Marek Olšák > > This may fix the GPU crashes. Reviewed-by: Alex Deucher > --- > src/gallium/drivers/radeonsi/si_state.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/gallium/drivers/radeonsi/si_state.c > b/src/gall

Re: [Mesa-dev] [PATCH] glcpp: error on multiple #else directives

2013-12-17 Thread Erik Faye-Lund
On Fri, Oct 11, 2013 at 10:48 PM, Ian Romanick wrote: > Carl, > > Can you look at this patch and Erik's follow-up patch? You (still) know > the glcpp much better than any of the rest of us. > > (Carl is currently out of town, so I know his response will be slow...) > I guess "slow" doesn't mean

[Mesa-dev] [PATCH] st/dri: Check for kernel support before enabling fd sharing

2013-12-17 Thread Thomas Hellstrom
The dri2 state tracker is checking for driver support before enabling dri2ImageExtension version 7. This commit adds a check that also the kernel driver supports fd sharing through prime. Note that this adds a libdrm dependency on dri2.c. Signed-off-by: Thomas Hellstrom --- src/gallium/state_tr

[Mesa-dev] [PATCH] radeonsi: set CB_DISABLE if the color mask is 0

2013-12-17 Thread Marek Olšák
From: Marek Olšák Also needed for the DB in-place decompression according to hw docs. --- src/gallium/drivers/radeonsi/si_state.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c ind

[Mesa-dev] [PATCH] radeonsi: add the htile buffer to the CS ioctl buffer list

2013-12-17 Thread Marek Olšák
From: Marek Olšák This may fix the GPU crashes. --- src/gallium/drivers/radeonsi/si_state.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c index c1107c6..5c18538 100644 --- a/src/gallium/drivers/radeonsi/si

Re: [Mesa-dev] [PATCH 1/3] radeonsi: flush HTILE when appropriate

2013-12-17 Thread Marek Olšák
It doesn't fix the corruption in Sanctuary. I haven't been able to reproduce the GPU crashes though (I don't have Serious Sam 3 or Skyrim). I agree it should be disabled by default if we don't find a fix quickly. Marek On Tue, Dec 17, 2013 at 9:44 AM, Michel Dänzer wrote: > On Die, 2013-12-17 at

[Mesa-dev] [PATCH v3 19/23] glsl: Add image built-in function generator.

2013-12-17 Thread Francisco Jerez
Because of the combinatorial explosion of different image built-ins with different image dimensionalities and base data types, enumerating all the 242 possibilities would be annoying and a waste of .text space. Instead use a special path in the built-in builder that loops over all the known image

Re: [Mesa-dev] [PATCH 1/3] radeonsi: flush HTILE when appropriate

2013-12-17 Thread Michel Dänzer
On Die, 2013-12-17 at 03:31 +0100, Marek Olšák wrote: > From: Marek Olšák This series is Reviewed-by: Michel Dänzer Does this address bugs such as https://bugs.freedesktop.org/show_bug.cgi?id=72685 and https://bugzilla.kernel.org/show_bug.cgi?id=66981 ? If not, HyperZ should probably be disab

Re: [Mesa-dev] [PATCH 0/2] v3: Fix array overrun with too many uniforms

2013-12-17 Thread Petri Latvala
On 11/27/2013 03:28 PM, Petri Latvala wrote: Third version of this patch series sent in full Ping? Comments, NAKs, ACKs...? -- Petri Latvala ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mes