Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

On Thu, Jul 6, 2017 at 8:12 PM, Alex Deucher wrote: > On Thu, Jul 6, 2017 at 1:13 PM, Jan Vesely wrote: >> On Thu, 2017-07-06 at 12:09 +1000, Dave Airlie wrote: >>> From: Dave Airlie >>> >>> Use family, but only set xnack+ for

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

On Thu, Jul 6, 2017 at 10:14 PM, Matt Arsenault wrote: > >> On Jul 6, 2017, at 13:08, Dave Airlie wrote: >> >> On 7 July 2017 at 05:07, Matt Arsenault wrote: >>> On Jul 5, 2017, at 19:09, Dave Airlie wrote:

[Mesa-dev] [PATCH] st/dri: always initialize 'opencl_func_mutex' in struct dri_screen

The 'opencl_func_mutex' is destroyed in dri_destroy_screen_helper() unconditionally, so make sure it is always initialized to avoid any undefined behaviour. Signed-off-by: Aleksander Morgado --- src/gallium/state_trackers/dri/dri2.c | 1 +

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

> On Jul 6, 2017, at 13:08, Dave Airlie wrote: > > On 7 July 2017 at 05:07, Matt Arsenault wrote: >> >>> On Jul 5, 2017, at 19:09, Dave Airlie wrote: >>> >>> From: Dave Airlie >>> >>> Use family, but only set

[Mesa-dev] [PATCH 2/2] anv/image: Disable CCS if the image doesn't support rendering

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

[Mesa-dev] [PATCH 1/2] Revert "intel/isl: Only create a CCS buffer if the image supports rendering"

This reverts commit 8aaa13467dc289d35dc7900ab9fab9a7689c4178, which was based on an incorrect assumption. Unlike the restriction placed on image views in the Vulkan API, OpenGL allows you to render to texture views whose formats differ from the originals. Bugzilla:

[Mesa-dev] [PATCH mesa] build systems: move git_sha1_gen.sh to bin/

There was no reason for this script to live outside the scripts directory. Suggested-by: Brian Paul Signed-off-by: Eric Engestrom --- Makefile.am| 2 +- git_sha1_gen.sh => bin/git_sha1_gen.sh | 2 +- src/Makefile.am

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

On 7 July 2017 at 05:07, Matt Arsenault wrote: > >> On Jul 5, 2017, at 19:09, Dave Airlie wrote: >> >> From: Dave Airlie >> >> Use family, but only set xnack+ for gfx9. >> > > The driver shouldn’t be explicitly setting this. This should

[Mesa-dev] [PATCH] radv: don't overallocate depth/stencil formats

From: Dave Airlie For depth/stencil formats the surface layer allocates the stencil separately, so we don't need to include it in the bpe. This reduces the side of d32s8 allocates to something closer to pro. Signed-off-by: Dave Airlie ---

Re: [Mesa-dev] [PATCH 1/3] st/mesa: find proper mipmap level in st_ClearTexSubImage()

>From: Brian Paul >To: >CC: Neha Bhende >The Piglit arb_clear_texture-error test creates a texture with only >a 1x1 image at level=1, then tries to clear level 0 (non-existant) s/non-existant/non-existent/ >and level 1

[Mesa-dev] [PATCH 5/6] radv: reorder and add passes to match i965

From: Connor Abbott This makes the radv shader pipeline much closer to brw_preprocess_nir(). The main changes are: - Now we call nir_split_var_copies(), which is necessary for nir_lower_var_copies() to work correctly. The SPIR-V frontend never generates variable copies, so

[Mesa-dev] [PATCH 6/6] radv: don't lower indirect derefs

From: Connor Abbott Radeonsi doesn't either. As of the last commit, these should be handled properly as long as LLVM has scratch support. We also should use nir_lower_io_to_temporaries() for inputs instead of generating an if-ladder, since that should be more efficient. ---

[Mesa-dev] [PATCH 4/6] ac/nir: rewrite shared variable handling

