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: Dave Airlie
> ---
> src/amd/common/ac_llvm_util.c | 4 ++--
>
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: normal
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 --
src/intel/common/gen_device_info.h | 5 ++
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
b/src/mesa/drivers/dri/i965/brw_performance_query.c
index 81389db
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 a/src/c
https://bugs.freedesktop.org/show_bug.cgi?id=101703
Eric Engestrom changed:
What|Removed |Added
Attachment #132472|0 |1
is obsolete|
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 a/src/gallium/drivers/etnaviv/etnaviv_resource.c
b/src/gallium/driv
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: xlocale.h: No such file or
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
> ---
> src/gallium/drivers/etnaviv/etnaviv_resource.c | 10 +++---
> 1 file changed, 7 insertions(+), 3
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 the build script so that it
>>
>> Cc: Tim Rowley > Fixes: 0b80b025021 ("swr: relax c
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.")
>>> introduced CPU feature detection based Android cpufeatures libr
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 were
>> dropped somewhat recently in May 2017.
>>
>>
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
-Chris
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
> +++ b/src/mesa/drivers/dri/i965/inte
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 fro
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
> >
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 attrib/values are not correct.
Not
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 formats (on gpus with HALTI0).
>>
>> FWIW, I did ch
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: P
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
> ---
> src/egl/drivers/dri2/egl_dr
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 st
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 --git a/src/gallium/state_tracke
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: xlocale.h:
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 vbo_save_Notif
---
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
b/src/gallium/drivers/swr/rasterizer/jitter/fetch_jit.c
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
b
---
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
--
On Jul 6, 2017, at 5:39 AM, Emil Velikov
mailto:emil.l.veli...@gmail.com>> wrote:
On 5 July 2017 at 19:28, Rowley, Timothy O
mailto:timothy.o.row...@intel.com>> wrote:
On Jul 4, 2017, at 12:01 PM, Emil Velikov
mailto:emil.l.veli...@gmail.com>> wrote:
From: Emil Velikov
mailto:emil.veli...@c
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
b/src/gallium/auxiliary/util/u_helpers.c
ind
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 dri2_create_image_khr_{texture,renderbuffer}
>>
>> Signed-off-
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
__
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(-)
>
> diff --git a/src/amd/common/ac_llvm_util.
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 s
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
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(-)
>
> diff --git a/src/mesa/main
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
> ---
> src/gallium/
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
>> ---
>> src/amd/common/ac_llvm_util.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1
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 tiling for the test d
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 allocated in vaAssoci
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 since v4:
> > Gustaw Smolar
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
>> > Subject: [PAT
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 +++---
src/gallium/drivers
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 since v5:
>> >Actually include changes from v5 in patc
> 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
___
mesa-dev mailing li
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
> ---
> src/gallium/drivers/swr/rasterizer/common/os.h | 6 ++
>
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 changed, 4 deletions(-)
>
> diff --git a/src/gallium/drivers/
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 a/src/gallium/drivers/swr/rasterizer/common/simdlib_256_avx.inl
> b/s
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 over-aligned new
> feature.
>
> Cc: mesa-sta...@lists.freedesk
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.
---
src/compiler/nir/nir_lower_io_to_tempor
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 for local
variables an
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 a variable. Since
nir_
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.
---
src/amd/common/ac_nir
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.
---
src/amd/common/ac_nir_to_llvm.c |
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.
---
src/amd/vulkan/radv_p
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 we didn't notice this,
>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 (exists). The test only checks that the former generates
>an error but t
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
---
src/amd/vulkan/radv_image.c | 2 +-
1 file changed, 1
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 be set as part
> of the subtarget chosen
Well I expect Ma
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| 2 +-
src/SConscript
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
+++ b/src/intel/vulkan/anv_image.c
@@
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: https://bugzilla.freedesktop.org
> 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 xnack+ for gfx9.
>>>
>>
>> The driver shouldn’t be explicitly setting this. Thi
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 +
src/gallium/state_trackers/dri/drisw.c | 1 +
2 files c
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:
From: Dave Airlie
Use family, but only set xnack+ for gfx9.
>>
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 gfx9.
>>>
>>> Signed-off-by: Dave Airlie
>>> ---
>>> src/amd/common
On Thu, Jul 6, 2017 at 9:48 PM, Connor Abbott wrote:
> 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
I think the old code should handle indirect derefs just fine? See the
indir_index
Reviewed-by: Bas Nieuwenhuizen
On Thu, Jul 6, 2017 at 9:48 PM, Connor Abbott wrote:
> 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, meanin
LGTM.
Reviewed-by: Brian Paul
On 07/06/2017 02:08 PM, Eric Engestrom wrote:
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_sha
Patches 3-4 look technically correct to me, so for just using it for shared vars
Reviewed-by: Bas Nieuwenhuizen
On Thu, Jul 6, 2017 at 9:48 PM, Connor Abbott wrote:
> From: Connor Abbott
>
> Similar to before, do the direct NIR->LLVM translation instead of
> lowering to an array then back to a
On Thu, Jul 6, 2017 at 4:56 PM, Marek Olšák wrote:
> 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 gfx9.
On Thu, Jul 6, 2017 at 2:01 PM, Bas Nieuwenhuizen
wrote:
> On Thu, Jul 6, 2017 at 9:48 PM, Connor Abbott
> wrote:
>> 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
>
> I think the
Reviewed-by: Bas Nieuwenhuizen
On Thu, Jul 6, 2017 at 9:50 PM, Connor Abbott wrote:
> 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(
Despite being a member of the etna_screen struct, 'refcnt' is used by
the winsys-specific logic to track the reference count of the object
managed in a hash table. When the count reaches zero, the pipe screen
is removed from the table and destroyed.
Fix the logic by initializing the refcnt to 1 wh
On Thu, Jul 6, 2017 at 12:48 PM, Connor Abbott
wrote:
> 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 usi
On Thu, Jul 6, 2017 at 8:24 PM, Andres Gomez wrote:
> Marek, would we want this series in -stable or we shouldn't bother ?
Don't bother. Patch 1 isn't that important. Other patches are for
features not enabled in 17.1.
Marek
___
mesa-dev mailing list
m
On 2017-07-04 12:39 PM, Andres Rodriguez wrote:
On 2017-07-04 09:30 AM, Christian König wrote:
Am 04.07.2017 um 15:13 schrieb Nicolai Hähnle:
On 01.07.2017 01:03, Andres Rodriguez wrote:
From: Dave Airlie
Signed-off-by: Andres Rodriguez
---
src/gallium/drivers/radeon/r600_pipe_common.
From: Dave Airlie
Since radv uses compute rings and we can't know when we are setting
up the shaders what ring they are to be used on, we should just use
the default xnack setting. This may be suboptimal in some places,
but if we hit a problem, we likely should try and address this
between llvm a
On 6 July 2017 at 22:20, Connor Abbott wrote:
> On Thu, Jul 6, 2017 at 12:48 PM, Connor Abbott
> wrote:
>> 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
Reviewed-by: Bas Nieuwenhuizen
On Fri, Jul 7, 2017 at 12:10 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> Since radv uses compute rings and we can't know when we are setting
> up the shaders what ring they are to be used on, we should just use
> the default xnack setting. This may be suboptima
On Thu, Jul 6, 2017 at 9:50 PM, Connor Abbott wrote:
> 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
Reviewed-by: Jason Ekstrand
On Thu, Jul 6, 2017 at 12:48 PM, Connor Abbott
wrote:
> 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
Thanks! Pushed and cc'd it to stable.
Not pushing the first patch as I assume that is superseded by Connors patches.
On Fri, Jun 30, 2017 at 12:15 PM, Alex Smith
wrote:
> The NIR parameters are ordered "compare, data", matching GLSL, but both
> the image and buffer LLVM intrinsics take them the
On Thu, Jul 6, 2017 at 11:18 PM, Aleksander Morgado
wrote:
> Despite being a member of the etna_screen struct, 'refcnt' is used by
> the winsys-specific logic to track the reference count of the object
> managed in a hash table. When the count reaches zero, the pipe screen
> is removed from the ta
https://bugs.freedesktop.org/show_bug.cgi?id=101712
Bug ID: 101712
Summary: CPU lockup after ring 0 stalled
Product: Mesa
Version: unspecified
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Severity:
https://bugs.freedesktop.org/show_bug.cgi?id=101712
guiscara...@gmail.com changed:
What|Removed |Added
Priority|medium |high
--- Comment #1 from guiscar
https://bugs.freedesktop.org/show_bug.cgi?id=101712
--- Comment #2 from guiscara...@gmail.com ---
Created attachment 132488
--> https://bugs.freedesktop.org/attachment.cgi?id=132488&action=edit
DSMEG (more detailed log)
Here it is a more detailed log
--
You are receiving this mail because:
Yo
Thanks for the patches, Olivier!
I'm a bit short on time, but I'll test/commit them ASAP. I'll take a
closer look at the VBO issue too.
-Brian
On 07/06/2017 09:45 AM, Olivier Lauffenburger wrote:
glPrimitiveRestartNV crashes when it is called during the compilation
of a render list.
There
From: Ben Widawsky
Starting with GEN9, Memory Object Control State (MOCS) becomes an index
into a table as opposed to the direct programming within the command.
The table has 62 usable entries (ie 6 bits can represent all settings),
and each buffer type may use one of these 62 entries to describe
https://bugs.freedesktop.org/show_bug.cgi?id=101703
Brian Paul changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
Signed-off-by: Ben Widawsky
---
src/intel/drm/i915_drm.h | 8
1 file changed, 8 insertions(+)
diff --git a/src/intel/drm/i915_drm.h b/src/intel/drm/i915_drm.h
index c26bf7c125..69e38ce89f 100644
--- a/src/intel/drm/i915_drm.h
+++ b/src/intel/drm/i915_drm.h
@@ -431,6 +431,14 @@ typedef s
We don't yet have optimal MOCS settings, but we have enough to know how
to at least determine when we might have non-optimal settings within our
driver.
Signed-off-by: Ben Widawsky
---
src/intel/vulkan/anv_device.c | 12
src/intel/vulkan/anv_private.h| 2 ++
Copying the kernel commit message:
Starting with GEN9, Memory Object Control State (MOCS) becomes an index
into a table as opposed to the direct programming within the command.
The table has 62 usable entries (ie 6 bits can represent all settings),
and each buffer type may use one of these 62 entr
These are intrinsics rather than opcodes, because they operate across
channels.
---
src/compiler/glsl/glsl_to_nir.cpp | 22 ++
src/compiler/nir/nir_intrinsics.h | 5 +
2 files changed, 27 insertions(+)
diff --git a/src/compiler/glsl/glsl_to_nir.cpp
b/src/compiler/glsl/gl
Specifically, constant fold intrinsics from ARB_shader_group_vote, but I
suspect it'll be useful for other things in the future.
---
src/compiler/Makefile.sources | 1 +
src/compiler/nir/nir.h| 2 +
src/compiler/nir/nir_opt_intrinsics.c | 102 +++
I don't expect anyone is going to care about using this in vec4 programs
(vertex/tessellation/geometry on Gen6/7), no one has come up with a good
way to implement it much less test it.
---
src/intel/compiler/brw_compiler.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/intel/compiler/b
... trivially (as allowed by the spec!) by reusing the existing
nir_opt_intrinsics code.
---
src/compiler/nir/nir.h| 4
src/compiler/nir/nir_opt_intrinsics.c | 6 +++---
2 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/ni
The implementations of the ARB_shader_group_vote intrinsics will
explicitly write the flag as the destination register.
---
src/intel/compiler/brw_fs.cpp | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp
---
src/intel/compiler/brw_fs_nir.cpp | 36
1 file changed, 36 insertions(+)
diff --git a/src/intel/compiler/brw_fs_nir.cpp
b/src/intel/compiler/brw_fs_nir.cpp
index a9dce42c38..264398f38e 100644
--- a/src/intel/compiler/brw_fs_nir.cpp
+++ b/src/intel/compile
This function will be used to implement read_invocation (by specifying a
specific channel) and read_first_invocation (by not specifying a
channel).
---
src/intel/compiler/brw_fs_builder.h | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/src/intel/compiler/brw_fs_builder
i965 will want these to be scalar operations.
---
src/compiler/Makefile.sources | 1 +
src/compiler/nir/nir.h | 2 +-
.../nir/nir_lower_read_invocation_to_scalar.c | 112 +
3 files changed, 114 insertions(+), 1 deletion(
1 - 100 of 168 matches
Mail list logo
