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
+++
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
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
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 ++--
> > >
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
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
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
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
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
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
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
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
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
> ++
>
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
+++
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
>>
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
https://bugs.freedesktop.org/show_bug.cgi?id=101110
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
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(-)
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
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
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,
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
>
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 :
>
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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(-)
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
---
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
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
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
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
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
---
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 +--
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
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(+)
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
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 ++
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
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
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
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
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
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
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
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
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
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
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
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
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
---
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
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
---
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
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
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
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
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
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 +
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
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
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 +--
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
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
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
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
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
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
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:
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
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
---
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 - 100 of 200 matches
Mail list logo