Re: [Mesa-dev] [PATCH v3 3/6] vulkan: enum generator: make registry more flexible

2017-09-21 Thread Chad Versace
On Thu 21 Sep 2017, Jason Ekstrand wrote: > From: Lionel Landwerlin > > It will be used to store extension numbers as well. > > Signed-off-by: Lionel Landwerlin > Reviewed-by: Jason Ekstrand > --- >

[Mesa-dev] [PATCH v3 4/6] vulkan: enum generator: generate extension number defines

2017-09-21 Thread Jason Ekstrand
From: Lionel Landwerlin New extensions can introduce additional enums. Most of the new enums will have disjoint numbers from the initial enums. For example new formats introduced by VK_IMG_format_pvrtc : VK_FORMAT_ASTC_10x8_UNORM_BLOCK = 177,

[Mesa-dev] [PATCH v3 3/6] vulkan: enum generator: make registry more flexible

2017-09-21 Thread Jason Ekstrand
From: Lionel Landwerlin It will be used to store extension numbers as well. Signed-off-by: Lionel Landwerlin Reviewed-by: Jason Ekstrand --- src/vulkan/util/gen_enum_to_str.py | 20 ++-- 1

[Mesa-dev] [PATCH v3 5/6] vulkan: enum generator: Stop using iterparse

2017-09-21 Thread Jason Ekstrand
While using iterparse is potentially a little more efficient, the Vulkan registry XML is not large and using regular element tree simplifies the parsing logic substantially. --- src/vulkan/util/gen_enum_to_str.py | 36 ++-- 1 file changed, 10 insertions(+), 26

[Mesa-dev] [PATCH v3 6/6] vulkan: enum generator: Generate entries for extended enums

2017-09-21 Thread Jason Ekstrand
--- src/vulkan/util/gen_enum_to_str.py | 48 +++--- 1 file changed, 40 insertions(+), 8 deletions(-) diff --git a/src/vulkan/util/gen_enum_to_str.py b/src/vulkan/util/gen_enum_to_str.py index 07d7685..fc93136 100644 --- a/src/vulkan/util/gen_enum_to_str.py +++

[Mesa-dev] [PATCH v3 1/6] vulkan: enum generator: align function declarations/prototypes

2017-09-21 Thread Jason Ekstrand
From: Lionel Landwerlin Signed-off-by: Lionel Landwerlin Acked-by: Jason Ekstrand --- src/vulkan/util/gen_enum_to_str.py | 42 +++--- 1 file changed, 21 insertions(+), 21

[Mesa-dev] [PATCH 9/9] swr/rast: Handle instanceID offset / Instance Stride enable

2017-09-21 Thread Tim Rowley
Supported in JitGatherVertices(); FetchJit::JitLoadVertices() may require similar changes, will need address this if it is determined that this path is still in use. Handle Force Sequential Access in FetchJit::Create. --- .../drivers/swr/rasterizer/jitter/fetch_jit.cpp| 46

[Mesa-dev] [PATCH 2/9] swr/rast: New GS state/context API

2017-09-21 Thread Tim Rowley
One piglit regression, which was a false pass: spec@glsl-1.50@execution@geometry@dynamic_input_array_index --- .../drivers/swr/rasterizer/core/frontend.cpp | 227 - src/gallium/drivers/swr/rasterizer/core/state.h| 55 +++-- src/gallium/drivers/swr/swr_shader.cpp

[Mesa-dev] [PATCH 6/9] swr/rast: Slightly more efficient blend jit

2017-09-21 Thread Tim Rowley
--- .../drivers/swr/rasterizer/jitter/blend_jit.cpp| 30 -- 1 file changed, 10 insertions(+), 20 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/jitter/blend_jit.cpp b/src/gallium/drivers/swr/rasterizer/jitter/blend_jit.cpp index f2e6e53..3258639 100644 ---

[Mesa-dev] [PATCH 5/9] swr/rast: Properly sized null GS buffer

2017-09-21 Thread Tim Rowley
--- src/gallium/drivers/swr/rasterizer/core/frontend.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/frontend.cpp b/src/gallium/drivers/swr/rasterizer/core/frontend.cpp index 15bc93d..22a5705 100644 ---

[Mesa-dev] [PATCH 3/9] swr/rast: Fetch compile state changes

2017-09-21 Thread Tim Rowley
Add ForceSequentialAccessEnable and InstanceIDOffsetEnable bools to FETCH_COMPILE_STATE. --- src/gallium/drivers/swr/rasterizer/jitter/fetch_jit.cpp | 6 ++ src/gallium/drivers/swr/rasterizer/jitter/fetch_jit.h | 7 ++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 8/9] swr/rast: Remove code supporting legacy llvm (<3.9)