From: Connor Abbott Similar to before, do the direct NIR->LLVM translation instead of lowering to an array then back to a variable. This should fix indirect dereferences, make shared variables more tightly packed, and make LLVM's alias analysis more precise. ---

[Mesa-dev] [PATCH 3/6] ac/nir: rewrite local variable handling

From: Connor Abbott The old way was very TGSI-based, and couldn't handle indirect dereferences at all. Instead, pass through the type information NIR has about local variables to LLVM, and translate NIR dereferences directly into the equivalent GEP instructions in LLVM. ---

[Mesa-dev] [PATCH 0/6] radv: Rewrite local & shared variable handling

From: Connor Abbott This series grew out of trying to get rid of the copy-n-pasted index calculation code in radv's NIR-to-LLVM path, in particular in radv_get_deref_offset(). I realized for IO it's probably better to switch to using nir_lower_io after Nicolai's series, but

[Mesa-dev] [PATCH 1/6] nir: don't segfault when printing variables with no name

From: Connor Abbott While normally we give variables whose name field is NULL a temporary name when called from nir_print_shader(), when we were calling from nir_print_instr() we never bothered, meaning that we just segfaulted when trying to print out instructions with such

[Mesa-dev] [PATCH 2/6] nir/lower_io_to_temporaries: don't set compact on shadow vars

From: Connor Abbott The compact flag doesn't make sense on local variables, since the packing on them is up to the driver. This fixes nir_validate assertions in some cases, particularly when lower_io_to_temporaries is used on per-vertex inputs/outputs. ---

Re: [Mesa-dev] [PATCH] swr/rast: Correctly allocate SWR_STATS memory as cacheline aligned

Reviewed-by: Bruce Cherniak > On Jul 6, 2017, at 1:55 PM, Tim Rowley wrote: > > Cacheline alignment of SWR_STATS to prevent sharing of cachelines > between threads (performance). > > Gets rid of gcc-7.1 warning about using c++17's

Re: [Mesa-dev] [PATCH 2/3] swr/rast: don't use _mm256_fmsub_ps in AVX code

Reviewed-by: Bruce Cherniak > On Jul 6, 2017, at 11:26 AM, Tim Rowley wrote: > > --- > src/gallium/drivers/swr/rasterizer/common/simdlib_256_avx.inl | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git

Re: [Mesa-dev] [PATCH 3/3] swr/rast: remove unused variables

Reviewed-by: Bruce Cherniak > On Jul 6, 2017, at 11:26 AM, Tim Rowley wrote: > > --- > src/gallium/drivers/swr/rasterizer/jitter/fetch_jit.cpp | 2 -- > src/gallium/drivers/swr/rasterizer/jitter/streamout_jit.cpp | 2 -- > 2 files

Re: [Mesa-dev] [PATCH 1/3] swr/rast: _mm*_undefined_* implementations for gcc<4.9

Reviewed-by: Bruce Cherniak > On Jul 6, 2017, at 11:26 AM, Tim Rowley wrote: > > Define these in terms of setzero for ancient gcc versions which don't > have the undefined intrinsics. > > Cc: mesa-sta...@lists.freedesktop.org > --- >

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

> On Jul 5, 2017, at 19:09, Dave Airlie wrote: > > From: Dave Airlie > > Use family, but only set xnack+ for gfx9. > The driver shouldn’t be explicitly setting this. This should be set as part of the subtarget chosen -Matt

Re: [Mesa-dev] [PATCH v6 1/3] util: Add util_strlcpy

2017-07-06 20:52 GMT+02:00 Robert Foss : > On Wed, 2017-07-05 at 08:46 -0600, Brian Paul wrote: >> On 07/05/2017 12:57 AM, Robert Foss wrote: >> > Add local strlcpy implementation. >> > >> > Signed-off-by: Robert Foss >> > --- >> > Changes

[Mesa-dev] [PATCH] swr/rast: Correctly allocate SWR_STATS memory as cacheline aligned

Cacheline alignment of SWR_STATS to prevent sharing of cachelines between threads (performance). Gets rid of gcc-7.1 warning about using c++17's over-aligned new feature. Cc: mesa-sta...@lists.freedesktop.org --- src/gallium/drivers/swr/rasterizer/core/api.cpp | 6 +++---

