Re: [Mesa-dev] [PATCH] i965: Add an INTEL_DEBUG=reemit option.

Reviewed-by: Matt Turner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH v4 1/1] clover: Wait for requested operation if blocking flag is set

On Thu, 2017-09-14 at 14:39 -0700, Francisco Jerez wrote: > Jan Vesely writes: > > > On Mon, 2017-09-04 at 13:23 -0700, Francisco Jerez wrote: > > > Jan Vesely writes: > > > > > > > v2: wait in map_buffer and map_image as well > > > > v3: use

Re: [Mesa-dev] [PATCH 7/8] nir: add serialization and deserialization

On Fri, Sep 15, 2017 at 6:39 PM, Jason Ekstrand wrote: > From: Connor Abbott > > v2 (Jason Ekstrand): > - Various whitespace cleanups > - Add helpers for reading/writing objects > - Rework derefs > - [de]serialize nir_shader::num_* > - Fix uses of

Re: [Mesa-dev] [PATCH 1/1] clover: Fix build after LLVM r313390

On Fri, 2017-09-15 at 17:59 -0700, Francisco Jerez wrote: > Jan Vesely writes: > > > On Fri, 2017-09-15 at 17:48 -0700, Francisco Jerez wrote: > > > Jan Vesely writes: > > > > > > > Signed-off-by: Jan Vesely > > > > --- >

Re: [Mesa-dev] [PATCH 8/8] nir: Add hooks for testing serialization

On Fri, Sep 15, 2017 at 6:39 PM, Jason Ekstrand wrote: > --- > src/compiler/nir/nir.h | 17 + > src/compiler/nir/nir_serialize.c | 18 ++ > 2 files changed, 35 insertions(+) > > diff --git a/src/compiler/nir/nir.h

[Mesa-dev] [PATCH 5/8] nir/intrinsics: Set the correct num_indices for load_output

Cc: mesa-sta...@lists.freedesktop.org --- src/compiler/nir/nir_intrinsics.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compiler/nir/nir_intrinsics.h b/src/compiler/nir/nir_intrinsics.h index ea51525..e461153 100644 --- a/src/compiler/nir/nir_intrinsics.h +++

[Mesa-dev] [PATCH 4/8] nir: Get rid of nir_shader::stage

It's redundant with nir_shader::info::stage. --- src/amd/common/ac_nir_to_llvm.c| 30 ++--- src/amd/common/ac_shader_info.c| 2 +- src/amd/vulkan/radv_shader.c | 4 +- src/compiler/glsl/glsl_to_nir.cpp | 10

[Mesa-dev] [PATCH 6/8] compiler/types: Support [de]serializing void types