2017-09-21 Thread Tim Rowley
--- .../drivers/swr/rasterizer/jitter/JitManager.cpp | 11 ++- .../drivers/swr/rasterizer/jitter/JitManager.h | 7 -- .../drivers/swr/rasterizer/jitter/builder_misc.cpp | 102 ++--- 3 files changed, 15 insertions(+), 105 deletions(-) diff --git

[Mesa-dev] [PATCH 1/9] swr/rast: Add support for R10G10B10_FLOAT_A2_UNORM pixel format

2017-09-21 Thread Tim Rowley
--- .../drivers/swr/rasterizer/common/formats.cpp | 27 +++--- .../drivers/swr/rasterizer/core/format_traits.h| 2 +- .../drivers/swr/rasterizer/jitter/builder_misc.cpp | 16 ++--- 3 files changed, 28 insertions(+), 17 deletions(-) diff --git

[Mesa-dev] [PATCH 4/9] swr/rast: Move SWR_GS_CONTEXT from thread local storage to stack

2017-09-21 Thread Tim Rowley
Move structure, as the size is significantly reduced due to dynamic allocation of the GS buffers. --- .../drivers/swr/rasterizer/core/frontend.cpp | 23 +++--- 1 file changed, 11 insertions(+), 12 deletions(-) diff --git

[Mesa-dev] [PATCH 7/9] swr/rast: Fix allocation of DS output data for USE_SIMD16_FRONTEND

2017-09-21 Thread Tim Rowley
--- src/gallium/drivers/swr/rasterizer/core/frontend.cpp | 16 ++-- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/frontend.cpp b/src/gallium/drivers/swr/rasterizer/core/frontend.cpp index 22a5705..aea8e88 100644 ---

[Mesa-dev] [PATCH 0/9] swr: update rasterizer

2017-09-21 Thread Tim Rowley
Highlights: large change in the geometry shader api, cleanups. Tim Rowley (9): swr/rast: Add support for R10G10B10_FLOAT_A2_UNORM pixel format swr/rast: New GS state/context API swr/rast: Fetch compile state changes swr/rast: Move SWR_GS_CONTEXT from thread local storage to stack

Re: [Mesa-dev] [PATCH v3 2/6] vulkan: enum generator: sort enums by names

2017-09-21 Thread Chad Versace
On Thu 21 Sep 2017, Jason Ekstrand wrote: > From: Lionel Landwerlin > > Signed-off-by: Lionel Landwerlin > Reviewed-by: Jason Ekstrand > --- > src/vulkan/util/gen_enum_to_str.py | 3 ++- > 1 file changed, 2

Re: [Mesa-dev] [PATCH 3/6] vulkan: enum generator: make registry more flexible

2017-09-21 Thread Jason Ekstrand
On Thu, Sep 21, 2017 at 10:25 AM, Dylan Baker wrote: > Quoting Jason Ekstrand (2017-09-21 08:32:20) > > From: Lionel Landwerlin > > > > It will be used to store extension numbers as well. > > > > Signed-off-by: Lionel Landwerlin

Re: [Mesa-dev] [PATCH v3 1/6] vulkan: enum generator: align function declarations/prototypes

2017-09-21 Thread Dylan Baker
Thanks for updating. For the series: Reviewed-by: Dylan Baker Quoting Jason Ekstrand (2017-09-21 14:19:43) > From: Lionel Landwerlin > > Signed-off-by: Lionel Landwerlin > Acked-by: Jason Ekstrand

[Mesa-dev] [PATCH 4/3] glsl: tidy up IR after loop unrolling

2017-09-21 Thread Timothy Arceri
c7affbf6875622a enabled GLSLOptimizeConservatively on some drivers. The idea was to speed up compile times by running the GLSL IR passes only once each time do_common_optimization() is called. However loop unrolling can create a big mess and with large loops can actually case compile times to

Re: [Mesa-dev] [PATCH 2/5] util: Add tests for the string buffer

2017-09-21 Thread Dieter Nützel
If you (all) think, an AK from me (Dieter) is enough, I'm running this from the beginning, then go ahead. Greetings, Dieter Am 21.09.2017 21:13, schrieb Thomas Helland: Fixed the missing newline at the end of this cpp file locally. This is the only patch left in the series without an RB. If

[Mesa-dev] [PATCH v3 2/6] vulkan: enum generator: sort enums by names

