[Mesa-dev] [PATCH v2] mesa: fix ARRAY_SIZE query for GetProgramResourceiv

Fixes rest of the failures with ES31-CTS.program_interface_query.no-locations v2: make additional check only for GS inputs Signed-off-by: Tapani Pälli --- src/mesa/main/shader_query.cpp | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/mesa/main/shader_query

Re: [Mesa-dev] [PATCH] vl/mpeg12: Silence GCC unused-variable warning.

On 01.08.2015 20:45, Vinson Lee wrote: vl/vl_mpeg12_bitstream.c: In function 'decode_slice': vl/vl_mpeg12_bitstream.c:928:19: warning: unused variable 'extra' [-Wunused-variable] unsigned extra = vl_vlc_get_uimsbf(&bs->vlc, 1); ^ Signed-off-by: Vinson Lee Revie

Re: [Mesa-dev] [PATCH 4/4] i965/es3.1: Implement glMemoryBarrierByRegion

> -Original Message- > From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On > Behalf Of Matt Turner > Sent: Friday, July 31, 2015 7:48 PM > To: Ilia Mirkin > Cc: mesa-dev@lists.freedesktop.org > Subject: Re: [Mesa-dev] [PATCH 4/4] i965/es3.1: Implement > glMemoryBarrierByRegion

[Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

_mesa_get_program_resource_name has logic to append '[0]' in name if variable is an array, this should be skipped for XFB varyings that have array index already appended. Fixes: ES31-CTS.program_interface_query.transform-feedback-types Signed-off-by: Tapani Pälli --- src/mesa/main/shader_que

Re: [Mesa-dev] [PATCH] gallium/radeon: always use the llvm. prefix in intrinsic names

On 01.08.2015 00:35, Marek Olšák wrote: > From: Marek Olšák [...] > @@ -1176,8 +1176,18 @@ static void emit_frac( > struct lp_build_emit_data * emit_data) > { > LLVMBuilderRef builder = bld_base->base.gallivm->builder; > - > - LLVMValueRef floor = lp_build_intrinsic(buil

Re: [Mesa-dev] [PATCH 2/2] radeon/winsys: increase the IB size for VM

On 01.08.2015 00:16, Marek Olšák wrote: > From: Marek Olšák > > Luckily, there is a kernel query, so use the size from that. > It currently returns 256KB. It can be increased in the kernel. Both patches are Reviewed-by: Michel Dänzer -- Earthling Michel Dänzer |

Re: [Mesa-dev] [PATCH] gallium/radeon: suspend timer queries between IBs

On 01.08.2015 00:15, Marek Olšák wrote: > From: Marek Olšák > > When we are measuring the time spent in a draw call, an unexpected flush > can distort the result. Reviewed-by: Michel Dänzer -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast

Re: [Mesa-dev] [PATCH] nir: Use a single bit for the dual-source blend index

Reviewed-by: Tapani Pälli On 08/02/2015 08:04 AM, Timothy Arceri wrote: The only values allowed are 0 and 1, and the value is checked before assigning. This is a copy of 8eeca7a56c that seems to have been made to the glsl ir type after it was copied for use in nir but before nir landed. ---

Re: [Mesa-dev] [PATCH] glsl: replace old hash table with new and faster one

On Sun, 2015-08-02 at 20:39 -0400, Ilia Mirkin wrote: > Given that this is a debug-only thing, I doubt perf numbers are that > interesting. > > I have no clue what the diff is between the two hash tables, but if > one is allegedly faster than the other, that should be determined, and > we should j

Re: [Mesa-dev] [PATCH] glsl: replace old hash table with new and faster one

On Mon, 2015-08-03 at 03:54 +0200, Alejandro Seguí Díaz wrote: > I'm starting to contribute and Timothy Arceri told me about this > task. The change was made at that sole file because we talked > about no need to migrate all tables at once, but we discuss this > and I take note to migrate all hash

Re: [Mesa-dev] [PATCH] clover: clEnqueue* should block when asked for

EdB writes: > As a side effect, this fix clRetain/ReleaseEvent Piglit test > --- > src/gallium/state_trackers/clover/api/transfer.cpp | 29 > -- > 1 file changed, 27 insertions(+), 2 deletions(-) > > diff --git a/src/gallium/state_trackers/clover/api/transfer.cpp > b/src/ga

Re: [Mesa-dev] [PATCH] glsl: replace old hash table with new and faster one

On Mon, 2015-08-03 at 19:32 +1000, Timothy Arceri wrote: > On Sun, 2015-08-02 at 20:39 -0400, Ilia Mirkin wrote: > > Given that this is a debug-only thing, I doubt perf numbers are that > > interesting. > > > > I have no clue what the diff is between the two hash tables, but if > > one is allegedl

Re: [Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

On 03/08/15 11:10, Tapani Pälli wrote: _mesa_get_program_resource_name has logic to append '[0]' in name if variable is an array, this should be skipped for XFB varyings that have array index already appended. Fixes: ES31-CTS.program_interface_query.transform-feedback-types Signed-off-by: T

Re: [Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

On 08/03/2015 01:48 PM, Martin Peres wrote: On 03/08/15 11:10, Tapani Pälli wrote: _mesa_get_program_resource_name has logic to append '[0]' in name if variable is an array, this should be skipped for XFB varyings that have array index already appended. Fixes: ES31-CTS.program_interface_q

Re: [Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

On 03/08/15 13:50, Tapani Pälli wrote: On 08/03/2015 01:48 PM, Martin Peres wrote: On 03/08/15 11:10, Tapani Pälli wrote: _mesa_get_program_resource_name has logic to append '[0]' in name if variable is an array, this should be skipped for XFB varyings that have array index already appended.

Re: [Mesa-dev] [PATCH v2 00/78] i965: A new vec4 backend based on NIR

On 08/01/2015 02:59 PM, Eduardo Lima Mitev wrote: > On 07/31/2015 10:48 AM, Eduardo Lima Mitev wrote: >> On 07/30/2015 09:48 PM, Jason Ekstrand wrote: >>> >>> On Jul 27, 2015 3:39 PM, "Jason Ekstrand" >> > wrote: On Mon, Jul 27, 2015 at 2:07 PM, Eduardo Lima M

Re: [Mesa-dev] [Mesa-stable] [PATCH] i965/bdw: Fix 3DSTATE_VF_INSTANCING when the edge flag is used

Hi Emil, I posted a v2 of this patch because the v1 wouldn't work in combination with a fix for bug 84677. I think it would be better to wait for review on those two and land both of them instead of this patch. Neither of them are particulary urgent bugs so it's probably not worth worrying about

[Mesa-dev] [PATCH] clover: fix image resource depth and array_size

--- src/gallium/state_trackers/clover/core/memory.cpp | 2 +- src/gallium/state_trackers/clover/core/resource.cpp | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/clover/core/memory.cpp b/src/gallium/state_trackers/clover/core/memory.cpp index 0553

Re: [Mesa-dev] [PATCH 2/3] r600g: fix the CB_SHADER_MASK setup

On 2 August 2015 at 14:36, Marek Olšák wrote: > From: Marek Olšák > > This fixes the single-sample fast clear hang. > > Cc: 10.6 > --- > src/gallium/drivers/r600/evergreen_state.c | 8 > src/gallium/drivers/r600/r600_shader.c | 1 + > 2 files changed, 5 insertions(+), 4 deletions(-

Re: [Mesa-dev] [PATCH] clover: fix image resource depth and array_size

Zoltan Gilian writes: > --- > src/gallium/state_trackers/clover/core/memory.cpp | 2 +- > src/gallium/state_trackers/clover/core/resource.cpp | 1 + > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/gallium/state_trackers/clover/core/memory.cpp > b/src/gallium/state_track

Re: [Mesa-dev] [Mesa-stable] [PATCH] clover: Fix bug with computing hard_event status

On 1 August 2015 at 19:18, Marek Olšák wrote: > On Sat, Aug 1, 2015 at 7:29 PM, Emil Velikov wrote: >> On 1 August 2015 at 17:57, Marek Olšák wrote: >>> No, we don't need this. radeonsi doesn't return a NULL fence anymore. >>> >> Glad to hear. On the other hand is that fix in stable ? I believe

Re: [Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

On 08/03/2015 01:57 PM, Martin Peres wrote: On 03/08/15 13:50, Tapani Pälli wrote: On 08/03/2015 01:48 PM, Martin Peres wrote: On 03/08/15 11:10, Tapani Pälli wrote: _mesa_get_program_resource_name has logic to append '[0]' in name if variable is an array, this should be skipped for XFB va

Re: [Mesa-dev] [PATCH] clover: fix image resource depth and array_size

> Any reason you didn't fix this other branch to do the same? Or maybe > just init it to one after the if? I've only observed problems with images if that line is missing (the surface is rejected because of 0 buffer size), but it makes sense to set the array_size outside the if. On Mon, Aug 3,

Re: [Mesa-dev] [PATCH 1/2] clover: move find_kernels to functions

Meanwhile I've obtained commit permission, so committed as d2cd2c6. On Fri, Jul 31, 2015 at 4:06 PM, Zoltán Gilián wrote: > Could you please commit this? > > On Mon, Jul 27, 2015 at 1:20 PM, Francisco Jerez > wrote: >> Zoltan Gilian writes: >> >>> --- >>> .../state_trackers/clover/llvm/invoca

Re: [Mesa-dev] [PATCH 2/2] clover: pass image attributes to the kernel

Committed as 9ef5b7a. On Fri, Jul 31, 2015 at 4:07 PM, Zoltán Gilián wrote: > Could you please commit this? > > On Mon, Jul 27, 2015 at 1:28 PM, Francisco Jerez > wrote: >> Zoltan Gilian writes: >> >>> Read-only and write-only image arguments are recognized and >>> distinguished. >>> Attribute

Re: [Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

On 03/08/15 14:35, Tapani Pälli wrote: On 08/03/2015 01:57 PM, Martin Peres wrote: On 03/08/15 13:50, Tapani Pälli wrote: On 08/03/2015 01:48 PM, Martin Peres wrote: On 03/08/15 11:10, Tapani Pälli wrote: _mesa_get_program_resource_name has logic to append '[0]' in name if variable is a

Re: [Mesa-dev] [PATCH] clover: fix image resource depth and array_size

Fixed it. Committed as aa46fba. On Mon, Aug 3, 2015 at 1:36 PM, Zoltán Gilián wrote: >> Any reason you didn't fix this other branch to do the same? Or maybe >> just init it to one after the if? > > I've only observed problems with images if that line is missing (the > surface is rejected because

Re: [Mesa-dev] [PATCH] clover: handle setKernelArg errors

Committed as be3622d. On Fri, Jul 31, 2015 at 4:09 PM, Francisco Jerez wrote: > Zoltán Gilián writes: > >> Could you please commit this? I don't have permissions. >> > Sure, I'll put them into my queue. > >> On Fri, Jul 31, 2015 at 3:55 PM, Francisco Jerez >> wrote: >>> Zoltan Gilian writes:

Re: [Mesa-dev] [PATCH] mesa: add missing queries for ARB_direct_state_access

Ping. Can someone please review and/or commit this. I noticed ARB_dsa is already exposed in Mesa 10.6 - should this patch be Cc'd to that? -- Daniel On 2015-07-25 08:12, Daniel Scharrer wrote: > --- > > ARB_dsa / GL 4.5 add indexed GL_TEXTURE_BINDING_* and GL_SAMPLER_BINDING > queries, as wel

Re: [Mesa-dev] [PATCH 1/3] egl/x11: fix use of EGL_BAD_NATIVE_WINDOW

On 31 July 2015 at 13:34, Frank Binns wrote: > On 31/07/15 12:53, Emil Velikov wrote: >> On 31 July 2015 at 11:02, Frank Binns wrote: >>> Commit 4ed23fd590 introduced some calls to _eglError inappropriately >>> passing it EGL_BAD_NATIVE_WINDOW. This was actually harmless in two of the >>> cases a

Re: [Mesa-dev] [PATCH 2/3] r600g: fix the CB_SHADER_MASK setup

On Mon, Aug 3, 2015 at 1:28 PM, Emil Velikov wrote: > On 2 August 2015 at 14:36, Marek Olšák wrote: >> From: Marek Olšák >> >> This fixes the single-sample fast clear hang. >> >> Cc: 10.6 >> --- >> src/gallium/drivers/r600/evergreen_state.c | 8 >> src/gallium/drivers/r600/r600_shader

Re: [Mesa-dev] [Mesa-stable] [PATCH] clover: Fix bug with computing hard_event status

On Mon, Aug 3, 2015 at 1:33 PM, Emil Velikov wrote: > On 1 August 2015 at 19:18, Marek Olšák wrote: >> On Sat, Aug 1, 2015 at 7:29 PM, Emil Velikov >> wrote: >>> On 1 August 2015 at 17:57, Marek Olšák wrote: No, we don't need this. radeonsi doesn't return a NULL fence anymore. >>> Gl

Re: [Mesa-dev] [PATCH 2/3] egl/x11: set EGL_BAD_NATIVE_(PIXMAP|WINDOW) for invalid pixmaps/windows

On 31 July 2015 at 11:02, Frank Binns wrote: > Both eglCreatePixmapSurface and eglCreateWindowSurface were incorrectly > setting the EGL error to be EGL_BAD_ALLOC when an invalid native drawable > handle was being passed in. The EGL spec states the following for > eglCreatePixmapSurface: > >

Re: [Mesa-dev] [PATCH 3/3] egl/x11: don't abort when creating a DRI2 drawable fails

On 31 July 2015 at 11:02, Frank Binns wrote: > When calling either eglCreateWindowSurface or eglCreatePixmapSurface it > was possible for an application to be aborted as a result of it failing > to create a DRI2 drawable on the server. This could happen due to an > application passing in an invali

Re: [Mesa-dev] [Mesa-stable] [PATCH] clover: Fix bug with computing hard_event status

On 3 August 2015 at 13:17, Marek Olšák wrote: > On Mon, Aug 3, 2015 at 1:33 PM, Emil Velikov wrote: >> On 1 August 2015 at 19:18, Marek Olšák wrote: >>> On Sat, Aug 1, 2015 at 7:29 PM, Emil Velikov >>> wrote: On 1 August 2015 at 17:57, Marek Olšák wrote: > No, we don't need this. rad

[Mesa-dev] [PATCH] gallium/radeon: add a debug flag not to use write combining

From: Marek Olšák --- src/gallium/drivers/radeon/r600_buffer_common.c | 6 -- src/gallium/drivers/radeon/r600_pipe_common.c | 1 + src/gallium/drivers/radeon/r600_pipe_common.h | 1 + 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeon/r600_buffer_co

[Mesa-dev] [PATCH 00/11] RadeonSI and Gallium HUD improvements

This improves the HUD for RadeonSI. Hopefully, an introductory message like this makes it easier for Michel not to miss this. Marek ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 11/11] radeonsi: add a HUD query showing the number of shaders created

From: Marek Olšák --- src/gallium/drivers/radeon/r600_pipe_common.c | 2 ++ src/gallium/drivers/radeon/r600_pipe_common.h | 5 + src/gallium/drivers/radeon/r600_query.c | 9 + src/gallium/drivers/radeonsi/si_state_shaders.c | 1 + 4 files changed, 17 insertions(+) diff -

[Mesa-dev] [PATCH 09/11] gallium/radeon: display cumulative results for some driver queries

From: Marek Olšák --- src/gallium/drivers/radeon/r600_pipe_common.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c b/src/gallium/drivers/radeon/r600_pipe_common.c index 116e1f7..8f95444 100644 --- a/src/gallium/drivers/rad

[Mesa-dev] [PATCH 04/11] gallium/hud: replace assertions with clamping the unit index

From: Marek Olšák --- src/gallium/auxiliary/hud/hud_context.c | 42 ++--- 1 file changed, 23 insertions(+), 19 deletions(-) diff --git a/src/gallium/auxiliary/hud/hud_context.c b/src/gallium/auxiliary/hud/hud_context.c index decc055..d3ad271 100644 --- a/src/gallium

[Mesa-dev] [PATCH 10/11] radeonsi: add a HUD query showing the number of compiler invocations

From: Marek Olšák --- src/gallium/drivers/radeon/r600_pipe_common.c | 4 +++- src/gallium/drivers/radeon/r600_pipe_common.h | 6 ++ src/gallium/drivers/radeon/r600_query.c | 9 + src/gallium/drivers/radeonsi/si_state_shaders.c | 1 + 4 files changed, 19 insertions(+), 1 d

[Mesa-dev] [PATCH 02/11] gallium/hud: fix printing byte units

From: Marek Olšák --- src/gallium/auxiliary/hud/hud_context.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/hud/hud_context.c b/src/gallium/auxiliary/hud/hud_context.c index e10152e..e0ca29d 100644 --- a/src/gallium/auxiliary/hud/hud_context.c +++ b/s

[Mesa-dev] [PATCH 06/11] gallium/hud: automatically print % if max_value == 100

From: Marek Olšák --- src/gallium/auxiliary/hud/hud_context.c | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/src/gallium/auxiliary/hud/hud_context.c b/src/gallium/auxiliary/hud/hud_context.c index 275070d..95eed26 100644 --- a/src/gallium/auxiliary/hud/hu

[Mesa-dev] [PATCH 03/11] gallium, hud: allow displaying cumulative values instead of average

From: Marek Olšák The cumulative value is useful for queries like the number of shader compilations. --- src/gallium/auxiliary/hud/hud_context.c | 9 ++--- src/gallium/auxiliary/hud/hud_driver_query.c | 22 ++ src/gallium/auxiliary/hud/hud_private.h | 3 ++- s

[Mesa-dev] [PATCH 01/11] gallium, hud: add support for Hz units in driver queries

From: Marek Olšák --- src/gallium/auxiliary/hud/hud_context.c | 6 ++ src/gallium/include/pipe/p_defines.h| 2 ++ 2 files changed, 8 insertions(+) diff --git a/src/gallium/auxiliary/hud/hud_context.c b/src/gallium/auxiliary/hud/hud_context.c index 4631cd3..e10152e 100644 --- a/src/gall

[Mesa-dev] [PATCH 05/11] gallium/hud: fix printing % next to panes

From: Marek Olšák --- src/gallium/auxiliary/hud/hud_context.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/hud/hud_context.c b/src/gallium/auxiliary/hud/hud_context.c index d3ad271..275070d 100644 --- a/src/gallium/auxiliary/hud/hud_context.c +++ b/s

[Mesa-dev] [PATCH 07/11] gallium/radeon: change some driver query types to Hz

From: Marek Olšák --- src/gallium/drivers/radeon/r600_pipe_common.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c b/src/gallium/drivers/radeon/r600_pipe_common.c index b12992d..e143132 100644 --- a/src/gallium/drivers/radeo

[Mesa-dev] [PATCH 08/11] gallium/radeon: switch the buffer-wait-time query to microseconds

From: Marek Olšák This display the units in the HUD. --- src/gallium/drivers/radeon/r600_pipe_common.c | 2 +- src/gallium/drivers/radeon/r600_query.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c b/src/gallium/drive

[Mesa-dev] [PATCH] winsys/radeon: loosen up the requirements for how much memory IBs can use

From: Marek Olšák --- src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_cs.c b/src/gallium/winsys/radeon/drm/radeon_drm_cs.c index 45eef29..7e5fff4 100644 --- a/src/gallium/win

[Mesa-dev] [PATCH] radeonsi: before storing tess levels, load them from LDS instead of temporary

From: Marek Olšák Also use only one store if stride <= 4. All the fetches from and stores to temporaries can be removed now. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91461 --- src/gallium/drivers/radeonsi/si_shader.c | 136 +-- 1 file changed, 57 insert

Re: [Mesa-dev] [PATCH 06/11] gallium/hud: automatically print % if max_value == 100

Patches 1-6 are: Reviewed-by: Samuel Pitoiset But please, fix the commit message for patches 1 and 3 (ie. gallium/hud instead of gallium, hud). Btw, it would be good to display floating point numbers when percentage is used. What do you think ? On 08/03/2015 02:42 PM, Marek Olšák wrote:

Re: [Mesa-dev] [PATCH 1/2] glsl: add variable mode check to build_stageref

On Mon, 2015-08-03 at 09:02 +0300, Tapani Pälli wrote: > Currently stage reference mask is built using the variable name > only. However it can happen that input of one stage has same name > as output from another stage. Adding check of variable mode makes > sure we do not pick wrong variable. > >

Re: [Mesa-dev] [PATCH] r600, compute: force tiling on 2D and 3D texture compute resources

Committed as 44e90f2. On Fri, Jul 31, 2015 at 12:11 PM, Marek Olšák wrote: > Reviewed-by: Marek Olšák > > Marek > > On Fri, Jul 31, 2015 at 11:59 AM, Zoltan Gilian > wrote: >> To circumvent a problem occuring when LINEAR_ALIGNED array mode is >> selected on a TEXTURE_2D RAT. >> This configurat

Re: [Mesa-dev] [PATCH 06/11] gallium/hud: automatically print % if max_value == 100

Series looks good to me. Reviewed-by: Brian Paul On 08/03/2015 06:42 AM, Marek Olšák wrote: From: Marek Olšák --- src/gallium/auxiliary/hud/hud_context.c | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/src/gallium/auxiliary/hud/hud_context.c b/src/g

Re: [Mesa-dev] [PATCH] mesa: add missing queries for ARB_direct_state_access

On 08/03/2015 06:01 AM, Daniel Scharrer wrote: Ping. Can someone please review and/or commit this. I noticed ARB_dsa is already exposed in Mesa 10.6 - should this patch be Cc'd to that? Probably. -- Daniel On 2015-07-25 08:12, Daniel Scharrer wrote: --- ARB_dsa / GL 4.5 add indexed GL_

Re: [Mesa-dev] [PATCH] mesa: add missing queries for ARB_direct_state_access

Hi, thanks for looking at this. On 2015-08-03 16:18, Brian Paul wrote: > On 08/03/2015 06:01 AM, Daniel Scharrer wrote: >> Ping. >> >> Can someone please review and/or commit this. >> >> I noticed ARB_dsa is already exposed in Mesa 10.6 - should this patch be >> Cc'd to that? > > Probably. > >

Re: [Mesa-dev] [PATCH 05/21] mesa: Resolve GCC sign-compare warning.

On 08/03/2015 12:09 AM, Rhys Kidd wrote: mesa/src/mesa/drivers/dri/common/utils.c: In function 'driGetConfigAttrib': mesa/src/mesa/drivers/dri/common/utils.c:457:19: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] for (i = 0; i < ARRAY_SIZE(attribMap);

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

https://bugs.freedesktop.org/show_bug.cgi?id=91254 --- Comment #10 from di...@gmx.net --- Can someone please look into this issue. On every start of mpv my system freezes for some seconds and every instance of the flashplugin crashes in my browser. I'm always using mpv for watching flashvideos sin

Re: [Mesa-dev] [PATCH v2 00/78] i965: A new vec4 backend based on NIR

On Aug 3, 2015 4:17 AM, "Eduardo Lima Mitev" wrote: > > On 08/01/2015 02:59 PM, Eduardo Lima Mitev wrote: > > On 07/31/2015 10:48 AM, Eduardo Lima Mitev wrote: > >> On 07/30/2015 09:48 PM, Jason Ekstrand wrote: > >>> > >>> On Jul 27, 2015 3:39 PM, "Jason Ekstrand" >>>

Re: [Mesa-dev] [PATCH] mesa: add missing queries for ARB_direct_state_access

On 08/03/2015 08:27 AM, Daniel Scharrer wrote: Hi, thanks for looking at this. On 2015-08-03 16:18, Brian Paul wrote: On 08/03/2015 06:01 AM, Daniel Scharrer wrote: Ping. Can someone please review and/or commit this. I noticed ARB_dsa is already exposed in Mesa 10.6 - should this patch be C

Re: [Mesa-dev] [PATCH 4/4] i965/es3.1: Implement glMemoryBarrierByRegion

On Mon, Aug 3, 2015 at 1:04 AM, Lofstedt, Marta wrote: > However, I don't agree that the driver interface should be removed. > In my opinion, it is a driver specific decision if glMemoryBarrierByRegion > should be > implemented directly on top of glMemoryBarrier or if you want to do some > optimi

Re: [Mesa-dev] [PATCH 06/11] gallium/hud: automatically print % if max_value == 100

On Mon, Aug 3, 2015 at 2:58 PM, Samuel Pitoiset wrote: > Patches 1-6 are: > > Reviewed-by: Samuel Pitoiset > > But please, fix the commit message for patches 1 and 3 (ie. gallium/hud > instead of gallium, hud). > > Btw, it would be good to display floating point numbers when percentage is > used.

Re: [Mesa-dev] [PATCH 1/2] mesa: clear existing swizzle info before bitwise-OR

On 2 August 2015 at 11:37, Oded Gabbay wrote: > This patch fixes a bug in big-endian handling, where the previous > swizzle info wasn't cleared before a new swizzle info was inserted into > the format field using a bitwise-OR operation. > > Signed-off-by: Oded Gabbay > CC: "10.5 10.6" > --- > s

Re: [Mesa-dev] [PATCH 2/2] mesa: _mesa_format_convert should be endian agnostic

Hi Oded, On 2 August 2015 at 11:37, Oded Gabbay wrote: > This patch fixes a bug that is manifested in the read path of mesa when > running on big-endian machines. The effects can be seen when running > piglit sanity test and/or taking a screen capture. > > The bug is caused when _mesa_format_conv

Re: [Mesa-dev] [PATCH v2 00/78] i965: A new vec4 backend based on NIR

On 08/03/2015 04:57 PM, Jason Ekstrand wrote: > > On Aug 3, 2015 4:17 AM, "Eduardo Lima Mitev" > wrote: >> >> On 08/01/2015 02:59 PM, Eduardo Lima Mitev wrote: >> > On 07/31/2015 10:48 AM, Eduardo Lima Mitev wrote: >> >> On 07/30/2015 09:48 PM, Jason Ekstrand wrote: >> >>

Re: [Mesa-dev] [PATCH 06/11] gallium/hud: automatically print % if max_value == 100

On 08/03/2015 05:28 PM, Marek Olšák wrote: On Mon, Aug 3, 2015 at 2:58 PM, Samuel Pitoiset wrote: Patches 1-6 are: Reviewed-by: Samuel Pitoiset But please, fix the commit message for patches 1 and 3 (ie. gallium/hud instead of gallium, hud). Btw, it would be good to display floating point

Re: [Mesa-dev] [PATCH 00/12] bmake inspired fixes

On Fri, Jul 17, 2015 at 11:53 AM, Emil Velikov wrote: > On 17 July 2015 at 19:09, Matt Turner wrote: >> On Fri, Jul 17, 2015 at 10:29 AM, Emil Velikov >> wrote: >>> Hello all, >>> >>> A few days ago I realised that BSD make (bmake) is available in the >>> Archlinux repos, so I decided to give i

Re: [Mesa-dev] [PATCH 3/7] mesa: Replace F_TO_I() with _mesa_lroundevenf().

On Fri, Jul 31, 2015 at 5:46 PM, Roland Scheidegger wrote: >> + */ >> +static inline long >> +_mesa_lroundevenf(float x) >> +{ >> + return lrintf(x); >> +} >> + >> +/** >> + * \brief Rounds \c x to the nearest integer, with ties to the even integer, >> + * and returns the value as a long int. >>

Re: [Mesa-dev] [PATCH 5/7] util: Use SSE intrinsics in _mesa_lroundeven{f, }.

On Fri, Jul 31, 2015 at 6:37 PM, Roland Scheidegger wrote: > Am 01.08.2015 um 03:02 schrieb Matt Turner: >> On Fri, Jul 31, 2015 at 5:50 PM, Roland Scheidegger >> wrote: >>> Am 01.08.2015 um 01:26 schrieb Matt Turner: gcc actually generates this for us now that we use -fno-math-errno (

[Mesa-dev] [PATCH 5/7] util: Use SSE intrinsics in _mesa_lroundeven{f, }.

gcc actually generates this for us now that we use -fno-math-errno (which is weird, since lrintf()/lrint() don't set errno) but clang still does not. Presumably helps MSVC as well. Reduced .text size by 8.5k with gcc before -fno-math-errno. text data bss dec hex filename 49

Re: [Mesa-dev] [PATCH v2 00/78] i965: A new vec4 backend based on NIR

On Mon, Aug 3, 2015 at 8:54 AM, Eduardo Lima Mitev wrote: > On 08/03/2015 04:57 PM, Jason Ekstrand wrote: >> >> On Aug 3, 2015 4:17 AM, "Eduardo Lima Mitev" > > wrote: >>> >>> On 08/01/2015 02:59 PM, Eduardo Lima Mitev wrote: >>> > On 07/31/2015 10:48 AM, Eduardo Lima Mite

Re: [Mesa-dev] [PATCH] glsl: when generating out/inout parameter fixups, do indexing before the call

On Friday, July 31, 2015 04:37:16 PM Chris Forbes wrote: > If the indexing expression involves anything modified during the call, > the fixup code to copy back after the call would use the new values. > > This fixes the cases where the first expression node encountered is > ir_binop_vector_extract

[Mesa-dev] [PATCH 1/2] eglinfo: print client extensions

From: Marek Olšák --- src/egl/opengl/eglinfo.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c index 961fd9c..8e82e73 100644 --- a/src/egl/opengl/eglinfo.c +++ b/src/egl/opengl/eglinfo.c @@ -162,9 +162,12 @@ PrintExtens

[Mesa-dev] [PATCH 0/2] glxinfo fix & eglinfo improvement

Please review. Marek ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 2/2] glxinfo: fix printing core profile extensions

From: Marek Olšák Nobody supports GLX 3.x. Broken by: commit fca824431608101298ff0735db4c09954dcf64ab Author: Julien Isorce Date: Tue May 19 08:35:10 2015 +0100 glxinfo: fix segfault when core profile is unavailable --- src/xdemos/glxinfo.c | 2 +- 1 file changed, 1 insertion(+), 1 del

[Mesa-dev] [PATCH 3/3] radeonsi: add fiji pci id

Reviewed-by: Marek Olšák Reviewed-by: Christian König Reviewed-by: David Zhang Signed-off-by: Alex Deucher --- include/pci_ids/radeonsi_pci_ids.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/pci_ids/radeonsi_pci_ids.h b/include/pci_ids/radeonsi_pci_ids.h index 197ce8e..c01ee2

[Mesa-dev] [PATCH 2/3] radeonsi: add support for FIJI (v4)

v2: incorporate comments from Marek v3: add missing fiji case in winsys init use tonga raster config (double check this) v4: rebase on harvest patch Reviewed-by: Marek Olšák (v3) Reviewed-by: Christian König (v3) Reviewed-by: David Zhang (v3) Signed-off-by: Alex Deucher --- src/gallium/dr

[Mesa-dev] [PATCH 1/3] addrlib: add support for Fiji (v2)

v2: fix tonga chip check Reviewed-by: Marek Olšák Reviewed-by: Christian König Reviewed-by: David Zhang Signed-off-by: Alex Deucher --- src/gallium/winsys/amdgpu/drm/addrlib/r800/ciaddrlib.cpp | 5 + src/gallium/winsys/amdgpu/drm/addrlib/r800/ciaddrlib.h | 1 + src/gallium/winsys/amdgpu

Re: [Mesa-dev] [PATCH 2/2] glxinfo: fix printing core profile extensions

This patch is Reviewed-by: Ian Romanick On 08/03/2015 10:33 AM, Marek Olšák wrote: > From: Marek Olšák > > Nobody supports GLX 3.x. > > Broken by: > > commit fca824431608101298ff0735db4c09954dcf64ab > Author: Julien Isorce > Date: Tue May 19 08:35:10 2015 +0100 > > glxinfo: fix segfa

Re: [Mesa-dev] [PATCH] includes/GL: remove duplicated extension declarations from glx.h

On 08/01/2015 07:52 AM, Emil Velikov wrote: > On 30 July 2015 at 16:44, Ian Romanick wrote: >> On 07/30/2015 07:50 AM, Emil Velikov wrote: >>> On 30 July 2015 at 15:22, Emil Velikov wrote: All three of GLX_NV_float_buffer, GLX_EXT_texture_from_pixmap and GLX_MESA_query_renderer have bee

Re: [Mesa-dev] [PATCH 00/12] bmake inspired fixes

On 3 August 2015 at 17:17, Matt Turner wrote: > On Fri, Jul 17, 2015 at 11:53 AM, Emil Velikov > wrote: >> On 17 July 2015 at 19:09, Matt Turner wrote: >>> On Fri, Jul 17, 2015 at 10:29 AM, Emil Velikov >>> wrote: Hello all, A few days ago I realised that BSD make (bmake) is av

Re: [Mesa-dev] [PATCH 06/11] gallium/hud: automatically print % if max_value == 100

On Mon, Aug 3, 2015 at 2:58 PM, Samuel Pitoiset wrote: > Patches 1-6 are: > > Reviewed-by: Samuel Pitoiset > > But please, fix the commit message for patches 1 and 3 (ie. gallium/hud > instead of gallium, hud). "gallium,hud" means "gallium and gallium/hud". :) I can change it, but that would rem

Re: [Mesa-dev] [PATCH 06/11] gallium/hud: automatically print % if max_value == 100

On 08/03/2015 08:14 PM, Marek Olšák wrote: On Mon, Aug 3, 2015 at 2:58 PM, Samuel Pitoiset wrote: Patches 1-6 are: Reviewed-by: Samuel Pitoiset But please, fix the commit message for patches 1 and 3 (ie. gallium/hud instead of gallium, hud). "gallium,hud" means "gallium and gallium/hud".

[Mesa-dev] [PATCH] amdgpu: add Fiji support

Signed-off-by: Alex Deucher --- This should be 1/16 speed model. Not sure how to adjust that. Tom? lib/Target/AMDGPU/Processors.td | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/Target/AMDGPU/Processors.td b/lib/Target/AMDGPU/Processors.td index c0ffede..0eac121 100644 --- a/lib/Ta

Re: [Mesa-dev] [PATCH 1/2] glsl: add variable mode check to build_stageref

On Mon, 2015-08-03 at 23:16 +1000, Timothy Arceri wrote: > On Mon, 2015-08-03 at 09:02 +0300, Tapani Pälli wrote: > > Currently stage reference mask is built using the variable name > > only. However it can happen that input of one stage has same name > > as output from another stage. Adding check

[Mesa-dev] [PATCH] mesa: handle no-op cases sooner in _mesa_[Client]ActiveTexture()

If the new texture unit is the current texture unit, we can return before error checking. --- src/mesa/main/texstate.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/mesa/main/texstate.c b/src/mesa/main/texstate.c index 1e75e0a..9b5928c 100644 --- a/

[Mesa-dev] [PATCH] st/mesa: implement query pause/resume for meta operations

glClear, blitting, mipmap generation, etc. implemented with quad drawing should not effect queries like occlusion count. To accomplish that, this patch implements a simple pause/resume mechanism for active queries. We now keep a list of active queries. When we "pause" queries we actually end the

Re: [Mesa-dev] [PATCH] st/mesa: implement query pause/resume for meta operations

I think I brought this up recently (perhaps this is even in response to my comments about this)... my suggestion was to introduce a draw flag that says whether the draw should be counted. mipmap (and blit) should be using pipe->blit, which should never get counted anyways, so all that's left are c

Re: [Mesa-dev] [PATCH 5/7] util: Use SSE intrinsics in _mesa_lroundeven{f, }.

Am 03.08.2015 um 18:43 schrieb Matt Turner: > gcc actually generates this for us now that we use -fno-math-errno > (which is weird, since lrintf()/lrint() don't set errno) but clang still > does not. Presumably helps MSVC as well. > > Reduced .text size by 8.5k with gcc before -fno-math-errno. >

Re: [Mesa-dev] [PATCH 5/5] vc4: add missing nir include, to fix the build

Emil Velikov writes: > On 17/07/15 18:17, Emil Velikov wrote: >> Cc: 10.6 >> Cc: Eric Anholt >> Signed-off-by: Emil Velikov >> --- >> src/gallium/drivers/vc4/Makefile.am | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/src/gallium/drivers/vc4/Makefile.am >> b/src/gallium/drivers/

[Mesa-dev] [PATCH 2/2] nir: Don't try to scalarize unpack ops.

--- src/glsl/nir/nir_lower_alu_to_scalar.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/glsl/nir/nir_lower_alu_to_scalar.c b/src/glsl/nir/nir_lower_alu_to_scalar.c index 5d15fb2..96c84e8 100644 --- a/src/glsl/nir/nir_lower_alu_to_scalar.c +++ b/src/glsl/nir/nir_lower_alu_to_scala

[Mesa-dev] [PATCH 1/2] nir: Add algebraic opt for no-op iand.

I lazily generated some of these in VC4 NIR lowering. --- src/glsl/nir/nir_opt_algebraic.py | 1 + 1 file changed, 1 insertion(+) diff --git a/src/glsl/nir/nir_opt_algebraic.py b/src/glsl/nir/nir_opt_algebraic.py index 3068445..d7c1740 100644 --- a/src/glsl/nir/nir_opt_algebraic.py +++ b/src/gls

Re: [Mesa-dev] [PATCH] st/mesa: implement query pause/resume for meta operations

My understanding was that pipe->clear, blit, and resource_copy_region should not increment query values (except for TIME_ELAPSED). Currently, all radeon drivers implement this in the same way as this patch does (it's done more efficiently though), but I've been meaning to get rid of that and just t

[Mesa-dev] [PATCH 3/5] i965/vec4-nir: Handle boolean resolvese on ILK-

The analysis code was already there and running, we just weren't doing anything with the result of it yet. --- src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 13 + 1 file changed, 13 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp b/src/mesa/drivers/dri/i965/brw_v

[Mesa-dev] [PATCH 0/5] i965/vec4_nir: Various fixes for ILK- and BDW+

This is a quick little series that fixes a bunch of piglit regressions with vec4 NIR on ILK- and BDW+. With this series, vec4 NIR has (I think) zero regressions versus non-NIR on all intel platforms. I haven't yet tested NIR with vec4 vertex shaders on BDW, but I think it's probably ok. We shoul

[Mesa-dev] [PATCH 2/5] i965/nir: Don't mark bany or ball instructions for resolve

--- .../dri/i965/brw_nir_analyze_boolean_resolves.c| 23 ++ 1 file changed, 23 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_nir_analyze_boolean_resolves.c b/src/mesa/drivers/dri/i965/brw_nir_analyze_boolean_resolves.c index d1fc06d..c995d2b 100644 --- a/src/me

[Mesa-dev] [PATCH 4/5] i965/vec4_nir: Do boolean source modifier resolves on BDW+

On BDW+, the negation source modifier on NOT, AND, OR, and XOR, is actually a boolean negate and not an integer negate. However, NIR's soruce modifiers are the integer version. We have to resolve it with a MOV prior to emitting the actual instruction. This is basically the same thing we do in th

[Mesa-dev] [PATCH 1/5] i965/nir: Use nir_op_info.output_type for determining when to resolve

Previously, we were explicitly listing every instruction that needs a resolve. However, those instructions were precicely the ones that returned booleans so there's no reason why we shouldn't just have that check. Also, all of the reduction opcodes such as bany and ball were missing so it didn't

[Mesa-dev] [PATCH 5/5] i965/vec4_nir: Properly handle integer multiplies on BDW+

--- src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 52 -- 1 file changed, 28 insertions(+), 24 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp index 1b7fb5e..7251ca0 100644 --- a/src/mesa/drivers/dri/i965

[Mesa-dev] [Bug 90264] [Regression, bisected] Tooltip corruption in Chrome

https://bugs.freedesktop.org/show_bug.cgi?id=90264 --- Comment #43 from Antoine Labour --- I see, the XResizeWindow would cause a DRI2 Invalidate event on the server side, and the XSync would make sure it's handled on the client before anything else happens. That seems like it would do the right

  1   2   >