--- src/compiler/glsl_types.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/compiler/glsl_types.cpp b/src/compiler/glsl_types.cpp index a7fc7ff..704b63c 100644 --- a/src/compiler/glsl_types.cpp +++ b/src/compiler/glsl_types.cpp @@ -2149,6 +2149,8 @@ encode_type_to_blob(struct blob

[Mesa-dev] [PATCH 3/8] compiler/blob: Rework blob_reserve_bytes

This commit primarily makes two changes. First, blob_reserve_bytes now takes an alignment parameter. Second, we now return an offset into the blob instead of a pointer and leave the pointer arithmetic to the caller. This way you can call blob_reserve_bytes, emit a bunch more stuff, and then

[Mesa-dev] [PATCH 2/8] glsl: move shader_cache type handling to glsl_types

From: Connor Abbott Not sure if this is the best place to put it, but we're going to need this for NIR too. --- src/compiler/glsl/shader_cache.cpp | 171 - src/compiler/glsl_types.cpp| 171 +

[Mesa-dev] [PATCH 8/8] nir: Add hooks for testing serialization

--- src/compiler/nir/nir.h | 17 + src/compiler/nir/nir_serialize.c | 18 ++ 2 files changed, 35 insertions(+) diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h index e6104fc..57cd0d9 100644 --- a/src/compiler/nir/nir.h +++

[Mesa-dev] [PATCH 7/8] nir: add serialization and deserialization

From: Connor Abbott v2 (Jason Ekstrand): - Various whitespace cleanups - Add helpers for reading/writing objects - Rework derefs - [de]serialize nir_shader::num_* - Fix uses of blob_reserve_bytes - Use a bitfield struct for packing tex_instr data ---

[Mesa-dev] [PATCH 1/8] glsl: move blob up to the compiler directory

From: Connor Abbott I want to use this for NIR too. --- src/compiler/Makefile.sources| 4 ++-- src/compiler/{glsl => }/blob.c | 0 src/compiler/{glsl => }/blob.h | 0 src/mesa/state_tracker/st_shader_cache.h | 2 +- 4 files changed, 3

[Mesa-dev] [PATCH] i965: Add an INTEL_DEBUG=reemit option.

Jason and I use this for debugging all the time. Recompiling the driver to enable it is kind of annoying. It's a great thing to try along with always_flush_batch=true and always_flush_cache=true to detect a class of problems - namely, atoms listening to an insufficient set of dirty bits. ---

Re: [Mesa-dev] [PATCH 1/1] clover: Fix build after LLVM r313390

Jan Vesely writes: > On Fri, 2017-09-15 at 17:48 -0700, Francisco Jerez wrote: >> Jan Vesely writes: >> >> > Signed-off-by: Jan Vesely >> > --- >> > src/gallium/state_trackers/clover/llvm/compat.hpp | 10 ++

Re: [Mesa-dev] [PATCH 1/1] clover: Fix build after LLVM r313390

On Fri, 2017-09-15 at 17:48 -0700, Francisco Jerez wrote: > Jan Vesely writes: > > > Signed-off-by: Jan Vesely > > --- > > src/gallium/state_trackers/clover/llvm/compat.hpp | 10 ++ > >

Re: [Mesa-dev] [PATCH 1/1] clover: Fix build after LLVM r313390

Jan Vesely writes: > Signed-off-by: Jan Vesely > --- > src/gallium/state_trackers/clover/llvm/compat.hpp | 10 ++ > src/gallium/state_trackers/clover/llvm/invocation.cpp | 2 +- > 2 files changed, 11 insertions(+), 1 deletion(-) > >

[Mesa-dev] [PATCH 1/1] clover: Fix build after LLVM r313390

Signed-off-by: Jan Vesely --- src/gallium/state_trackers/clover/llvm/compat.hpp | 10 ++ src/gallium/state_trackers/clover/llvm/invocation.cpp | 2 +- 2 files changed, 11 insertions(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH v2 0/6] Flag new aux state on aux state updates

I don't think we need patches 2 or 6 and the commit message from 2 should probably go on 5. I left a comment on patch 4. Other than that, 1, 3, 4, and 5 are Reviewed-by: Jason Ekstrand On Fri, Sep 15, 2017 at 3:02 AM, Iago Toral Quiroga wrote: >

Re: [Mesa-dev] [PATCH v2 4/6] i965: emit BRW_NEW_AUX_STATE when we change the fast clear value

On Fri, Sep 15, 2017 at 3:02 AM, Iago Toral Quiroga wrote: > --- > src/mesa/drivers/dri/i965/brw_blorp.c | 12 ++-- > src/mesa/drivers/dri/i965/brw_clear.c | 2 +- > src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 28 > +++ > 3

Re: [Mesa-dev] [PATCH v2 2/6] i965: emit BRW_NEW_AUX_STATE when we allocate aux surfaces

Given the fact that we're whacking BRW_NEW_AUX_USAGE whenever aux_state changes, I don't think we need this. First off, for most surfaces, the aux buffer is created when the surface is created at which point it's never been referenced with a SURFACE_STATE and we don't need to flag new state. The

Re: [Mesa-dev] [PATCH v2 6/6] i965: emit BRW_NEW_AUX_STATE for textures without unresolved colors

On Fri, Sep 15, 2017 at 3:02 AM, Iago Toral Quiroga wrote: > intel_miptree_texture_aux_usage() will report ISL_AUX_USAGE_NONE for > textures that have a fast clear surface when they have been fully > resolved in order to avoid unnecesary aux surface lookups and save >

Re: [Mesa-dev] [PATCH 00/15] anv: implement KHR_sampler_ycbcr_conversion

On Fri, Sep 15, 2017 at 7:10 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Hi all, > > This series implements KHR_sampler_ycbcr_conversion (and > KHR_bind_memory2 as requirement) for the Anv driver. Apologies for > this series which is a bit invasive as it needs to introduce the

Re: [Mesa-dev] [PATCH 14/15] anv: enable multiple planes per image/imageView

On Fri, Sep 15, 2017 at 7:11 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > This change introduce the concept of planes for image & views. It > matches the planes available in new formats. > > We also refactor depth & stencil support through the usage of planes > for the sake of

[Mesa-dev] [PATCH 1/2] radv: Add code to check if two formats can share DCC metadata.

Ported from radeonsi. --- src/amd/vulkan/radv_formats.c | 85 +++ src/amd/vulkan/radv_private.h | 2 + 2 files changed, 87 insertions(+) diff --git a/src/amd/vulkan/radv_formats.c b/src/amd/vulkan/radv_formats.c index c19a9a37a49..24445412813 100644 ---

[Mesa-dev] [PATCH 2/2] radv: Add support for more DCC compression with VK_KHR_image_format_list.

--- src/amd/vulkan/radv_device.c | 4 src/amd/vulkan/radv_image.c | 24 +--- 2 files changed, 25 insertions(+), 3 deletions(-) diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c index c112453e395..e6d595dfbe5 100644 ---

Re: [Mesa-dev] [PATCH 12/15] anv: add descriptor support for multiplanar image/sampler

On Fri, Sep 15, 2017 at 7:11 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Signed-off-by: Lionel Landwerlin > --- > src/intel/vulkan/anv_descriptor_set.c| 107 > +-- >

Re: [Mesa-dev] Vulkan extensions

On Fri, Sep 15, 2017 at 1:31 AM, Romain Failliot wrote: > 2017-09-14 19:19 GMT-04:00 Bas Nieuwenhuizen : >> >> Also you can implement 0% of the feature list and still be vulkan 1.0 >> compliant ;) > > > Oh... is that so? > Thanks for all your