Re: [Mesa-dev] [st/va] Fix leak in VAAPI subpictures

On Thu, Jul 6, 2017 at 2:37 PM, Andres Gomez wrote: > It looks like we could want this into -stable (?) Yes, I think so. Alex > > On Thu, 2017-07-06 at 21:10 +0300, Andres Gomez wrote: >> On Thu, 2017-06-22 at 09:25 +, Namburu, Chandu-babu wrote: >> > From: Chandu Babu N

Re: [Mesa-dev] [PATCH v6 1/3] util: Add util_strlcpy

On Wed, 2017-07-05 at 08:46 -0600, Brian Paul wrote: > On 07/05/2017 12:57 AM, Robert Foss wrote: > > Add local strlcpy implementation. > > > > Signed-off-by: Robert Foss > > --- > > Changes since v5: > >    Actually include changes from v5 in patch > > > > Changes

Re: [Mesa-dev] [st/va] Fix leak in VAAPI subpictures

It looks like we could want this into -stable (?) On Thu, 2017-07-06 at 21:10 +0300, Andres Gomez wrote: > On Thu, 2017-06-22 at 09:25 +, Namburu, Chandu-babu wrote: > > From: Chandu Babu N > > Subject: [PATCH] [st/va] Fix leak in VAAPI subpictures > >   > > sampler view

Re: [Mesa-dev] [PATCH 1/4] radeonsi: fix DCC fast clear for luminance and alpha formats

Marek, would we want this series in -stable or we shouldn't bother ? On Tue, 2017-06-20 at 19:06 +0200, Marek Olšák wrote: > From: Marek Olšák > > I reproduced this bug on Polaris11 and Raven. > > I can't get this bug on Fiji. The reason might be that Fiji doesn't use > 2D

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

On Thu, Jul 6, 2017 at 1:13 PM, Jan Vesely wrote: > On Thu, 2017-07-06 at 12:09 +1000, Dave Airlie wrote: >> From: Dave Airlie >> >> Use family, but only set xnack+ for gfx9. >> >> Signed-off-by: Dave Airlie >> --- >>

Re: [Mesa-dev] [st/va] Fix leak in VAAPI subpictures

On Thu, 2017-06-22 at 09:25 +, Namburu, Chandu-babu wrote: > From: Chandu Babu N > Subject: [PATCH] [st/va] Fix leak in VAAPI subpictures >   > sampler view allocated in vaAssociateSubpicture is not cleared > in vaiDeassociateSubpicture. >   > Reviewed-by: Christian König

Re: [Mesa-dev] [PATCH 1/8] mesa: check for allocation failures in _mesa_new_texture_object()

It looks like we could want patches 1 and 3-8 from this series into -stable (?) On Wed, 2017-06-21 at 11:04 +0200, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/mesa/main/texobj.c | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) >

[Mesa-dev] [Bug 101709] [llvmpipe] piglit gl-1.0-scissor-offscreen regression

https://bugs.freedesktop.org/show_bug.cgi?id=101709 Bug ID: 101709 Summary: [llvmpipe] piglit gl-1.0-scissor-offscreen regression Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW

[Mesa-dev] BCM7268 V3D3.3 ("vc5") driver release

I've just pushed a "vc5" branch to my Mesa tree (https://github.com/anholt/mesa/commits/vc5). This is the culmination of a couple of months of work on building a new driver for Broadcom's V3D 3.3. V3D 3.3 is a GLES3.1 part, though I'm nowhere near conformance yet. This driver is for BCM7268, a

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

On Thu, 2017-07-06 at 12:09 +1000, Dave Airlie wrote: > From: Dave Airlie > > Use family, but only set xnack+ for gfx9. > > Signed-off-by: Dave Airlie > --- > src/amd/common/ac_llvm_util.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > >

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/3] mesa/main: Move NULL pointer check.