2017-09-21 Thread Jason Ekstrand
From: Lionel Landwerlin Signed-off-by: Lionel Landwerlin Reviewed-by: Jason Ekstrand --- src/vulkan/util/gen_enum_to_str.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH 5/5] radeonsi: allow out-of-order rasterization in commutative blending cases

2017-09-21 Thread Matias N. Goldberg
I'm reviewing the patch and there's something that confuses me about radeonsi_assume_no_z_fights (which got implemented in a later patch). It appears to be that part of the logic is flawed. Please correct me whenever you feel I misunderstood what is going on. Assuming colour writes are

Re: [Mesa-dev] [PATCH] swr/rast: remove llvm fence/atomics from generated files

2017-09-21 Thread Cherniak, Bruce
> On Sep 20, 2017, at 8:13 AM, Laurent Carlier wrote: > > Le mardi 19 septembre 2017, 21:25:42 CEST Tim Rowley a écrit : >> We currently don't use these instructions, and since their API >> changed in llvm-5.0 having them in the autogen files broke the mesa >> release

Re: [Mesa-dev] [PATCH v2 2/3] glsl: check if induction var incremented before use in terminator

2017-09-21 Thread Timothy Arceri
On 22/09/17 01:51, Michael Schellenberger Costa wrote: Out of curriosity what about decrement and other shenanigans? My commit message is a little misleading. The method in which the induction var changes doesn't matter, we just check if something was assigned to it (we trust the rest of the

Re: [Mesa-dev] [PATCH v3 1/6] vulkan: enum generator: align function declarations/prototypes

2017-09-21 Thread Chad Versace
On Thu 21 Sep 2017, Jason Ekstrand wrote: > From: Lionel Landwerlin > > Signed-off-by: Lionel Landwerlin > Acked-by: Jason Ekstrand > --- > src/vulkan/util/gen_enum_to_str.py | 42 >

Re: [Mesa-dev] [PATCH] anv: Implement VK_ANDROID_native_buffer (v5)

2017-09-21 Thread Chad Versace
On Thu 21 Sep 2017, Tapani Pälli wrote: > Hi Chad; > > The build works ok now on Android-IA. There is still something wrong with > 'exec async' though. It behaves differently with small/big apps but > eventually I think it just starts to block .. somewhere. I still need the > big hammer to set

Re: [Mesa-dev] [PATCH v2] i965 : optimized bucket index calculation

2017-09-21 Thread Marathe, Yogesh
+ all v1 reviewers. Does this look ok? -Yogesh. >-Original Message- >From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of >aravindan.muthuku...@intel.com >Sent: Thursday, September 14, 2017 12:13 PM >To: mesa-dev@lists.freedesktop.org >Cc: Muthukumar, Aravindan

Re: [Mesa-dev] initial meson port

2017-09-21 Thread Jakob Bornecrantz
On Thu, Sep 21, 2017 at 2:20 AM, Eric Anholt wrote: > Dylan Baker writes: >> Results >> autotools : sh -c 535.34s user 30.33s system 310% cpu 3:02.05 total >> meson : sh -c 136.58s user 11.98s system 372% cpu 39.895 total > > I just want to point at

Re: [Mesa-dev] initial meson port

2017-09-21 Thread Nirbheek Chauhan
On Thu, Sep 21, 2017 at 5:36 PM, Jakob Bornecrantz wrote: > Wasn't lacking distcheck support one of the arguments against moving > to only a scons build when this was brought up all those years ago? > Does Meson provide something similar, or do people just now get all > of

Re: [Mesa-dev] [PATCH 00/14] Patches for VA-API State Tracker Postproc

2017-09-21 Thread Leo Liu
On 09/21/2017 03:03 AM, Mark Thompson wrote: On 21/09/17 03:17, Leo Liu wrote: On 09/20/2017 06:11 PM, Mark Thompson wrote: On 19/09/17 20:04, Leo Liu wrote: This series are for VA-API State Tracker Postproc, including: Deinterlacing I video for transcode; Scaling support in postproc for

[Mesa-dev] [PATCH 05/22] glsl: Convert lower_vec_index_to_cond_assign to using ir_builder

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- .../glsl/lower_vec_index_to_cond_assign.cpp | 56 ++- 1 file changed, 17 insertions(+), 39 deletions(-) diff --git

[Mesa-dev] [PATCH 04/22] glsl: Return ir_variable from compare_index_block

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick This is basically a wash now, but it simplifies later patches that convert to using ir_builder. Signed-off-by: Ian Romanick --- src/compiler/glsl/ir_optimization.h