Re: [Mesa-dev] [PATCH 11/15] anv: add new formats KHR_sampler_ycbcr_conversion

On Fri, Sep 15, 2017 at 7:11 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Adding new downsampling factors for each planes. > > Signed-off-by: Lionel Landwerlin > --- > src/intel/vulkan/anv_formats.c| 158 ++ >

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

https://bugs.freedesktop.org/show_bug.cgi?id=99856 Jan Vesely changed: What|Removed |Added Status|NEW |RESOLVED

Re: [Mesa-dev] [PATCH 09/15] isl: add surface layout for yuv formats

On Fri, Sep 15, 2017 at 7:11 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Signed-off-by: Lionel Landwerlin > --- > src/intel/isl/isl_format_layout.csv | 4 > 1 file changed, 4 insertions(+) > > diff --git

Re: [Mesa-dev] [PATCH 07/15] anv: add support for VK_KHR_bind_memory2

So, yeah, I also sent an implementation of this one... I'm not sure which is better. Mine just does a bit more refactoring to keep things clean. I hadn't even though about the fact that of course you implemented it too. --Jason On Fri, Sep 15, 2017 at 7:10 AM, Lionel Landwerlin <

Re: [Mesa-dev] [PATCH 06/15] vulkan: util: add macros to extract extension/offset number from enums

On Fri, Sep 15, 2017 at 7:10 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Signed-off-by: Lionel Landwerlin > --- > src/vulkan/util/vk_util.h | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/src/vulkan/util/vk_util.h

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

On Fri, Sep 15, 2017 at 7:10 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > It will be used to store extension numbers as well. > > Signed-off-by: Lionel Landwerlin > --- > src/vulkan/util/gen_enum_to_str.py | 19 +-- > 1 file

[Mesa-dev] [Bug 102710] vkCmdBlitImage with arrayLayers > 1 fails

https://bugs.freedesktop.org/show_bug.cgi?id=102710 --- Comment #5 from mais...@archlinux.us --- This is clearly missing conformance coverage: https://github.com/KhronosGroup/VK-GL-CTS/issues/50 -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for

Re: [Mesa-dev] [PATCH 02/15] vulkan: enum generator: align function declarations/prototypes

I don't know if this really makes things any better or worse. TBH, I'd kind-of like to get rid of the dedent entirely and just have the C fully left-aligned. In any case, this is fine. Acked-by: Jason Ekstrand On Fri, Sep 15, 2017 at 7:10 AM, Lionel Landwerlin <

[Mesa-dev] [Bug 102710] vkCmdBlitImage with arrayLayers > 1 fails

https://bugs.freedesktop.org/show_bug.cgi?id=102710 --- Comment #4 from mais...@archlinux.us --- The implementation is clearly wrong. Image views are setup incorrectly for 2D arrays. The hack patch fixes the issue, but it's probably broken for other cases. -- You are receiving this mail

[Mesa-dev] [Bug 102710] vkCmdBlitImage with arrayLayers > 1 fails

https://bugs.freedesktop.org/show_bug.cgi?id=102710 --- Comment #3 from mais...@archlinux.us --- Created attachment 134269 --> https://bugs.freedesktop.org/attachment.cgi?id=134269=edit Hack patch -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee

Re: [Mesa-dev] [PATCH 2/2] anv/gen9: expose VK_EXT_post_depth_coverage

On Fri, Sep 15, 2017 at 11:18 AM, Ilia Mirkin wrote: > Signed-off-by: Ilia Mirkin > --- > > Note that the use of ICMS_INNER_CONSERVATIVE disagrees with the GL driver. > Perhaps it's more performant than ICMS_NORMAL and is otherwise permitted? > Not