On 6 July 2017 at 15:35, Andres Gomez wrote: > It looks like we could want this into -stable (?) > _mesa_update_draw_buffer_bounds is a no-op if the buffer pointer is NULL. There's no issue to fix, but making Coverity happy is not a bad idea either ;-) -Emil

Re: [Mesa-dev] [PATCH 09/10] egl: enhance KHR_gl_image extensions checks

On 6 July 2017 at 15:45, Eric Engestrom wrote: > On Friday, 2017-06-30 12:15:19 +0100, Emil Velikov wrote: >> From: Emil Velikov >> >> Drop the (duplicate) top-level check in dri2_create_image_khr() and add >> the respective checks in

[Mesa-dev] [PATCH] gallium/util: remove unused util_{begin, end}_pipestat_query() helpers

Signed-off-by: Samuel Pitoiset --- src/gallium/auxiliary/util/u_helpers.c | 50 -- src/gallium/auxiliary/util/u_helpers.h | 7 - 2 files changed, 57 deletions(-) diff --git a/src/gallium/auxiliary/util/u_helpers.c

Re: [Mesa-dev] [RFC] travis: lower SWR requirement to GCC 4.8, aka std=c++11

On Jul 6, 2017, at 5:39 AM, Emil Velikov > wrote: On 5 July 2017 at 19:28, Rowley, Timothy O > wrote: On Jul 4, 2017, at 12:01 PM, Emil Velikov

[Mesa-dev] [PATCH 2/3] swr/rast: don't use _mm256_fmsub_ps in AVX code

--- src/gallium/drivers/swr/rasterizer/common/simdlib_256_avx.inl | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/swr/rasterizer/common/simdlib_256_avx.inl b/src/gallium/drivers/swr/rasterizer/common/simdlib_256_avx.inl index aec79e3..16eb521 100644

[Mesa-dev] [PATCH 1/3] swr/rast: _mm*_undefined_* implementations for gcc<4.9

Define these in terms of setzero for ancient gcc versions which don't have the undefined intrinsics. Cc: mesa-sta...@lists.freedesktop.org --- src/gallium/drivers/swr/rasterizer/common/os.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/gallium/drivers/swr/rasterizer/common/os.h

[Mesa-dev] [PATCH 3/3] swr/rast: remove unused variables

--- src/gallium/drivers/swr/rasterizer/jitter/fetch_jit.cpp | 2 -- src/gallium/drivers/swr/rasterizer/jitter/streamout_jit.cpp | 2 -- 2 files changed, 4 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/jitter/fetch_jit.cpp

[Mesa-dev] [PATCH 3/3] mesa: fix glPrimitiveRestartNV crash inside a render list.

glPrimitiveRestartNV crashes when it is called during the compilation of a render list. There are two reasons: - ctx->Driver.CurrentSavePrimitive is not set to the current primitive - save_PrimitiveRestartNV() calls _save_Begin() which only sets an OpenGL error, instead of calling

Re: [Mesa-dev] [PATCH] scons: Check for xlocale.h before defining HAVE_XLOCALE_H.

On Wednesday, 2017-07-05 21:25:12 +, Vinson Lee wrote: > glibc 2.26 removed xlocale.h. > https://sourceware.org/glibc/wiki/Release/2.26#Removal_of_.27xlocale.h.27 > > Fix this build error with glibc 2.26. > > Compiling src/util/strtod.c ... > src/util/strtod.c:32:10: fatal error:

[Mesa-dev] [PATCH 2/2] gallium: Implement wglUseFontBitmaps.

wglUseFontBitmaps is currently a noop. This patch implements this function for Windows. Signed-off-by: Olivier Lauffenburger --- src/gallium/state_trackers/wgl/stw_wgl.c | 50 1 file changed, 44 insertions(+), 6 deletions(-) diff

[Mesa-dev] [PATCH] gallium: improve selection of pixel format

Current selection of pixel format does not enforce the request of stencil or depth buffer if the color depth is not the same as requested. For instance, GLUT requests a 32-bit color buffer with an 8-bit stencil buffer, but because color buffers are only 24-bit, no priority is given to creating a

