Potentially more efficient as is may avoid the struct being initialised
twice.
---
src/compiler/glsl/ir.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/compiler/glsl/ir.cpp b/src/compiler/glsl/ir.cpp
index 123de99..e55635b 100644
--- a/src/compiler/glsl/ir.cpp
+++ b/s
This will be used for things such as adding driver specific environment
variables to the key. Allowing us to set environment vars that change
the shader and not have the driver ignore them if it finds existing
shaders in the cache.
---
src/compiler/glsl/tests/cache_test.c | 20 ++-
I wasn't sure if I should filter the flags so that we only use
flags that actually change the shader output. To avoid manual
updates we just pass in everything for now.
---
src/gallium/drivers/radeon/r600_pipe_common.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/galli
If info->index_size is zero, info->index will point to uninitialized
memory.
Fatal signal 11 (SIGSEGV), code 2, fault addr 0xab5d07a3 in tid 20456
(surfaceflinger)
Signed-off-by: Tomeu Vizoso
Cc: etna...@lists.freedesktop.org
Cc: Marek Olšák
Fixes: 330d0607ed60 ("gallium: remove pipe_index_buf
On Fri, May 19, 2017 at 05:18:12PM -0700, Jason Ekstrand wrote:
> On Fri, May 19, 2017 at 4:51 PM, Nanley Chery wrote:
>
> > On Thu, May 18, 2017 at 02:00:51PM -0700, Jason Ekstrand wrote:
> > > ---
> > > src/intel/vulkan/anv_image.c | 12 ++--
> > > src/intel/vulkan/genX_cmd_buffe
On Fri, May 19, 2017 at 05:32:07PM -0700, Ian Romanick wrote:
> On 05/19/2017 05:07 PM, Ilia Mirkin wrote:
> > On Fri, May 19, 2017 at 7:29 PM, Ian Romanick wrote:
> >> I missed that glsl_parser_extras.cpp has its own implementation of the
> >> has_XXX_foo() functions that take the API and version
On 05/19/2017 05:07 PM, Ilia Mirkin wrote:
> On Fri, May 19, 2017 at 7:29 PM, Ian Romanick wrote:
>> I missed that glsl_parser_extras.cpp has its own implementation of the
>> has_XXX_foo() functions that take the API and version as explicit
>> parameters. Your patch predates that change. There's
https://bugs.freedesktop.org/show_bug.cgi?id=101110
--- Comment #4 from Emil Velikov ---
Correction - needs the following brown paper bag commit
48cd1919ff1 ("configure.ac: s/xcb-fixes/xcb-xfixes/")
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact
https://bugs.freedesktop.org/show_bug.cgi?id=101110
--- Comment #3 from Dieter Nützel ---
Hello Emil,
shouldn't this look like this:
diff --git a/configure.ac b/configure.ac
index b788137d51..06883a9667 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1723,7 +1723,7 @@ fi
if test x"$enable_dri
On Fri, May 19, 2017 at 4:51 PM, Nanley Chery wrote:
> On Thu, May 18, 2017 at 02:00:51PM -0700, Jason Ekstrand wrote:
> > ---
> > src/intel/vulkan/anv_image.c | 12 ++--
> > src/intel/vulkan/genX_cmd_buffer.c | 9 +
> > 2 files changed, 11 insertions(+), 10 deletions(-)
>
On Fri, May 19, 2017 at 12:57 PM, Emil Velikov wrote:
> On 18 May 2017 at 23:01, Rob Herring wrote:
>> On Thu, May 18, 2017 at 5:25 AM, Emil Velikov
>> wrote:
>>> On 18 May 2017 at 05:10, Chih-Wei Huang wrote:
2017-05-18 12:01 GMT+08:00 Xu, Randy :
>
>> -Original Message-
On Fri, May 19, 2017 at 7:29 PM, Ian Romanick wrote:
> I missed that glsl_parser_extras.cpp has its own implementation of the
> has_XXX_foo() functions that take the API and version as explicit
> parameters. Your patch predates that change. There's room for some
> modest savings there too.
The
On Thu, May 18, 2017 at 02:00:51PM -0700, Jason Ekstrand wrote:
> ---
> src/intel/vulkan/anv_image.c | 12 ++--
> src/intel/vulkan/genX_cmd_buffer.c | 9 +
> 2 files changed, 11 insertions(+), 10 deletions(-)
>
> diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/
On Thu, May 18, 2017 at 02:00:50PM -0700, Jason Ekstrand wrote:
> This causes dEQP-VK.api.copy_and_blit.resolve_image.partial.* to start
> failing due to test bugs. See CL 1031 for a test fix.
> ---
> src/intel/vulkan/anv_blorp.c | 40
> ++
> src/intel/v
Cc: Anuj Phogat
Signed-off-by: Nanley Chery
---
docs/features.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/docs/features.txt b/docs/features.txt
index e18bf54a48..05d776be08 100644
--- a/docs/features.txt
+++ b/docs/features.txt
@@ -306,8 +306,8 @@ Khronos, ARB, an
On 05/19/2017 10:28 AM, Nanley Chery wrote:
> On Fri, May 19, 2017 at 06:38:03AM -0700, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> The previous code handled everything with the general case. I noticed
>> that every time I converted an open-coded check to use a
>> _mesa_has_EXT_foo() function
On 19 May 2017 at 20:50, Rowley, Timothy O wrote:
> Thanks for doing this; I would have been hunting for the dist-hook: magic
> for a while.
>
> Tested “make dist” on llvm-3.9.0 (works) and llvm-4.0/llvm-svn (fails,
> expected desired behavior).
>
> Built result of llvm-3.9.0 “make dist” with llvm
https://bugs.freedesktop.org/show_bug.cgi?id=101110
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Fri, May 19, 2017 at 2:53 PM, Gustaw Smolarczyk
wrote:
> 2017-05-18 23:01 GMT+02:00 Jason Ekstrand :
> > ---
> > src/intel/vulkan/anv_device.c | 42 ++
> ++--
> > 1 file changed, 36 insertions(+), 6 deletions(-)
> >
> > diff --git a/src/intel/vulkan/anv_dev
So, I think I'm going to end up doing a fairly significant rework of
resolves over the course of the next couple of weeks. Carry on with the
branch as is and I'll figure out how to rebase it on top of whatever
changes I do later. But it'll probably be significant. Just a heads up.
--Jason
On F
https://bugs.freedesktop.org/show_bug.cgi?id=101110
--- Comment #1 from Emil Velikov ---
Seems like I've missed some corner case - I'm looking into it.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.__
On 19 May 2017 at 23:41, Emil Velikov wrote:
> On 19 May 2017 at 23:21, Eric Anholt wrote:
>> Emil Velikov writes:
>>
>>> On 17 May 2017 at 20:13, Emil Velikov wrote:
On 17 May 2017 at 18:53, Eric Anholt wrote:
> Emil Velikov writes:
>
>> Hi Eric,
>>
>> On 11 May 2017
On 19 May 2017 at 23:21, Eric Anholt wrote:
> Emil Velikov writes:
>
>> On 17 May 2017 at 20:13, Emil Velikov wrote:
>>> On 17 May 2017 at 18:53, Eric Anholt wrote:
Emil Velikov writes:
> Hi Eric,
>
> On 11 May 2017 at 00:06, Eric Anholt wrote:
>> This follows the mo
https://bugs.freedesktop.org/show_bug.cgi?id=101110
Bug ID: 101110
Summary: Build failure in GNOME Continuous
Product: Mesa
Version: git
Hardware: Other
OS: All
Status: NEW
Severity: normal
Prior
Emil Velikov writes:
> On 17 May 2017 at 20:13, Emil Velikov wrote:
>> On 17 May 2017 at 18:53, Eric Anholt wrote:
>>> Emil Velikov writes:
>>>
Hi Eric,
On 11 May 2017 at 00:06, Eric Anholt wrote:
> This follows the model of imx (display) and etnaviv (render): pl111 is a
>>
Before the swapchain event queue is destroyed, all proxy objects that reference
it must be dropped. Otherwise we risk a use-after-free if a frame callback event
or buffer release events are received afterwards.
This happens when an application destroys and recreates a swapchain in FIFO
mode between
2017-05-18 23:01 GMT+02:00 Jason Ekstrand :
> ---
> src/intel/vulkan/anv_device.c | 42 --
> 1 file changed, 36 insertions(+), 6 deletions(-)
>
> diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
> index 6ea8dfe..8eed7f3 100644
> ---
Ilia Mirkin writes:
> I kinda see it both ways - yeah, the functions are the same and it's
> all shared, so your patch makes sense. OTOH, all of these functions
> (which do anything) have a nv04/nv10/nv20 prefix, which makes it
> easier to separate stuff out by generation if need be. So I think i
It's a bit annoying, but I think this would all make more sense if we moved
23 and 24 to before 19. Then we could drop no_aux from 21. Then again,
I'm sure Ben has some good reason why that's not practical and I'm fine
with leaving the order as-is. It's hard to review but there's a lot of
detang
On Fri, May 19, 2017 at 10:41:22AM -0700, Anuj Phogat wrote:
> On Thu, May 18, 2017 at 3:53 PM, Nanley Chery wrote:
> > This platform passes the following GLES3 tests:
> > ES3-CTS.functional.texture.compressed.astc.endpoint_value_hdr_cem_*
> >
> We've hdr tests in piglit as well. With those passin
On Fri, May 19, 2017 at 2:38 AM, Daniel Stone wrote:
> From: Ben Widawsky
>
> Make the code only disable CCS when it has to, unlike before where it
> disabled CCS and enabled it when it could. This is much more inline with
> how it should work in a few patches, where we have fewer restrictions a
Thanks for doing this; I would have been hunting for the dist-hook: magic for a
while.
Tested “make dist” on llvm-3.9.0 (works) and llvm-4.0/llvm-svn (fails, expected
desired behavior).
Built result of llvm-3.9.0 “make dist” with llvm-4.0 and llvm-svn and it
compiles/works.
Reviewed-by: Tim R
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote:
> Intel hardware requires that all planes of an image come from the same
> buffer, which is currently implemented by testing that all FDs are
> numerically the same.
>
> However, when going through a winsys (e.g.) or anything which transits
> F
Just FYI: I'm not checking that you implement the EGL API correctly. I'm
mostly checking for whether or not it's using DRI correctly.
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote:
> From: Varad Gautam
>
> query and return supported dmabuf format modifiers for
> EGL_EXT_image_dma_buf_imp
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote:
> From: Varad Gautam
>
> allow egl clients to query the dmabuf formats supported on this platform.
>
> v2: return EGLBoolean.
> v3: Check DRIimageExtension version before querying (Daniel Stone)
>
> Signed-off-by: Louis-Francis Ratté-Boulianne
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote:
> From: Varad Gautam
>
> these allow querying the driver for supported dmabuf formats and
> modifiers.
>
> v2: move to __DRIimageExtension version 16.
> v3: return GLBoolean for error reporting, document params better.
>
> Signed-off-by: Varad
Jfyi: feel free to add Fixes tag as you see fit. We parse though and
automatically nominate patches for -stable :-)
On 19 May 2017 at 13:04, Rob Herring wrote:
> On Fri, May 12, 2017 at 11:55 AM, Rob Herring wrote:
>> There's still an error after my recent clean-up if LLVM is not patched to
>> en
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote:
> From: Varad Gautam
>
> Allow creating EGLImages with dmabuf format modifiers when target is
> EGL_LINUX_DMA_BUF_EXT for EGL_EXT_image_dma_buf_import_modifiers.
>
> v2:
> - clear modifier assembling and error label name (Eric Engestrom)
> v3
On Fri, May 19, 2017 at 2:37 AM, Daniel Stone wrote:
> From: Pekka Paalanen
>
> add createImageFromDmaBufs2 function which accepts per-plane dmabuf
> format modifiers.
>
> Signed-off-by: Pekka Paalanen
> Signed-off-by: Varad Gautam
> Reviewed-by: Daniel Stone
> Signed-off-by: Daniel Stone
>
On 18 May 2017 at 23:01, Rob Herring wrote:
> On Thu, May 18, 2017 at 5:25 AM, Emil Velikov
> wrote:
>> On 18 May 2017 at 05:10, Chih-Wei Huang wrote:
>>> 2017-05-18 12:01 GMT+08:00 Xu, Randy :
> -Original Message-
> From: Palli, Tapani
> >
> > It's just applied. Is
On Thu, May 18, 2017 at 2:01 PM, Jason Ekstrand
wrote:
> ---
> src/intel/vulkan/anv_device.c | 42 ++
> ++--
> 1 file changed, 36 insertions(+), 6 deletions(-)
>
> diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
> index 6ea8dfe..8eed
From: Emil Velikov
The compiler can discard the shared ones from the link chain, since
there is no user (the static libraries) before it on the command line.
Cc: mesa-sta...@lists.freedesktop.org
Reported-by: Laurent Carlier
Signed-off-by: Emil Velikov
---
src/intel/Makefile.vulkan.am | 31 ++
From: Emil Velikov
Analogous to previous commit - the compiler can discard xcb + wayland
libs, since there is no user (the static libraries) before it on the
command line.
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Emil Velikov
---
src/amd/vulkan/Makefile.am | 35
On Thu, May 18, 2017 at 3:53 PM, Nanley Chery wrote:
> This platform passes the following GLES3 tests:
> ES3-CTS.functional.texture.compressed.astc.endpoint_value_hdr_cem_*
>
We've hdr tests in piglit as well. With those passing, both patches are:
Reviewed-by: Anuj Phogat
> Signed-off-by: Nanley
On 19 May 2017 at 17:46, Rowley, Timothy O wrote:
>
> > On May 19, 2017, at 10:26 AM, Emil Velikov wrote:
> > The original idea by Tim sounds OK imho and I'm actually giving it a try.
> Are you referring to using a llvm-3.9 generated version?
Yes. Unless I've completely misinterpret and you had
From: Emil Velikov
As gen_builder.hpp file is generated, it contains information that is
specific to the LLVM version it originates from.
As suggested by Tim, the file seems to be forwards compatible. So in
order to produce ship a file which will work everywhere we should be
using earlies suppor
Reviewed-by: Jason Ekstrand
On Fri, May 19, 2017 at 4:25 AM, Tapani Pälli
wrote:
> Just like is done on desktop and what is expected by the build-id code.
>
> Signed-off-by: Tapani Pälli
> ---
>
> Jason, I commented before that this did not work but it seems I had other
> changes when testing
On Fri, May 19, 2017 at 1:28 PM, Nanley Chery wrote:
> The only thing I think may need
> addressing is endianness.
FYI, I'm happy to provide BE support, both reviewing code and testing.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://li
On Fri, May 19, 2017 at 06:38:03AM -0700, Ian Romanick wrote:
> From: Ian Romanick
>
> The previous code handled everything with the general case. I noticed
> that every time I converted an open-coded check to use a
> _mesa_has_EXT_foo() function, the text size of the driver increased.
>
> Almo
On 05/19/2017 07:05 PM, Ilia Mirkin wrote:
Great work, Samuel! Is this available in a branch somewhere?
Thanks Ilia!
Here's the branch:
https://cgit.freedesktop.org/~hakzsam/mesa/log/?h=arb_bindless_texture
On Fri, May 19, 2017 at 12:52 PM, Samuel Pitoiset
wrote:
Hi,
This series implem
Great work, Samuel! Is this available in a branch somewhere?
On Fri, May 19, 2017 at 12:52 PM, Samuel Pitoiset
wrote:
> Hi,
>
> This series implements ARB_bindless_texture for RadeonSI.
>
> Reminder: the GLSL compiler part is already upstream.
>
> This series has been mainly tested with Feral gam
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_atom_constbuf.c | 6 ++
src/mesa/state_tracker/st_texture.c | 94 +++
src/mesa/state_tracker/st_texture.h | 8 +++
3 files changed, 108 insertions(+)
diff --git a/src/mesa/state_tracker/st_atom
This will also be used for looping over bindless samplers bound
to texture units.
Signed-off-by: Samuel Pitoiset
---
src/mesa/main/texstate.c | 36 +++-
1 file changed, 23 insertions(+), 13 deletions(-)
diff --git a/src/mesa/main/texstate.c b/src/mesa/main/texsta
To share some common code between bound and bindless textures.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_descriptors.c | 95 +++
1 file changed, 52 insertions(+), 43 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c
b/src/g
When current bound shaders don't use any bindless textures or
images, we shouldn't try to decompress the resident resources.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_blit.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/src/gallium/driv
Signed-off-by: Samuel Pitoiset
---
src/gallium/include/pipe/p_context.h | 16
1 file changed, 16 insertions(+)
diff --git a/src/gallium/include/pipe/p_context.h
b/src/gallium/include/pipe/p_context.h
index 4b75386a65..83eb1c9b08 100644
--- a/src/gallium/include/pipe/p_context.h
When texture buffers are invalidated the addr in the resident
descriptor has to be updated but we can't create a new descriptor
because the resident handle has to be the same.
Instead, use the WRITE_DATA packet which allows to update memory
directly but graphics/compute have to be idle in case the
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_atom_texture.c | 15 +--
src/mesa/state_tracker/st_texture.h | 5 +
2 files changed, 14 insertions(+), 6 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_texture.c
b/src/mesa/state_tracker/st_atom_texture
Signed-off-by: Samuel Pitoiset
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 37 ++
src/gallium/auxiliary/tgsi/tgsi_scan.h | 2 ++
2 files changed, 39 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/tgsi_scan.c
index
This has only been tested on RX480.
Signed-off-by: Samuel Pitoiset
---
docs/features.txt | 2 +-
docs/relnotes/17.2.0.html | 1 +
src/gallium/drivers/radeonsi/si_pipe.c | 4 +++-
3 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/docs/features.txt b/
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index ee1afdcd08..c69eefe013 100644
--- a/src/mesa/state_tracker/st_glsl
Similar to the existing decompression code path except that it
loops over the list of resident textures/images.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_blit.c| 67 +--
src/gallium/drivers/radeonsi/si_descriptors.c | 44 ++
Resident buffers have to be added to every new command stream.
Though, this could be slightly improved when current shaders
don't use any bindless textures/images but usually applications
tend to use bindless for almost every draw call, and the winsys
thread might help when buffers are added early.
This won't help much except for applications that use a ton
of resident handles. Though, this will reduce the winsys
overhead a little bit.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_descriptors.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/driver
When a buffer becomes resident, check if it has been invalidated,
if so update the descriptor and the dirty flag.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_descriptors.c | 34 +++
1 file changed, 34 insertions(+)
diff --git a/src/gallium/drivers/
This adds some new helper functions to know if the current draw
call (or dispatch compute) is using bindless samplers/images,
based on TGSI analysis.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_compute.c | 2 ++
src/gallium/drivers/radeonsi/si_compute.h | 14 +
When a texture is referenced by one or more texture handles,
it might be resident and we shouldn't release the sampler views.
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_sampler_view.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/state_tracker/st_sampler_vi
To share some common code between bound and bindless images.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_descriptors.c | 78 ---
1 file changed, 46 insertions(+), 32 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c
b/src/gal
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 31 ++-
1 file changed, 25 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c
b/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c
index c9397c2fb8
The ARB_bindless_texture spec say:
"If ARB_seamless_cubemap (or OpenGL 4.0, which includes it) is
supported, the per-context seamless cubemap enable is ignored
and treated as disabled when using texture handles."
"If AMD_seamless_cubemap_per_texture is supported, the seamless
cu
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c
b/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c
index bd8ecb70f8..c9397
For each texture/image handles, we need to allocate a new
buffer for the resident descriptor. But when the number of
buffers added to the current CS becomes high, the overhead
in the winsys (and in the kernel) is important.
To reduce this bottleneck, the idea is to suballocate the
resident descrip
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 264b43c10b..f30544eb4c 100644
--- a/src/mesa/st
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_texture.c | 42 +
1 file changed, 42 insertions(+)
diff --git a/src/mesa/state_tracker/st_texture.c
b/src/mesa/state_tracker/st_texture.c
index 2e9856dcdf..65f86f2b4f 100644
--- a/src/mesa/state_tra
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_atom_image.c | 33 ++---
src/mesa/state_tracker/st_texture.h| 5 +
2 files changed, 27 insertions(+), 11 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_image.c
b/src/mesa/state_tracker/st
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_atom_sampler.c | 14 --
src/mesa/state_tracker/st_texture.h | 5 +
2 files changed, 13 insertions(+), 6 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_sampler.c
b/src/mesa/state_tracker/st_atom_sampler.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/trace/tr_context.c | 114 +
1 file changed, 114 insertions(+)
diff --git a/src/gallium/drivers/trace/tr_context.c
b/src/gallium/drivers/trace/tr_context.c
index c5563a4844..6639f783a5 100644
--- a/src/gallium
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_cb_texture.c | 84 ++
1 file changed, 84 insertions(+)
diff --git a/src/mesa/state_tracker/st_cb_texture.c
b/src/mesa/state_tracker/st_cb_texture.c
index 99c59f77a3..fabf78e7a7 100644
--- a/src/mesa/sta
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
index 2fa7ba7797..80695580cb 100644
--- a/src/mesa/state_tracker/st_extensions.c
+++ b/src
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_context.c | 2 +
src/mesa/state_tracker/st_context.h | 11 ++
src/mesa/state_tracker/st_texture.c | 77 +
src/mesa/state_tracker/st_texture.h | 5 +++
4 files changed, 95 insertions(+)
diff --g
Analogous to bound textures/images. We should also update the
resident descriptors and disable COMPRESSION_EN for avoiding
useless DCC fetches, but I postpone this optimization for a
separate series.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_blit.c| 52 ++
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_descriptors.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c
b/src/gallium/drivers/radeonsi/si_descriptors.c
index 44a4b16712..810dd8e89a 100644
-
When a stencil buffer is part of the framebuffer state, it is
decompressed but because it's bindles, all draw calls set
stencil_dirty_level_mask to 1.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_blit.c | 8
src/gallium/drivers/radeonsi/si_pipe.h | 1 +
src/gal
Old-style images are identified using TGSI_FILE_SAMPLER, but
bindless samplers can be TGSI_FILE_CONSTANT or TGSI_FILE_TEMPORARY.
To avoid backend compilers to be confused, this adds a new flag
that will only be set for bindless samplers.
Signed-off-by: Samuel Pitoiset
---
src/gallium/auxiliary/
This implements the Gallium interface. Decompression of resident
textures/images will follow in the next patches.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_descriptors.c | 340 ++
src/gallium/drivers/radeonsi/si_pipe.c| 12 +
src/gallium/
This will be used in order to initialize resident descriptors
for bindless textures/images.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_descriptors.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c
b/src/g
Whether bindless texture operations are supported by the
underlying driver.
Signed-off-by: Samuel Pitoiset
---
src/gallium/docs/source/screen.rst | 2 ++
src/gallium/drivers/etnaviv/etnaviv_screen.c | 1 +
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
src/gallium/driv
Signed-off-by: Samuel Pitoiset
---
src/gallium/auxiliary/util/u_threaded_context.c| 147 +
.../auxiliary/util/u_threaded_context_calls.h | 4 +
2 files changed, 151 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_threaded_context.c
b/src/gallium/auxiliary/
Old-style samplers (ie. bound samplers) are stored as
PROGRAM_SAMPLER, while bindless ones are PROGRAM_UNIFORM.
Signed-off-by: Samuel Pitoiset
---
src/mesa/program/ir_to_mesa.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/progra
Signed-off-by: Samuel Pitoiset
---
src/mesa/program/ir_to_mesa.cpp| 7 ---
src/mesa/program/ir_to_mesa.h | 4 ++--
src/mesa/state_tracker/st_glsl_to_nir.cpp | 3 +--
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 3 +--
src/mesa/state_tracker/st_shader_cache.c | 3 +-
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/ddebug/dd_context.c | 61 +
1 file changed, 61 insertions(+)
diff --git a/src/gallium/drivers/ddebug/dd_context.c
b/src/gallium/drivers/ddebug/dd_context.c
index 8260d4f869..8a07411c46 100644
--- a/src/galliu
Signed-off-by: Samuel Pitoiset
---
src/mesa/main/uniform_query.cpp | 122 ++--
1 file changed, 116 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/uniform_query.cpp b/src/mesa/main/uniform_query.cpp
index cc145f29e9..be04e48d53 100644
--- a/src/mesa/m
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 78 --
1 file changed, 63 insertions(+), 15 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 5734d0e456..264b43c10b
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 27 +--
1 file changed, 21 insertions(+), 6 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index ef6597e083..5734d0e456 1006
This will also be used for looping over bindless samplers bound
to texture units.
Signed-off-by: Samuel Pitoiset
---
src/mesa/main/uniforms.c | 67 +++-
1 file changed, 38 insertions(+), 29 deletions(-)
diff --git a/src/mesa/main/uniforms.c b/src/mesa
When a bindless sampler/image is bound to a texture/image unit,
we have to overwrite the constant value by the resident handle
directly in the constant buffer before the next draw.
One solution is to keep track of a pointer to the data.
Signed-off-by: Samuel Pitoiset
---
src/mesa/program/ir_to_
The ARB_bindless_texture spec says:
"When a sampler or image uniform's value is queried via any
of the GetUniform* commands, the returned value will reflect
the most recently set value through either UniformHandle* or
Uniform1i*, converted to the requested type."
Signed-off-by: Sam
For example, TGSI_OPCODE_STORE for bindless images might use
a constant buffer or a shader input.
Signed-off-by: Samuel Pitoiset
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_ureg.c
b/src/gallium/auxiliary/tgsi/tg
Old-style images are identified using TGSI_FILE_IMAGE, but
bindless images can be TGSI_FILE_CONSTANT or TGSI_FILE_TEMPORARY.
To avoid backend compilers to be confused, this adds a new flag
that will only be set for bindless images.
Signed-off-by: Samuel Pitoiset
---
src/gallium/auxiliary/tgsi/t
When a texture (or an image) instruction uses a bindless sampler
(respectively a bindless image), make sure the DCE pass won't
remove code when the resource is a temporary variable.
Signed-off-by: Samuel Pitoiset
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 15 +++
1 file changed
1 - 100 of 200 matches
Mail list logo
