Re: [Mesa-dev] EGL: Question about deferred context and surface destroy

On 04/28/2017 08:08 AM, Tapani Pälli wrote: On 04/28/2017 08:05 AM, Tapani Pälli wrote: On 04/27/2017 06:59 PM, Mike Gorchak wrote: Hi Tapani, Could you please do modification like I suggested to eglut library and check how it is going? I'm asking because I see following code:

Re: [Mesa-dev] EGL: Question about deferred context and surface destroy

On 04/28/2017 08:05 AM, Tapani Pälli wrote: On 04/27/2017 06:59 PM, Mike Gorchak wrote: Hi Tapani, Could you please do modification like I suggested to eglut library and check how it is going? I'm asking because I see following code: EGLBoolean EGLAPIENTRY eglDestroySurface(EGLDisplay

Re: [Mesa-dev] EGL: Question about deferred context and surface destroy

On 04/27/2017 06:59 PM, Mike Gorchak wrote: Hi Tapani, Could you please do modification like I suggested to eglut library and check how it is going? I'm asking because I see following code: EGLBoolean EGLAPIENTRY eglDestroySurface(EGLDisplay dpy, EGLSurface surface) { _EGLDisplay

Re: [Mesa-dev] [PATCH 04/18] anv: Get rid of a bunch of uses of size_t

On Thu, Apr 27, 2017 at 9:23 AM, Juan A. Suarez Romero wrote: > On Wed, 2017-04-26 at 07:35 -0700, Jason Ekstrand wrote: > > We should only use size_t when referring to sizes of bits of CPU memory. > > Anything on the GPU or just a regular array length should be a type that

Re: [Mesa-dev] [PATCH] util/disk_cache: remove percentage based max cache limit

On 28/04/17 12:05 PM, Timothy Arceri wrote: > The more I think about it the more this seems like a bad idea. > When we were deleting old cache dirs this wasn't so bad as it > was unlikely we would ever hit the actual limit before things > were cleaned up. Now that we only start cleaning up old

Re: [Mesa-dev] [PATCH 01/18] anv/allocator: Add no-valgrind versions of state_pool_alloc/free

On Thu, Apr 27, 2017 at 9:20 AM, Juan A. Suarez Romero wrote: > On Wed, 2017-04-26 at 07:35 -0700, Jason Ekstrand wrote: > > --- > > src/intel/vulkan/anv_allocator.c | 24 +++- > > 1 file changed, 19 insertions(+), 5 deletions(-) > > > > diff --git

Re: [Mesa-dev] [PATCH 03/18] anv/allocator: Convert the state stream to pull from a state pool

On Wed, Apr 26, 2017 at 9:04 AM, Juan A. Suarez Romero wrote: > On Wed, 2017-04-26 at 07:35 -0700, Jason Ekstrand wrote: > > --- > > src/intel/vulkan/anv_allocator.c | 71 > +-- > > src/intel/vulkan/anv_cmd_buffer.c | 8 ++-- > >

[Mesa-dev] [PATCH] st/mesa: add more fallback gallium formats for GL integer formats

The VMware driver has a limited set of integer texture formats. We often have to fall back to 4-component formats when 1- or 2-component formats are missing. This fixes about 8 integer texture Piglit tests with the VMware driver on Linux. We've had this code in-house for a long time but I guess

[Mesa-dev] [PATCH] mesa: optimize color_buffer_writes_enabled()

Return as soon as we find an existing color channel that's enabled for writing. Typically, this allows us to return true on the first loop iteration intead of doing four iterations. No piglit regressions. --- src/mesa/main/clear.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-)

[Mesa-dev] [PATCH] util/disk_cache: remove percentage based max cache limit

The more I think about it the more this seems like a bad idea. When we were deleting old cache dirs this wasn't so bad as it was unlikely we would ever hit the actual limit before things were cleaned up. Now that we only start cleaning up old cache items once the limit is reached the a percentage

Re: [Mesa-dev] [PATCH 00/12] Android build cleanup/fixes

2017-04-28 3:43 GMT+08:00 Rob Herring : > This is a series of clean-ups and fixes to the Android build files. > Overall, it removes a lot of the if conditions that aren't necessary for > normal builds and also enables "mmma external/mesa3d" to build > everything for easier build

Re: [Mesa-dev] [PATCH 10/12] Android: default to building all drivers

