[Mesa-dev] [PATCH 2/9] i965: check malloc return value in intel_resolve_map_set()

2014-07-03 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila --- src/mesa/drivers/dri/i965/intel_resolve_map.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/mesa/drivers/dri/i965/intel_resolve_map.c b/src/mesa/drivers/dri/i965/intel_resolve_map.c index bf6bcf2..bb45231 100644 --- a/src/mesa/drivers/dri/i9

[Mesa-dev] [PATCH 4/9] glsl: Check calloc return value in link_intrastage_shaders()

2014-07-03 Thread Juha-Pekka Heikkila
Check calloc return value while adding build-in functions. Signed-off-by: Juha-Pekka Heikkila --- src/glsl/linker.cpp | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp index d588bc6..2f004f1 100644 --- a/src/glsl/linker

[Mesa-dev] [PATCH 3/9] i965: Avoid null access in intelMakeCurrent()

2014-07-03 Thread Juha-Pekka Heikkila
separate two null checks connected with && to their own if branches. Signed-off-by: Juha-Pekka Heikkila --- src/mesa/drivers/dri/i965/brw_context.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/

[Mesa-dev] [PATCH 0/9] Klocwork patches again

2014-07-03 Thread Juha-Pekka Heikkila
i965: Avoid null access in intelMakeCurrent() I was a bit iffy if its the same behavior which I replaced. i965: in set_read_rb_tex_image() check _mesa_meta_bind_rb_as_tex_image() did succeed - I tried to follow Topi's suggestion. I did not go touching brw_meta_updownsample() here, it is left fo

[Mesa-dev] [PATCH 9/9] mesa/main: Check allocations success in _mesa_one_time_init_extension_overrides()

2014-07-03 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila --- src/mesa/main/extensions.c | 9 + 1 file changed, 9 insertions(+) diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c index 92e3f0d..cd89e8f 100644 --- a/src/mesa/main/extensions.c +++ b/src/mesa/main/extensions.c @@ -602,6 +602,15

[Mesa-dev] [PATCH 1/9] i965: in set_read_rb_tex_image() check _mesa_meta_bind_rb_as_tex_image() did succeed

2014-07-03 Thread Juha-Pekka Heikkila
Check if _mesa_meta_bind_rb_as_tex_image() did give the texture. If no texture was given there is already either GL_INVALID_VALUE or GL_OUT_OF_MEMORY error set in context. Signed-off-by: Juha-Pekka Heikkila --- src/mesa/drivers/common/meta_blit.c | 10 +- src/mesa/drivers/d

[Mesa-dev] [PATCH 6/9] glsl: Check realloc return value in ir_function::matching_signature()

2014-07-03 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila --- src/glsl/ir_function.cpp | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/glsl/ir_function.cpp b/src/glsl/ir_function.cpp index 7d6c2f4..04ab5ab 100644 --- a/src/glsl/ir_function.cpp +++ b/src/glsl/ir_function.cpp @@ -24,6

[Mesa-dev] [PATCH 5/9] loader: Check dlsym() did not fail in libudev_get_device_name_for_fd()

2014-07-03 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila --- src/loader/loader.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/loader/loader.c b/src/loader/loader.c index 47e1f58..2205a9c 100644 --- a/src/loader/loader.c +++ b/src/loader/loader.c @@ -594,6 +594,9 @@ libudev_get_device_name_for_fd(int fd)

[Mesa-dev] [PATCH 8/9] i965: Check after malloc success in intel_miptree_alloc_hiz()

2014-07-03 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila --- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 4 1 file changed, 4 insertions(+) diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c index 2ab0faa..30030d1 100644 --- a/src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH 7/9] i965: Avoid null access in fs_generator::generate_code()

2014-07-03 Thread Juha-Pekka Heikkila
Avoid null access while printing debug infos. On the same go change local variable name to avoid confusion because there already is class member with same name. Signed-off-by: Juha-Pekka Heikkila --- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 8 ++-- 1 file changed, 6 insertions(+), 2

Re: [Mesa-dev] [PATCH demos 0/3] demos release plan and glxinfo: Print more limits

2014-07-03 Thread Andreas Boll
2014-07-02 21:59 GMT+02:00 Brian Paul : > On 07/02/2014 01:04 PM, Andreas Boll wrote: >> >> I'd like to make a new demos release on Friday, July 4th. >> The last release was on February 24th, 2013. Additionally this >> release is needed to fix the build with mesa 10.2. (fdo#78101) >> >> Any objecti

Re: [Mesa-dev] [PATCH demos 0/3] demos release plan and glxinfo: Print more limits

