[Mesa-dev] [Bug 96219] Using glXBindTexImageEXT with a GLXPixmap created out-of-process fails

2016-06-22 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=96219 --- Comment #1 from Matt Turner --- (In reply to Andrew Comminos from comment #0) > I suggest that driFetchDrawable is used in dri{2,3}_bind_tex_image, as it is > in dri{2,3}_bind_context. Have you tried that? If it works, I'd send such a patch

[Mesa-dev] [Bug 61907] Indirect rendering of multi-texture vertex arrays broken

2016-06-22 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61907 Matt Turner changed: What|Removed |Added QA Contact||mesa-dev@lists.freedesktop.

[Mesa-dev] [PATCH] i965: Preserve the internal format of the dri image

2016-06-22 Thread Jordan Justen
Since the OpenGLES API is strict about the internal format matching the for many operations, we need to preserve it. See _mesa_es3_error_check_format_and_type in src/mesa/main/glformats.c. Fixes ES2-CTS.gtf.GL2ExtensionTests.egl_image.egl_image Bugzilla: https://bugs.freedesktop.org/show_bug.cgi

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

2016-06-22 Thread Edward O'Callaghan
Thanks spending the time on doing this. With a lightly more descriptive commit summary fixup, Reviewed-by: Edward O'Callaghan On 06/23/2016 03:32 PM, Matt Turner wrote: > On Wed, Jun 22, 2016 at 10:05 PM, Giuseppe Bilotta > wrote: >> Miscellaneous repetitions in commits and fly-by typo fixes.

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

2016-06-22 Thread Mathias Fröhlich
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 bitmask makes functions iterating only active > >> > attr

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

2016-06-22 Thread Matt Turner
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 https://lists.freedesktop.org/mailman/list

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

2016-06-22 Thread Giuseppe Bilotta
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| 2 +- src/compil

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

2016-06-22 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=96573 Kenneth Graunke changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop. |i...@freedesktop.org

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

2016-06-22 Thread Kenneth Graunke
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: https://bugs.freedesktop.org/show_bug.cgi?id=95419 > --- > src/mesa/drivers/dri/i965/gen

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

2016-06-22 Thread Francisco Jerez
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 getting the size of a struct arg. vec3 types still >> > > > work >> > > >

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

2016-06-22 Thread Michel Dänzer
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 %) > Code Size: 22089908

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

2016-06-22 Thread Michel Dänzer
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. It's perfectly unique, > unle

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

2016-06-22 Thread Ilia Mirkin
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 >> +++ >> src/gallium/drivers/radeon/r600_pipe_co

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

2016-06-22 Thread Matt Turner
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 +++--- 1 file changed, 15 insertions(+), 11

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

2016-06-22 Thread Rhys Kidd
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 > Bugzilla: https://bugs.freedesktop

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

2016-06-22 Thread Michael Schellenberger Costa
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 ++ > src/gallium/drivers/radeonsi/si_debug.c | 39 +++

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

2016-06-22 Thread Jan Vesely
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 b/src/gallium/state_tra

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

2016-06-22 Thread Jan Vesely
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 b/src/gallium/state_trackers/clover/llvm/invocation.cpp index 57e

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

2016-06-22 Thread Kenneth Graunke
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 > > --- > > src/mesa/drivers/dri/i965/brw_state.h| 2

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

2016-06-22 Thread Jan Vesely
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 > > > > work > > > > ok. > > > > Only buit-in args

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