[Mesa-dev] [Bug 102710] vkCmdBlitImage with arrayLayers > 1 fails

https://bugs.freedesktop.org/show_bug.cgi?id=102710 --- Comment #2 from mais...@archlinux.us --- Here's the workaround I apply. N is not REMAINING_ARRAY_LAYERS. https://github.com/Themaister/Granite/blob/master/vulkan/command_buffer.cpp#L289 Only the first layer seems to be blitted. -- You

[Mesa-dev] [Bug 102665] test_glsl_to_tgsi_lifetime.cpp:53:67: error: ‘?=>>=?UTF-8?Q?’ should be ‘?=> >=?UTF-8?Q?’ within a nested template argument list

https://bugs.freedesktop.org/show_bug.cgi?id=102665 Vinson Lee changed: What|Removed |Added Status|RESOLVED|REOPENED

Re: [Mesa-dev] [PATCH] radv: Also fix up 1D images in the image_size intrinsic.

Okay, this patch was just nonsense. Superseded by https://patchwork.freedesktop.org/patch/177162/ On Fri, Sep 15, 2017 at 9:24 PM, Bas Nieuwenhuizen wrote: > Forgotten when implementing them using 2D textures. > > This fixes: >

[Mesa-dev] [PATCH] radv: Check for GFX9 for 1D arrays in image_size intrinsic.

Only on GFX9 we implement them as 2D images. This fixes: dEQP-VK.image.image_size.1d_array.readonly_12x34 dEQP-VK.image.image_size.1d_array.readonly_1x1 dEQP-VK.image.image_size.1d_array.readonly_32x32 dEQP-VK.image.image_size.1d_array.readonly_7x1

Re: [Mesa-dev] [PATCH mesa] docs/submittingpatches: add 'test each commit' instructions

This patch is: Reviewed-by: Thomas Helland 2017-09-15 19:09 GMT+02:00 Eric Engestrom : > Signed-off-by: Eric Engestrom > --- > docs/submittingpatches.html | 12 > 1 file changed, 12 insertions(+) > >

[Mesa-dev] [PATCH] radv: Also fix up 1D images in the image_size intrinsic.

Forgotten when implementing them using 2D textures. This fixes: dEQP-VK.image.image_size.1d_array.readonly_12x34 dEQP-VK.image.image_size.1d_array.readonly_1x1 dEQP-VK.image.image_size.1d_array.readonly_32x32 dEQP-VK.image.image_size.1d_array.readonly_7x1

Re: [Mesa-dev] [PATCH mesa] i965: drop unused variables

On Friday, September 15, 2017 10:11:11 AM PDT Eric Engestrom wrote: > Signed-off-by: Eric Engestrom > --- > src/mesa/drivers/dri/i965/brw_gs.c| 1 - > src/mesa/drivers/dri/i965/intel_batchbuffer.c | 1 - > 2 files changed, 2 deletions(-) > > diff --git

[Mesa-dev] [PATCH] anv: Implement VK_KHR_bind_memory2

--- src/intel/vulkan/anv_device.c | 41 +--- src/intel/vulkan/anv_extensions.py | 1 + src/intel/vulkan/anv_image.c | 43 ++ 3 files changed, 68 insertions(+), 17 deletions(-) diff --git

[Mesa-dev] [PATCH] radv: Don't redundantly emit pipelines after secondary cmd buffer.

--- src/amd/vulkan/radv_cmd_buffer.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 068247d04d4..5da866ca64e 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++

Re: [Mesa-dev] [PATCH mesa] i965: drop unused variables

Thanks! Rb On September 15, 2017 10:12:04 AM Eric Engestrom wrote: Signed-off-by: Eric Engestrom --- src/mesa/drivers/dri/i965/brw_gs.c| 1 - src/mesa/drivers/dri/i965/intel_batchbuffer.c | 1 - 2 files changed, 2

[Mesa-dev] [PATCH 2/2] anv/gen9: expose VK_EXT_post_depth_coverage

Signed-off-by: Ilia Mirkin --- Note that the use of ICMS_INNER_CONSERVATIVE disagrees with the GL driver. Perhaps it's more performant than ICMS_NORMAL and is otherwise permitted? Not sure, so I left it as-is. Also note that there are no tests for this, and I have not

[Mesa-dev] [PATCH 1/2] spirv: add support for SPV_KHR_post_depth_coverage

Allow the capability to be exposed, and convert the new execution mode into fs state. Signed-off-by: Ilia Mirkin --- src/compiler/spirv/nir_spirv.h| 1 + src/compiler/spirv/spirv_to_nir.c | 9 + 2 files changed, 10 insertions(+) diff --git