A typo in the subject? (s/building/build/) 2017-04-28 3:43 GMT+08:00 Rob Herring : > If BOARD_GPU_DRIVERS is empty, build all the drivers. This doesn't > enable building mesa as that is controlled by including libGLES_mesa in > the product. > > Signed-off-by: Rob Herring

Re: [Mesa-dev] [PATCH 07/12] Android: remove needless conditional including of child makefiles

2017-04-28 3:43 GMT+08:00 Rob Herring : > It is not necessary to filter driver and winsys directories based on the > list of enabled drivers. Selecting the included driver libraries or not is > sufficient to control what is built. Hmm.. The benefit is faster parsing time. Though

Re: [Mesa-dev] [PATCH] anv: Drop 'x11' prefix from non-X11 WSI funcs

Rb On April 28, 2017 1:54:58 AM Chad Versace wrote: Drop it from x11_anv_wsi_image_create and x11_anv_wsi_image_free. The functions are used by Wayland WSI too. --- src/intel/vulkan/anv_wsi.c | 32 1 file changed, 16 insertions(+),

[Mesa-dev] [RFC v2 1/2] drm/i915: Engine discovery uAPI

From: Tvrtko Ursulin Engine discovery uAPI allows userspace to probe for engine configuration and features without needing to maintain the internal PCI id based database. This enables removal of code duplications across userspace components. Probing is done via the

[Mesa-dev] [RFC v2 2/2] drm/i915: Select engines via class and instance in execbuffer2

From: Tvrtko Ursulin Building on top of the previous patch which exported the concept of engine classes and instances, we can also use this instead of the current awkward engine selection uAPI. This is primarily interesting for the VCS engine selection which is a)

[Mesa-dev] [PATCH v2] scons: update for LLVM 4.0

LLVMDemangle, LLVMGlobalISel, and LLVMDebugInfoMSF are new. Also update the comment to add irreader to the list of components. CC: Reviewed-by: Chuck Atkins Signed-off-by: Ben Boeckel --- scons/llvm.py | 22

[Mesa-dev] [PATCH] scons: update for LLVM 4.0

LLVMDemangle, LLVMGlobalISel, and LLVMDebugInfoMSF are new. Also update the comment to add irreader to the list of components. CC: Reviewed-by: Chuck Atkins Signed-off-by: Ben Boeckel --- scons/llvm.py | 22

Re: [Mesa-dev] [PATCH v2 30/31] glsl: disable tree grafting optimization for bindless images

On 26/04/17 18:27, Samuel Pitoiset wrote: On 04/26/2017 10:05 AM, Nicolai Hähnle wrote: On 24.04.2017 12:36, Samuel Pitoiset wrote: Because the variable declaration holds more information than the dereference. Note that an image is considered bindless either if it has been declared in the

Re: [Mesa-dev] [PATCH] Android: fix r300g only build

2017-04-27 17:36 GMT+02:00 Emil Velikov : > On 27 April 2017 at 16:11, Rob Herring wrote: > >> >> BTW, mesa lists this as the LLVM version requirements: >> >> LLVM_REQUIRED_GALLIUM=3.3.0 > src/gallium/auxiliary/{draw,gallivm}, used by > - nouveau

Re: [Mesa-dev] [PATCH v2] swr: move msaa resolve to generalized StoreTile

On Thu, Apr 27, 2017 at 8:45 PM, Cherniak, Bruce wrote: > >> On Apr 27, 2017, at 7:38 PM, Ilia Mirkin wrote: >> >> Erm, so ... what happens if I render to FB1, then render to FB2, then >> render to FB1 again (and I have blending enabled)? Doesn't

Re: [Mesa-dev] [PATCH v2] swr: move msaa resolve to generalized StoreTile

> On Apr 27, 2017, at 7:38 PM, Ilia Mirkin wrote: > > Erm, so ... what happens if I render to FB1, then render to FB2, then > render to FB1 again (and I have blending enabled)? Doesn't the resolve > lose the per-sample information? Or does the resolve merely precompute >

Re: [Mesa-dev] [PATCH v2] swr: move msaa resolve to generalized StoreTile

Erm, so ... what happens if I render to FB1, then render to FB2, then render to FB1 again (and I have blending enabled)? Doesn't the resolve lose the per-sample information? Or does the resolve merely precompute the resolved version on the off chance that it's needed, without losing the source

[Mesa-dev] [PATCH v2] swr: move msaa resolve to generalized StoreTile