Re: [Mesa-dev] [PATCH 09/10] egl: enhance KHR_gl_image extensions checks

On Friday, 2017-06-30 12:15:19 +0100, Emil Velikov wrote: > From: Emil Velikov > > Drop the (duplicate) top-level check in dri2_create_image_khr() and add > the respective checks in dri2_create_image_khr_{texture,renderbuffer} > > Signed-off-by: Emil Velikov

Re: [Mesa-dev] [PATCH 1/3] mesa/main: Move NULL pointer check.

It looks like we could want this into -stable (?) On Wed, 2017-06-14 at 19:33 +0300, Plamena Manolova wrote: > In blit_framebuffer we're already doing a NULL > pointer check for readFb and drawFb so it makes > sense to do it before we actually use the pointers. > > CID: 1412569 > Signed-off-by:

Re: [Mesa-dev] [PATCHv2] etnaviv: Add support for ETC2 texture compression

Hello, On Tue, Jun 27, 2017 at 5:47 PM, Christian Gmeiner wrote: > 2017-06-27 17:39 GMT+02:00 Wladimir J. van der Laan : >> On Tue, Jun 27, 2017 at 05:31:52PM +0200, Wladimir J. van der Laan wrote: >> >>> > sure if we should advertise the broken

Re: [Mesa-dev] [PATCH 01/10] egl: move eglCreateDRMImageMESA's malloc later

On Friday, 2017-06-30 12:15:11 +0100, Emil Velikov wrote: > From: Emil Velikov > > Don't bother allocating any memory until we're finished parsing and > sanitising all the attributes. > > As a nice side effect we now consistently set eglError when any of > the

Re: [Mesa-dev] [PATCH] glsl: gl_Max{Vertex, Fragment}UniformComponents exist in all desktop GL versions

Yes, I think so, thanks Andres. Iago On Thu, 2017-07-06 at 16:45 +0300, Andres Gomez wrote: > It looks like we could want this into -stable (?) > > On Fri, 2017-06-16 at 12:05 +0200, Iago Toral Quiroga wrote: > > The current implementation assumed that these were replaced in GLSL > > >= 4.10 >

Re: [Mesa-dev] [PATCH] glsl: gl_Max{Vertex, Fragment}UniformComponents exist in all desktop GL versions

It looks like we could want this into -stable (?) On Fri, 2017-06-16 at 12:05 +0200, Iago Toral Quiroga wrote: > The current implementation assumed that these were replaced in GLSL >= 4.10 > by gl_Max{Vertex,Fragment}UniformVectors, however this is not true: both > built-ins should be produced

Re: [Mesa-dev] [PATCH 6/7] i965: Don't use PREAD for glGetBufferSubData().

Quoting Kenneth Graunke (2017-07-05 21:56:53) > diff --git a/src/mesa/drivers/dri/i965/intel_buffer_objects.c > b/src/mesa/drivers/dri/i965/intel_buffer_objects.c > index a9ac29a6a81..2b0f7b9a698 100644 > --- a/src/mesa/drivers/dri/i965/intel_buffer_objects.c > +++

Re: [Mesa-dev] [PATCH 5/7] i965: Assert that we don't use CPU write maps to non-coherent buffers.

Quoting Kenneth Graunke (2017-07-05 21:56:52) > Using CPU maps of non-coherent buffers can get us in a lot of trouble, > and WC maps are a reasonable alternative anyway. Guard against shooting > ourselves in the foot by adding an assert, and comment. Reviewed-by: Chris Wilson

Re: [Mesa-dev] [PATCH 01/10] egl: move eglCreateDRMImageMESA's malloc later

On 30 June 2017 at 12:56, Daniel Stone wrote: > Hi Emil, > > On 30 June 2017 at 12:15, Emil Velikov wrote: >> Kristian can you shed some light? The extension seems quite sparse. >> >> Weston used the extension back in 2011. While the Glamor bit

Re: [Mesa-dev] [PATCH] gallium: auxiliary: Fix standalone Android build of u_cpu_detect

