Re: [Mesa-dev] [PATCH 29/29] vbo: Use a bitmask to track the active arrays in vbo_save*.

On Monday, June 20, 2016 11:42:19 Mark Janes wrote: > Mathias Fröhlich writes: > > > On Monday, June 20, 2016 10:33:42 Mark Janes wrote: > >> mathias.froehl...@gmx.net writes: > >> > >> > From: Mathias Fröhlich > >> > > >> > The use of a

Re: [Mesa-dev] [PATCH] if if is is, then then the the do do for for an a

On Wed, Jun 22, 2016 at 10:05 PM, Giuseppe Bilotta wrote: > Miscellaneous repetitions in commits and fly-by typo fixes. How about an actual commit title? ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

[Mesa-dev] [PATCH] if if is is, then then the the do do for for an a

Miscellaneous repetitions in commits and fly-by typo fixes. Signed-off-by: Giuseppe Bilotta --- include/GL/mesa_glinterop.h | 6 +++--- src/compiler/glsl/glsl_to_nir.cpp | 2 +- src/compiler/nir/nir.h

[Mesa-dev] [Bug 96573] Micro freezes resulting in crash

https://bugs.freedesktop.org/show_bug.cgi?id=96573 Kenneth Graunke changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop.

Re: [Mesa-dev] [PATCH] i965: Avoid division by zero.

On Wednesday, June 22, 2016 7:05:04 PM PDT Matt Turner wrote: > From: Ardinartsev Nikita > > Fixes regression introduced by af5ca43f2676bff7499f93277f908b681cb821d0 > > Reviewed-by: Matt Turner > Bugzilla:

Re: [Mesa-dev] [PATCH 2/2] clover: fix getting struct args api size

Jan Vesely writes: > On Wed, 2016-06-22 at 17:07 -0700, Francisco Jerez wrote: >> Jan Vesely writes: >> >> > On Mon, 2016-06-13 at 17:24 -0700, Francisco Jerez wrote: >> > > Serge Martin writes: >> > > >> > > > This fix

Re: [Mesa-dev] [PATCH 1/2] radeonsi: emit 1/sqrt for RSQ

On 22.06.2016 22:13, Marek Olšák wrote: > From: Marek Olšák > > We don't need the clamped version and we don't have to use any intrinsic. > > Stats on Tonga: > > 15382 shaders in 9128 tests > Totals: > SGPRS: 1230560 -> 1230560 (0.00 %) > VGPRS: 469577 -> 462504 (-1.51 %)

Re: [Mesa-dev] [PATCH 2/4] anv: use cache uuid based on the build timestamp.

On 22.06.2016 21:04, Emil Velikov wrote: > From: Emil Velikov > > Do not rely on the git sha1: > - its current truncated form makes it less unique > - it does not attribute for local (Vulkand or otherwise) changes > > Use a timestamp produced at the time of build.

Re: [Mesa-dev] [PATCH 1/6] radeonsi: extract IB and bo list saving into separate functions

On Wed, Jun 22, 2016 at 6:01 AM, Michael Schellenberger Costa wrote: > Hi Nicolai > > Am 22.06.2016 um 11:40 schrieb Nicolai Hähnle: >> From: Nicolai Hähnle >> >> --- >> src/gallium/drivers/radeon/r600_pipe_common.c | 53 >>

[Mesa-dev] [PATCH] i965: Avoid division by zero.

From: Ardinartsev Nikita Fixes regression introduced by af5ca43f2676bff7499f93277f908b681cb821d0 Reviewed-by: Matt Turner Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95419 --- src/mesa/drivers/dri/i965/gen7_urb.c | 26

Re: [Mesa-dev] [PATCH 1/2] i965/gen4: Pull texture formats from the texture object not the miptree

On 22 June 2016 at 15:07, Jason Ekstrand wrote: > This makes texture views sort-of work. It doesn't add full texture view > support for gen4-5 but it is enough to fix the GL_ARB_copy_image formats > piglit test on Iron Lake. > > Signed-off-by: Jason Ekstrand

Re: [Mesa-dev] [PATCH 1/6] radeonsi: extract IB and bo list saving into separate functions

Hi Nicolai Am 22.06.2016 um 11:40 schrieb Nicolai Hähnle: > From: Nicolai Hähnle > > --- > src/gallium/drivers/radeon/r600_pipe_common.c | 53 > +++ > src/gallium/drivers/radeon/r600_pipe_common.h | 12 ++ >

[Mesa-dev] [PATCH 2/2] clover: Constify llvm::Function handling

Signed-off-by: Jan Vesely --- This can be squashed with the previous one .../state_trackers/clover/llvm/invocation.cpp | 28 ++ 1 file changed, 13 insertions(+), 15 deletions(-) diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp

[Mesa-dev] [PATCH 1/2] clover: Fix kernel metadata retrieval after clang r273425