v2: Reword commit message to more closely adhere to community guidelines. This patch moves msaa resolve down into core/StoreTiles where the surface format conversion routines are available. The previous "experimental" resolve was limited to 8-bit unsigned render targets. This fixes a number of

Re: [Mesa-dev] [PATCH v02 26/37] i965: Port gen6+ 3DSTATE_WM to genxml.

On Wed, Apr 26, 2017 at 11:15:45PM -0700, Kenneth Graunke wrote: > On Monday, April 24, 2017 3:19:21 PM PDT Rafael Antognolli wrote: > [snip] > > diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c > > b/src/mesa/drivers/dri/i965/genX_state_upload.c > > index 1b9dedf..0f7a222 100644 > >

Re: [Mesa-dev] [PATCH 3/3] glsl: reject image qualifiers with non-image types inside uniform blocks

On 28/04/17 10:06, Timothy Arceri wrote: On 27/04/17 02:50, Samuel Pitoiset wrote: Interface blocks don't allow to declare opaque types and atomic counters/images are forbidden inside structures. The above comment seems unrelated to me. With that removed and the below comment changed to

Re: [Mesa-dev] [PATCH 3/3] glsl: reject image qualifiers with non-image types inside uniform blocks

On 27/04/17 02:50, Samuel Pitoiset wrote: Interface blocks don't allow to declare opaque types and atomic counters/images are forbidden inside structures. The above comment seems unrelated to me. With that removed and the below comment changed to something like: Fixes the following

[Mesa-dev] [PATCH] anv: Drop 'x11' prefix from non-X11 WSI funcs

Drop it from x11_anv_wsi_image_create and x11_anv_wsi_image_free. The functions are used by Wayland WSI too. --- src/intel/vulkan/anv_wsi.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/src/intel/vulkan/anv_wsi.c

Re: [Mesa-dev] [PATCH 08/21] anv/allocator: Add a BO cache

On Fri 14 Apr 2017, Jason Ekstrand wrote: > This cache allows us to easily ensure that we have a unique anv_bo for > each gem handle. We'll need this in order to support multiple-import of > memory objects and semaphores. > > v2 (Jason Ekstrand): > - Reject BO imports if the size doesn't match

Re: [Mesa-dev] [PATCH 03/21] anv: Set EXEC_OBJECT_ASYNC when available

On Fri 14 Apr 2017, Jason Ekstrand wrote: > --- > src/intel/vulkan/anv_allocator.c | 3 +++ > src/intel/vulkan/anv_device.c| 5 + > src/intel/vulkan/anv_private.h | 1 + > src/intel/vulkan/anv_wsi.c | 1 + > 4 files changed, 10 insertions(+) Patch 3 is Reviewed-by: Chad Versace

[Mesa-dev] [Bug 100854] YUV to RGB Color Space Conversion result is not precise

https://bugs.freedesktop.org/show_bug.cgi?id=100854 --- Comment #1 from Matt Turner --- Do you have a test program that shows the issue? -- You are receiving this mail because: You are the assignee for the bug.___ mesa-dev mailing

Re: [Mesa-dev] [PATCH 1/2] wglgears.c: add -srgb option

Series looks good. One typo below: >From: Brian Paul >Sent: Thursday, April 27, 2017 10:19 AM >To: mesa-dev@lists.freedesktop.org >Cc: Charmaine Lee; Neha Bhende >Subject: [PATCH 1/2] wglgears.c: add -srgb option >To test sRGB pixel format selection and sRGB rendering.

Re: [Mesa-dev] [PATCH v3 03/14] spirv: Add support for SPV_KHR_multiview

Patch 1-3 are Reviewed-by: Bas Nieuwenhuizen On Thu, Apr 27, 2017 at 6:31 PM, Jason Ekstrand wrote: > Reviewed-by: Iago Toral Quiroga > --- > src/compiler/spirv/nir_spirv.h | 1 + > src/compiler/spirv/spirv_to_nir.c | 4

Re: [Mesa-dev] [PATCH] glsl: set vector_elements to 1 for samplers

On 26.04.2017 13:49, Samuel Pitoiset wrote: I don't see any reasons why vector_elements is 1 for images and 0 for samplers. This increases consistency and allows to clean up some code a bit. This will also help for ARB_bindless_texture. No piglit regressions with RadeonSI. Signed-off-by:

[Mesa-dev] [PATCH 07/12] Android: remove needless conditional including of child makefiles

