[Mesa-dev] [PATCH] mesa: automake: replace $(RM) with rm -f

2015-07-18 Thread Jonathan Gray
$(RM) is set to 'rm -f' by GNU make, this is not true of other versions of make and RM is not one of the macros required by POSIX. Signed-off-by: Jonathan Gray --- Makefile.am | 2 +- src/gallium/targets/dri/Makefile.am | 6 +++--- src/gallium/targets/vdpau/Makefile.a

Re: [Mesa-dev] [PATCH] radeonsi: don't return NULL fence if no fence is available

2015-07-18 Thread Marek Olšák
It looks good. Would you push it please? Reviewed-by: Marek Olšák Marek On Fri, Jul 17, 2015 at 11:05 AM, Michel Dänzer wrote: > On 17.07.2015 06:03, Marek Olšák wrote: >> From: Marek Olšák >> >> An alternative (and ugly) solution to the current clover issue. > > How about something like this

[Mesa-dev] [PATCH 2/3] clover: override ret_object

2015-07-18 Thread EdB
Return an API object from an intrusive reference to a Clover object, incrementing the reference count of the object. Reviewed-by: Francisco Jerez --- src/gallium/state_trackers/clover/api/util.hpp | 12 1 file changed, 12 insertions(+) diff --git a/src/gallium/state_trackers/clover

[Mesa-dev] [PATCH 0/3 v4] add clLinkProgram

2015-07-18 Thread EdB
This serie add OpenCL 1.2 clLinkProgram. However, it lacks the binary type part that is mandatory for input validation and also for CL_PROGRAM_BINARY_TYPE query. This will be adressed in another serie once we agree on the way to store it. EdB (3): clover: separate compile and link stages clove

[Mesa-dev] [PATCH 3/3] clover: add clLinkProgramm (CL 1.2)

2015-07-18 Thread EdB
--- src/gallium/state_trackers/clover/api/program.cpp | 35 ++ src/gallium/state_trackers/clover/core/error.hpp | 7 + src/gallium/state_trackers/clover/core/program.cpp | 4 +++ src/gallium/state_trackers/clover/core/program.hpp | 1 + .../state_trackers/clover/llvm/

[Mesa-dev] [PATCH 1/3] clover: separate compile and link stages

2015-07-18 Thread EdB
--- src/gallium/state_trackers/clover/api/program.cpp | 19 +- .../state_trackers/clover/core/compiler.hpp| 7 +- src/gallium/state_trackers/clover/core/error.hpp | 7 + src/gallium/state_trackers/clover/core/program.cpp | 41 +++- src/gallium/state_trackers/clover/core/program.hp

[Mesa-dev] [Bug 91387] Mesa 10.6.1 implementation error: invalid target in _swrast_choose_texture_sample_func

2015-07-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91387 Bug ID: 91387 Summary: Mesa 10.6.1 implementation error: invalid target in _swrast_choose_texture_sample_func Product: Mesa Version: 10.6 Hardware: x86-64 (AMD64)

[Mesa-dev] [Bug 91385] gallium xvmc tries to symlink non existing libraries on OpenBSD

2015-07-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91385 Bug ID: 91385 Summary: gallium xvmc tries to symlink non existing libraries on OpenBSD Product: Mesa Version: git Hardware: Other OS: OpenBSD St

Re: [Mesa-dev] [PATCH] nouveau: use bool instead of boolean

2015-07-18 Thread samuel.pitoiset
On 17/07/2015 23:08, Ilia Mirkin wrote: On Fri, Jul 17, 2015 at 5:02 PM, Emil Velikov wrote: On 16/07/15 22:39, Samuel Pitoiset wrote: Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/codegen/nv50_ir.h | 2 +- .../drivers/nouveau/codegen/nv50_ir_driver.h | 14

[Mesa-dev] [PATCH 11/12] i965/fs: Reimplement emit_mcs_fetch() in terms of logical sends.

2015-07-18 Thread Francisco Jerez
--- src/mesa/drivers/dri/i965/brw_fs.h | 3 ++- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 36 ++-- 2 files changed, 15 insertions(+), 24 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.h b/src/mesa/drivers/dri/i965/brw_fs.h index 64f89d4..fed5d2

[Mesa-dev] [PATCH 10/12] i965/fs: Reimplement emit_texture() in terms of logical send messages.

2015-07-18 Thread Francisco Jerez
--- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 66 +--- 1 file changed, 49 insertions(+), 17 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp index 89fcc49..4011639 100644 --- a/src/mesa/drivers/dri/

[Mesa-dev] [PATCH 12/12] i965/fs: Remove the emit_texture_gen*() fs_visitor methods.

2015-07-18 Thread Francisco Jerez
This is now dead code. --- src/mesa/drivers/dri/i965/brw_fs.h | 21 - src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 608 --- 2 files changed, 629 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.h b/src/mesa/drivers/dri/i965/brw_fs.h index fed5d23..d

[Mesa-dev] [PATCH 01/12] i965/fs: Define logical texture sampling opcodes.

2015-07-18 Thread Francisco Jerez
Each logical variant is largely equivalent to the original opcode but instead of taking a single payload source it expects the arguments separately as individual sources, like: tex_logical dst, coordinates, shadow_c, lod, lod2, sample_index, mcs, sampler, offset,

[Mesa-dev] [PATCH 05/12] i965/fs: Implement lowering of logical texturing opcodes on Gen7+.