Signed-off-by: Jan Vesely --- .../state_trackers/clover/llvm/invocation.cpp | 35 +++--- 1 file changed, 31 insertions(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp

Re: [Mesa-dev] [PATCH 1/2] i965: Combine 3DSTATE_STREAMOUT emitters and genX_sol_state atoms.

On Wednesday, June 22, 2016 5:21:34 PM PDT Jason Ekstrand wrote: > On Wed, Jun 22, 2016 at 4:26 PM, Kenneth Graunke > wrote: > > > They're basically the same. Let's avoid the code duplication. > > > > Signed-off-by: Kenneth Graunke > > --- > >

Re: [Mesa-dev] [PATCH 2/2] clover: fix getting struct args api size

On Wed, 2016-06-22 at 17:07 -0700, Francisco Jerez wrote: > Jan Vesely writes: > > > On Mon, 2016-06-13 at 17:24 -0700, Francisco Jerez wrote: > > > Serge Martin writes: > > > > > > > This fix getting the size of a struct arg. vec3 types still > > >

Re: [Mesa-dev] [PATCH 2/6] glsl: Split arrays even in the presence of whole-array copies.

On Wednesday, June 22, 2016 2:23:00 PM PDT Timothy Arceri wrote: > On Tue, 2016-06-21 at 20:02 -0700, Kenneth Graunke wrote: > > Previously, we failed to split constant arrays. Code such as > > > >int[2] numbers = int[](1, 2); > > > > would generates a whole-array assignment: > > > >

Re: [Mesa-dev] [PATCH 1/2] i965: Combine 3DSTATE_STREAMOUT emitters and genX_sol_state atoms.

On Wed, Jun 22, 2016 at 4:26 PM, Kenneth Graunke wrote: > They're basically the same. Let's avoid the code duplication. > > Signed-off-by: Kenneth Graunke > --- > src/mesa/drivers/dri/i965/brw_state.h| 2 +- >

Re: [Mesa-dev] [PATCH 2/2] clover: fix getting struct args api size

Jan Vesely writes: > On Mon, 2016-06-13 at 17:24 -0700, Francisco Jerez wrote: >> Serge Martin writes: >> >> > This fix getting the size of a struct arg. vec3 types still work >> > ok. >> > Only buit-in args need to have power of two alignment, >> >

[Mesa-dev] [PATCH 1/2] i965: Combine 3DSTATE_STREAMOUT emitters and genX_sol_state atoms.

They're basically the same. Let's avoid the code duplication. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_state.h| 2 +- src/mesa/drivers/dri/i965/brw_state_upload.c | 2 +- src/mesa/drivers/dri/i965/gen7_sol_state.c | 32 --

[Mesa-dev] [PATCH 2/2] i965: Implement rasterizer discard via SOL unless required for queries.

We currently use CL_INVOCATION_COUNT for the GL_PRIMITIVES_GENERATED query, which involves passing all primitives to the clipper. When rasterizer discard is enabled, we program the clipper in REJECT_ALL mode, rather than using the SOL stage's "Rendering Disable" feature. See commit

Re: [Mesa-dev] [PATCH 2/2] clover: fix getting struct args api size

On Mon, 2016-06-13 at 17:24 -0700, Francisco Jerez wrote: > Serge Martin writes: > > > This fix getting the size of a struct arg. vec3 types still work > > ok. > > Only buit-in args need to have power of two alignment, > > getTypeAllocSize > > reports the correct size. > >

Re: [Mesa-dev] [PATCH 06/11] anv/cmd_buffer: Set depth/stencil extent based on the image

On Fri 17 Jun 2016, Jason Ekstrand wrote: > It used to be based on the framebuffer which isn't quite right. > > Signed-off-by: Jason Ekstrand > Cc: Chad Versace > Cc: "12.0" Yes, please. Reviewed-by: Chad Versace

Re: [Mesa-dev] [PATCH 08/64] i965/blorp: Only set src_z for gen8+ 3D textures

On Tue 21 Jun 2016, Pohjolainen, Topi wrote: > On Mon, Jun 20, 2016 at 08:52:50PM -0700, Kenneth Graunke wrote: > > On Friday, June 17, 2016 4:55:41 PM PDT Jason Ekstrand wrote: > > > On Thu, Jun 16, 2016 at 10:08 AM, Chad Versace > > > wrote: > > > > > > > On Sat 11 Jun

Re: [Mesa-dev] [BRANCH] Gallium Radeon cleanups

On Thu, Jun 23, 2016 at 12:16 AM, Vedran Miletić wrote: > On 06/22/2016 08:49 PM, Marek Olšák wrote: >> >> Hi, >> >> The top 6 patches in this branch are cleanups I'd like to push. I'm >> not sending them to the list because they are kinda boring. (and there >> is another

Re: [Mesa-dev] [PATCH 0/6] Gallium multithreaded queue - cleanup and multiple threads per queue

On Thu, Jun 23, 2016 at 12:29 AM, Marek Olšák wrote: > On Tue, Jun 21, 2016 at 4:47 PM, Nicolai Hähnle wrote: >> On 21.06.2016 14:17, Marek Olšák wrote: >>> >>> Hi, >>> >>> This improves u_queue to be more usable in more cases. >>> >>> With this, the size of

Re: [Mesa-dev] [PATCH 0/6] Gallium multithreaded queue - cleanup and multiple threads per queue

On Tue, Jun 21, 2016 at 4:47 PM, Nicolai Hähnle wrote: > On 21.06.2016 14:17, Marek Olšák wrote: >> >> Hi, >> >> This improves u_queue to be more usable in more cases. >> >> With this, the size of the queue (maximum number of jobs waiting) is >> configurable as well as the

Re: [Mesa-dev] [PATCH 1/4] clover: conditionally use MESA_GIT_SHA1

On 06/22/2016 02:04 PM, Emil Velikov wrote: From: Emil Velikov Considering how hard/annoying it was for many peoples' workflow to properly generate the macro, it will be demoted to conditionally available with follow-up commits. Cc:

Re: [Mesa-dev] [BRANCH] Gallium Radeon cleanups

On 06/22/2016 08:49 PM, Marek Olšák wrote: Hi, The top 6 patches in this branch are cleanups I'd like to push. I'm not sending them to the list because they are kinda boring. (and there is another series with 6 patches already) https://cgit.freedesktop.org/~mareko/mesa/log/?h=radeon-cleanups

[Mesa-dev] [PATCH] radeonsi: keep using v_rcp_f32 for division in future LLVM

From: Marek Olšák --- src/gallium/drivers/radeon/radeon_llvm.h | 3 +++ .../drivers/radeon/radeon_setup_tgsi_llvm.c| 28 -- 2 files changed, 29 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeon/radeon_llvm.h

Re: [Mesa-dev] [PATCH 3/3] radeon/vce: use vce structures for vce_52 firmware

OK, so I added get parameters call for each firmware versions, and moved all the value assignments to firmware specific file. As a result, changes made for specific version won't affect other version. For future firmware verison, we still can use the same structure but assign different values

Re: [Mesa-dev] [PATCH 2/3] vl: add parameters for VAAPI encode

Agree, please see the new patch set I just sent. -Original Message- From: Christian König [mailto:deathsim...@vodafone.de] Sent: June-22-16 3:29 AM To: Zhang, Boyuan; mesa-dev@lists.freedesktop.org Subject: Re: [PATCH 2/3] vl: add parameters for VAAPI encode Am 21.06.2016 um 16:50

[Mesa-dev] [PATCH 1/4] radeon/vce: add vce structures

Signed-off-by: Boyuan Zhang --- src/gallium/drivers/radeon/radeon_vce.h | 297 1 file changed, 297 insertions(+) diff --git a/src/gallium/drivers/radeon/radeon_vce.h b/src/gallium/drivers/radeon/radeon_vce.h index e438148..da61285 100644

[Mesa-dev] [PATCH 3/4] vl: add parameters for VAAPI encode

Signed-off-by: Boyuan Zhang --- src/gallium/include/pipe/p_video_state.h | 12 1 file changed, 12 insertions(+) diff --git a/src/gallium/include/pipe/p_video_state.h b/src/gallium/include/pipe/p_video_state.h index d353be6..2f2a4d9 100644 ---

[Mesa-dev] [PATCH 2/4] radeon/vce: use vce structure for vce 52 firmware

Signed-off-by: Boyuan Zhang --- src/gallium/drivers/radeon/radeon_vce.c| 4 + src/gallium/drivers/radeon/radeon_vce.h| 17 + src/gallium/drivers/radeon/radeon_vce_40_2_2.c | 4 + src/gallium/drivers/radeon/radeon_vce_50.c | 4 +

[Mesa-dev] [PATCH 4/4] radeon/vce: handle newly added parameters

Signed-off-by: Boyuan Zhang --- src/gallium/drivers/radeon/radeon_vce_52.c | 107 + 1 file changed, 79 insertions(+), 28 deletions(-) diff --git a/src/gallium/drivers/radeon/radeon_vce_52.c b/src/gallium/drivers/radeon/radeon_vce_52.c index

Re: [Mesa-dev] [PATCH v2 05/14] swr: [rasterizer jitter] cleanup supporting different llvm versions

On 22 June 2016 at 21:01, Rowley, Timothy O wrote: > >> On Jun 22, 2016, at 2:27 PM, Emil Velikov wrote: >> >> On 22 June 2016 at 20:19, Rowley, Timothy O >> wrote: >>> On Jun 22, 2016, at 1:52 PM, Emil

Re: [Mesa-dev] [PATCH] st/mesa: expose EXT_vertex_array_bgra when supported by backend

Thanks for the reviews.. I only need someone to push this patch :) 2016-06-20 16:46 GMT+02:00 Marek Olšák : > Reviewed-by: Marek Olšák > > Marek > > On Mon, Jun 20, 2016 at 9:44 AM, Christian Gmeiner > wrote: >> Signed-off-by:

Re: [Mesa-dev] [PATCH v2 2/4] anv: use cache uuid based on the build timestamp.

On 22 June 2016 at 21:02, Jason Ekstrand wrote: >> @@ -131,7 +131,14 @@ anv_entrypoints.c : anv_entrypoints_gen.py >> $(vulkan_include_HEADERS) >> $(AM_V_GEN) cat $(vulkan_include_HEADERS) |\ >> $(PYTHON2) $(srcdir)/anv_entrypoints_gen.py code > $@ >> >>

[Mesa-dev] [PATCH v2] r600g: move PA_SU_POLY_OFFSET_DB_FMT_CNTL to poly offset states for evergreen

Emit PA_SU_POLY_OFFSET_DB_FMT_CNTL with the other poly_offset states. This will be useful to implement PIPE_CAP_POLYGON_OFFSET_UNITS_UNSCALED. v2: Increase the num_dw field for the poly offset atom Signed-off-by: Axel Davy --- src/gallium/drivers/r600/evergreen_state.c | 38

[Mesa-dev] [PATCH v2] r600g: move PA_SU_POLY_OFFSET_DB_FMT_CNTL to poly offset states for r600

Emit PA_SU_POLY_OFFSET_DB_FMT_CNTL with the other poly_offset states. This will be useful to implement PIPE_CAP_POLYGON_OFFSET_UNITS_UNSCALED. v2: Increase the num_dw field for the poly offset atom Signed-off-by: Axel Davy --- src/gallium/drivers/r600/r600_state.c | 37

Re: [Mesa-dev] [Mesa-stable] [PATCH 4/4] swr: automake: don't ship LLVM version specific generated sources

On 22 June 2016 at 20:44, Rowley, Timothy O wrote: > Couple of minor comments inlined below. > > Tested-by: Tim Rowley > Great, thanks. >> On Jun 22, 2016, at 7:04 AM, Emil Velikov wrote: >> >> From: Emil Velikov

Re: [Mesa-dev] [PATCH v2 2/4] anv: use cache uuid based on the build timestamp.

On Wed, Jun 22, 2016 at 11:25 AM, Emil Velikov wrote: > From: Emil Velikov > > Do not rely on the git sha1: > - its current truncated form makes it less unique > - it does not attribute for local (Vulkand or otherwise) changes > > Use a

Re: [Mesa-dev] [PATCH v2 05/14] swr: [rasterizer jitter] cleanup supporting different llvm versions

> On Jun 22, 2016, at 2:27 PM, Emil Velikov wrote: > > On 22 June 2016 at 20:19, Rowley, Timothy O > wrote: >> >>> On Jun 22, 2016, at 1:52 PM, Emil Velikov wrote: >>> >>> On 20 June 2016 at 22:36, Tim Rowley

Re: [Mesa-dev] [Mesa-stable] [PATCH 4/4] swr: automake: don't ship LLVM version specific generated sources

Couple of minor comments inlined below. Tested-by: Tim Rowley > On Jun 22, 2016, at 7:04 AM, Emil Velikov wrote: > > From: Emil Velikov > > Otherwise things will fail to build, if the builder is using another

Re: [Mesa-dev] [BRANCH] Gallium Radeon cleanups

On Wed, Jun 22, 2016 at 2:49 PM, Marek Olšák wrote: > Hi, > > The top 6 patches in this branch are cleanups I'd like to push. I'm > not sending them to the list because they are kinda boring. (and there > is another series with 6 patches already) > >

Re: [Mesa-dev] [PATCH v2 05/14] swr: [rasterizer jitter] cleanup supporting different llvm versions

On 22 June 2016 at 20:19, Rowley, Timothy O wrote: > >> On Jun 22, 2016, at 1:52 PM, Emil Velikov wrote: >> >> On 20 June 2016 at 22:36, Tim Rowley wrote: >>> --- >>>

Re: [Mesa-dev] [PATCH v2] gallium: detect avx512 cpu features

On May 26, 2016, at 3:06 PM, Tim Rowley > wrote: v2: style code, add avx512 to cpu dump --- src/gallium/auxiliary/util/u_cpu_detect.c | 26 ++ src/gallium/auxiliary/util/u_cpu_detect.h | 10 ++ 2 files

Re: [Mesa-dev] [PATCH v2 05/14] swr: [rasterizer jitter] cleanup supporting different llvm versions

> On Jun 22, 2016, at 1:52 PM, Emil Velikov wrote: > > On 20 June 2016 at 22:36, Tim Rowley wrote: >> --- >> .../drivers/swr/rasterizer/jitter/JitManager.cpp | 9 +-- >> .../drivers/swr/rasterizer/jitter/JitManager.h | 7 - >>

Re: [Mesa-dev] [PATCH 1/2] i965/gen4: Pull texture formats from the texture object not the miptree

On Wednesday, June 22, 2016 12:07:05 PM PDT Jason Ekstrand wrote: > This makes texture views sort-of work. It doesn't add full texture view > support for gen4-5 but it is enough to fix the GL_ARB_copy_image formats > piglit test on Iron Lake. > > Signed-off-by: Jason Ekstrand

Re: [Mesa-dev] [Mesa-stable] [PATCH v2 01/64] i965: Drop the maximum 3D texture size to 512 on Sandy Bridge

This patch is Reviewed-by: Ian Romanick On 06/21/2016 03:45 PM, Jason Ekstrand wrote: > The RenderTargetViewExtent field of RENDER_SURFACE_STATE is supposed to be > set to the depth of a 3-D texture when rendering. Unfortunatley, that > field is only 9 bits on Sandy

[Mesa-dev] [PATCH 2/2] i965/gen4-6: Handle gl_texture_object::BaseLevel and MinLayer correctly

This is basically a direct translation of what we do for gen7. Signed-off-by: Jason Ekstrand Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=83036 Cc: "11.1 11.2 12.0" --- src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 4 +++-

[Mesa-dev] [PATCH 1/2] i965/gen4: Pull texture formats from the texture object not the miptree

This makes texture views sort-of work. It doesn't add full texture view support for gen4-5 but it is enough to fix the GL_ARB_copy_image formats piglit test on Iron Lake. Signed-off-by: Jason Ekstrand Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=83036 Cc: "11.1

Re: [Mesa-dev] [PATCH v2 05/14] swr: [rasterizer jitter] cleanup supporting different llvm versions

On 20 June 2016 at 22:36, Tim Rowley wrote: > --- > .../drivers/swr/rasterizer/jitter/JitManager.cpp | 9 +-- > .../drivers/swr/rasterizer/jitter/JitManager.h | 7 - > .../drivers/swr/rasterizer/jitter/blend_jit.cpp| 8 +- >

[Mesa-dev] [BRANCH] Gallium Radeon cleanups

Hi, The top 6 patches in this branch are cleanups I'd like to push. I'm not sending them to the list because they are kinda boring. (and there is another series with 6 patches already) https://cgit.freedesktop.org/~mareko/mesa/log/?h=radeon-cleanups Commits: gallium/radeon: use

Re: [Mesa-dev] [PATCH v2] swr: push/pop DEBUG macro around llvm includes

Reviewed-by: Bruce Cherniak > On Jun 17, 2016, at 11:01 AM, Tim Rowley wrote: > > llvm redefines DEBUG; adding push/pop prevents a undefined reference > to debug_refcnt_state in llvm-3.7+. > > v2: add undef DEBUG > --- >

Re: [Mesa-dev] [PATCH v2 00/14] update swr rasterizer

Reviewed-by: Bruce Cherniak > On Jun 20, 2016, at 4:36 PM, Tim Rowley wrote: > > v2: > add conservativeRast.h to Makefile.sources > minimize changes in llvm support cleanup > remove tabs that were added by the v1 patches >

[Mesa-dev] [PATCH 4/6] gallium/radeon: add state setup for a separate DCC buffer

From: Marek Olšák --- src/gallium/drivers/radeon/r600_pipe_common.h | 8 src/gallium/drivers/radeon/r600_texture.c | 18 +++--- src/gallium/drivers/radeonsi/si_descriptors.c | 11 ++- src/gallium/drivers/radeonsi/si_state.c | 9

[Mesa-dev] [PATCH 1/6] gallium/radeon: add flag R600_QUERY_HW_FLAG_BEGIN_RESUMES

From: Marek Olšák --- src/gallium/drivers/radeon/r600_query.c | 3 ++- src/gallium/drivers/radeon/r600_query.h | 2 ++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeon/r600_query.c b/src/gallium/drivers/radeon/r600_query.c index

[Mesa-dev] [PATCH 2/6] radeonsi: unreference framebuffer state with set_framebuffer_state

From: Marek Olšák --- src/gallium/drivers/radeonsi/si_pipe.c | 7 +-- src/gallium/drivers/radeonsi/si_state.c | 2 +- src/gallium/drivers/radeonsi/si_state.h | 1 - 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_pipe.c

[Mesa-dev] [PATCH 5/6] gallium/radeon: add a heuristic dynamically enabling DCC for scanout surfaces

From: Marek Olšák DCC for displayable surfaces is allocated in a separate buffer and is enabled or disabled based on PS invocations from 2 frames ago (to let queries go idle) and the number of slow clears from the current frame. At least an equivalent of 5 fullscreen draws

[Mesa-dev] [PATCH 6/6] gallium/radeon: add a HUD query for PS draw ratio stats from separate DCC

From: Marek Olšák --- src/gallium/drivers/radeon/r600_pipe_common.h | 1 + src/gallium/drivers/radeon/r600_query.c | 5 + src/gallium/drivers/radeon/r600_query.h | 1 + src/gallium/drivers/radeon/r600_texture.c | 1 + 4 files changed, 8 insertions(+)

[Mesa-dev] [PATCH 3/6] radeonsi: always calculate DCC info even if it's not used immediately

From: Marek Olšák for a later use --- src/gallium/drivers/radeon/r600_texture.c | 3 ++- src/gallium/winsys/amdgpu/drm/amdgpu_surface.c | 1 - 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeon/r600_texture.c

[Mesa-dev] [PATCH 0/6] RadeonSI: DCC for displayable surfaces

Hi, This series allows enabling DCC for displayable (currently all window) surfaces depending on the circumstances. There is a heuristic that determines whether it pays off based on the pipeline statistics query, because DCC decompression is expensive and can outweigh the benefit of DCC. Most of

[Mesa-dev] [PATCH v2 2/4] anv: use cache uuid based on the build timestamp.

From: Emil Velikov Do not rely on the git sha1: - its current truncated form makes it less unique - it does not attribute for local (Vulkand or otherwise) changes Use a timestamp produced at the time of build. It's perfectly unique, unless someone explicitly

Re: [Mesa-dev] [PATCH 2/4] anv: use cache uuid based on the build timestamp.

On 22 June 2016 at 17:07, Eric Engestrom wrote: > On Wed, Jun 22, 2016 at 04:57:39PM +0100, Eric Engestrom wrote: >> On Wed, Jun 22, 2016 at 01:04:41PM +0100, Emil Velikov wrote: >> > From: Emil Velikov > > [...] > >> > diff --git

[Mesa-dev] [Bug 96629] dEQP-GLES2.functional.texture.completeness.cube.not_positive_level_0: Assertion `width >= 1' failed.

https://bugs.freedesktop.org/show_bug.cgi?id=96629 Ilia Mirkin changed: What|Removed |Added CC||nhaeh...@gmail.com

[Mesa-dev] [Bug 96634] Mesa or libGL very slow rendering with kabini vga card

https://bugs.freedesktop.org/show_bug.cgi?id=96634 Alex Deucher changed: What|Removed |Added Component|Mesa core |DRM/Radeon

[Mesa-dev] [Bug 96639] st/mesa: transfer_map with too-high level with dEQP-GLES2.functional.texture.completeness.cube.extra_level

https://bugs.freedesktop.org/show_bug.cgi?id=96639 Bug ID: 96639 Summary: st/mesa: transfer_map with too-high level with dEQP-GLES2.functional.texture.completeness.cube.extra_ level Product: Mesa Version: git

[Mesa-dev] [Bug 96629] dEQP-GLES2.functional.texture.completeness.cube.not_positive_level_0: Assertion `width >= 1' failed.

https://bugs.freedesktop.org/show_bug.cgi?id=96629 --- Comment #2 from Ilia Mirkin --- Be that as it may, we shouldn't be asserting for reachable situations. I think another one that asserts is on that list, since I was running with the master list when I hit it (admittedly

[Mesa-dev] [Bug 96629] dEQP-GLES2.functional.texture.completeness.cube.not_positive_level_0: Assertion `width >= 1' failed.

https://bugs.freedesktop.org/show_bug.cgi?id=96629 --- Comment #1 from Mark Janes --- FWIW, this test is not in dEQP's android "must-pass" list: https://android.googlesource.com/platform/external/deqp/+/master/android/cts/master/gles2-master.txt -- You are receiving

Re: [Mesa-dev] [PATCH 2/4] anv: use cache uuid based on the build timestamp.

On Wed, Jun 22, 2016 at 04:57:39PM +0100, Eric Engestrom wrote: > On Wed, Jun 22, 2016 at 01:04:41PM +0100, Emil Velikov wrote: > > From: Emil Velikov [...] > > diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am > > index 4d9ff90..8332ae5 100644

Re: [Mesa-dev] [PATCH 3/4] automake: don't mandate git_sha1.h/MESA_GIT_SHA1

On Wed, Jun 22, 2016 at 01:04:42PM +0100, Emil Velikov wrote: > From: Emil Velikov > > It has proven subtle to get it right both form the build side POV (see s/form/from/ Reviewed-by: Eric Engestrom > commit list below) and builders due

Re: [Mesa-dev] [PATCH 2/4] anv: use cache uuid based on the build timestamp.

On Wed, Jun 22, 2016 at 01:04:41PM +0100, Emil Velikov wrote: > From: Emil Velikov > > Do not rely on the git sha1: > - its current truncated form makes it less unique > - it does not attribute for local (Vulkand or otherwise) changes > > Use a timestamp produced

Re: [Mesa-dev] [PATCH 3/3] radeon/vce: use vce structures for vce_52 firmware

Am 22.06.2016 um 17:43 schrieb Zhang, Boyuan: We should write the encode structure directly without the use of the RVCE_CS() macros. Otherwise all of that doesn't make much sense and is just another layer of abstraction. Different from UVD where firmware takes the address of the IB structure,

Re: [Mesa-dev] [PATCH] st/mesa: fix readpixels regression with MESA_pack_invert

Reviewed-by: Marek Olšák Marek On Wed, Jun 22, 2016 at 5:39 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > Fixes an error introduced in commit 3948cd37973696dc319170877382676809659465. > > Reported-by: Marek Olšák

Re: [Mesa-dev] [PATCH] st/mesa: fix readpixels regression with MESA_pack_invert

On 06/22/2016 09:39 AM, Nicolai Hähnle wrote: From: Nicolai Hähnle Fixes an error introduced in commit 3948cd37973696dc319170877382676809659465. Reported-by: Marek Olšák --- src/mesa/state_tracker/st_cb_readpixels.c | 2 +- 1 file changed, 1

Re: [Mesa-dev] [PATCH 1/4] clover: conditionally use MESA_GIT_SHA1

On Wed, Jun 22, 2016 at 01:04:40PM +0100, Emil Velikov wrote: > From: Emil Velikov > > Considering how hard/annoying it was for many peoples' workflow to > properly generate the macro, it will be demoted to conditionally > available with follow-up commits. > > Cc:

Re: [Mesa-dev] [PATCH 3/3] radeon/vce: use vce structures for vce_52 firmware

> We should write the encode structure directly without the use of the > RVCE_CS() macros. > > Otherwise all of that doesn't make much sense and is just another layer of > abstraction. Different from UVD where firmware takes the address of the IB structure, VCE firmware directly takes the

[Mesa-dev] [PATCH] st/mesa: fix readpixels regression with MESA_pack_invert

From: Nicolai Hähnle Fixes an error introduced in commit 3948cd37973696dc319170877382676809659465. Reported-by: Marek Olšák --- src/mesa/state_tracker/st_cb_readpixels.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH v2] glsl: reuse main extension table to appropriate restrict extensions

So I have encountered bug that should be fixed by this. In my case its a happy little accident that I can use GL_ARB_gpu_shader5 in a compatibility context (since I want the textureGatherOffset functions). So while one part of me is happy that the bug exist. Cheers, Jakob.

Re: [Mesa-dev] [PATCH v2] glsl: reuse main extension table to appropriate restrict extensions

Urgh, just realized that I also need to teach glcpp about this stuff, otherwise we can end up with e.g. GL_ARB_gpu_shader5 defined but not enableable in a compat context. Also a discussion with Jakob on IRC pointed out that we currently allow GL_ARB_gpu_shader5 to be enabled in compat contexts,

Re: [Mesa-dev] [PATCH 2/2] glsl/mesa: stop duplicating geom and tcs layout values

On Wed, 2016-06-22 at 12:41 +1000, Timothy Arceri wrote: > We already store these in gl_shader and gl_program here we > remove it from gl_shader_program and just use the values > from gl_shader. > > This will allow us to keep the shader cache restore code as > simple as it can be while making it

Re: [Mesa-dev] [PATCH V2 1/2] glsl/mesa: stop duplicating tes layout values

On Wed, 2016-06-22 at 12:41 +1000, Timothy Arceri wrote: > We already store this in gl_shader and gl_program here we > remove it from gl_shader_program and just use the values > from gl_shader. > > This will allow us to keep the shader cache restore code as > simple as it can be while making it

Re: [Mesa-dev] [PATCH 6/6] winsys/radeon: add guard pages when R600_DEBUG=check_vm is enabled

For the series: Reviewed-by: Marek Olšák Marek On Wed, Jun 22, 2016 at 11:40 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > This should help flush out GPU VM faults. > --- > src/gallium/winsys/radeon/drm/radeon_drm_bo.c

Re: [Mesa-dev] [PATCH] st/vdpau: use bicubic filter for scaling

Andy Furniss wrote: re-testing with an unpatched mesa and return false after PREFERS I see the same. s/w decode mplayer or mpv will segfault - bt below. No asserts with debug build of mesa. Sometimes/depending on input file it doesn't instantly segfault brief output is corrupt and I get 00s

[Mesa-dev] [PATCH 2/2] radeonsi: set LLVM denormal flags

From: Marek Olšák - make sure FP32 denormals will stay disabled in LLVM in the future (the current default is disabled) - tell LLVM that FP64 denormals are enabled --- src/gallium/drivers/radeonsi/si_pipe.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-)

[Mesa-dev] [PATCH 1/2] radeonsi: emit 1/sqrt for RSQ

From: Marek Olšák We don't need the clamped version and we don't have to use any intrinsic. Stats on Tonga: 15382 shaders in 9128 tests Totals: SGPRS: 1230560 -> 1230560 (0.00 %) VGPRS: 469577 -> 462504 (-1.51 %) Code Size: 22089908 -> 21730052 (-1.63 %) bytes LDS: 598 ->

Re: [Mesa-dev] [PATCH] st/mesa: use a single memcpy in st_ReadPixels when possible

Hi, FYI, piglit "mesa_pack_invert-readpixels -auto -fbo" crashes with this commit. Marek On Tue, Jun 21, 2016 at 11:08 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > This avoids costly address recomputations, function overhead, and may

Re: [Mesa-dev] [PATCH 1/6] gallium/util: fix some 4-space indentation in blitter code

Reviewed-by: Marek Olšák Marek On Wed, Jun 22, 2016 at 12:16 AM, Brian Paul wrote: > --- > src/gallium/auxiliary/util/u_blitter.c | 42 > +- > 1 file changed, 21 insertions(+), 21 deletions(-) > > diff --git

Re: [Mesa-dev] [RFC] Coding style scripts (Was Re: [PATCH 1/2] gallium: replace [0-9]*.f with [0-9]*.0f)

On Wed, Jun 22, 2016 at 8:25 AM, Emil Velikov wrote: > Hi All, > > Seems like we have a few people are keen on the idea of having some > form of at least semi-automated way to handle coding style issues. > > Some options/ideas: > - Combine the emacs .dir-local.el +

[Mesa-dev] [RFC] Coding style scripts (Was Re: [PATCH 1/2] gallium: replace [0-9]*.f with [0-9]*.0f)

Hi All, Seems like we have a few people are keen on the idea of having some form of at least semi-automated way to handle coding style issues. Some options/ideas: - Combine the emacs .dir-local.el + emacs -batch to do the checking: Pros: rules aren't duplicated in multiple places (like the

[Mesa-dev] [PATCH 4/4] swr: automake: don't ship LLVM version specific generated sources

From: Emil Velikov Otherwise things will fail to build, if the builder is using another version of LLVM. v2: annotate all the dependencies of builder_gen.h v3: clean the generated files as needed Cc: "12.0" Cc: Tim Rowley

[Mesa-dev] [PATCH 1/4] clover: conditionally use MESA_GIT_SHA1

From: Emil Velikov Considering how hard/annoying it was for many peoples' workflow to properly generate the macro, it will be demoted to conditionally available with follow-up commits. Cc: mesa-sta...@lists.freedesktop.org Cc: Vedran Miletić Cc:

[Mesa-dev] [PATCH 3/4] automake: don't mandate git_sha1.h/MESA_GIT_SHA1

From: Emil Velikov It has proven subtle to get it right both form the build side POV (see commit list below) and builders due to their varying workflows. Furthermore it does not fully fulfil the reason why it was enforced - to detect uniqueness between different

[Mesa-dev] [PATCH 2/4] anv: use cache uuid based on the build timestamp.

From: Emil Velikov Do not rely on the git sha1: - its current truncated form makes it less unique - it does not attribute for local (Vulkand or otherwise) changes Use a timestamp produced at the time of build. It's perfectly unique, unless someone explicitly

Re: [Mesa-dev] [GSoC lib soft float] Blog update

2016-06-16 1:32 GMT+02:00 Rhys Kidd : > On Monday, June 13, 2016, tournier.elie wrote: > >> Hi, >> >> After a few weeks of coding, I wrote a new post on my blog ( >> https://hopetech.github.io/). I speak about the progress of my project, >> the

Re: [Mesa-dev] [PATCH v4] st/vdpau: use bicubic filter for scaling

Thanks for the review. On Wed, Jun 22, 2016 at 1:17 PM, Christian König wrote: > Am 21.06.2016 um 21:21 schrieb Nayan Deshmukh: > >> use bicubic filtering as high quality scaling L1. >> >> v2: fix a typo and add a newline to code >> >> v3: -render the unscaled image on

[Mesa-dev] [Bug 96634] Mesa or libGL very slow rendering with kabini vga card

https://bugs.freedesktop.org/show_bug.cgi?id=96634 Bug ID: 96634 Summary: Mesa or libGL very slow rendering with kabini vga card Product: Mesa Version: 11.2 Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW

Re: [Mesa-dev] [PATCH] glsl/mesa: stop duplicating tess layout values

On Wed, 2016-06-22 at 09:30 +0200, Iago Toral wrote: > On Wed, 2016-06-22 at 09:54 +1000, Timothy Arceri wrote: > > > > On Tue, 2016-06-21 at 17:45 +0200, Iago Toral wrote: > > > > > > On Tue, 2016-06-21 at 12:21 +1000, Timothy Arceri wrote: > > > > > > > > > > > > We already store this in

  1   2   >