It is not necessary to filter driver and winsys directories based on the list of enabled drivers. Selecting the included driver libraries or not is sufficient to control what is built. Signed-off-by: Rob Herring --- Android.mk | 8 --

[Mesa-dev] [PATCH 12/12] Android: Drop linking libgcc

Including libgcc breaks on Android O (master). Not exactly sure if it is still needed on older versions or not. N is build with clang, so it should be fine at least. Signed-off-by: Rob Herring --- src/gallium/targets/dri/Android.mk | 4 1 file changed, 4 deletions(-) diff

[Mesa-dev] [PATCH 11/12] Android: Add LLVM support for Android O

Android O moves to LLVM 3.9 and also has some differences in header dependencies as LLVM has moved to blueprint files. It seems libLLVMCore was only needed for header dependencies. Signed-off-by: Rob Herring --- Android.common.mk| 5 +++--

[Mesa-dev] [PATCH 05/12] Android: drop LLVM support on Lollipop

Mesa no longer supports LLVM 3.5 for any targets we support. Android-x86 adds support for llvmpipe which could work, but android-x86 for L is using mesa 11.0 anyway. Dropping this support enables clean-up of libelf dependencies. Signed-off-by: Rob Herring --- Android.common.mk

[Mesa-dev] [PATCH 10/12] Android: default to building all drivers

If BOARD_GPU_DRIVERS is empty, build all the drivers. This doesn't enable building mesa as that is controlled by including libGLES_mesa in the product. Signed-off-by: Rob Herring --- Android.mk | 8 1 file changed, 8 insertions(+) diff --git a/Android.mk b/Android.mk

[Mesa-dev] [PATCH 08/12] Android: Fix swrast only build

A build of only swrast is broken as the Android EGL now depends on libdrm as does GBM. While we could make EGL conditionally depend on libdrm, we probably want to enable kms_dri winsys as well and that will need libdrm enabled. So just always enable libdrm and simplify the Android makefiles a bit.

[Mesa-dev] [PATCH 01/12] android: drop static linking of R600 LLVM libraries

From: Mauro Rossi Inspired by Chih-Wei Huang and Zhen Wu similar patches Linking against llvm with both static and shared may be avoided, provided that libLLVM shared library for device supports whole static R600/AMDGPU libraries, necessary for radeonsi/amdgpu.

[Mesa-dev] [PATCH 00/12] Android build cleanup/fixes

This is a series of clean-ups and fixes to the Android build files. Overall, it removes a lot of the if conditions that aren't necessary for normal builds and also enables "mmma external/mesa3d" to build everything for easier build testing. This fixes LLVM enabled builds on master though that

[Mesa-dev] [PATCH 03/12] Android: amd: use exported include dirs instead of explicit includes

Add exported include paths rather than explicitly adding the includes in each user of the common AMD libs. Signed-off-by: Rob Herring --- src/amd/Android.addrlib.mk | 6 ++ src/amd/Android.common.mk| 3 +++ src/gallium/drivers/r600/Android.mk

[Mesa-dev] [PATCH 06/12] Android: rework libelf dependencies

Add libelf as a library dependency rather than explicitly listing its include paths. This should work for Android M and later which have the necessary exported directories in libelf. Signed-off-by: Rob Herring --- Android.common.mk | 2 --

[Mesa-dev] [PATCH 04/12] Android: amd/common: fix dependency on libmesa_nir

Building libmesa_amd_common fails with: external/mesa/src/amd/common/ac_shader_info.c:23:10: fatal error: 'nir/nir.h' file not found ^ external/mesa/src/compiler/nir/nir.h:48:10: fatal error: 'nir_opcodes.h' file not found ^ libmesa_amd_common now depends on libmesa_nir, so

[Mesa-dev] [PATCH 09/12] Android: push driver build details to driver makefiles

src/gallium/targets/dri/Android.mk contains lots of conditional for individual drivers. Let's move these details into the individual driver makefiles. In the process, align the make driver conditionals with automake (i.e. HAVE_GALLIUM_*). Signed-off-by: Rob Herring ---

[Mesa-dev] [PATCH 02/12] android: define required __STDC* macros as cflags

From: Mauro Rossi Necessary to fix the following radeonsi building errors: In file included from external/mesa/src/gallium/drivers/radeonsi/si_blit.c:24: In file included from external/mesa/src/gallium/drivers/radeonsi/si_pipe.h:29: In file included from

Re: [Mesa-dev] [PATCH 1/4] r600g: avoid redundant DB registerupdates

