Re: [Mesa-dev] [PATCH 2/2] radeonsi: Compile dummy pixel shader on demand

On 01.09.2014 19:08, Marek Olšák wrote: On Wed, Aug 27, 2014 at 10:57 AM, Michel Dänzer wrote: From: Michel Dänzer It's never used under normal circumstances. Signed-off-by: Michel Dänzer --- src/gallium/drivers/radeonsi/si_pipe.c | 7 --- src/gallium/drivers/radeonsi/si_state.c |

Re: [Mesa-dev] [PATCH] mesa/program_cache: calloc the correct size for the cache.

Yep, it should be allocating just the pointers. Reviewed-by: Tapani Pälli On 09/02/2014 02:22 AM, Dave Airlie wrote: > From: Dave Airlie > > Coverity reported this, and I think this is the right solution, > since cache->items is struct cache_item ** not struct cache_item *, > we also realloc it

Re: [Mesa-dev] [PATCH 07/37] i965/gen6/gs: Implement GS_OPCODE_FF_SYNC.

On lun, 2014-09-01 at 11:17 -0700, Kenneth Graunke wrote: > On Thursday, August 14, 2014 01:11:39 PM Iago Toral Quiroga wrote: > > This implements the FF_SYNC message required in gen6 geometry shaders to > > get the initial URB handle. > > --- > > src/mesa/drivers/dri/i965/brw_defines.h

Re: [Mesa-dev] [PATCH] glsl: free uniform_map on failure path.

Reviewed-by: Tapani Pälli On 09/02/2014 02:55 AM, Dave Airlie wrote: > From: Dave Airlie > > If we fails in reserve_explicit_locations, we leak uniform_map. > > Reported-by: coverity scanner. > > Signed-off-by: Dave Airlie > --- > src/glsl/linker.cpp | 4 +++- > 1 file changed, 3 insertions(+)

[Mesa-dev] [Bug 82327] FAIL: glcpp/tests/glcpp-test-cr-lf

https://bugs.freedesktop.org/show_bug.cgi?id=82327 Vinson Lee changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 83382] New: /usr/include/string.h:82:1: error: unknown type name '__extern_always_inline'

https://bugs.freedesktop.org/show_bug.cgi?id=83382 Priority: medium Bug ID: 83382 Assignee: mesa-dev@lists.freedesktop.org Summary: /usr/include/string.h:82:1: error: unknown type name '__extern_always_inline' Severity: norma

[Mesa-dev] [Bug 83381] New: dri_screen.c:72:10: error: expected '}'

https://bugs.freedesktop.org/show_bug.cgi?id=83381 Priority: medium Bug ID: 83381 Assignee: mesa-dev@lists.freedesktop.org Summary: dri_screen.c:72:10: error: expected '}' Severity: normal Classification: Unclassified OS: Lin

Re: [Mesa-dev] [PATCH] i965: add missing parens in vec4 visitor

Oops. Reviewed-by: Chris Forbes On Tue, Sep 2, 2014 at 12:14 PM, Dave Airlie wrote: > From: Dave Airlie > > coverity reported this, Matt said it look like missing parens, > not bad identing, so lets try that. > > Signed-off-by: Dave Airlie > --- > src/mesa/drivers/dri/i965/brw_vec4_visitor.c

[Mesa-dev] [PATCH] i965: add missing parens in vec4 visitor

From: Dave Airlie coverity reported this, Matt said it look like missing parens, not bad identing, so lets try that. Signed-off-by: Dave Airlie --- src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw

[Mesa-dev] [PATCH] glsl: free uniform_map on failure path.

From: Dave Airlie If we fails in reserve_explicit_locations, we leak uniform_map. Reported-by: coverity scanner. Signed-off-by: Dave Airlie --- src/glsl/linker.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp index d5473ad..5

[Mesa-dev] uninitialised fields in ir_variable::ir_variable

coverity gives out that we initialize a bunch of fields in the ir_variable constructor, but we don't initialize a bunch more of them, precise, assigned etc. Someone should probably take a look who knows C++ better than me :-) Dave. ___ mesa-dev mailing