Re: [Mesa-dev] [PATCH 2/3] egl: Wrap dri3 surface primitive around dri2 egl surface

On 15 September 2017 at 17:37, Marathe, Yogesh wrote: > Hi Emil, > >>-Original Message- >>From: Emil Velikov [mailto:emil.l.veli...@gmail.com] >>Sent: Friday, September 15, 2017 9:52 PM >>To: Marathe, Yogesh >>Cc: Eric Engestrom

[Mesa-dev] [PATCH 1/3] egl: Wrap dri3 surface primitive around dri2 egl surface

From: Yogesh Marathe Originally dri3 egl surface was wrapped around _EGLSurface. With next commit we'll add additional attributes, which will be checked from generic code. Thus in order to access that we need to use dri2_egl_surface. The name of the latter is a

[Mesa-dev] [PATCH 3/3] egl/android: Use per surface out fence

From: Zhongmin Wu Use the plumbing introduced with previous patch, to interact with the Android framework. Namely: currently we use an invalid fd or -1 for our calls to ANativeWindow::{queue,cancel}Buffer. At the same time applications (like flatland) may rely on it

[Mesa-dev] [PATCH 2/3] egl: Allow creation of per surface out fence

From: Zhongmin Wu Add plumbing to allow creation of per display surface out fence. This can be used to implement explicit sync. One user of which is Android - which will be addressed with next commit. Signed-off-by: Zhongmin Wu Signed-off-by:

Re: [Mesa-dev] [PATCH 3/3] egl: dri3 changes to support surface primitive wrap around dri2

>-Original Message- >From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of >Emil Velikov >Sent: Friday, September 15, 2017 10:00 PM >To: Marathe, Yogesh >Cc: ML mesa-dev >Subject: Re: [Mesa-dev] [PATCH 3/3]

[Mesa-dev] [PATCH 3/3] i965: Force outputs_written to contain varyings needed by stream-out.

If transform feedback is recording a varying, it needs a slot in the VUE map, regardless of whether or not the shader writes it. Together with the previous patch, this fixes: - KHR-GL45.enhanced_layouts.xfb_capture_struct The test captures a structure where the vertex shader writes the first and

[Mesa-dev] [PATCH 1/3] i965: Handle unwritten PSIZ/VIEWPORT/LAYER outputs in vec4 shaders.

This can occur if the shader is capturing some of the values from the VUE header for transform feedback, but the shader hasn't written all of them. --- src/intel/compiler/brw_vec4_visitor.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[Mesa-dev] [PATCH 2/3] i965: Compute VS/GS output VUE map from the NIR info.

unify_interfaces() only updates the NIR program info, not the copy in the gl_program itself. So, by using the old copy, we were missing out on these updates. The TCS/TES ones already did this correctly. --- src/mesa/drivers/dri/i965/brw_gs.c | 2 +- src/mesa/drivers/dri/i965/brw_vs.c | 2 +- 2

[Mesa-dev] [PATCH mesa] i965: drop unused variables

Signed-off-by: Eric Engestrom --- src/mesa/drivers/dri/i965/brw_gs.c| 1 - src/mesa/drivers/dri/i965/intel_batchbuffer.c | 1 - 2 files changed, 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_gs.c b/src/mesa/drivers/dri/i965/brw_gs.c index

[Mesa-dev] [PATCH mesa] docs/submittingpatches: add 'test each commit' instructions

Signed-off-by: Eric Engestrom --- docs/submittingpatches.html | 12 1 file changed, 12 insertions(+) diff --git a/docs/submittingpatches.html b/docs/submittingpatches.html index ecf9590a95..0581391b29 100644 --- a/docs/submittingpatches.html +++

Re: [Mesa-dev] [PATCH] i965: fix build warning on clang

On Thu, Sep 14, 2017 at 12:26 AM, Tapani Pälli wrote: > fixes following warning: >warning: format specifies type 'long' but the argument has type 'uint64_t' > (aka 'unsigned long long') > > cast is needed to avoid this turning in to another warning on 32bit build: >

Re: [Mesa-dev] [PATCH 2/2] util/u_atomic: Add implementation of __sync_val_compare_and_swap_8

On Fri, Sep 15, 2017 at 2:04 AM, Grazvydas Ignotas wrote: > It was because on some platforms ./configure test looking for > __atomic_* passes because it's a compile (not link) test, even when > the compiler inserts library calls instead of actual implementation. > It looks like

Re: [Mesa-dev] [PATCH 2/3] egl: Wrap dri3 surface primitive around dri2 egl surface