On 27.04.2017 11:32, Marc Dietrich wrote: > Am Donnerstag, 27. April 2017, 10:05:32 CEST schrieb Marc Dietrich: >> Am Mittwoch, 26. April 2017, 19:48:46 CEST schrieb Constantine Kharlamov: >>> On 26.04.2017 17:51, Marc Dietrich wrote: Am Mittwoch, 26. April 2017, 16:36:39 CEST schrieb

[Mesa-dev] [PATCH 13/15] travis: add Gallium state-tracker targets

From: Emil Velikov Split into OpenCL and others, since the former is quite time consuming. v2: - explicitly enable/disable components - build libvdpau 1.1 requirement - enable st/vdpau - build libva 1.6.2 (API 0.38) requirement Signed-off-by: Emil Velikov

[Mesa-dev] [PATCH 15/15] travis: bump MAKEFLAGS to -j4

From: Emil Velikov The instance has 2 cores, yet bumping the jobs to 4 gives a minor speed improvement. Signed-off-by: Emil Velikov --- Yay, new patch. --- .travis.yml | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-)

[Mesa-dev] [PATCH 09/15] travis: add separate "scons" and "scons llvm" targets

From: Emil Velikov The former does not require any LLVM, while the latter uses LLVM 3.3. This way we'll quickly catch any LLVM 3.3+ functionality that gets introduced where it shouldn't. Add the full list of addons for each build permutation. v2: Keep libedit-dev,

[Mesa-dev] [PATCH 12/15] travis: split the make target to three separate ones

From: Emil Velikov Split the target to allow faster builds for each run. The overall build time will be more, yet Travis runs multiple builds in parallel so we're limited by the slowest one. Things are split roughly as: - DRI loaders, classic DRI drivers, classic

[Mesa-dev] [PATCH 10/15] travis: add "scons swr" to the build matrix

From: Emil Velikov Requires GCC 5.0 (due to the C++14 requirement) and LLVM 3.9. v2: Enable the target, add libedit-dev, rework check target. v3: Comment the current check target, add -j4 SCONSFLAGS, quote OVERRIDE variables. Cc: Tim Rowley

[Mesa-dev] [PATCH 07/15] travis: rework "if test" blocks in the script section

From: Emil Velikov Split the "if test" blocks so that we get more sensible output in case of a failure. Signed-off-by: Emil Velikov --- .travis.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.travis.yml

[Mesa-dev] [PATCH 14/15] travis: enable wayland support

From: Emil Velikov Signed-off-by: Emil Velikov --- .travis.yml | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/.travis.yml b/.travis.yml index 86f88e86b05..32ef3e27651 100644 --- a/.travis.yml +++

[Mesa-dev] [PATCH 05/15] travis: automatically manage ccache caching

From: Emil Velikov According to the manual "If you are using ccache, use: language: c # or other C/C++ variants cache: ccache to cache $HOME/.ccache and automatically add /usr/lib/ccache to your $PATH." Signed-off-by: Emil Velikov

[Mesa-dev] [PATCH 11/15] travis: add "make swr" to the build matrix

From: Emil Velikov v2: Quote OVERRIDE variables. Signed-off-by: Emil Velikov --- .travis.yml | 39 --- 1 file changed, 36 insertions(+), 3 deletions(-) diff --git a/.travis.yml b/.travis.yml index

[Mesa-dev] [PATCH 08/15] travis: split out matrix from env

From: Emil Velikov With next commits we'll add a couple of more options. v2: Rework check target. v3: Comment the current check target, add -j4 SCONSFLAGS Signed-off-by: Emil Velikov --- .travis.yml | 20 +++- 1 file

[Mesa-dev] [PATCH 06/15] travis: remove unused -dev packages

From: Emil Velikov We effectively override libdrm-dev and libxcb-dri2-0-dev since we build and install the package locally. Signed-off-by: Emil Velikov --- .travis.yml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.travis.yml

[Mesa-dev] [PATCH 01/15] travis: explicitly LD_LIBRARY_PATH the local libraries

From: Emil Velikov Some of the libraries may be dlopened, which may not always work due to the non-standard prefix that we're using. Signed-off-by: Emil Velikov --- Adding this is a good idea imho, regardless if we opt for or against

[Mesa-dev] [PATCH 04/15] travis: enable apt cache