[Mesa-dev] [PATCH] omx/h264: remove stray semicolon after if

From: Dave Airlie Coverity reported this, looks wrong to me. Signed-off-by: Dave Airlie --- src/gallium/state_trackers/omx/vid_dec_h264.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/omx/vid_dec_h264.c b/src/gallium/state_trackers/omx/vid_dec_

Re: [Mesa-dev] [PATCH 0/3] cl workdim v2

On Sat, 2014-08-16 at 13:13 +0300, Francisco Jerez wrote: > Jan Vesely writes: > > > On Thu, 2014-08-07 at 16:02 +0300, Francisco Jerez wrote: > >> Jan Vesely writes: > >> > >> > This respin includes Francisco's approach of providing implicit > >> > in the arg vector passed from clover, and Tom

[Mesa-dev] [PATCH] mesa/program_cache: calloc the correct size for the cache.

From: Dave Airlie Coverity reported this, and I think this is the right solution, since cache->items is struct cache_item ** not struct cache_item *, we also realloc it using struct cache_item * at some point. Signed-off-by: Dave Airlie --- src/mesa/program/prog_cache.c | 2 +- 1 file changed,

[Mesa-dev] [PATCH] r300g: pointless assignment of info.indexed

From: Dave Airlie Did this code mean to do something else, you tell me! Signed-off-by: Dave Airlie --- src/gallium/drivers/r300/r300_render.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/gallium/drivers/r300/r300_render.c b/src/gallium/drivers/r300/r300_render.c index 6e5b381..4c9

Re: [Mesa-dev] [PATCH] nouveau: don't leak dec struct on error

On Mon, Sep 1, 2014 at 7:08 PM, Dave Airlie wrote: > From: Dave Airlie > > This one path doesn't goto fail, so it seems to leak dec. > > Signed-off-by: Dave Airlie Reviewed-by: Ilia Mirkin > --- > src/gallium/drivers/nouveau/nouveau_video.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletio

[Mesa-dev] [PATCH] i965: possible typo in ir_unop_f2b case.

From: Dave Airlie Coverity reported this, I'm not sure this patch is correct, but I'm sure someone who knows can fix this or push my fix. Signed-off-by: Dave Airlie --- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/dri

[Mesa-dev] [PATCH] nouveau: don't leak dec struct on error

From: Dave Airlie This one path doesn't goto fail, so it seems to leak dec. Signed-off-by: Dave Airlie --- src/gallium/drivers/nouveau/nouveau_video.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/nouveau_video.c b/src/gallium/drivers/nouveau/

[Mesa-dev] [PATCH] vdpau: unlock the mutex on error paths in attribute setting.

From: Dave Airlie Coverity pointed out we never dropped the lock here, so fix it by using a common exit path. Signed-off-by: Dave Airlie --- src/gallium/state_trackers/vdpau/mixer.c | 38 ++-- 1 file changed, 26 insertions(+), 12 deletions(-) diff --git a/src/galli

Re: [Mesa-dev] [PATCH v2 00/14] i965/gen7+ hiz cleanup + gen8 hiz improvements

Ken, I think you were concerned about: [PATCH v2 06/14] i965/gen8: Enable hiz for all depth levels but, you weren't able to reproduce any issues with it. Should we try this series on master now that 10.3 has branched? -Jordan On Mon, Jul 21, 2014 at 11:00 PM, Jordan Justen wrote: > 1. No longe

Re: [Mesa-dev] [Mesa-stable] [PATCH v4] winsys/radeon: fix nop packet padding for hawaii

Hi Emil, The first attachment is the patch for 10.2. The other two patches fix build with latest LLVM for 10.2. Marek On Tue, Aug 26, 2014 at 10:09 PM, Emil Velikov wrote: > On 06/08/14 17:16, Alex Deucher wrote: >> On Mon, Aug 4, 2014 at 6:48 AM, Andreas Boll >> wrote: >>> The initial firmwa

[Mesa-dev] [PATCH] r600g: Implement sm5 interpolation functions