2014-07-03 Thread Andreas Boll
2014-07-03 3:09 GMT+02:00 Ilia Mirkin : > On Wed, Jul 2, 2014 at 3:04 PM, Andreas Boll > wrote: >> I'd like to make a new demos release on Friday, July 4th. >> The last release was on February 24th, 2013. Additionally this >> release is needed to fix the build with mesa 10.2. (fdo#78101) >> >> An

Re: [Mesa-dev] [PATCH demos 0/3] demos release plan and glxinfo: Print more limits

2014-07-03 Thread Andreas Boll
2014-07-03 7:39 GMT+02:00 Steven Newbury : > On Wed, 2014-07-02 at 21:04 +0200, Andreas Boll wrote: > >> I'd like to make a new demos release on Friday, July 4th. >> The last release was on February 24th, 2013. Additionally this >> release is needed to fix the build with mesa 10.2. (fdo#78101) >> >

Re: [Mesa-dev] [PATCH v3 0/3] Software rendering in EGL-DRM

2014-07-03 Thread Boris BREZILLON
Hello Giovanni, I have recently been working on a DRM/KMS driver which does not support OpenGL rendering (it only provides plane composition functionalities): [1]. If I understand correctly you patch series might solve some of the issues I am facing. I'm trying to get wayland working with HW cur

[Mesa-dev] [PATCH] st/xa: Don't close the drm fd on failure

2014-07-03 Thread Thomas Hellstrom
If XA fails to initialize with pipe_loader enabled, the pipe_loader's cleanup function will close the drm file descriptor. That's pretty bad because the file descriptor will probably be the X server driver's only connection to drm. Temporarily solve this by dup()'ing the file descriptor before hand

Re: [Mesa-dev] [PATCH v3 0/3] Software rendering in EGL-DRM

2014-07-03 Thread Pekka Paalanen
On Thu, 3 Jul 2014 10:48:26 +0200 Boris BREZILLON wrote: > Hello Giovanni, > > I have recently been working on a DRM/KMS driver which does not support > OpenGL rendering (it only provides plane composition functionalities): > [1]. > > If I understand correctly you patch series might solve some

[Mesa-dev] [PATCH] st/xa: Don't close the drm fd on failure v2

2014-07-03 Thread Thomas Hellstrom
If XA fails to initialize with pipe_loader enabled, the pipe_loader's cleanup function will close the drm file descriptor. That's pretty bad because the file descriptor will probably be the X server driver's only connection to drm. Temporarily solve this by dup()'ing the file descriptor before hand

Re: [Mesa-dev] [PATCH v2 1/5] mesa/main: add ARB_clear_texture entrypoints

2014-07-03 Thread Neil Roberts
Ilia Mirkin writes: > I believe what this is saying is akin to "Danger! There is a hole from > 145 to 146". Since ARB_clear_texture is 145, the hole is now just 146 > (ARB_enhanced_layouts)... Whereas your comments indicate the numbers > of the actually-listed extensions. > > Note that I don't re

Re: [Mesa-dev] [PATCH] st/xa: Don't close the drm fd on failure v2

2014-07-03 Thread Jakob Bornecrantz
- Original Message - > If XA fails to initialize with pipe_loader enabled, the pipe_loader's > cleanup function will close the drm file descriptor. That's pretty bad > because the file descriptor will probably be the X server driver's only > connection to drm. Temporarily solve this by dup(

Re: [Mesa-dev] [PATCH v3 0/3] Software rendering in EGL-DRM

2014-07-03 Thread Boris BREZILLON
On Thu, 3 Jul 2014 11:14:48 +0200 Giovanni Campagna wrote: > 2014-07-03 10:48 GMT+02:00 Boris BREZILLON > : > > Hello Giovanni, > > > > I have recently been working on a DRM/KMS driver which does not support > > OpenGL rendering (it only provides plane composition functionalities): > > [1]. > >

Re: [Mesa-dev] [PATCH v3 0/3] Software rendering in EGL-DRM

2014-07-03 Thread Boris BREZILLON
On Thu, 3 Jul 2014 12:24:44 +0300 Pekka Paalanen wrote: > On Thu, 3 Jul 2014 10:48:26 +0200 > Boris BREZILLON wrote: > > > Hello Giovanni, > > > > I have recently been working on a DRM/KMS driver which does not support > > OpenGL rendering (it only provides plane composition functionalities):

[Mesa-dev] Another important XA patch