From: Emil Velikov Signed-off-by: Emil Velikov --- .travis.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.travis.yml b/.travis.yml index e317a027233..061aed1bc7c 100644 --- a/.travis.yml +++ b/.travis.yml @@ -4,6 +4,7 @@ sudo:

[Mesa-dev] [PATCH 02/15] travis: replace Trusty-based LLVM toolchain apt-get with apt addon

From: Andres Gomez Trusty's LLVM toochain repository was whitelisted some time ago. See: https://github.com/travis-ci/apt-source-whitelist/commit/479067c5e74cb0c1e2419209179b1afe2edce274 Signed-off-by: Andres Gomez [Emil Velikov] - set sudo to false -

[Mesa-dev] [PATCH 03/15] travis: add the possibility of using the txc-dxtn library

From: Andres Gomez The txc-dxtn library implements the patented S3 Texture Compression algorithm. By default it won't be used but we add the possibility of setting the USE_TXC_DXTN variable to yes in the travis web UI so it will be installed and used for the scons tests. Cc:

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

https://bugs.freedesktop.org/show_bug.cgi?id=90264 --- Comment #74 from Vasily Khoruzhick --- (In reply to Joseph Yasi from comment #73) > I'm still seeing this with Mesa 17.1.0rc2 and xf86-video-intel from git > 2017-04-18c72bb27a3a68ecc616ce2dc8e9a1d20354504562 on

[Mesa-dev] [PATCH 07/22] anv/image: Initialize the clear values buffer

Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_image.c | 75 +++--- src/intel/vulkan/anv_private.h | 5 +++ 2 files changed, 69 insertions(+), 11 deletions(-) diff --git a/src/intel/vulkan/anv_image.c

[Mesa-dev] [PATCH 20/22] anv/pass: Get rid of anv_subpass_usage

Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_pass.c| 14 -- src/intel/vulkan/anv_private.h | 10 -- 2 files changed, 24 deletions(-) diff --git a/src/intel/vulkan/anv_pass.c b/src/intel/vulkan/anv_pass.c index 02b8f03fa5..de279d9d64

[Mesa-dev] [PATCH 09/22] anv/cmd_buffer: Don't partially fast-clear image layers

Signed-off-by: Nanley Chery --- src/intel/vulkan/genX_cmd_buffer.c | 18 +++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index 0ea378fde2..a981b00f67 100644 ---

[Mesa-dev] [PATCH 22/22] anv: Predicate fast-clear resolves

There's no image layout to represent a full-RT-cleared color attachment. That's one reason we can end up with redundant resolves. Testing has shown that such resolves can measurably hurt performance and that predicating them can avoid the penalty. Signed-off-by: Nanley Chery

[Mesa-dev] [PATCH 21/22] intel/blorp: Allow BLORP calls to be predicated

Signed-off-by: Nanley Chery --- src/intel/blorp/blorp.h | 3 +++ src/intel/blorp/blorp_genX_exec.h | 3 +++ 2 files changed, 6 insertions(+) diff --git a/src/intel/blorp/blorp.h b/src/intel/blorp/blorp.h index 8b8227bb1c..36acbb8222 100644 ---

[Mesa-dev] [PATCH 18/22] anv/cmd_buffer: Perform color buffer layout transitions

Signed-off-by: Nanley Chery --- src/intel/vulkan/genX_cmd_buffer.c | 65 ++ 1 file changed, 52 insertions(+), 13 deletions(-) diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index

[Mesa-dev] [PATCH 08/22] anv/cmd_buffer: Always enable CCS_D in render passes

The lifespan of the fast-clear data will surpass the render pass scope. We need CCS_D to be enabled in order to invalidate blocks previously marked as cleared and to sample cleared data correctly. Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_blorp.c | 15

[Mesa-dev] [PATCH 03/22] intel/isl: Add surface state clear value information

This will be used to load and store clear values from surface state objects. Signed-off-by: Nanley Chery --- src/intel/isl/isl.c | 9 + src/intel/isl/isl.h | 4 2 files changed, 13 insertions(+) diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c index

[Mesa-dev] [PATCH 19/22] anv/blorp: Stop resolving CCS implicitly

With the previous patch, resolves are additionally performed on layout transitions. Remove the now unnecessary implicit resolves within render passes. Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_blorp.c | 139 --- 1 file

[Mesa-dev] [PATCH 15/22] anv/cmd_buffer: Adjust the image view reloc function

Make the function take in an image instead of an image view. This enables us to record relocations for surfaces states created outside of the anv_CreateImageView path. Signed-off-by: Nanley Chery --- src/intel/vulkan/genX_cmd_buffer.c | 45