[Mesa-dev] [PATCH 01/22] glsl: Fix coding standards issues in lower_if_to_cond_assign

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Mostly tabs-before-spaces issues. Signed-off-by: Ian Romanick --- src/compiler/glsl/lower_if_to_cond_assign.cpp | 95 +-- 1 file changed, 47 insertions(+),

[Mesa-dev] [PATCH 07/22] glsl: Fix coding standards issues in lower_variable_index_to_cond_assign

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Mostly tabs-before-spaces, but there was some other trivium too. Signed-off-by: Ian Romanick --- .../lower_variable_index_to_cond_assign.cpp | 154 +- 1

[Mesa-dev] [PATCH 02/22] glsl: Lower ifs to conditional-select instead of conditional-assign

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/lower_if_to_cond_assign.cpp | 22 --- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git

[Mesa-dev] [PATCH 08/22] glsl: Convert lower_variable_index_to_cond_assign to ir_builder

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/ir_optimization.h | 8 +- .../lower_variable_index_to_cond_assign.cpp | 158 +++---

[Mesa-dev] [PATCH 00/22] Send ir_assignment::condition to the Upside Down

2017-09-21 Thread Ian Romanick
From: Ian Romanick I have also pushed this series to https://cgit.freedesktop.org/~idr/mesa/log/?h=remove-ir_assignment-condition I think -201 lines speaks for itself, but... The condition field pre-dates the ir_triop_csel operation by a few years. The idea was that

[Mesa-dev] [PATCH 12/22] glsl/ast: Convert ast_case_label::hir to ir_builder

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/ast_to_hir.cpp | 35 ++-- 1 file changed, 11 insertions(+), 24 deletions(-) diff --git

[Mesa-dev] [PATCH 14/22] glsl/ast: Generate a more compact expression to disable execution of default case

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Instead of generating a sequence like: run_default = true; if (i == 3) // some label that appears after default run_default = false; if (i == 4) // some label that appears after

[Mesa-dev] [PATCH 18/22] glsl: Move 'foo = foo; ' optimization to opt_dead_code_local

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick The optimization as done in opt_copy_propagation would have to be removed in the next patch, and doing so would lead to some substantial regressions. Signed-off-by: Ian Romanick

[Mesa-dev] [PATCH 22/22] glsl: Simplify ir_assignment::clone

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Use the ir_assignment constructor that takes the write mask as a parameter. This skips some work that was done in the other constructor. Signed-off-by: Ian Romanick ---

[Mesa-dev] [PATCH v2 3/3] glsl: make loop unrolling more like the nir unrolling path

2017-09-21 Thread Timothy Arceri
The old code assumed that loop terminators will always be at the start of the loop, resulting in otherwise unrollable loops not being unrolled at all. For example the current code would unroll: int j = 0; do { if (j > 5) break; ... do stuff ... j++; } while (j < 4);

[Mesa-dev] [PATCH v2 1/3] glsl: don't drop instructions from unreachable terminators continue branch

2017-09-21 Thread Timothy Arceri
These instructions will be executed on every iteration of the loop we cannot drop them. V2: - move removal of unreachable terminators from the terminator list to the same place they are removed from the IR as suggested by Nicolai. --- src/compiler/glsl/loop_analysis.h | 7 +++

[Mesa-dev] [PATCH v2 2/3] glsl: check if induction var incremented before use in terminator

2017-09-21 Thread Timothy Arceri
do-while loops can increment the starting value before the condition is checked. e.g. do { ndx++; } while (ndx < 3); This commit changes the code to detect this and reduces the iteration count by 1 if found. V2: fix terminator spelling Reviewed-by: Nicolai Hähnle

Re: [Mesa-dev] [PATCH] Revert "intel: Remove unused Kabylake pci ids

2017-09-21 Thread Rodrigo Vivi
On Wed, Sep 20, 2017 at 08:17:32PM +, Anuj Phogat wrote: > drm-intel is in favor of keeping the unused pci-id's which > are still listed in the h/w specs. To keep it uniform > across multiple gfx stack components, I'm reverting below > Mesa patches: > b2dae9f8fd310c19e66b161a7ee9845af78f73e0 >

[Mesa-dev] [PATCH 19/22] glsl: Kill ir_assignment::condition with fire

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/glsl_to_nir.cpp | 16 ++ src/compiler/glsl/ir.cpp | 5 ++---

[Mesa-dev] [PATCH 06/22] glsl: Lower vector indexing to conditional-select instead of conditional-assign

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick This changes the generated GLSL IR from something like bvec4 eq = equal(vec4(index), vec4(0, 1, 2, 3)); if (eq.x) lhs = rhs.x; if (eq.y) lhs = rhs.y; if (eq.z)