On 6 July 2017 at 02:18, Tomasz Figa wrote: > On Thu, Jul 6, 2017 at 2:49 AM, Emil Velikov wrote: >> Hi Tomasz, >> >> On 5 July 2017 at 15:58, Tomasz Figa wrote: >>> Commit 463b7d0332c5("gallium: Enable ARM NEON CPU detection.")

Re: [Mesa-dev] [RFC] travis: lower SWR requirement to GCC 4.8, aka std=c++11

On 5 July 2017 at 19:28, Rowley, Timothy O wrote: > >> On Jul 4, 2017, at 12:01 PM, Emil Velikov wrote: >> >> From: Emil Velikov >> >> With ealier commit we relaxed the requirement from C++14 to C++11. >> Update

Re: [Mesa-dev] [PATCH] etnaviv: don't dereference etna_resource pointer if allocation fails

2017-07-06 11:35 GMT+02:00 Aleksander Morgado : > The check for the pointer being non-NULL was being done too late. > > Signed-off-by: Aleksander Morgado Reviewed-by: Christian Gmeiner > --- >

Re: [Mesa-dev] [PATCH] scons: Check for xlocale.h before defining HAVE_XLOCALE_H.

On 5 July 2017 at 22:25, Vinson Lee wrote: > glibc 2.26 removed xlocale.h. > https://sourceware.org/glibc/wiki/Release/2.26#Removal_of_.27xlocale.h.27 > > Fix this build error with glibc 2.26. > > Compiling src/util/strtod.c ... > src/util/strtod.c:32:10: fatal error:

[Mesa-dev] [PATCH] etnaviv: don't dereference etna_resource pointer if allocation fails

The check for the pointer being non-NULL was being done too late. Signed-off-by: Aleksander Morgado --- src/gallium/drivers/etnaviv/etnaviv_resource.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git

[Mesa-dev] [Bug 101703] No stencil buffer allocated when requested by GLUT

https://bugs.freedesktop.org/show_bug.cgi?id=101703 Eric Engestrom changed: What|Removed |Added Attachment #132472|0 |1 is

[Mesa-dev] [PATCH] glsl: look up for transform feedback varyings after linking

Check if shaders have transform feedback varyings also after the post-link step. This fixes: KHR-GL45.enhanced_layouts.xfb_vertex_streams piglit/spec/arb_enhanced_layouts/gs-stream-location-aliasing --- src/compiler/glsl/glsl_to_nir.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git

[Mesa-dev] [PATCH 2/2] i965: perf: use new subslices numbers from device info

Signed-off-by: Lionel Landwerlin --- src/mesa/drivers/dri/i965/brw_performance_query.c | 49 --- 1 file changed, 17 insertions(+), 32 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_performance_query.c

[Mesa-dev] [PATCH 1/2] intel: add number of subslices to device info

We could have used a single integer to store that value, but Cannonlake has different number of subslices per slice depending on the GT. Signed-off-by: Lionel Landwerlin --- src/intel/common/gen_device_info.c | 54 --

[Mesa-dev] [Bug 101703] No stencil buffer allocated when requested by GLUT

https://bugs.freedesktop.org/show_bug.cgi?id=101703 Bug ID: 101703 Summary: No stencil buffer allocated when requested by GLUT Product: Mesa Version: 17.0 Hardware: All OS: All Status: NEW Severity:

Re: [Mesa-dev] [PATCH 1/4] ac/radv: change api to create target machine

Series is Reviewed-by: Bas Nieuwenhuizen On Thu, Jul 6, 2017 at 4:09 AM, Dave Airlie wrote: > From: Dave Airlie > > This just modifies the API to make it easier to add other flags > to target machine creation. > > Signed-off-by:

[Mesa-dev] [PATCH] radv: don't overallocate 5 bpe formats (d32/s8)

From: Dave Airlie This was getting passed into the ac_surface code, and used to calculate bpp for the surface, however in this case the bpe is really only 4, the surface code will allocate stencil separately. This reduces the side of d32s8 allocates to something closer to

<    1   2