Requires evergreen/cayman Signed-off-by: Glenn Kennard --- This patch depends on r600g: Implement GL_ARB_sample_shading Implementation note: interpolateAtSample/Offset is doing affine rather than perspective correct reverse projection, I think for the small intra-pixel offsets used this should t

Re: [Mesa-dev] [PATCH] r600g: Implement sm5 geometry shader instancing

Pushed, thanks. Marek On Mon, Aug 25, 2014 at 11:05 AM, Glenn Kennard wrote: > Requires Evergreen or later hardware. > > Signed-off-by: Glenn Kennard > --- > docs/GL3.txt | 2 +- > src/gallium/drivers/r600/evergreen_state.c | 4 ++-- > src/gallium/drivers/r600/r

[Mesa-dev] [Bug 66184] src/mesa/state_tracker/st_glsl_to_tgsi.cpp:3216:simplify_cmp: Assertion `inst->dst.index < 4096' failed.

https://bugs.freedesktop.org/show_bug.cgi?id=66184 Marek Olšák changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [PATCH] nv50/ir: use unordered_set instead of list to keep track of var defs

The set of variable defs does not need to be ordered in any way, and removing/adding elements is a fairly common operation in various optimization passes. This shortens runtime of piglit test fp-long-alu to ~11s from ~22s No piglit regressions observed on nvc0! Signed-off-by: Tobias Klausmann -

Re: [Mesa-dev] [PATCH 07/37] i965/gen6/gs: Implement GS_OPCODE_FF_SYNC.

On Thursday, August 14, 2014 01:11:39 PM Iago Toral Quiroga wrote: > This implements the FF_SYNC message required in gen6 geometry shaders to > get the initial URB handle. > --- > src/mesa/drivers/dri/i965/brw_defines.h | 14 + > src/mesa/drivers/dri/i965/brw_shader.cpp |

[Mesa-dev] [PATCH] mesa: invalidate draw state in glPopClientAttrib

From: Marek Olšák Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=82538 Cc: mesa-sta...@lists.freedesktop.org --- src/mesa/main/attrib.c | 4 1 file changed, 4 insertions(+) diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c index c656845..2e289b6 100644 --- a/src/mesa/ma

Re: [Mesa-dev] TGSI and Tessellation Control Shader outputs

Am 01.09.2014 18:53, schrieb Ilia Mirkin: > On Mon, Sep 1, 2014 at 12:47 PM, Roland Scheidegger > wrote: >> Am 01.09.2014 18:19, schrieb Ilia Mirkin: >>> On Mon, Sep 1, 2014 at 12:00 PM, Roland Scheidegger >>> wrote: Am 29.08.2014 22:44, schrieb Ilia Mirkin: > Hello, > > I've b

Re: [Mesa-dev] [PATCH 13/37] i965/gen6/gs: Implement GS_OPCODE_SET_DWORD_2.

On Thursday, August 14, 2014 01:11:45 PM Iago Toral Quiroga wrote: > We have GS_OPCODE_SET_DWORD_2_IMMED but this requires its source argument to > be > an immediate. In gen6 we need to set dword 2 of the URB write message header > from values stored in separate register, so we need something more

Re: [Mesa-dev] [PATCH] loader: fds can be 0

On Monday, September 01, 2014 07:45:26 PM Dave Airlie wrote: > Possible resource leak reported by coverity. > > Reported-by: Coverity scanner. > Signed-off-by: Dave Airlie > --- > src/loader/loader.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/loader/loader.c b/

Re: [Mesa-dev] [PATCH 2/2] i965/fs: Don't segfault when debug-logging a null program

On Monday, September 01, 2014 01:36:16 AM Jason Ekstrand wrote: > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp > b/src/mesa/drivers/dri/i965/brw_fs.cpp > index 43aee66..8b76c3e 100644

[Mesa-dev] [PATCH 3/3] nv50: attach the buffer bo to the miptree structures

The current code... makes no sense. Use nouveau_bo_ref to attach the bo to the exposed resource so as to have the proper lifetime guarantees. Tested-by: Emil Velikov Signed-off-by: Ilia Mirkin Cc: "10.2 10.3" --- src/gallium/drivers/nouveau/nv50/nv84_video.c | 13 + 1 file changed,

[Mesa-dev] [PATCH 1/3] nv50: set the miptree address when clearing bo's in vp2 init

The mt address is about to be used more, make sure it's set appropriately. Reported-by: Emil Velikov Tested-by: Emil Velikov Signed-off-by: Ilia Mirkin Cc: "10.2 10.3" --- src/gallium/drivers/nouveau/nv50/nv84_video.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/n

[Mesa-dev] [PATCH 2/3] nv50: mt address may not be the underlying bo's start address

With VP2, nv50_miptree is faked because the underlying bo's have to be laid out in a certain way. This is done by adjusting the address. Make sure that blits (and everything else for consistency) use the mt address rather than the bo address as a base. This fixes retrieving chroma plane with VDPAU

Re: [Mesa-dev] TGSI and Tessellation Control Shader outputs

On Mon, Sep 1, 2014 at 12:47 PM, Roland Scheidegger wrote: > Am 01.09.2014 18:19, schrieb Ilia Mirkin: >> On Mon, Sep 1, 2014 at 12:00 PM, Roland Scheidegger >> wrote: >>> Am 29.08.2014 22:44, schrieb Ilia Mirkin: Hello, I've been thinking a bit about how to properly implement TCS

Re: [Mesa-dev] TGSI and Tessellation Control Shader outputs

Am 01.09.2014 18:19, schrieb Ilia Mirkin: > On Mon, Sep 1, 2014 at 12:00 PM, Roland Scheidegger > wrote: >> Am 29.08.2014 22:44, schrieb Ilia Mirkin: >>> Hello, >>> >>> I've been thinking a bit about how to properly implement TCS outputs >>> in TGSI. As a quick reminder, there are per-vertex (i.e

[Mesa-dev] [PATCH 00/14] First steps to generalize scalar code gen

This starts the process of generalizing the scalar code gen so it can be used for other stages beyond FS. These changes are enough to allow code for a basic compute shader to be generated. (With some more CS patches added...) No piglit regressions were seen on gen7. git://people.freedesktop.org/

[Mesa-dev] [PATCH 06/14] i965: Add brw_scalar_prog_data structure

All fields were migrated from brw_wm_prog_data. In future updates, we can move these FS specific fields back into brw_wm_prog_data. The scalar_visitor and scalar_generator class mainly use these structures now. Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/Makefile.sources

[Mesa-dev] [PATCH 01/14] i965: Rename brw_fs.h to brw_scalar.h

Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h | 2 +- src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +- src/mesa/drivers/dri/i965/brw_fs.h | 699 - .../drivers/dri/i965/brw_fs_copy_propagation.cpp | 2 +- s

[Mesa-dev] [PATCH 14/14] i965: Don't store gl_fragment_program in scalar_visitor

Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_fs_fp.cpp | 8 ++-- src/mesa/drivers/dri/i965/brw_scalar.h | 1 - src/mesa/drivers/dri/i965/brw_scalar_visitor.cpp | 1 - src/mesa/drivers/dri/i965/brw_wm_iz.cpp | 2 ++ 4 files changed, 8 insertions(+)

[Mesa-dev] [PATCH 13/14] i965: Remove gl_fragment_program dependence in scalar_generator

Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +- src/mesa/drivers/dri/i965/brw_scalar.h | 4 ++-- src/mesa/drivers/dri/i965/brw_scalar_generator.cpp | 10 +- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/mesa/dr

[Mesa-dev] [PATCH 05/14] i965: Allow brw_scalar.h to be included in C files

Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_scalar.h | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_scalar.h b/src/mesa/drivers/dri/i965/brw_scalar.h index 7d3ebdb..be71101 100644 --- a/src/mesa/drivers/dri/

[Mesa-dev] [PATCH 09/14] i965/scalar_visitor: Use prog rather than fp->Base

Reduce scalar_visitor's dependence on gl_fragment_program. Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_fs.cpp | 28 src/mesa/drivers/dri/i965/brw_scalar_visitor.cpp | 2 +- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/s

[Mesa-dev] [PATCH 12/14] i965: Add uses_dfdy to brw_scalar_prog_data

Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_context.h| 1 + src/mesa/drivers/dri/i965/brw_scalar_generator.cpp | 4 ++-- src/mesa/drivers/dri/i965/brw_wm.c | 1 + 3 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i96

[Mesa-dev] [PATCH 08/14] i965/scalar_visitor: Use stage_prog_data instead of prog_data->base

Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_scalar_visitor.cpp | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_scalar_visitor.cpp b/src/mesa/drivers/dri/i965/brw_scalar_visitor.cpp index e1b1fd7..f021dcb 100644 ---

[Mesa-dev] [PATCH 07/14] i965: Add brw_scalar_prog_key structure

All fields were migrated from brw_wm_prog_key. In future updates, we can move these FS specific fields back into brw_wm_prog_key. The scalar_visitor and scalar_generator class mainly use these structures now. Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_blorp_blit_eu.cpp|

[Mesa-dev] [PATCH 11/14] i965: Add uses_kill to brw_scalar_prog_data

Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_context.h| 1 + src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +- src/mesa/drivers/dri/i965/brw_scalar_generator.cpp | 2 +- src/mesa/drivers/dri/i965/brw_scalar_visitor.cpp | 12 ++-- src/mesa/driver

[Mesa-dev] [PATCH 10/14] i965/fs: Use gl_program* rather than fp->Base

Reduce brw_fs_precompile's dependence on gl_fragment_program. Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_fs.cpp | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp i

[Mesa-dev] [PATCH 04/14] i965/scalar_visitor: Add init function

This common init routine can be used by constructors for multiple program types. Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_scalar.h | 1 + src/mesa/drivers/dri/i965/brw_scalar_visitor.cpp | 6 ++ 2 files changed, 7 insertions(+) diff --git a/src/mesa/drivers/d

Re: [Mesa-dev] TGSI and Tessellation Control Shader outputs

On Mon, Sep 1, 2014 at 12:00 PM, Roland Scheidegger wrote: > Am 29.08.2014 22:44, schrieb Ilia Mirkin: >> Hello, >> >> I've been thinking a bit about how to properly implement TCS outputs >> in TGSI. As a quick reminder, there are per-vertex (i.e. invocation) >> and per-patch outputs in TCS. And w

Re: [Mesa-dev] The long road to 'make dist' (gallium#2)

Am 01.09.2014 17:55, schrieb Emil Velikov: > On 01/09/14 16:31, Roland Scheidegger wrote: >> Am 30.08.2014 01:44, schrieb Emil Velikov: >>> Hello list, >>> >>> Another boring yet important series, if we are to have properly working >>> solution to package a mesa release tarball via 'make dist'. >>>

[Mesa-dev] [PATCH] automake: check if the linker supports --dynamic-list

As older versions of gnu ld did not support --dynamic-list check to see if it is supported before using it. Non gnu linkers such the apple one likely lack this option as well. Fixes the build on OpenBSD which has binutils 2.15 and 2.17. The --dynamic-list option seems to been have introduced some

Re: [Mesa-dev] The long road to 'make dist' (gallium#2)

On Mon, Sep 1, 2014 at 8:31 AM, Roland Scheidegger wrote: > Am 30.08.2014 01:44, schrieb Emil Velikov: >> Hello list, >> >> Another boring yet important series, if we are to have properly working >> solution to package a mesa release tarball via 'make dist'. >> >> This series covers a few state-tr

Re: [Mesa-dev] TGSI and Tessellation Control Shader outputs

Am 29.08.2014 22:44, schrieb Ilia Mirkin: > Hello, > > I've been thinking a bit about how to properly implement TCS outputs > in TGSI. As a quick reminder, there are per-vertex (i.e. invocation) > and per-patch outputs in TCS. And while you can only write to the > current invocation's per-vertex o

Re: [Mesa-dev] The long road to 'make dist' (gallium#2)

On 01/09/14 16:31, Roland Scheidegger wrote: > Am 30.08.2014 01:44, schrieb Emil Velikov: >> Hello list, >> >> Another boring yet important series, if we are to have properly working >> solution to package a mesa release tarball via 'make dist'. >> >> This series covers a few state-tracker files mi

Re: [Mesa-dev] The long road to 'make dist' (gallium#2)

Am 30.08.2014 01:44, schrieb Emil Velikov: > Hello list, > > Another boring yet important series, if we are to have properly working > solution to package a mesa release tarball via 'make dist'. > > This series covers a few state-tracker files missed out previously, > picks up all the gallium te

[Mesa-dev] [PATCH] nv50/ir: avoid creating instructions that can't be emitted

When constant folding a MAD operation, we first fold the multiply and generate an ADD. However we do so without making sure that the immediate can be handled in the saturate case. If it can't, load the immediate in a separate instruction. Reported-by: Tiziano Bacocco Signed-off-by: Ilia Mirkin C

[Mesa-dev] [Bug 83355] FTBFS: src/mesa/program/program_lexer.l:122:64: error: unknown type name 'YYSTYPE'

https://bugs.freedesktop.org/show_bug.cgi?id=83355 --- Comment #3 from Emil Velikov --- (In reply to comment #2) > Thanks Emil, the patch fixes the build again. You can have my > Tested-by: Kai Wasserbäch > Thanks. > With regards to the parallel build: I always build in parallel and consider

[Mesa-dev] [Bug 83355] FTBFS: src/mesa/program/program_lexer.l:122:64: error: unknown type name 'YYSTYPE'

https://bugs.freedesktop.org/show_bug.cgi?id=83355 --- Comment #2 from Kai --- Thanks Emil, the patch fixes the build again. You can have my Tested-by: Kai Wasserbäch With regards to the parallel build: I always build in parallel and consider it a valuable feature, since it speeds up build ti

[Mesa-dev] [Bug 83355] FTBFS: src/mesa/program/program_lexer.l:122:64: error: unknown type name 'YYSTYPE'

https://bugs.freedesktop.org/show_bug.cgi?id=83355 --- Comment #1 from Emil Velikov --- Parallel 'make install' was never tested to work (see the release announcements). With that aside a the following patch [1] will revert the commit and resolve the original issue in another way. Feel free to gi

[Mesa-dev] [Bug 83355] New: FTBFS: src/mesa/program/program_lexer.l:122:64: error: unknown type name 'YYSTYPE'

https://bugs.freedesktop.org/show_bug.cgi?id=83355 Priority: medium Bug ID: 83355 Assignee: mesa-dev@lists.freedesktop.org Summary: FTBFS: src/mesa/program/program_lexer.l:122:64: error: unknown type name 'YYSTYPE' Severity:

[Mesa-dev] [Bug 79662] [DRI3 all Bisected] Many webglc cases fail and X fails on PNV

https://bugs.freedesktop.org/show_bug.cgi?id=79662 Eero Tamminen changed: What|Removed |Added CC||eero.t.tammi...@intel.com -- You are re

[Mesa-dev] [Bug 76188] EGL_EXT_image_dma_buf_import fd ownership is incorrect

https://bugs.freedesktop.org/show_bug.cgi?id=76188 --- Comment #11 from Pekka Paalanen --- Sent to Mesa-stable: http://lists.freedesktop.org/archives/mesa-stable/2014-September/001683.html -- You are receiving this mail because: You are the assignee for the bug.

[Mesa-dev] [PATCH] Revert "mesa: fix make tarballs"

This reverts commit 0fbb9a599df898d4e1166d6d6f00cb34a0524bea. Rather than adding hacks around the issue drop the sources from the final tarball, and re-add them back with 'make dist'. This fixes a problem when running parallel 'make install' fails as it recreates sources and triggers partial recom

Re: [Mesa-dev] [PATCH] mesa: fix fallback texture for cube map array

On Thu, Aug 28, 2014 at 3:36 PM, Roland Scheidegger wrote: > Oops there's a bug in there, I'll send out a new version (we only pass a > one pixel sized array). > FWIW the msaa case also has its issues (generates a warning in > store_texsubimage, I guess that's pretty harmless though at the very >

Re: [Mesa-dev] [PATCH] r600g, radeonsi: Always use GTT again for PIPE_USAGE_STREAM buffers

Perhaps Valley uses scattered writes, which the write-combining cache doesn't like? u_upload_mgr mostly uses sequential writes, so I think it could still benefit from VRAM placement. Just guessing. Does the STREAM flag which causes problems come from Valley (glBufferData) or from Mesa (u_upload_mg

Re: [Mesa-dev] [PATCH] r600g, radeonsi: Inform the kernel if a BO will likely be accessed by the CPU

Reviewed-by: Marek Olšák Marek On Thu, Aug 28, 2014 at 8:56 AM, Michel Dänzer wrote: > From: Michel Dänzer > > This allows the kernel to prevent such BOs from ever being stored in the > CPU inaccessible part of VRAM. > > Signed-off-by: Michel Dänzer > --- > src/gallium/drivers/radeon/r600_bu

Re: [Mesa-dev] [PATCH] mesa: Convert NewDriverState to 64-bits

Reviewed-by: Marek Olšák Marek On Thu, Aug 28, 2014 at 6:44 AM, Jordan Justen wrote: > i965 will have more than 32 bits when BRW_STATE_COMPUTE_PROGRAM is added. > > Signed-off-by: Jordan Justen > --- > Ken, > > How about this to replace "i965: Convert brw state dirty bits to > 64-bits"? > > -J

Re: [Mesa-dev] [PATCH 2/7] radeon/compute: Report a value for PIPE_SHADER_CAP_MAX_CONST_BUFFER_SIZE

PIPE_SHADER_MAX_CONST_BUFFER_SIZE replaced MAX_CONSTS. The former is in bytes, while the latter was in vec4s (16 bytes). Marek On Wed, Aug 27, 2014 at 8:05 AM, Dave Airlie wrote: > I think Tom cc'ed the wrong stable maybe, not sure, > > 10.2 doesn't have PIPE_SHADER_CAP_MAX_CONST_BUFFER_SIZE, so

Re: [Mesa-dev] [PATCH 2/2] radeonsi: Compile dummy pixel shader on demand

On Wed, Aug 27, 2014 at 10:57 AM, Michel Dänzer wrote: > From: Michel Dänzer > > It's never used under normal circumstances. > > Signed-off-by: Michel Dänzer > --- > src/gallium/drivers/radeonsi/si_pipe.c | 7 --- > src/gallium/drivers/radeonsi/si_state.c | 11 ++- > 2 files chang

[Mesa-dev] [PATCH] loader: fds can be 0

Possible resource leak reported by coverity. Reported-by: Coverity scanner. Signed-off-by: Dave Airlie --- src/loader/loader.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/loader/loader.c b/src/loader/loader.c index 47e1f58..bdd3906 100644 --- a/src/loader/loader.c +++

[Mesa-dev] [PATCH 2/2] i965/fs: Don't segfault when debug-logging a null program

--- src/mesa/drivers/dri/i965/brw_fs.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index 43aee66..8b76c3e 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.cpp +++ b/src/mesa/drivers/dri/i965/brw

[Mesa-dev] [PATCH] main: Don't leak temporary texture rows

--- src/mesa/main/texstore.c | 4 1 file changed, 4 insertions(+) diff --git a/src/mesa/main/texstore.c b/src/mesa/main/texstore.c index f2eb0de..4b8158a 100644 --- a/src/mesa/main/texstore.c +++ b/src/mesa/main/texstore.c @@ -1634,6 +1634,8 @@ texstore_via_float(TEXSTORE_PARAMS) }

[Mesa-dev] [PATCH 1/2] i965/vec4: Don't segfault when debug-logging a null program

--- src/mesa/drivers/dri/i965/brw_vec4.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp b/src/mesa/drivers/dri/i965/brw_vec4.cpp index 5f8f399..ebe1216 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4.cpp +++ b/src/mesa/drivers/dri/