[Mesa-dev] [PATCH 17/22] glsl: Eliminate ir_assignment constructors that have a condition

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/ast_function.cpp| 15 +++ src/compiler/glsl/ir.cpp | 19 ++-

[Mesa-dev] [PATCH 03/22] glsl: Fix coding standards issues in lower_vec_index_to_cond_assign

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Mostly tabs-before-spaces, but there was some other trivium too. Signed-off-by: Ian Romanick --- .../glsl/lower_vec_index_to_cond_assign.cpp | 28 --- 1

[Mesa-dev] [PATCH 16/22] glsl: Eliminate ir_builder assign overloads that have a condition

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/ir_builder.cpp | 16 ++-- src/compiler/glsl/ir_builder.h | 2 -- 2 files changed, 2 insertions(+), 16

[Mesa-dev] [PATCH 10/22] glsl: Don't pass NULL to ir_assignment constructor when not necessary

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/ast_function.cpp| 22 +-- .../glsl/ir_expression_flattening.cpp | 4 +---

[Mesa-dev] [PATCH 15/22] glsl/ast: Use logical-or instead of conditional assignment to set fallthru_var

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/ast_to_hir.cpp | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/compiler/glsl/ast_to_hir.cpp

[Mesa-dev] [PATCH 20/22] glsl: Fix indentation left weird from the previous commit

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/opt_dead_code_local.cpp | 168 +++--- src/mesa/program/ir_to_mesa.cpp | 10 +- 2 files changed,

[Mesa-dev] [PATCH 13/22] glsl/ast: Explicitly track the set of case labels that occur after default

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Previously the instruction stream was walked looking for comparisons with case-label values. This should generate nearly identical code. For at least fs-default-notlast-fallthrough.shader_test, the

[Mesa-dev] [PATCH 11/22] glsl/ast: Use ir_binop_equal instead of ir_binop_all_equal

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick The values being compared are scalars, so these are the same. While I'm here, simplify the run_default condition to just deref the flag (instead of comparing a scalar bool with true). There is a bit

[Mesa-dev] [PATCH 09/22] glsl: Lower array indexing to conditional-select instead of conditional-assign

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/lower_variable_index_to_cond_assign.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH 21/22] glsl: Remove spurious assertions

2017-09-21 Thread Ian Romanick
From: "\"Ian Romanick\"" From: Ian Romanick It's inside an if-statement that already checks that the variables are not NULL. Signed-off-by: Ian Romanick --- src/compiler/glsl/opt_copy_propagation.cpp | 2 -- 1 file

Re: [Mesa-dev] [PATCH 1/3] glsl: Silence unused parameter warnings

2017-09-21 Thread Ian Romanick
On 09/20/2017 09:15 PM, Timothy Arceri wrote: > Fix sent: > > https://lists.freedesktop.org/archives/mesa-dev/2017-September/170304.html Ugh... which I already deleted from my inbox due to the tag. :( I'll bet this fixes https://bugs.freedesktop.org/show_bug.cgi?id=102910. That patch is

[Mesa-dev] [PATCH v2] Android: move libraries to /vendor

2017-09-21 Thread Rob Herring
As part of Treble project in Android O, all the device specific files have to be located in a separate vendor partition. This is done by setting LOCAL_PROPRIETARY_MODULE (the name is misleading). This change will not break existing platforms without a vendor partition as it will just move files to

[Mesa-dev] [PATCH 1/6] vulkan: enum generator: align function declarations/prototypes

2017-09-21 Thread Jason Ekstrand
From: Lionel Landwerlin Signed-off-by: Lionel Landwerlin Acked-by: Jason Ekstrand --- src/vulkan/util/gen_enum_to_str.py | 42 +++--- 1 file changed, 21 insertions(+), 21

[Mesa-dev] [PATCH 2/6] vulkan: enum generator: sort enums by names

2017-09-21 Thread Jason Ekstrand
From: Lionel Landwerlin Signed-off-by: Lionel Landwerlin Reviewed-by: Jason Ekstrand --- src/vulkan/util/gen_enum_to_str.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 3/6] vulkan: enum generator: make registry more flexible

2017-09-21 Thread Jason Ekstrand
From: Lionel Landwerlin It will be used to store extension numbers as well. Signed-off-by: Lionel Landwerlin Reviewed-by: Jason Ekstrand --- src/vulkan/util/gen_enum_to_str.py | 22 +++--- 1

[Mesa-dev] [PATCH 4/6] vulkan: enum generator: generate extension number defines