2015-07-18 Thread Francisco Jerez
This should be largely equivalent to emit_texture_gen7() except that we now get i965 sampling opcodes directly rather than ir_texture_opcode enum values. The mapping is as follows: - ir_tex -> SHADER_OPCODE_TEX - ir_txb -> FS_OPCODE_TXB - ir_txl -> SHADER_OPCODE_TXL - ir_txd -> SHADER_OPCODE_

[Mesa-dev] [PATCH 04/12] i965/fs: Pass a BAD_FILE header source to LOAD_PAYLOAD in emit_texture_gen7().

2015-07-18 Thread Francisco Jerez
So that it's left uninitialized by LOAD_PAYLOAD, we only need to reserve space for it in the message since it will be initialized implicitly by the generator. --- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/mesa/driver

[Mesa-dev] [PATCH 08/12] i965/fs: Implement lowering of logical texturing opcodes on Gen4.

2015-07-18 Thread Francisco Jerez
Unlike its Gen5 and Gen7 counterparts this patch isn't a plain refactor of the previous Gen4 texturing code, it's more of a rewrite largely based on emit_texture_gen4_simd16(). The reason is that on the one hand the original emit_texture_gen4() code didn't seem easily fixable to be SIMD width-inva

[Mesa-dev] [PATCH 09/12] i965/fs: Hook up SIMD lowering to handle texturing opcodes of unsupported width.

2015-07-18 Thread Francisco Jerez
This should match the set of cases in which we currently call fail() or no16() from the emit_texture_*() methods and the ones in which emit_texture_gen4() enables the SIMD16 workaround. Hint for reviewers: It's not a big deal if I happen to have missed some case here, it will just lead to an asser

[Mesa-dev] [PATCH 06/12] i965/fs: Lower SHADER_OPCODE_TXF_UMS/MCS_LOGICAL too on Gen7+.

2015-07-18 Thread Francisco Jerez
These weren't being handled by emit_texture_gen7() but we can easily lower them here for consistency with other texturing opcodes. --- src/mesa/drivers/dri/i965/brw_fs.cpp | 16 +++- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/

[Mesa-dev] [PATCH 07/12] i965/fs: Implement lowering of logical texturing opcodes on Gen5-6.

2015-07-18 Thread Francisco Jerez
This should be largely equivalent to emit_texture_gen5() except for slight codestyle changes and the use i965 opcodes instead of the ir_texture_opcode enum, see "i965/fs: Implement lowering of logical texturing opcodes on Gen7+." for the mapping between them. --- src/mesa/drivers/dri/i965/brw_fs.c

[Mesa-dev] [PATCH 02/12] i965/fs: Use exec_size instead of dispatch_width to determine the message variant.

2015-07-18 Thread Francisco Jerez
dispatch_width is global for a single compilation and doesn't necessarily match the desired execution width if we had to lower the original full-width instruction due to hardware limitations. These were all inside a Gen4-specific branch so this patch shouldn't have any effect on more recent hardwa

[Mesa-dev] [PATCH 03/12] i965/fs: Fix opt_zero_samples() for texturing ops not matching dispatch_width.

2015-07-18 Thread Francisco Jerez
--- src/mesa/drivers/dri/i965/brw_fs.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index 6afb9fe..c31a0e1 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.cpp +++ b/src/mesa/drivers/dri/i965/b

Re: [Mesa-dev] [PATCH] radeonsi: don't return NULL fence if no fence is available

2015-07-18 Thread Francisco Jerez
Michel Dänzer writes: > On 17.07.2015 06:03, Marek Olšák wrote: >> From: Marek Olšák >> >> An alternative (and ugly) solution to the current clover issue. > > How about something like this instead? (Compile tested only) > I'm rather unfamiliar with the radeonsi pipe driver code so I should pro

[Mesa-dev] [PATCH excerpt] mesa: Rename _mesa_lookup_enum_by_nr() to _mesa_enum_to_string().

2015-07-18 Thread Kenneth Graunke
Generated by sed; no manual changes. Signed-off-by: Kenneth Graunke --- We talked about doing this back in 2013, but the patches never quite materialized. Here's the obvious sed job. Actual patch is here: http://cgit.freedesktop.org/~kwg/mesa/commit/?h=enumtostring I figured I'd spare the ma

Re: [Mesa-dev] [Mesa-stable] [PATCH 16/18] i965: Prevent coordinate overflow in intel_emit_linear_blit

2015-07-18 Thread Chris Wilson
On Fri, Jul 17, 2015 at 05:12:54PM -0700, Anuj Phogat wrote: > On Mon, Jul 6, 2015 at 3:33 AM, Chris Wilson wrote: > > + do { > > + /* The pitch given to the GPU must be DWORD aligned, and > > + * we want width to match pitch. Max width is (1 << 15 - 1), > > + * rounding that do

[Mesa-dev] [Bug 91254] (regresion) video using VA-API on Intel slow and freeze system with mesa 10.6 or 10.6.1

2015-07-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91254 me...@frugalware.org changed: What|Removed |Added CC||me...@frugalware.org --- Comment #

Re: [Mesa-dev] [PATCH 1/2] mesa: Detect and provide macros for function attributes pure and const.

2015-07-18 Thread Jose Fonseca
On 18/07/15 01:38, Eric Anholt wrote: Emil Velikov writes: On 14/07/15 19:45, Eric Anholt wrote: These are really useful hints to the compiler in the absence of link-time optimization, and I'm going to use them in VC4. I've made the const attribute be ATTRIBUTE_CONST unlike other function at