2014-07-03 Thread Thomas Hellstrom
Hi, Carl, Patch st/xa: Don't close the drm fd on failure v2 causes some conflicts when backported to 10.2. Let me know if you need help resolving them, or if you want me to push a backported version to 10.2. Thanks, Thomas ___ mesa-dev mailing list mesa

Re: [Mesa-dev] [PATCH 4/5] r600g: allow vs to write to gl_ViewportIndex

2014-07-03 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Wed, Jul 2, 2014 at 10:30 PM, Ilia Mirkin wrote: > Signed-off-by: Ilia Mirkin > Tested-by: Tobias Droste > --- > > In addition to the existing ARB_fragment_layer_viewport piglits passing and > the new AMD_vertex_shader_viewport_index piglit that is on the lis

Re: [Mesa-dev] [PATCH 5/5] mesa/st: enable AMD_vertex_shader_viewport_index

2014-07-03 Thread Marek Olšák
Reviewed-by: Marek Olšák Renaming the cap to include "VIEWPORT" would not be a bad idea. Marek On Wed, Jul 2, 2014 at 10:30 PM, Ilia Mirkin wrote: > The assumption is that any driver capable of emitting layer from the > vertex shader and supporting viewports should be able to also handle > emi

Re: [Mesa-dev] [PATCH v3 0/3] Software rendering in EGL-DRM

2014-07-03 Thread Pekka Paalanen
On Thu, 3 Jul 2014 12:10:36 +0200 Boris BREZILLON wrote: > On Thu, 3 Jul 2014 12:24:44 +0300 > Pekka Paalanen wrote: > > > On Thu, 3 Jul 2014 10:48:26 +0200 > > Boris BREZILLON wrote: > > > > > Hello Giovanni, > > > > > > I have recently been working on a DRM/KMS driver which does not suppor

Re: [Mesa-dev] [PATCH v3 0/3] Software rendering in EGL-DRM

2014-07-03 Thread Boris BREZILLON
On Thu, 3 Jul 2014 13:49:06 +0300 Pekka Paalanen wrote: > On Thu, 3 Jul 2014 12:10:36 +0200 > Boris BREZILLON wrote: > > > On Thu, 3 Jul 2014 12:24:44 +0300 > > Pekka Paalanen wrote: > > > > > On Thu, 3 Jul 2014 10:48:26 +0200 > > > Boris BREZILLON wrote: > > > > > > > Hello Giovanni, > > >

Re: [Mesa-dev] [PATCH 1/9] i965: in set_read_rb_tex_image() check _mesa_meta_bind_rb_as_tex_image() did succeed

2014-07-03 Thread Pohjolainen, Topi
On Thu, Jul 03, 2014 at 11:13:11AM +0300, Juha-Pekka Heikkila wrote: > Check if _mesa_meta_bind_rb_as_tex_image() did give the texture. > If no texture was given there is already either > GL_INVALID_VALUE or GL_OUT_OF_MEMORY error set in context. > > Signed-off-by: Juha-Pekka Heikkila > --- > sr

Re: [Mesa-dev] [PATCH v3 0/3] Software rendering in EGL-DRM

2014-07-03 Thread Pekka Paalanen
On Thu, 3 Jul 2014 14:15:34 +0200 Boris BREZILLON wrote: > On Thu, 3 Jul 2014 13:49:06 +0300 > Pekka Paalanen wrote: > > > On Thu, 3 Jul 2014 12:10:36 +0200 > > Boris BREZILLON wrote: > > > > > On Thu, 3 Jul 2014 12:24:44 +0300 > > > Pekka Paalanen wrote: > > > > Weston's overlay planes code

Re: [Mesa-dev] [PATCH v3 0/3] Software rendering in EGL-DRM

2014-07-03 Thread Boris BREZILLON
On Thu, 3 Jul 2014 15:46:14 +0300 Pekka Paalanen wrote: > On Thu, 3 Jul 2014 14:15:34 +0200 > Boris BREZILLON wrote: > > > On Thu, 3 Jul 2014 13:49:06 +0300 > > Pekka Paalanen wrote: > > > > > On Thu, 3 Jul 2014 12:10:36 +0200 > > > Boris BREZILLON wrote: > > > > > > > On Thu, 3 Jul 2014 12

Re: [Mesa-dev] [PATCH 7/9] i965: Avoid null access in fs_generator::generate_code()

2014-07-03 Thread Pohjolainen, Topi
On Thu, Jul 03, 2014 at 11:13:17AM +0300, Juha-Pekka Heikkila wrote: > Avoid null access while printing debug infos. On the same go > change local variable name to avoid confusion because there > already is class member with same name. > > Signed-off-by: Juha-Pekka Heikkila > --- > src/mesa/driv