2017-09-21 Thread Jason Ekstrand
From: Lionel Landwerlin New extensions can introduce additional enums. Most of the new enums will have disjoint numbers from the initial enums. For example new formats introduced by VK_IMG_format_pvrtc : VK_FORMAT_ASTC_10x8_UNORM_BLOCK = 177,

[Mesa-dev] [PATCH 6/6] vulkan: enum generator: Generate entries for extended enums

2017-09-21 Thread Jason Ekstrand
--- src/vulkan/util/gen_enum_to_str.py | 50 +++--- 1 file changed, 41 insertions(+), 9 deletions(-) diff --git a/src/vulkan/util/gen_enum_to_str.py b/src/vulkan/util/gen_enum_to_str.py index e5f8964..57d12e6 100644 --- a/src/vulkan/util/gen_enum_to_str.py +++

[Mesa-dev] [Bug 102891] [radv] glitches on rpcs3 emulator (green zones)

2017-09-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102891 --- Comment #5 from Bas Nieuwenhuizen --- IIRC that SpvCapabilityImageMSArray warning is always there with renderdoc though, so probably a red herring. -- You are receiving this mail because: You are the assignee for

[Mesa-dev] [PATCH 5/6] vulkan: enum generator: Stop using iterparse

2017-09-21 Thread Jason Ekstrand
While using iterparse is potentially a little more efficient, the Vulkan registry XML is not large and using regular element tree simplifies the parsing logic substantially. --- src/vulkan/util/gen_enum_to_str.py | 41 +++--- 1 file changed, 16 insertions(+), 25

Re: [Mesa-dev] [PATCH 1/3] mesa: GL_TEXTURE_BORDER_COLOR exists in OpenGL 1.0, so don't depend on GL_ARB_texture_border_clamp

2017-09-21 Thread Ian Romanick
On 09/20/2017 03:12 AM, Juan A. Suarez Romero wrote: > On Sat, 2017-07-08 at 02:03 +0300, Andres Gomez wrote: >> Ian, it looks like we could want this patch (and the others from the >> series when they land) in -stable (?) >> > > As we are preparing a new stable 17.1 release, gently pinging. >

Re: [Mesa-dev] [PATCH 19/22] glsl: Kill ir_assignment::condition with fire

2017-09-21 Thread Ian Romanick
And I just pushed an update to my branch that rebases this on the merge of loop_analysis.cpp and loop_controls.cpp. On 09/21/2017 09:34 AM, Ian Romanick wrote: > From: "\"Ian Romanick\"" > > From: Ian Romanick > > Signed-off-by: Ian Romanick

Re: [Mesa-dev] [PATCH v2] Android: move libraries to /vendor

2017-09-21 Thread Tapani Pälli
Reviewed-by: Tapani Pälli On 09/21/2017 06:28 PM, Rob Herring wrote: As part of Treble project in Android O, all the device specific files have to be located in a separate vendor partition. This is done by setting LOCAL_PROPRIETARY_MODULE (the name is misleading). This

Re: [Mesa-dev] [PATCH] amd/addrlib: fix missing va_end() after va_copy()

2017-09-21 Thread Nicolai Hähnle
On 21.09.2017 12:03, Eric Engestrom wrote: Hmm, just noticed the title should be fixed. Something like this? amd/addrlib: drop unnecessary va_copy() Makes sense, but I already pushed it... On Wednesday, 2017-09-20 14:48:46 +, Nicolai Hähnle wrote: From: Nicolai Hähnle

Re: [Mesa-dev] [PATCH 00/14] Patches for VA-API State Tracker Postproc

2017-09-21 Thread Mark Thompson
On 21/09/17 03:17, Leo Liu wrote: > On 09/20/2017 06:11 PM, Mark Thompson wrote: >> On 19/09/17 20:04, Leo Liu wrote: >>> This series are for VA-API State Tracker Postproc, including: >>> >>> Deinterlacing I video for transcode; >>> Scaling support in postproc for transcode; >>> Frame grabber in

[Mesa-dev] [PATCH] i965/screen: Check that given format is valid

2017-09-21 Thread Topi Pohjolainen
CID: 1418110 Fixes: 939b53d3325 "i965/screen: Implement queryDmaBufFormatModifierAttirbs" CC: Jason Ekstrand Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/intel_screen.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [Mesa-dev] [PATCH] mesa/st: fix infinite loops

