https://bugs.freedesktop.org/show_bug.cgi?id=99119
Bug ID: 99119
Summary: swr_fence_work.cpp(42): error: argument of type
"std::nullptr_t" is incompatible with parameter of
type "unsigned long"
Product: Mesa
From: Rainer Hochecker
This allows eglCreateImageKHR to access P010 surfaces created by vaapi
patch for drm, fourcc:
http://paste.ubuntu.com/23638632/
Signed-off-by: Rainer Hochecker
---
include/GL/internal/dri_interface.h | 4
Users/tests relying on the total reset count will start seeing a smaller
number since most of the hangs can be handled by engine reset.
Note that if reset engine x, context a running on engine y will be unaware
and unaffected.
To start the discussion, include just a total engine reset count. If
stage_mask is a bitmask of shader stages, so the proper comparison would
be (1 << MESA_SHADER_VERTEX), not MESA_SHADER_VERTEX itself.
But we only care for structure types, and VS inputs cannot be structs.
So we can just drop this entirely.
Signed-off-by: Kenneth Graunke
This fixes 555 dEQP tests (using the nougat-cts-dev branch), Piglit's
arb_program_interface_query/arb_program_interface_query-resource-query,
and GL45-CTS.program_interface_query.separate-programs-{tess-control,
tess-eval,geometry}. Only one dEQP program interface failure remains.
I would have
On Fri 16 Dec 2016, Chad Versace wrote:
> Happy Christmas to everyone who's busy squashing dEQP bugs.
>
> I wrote a new GBM backend for dEQP. [...]
> There's probably bugs. No surprises there.
You probably want this Mesa patch before using dEQP on GBM. It fixes
some EGL failures.
Subject:
If the provided EGLConfig does not support the requested surface type,
then emit EGL_BAD_MATCH.
Fixes dEQP-EGL.functional.negative_api.create_pbuffer_surface
on GBM.
Cc: "13.0"
---
src/egl/main/eglapi.c | 10 ++
1 file changed, 10 insertions(+)
diff
Happy Christmas to everyone who's busy squashing dEQP bugs.
I wrote a new GBM backend for dEQP. I even submitted it to dEQP's
upstream Gerrit. Pyry, dEQP's maintainer, told me over beer earlier
this year that he would accept it if I submitted it, and if it wasn't
too crazy. So, maybe it'll be
This series is,
Reviewed-by: Edward O'Callaghan
On 12/16/2016 02:42 AM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Should have no effect (other than perhaps on power consumption), but
> Vulkan does this.
> ---
>
On 15 December 2016 at 12:23, Ilia Mirkin wrote:
> ping
>
> On Sun, Nov 27, 2016 at 4:22 PM, Ilia Mirkin wrote:
>> The strategy is to do the same thing that the GLSL lower_offset_arrays
>> pass does - create 4 separate texture gather ops, one per
On 17 December 2016 at 08:14, Bas Nieuwenhuizen
wrote:
> The spec says we have to try to create all, and only set failed
> pipelines to VK_NULL_HANDLE. If one of them fails, we have to return
> an error, but as far as I can see, the spec does not care which of
> the
https://bugs.freedesktop.org/show_bug.cgi?id=99116
--- Comment #1 from Ilia Mirkin ---
Replays fine on Intel SKL GT2 (Mesa 13.0.2 with Intel DDX + DRI3). Probably an
issue with st/mesa's logic somewhere.
--
You are receiving this mail because:
You are the QA Contact for
https://bugs.freedesktop.org/show_bug.cgi?id=99116
Fabian Maurer changed:
What|Removed |Added
CC||dark.shad...@web.de
https://bugs.freedesktop.org/show_bug.cgi?id=99116
Bug ID: 99116
Summary: Wine program showing only a blackscreen when using
mesa
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
The spec says we have to try to create all, and only set failed
pipelines to VK_NULL_HANDLE. If one of them fails, we have to return
an error, but as far as I can see, the spec does not care which of
the suberrors.
Fixes
dEQP-VK.api.object_management.alloc_callback_fail_multiple.compute_pipeline
On 12/15/2016 05:25 PM, Xu, Randy wrote:
> Thanks, Matt
>
> I have run "git config --global user.name "Randy Xu"" and see it in
> ~/.gitconfig
> [user]
> email = randy...@intel.com
> name = Randy Xu
Perhaps you need quotes around the name? My ~/.gitconfig has:
[user]
On Fri, Dec 16, 2016 at 1:26 PM, Francisco Jerez
wrote:
> Jason Ekstrand writes:
>
> > On Dec 15, 2016 3:22 PM, "Francisco Jerez"
> wrote:
> >
> > This fixes an apparent regression in a bunch of image store vulkan CTS
> >
On 12/16/2016 12:49 PM, Chad Versace wrote:
> On Fri 16 Dec 2016, Chad Versace wrote:
>> On Fri 16 Dec 2016, Randy Xu wrote:
>>> From: "Xu,Randy"
>>>
>>> The ES specification says that TexImage3D should return GL_INVALID_OPERATION
>>> if the internal format is DEPTH_COMPONENT,
Vedran Miletić writes:
> Current implementation of event handling keeps an extra reference to
> the hardware event, in addition to the reference returned via the OpenCL
> API. This additional reference is internal and should not be counted
> when queried via the
Jason Ekstrand writes:
> On Dec 15, 2016 3:22 PM, "Francisco Jerez" wrote:
>
> This fixes an apparent regression in a bunch of image store vulkan CTS
> tests from commit ad38ba113491869ab0dffed937f7b3dd50e8a735, which
> started using OWORD block read
On Fri 16 Dec 2016, Chad Versace wrote:
> On Fri 16 Dec 2016, Randy Xu wrote:
> > From: "Xu,Randy"
> >
> > The ES specification says that TexImage3D should return GL_INVALID_OPERATION
> > if the internal format is DEPTH_COMPONENT, DEPTH_-STENCIL or STENCIL_INDEX.
> > The
On Fri 16 Dec 2016, Randy Xu wrote:
> From: "Xu,Randy"
>
> The ES specification says that TexImage3D should return GL_INVALID_OPERATION
> if the internal format is DEPTH_COMPONENT, DEPTH_-STENCIL or STENCIL_INDEX.
> The current code returns INVALID_ENUM as
On Thu 15 Dec 2016, Emil Velikov wrote:
> Hi Chad,
>
> On 10 December 2016 at 00:40, Chad Versace wrote:
> > intel_miptree_make_shareable() discarded and disabled CCS. Fix it so
> > that it discards and disables HiZ too.
> >
> > Fixes
> >
On Thu, Dec 15, 2016 at 10:52 PM, Thomas Helland
wrote:
> Could I bother one of you to push it?
> I haven't yet acquired commit access.
>
Done.
> 2016-12-16 7:26 GMT+01:00 Jason Ekstrand :
> > Thanks! Rb.
> >
> > On Dec 15, 2016 3:30 PM, "Eric
On Fri, Dec 16, 2016 at 11:08 AM, Jason Ekstrand
wrote:
> On Thu, Dec 15, 2016 at 6:29 PM, Jason Ekstrand
> wrote:
>
>> On Dec 15, 2016 3:08 PM, "Eric Anholt" wrote:
>>
>> Jason Ekstrand writes:
>>
>> > This
Ken, Eric,
Any thoughts on whether or not this should go to stable? It does fix a
real bug but the result of the bug is that we end up generating a bunch of
duplicate phi nodes instead of just one for the block. In the end it
doesn't matter since CSE cleans them up.
--Jason
On Fri, Dec 16,
On Thu, Dec 15, 2016 at 6:29 PM, Jason Ekstrand
wrote:
> On Dec 15, 2016 3:08 PM, "Eric Anholt" wrote:
>
> Jason Ekstrand writes:
>
> > This keeps some of Connor's original code. However, while I was at it,
> > I updated this very
After we figure out the value that we are going to return, we have a
loop that walks up the dominance tree and sets the value in each of the
blocks that doesn't have one yet. In the case of the phi, the def is
set to NEEDS_PHI not NULL, so the last one where the phi node actually
goes never gets
This keeps some of Connor's original code. However, while I was at it,
I updated this very old pass to a bit more modern NIR.
---
src/compiler/nir/nir_lower_regs_to_ssa.c | 589 +--
1 file changed, 167 insertions(+), 422 deletions(-)
diff --git
---
src/compiler/nir/nir.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index f1c99ce..9402863 100644
--- a/src/compiler/nir/nir.h
+++ b/src/compiler/nir/nir.h
@@ -373,6 +373,11 @@ typedef struct nir_register {
struct list_head
This matches the naming of nir_lower_vars_to_ssa, the other to-SSA pass.
---
src/compiler/Makefile.sources | 2 +-
src/compiler/nir/nir.h | 4 +---
src/compiler/nir/{nir_to_ssa.c => nir_lower_regs_to_ssa.c} | 8
https://bugs.freedesktop.org/show_bug.cgi?id=99076
Chad Versace changed:
What|Removed |Added
CC|
Current implementation of event handling keeps an extra reference to
the hardware event, in addition to the reference returned via the OpenCL
API. This additional reference is internal and should not be counted
when queried via the clGetEventInfo() function.
Fixes Piglit's
For the series:
Reviewed-by: Marek Olšák
Marek
On Thu, Dec 15, 2016 at 4:42 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> The clipper hardware doesn't consider points as primitives that can be
> clipped. Simply setting
2016-12-16 18:12 GMT+01:00 Gustaw Smolarczyk :
> 2016-12-16 17:57 GMT+01:00 Emil Velikov :
>> On 16 December 2016 at 15:27, Gustaw Smolarczyk wrote:
>>> 2016-12-16 14:50 GMT+01:00 Emil Velikov :
On 16 December 2016 at 15:36, Nicolai Hähnle wrote:
> On 04.12.2016 11:34, Gustaw Smolarczyk wrote:
>>
>> They are picked automatically by the provided llvm-config flags, but are
>> not needed.
>>
>> Fixes loading radv through a vulkan loader.
>>
>> Cc: 13.0
Hi Tomasz,
does this actually fix anything?
Because right now llvm-config.h does not include anything and I doubt it will
in the future, as it's just a collection of defines.
The path to the header file itself is given by llvm-config ($LLVM_INCLUDEDIR).
Did you just happen to see this or do
2016-12-16 17:57 GMT+01:00 Emil Velikov :
> On 16 December 2016 at 15:27, Gustaw Smolarczyk wrote:
>> 2016-12-16 14:50 GMT+01:00 Emil Velikov :
>>>
>>> On 5 October 2016 at 23:12, Gustaw Smolarczyk
Hey Emil,
did you have to time look at this?
What should we do?
I would argue to first merge what I have now and if you really want to
disallow llvm-less gallium with llvm-full radv we could do this later and
revert the changes of patch 4.
Personally I think it's ok how it is now. Gallium
On 16 December 2016 at 15:27, Gustaw Smolarczyk wrote:
> 2016-12-16 14:50 GMT+01:00 Emil Velikov :
>>
>> On 5 October 2016 at 23:12, Gustaw Smolarczyk
>> wrote:
>> > 2016-10-06 0:05 GMT+02:00 Emil Velikov
On 04.12.2016 11:34, Gustaw Smolarczyk wrote:
They are picked automatically by the provided llvm-config flags, but are
not needed.
Fixes loading radv through a vulkan loader.
Cc: 13.0
---
It's work-around for:
2016-12-16 14:50 GMT+01:00 Emil Velikov :
> On 5 October 2016 at 23:12, Gustaw Smolarczyk
> wrote:
> > 2016-10-06 0:05 GMT+02:00 Emil Velikov :
> >> On 5 October 2016 at 21:45, Gustaw Smolarczyk
>
https://bugs.freedesktop.org/show_bug.cgi?id=99014
Vedran Miletić changed:
What|Removed |Added
Summary|clover is broken in 12.0.4 |clover is broken
https://bugs.freedesktop.org/show_bug.cgi?id=99014
--- Comment #9 from Vedran Miletić ---
Not reproducible on Hawaii.
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.___
From: Emil Velikov
When using static link with SWR et al. causes huge sprike in processing
power/memory gets leading to hude increase in the test times.
Revert to the default value, which is used by the vast majority.
Cc: 12.0 13.0
From: Emil Velikov
Typos do happen as people nominate patches for stable. This script aims
to catch most of those.
Due to the subtle nature of things, one has to pay special attention to
the output, similar to get-extra-pick-list.sh.
At the moment only the following
From: Samuel Iglesias Gonsálvez
We need to pick two 32-bit values per component to perform the right shuffle
operation.
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c | 25 +
1 file changed,
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/src/compiler/spirv/spirv_to_nir.c
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/compiler/spirv/spirv_to_nir.c
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c | 1 +
src/compiler/spirv/vtn_private.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/compiler/spirv/spirv_to_nir.c
From: Samuel Iglesias Gonsálvez
Currently, gen8+ supports ARB_gpu_shader_fp64 in mesa master.
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/mesa/drivers/dri/i965/brw_compiler.c | 36 +---
1 file changed, 24
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/intel/vulkan/anv_device.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
index
So far, input_reads was a bitmap tracking which vertex input locations
were being used.
In OpenGL, an attribute bigger than a vec4 (like a dvec3 or dvec4)
consumes just one location, any other small attribute. So we mark the
proper bit in inputs_read, and also the same bit in double_inputs_read
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/nir/nir.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index 9f3abb7..c369db3 100644
---
From: Samuel Iglesias Gonsálvez
We use *64*_PASSTHRU formats to upload vertex attributes of 64 bits
to avoid conversions. From the BDW PRM, Volume 2d, page 586
(VERTEX_ELEMENT_STATE):
"When SourceElementFormat is set to one of the *64*_PASSTHRU
formats, 64-bit
---
src/intel/vulkan/anv_pipeline.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/intel/vulkan/anv_pipeline.c b/src/intel/vulkan/anv_pipeline.c
index 30ac19a..6a141b6 100644
--- a/src/intel/vulkan/anv_pipeline.c
+++ b/src/intel/vulkan/anv_pipeline.c
@@ -489,6 +489,7 @@
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/vtn_glsl450.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/compiler/spirv/vtn_glsl450.c b/src/compiler/spirv/vtn_glsl450.c
index
From: Samuel Iglesias Gonsálvez
SPIR-V does not have special opcodes for DF conversions. We need to identify
them by checking the bit size of the operand and the result.
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/amd/vulkan/radv_pipeline.c| 5 +++-
src/compiler/spirv/nir_spirv.h| 5 +++-
src/compiler/spirv/spirv_to_nir.c | 51 +++
From: Samuel Iglesias Gonsálvez
This function returns the nir_op corresponding to the conversion between
the given nir_alu_type arguments.
This function lacks support for integer-based types with bit_size != 32
and for float16 conversion ops.
Signed-off-by: Samuel
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/nir/nir.h | 19 +++
1 file changed, 19 insertions(+)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index 544d4ba..9310dab 100644
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/vtn_variables.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/compiler/spirv/vtn_variables.c
b/src/compiler/spirv/vtn_variables.c
index
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/nir_types.cpp | 15 +++
src/compiler/nir_types.h | 2 ++
2 files changed, 17 insertions(+)
diff --git a/src/compiler/nir_types.cpp
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/compiler/spirv/spirv_to_nir.c
b/src/compiler/spirv/spirv_to_nir.c
index
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/vtn_variables.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/compiler/spirv/vtn_variables.c
b/src/compiler/spirv/vtn_variables.c
index
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/vtn_variables.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/compiler/spirv/vtn_variables.c
b/src/compiler/spirv/vtn_variables.c
index
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/compiler/spirv/spirv_to_nir.c
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/src/compiler/spirv/spirv_to_nir.c
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/compiler/spirv/spirv_to_nir.c
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/compiler/spirv/spirv_to_nir.c
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/compiler/spirv/spirv_to_nir.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/compiler/spirv/spirv_to_nir.c
This patch series is a second iteration of previous one:
https://lists.freedesktop.org/archives/mesa-dev/2016-November/136507.html
Main changes are the ones suggested by Jason, and also a refactor of the way
inputs_read bitmap is used in NIR.
If you want to test these patches, you can clone our
On 5 October 2016 at 23:12, Gustaw Smolarczyk wrote:
> 2016-10-06 0:05 GMT+02:00 Emil Velikov :
>> On 5 October 2016 at 21:45, Gustaw Smolarczyk wrote:
>>> Hello,
>>>
>>> I have encountered a following problem while trying to
Ugh! Thanks for this.
This is
Reviewed-by: Andres Gomez
Br.
On Thu, 2016-12-15 at 20:10 -0800, Ian Romanick wrote:
> From: Ian Romanick
>
> glsl/ast_type.cpp: In function ‘bool validate_point_mode(YYLTYPE*,
> _mesa_glsl_parse_state*, const
vl_compositor_set_csc_matrix may return false
Signed-off-by: Nayan Deshmukh
---
src/gallium/state_trackers/xvmc/attributes.c | 3 ++-
src/gallium/state_trackers/xvmc/context.c| 4 +++-
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git
handle the cases when vl_compositor_set_csc_matrix(),
vl_compositor_init_state() and vl_compositor_init() fail
Signed-off-by: Nayan Deshmukh
---
src/gallium/state_trackers/va/context.c | 18 +++---
1 file changed, 15 insertions(+), 3 deletions(-)
diff
handle the cases when vl_compositor_set_csc_matrix(),
vl_compositor_init_state() and vl_compositor_init() fail
Signed-off-by: Nayan Deshmukh
---
src/gallium/state_trackers/vdpau/device.c | 8 +-
src/gallium/state_trackers/vdpau/mixer.c | 43
v2: add missing semicolon
Signed-off-by: Nayan Deshmukh
---
src/gallium/state_trackers/xvmc/context.c | 40 +++
1 file changed, 19 insertions(+), 21 deletions(-)
diff --git a/src/gallium/state_trackers/xvmc/context.c
pipe_buffer_map and pipe_buffer_create may return NULL
Signed-off-by: Nayan Deshmukh
---
src/gallium/auxiliary/vl/vl_compositor.c | 13 +++--
src/gallium/auxiliary/vl/vl_compositor.h | 2 +-
2 files changed, 12 insertions(+), 3 deletions(-)
diff --git
Thanks for taking care of this! Patches 4-6 are also:
Reviewed-by: Nicolai Hähnle
On 16.12.2016 10:52, Michel Dänzer wrote:
From: Michel Dänzer
This reverts commit 6dc96de303290e8d1fc294da478c4f370be98dea. No longer
necessary with the
On 16.12.2016 10:52, Michel Dänzer wrote:
From: Michel Dänzer
Only copy/memset the pointers that actually need to be.
Signed-off-by: Michel Dänzer
---
src/gallium/auxiliary/cso_cache/cso_context.c | 21 +
1 file changed, 17
On 16.12.2016 10:52, Michel Dänzer wrote:
From: Michel Dänzer
Preparation for following changes, no functional change intended.
Signed-off-by: Michel Dänzer
---
src/gallium/auxiliary/cso_cache/cso_context.c | 33 +++
1
On 16.12.2016 10:52, Michel Dänzer wrote:
From: Michel Dänzer
If info->nr_samplers > ctx->nr_fragment_samplers_saved, the assignment
would prevent cso_single_sampler_done from unbinding the no longer used
samplers from the driver, which could result in use-after-free.
On 15.12.2016 21:35, Tom Stellard wrote:
Also add a need_range_checks parameter to this function, which can be
set to false to enable some additional optimizations. Currently, this
will cause the compiler to emit the llvm.SI.buffer.load.dword intrinsic
instead of llvm.amdgcn.buffer.load.
On 15.12.2016 19:11, Tom Stellard wrote:
This prevents LLVM from using sext instructions for local memory offsets
and allows the backend to fold immediate offsets into the instruction.
This also prevents some incorrect code generation for ptrtoint and
inttoptr instructions.
Reviewed-by:
From: Michel Dänzer
Remove currently bound sampler states from the hash table before pruning
entries from the hash table, so currently bound states cannot
accidentally be deleted by the pruning.
Signed-off-by: Michel Dänzer
---
From: Michel Dänzer
Preparation for following changes, no functional change intended.
Signed-off-by: Michel Dänzer
---
src/gallium/auxiliary/cso_cache/cso_context.c | 33 +++
1 file changed, 18 insertions(+), 15
From: Michel Dänzer
It turned out to be slightly more complicated than I'd expected, but I
think I've found a good solution with low (hopefully insignificant)
additional overhead. In fact, thanks to the optimization in patch 3, the
overhead may be slightly lower in
From: Michel Dänzer
If info->nr_samplers > ctx->nr_fragment_samplers_saved, the assignment
would prevent cso_single_sampler_done from unbinding the no longer used
samplers from the driver, which could result in use-after-free. This is
probably unlikely to happen in
From: Michel Dänzer
Only copy/memset the pointers that actually need to be.
Signed-off-by: Michel Dänzer
---
src/gallium/auxiliary/cso_cache/cso_context.c | 21 +
1 file changed, 17 insertions(+), 4 deletions(-)
diff --git
From: Michel Dänzer
This reverts commit 6dc96de303290e8d1fc294da478c4f370be98dea. No longer
necessary with the previous change.
Signed-off-by: Michel Dänzer
---
src/gallium/auxiliary/cso_cache/cso_cache.c | 4 +---
1 file changed, 1
From: Michel Dänzer
Preparation for following changes, no functional change intended.
Signed-off-by: Michel Dänzer
---
src/gallium/auxiliary/cso_cache/cso_cache.h | 1 +
src/gallium/auxiliary/cso_cache/cso_context.c | 1 +
2 files changed, 2
https://bugs.freedesktop.org/show_bug.cgi?id=99027
Timo Aaltonen changed:
What|Removed |Added
Status|REOPENED|RESOLVED
93 matches
Mail list logo