Re: [Mesa-dev] [PATCH] glsl: fix duplicated layout qualifier detection for GS

2014-07-03 Thread Samuel Iglesias Gonsálvez
On Wed, 2014-07-02 at 12:47 -0700, Jordan Justen wrote: > Reviewed-by: Jordan Justen > I don't have commit access to the repository. Would you mind pushing this patch to master? Sam signature.asc Description: This is a digitally signed message part ___

[Mesa-dev] [PATCH 1/2] glxinfo: remove query of GL_MAX_VERTEX_ATTRIB_STRIDE

2014-07-03 Thread Brian Paul
This is not part of the GL_ARB_vertex_attrib_binding extension. It's part of OpenGL 4.4. Fixes compilation failure if glext.h doesn't have the GL 4.4 #defines. --- src/xdemos/glinfo_common.c |1 - 1 file changed, 1 deletion(-) diff --git a/src/xdemos/glinfo_common.c b/src/xdemos/glinfo_common

[Mesa-dev] [PATCH 2/2] glxinfo: add query for OpenGL 4.4 GL_MAX_VERTEX_ATTRIB_STRIDE

2014-07-03 Thread Brian Paul
And add support for qualifying a limit query with an OpenGL version instead of an extension string. --- src/xdemos/glinfo_common.c | 30 +- 1 file changed, 29 insertions(+), 1 deletion(-) diff --git a/src/xdemos/glinfo_common.c b/src/xdemos/glinfo_common.c index 4360

Re: [Mesa-dev] [PATCH 7/9] i965: Avoid null access in fs_generator::generate_code()

2014-07-03 Thread Juha-Pekka Heikkila
On 03.07.2014 16:26, Pohjolainen, Topi wrote: > On Thu, Jul 03, 2014 at 11:13:17AM +0300, Juha-Pekka Heikkila wrote: >> Avoid null access while printing debug infos. On the same go >> change local variable name to avoid confusion because there >> already is class member with same name. >> >> Signed

Re: [Mesa-dev] [PATCH 5/5] clover: Enable cl_khr_fp64 for devices that support doubles v2