2017-09-21 Thread Kenneth Graunke
On Wednesday, September 20, 2017 7:13:29 PM PDT Timothy Arceri wrote: > Fixes: 9ac8fece63a9 (glsl: Unify ir_constant::const_elements and ::components) > --- > src/mesa/program/ir_to_mesa.cpp| 2 +- > src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 +- > 2 files changed, 2

Re: [Mesa-dev] [PATCH] i965/screen: Check that given format is valid

2017-09-21 Thread Tapani Pälli
Reviewed-by: Tapani Pälli On 09/21/2017 10:22 AM, Topi Pohjolainen wrote: CID: 1418110 Fixes: 939b53d3325 "i965/screen: Implement queryDmaBufFormatModifierAttirbs" CC: Jason Ekstrand Signed-off-by: Topi Pohjolainen

Re: [Mesa-dev] [PATCH] mesa/st: fix infinite loops

2017-09-21 Thread Timothy Arceri
On 21/09/17 16:55, Kenneth Graunke wrote: On Wednesday, September 20, 2017 7:13:29 PM PDT Timothy Arceri wrote: Fixes: 9ac8fece63a9 (glsl: Unify ir_constant::const_elements and ::components) --- src/mesa/program/ir_to_mesa.cpp| 2 +- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2

Re: [Mesa-dev] initial meson port

2017-09-21 Thread Eric Anholt
Matt Turner writes: > On Thu, Sep 21, 2017 at 5:06 AM, Jakob Bornecrantz > wrote: >> On Thu, Sep 21, 2017 at 2:20 AM, Eric Anholt wrote: >>> Dylan Baker writes: Results autotools : sh -c 535.34s user

Re: [Mesa-dev] [PATCH 2/2] st/va: Implement vaExportSurfaceHandle()

2017-09-21 Thread Mark Thompson
On 20/09/17 09:14, Christian König wrote: > Am 20.09.2017 um 00:01 schrieb Mark Thompson: >> This is a new interface in libva2 to support wider use-cases of passing >> surfaces to external APIs.  In particular, this allows export of NV12 and >> P010 surfaces. > > First of all thanks a lot for

[Mesa-dev] [Mesa-announce] Mesa 17.1.10 release candidate

2017-09-21 Thread Juan A. Suarez Romero
Hello list, The candidate for the Mesa 17.1.10 is now available. Currently we have: - 41 queued - 0 nominated (outstanding) - and 5 rejected patches This is the last release for the 17.1 series. In the current queue we have: In build and integration system, we add a dependency on libunwind

Re: [Mesa-dev] [PATCH 5/6] vulkan: enum generator: Stop using iterparse

2017-09-21 Thread Jason Ekstrand
On Thu, Sep 21, 2017 at 10:34 AM, Dylan Baker wrote: > Quoting Jason Ekstrand (2017-09-21 08:32:22) > > While using iterparse is potentially a little more efficient, the Vulkan > > registry XML is not large and using regular element tree simplifies the > > parsing logic

Re: [Mesa-dev] [PATCH 2/5] util: Add tests for the string buffer

2017-09-21 Thread Thomas Helland
Fixed the missing newline at the end of this cpp file locally. This is the only patch left in the series without an RB. If there's no objections I plan on pushing this once I get an RB on this. Someone mind having a look at it? 2017-09-11 22:21 GMT+02:00 Thomas Helland

[Mesa-dev] [PATCH 05/15] st/omx: use new vl_compositor_yuv_deint_full() to deint

2017-09-21 Thread Leo Liu
v2: add dst rect to make sure no scale Acked-by: Christian König --- src/gallium/state_trackers/omx_bellagio/vid_dec.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/omx_bellagio/vid_dec.c

[Mesa-dev] [PATCH 1/2] glx: Sort the GLX extension bit enum and table

2017-09-21 Thread Adam Jackson
Not quite asciibetical: ARB, then EXT, then vendor, just like the GL extension enum just below. No functional change, but it bothered me. Signed-off-by: Adam Jackson --- src/glx/glxextensions.c | 22 +++--- src/glx/glxextensions.h | 24

[Mesa-dev] [PATCH 2/2] glx: Implement GLX_EXT_no_config_context

2017-09-21 Thread Adam Jackson
This more or less ports EGL_KHR_no_config_context to GLX. This is currently a draft extension, under review at: https://github.com/KhronosGroup/OpenGL-Registry/pull/102 Signed-off-by: Adam Jackson --- src/glx/create_context.c | 37 +++--

Re: [Mesa-dev] [PATCH] radv: Implement VK_AMD_rasterization_order