[Mesa-dev] [PATCH 16/22] anv/cmd_buffer: Add transition_color_buffer()

Signed-off-by: Nanley Chery --- src/intel/vulkan/genX_cmd_buffer.c | 93 ++ 1 file changed, 93 insertions(+) diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index d5cc358aec..1ae0c3256e 100644

[Mesa-dev] [PATCH 12/22] anv/gpu_memcpy: Add a lighter-weight memcpy path

We're now performing a GPU memcpy in more places to copy small amounts of data. Add a path to thrash less state. Signed-off-by: Nanley Chery --- src/intel/vulkan/genX_gpu_memcpy.c | 38 ++ 1 file changed, 38 insertions(+) diff --git

[Mesa-dev] [PATCH 11/22] anv/cmd_buffer: Ensure the fast clear values are correct

Signed-off-by: Nanley Chery --- src/intel/vulkan/genX_cmd_buffer.c | 76 ++ 1 file changed, 76 insertions(+) diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index e3b1687121..4698270abb 100644

[Mesa-dev] [PATCH 14/22] anv/blorp: Add a surface-state-based CCS resolve function

This will be used in conjunction with the buffer of clear values. Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_blorp.c | 41 + src/intel/vulkan/anv_private.h | 6 ++ 2 files changed, 47 insertions(+) diff --git

[Mesa-dev] [PATCH 17/22] anv: Add anv_get_sliceCount()

Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_private.h | 4 1 file changed, 4 insertions(+) diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h index 667176270a..819fbea567 100644 --- a/src/intel/vulkan/anv_private.h +++

[Mesa-dev] [PATCH 10/22] anv/cmd_buffer: Disable fast clears in the GENERAL layout

Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_pass.c| 22 ++ src/intel/vulkan/anv_private.h | 1 + src/intel/vulkan/genX_cmd_buffer.c | 20 ++-- 3 files changed, 37 insertions(+), 6 deletions(-) diff --git

[Mesa-dev] [PATCH 04/22] anv: Add color auxiliary buffer helpers

Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_private.h | 42 ++ 1 file changed, 42 insertions(+) diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h index dbe1e0475e..ac71537e88 100644 ---

[Mesa-dev] [PATCH 05/22] anv/image: Append CCS/MCS with a clear value buffer

Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_image.c | 67 1 file changed, 67 insertions(+) diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index cf34dbe3b0..8d946e8e93 100644 ---

[Mesa-dev] [PATCH 13/22] blorp/clear: Add a binding-table-based CCS resolve function

Signed-off-by: Nanley Chery --- src/intel/blorp/blorp.h | 9 ++ src/intel/blorp/blorp_clear.c | 64 --- 2 files changed, 57 insertions(+), 16 deletions(-) diff --git a/src/intel/blorp/blorp.h b/src/intel/blorp/blorp.h

[Mesa-dev] [PATCH 02/22] intel/isl: Only create a CCS buffer if the image supports rendering

This prevents assertion failures when initializing the clear value buffer on images with the E5B9G9R9 format. Signed-off-by: Nanley Chery --- src/intel/isl/isl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/isl/isl.c

[Mesa-dev] [PATCH 06/22] anv/image: Remove incorrect assertion in anv_BindImage

According to the Linux kernel sources, the ioctl in anv_gem_mmap() will not fail if the size isn't a multiple of 4KB. Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_image.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git

[Mesa-dev] [PATCH 01/22] intel/isl: Limit CCS to one subresource on gen7

Signed-off-by: Nanley Chery --- src/intel/isl/isl.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c index f89f351c15..ce5b35c47c 100644 --- a/src/intel/isl/isl.c +++ b/src/intel/isl/isl.c @@ -1610,14

[Mesa-dev] [PATCH 00/22] anv: Move CCS resolves to layout transitions

Resolves of CCS-enabled image subresources are currently tied to the scope of a subpass. This can cause us to lose clear data compression prematurely in some cases. For example, an application can record a render pass that only clears followed by another render pass that only draws. The driver

Re: [Mesa-dev] [PATCH 0/2] anv/i965: drop libdrm dependency completely

On 27/04/17 08:20, Eric Anholt wrote: Emil Velikov writes: On 25 April 2017 at 23:56, Lionel Landwerlin wrote: Hi, While working with changes that span from kernel to user space, I've been wondering whether we need to depend on