2014-07-03 Thread Tom Stellard
On Thu, Jul 03, 2014 at 01:12:07AM +0200, Francisco Jerez wrote: > Tom Stellard writes: > > > On Thu, Jun 26, 2014 at 04:15:39PM +0200, Francisco Jerez wrote: > >> Tom Stellard writes: > >> > >> > v2: > >> > - Report correct values for CL_DEVICE_NATIVE_VECTOR_WIDTH_DOUBLE and > >> > CL_DE

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Tom Stellard
On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: > Previously, we were assuming that kernel metadata nodes only had 1 operand. > > Kernels which have attributes can have more than 1, e.g.: > !0 = metadata !{void (i32 addrspace(1)*)* @testKernel, metadata !1} > !1 = metadata !{metadata

[Mesa-dev] [PATCH 6/5] gallium: rename PIPE_CAP_TGSI_VS_LAYER to also have _VIEWPORT

2014-07-03 Thread Ilia Mirkin
Now that this cap is used to determine the availability of both, adjust its name to reflect the new reality. Signed-off-by: Ilia Mirkin --- src/gallium/auxiliary/util/u_blitter.c | 2 +- src/gallium/docs/source/screen.rst | 6 -- src/gallium/drivers/freedreno/freedren

Re: [Mesa-dev] [PATCH 6/5] gallium: rename PIPE_CAP_TGSI_VS_LAYER to also have _VIEWPORT

2014-07-03 Thread Roland Scheidegger
Reviewed-by: Roland Scheidegger Am 03.07.2014 17:17, schrieb Ilia Mirkin: > Now that this cap is used to determine the availability of both, adjust > its name to reflect the new reality. > > Signed-off-by: Ilia Mirkin > --- > src/gallium/auxiliary/util/u_blitter.c | 2 +- > src/galli

Re: [Mesa-dev] [PATCH] gallium: pass in per-sample interpolation qualifier

2014-07-03 Thread Roland Scheidegger
Am 03.07.2014 03:19, schrieb Ilia Mirkin: > On Wed, Jul 2, 2014 at 4:03 PM, Roland Scheidegger wrote: >> Actually on second thought, because centroid and sample are mutually >> exclusive (or more generally, the "Interpolate" member determines "how" >> interpolation is done, whereas these two deter

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Aaron Watry
On Thu, Jul 3, 2014 at 9:56 AM, Tom Stellard wrote: > On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: >> Previously, we were assuming that kernel metadata nodes only had 1 operand. >> >> Kernels which have attributes can have more than 1, e.g.: >> !0 = metadata !{void (i32 addrspace(1

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Alex Deucher
On Thu, Jul 3, 2014 at 11:46 AM, Aaron Watry wrote: > On Thu, Jul 3, 2014 at 9:56 AM, Tom Stellard wrote: >> On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: >>> Previously, we were assuming that kernel metadata nodes only had 1 operand. >>> >>> Kernels which have attributes can have

Re: [Mesa-dev] [PATCH 6/5] gallium: rename PIPE_CAP_TGSI_VS_LAYER to also have _VIEWPORT

2014-07-03 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Thu, Jul 3, 2014 at 5:17 PM, Ilia Mirkin wrote: > Now that this cap is used to determine the availability of both, adjust > its name to reflect the new reality. > > Signed-off-by: Ilia Mirkin > --- > src/gallium/auxiliary/util/u_blitter.c | 2 +- >

[Mesa-dev] [PATCH 1/2] R600/SI: fix shadow mapping for 1D and 2D array textures

2014-07-03 Thread Marek Olšák
From: Marek Olšák It was conflicting with def TEX_SHADOW_ARRAY, which also handles them. --- lib/Target/R600/R600Instructions.td | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/Target/R600/R600Instructions.td b/lib/Target/R600/R600Instructions.td index 73fa345..704507d 1

[Mesa-dev] [PATCH] gallium: fix u_default_transfer_inline_write for textures

2014-07-03 Thread Marek Olšák
From: Marek Olšák This doesn't fix any known issue. In fact, radeon drivers ignore all the discard flags for textures and implicitly do "discard range" for any write transfer. Cc: mesa-sta...@lists.freedesktop.org --- src/gallium/auxiliary/util/u_transfer.c | 4 ++-- 1 file changed, 2 insertion

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Tom Stellard
On Thu, Jul 03, 2014 at 10:56:24AM -0400, Tom Stellard wrote: > On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: > > Previously, we were assuming that kernel metadata nodes only had 1 operand. > > > > Kernels which have attributes can have more than 1, e.g.: > > !0 = metadata !{void (i

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands\

2014-07-03 Thread Tom Stellard
On Thu, Jul 03, 2014 at 11:59:00AM -0400, Alex Deucher wrote: > On Thu, Jul 3, 2014 at 11:46 AM, Aaron Watry wrote: > > On Thu, Jul 3, 2014 at 9:56 AM, Tom Stellard wrote: > >> On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: > >>> Previously, we were assuming that kernel metadata nod

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Aaron Watry
On Thu, Jul 3, 2014 at 11:29 AM, Tom Stellard wrote: > On Thu, Jul 03, 2014 at 10:56:24AM -0400, Tom Stellard wrote: >> On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: >> > Previously, we were assuming that kernel metadata nodes only had 1 operand. >> > >> > Kernels which have attribu

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Aaron Watry
On Thu, Jul 3, 2014 at 10:59 AM, Alex Deucher wrote: > Someone mentioned stability issues with cedar with the golden register > kernel patch. Can you see if skipping the golden register setup > helps? If so can you narrow down which registers are problematic? I'll give it a shot and see if it h

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Tom Stellard
On Thu, Jul 03, 2014 at 11:55:25AM -0500, Aaron Watry wrote: > On Thu, Jul 3, 2014 at 10:59 AM, Alex Deucher wrote: > > Someone mentioned stability issues with cedar with the golden register > > kernel patch. Can you see if skipping the golden register setup > > helps? If so can you narrow down

Re: [Mesa-dev] [PATCH] glsl: fix duplicated layout qualifier detection for GS

2014-07-03 Thread Jordan Justen
On 2014-07-03 07:18:42, Samuel Iglesias Gonsálvez wrote: > On Wed, 2014-07-02 at 12:47 -0700, Jordan Justen wrote: > > Reviewed-by: Jordan Justen > > I don't have commit access to the repository. > > Would you mind pushing this patch to master? Pushed. Thanks for the patch! -Jordan signature

[Mesa-dev] [PATCH 2/2] i965: Viewport extents print (don't push)

2014-07-03 Thread Ben Widawsky
Just to verify the code does what we want... --- src/mesa/drivers/dri/i965/gen8_viewport_state.c | 4 1 file changed, 4 insertions(+) diff --git a/src/mesa/drivers/dri/i965/gen8_viewport_state.c b/src/mesa/drivers/dri/i965/gen8_viewport_state.c index 2bf5fbb..cfbcb12 100644 --- a/src/mesa/d

[Mesa-dev] [PATCH 1/2] i965: Viewport extents on gen8

2014-07-03 Thread Ben Widawsky
Viewport extents are a 3rd rectangle that defines which pixels get discarded as part of the rasterization process. This can potentially improve performance by reducing cache usage, and freeing up PS cycles. This will get hit if one's viewport is smaller than the full renderbuffer, and scissoring is

[Mesa-dev] [PATCH v3 2/2] nv50/ir: Handle OP_CVT when folding constant expressions

2014-07-03 Thread Tobias Klausmann
Folding for conversions: F32/64->(U16/32, S16/32) and (U16/32, S16/32)->F32 No piglit regressions observed on nv50 and nvc0! Signed-off-by: Tobias Klausmann --- .../drivers/nouveau/codegen/nv50_ir_peephole.cpp | 74 ++ 1 file changed, 74 insertions(+) diff --git a/src/gall

[Mesa-dev] [PATCH v3 1/2] nv50/ir: Add support for the double Type to BuildUtil

2014-07-03 Thread Tobias Klausmann
Signed-off-by: Tobias Klausmann --- .../drivers/nouveau/codegen/nv50_ir_build_util.cpp | 17 + .../drivers/nouveau/codegen/nv50_ir_build_util.h| 2 ++ 2 files changed, 19 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_build_util.cpp b/src/ga

[Mesa-dev] [PATCH] svga: Don't unnecessarily reemit BindGBShader commands v2

2014-07-03 Thread Thomas Hellstrom
The Linux winsys can no longer relocate shader code, so avoid reemitting BindGBShader commands. They are costly. v2: Correctly handle errors from SVGA3D_BindGBShader() Reported-by: Michael Banack Signed-off-by: Thomas Hellstrom Tested-by: Brian Paul Reviewed-by: Brian Paul Reviewed-by: Jakob

[Mesa-dev] [PATCH] i915: Fix up intelInitScreen2 for DRI3

2014-07-03 Thread Adel Gadllah
Commit 442442026eb updated both i915 and i965 for DRI3 support, but one check in intelInitScreen2 was missed for i915 causing crashes when trying to use i915 with DRI3. So fix that up. Reported-by: Igor Gnatenko Tested-by: František Zatloukal Reference: https://bugzilla.redhat.com/show_bug.cgi?

[Mesa-dev] [PATCH] svga: Don't unnecessarily reemit BindGBShader commands v2

2014-07-03 Thread Thomas Hellstrom
The Linux winsys can no longer relocate shader code, so avoid reemitting BindGBShader commands. They are costly. v2: Correctly handle errors from SVGA3D_BindGBShader() Reported-by: Michael Banack Signed-off-by: Thomas Hellstrom Tested-by: Brian Paul Reviewed-by: Brian Paul Reviewed-by: Jakob

Re: [Mesa-dev] [PATCH 1/2] i965: Don't enable SOL statistics during meta operations.

2014-07-03 Thread Kenneth Graunke
On Wednesday, July 02, 2014 09:08:04 PM Kristian Høgsberg wrote: > On Tue, Jul 1, 2014 at 5:25 PM, Kenneth Graunke wrote: > > The GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN counter is not supposed to > > increment during glGenerateMipmap(). I don't think either counter is > > supposed to increment

[Mesa-dev] [PATCH] glsl/glcpp: Integrate recent glcpp-test-cr-lf test into "make check"

2014-07-03 Thread Carl Worth
Beyond just listing this in the TESTS variable in Makefile.am, only minor changes were needed to make this work. The primary issue is that the build system runs the test script from a different directory than the script itself. So we have to use the $srcdir variable to find the test input files. U

Re: [Mesa-dev] [PATCH 2/2] i965: Disable SOL buffers and decls when not doing transform feedback.

2014-07-03 Thread Kenneth Graunke
On Wednesday, July 02, 2014 10:09:49 AM Iago Toral wrote: > On Wed, 2014-07-02 at 08:49 +0200, Iago Toral wrote: > > Hello Kenneth, Steven: > > > > On Tue, 2014-07-01 at 17:25 -0700, Kenneth Graunke wrote: > > > Previously, we only emitted 3DSTATE_SO_BUFFER and 3DSTATE_SO_DECL_LIST > > > when tran

Re: [Mesa-dev] [PATCH 8/9] i965: Check after malloc success in intel_miptree_alloc_hiz()

2014-07-03 Thread Kenneth Graunke
On Thursday, July 03, 2014 11:13:18 AM Juha-Pekka Heikkila wrote: > Signed-off-by: Juha-Pekka Heikkila > --- > src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/int

[Mesa-dev] [PATCH] glsl/glcpp: Fix handling of commas that result from macro expansion

2014-07-03 Thread Carl Worth
Here is some additional stress testing of nested macros where the expansion of macros involves commas, (and whether those commas are interpreted as argument separators or not in subsequent function-like macro calls). Credit to the GCC documentation that directed my attention toward this issue:

Re: [Mesa-dev] [PATCH 2/9] i965: check malloc return value in intel_resolve_map_set()

2014-07-03 Thread Kenneth Graunke
On Thursday, July 03, 2014 11:13:12 AM Juha-Pekka Heikkila wrote: > Signed-off-by: Juha-Pekka Heikkila > --- > src/mesa/drivers/dri/i965/intel_resolve_map.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/src/mesa/drivers/dri/i965/intel_resolve_map.c b/src/mesa/drivers/dri/i965/i

[Mesa-dev] [PATCH] ff_fragment_shader: Access glsl_types directly.

2014-07-03 Thread Kenneth Graunke
Originally, we didn't have direct accessors for all of the GLSL types, so the only way to get at them was to use the symbol table. Now, we can just get at them directly, which is simpler and faster. --- src/mesa/main/ff_fragment_shader.cpp | 30 +++--- 1 file changed, 15 i

[Mesa-dev] [PATCH 1/2] radeonsi: properly implement texture opcodes that take an offset

2014-07-03 Thread Marek Olšák
From: Marek Olšák Instead of using intr_name in lp_build_tgsi_action, this selects the names with a switch statement in the emit function. This allows emitting llvm.SI.sample for instructions without offsets and llvm.SI.image.sample.*.o otherwise. This depends on my LLVM changes. When LLVM 3.5

[Mesa-dev] [PATCH 2/2] radeonsi: fix texture fetches with derivatives for 1DArray and 3D textures

2014-07-03 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.c | 34 1 file changed, 30 insertions(+), 4 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index 335e43b..b4d4f22 100644 --- a/src/galliu

Re: [Mesa-dev] [PATCH 5/5] clover: Enable cl_khr_fp64 for devices that support doubles v2

2014-07-03 Thread Francisco Jerez
Tom Stellard writes: > On Thu, Jul 03, 2014 at 01:12:07AM +0200, Francisco Jerez wrote: >> Tom Stellard writes: >> >> > On Thu, Jun 26, 2014 at 04:15:39PM +0200, Francisco Jerez wrote: >> >> Tom Stellard writes: >> >> >> >> > v2: >> >> > - Report correct values for CL_DEVICE_NATIVE_VECTOR_W

Re: [Mesa-dev] [PATCH 2/3] radeonsi: add sampling of 4:2:2 subsampled textures

2014-07-03 Thread Andy Furniss
Andy Furniss wrote: Grigori Goronzy wrote: On 02.07.2014 22:18, Andy Furniss wrote: Before I knew how to get field sync to use my TVs deinterlacer I had to modify mesa so that I could use the vdpau de-interlacer(s), when I did this I noticed that 422 didn't work and looked the same as it does

Re: [Mesa-dev] [PATCH] ff_fragment_shader: Access glsl_types directly.

2014-07-03 Thread Ben Widawsky
Reviewed-by: Ben Widawsky On Thu, Jul 03, 2014 at 02:47:14PM -0700, Kenneth Graunke wrote: > Originally, we didn't have direct accessors for all of the GLSL types, > so the only way to get at them was to use the symbol table. Now, we > can just get at them directly, which is simpler and faster.

Re: [Mesa-dev] [PATCH 5/5] clover: Enable cl_khr_fp64 for devices that support doubles v2

2014-07-03 Thread Tom Stellard
On Fri, Jul 04, 2014 at 12:28:20AM +0200, Francisco Jerez wrote: > Tom Stellard writes: > > > On Thu, Jul 03, 2014 at 01:12:07AM +0200, Francisco Jerez wrote: > >> Tom Stellard writes: > >> > >> > On Thu, Jun 26, 2014 at 04:15:39PM +0200, Francisco Jerez wrote: > >> >> Tom Stellard writes: > >

Re: [Mesa-dev] [PATCH 2/3] radeonsi: add sampling of 4:2:2 subsampled textures

2014-07-03 Thread Andy Furniss
Andy Furniss wrote: Andy Furniss wrote: Grigori Goronzy wrote: On 02.07.2014 22:18, Andy Furniss wrote: Before I knew how to get field sync to use my TVs deinterlacer I had to modify mesa so that I could use the vdpau de-interlacer(s), when I did this I noticed that 422 didn't work and looked

Re: [Mesa-dev] [Mesa-stable] [PATCH] mesa: Don't use derived vertex state in api_arrayelt.c

2014-07-03 Thread Carl Worth
Fredrik Höglund writes: > Cc: "10.1" (And "10.2" as pushed to master) Hi Fredrik, I helped prod you into pushing this patch to master, which brought it more directly into my view as a candidate for the 10.2 branch. Now that I'm taking a closer look at the patch, I think it's missing something

Re: [Mesa-dev] OpenGL on Wayland

2014-07-03 Thread Kalrish Bäakjen
On Wed, Jul 2, 2014 at 6:37 PM, Thomas Daede wrote: > What Mesa version are you using? IIRC OpenGL 3.3 is only supported in > llvmpipe in mesa 10.3 and newer. Using MESA_GL_VERISON_OVERRIDE just fakes > the version string, it doesn't change what functions you can actually link > to. > > Also, tha

Re: [Mesa-dev] [PATCH v3 0/3] Software rendering in EGL-DRM

2014-07-03 Thread Giovanni Campagna
2014-07-03 10:48 GMT+02:00 Boris BREZILLON : > Hello Giovanni, > > I have recently been working on a DRM/KMS driver which does not support > OpenGL rendering (it only provides plane composition functionalities): > [1]. > > If I understand correctly you patch series might solve some of the > issues

Re: [Mesa-dev] Another important XA patch

2014-07-03 Thread Carl Worth
Thomas Hellstrom writes: > Patch st/xa: Don't close the drm fd on failure v2 causes some conflicts > when backported to 10.2. Let me know if > you need help resolving them, or if you want me to push a backported > version to 10.2. Thanks for the heads-up, Thomas. The conflict resolution seemed q

[Mesa-dev] Patches prepped for 10.2.3 release

2014-07-03 Thread Carl Worth
I've just made a pass over the patches nominated for inclusion in the 10.2 branch. The results can be seen here: http://cworth.org/~cworth/mesa-stable-queue/ After a bit more testing, I expect to push the queued patches to the 10.2 branch. Then I'll likely let this sit over the weekend an

[Mesa-dev] What are some good beginner's tasks for Mesa?

2014-07-03 Thread Darius Goad
Hello. I'm trying to get my feet wet with Mesa, and I was wondering what some good tasks for me would be. Thanks again. - Darius Goad ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] What are some good beginner's tasks for Mesa?

2014-07-03 Thread Ilia Mirkin
On Thu, Jul 3, 2014 at 11:33 PM, Darius Goad wrote: > Hello. I'm trying to get my feet wet with Mesa, and I was wondering what > some good tasks for me would be. Thanks again. Take a look at http://wiki.freedesktop.org/dri/NewbieProjects/ Also depending on the hardware you have available to you

[Mesa-dev] [Bug 80266] Many instances of 1<<31, which is undefined in C99

2014-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=80266 --- Comment #13 from Vittorio --- More issues: u_pack_color.h:365 "uc->ui = (a << 24) | (r << 16) | (g << 8) | b;" should be "uc->ui = ((unsigned int)a << 24) | (r << 16) | (g << 8) | b;" m_matrix.c line 1156 "#define ONE(x) (1<<(x+16))" should

[Mesa-dev] [Bug 80888] New: draw_cliptest_tmp.h computes zero/zero It is better to use NAN macro

2014-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=80888 Priority: medium Bug ID: 80888 Assignee: mesa-dev@lists.freedesktop.org Summary: draw_cliptest_tmp.h computes zero/zero It is better to use NAN macro Severity: normal Cla

[Mesa-dev] [Bug 80889] New: draw_pipe_clip.c division by zero in interp

2014-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=80889 Priority: medium Bug ID: 80889 Assignee: mesa-dev@lists.freedesktop.org Summary: draw_pipe_clip.c division by zero in interp Severity: normal Classification: Unclassified OS:

Re: [Mesa-dev] Another important XA patch

2014-07-03 Thread Thomas Hellstrom
Hi! On 07/04/2014 04:59 AM, Carl Worth wrote: > Thomas Hellstrom writes: >> Patch st/xa: Don't close the drm fd on failure v2 causes some conflicts >> when backported to 10.2. Let me know if >> you need help resolving them, or if you want me to push a backported >> version to 10.2. > > Thanks fo

[Mesa-dev] [Bug 80889] draw_pipe_clip.c division by zero in interp

2014-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=80889 Roland Scheidegger changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 80888] draw_cliptest_tmp.h computes zero/zero It is better to use NAN macro

2014-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=80888 Roland Scheidegger changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---