2017-09-21 Thread Bas Nieuwenhuizen
Pushed, thanks. On Mon, Sep 18, 2017 at 6:26 PM, Nicholas Miell wrote: > Tested with AMD's Anvil OutOfOrderRasterization demo on a RX 560. > > Signed-off-by: Nicholas Miell > --- > src/amd/vulkan/radv_device.c | 17 + >

Re: [Mesa-dev] initial meson port

2017-09-21 Thread Matt Turner
On Thu, Sep 21, 2017 at 5:06 AM, Jakob Bornecrantz wrote: > On Thu, Sep 21, 2017 at 2:20 AM, Eric Anholt wrote: >> Dylan Baker writes: >>> Results >>> autotools : sh -c 535.34s user 30.33s system 310% cpu 3:02.05 total >>> meson

Re: [Mesa-dev] [PATCH v2 2/3] glsl: check if induction var incremented before use in terminator

2017-09-21 Thread Michael Schellenberger Costa
Out of curriosity what about decrement and other shenanigans? --Michael Am 21.09.2017 um 12:55 schrieb Timothy Arceri: do-while loops can increment the starting value before the condition is checked. e.g. do { ndx++; } while (ndx < 3); This commit changes the code to detect this

Re: [Mesa-dev] [PATCH 3/6] vulkan: enum generator: make registry more flexible

2017-09-21 Thread Dylan Baker
Quoting Jason Ekstrand (2017-09-21 08:32:20) > From: Lionel Landwerlin > > It will be used to store extension numbers as well. > > Signed-off-by: Lionel Landwerlin > Reviewed-by: Jason Ekstrand > --- >

Re: [Mesa-dev] [PATCH 4/6] vulkan: enum generator: generate extension number defines

2017-09-21 Thread Dylan Baker
Quoting Jason Ekstrand (2017-09-21 08:32:21) > From: Lionel Landwerlin > > New extensions can introduce additional enums. Most of the new enums > will have disjoint numbers from the initial enums. For example new > formats introduced by VK_IMG_format_pvrtc : > >

Re: [Mesa-dev] [PATCH 5/6] vulkan: enum generator: Stop using iterparse

2017-09-21 Thread Dylan Baker
Quoting Jason Ekstrand (2017-09-21 08:32:22) > While using iterparse is potentially a little more efficient, the Vulkan > registry XML is not large and using regular element tree simplifies the > parsing logic substantially. > --- > src/vulkan/util/gen_enum_to_str.py | 41 >

Re: [Mesa-dev] [PATCH] anv: Implement VK_ANDROID_native_buffer (v5)

2017-09-21 Thread Tapani Pälli
Hi Chad; The build works ok now on Android-IA. There is still something wrong with 'exec async' though. It behaves differently with small/big apps but eventually I think it just starts to block .. somewhere. I still need the big hammer to set device->has_exec_async false to fix that. Please

Re: [Mesa-dev] [PATCH 3/3] glsl: make loop unrolling more like the nir unrolling path

2017-09-21 Thread Timothy Arceri
On 18/09/17 21:43, Timothy Arceri wrote: On 18/09/17 20:50, Nicolai Hähnle wrote: On 14.09.2017 06:47, Timothy Arceri wrote: The old code incorrectly assumed that loop terminators will always be at the start of the loop. It really seems to be just luck that we haven't triggered any bugs

Re: [Mesa-dev] Meson mesademos (Was: [RFC libdrm 0/2] Replace the build system with meson)

2017-09-21 Thread Nirbheek Chauhan
On Thu, Sep 21, 2017 at 1:53 PM, Jose Fonseca wrote: > On 12/04/17 18:57, Nirbheek Chauhan wrote: >> >> Hi Jose, >> >> On Wed, Apr 12, 2017 at 11:08 PM, Jose Fonseca >> wrote: >>> >>> One newbie question: what's your workflow to update a wrap patch? Can

Re: [Mesa-dev] Meson mesademos (Was: [RFC libdrm 0/2] Replace the build system with meson)

2017-09-21 Thread Jose Fonseca
On 12/04/17 18:57, Nirbheek Chauhan wrote: Hi Jose, On Wed, Apr 12, 2017 at 11:08 PM, Jose Fonseca wrote: One newbie question: what's your workflow to update a wrap patch? Can we prototype changes locally without tarballing the patch? Any changes you make in

Re: [Mesa-dev] [PATCH] amd/addrlib: fix missing va_end() after va_copy()

2017-09-21 Thread Eric Engestrom
Hmm, just noticed the title should be fixed. Something like this? > amd/addrlib: drop unnecessary va_copy() On Wednesday, 2017-09-20 14:48:46 +, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > There's no reason to use va_copy here. > > CID: 1418113 > --- > I have