2016-06-22 Thread Kenneth Graunke
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: > > > > (ass

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

2016-06-22 Thread Jason Ekstrand
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 +- > src/mesa/drivers/dri/i965/brw_state_upload.c | 2 +- > src/mesa/drivers/dr

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

2016-06-22 Thread Francisco Jerez
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, >> > getTypeAllocSize >> > reports the correct siz

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

2016-06-22 Thread Kenneth Graunke
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 -- src/mesa/drivers/dri/i965/ge

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

2016-06-22 Thread Kenneth Graunke
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 f09b91f7824740

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

2016-06-22 Thread Jan Vesely
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. > > --- > >  src/gallium/st

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

2016-06-22 Thread Chad Versace
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 ___ mesa-dev mailing list mesa-dev@

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

2016-06-22 Thread Chad Versace
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 2016, Jason Ekstrand wrot

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

2016-06-22 Thread Marek Olšák
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 series with 6 patches alr

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

2016-06-22 Thread Marek Olšák
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 the queue (maximum number of jobs wait

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

2016-06-22 Thread Marek Olšák
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 number of threads execut

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

2016-06-22 Thread Vedran Miletić
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: mesa-sta...@lists.freedesktop.org Cc: Vedran Miletić Cc: Fra

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

2016-06-22 Thread Vedran Miletić
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 C

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

2016-06-22 Thread Marek Olšák
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 b/src/gallium/drivers/radeo

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

2016-06-22 Thread Zhang, Boyuan
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 in

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

2016-06-22 Thread Zhang, Boyuan
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 schrie

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

2016-06-22 Thread Boyuan Zhang
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 --- a/src/gallium/driver

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

2016-06-22 Thread Boyuan Zhang
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 --- a/src/gallium/include/pipe/p_video_stat

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

2016-06-22 Thread Boyuan Zhang
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 56b6dd8..ac577e3 100644 --

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

2016-06-22 Thread Boyuan Zhang
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 + src/gallium/drivers/radeon/radeon_vce_52.

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

2016-06-22 Thread Emil Velikov
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 Velikov wrote: On 20 June 2016 at 22:36, Tim Rowley wrote: > --- >

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

2016-06-22 Thread Christian Gmeiner
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: Christian Gmeiner >> --- >> src/mesa/state_tracker/st_extensions.c

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

2016-06-22 Thread Emil Velikov
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 > $@ >> >> -BUILT_SOURCES = $(VULKAN_

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

2016-06-22 Thread Axel Davy
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

2016-06-22 Thread Axel Davy
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

2016-06-22 Thread Emil Velikov
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 >> >> Otherwise things will fail to build, if the builder is using another >> ver

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

2016-06-22 Thread Jason Ekstrand
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 timestamp produced at the time of build. It's perfectly u

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

2016-06-22 Thread Rowley, Timothy O
> 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 wrote: --- .../drivers/swr/rasterizer/jitter/JitManager.cpp | 9

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

2016-06-22 Thread Rowley, Timothy O
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 > version of LLVM. > > v2: annotate all the dependencies of builder_gen.h > v3: cl

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

2016-06-22 Thread Alex Deucher
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) > > https://cgit.freedesktop.org/~mareko/mesa/log/?

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

2016-06-22 Thread Emil Velikov
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: >>> --- >>> .../drivers/swr/rasterizer/jitter/JitManager.cpp | 9 +-- >>> .../drivers/swr/rasterizer/jitter/JitManager.h | 7 -

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

2016-06-22 Thread Cherniak, Bruce
On May 26, 2016, at 3:06 PM, Tim Rowley mailto:timothy.o.row...@intel.com>> 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 changed, 36 insertions(+) di

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

2016-06-22 Thread Rowley, Timothy O
> 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 - >> .../drivers/swr/rasterizer/jitter/blend_jit.cpp|

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

2016-06-22 Thread Kenneth Graunke
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 > Bugzilla: https:/

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

2016-06-22 Thread Ian Romanick
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 Bridge and prior so we can't

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

2016-06-22 Thread Jason Ekstrand
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 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --gi

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

2016-06-22 Thread Jason Ekstrand
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 11.2 12.0" --- src/mes

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

2016-06-22 Thread Emil Velikov
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 +- > .../drivers/swr/rasterizer/jitter/builder_misc.

[Mesa-dev] [BRANCH] Gallium Radeon cleanups

2016-06-22 Thread Marek Olšák
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 r600_resour

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

2016-06-22 Thread Cherniak, Bruce
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 > --- > src/gallium/drivers/swr/swr_shader.cpp | 10 +++--- > src/gallium/driv

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

2016-06-22 Thread Cherniak, Bruce
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 > > Tim Rowley (14): > swr: [rasterizer common] workar

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

2016-06-22 Thread Marek Olšák
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 - 4 files changed,

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

2016-06-22 Thread Marek Olšák
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 2e06746..d83d505 100644 --

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

2016-06-22 Thread Marek Olšák
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 b/src/gallium/drivers/r

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

2016-06-22 Thread Marek Olšák
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 or slow clears must be

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

2016-06-22 Thread Marek Olšák
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(+) diff --git a/src/gallium

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

2016-06-22 Thread Marek Olšák
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 b/src/gallium/drivers/radeon/r600_texture.

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

2016-06-22 Thread Marek Olšák
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.

2016-06-22 Thread Emil Velikov
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 thinkers with their system clock. E

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

2016-06-22 Thread Emil Velikov
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 a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am >> >

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

2016-06-22 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=96629 Ilia Mirkin changed: What|Removed |Added CC||nhaeh...@gmail.com --- Comment #3 from Ili

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

2016-06-22 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=96634 Alex Deucher changed: What|Removed |Added Component|Mesa core |DRM/Radeon QA Contact|mesa-dev@li

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

2016-06-22 Thread bugzilla-daemon
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.

2016-06-22 Thread bugzilla-daemon
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 on a NV34 with force-e

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

2016-06-22 Thread bugzilla-daemon
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 this mail because: You are

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

2016-06-22 Thread Eric Engestrom
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 > > --- a/src/intel/vulkan/M

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

2016-06-22 Thread Eric Engestrom
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 to their varying workflows. > > Furthermore it does not

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

2016-06-22 Thread Eric Engestrom
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 at the time of build. It's per

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

2016-06-22 Thread Christian König
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

2016-06-22 Thread Marek Olšák
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 > --- > src/mesa/state_tracker/st_cb_readpixels.c | 2 +- > 1 file chang

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

2016-06-22 Thread Brian Paul
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 insertion(+), 1 deletion(-) diff --git a/src/me

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

2016-06-22 Thread Eric Engestrom
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: mesa-sta...@lists.freedesktop.o

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

2016-06-22 Thread 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, VCE firmware directly takes the value

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

2016-06-22 Thread Nicolai Hähnle
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 a/src/mesa/state_tracker/st_cb_readpixels.c b/src/mesa/st

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

2016-06-22 Thread Jakob Bornecrantz
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

2016-06-22 Thread Ilia Mirkin
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, but

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

2016-06-22 Thread Iago Toral
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 s

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

2016-06-22 Thread Iago Toral
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 so

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

2016-06-22 Thread Marek Olšák
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 | 4 +++- > src/gallium/winsys/radeon/drm/radeon_drm_winsys.c

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

2016-06-22 Thread Andy Furniss
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 of

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

2016-06-22 Thread Marek Olšák
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(-) diff --git a/src/gallium/

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

2016-06-22 Thread Marek Olšák
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 -> 598 (0.00 %) blocks Sc

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

2016-06-22 Thread Marek Olšák
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 trigger > large copy optimizations. > --- > src/mes

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

2016-06-22 Thread Marek Olšák
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 a/src/gallium/auxiliary/util/u_blitter.c > b/src/gal

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

2016-06-22 Thread Rob Clark
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 + emacs -batch to do the checking

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

2016-06-22 Thread Emil Velikov
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 secon

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

2016-06-22 Thread Emil Velikov
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 Cc: Chuck Atkins Tested-by: Chuck Atkins (v2) Reported-by: Chuck Atki

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

2016-06-22 Thread Emil Velikov
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: Francisco Jerez Signed-off-by: Emil Velikov ---

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

2016-06-22 Thread Emil Velikov
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 builds, in order to distinguish a

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

2016-06-22 Thread Emil Velikov
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 thinkers with their system clock. E

  1   2   >