Hi Emil, >-Original Message- >From: Emil Velikov [mailto:emil.l.veli...@gmail.com] >Sent: Friday, September 15, 2017 9:52 PM >To: Marathe, Yogesh >Cc: Eric Engestrom ; mesa- >d...@lists.freedesktop.org >Subject: Re: [Mesa-dev] [PATCH

Re: [Mesa-dev] [PATCH 1/3] egl: Allow creation of per surface out fence

On 15 September 2017 at 07:36, wrote: > From: Zhongmin Wu > > Add plumbing to allow creation of per display surface out fence. > > Currently enabled only on android, since the system expects a valid > fd in ANativeWindow::{queue,cancel}Buffer. We

Re: [Mesa-dev] [PATCH 3/3] egl: dri3 changes to support surface primitive wrap around dri2

On 15 September 2017 at 07:36, wrote: > From: Yogesh Marathe > > As base is moved one level down corresponding implementation in > dri3 needs a change. > > Tested with Intel Mesa CI > > Signed-off-by: Yogesh Marathe

Re: [Mesa-dev] [PATCH 2/3] egl: Wrap dri3 surface primitive around dri2 egl surface

On 15 September 2017 at 16:48, Marathe, Yogesh wrote: > Hi Eric, > >>-Original Message- >>From: Eric Engestrom [mailto:eric.engest...@imgtec.com] >>Sent: Friday, September 15, 2017 7:13 PM >>To: Marathe, Yogesh >>Cc:

[Mesa-dev] [PATCH] anv: Implement VK_KHR_image_format_list

--- src/intel/vulkan/anv_extensions.py | 1 + src/intel/vulkan/anv_image.c | 37 +++-- 2 files changed, 36 insertions(+), 2 deletions(-) diff --git a/src/intel/vulkan/anv_extensions.py b/src/intel/vulkan/anv_extensions.py index acec785..ba7bb56 100644 ---

[Mesa-dev] Mesa 17.2.1 release candidate

Hello list, The candidate for the Mesa 17.2.1 is now available. Currently we have: - 53 queued - 0 nominated (outstanding) - and 7 rejected patches The current queue consists of: Nearly a multiple fixes, making the GLSL shader cache more robust. The RADV driver has seen a few dozen patches,

[Mesa-dev] [PATCH v2 2/2] etnaviv: remove flat shading workaround

It turned out not to be a hardware bug, but the shader compiler emitting wrong varying component use information. With that fixed we can turn flat shading back on. Signed-off-by: Lucas Stach Reviewed-by: Philipp Zabel ---

[Mesa-dev] [PATCH v2 1/2] etnaviv: fix varying interpolation

It seems that newer cores don't use the PA_ATTRIBUTES to decide if the varying should bypass the flat shading, but derive this from the component use. This fixes flat shading on GC880+. VARYING_COMPONENT_USE_POINTCOORD is a bit of a misnomer now, as it isn't only used for pointcoords, but missing

[Mesa-dev] [PATCH 09/12] intel/blorp: Internally expose surf_convert_to_uncompressed

--- src/intel/blorp/blorp_blit.c | 29 - src/intel/blorp/blorp_priv.h | 5 + 2 files changed, 21 insertions(+), 13 deletions(-) diff --git a/src/intel/blorp/blorp_blit.c b/src/intel/blorp/blorp_blit.c index 9c921cc..11c2116a 100644 ---

[Mesa-dev] [PATCH 11/12] anv/image: Use RENDER_SURFACE_STATE::X/Y Offset on SKL+

The Broadawell method of handling uncompressed views of compressed textures was to make the texture linear and have a tiled shadow copy. This isn't needed on Sky Lake because the HALIGN and VALIGN parameters are specified in surface elements and required to be a multiple of 4. This means that we

[Mesa-dev] [PATCH 08/12] anv/image: Support creating uncompressed views of compressed images

In order to get support everywhere, this gets a bit complicated. On Sky Lake and later, everything is fine because HALIGN/VALIGN are specified in surface elements and are required to be at least 4 so any offsetting we may need to do falls neatly within the heavy restrictions placed on the X/Y

[Mesa-dev] [PATCH 10/12] intel/blorp: Handle clearing compressed surfaces

--- src/intel/blorp/blorp_clear.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/src/intel/blorp/blorp_clear.c b/src/intel/blorp/blorp_clear.c index 0feebef..e8b1e32 100644 --- a/src/intel/blorp/blorp_clear.c +++ b/src/intel/blorp/blorp_clear.c @@

[Mesa-dev] [PATCH 12/12] anv: Advertise VK_KHR_maintenance2

--- src/intel/vulkan/anv_extensions.py | 1 + 1 file changed, 1 insertion(+) diff --git a/src/intel/vulkan/anv_extensions.py b/src/intel/vulkan/anv_extensions.py index acec785..52390fe 100644 --- a/src/intel/vulkan/anv_extensions.py +++ b/src/intel/vulkan/anv_extensions.py @@ -63,6 +63,7 @@

[Mesa-dev] [PATCH 07/12] anv: Add a new anv_surface_state struct

This struct represents a full surface state including the addresses of the referenced main and auxiliary surfaces (if any). This makes relocation setup substantially simpler and allows us to move 100% of the surface state setup logic into anv_image where it belongs. Before, we were manually

[Mesa-dev] [PATCH 01/12] anv/image: Add support for the new depth/stencil layouts

--- src/intel/vulkan/anv_image.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index 492b341..63e507a 100644 --- a/src/intel/vulkan/anv_image.c +++ b/src/intel/vulkan/anv_image.c @@ -581,6 +581,7 @@ anv_layout_to_aux_usage(const

[Mesa-dev] [PATCH 03/12] anv: Add support for tessellation domain origin control

--- src/intel/vulkan/genX_pipeline.c | 31 +-- 1 file changed, 21 insertions(+), 10 deletions(-) diff --git a/src/intel/vulkan/genX_pipeline.c b/src/intel/vulkan/genX_pipeline.c index 844c118..c2fa9c0 100644 --- a/src/intel/vulkan/genX_pipeline.c +++

[Mesa-dev] [PATCH 06/12] anv/image: Break surface state fill logic into a helper

This gives us a single centralized place where we take an image view and use it to fill out a surface state. Reviewed-by: Lionel Landwerlin --- src/intel/vulkan/anv_image.c | 173 + src/intel/vulkan/anv_private.h | 16

[Mesa-dev] [PATCH 05/12] anv/image: Add support for the VkImageViewUsageCreateInfoKHR struct

--- src/intel/vulkan/anv_image.c | 23 +++ 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index 63e507a..202becc 100644 --- a/src/intel/vulkan/anv_image.c +++ b/src/intel/vulkan/anv_image.c @@ -30,6

[Mesa-dev] [PATCH 04/12] anv: return device point clipping properties on GetPhysicalDeviceProperties2KHR()

From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez Reviewed-by: Jason Ekstrand --- src/intel/vulkan/anv_device.c | 8 1 file changed, 8 insertions(+) diff --git a/src/intel/vulkan/anv_device.c

[Mesa-dev] [PATCH 02/12] spirv: Flip the tessellation winding order

It's not SPIR-V that's backwards from GLSL, it's Vulkan that's backwards from GL. Let's make NIR consistent with the source language and do the flipping inside the Vulkan driver instead. Cc: Kenneth Graunke Cc: Dave Airlie ---

[Mesa-dev] [PATCH 2/3] etnaviv: add resource subregion copy

This is useful if we only need to copy part of a larger resource, mostly when using the RS engine to de-/tile on pipe transfers. Signed-off-by: Lucas Stach --- src/gallium/drivers/etnaviv/etnaviv_clear_blit.c | 27

[Mesa-dev] [PATCH 3/3] etnaviv: optimize RS transfers

Currently we are blitting the whole resource when the RS is used to de-/tile a resource. This can be very inefficient for large resources where the transfer is only changing a small part of the resource (happens a lot with glTexSubImage2D). Optimize this by only blitting the tile aligned

[Mesa-dev] [PATCH 0/3] Etnaviv partial transfer optimization

Hi all, this small series optimizes transfers where we de-/tile the resource with the RS. Currently we are always blitting the whole resource between the transfer and data buffer. This is really inefficient when the transfer only targets part of the resource. A prime example exposing the issue

[Mesa-dev] [PATCH 1/3] etnaviv: support tile aligned RS blits

The RS can blit abitrary tile aligned subregions of a resource by adjusting the buffer offset. Signed-off-by: Lucas Stach --- src/gallium/drivers/etnaviv/etnaviv_clear_blit.c | 75 +--- 1 file changed, 67 insertions(+), 8 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 2/3] egl: Wrap dri3 surface primitive around dri2 egl surface

Hi Eric, >-Original Message- >From: Eric Engestrom [mailto:eric.engest...@imgtec.com] >Sent: Friday, September 15, 2017 7:13 PM >To: Marathe, Yogesh >Cc: mesa-dev@lists.freedesktop.org >Subject: Re: [Mesa-dev] [PATCH 2/3] egl: Wrap dri3 surface primitive around

Re: [Mesa-dev] [PATCH 7/9] wayland-drm: static inline wayland_drm_buffer_get

Hi, On 15 September 2017 at 16:13, Emil Velikov wrote: > On 15 September 2017 at 15:13, Daniel Stone wrote: >> But if you avoid wl_resource_instance_of() and replace it with >> wl_resource_get_destroy_listener(), you can side-step the problem, by

Re: [Mesa-dev] [PATCH 2/3] radv: only copy the dynamic states that changed

On Thursday 14 September 2017, Samuel Pitoiset wrote: > > On 09/13/2017 06:34 PM, Fredrik Höglund wrote: > > On Wednesday 13 September 2017, Samuel Pitoiset wrote: > >> When binding a new pipeline, we applied all dynamic states > >> without checking if they really need to be re-emitted. This > >>

Re: [Mesa-dev] [RFC] NIR serialization

Hi, On 13.09.2017 19:08, Jason Ekstrand wrote: On Wed, Sep 13, 2017 at 8:42 AM, Nicolai Hähnle > wrote: On 13.09.2017 11:54, Eero Tamminen wrote: On 12.09.2017 09:55, Jordan Justen wrote: On 2017-09-11 21:44:32, Timothy

Re: [Mesa-dev] [PATCH 1/4] egl: simplify _eglDebugReport* API

On 09/15/2017 07:42 AM, Emil Velikov wrote: Hi Kyle, On 8 September 2017 at 13:09, Emil Velikov wrote: On 7 September 2017 at 19:21, Kyle Brenneman wrote: On 09/07/2017 11:56 AM, Emil Velikov wrote: On 7 September 2017 at 18:36, Kyle

[Mesa-dev] [Bug 102677] [OpenGL CTS] KHR-GL45.CommonBugs.CommonBug_PerVertexValidation fails

https://bugs.freedesktop.org/show_bug.cgi?id=102677 Juan A. Suarez changed: What|Removed |Added CC|

Re: [Mesa-dev] [Mesa-stable] [PATCH 4/4] ac/surface: match Z and stencil tile config

On 13.09.2017 22:45, Emil Velikov wrote: On 13 September 2017 at 20:48, Nicolai Hähnle wrote: On 13.09.2017 20:28, Emil Velikov wrote: Hi Nicolai, I'm doing a final run through the lists for 17.2.1 and doesn't seem like this landed in master. Just a friendly poke, in

Re: [Mesa-dev] [PATCH 7/9] wayland-drm: static inline wayland_drm_buffer_get

On 15 September 2017 at 15:13, Daniel Stone wrote: > Hi, > > On 15 September 2017 at 14:30, Emil Velikov wrote: >> On 15 September 2017 at 14:04, Daniel Stone wrote: >>> I'd be slightly wary of this current series. Currently,

Re: [Mesa-dev] [PATCH] glsl: buffer variables can be readonly and writeonly

On Fri, 2017-09-15 at 13:04 +0200, Juan A. Suarez Romero wrote: > On Fri, 2017-09-15 at 12:47 +0200, Samuel Pitoiset wrote: > > I guess this will break some piglits. > > > > > Right. I'll check them and provide fixes After running piglit with and without this patch, I didn't find any

[Mesa-dev] [Bug 100151] Front buffer drawing mode shows black window with gallium software rasterizers

https://bugs.freedesktop.org/show_bug.cgi?id=100151 --- Comment #11 from Bruce Cherniak --- I tried this again, and can only get the failure as you describe on CentOS. Seems to work on other OSs. It also appears fixed in 17.1 and 17.2 mesa releases. At this point, I

Re: [Mesa-dev] [PATCH v2 0/2] Add libunwind to travis gallium build and fix it

Am Freitag, den 15.09.2017, 13:07 +0100 schrieb Emil Velikov: > On 14 September 2017 at 11:27, Gert Wollny > wrote: > > The second version of the two patches now forces llvm-3.3 for > > "make Gallium ST Other" and no longer requires changing any > > Makefile.am. > > In

Re: [Mesa-dev] [PATCH 7/9] wayland-drm: static inline wayland_drm_buffer_get

Hi, On 15 September 2017 at 14:30, Emil Velikov wrote: > On 15 September 2017 at 14:04, Daniel Stone wrote: >> I'd be slightly wary of this current series. Currently, both >> libwayland-server.so and libwayland-client.so define >>

[Mesa-dev] [PATCH 15/15] anv: enable VK_KHR_sampler_ycbcr_conversion

Signed-off-by: Lionel Landwerlin --- src/intel/vulkan/anv_device.c | 51 ++-- src/intel/vulkan/anv_extensions.py | 1 + src/intel/vulkan/anv_formats.c | 59 ++ src/intel/vulkan/anv_image.c

  1   2   >