Re: [Mesa-dev] [PATCH] autogen.sh: set default sendemail.to

On Thu, Apr 27, 2017 at 1:32 PM, Kenneth Graunke wrote: > On Monday, April 24, 2017 12:29:32 PM PDT Jason Ekstrand wrote: >> This seems like something that would be more appropriate to put on a >> "getting started" page than autogen.sh. The very last thing I (as a user >>

Re: [Mesa-dev] [PATCH] autogen.sh: set default sendemail.to

On Monday, April 24, 2017 12:29:32 PM PDT Jason Ekstrand wrote: > This seems like something that would be more appropriate to put on a > "getting started" page than autogen.sh. The very last thing I (as a user > of it) would expect autogen.sh to do is monkey with my git config; local or >

Re: [Mesa-dev] [PATCH 2/2] wglinfo: query/print sRGB and float pixel format info

series looks good to me. Reviewed-by : Neha Bhende Regards, Neha From: Brian Paul Sent: Thursday, April 27, 2017 10:19:07 AM To: mesa-dev@lists.freedesktop.org Cc: Charmaine Lee; Neha Bhende Subject: [PATCH 2/2]

Re: [Mesa-dev] [PATCH] i965: Set point rasterization rule to UPPER_RIGHT on Gen6-7.5.

Makes sense to me. Reviewed-by: Rafael Antognolli On Wed, Apr 26, 2017 at 11:14:47PM -0700, Kenneth Graunke wrote: > Gen4-5 and Gen8+ already set this, but Gen6-7.5 did not. We ought to > be consistent - the answer depends on the API, not the hardware generation. >

Re: [Mesa-dev] [PATCH] Android: fix r300g only build

On Thu, Apr 27, 2017 at 12:31 PM, Rob Herring wrote: > On Thu, Apr 27, 2017 at 11:18 AM, Ilia Mirkin wrote: >> On Thu, Apr 27, 2017 at 11:36 AM, Emil Velikov >> wrote: >>> On 27 April 2017 at 16:11, Rob Herring

[Mesa-dev] [PATCH 1/2] wglgears.c: add -srgb option

To test sRGB pixel format selection and sRGB rendering. This involves choosing a new pixel format, creating a new context, etc. --- src/wgl/wglgears.c | 91 -- 1 file changed, 88 insertions(+), 3 deletions(-) diff --git a/src/wgl/wglgears.c

[Mesa-dev] [PATCH 2/2] wglinfo: query/print sRGB and float pixel format info

Print the columns in the same place as glxinfo. Use 3 digit hexadecimal field for printing pixel format number. Remove the 'caveat' field which is non-existant in WGL. This also fixes a minor bug when we query WGL_PIXEL_TYPE_ARB. We previously stored the result in info->pfd.iPixelType which is a

Re: [Mesa-dev] [PATCH 1/1] clover: Fix build since clang r301442

On Thu, 2017-04-27 at 09:37 -0700, Francisco Jerez wrote: > Jan Vesely writes: > > > Signed-off-by: Jan Vesely > > --- > > src/gallium/state_trackers/clover/llvm/compat.hpp | 2 ++ > > src/gallium/state_trackers/clover/llvm/invocation.cpp | 2

Re: [Mesa-dev] [PATCH 1/1] clover: Fix build since clang r301442

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

[Mesa-dev] [PATCH 1/2] renderonly: close transfer prime_fd

prime_fd is only used to transfer the scanout buffer to the GPU inside renderonly_create_kms_dumb_buffer_for_resource. It should be closed immediately to avoid leaking the DMA-BUF file handle. Signed-off-by: Philipp Zabel --- src/gallium/auxiliary/renderonly/renderonly.c

[Mesa-dev] [PATCH 2/2] renderonly: drop resources on destroy

The renderonly_scanout holds a reference on its prime pipe resource, which should be released when it is destroyed. If it was created by renderonly_create_kms_dumb_buffer_for_resource, the dumb BO also has to be destroyed. Signed-off-by: Philipp Zabel ---

Re: [Mesa-dev] [PATCH 18/18] anv: Drop the instruction pool block size

Reviewed-by: Juan A. Suarez Romero On Wed, 2017-04-26 at 07:35 -0700, Jason Ekstrand wrote: > Now that we can allocate states larger than the block size, we no longer > need a block size of 1MB which can be rather wasteful. > --- > src/intel/vulkan/anv_device.c | 3 +-- >

  1   2   >