https://bugs.freedesktop.org/show_bug.cgi?id=95215
Timothy Arceri changed:
What|Removed |Added
Resolution|--- |WONTFIX
Fixes: a017c7ecb7ae "mesa: display list support for uint uniforms"
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78097
---
Piglit test: https://patchwork.freedesktop.org/patch/221794/
src/mesa/main/dlist.c | 50 +++
1 file changed, 17
This seems to be broken at the moment for opengl interop. Which is
used by steamvr opengl applications.
---
Not sure if we want to merge this or not. It fixes steamvr opengl applications
on vega, but the proper solution is probably in some metadata bug.
Mailing it out mostly as an awareness
From: Dave Airlie
This enabled the vertex streams out when the host supports
GL4.0.
---
src/gallium/drivers/virgl/virgl_encode.c | 2 +-
src/gallium/drivers/virgl/virgl_screen.c | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git
On 4 April 2018 at 02:06, Jakob Bornecrantz wrote:
> Signed-off-by: Jakob Bornecrantz
not sure if still needed or not, but
Reviewed-by: Dave Airlie
> ---
> src/gallium/drivers/virgl/virgl_hw.h | 1 +
> 1 file changed, 1
The vertex array Size and Stride attributes are now ubyte and short,
respectively. The glGet code needed to be updated to handle those
types, but wasn't.
Fixes the new piglit test gl-1.5-get-array-attribs test.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106450
Fixes: d5f42f96e16
https://bugs.freedesktop.org/show_bug.cgi?id=106450
--- Comment #4 from Brian Paul ---
You're missing fixes for glGetFloat/Double/Booleanv(). And there are a few
more arrays like GL_SECONDARY_COLOR, GL_FOG_COORD, GL_EDGE_FLAG, etc.
I've got a patch which covers all that
https://bugs.freedesktop.org/show_bug.cgi?id=62612
Matt Turner changed:
What|Removed |Added
Resolution|--- |WONTFIX
https://bugs.freedesktop.org/show_bug.cgi?id=106450
--- Comment #3 from xuelian@samsung.com ---
Created attachment 139455
--> https://bugs.freedesktop.org/attachment.cgi?id=139455=edit
Patch for this bug
This is my patch, please check, thanks!
--
You are receiving this mail because:
You
Your idea of renaming ST_NEW_SAMPLE_MASK and using that sounds good. (we
already have 64 flags)
Marek
On Wed, May 9, 2018 at 12:21 AM, Marek Olšák wrote:
> I'd like a separate state for this, because set_framebuffer_state is a
> memory and execution barrier, so it decreases
Quite a sneaky little bug, can't hurt to make undefined behavior a bit
more defined :-).
Reviewed-by: Roland Scheidegger
Am 10.05.2018 um 00:20 schrieb Dave Airlie:
> From: Dave Airlie
>
> If you have an indirect access to a constant buffer on r600/eg
>
Reviewed-by: Bas Nieuwenhuizen
On Wed, May 9, 2018 at 11:52 PM, Samuel Pitoiset
wrote:
> It's a per-application optimization, so it makes more sense
> to do that in radv_handle_per_app_options().
>
> Signed-off-by: Samuel Pitoiset
For the series:
Reviewed-by: Marek Olšák
Marek
On Wed, May 9, 2018 at 3:29 PM, Jan Vesely wrote:
> Fixes memory leak on module unload.
> v2: Use util_hash_table helper function
>
> CC:
> Signed-off-by: Jan
From: Dave Airlie
This fixes 4 out of 5 cases in:
arb_framebuffer_no_attachments-atomic on cayman.
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
From: Thomas Petazzoni
The configure.ac logic added in commit
2ef7f23820a67e958c2252bd81eb0458903ebf33 ("configure: check if
-latomic is needed for __atomic_*") makes the assumption that if a
64-bit atomic intrinsic test program fails to link without -latomic,
it is
Thanks.
Reviewed-by: Matt Turner
Cc'd stable and pushed.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
https://bugs.freedesktop.org/show_bug.cgi?id=106445
Timothy Arceri changed:
What|Removed |Added
Resolution|--- |INVALID
On 10/05/18 02:46, Eric Anholt wrote:
Timothy Arceri writes:
---
src/mesa/state_tracker/st_program.c | 65 +
1 file changed, 58 insertions(+), 7 deletions(-)
diff --git a/src/mesa/state_tracker/st_program.c
On 09/05/18 23:18, Jason Ekstrand wrote:
>From the bspec docs for "Indirect State Pointers Disable":
"At the completion of the post-sync operation associated with this
pipe control packet, the indirect state pointers in the hardware are
considered invalid"
So the ISP disable is
From: Dave Airlie
If you have an indirect access to a constant buffer on r600/eg
use a vertex fetch in the shader. However apps have expected
behaviour on those out of bounds accessess (even if illegal).
If the constants were being uploaded as part of a larger
upload buffer,
With Michel's suggestion I've not seen a freeze in Plasmashell or Steam so
far
Tested-by: Mike Lothian
On Wed, 9 May 2018 at 21:47 Mike Lothian wrote:
> On Tue, 8 May 2018 at 20:47 Adam Jackson wrote:
>
>> dri*_bind_context, when
From the bspec docs for "Indirect State Pointers Disable":
"At the completion of the post-sync operation associated with this
pipe control packet, the indirect state pointers in the hardware are
considered invalid"
So the ISP disable is a post-sync type of operation which means that
It's a per-application optimization, so it makes more sense
to do that in radv_handle_per_app_options().
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_device.c | 18 ++
1 file changed, 6 insertions(+), 12 deletions(-)
diff --git
Reviewed-by: Samuel Pitoiset
On 05/09/2018 11:32 PM, Grazvydas Ignotas wrote:
Trivial.
---
src/amd/vulkan/radv_cmd_buffer.c | 4 ++--
src/amd/vulkan/radv_descriptor_set.c | 2 +-
src/amd/vulkan/radv_device.c | 6 +++---
On 05/07/2018 03:45 PM, Rhys Perry wrote:
> Could you merge this for me (with the change)? I don't have write access.
Pushed as
5ac16ed0476 mesa: fix error handling in get_framebuffer_parameteriv
Thanks!
> ___
> mesa-stable mailing list
>
Trivial.
---
src/amd/vulkan/radv_cmd_buffer.c | 4 ++--
src/amd/vulkan/radv_descriptor_set.c | 2 +-
src/amd/vulkan/radv_device.c | 6 +++---
src/amd/vulkan/radv_entrypoints_gen.py | 2 +-
src/amd/vulkan/radv_image.c| 4 ++--
src/amd/vulkan/radv_nir_to_llvm.c |
Out of tree builds can try to write into a directory that doesn't exist yet:
| Traceback (most recent call last):
| File "../../../mesa-18.0.2/src/intel/vulkan/anv_icd.py", line 46, in
| with open(args.out, 'w') as f:
| IOError: [Errno 2] No such file or directory:
On Tue, 8 May 2018 at 20:47 Adam Jackson wrote:
> dri*_bind_context, when switching current drawables, will drop the
> reference on the old one; since that refcount has probably now gone to
> zero that means we lose all the state we applied to that drawable
> before, like when
Use pipe_refernce to release old RAT surfaces.
RAT surface adds a reference to pool bo. So use reference counting for pool->bo
as well.
v2: Use the same pattern for both defrag paths
Drop confusing comment
CC:
Signed-off-by: Jan Vesely
Fixes memory leak on module unload.
v2: Use util_hash_table helper function
CC:
Signed-off-by: Jan Vesely
---
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git
CC:
Signed-off-by: Jan Vesely
---
src/gallium/auxiliary/util/u_hash_table.c | 17 +
src/gallium/auxiliary/util/u_hash_table.h | 5 +
2 files changed, 22 insertions(+)
diff --git
Pushed with the title changed.
Thanks again.
On 09/05/18 15:39, Rafael Antognolli wrote:
In the summary log, you should drop the mention to post sync operation
too, and mention stall on pixel scoreboard, right?
With that, this and the next patches are
Reviewed-by: Rafael Antognolli
https://bugs.freedesktop.org/show_bug.cgi?id=106283
--- Comment #9 from i...@yahoo.com ---
(In reply to Tapani Pälli from comment #8)
> I'm working on something different ATM and would not like to context switch
> but will look at this later. If changes from b...@besd.de make sense, it
> would be
Looks good to me.
On Wed, May 9, 2018 at 10:47 AM, Nanley Chery wrote:
> v2: Inline the switch statement (Jason)
>
> Reviewed-by: Jason Ekstrand
> ---
> src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 90 ---
> 1 file changed, 38
https://bugs.freedesktop.org/show_bug.cgi?id=100960
--- Comment #4 from Fabian Maurer ---
Yes, the screenshot from comment 1 is from the same apitrace, but taken on
windows.
I also retested on linux, and as of Mesa 18.2.0-devel (git-bd35345e85) the
issue is still present.
On Tuesday, May 8, 2018 10:33:45 AM PDT Francisco Jerez wrote:
> Kenneth Graunke writes:
>
> > On Tuesday, May 8, 2018 1:23:32 AM PDT Eero Tamminen wrote:
> >> Hi,
> >>
> >> On 08.05.2018 06:45, Matt Turner wrote:
> >> > On Mon, May 7, 2018 at 8:02 PM, Brian Paul
v2: Inline the switch statement (Jason)
Reviewed-by: Jason Ekstrand
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 90 ---
1 file changed, 38 insertions(+), 52 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
On Tue, 2018-05-08 at 11:42 +0200, Michel Dänzer wrote:
> Idle notify events shouldn't need special treatment, since the pixmap
> XIDs of the buffers will be different between loader_dri3_drawable
> incarnations, aren't they?
We're destroying loader_dri3_drawable structs at MakeCurrent time, but
On Fri, Jan 26, 2018 at 05:59:58PM -0800, Jason Ekstrand wrote:
> On CNL and above, CCS_E supports 1010102 formats and R11G11B10F. We had
> shut them off during early enabling because blorp_copy couldn't handle
> them. Now it can so we can turn them back on.
> ---
> src/intel/isl/isl_format.c |
It looks like get_sample_position should be in pipe_screen too, but you
don't have to worry about that.
Marek
On Wed, May 9, 2018 at 9:26 AM, Rhys Perry wrote:
> It was put in pipe_context because get_sample_position was there. It's
> not really a strong reason to
On Tue, May 08, 2018 at 04:59:50PM -0700, Jason Ekstrand wrote:
> I made a small smattering of comments. With those resolved, the series is
>
> Reviewed-by: Jason Ekstrand
>
> I really like the way it all came together in the end. Patch 13 was one I
> didn't expect to
On Tue, May 08, 2018 at 04:52:09PM -0700, Jason Ekstrand wrote:
> On Thu, May 3, 2018 at 12:03 PM, Nanley Chery wrote:
>
> > There isn't much that changes between the aux allocation functions.
> > Remove the duplicated code.
> > ---
> >
Timothy Arceri writes:
> ---
> src/mesa/state_tracker/st_program.c | 65 +
> 1 file changed, 58 insertions(+), 7 deletions(-)
>
> diff --git a/src/mesa/state_tracker/st_program.c
> b/src/mesa/state_tracker/st_program.c
> index
On 2018-05-09 08:20:59, Scott D Phillips wrote:
> Jason Ekstrand writes:
>
> > On Tue, May 8, 2018 at 10:58 AM, Jordan Justen
> > wrote:
> >
> >> On 2018-05-07 17:30:43, Scott D Phillips wrote:
> >> > From: Jason Ekstrand
On Tue, May 01, 2018 at 02:34:15PM -0700, Jason Ekstrand wrote:
> On Wed, Mar 7, 2018 at 5:08 AM, Pohjolainen, Topi <
> topi.pohjolai...@gmail.com> wrote:
>
> > On Fri, Jan 26, 2018 at 05:59:55PM -0800, Jason Ekstrand wrote:
> > > By making use of the NIR helper for uint vector casts, we should
On 2018-05-08 09:47 PM, Adam Jackson wrote:
> dri*_bind_context, when switching current drawables, will drop the
> reference on the old one; since that refcount has probably now gone to
> zero that means we lose all the state we applied to that drawable
> before, like when swaps are expected to
Reviewed-by: Caio Marcelo de Oliveira Filho
On Wed, May 09, 2018 at 04:40:37PM +0100, Lionel Landwerlin wrote:
> Signed-off-by: Lionel Landwerlin
> Fixes: 2dc29e095f9da ("i965: Don't leak blorp on Gen4-5.")
> ---
>
If the glsl is something like this:
in vec4 some_input;
interpolateAtCentroid(some_input[idx])
then it now gets generated as if it were:
interpolateAtCentroid(some_input)[idx]
This is necessary because the index will get generated as a series of
nir_bcsel instructions so it would no
Signed-off-by: Lionel Landwerlin
Fixes: 2dc29e095f9da ("i965: Don't leak blorp on Gen4-5.")
---
src/mesa/drivers/dri/i965/brw_context.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
https://bugs.freedesktop.org/show_bug.cgi?id=106450
--- Comment #2 from Brian Paul ---
FYI: I've posted a piglit patch for a new test to check this.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the
Jason Ekstrand writes:
> On Tue, May 8, 2018 at 10:58 AM, Jordan Justen
> wrote:
>
>> On 2018-05-07 17:30:43, Scott D Phillips wrote:
>> > From: Jason Ekstrand
>> >
>> > This is simple linear-walk first-fit allocator
This is needed for fixing CTS:
dEQP-GLES3.functional.occlusion_query.conservative*
Signed-off-by: Gert Wollny
---
This needs an additional patch in virglrenderer:
https://patchwork.freedesktop.org/patch/221718/
Best,
Gert
Duh! Thanks a lot :)
-
Lionel
On 09/05/18 15:39, Rafael Antognolli wrote:
In the summary log, you should drop the mention to post sync operation
too, and mention stall on pixel scoreboard, right?
With that, this and the next patches are
Reviewed-by: Rafael Antognolli
In the summary log, you should drop the mention to post sync operation
too, and mention stall on pixel scoreboard, right?
With that, this and the next patches are
Reviewed-by: Rafael Antognolli
On Wed, May 09, 2018 at 12:39:51AM +0100, Lionel Landwerlin wrote:
>
https://bugs.freedesktop.org/show_bug.cgi?id=106450
--- Comment #1 from Brian Paul ---
Can you please attach the patch here, or post it to mesa-dev? I can't access
the URL you gave.
BTW, I've confirmed the bug here.
--
You are receiving this mail because:
You are the
Reviewed-by: Leo Liu
On 05/08/2018 02:35 PM, boyuan.zh...@amd.com wrote:
From: Boyuan Zhang
All vce firmwares with major version greater than or equal to 53 are supported
Signed-off-by: Boyuan Zhang
---
On Wed, May 09, 2018 at 03:08:03PM +0100, Lionel Landwerlin wrote:
> It's just not possible to have a device with no subslices.
Reviewed-by: Rafael Antognolli
> CID: 1433511
> Signed-off-by: Lionel Landwerlin
> ---
>
It's just not possible to have a device with no subslices.
CID: 1433511
Signed-off-by: Lionel Landwerlin
---
src/intel/dev/gen_device_info.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/intel/dev/gen_device_info.c b/src/intel/dev/gen_device_info.c
index
https://bugs.freedesktop.org/show_bug.cgi?id=106157
Bug 106157 depends on bug 105906, which changed state.
Bug 105906 Summary: [DRI3] Compiz segfaults in intel_destroy_image()
https://bugs.freedesktop.org/show_bug.cgi?id=105906
What|Removed |Added
It was put in pipe_context because get_sample_position was there. It's
not really a strong reason to have it there though, so I'll put it in
pipe_screen.
On Wed, May 9, 2018 at 2:37 AM, Marek Olšák wrote:
> BTW, is there any reason for get_sample_pixel_grid to be in
On 09.05.2018 14:39, Ilia Mirkin wrote:
On Wed, May 9, 2018 at 2:12 AM, Tapani Pälli wrote:
Functionality already covered by ARB_texture_view, patch also
adds missing 'gles guard' for enums (added in f1563e6392).
Tested via arb_texture_view.*_gles3 tests and
https://bugs.freedesktop.org/show_bug.cgi?id=106452
--- Comment #8 from Samuel Pitoiset ---
I do agree with Alex. Though, as a temporary solution we could force that
behaviour in radv when dxvk is detected until we found a better option?
--
You are receiving this
On Wed, May 9, 2018 at 2:12 AM, Tapani Pälli wrote:
> Functionality already covered by ARB_texture_view, patch also
> adds missing 'gles guard' for enums (added in f1563e6392).
>
> Tested via arb_texture_view.*_gles3 tests and individual app
> utilizing texture view with
On 09.05.2018 11:22, Kenneth Graunke wrote:
On Tuesday, May 8, 2018 11:12:33 PM PDT Tapani Pälli wrote:
Signed-off-by: Tapani Pälli
---
src/mesa/drivers/dri/i965/intel_extensions.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
https://bugs.freedesktop.org/show_bug.cgi?id=106452
--- Comment #7 from Alex Smith ---
Yup, have to conditionalize any later side effects as well.
A workaround activated based on app name isn't so nice. I don't like the idea
of us shipping a game assuming that the
https://bugs.freedesktop.org/show_bug.cgi?id=106452
--- Comment #6 from Philip Rebohle ---
Hm, I wasn't aware of that.
In general I'd prefer a general workaround on my end since other drivers are
probably affectes as well, but i'm not sure if that's viable.
Alex
https://bugs.freedesktop.org/show_bug.cgi?id=106452
--- Comment #5 from Samuel Pitoiset ---
Yes, I can implement a similar behaviour on RADV which can be enabled based on
the executable name (and using a debug option).
--
You are receiving this mail because:
You are
On 2018-05-05 12:59 PM, Jason Ekstrand wrote:
> On Wed, May 2, 2018 at 2:35 AM, Michel Dänzer wrote:
>
>> From: Michel Dänzer
>>
>> And only free no longer needed back buffers there as well.
>>
>> We want to stick to the same back buffer throughout a
https://bugs.freedesktop.org/show_bug.cgi?id=106452
--- Comment #4 from Alex Smith ---
Derivatives are undefined after a discard in non-uniform control flow. SPIR-V's
OpKill says "all subsequent control flow is non-uniform", and derivatives are
only defined in
On Tue, 2018-05-08 at 08:43 -0600, Brian Paul wrote:
> Change the size of the bitset from 128 bits to 96. This works around an
> apparent GCC 5.4 bug in which bad SSE code is generated, leading to a
> crash in ast_type_qualifier::validate_in_qualifier() (ast_type.cpp:654).
>
> This can be
https://bugs.freedesktop.org/show_bug.cgi?id=106452
--- Comment #3 from Philip Rebohle ---
To avoid any confusion: The screenshots show the expected vs. actual contents
of the first color attachment at EID 8648 in the Renderdoc capture linked
above.
--
You are
https://bugs.freedesktop.org/show_bug.cgi?id=106452
--- Comment #2 from Philip Rebohle ---
Created attachment 139442
--> https://bugs.freedesktop.org/attachment.cgi?id=139442=edit
Expected rendering
--
You are receiving this mail because:
You are the assignee
https://bugs.freedesktop.org/show_bug.cgi?id=106452
--- Comment #1 from Philip Rebohle ---
Created attachment 139441
--> https://bugs.freedesktop.org/attachment.cgi?id=139441=edit
Incorrect rendering
--
You are receiving this mail because:
You are the QA
https://bugs.freedesktop.org/show_bug.cgi?id=106452
Bug ID: 106452
Summary: radv: Early discard in fragment shader breaks
derivatives
Product: Mesa
Version: git
Hardware: Other
OS: All
Status:
https://bugs.freedesktop.org/show_bug.cgi?id=106445
--- Comment #5 from Eero Tamminen ---
(In reply to Zacharybinx34 from comment #2)
> Could you add it for Cemu with Wine?
As noted above, somebody needs first to do extensive enough testing that
threading:
* doesn't
Hi,
On 08.05.2018 20:33, Francisco Jerez wrote:
Kenneth Graunke writes:
On Tuesday, May 8, 2018 1:23:32 AM PDT Eero Tamminen wrote:
On 08.05.2018 06:45, Matt Turner wrote:
On Mon, May 7, 2018 at 8:02 PM, Brian Paul wrote:
This probably prevents
Looks good to me.
Reviewed-by: Samuel Pitoiset
On 05/09/2018 02:45 AM, Dave Airlie wrote:
From: Dave Airlie
This fixes the fmask descriptor generation to handle 2d ms arrays.
---
src/amd/vulkan/radv_image.c | 2 +-
1 file changed, 1
https://bugs.freedesktop.org/show_bug.cgi?id=91106
Jeremy Huddleston Sequoia changed:
What|Removed |Added
Status|NEEDINFO
On Tuesday, May 8, 2018 11:12:33 PM PDT Tapani Pälli wrote:
> Signed-off-by: Tapani Pälli
> ---
> src/mesa/drivers/dri/i965/intel_extensions.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
>
https://bugs.freedesktop.org/show_bug.cgi?id=106450
Bug ID: 106450
Summary: glGetIntegerv return wrong value in some cases
Product: Mesa
Version: unspecified
Hardware: ARM
OS: Linux (All)
Status: NEW
On Fri, 2018-05-04 at 15:45 +0200, Mario Kleiner wrote:
> The real problem, if i understand it correctly, is the way the life-time
> of dri3_drawables and loader_dri3_drawables is managed atm. by Mesa's
> bindContext() functions. Whenever glXMakeCurrent() etc. are called to
> assign new/different
https://bugs.freedesktop.org/show_bug.cgi?id=106445
--- Comment #4 from Timothy Arceri ---
Cemu uses OpenGLs compatibility profile (requires at least 4.1) which is
unsupported in Mesa. Are you running it with overrides? If not it's going to be
slow and likely buggy as it
https://bugs.freedesktop.org/show_bug.cgi?id=100960
Timothy Arceri changed:
What|Removed |Added
Status|NEW |NEEDINFO
---
https://bugs.freedesktop.org/show_bug.cgi?id=106445
zacharybin...@hotmail.com changed:
What|Removed |Added
Resolution|INVALID |---
https://bugs.freedesktop.org/show_bug.cgi?id=106445
--- Comment #3 from zacharybin...@hotmail.com ---
Could you optimize it for better performance with Cemu please? Just compare amd
on linux vs nvidia.
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA
Signed-off-by: Tapani Pälli
---
src/mesa/drivers/dri/i965/intel_extensions.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index b5860f13cb..192971f32c
Functionality already covered by ARB_texture_view, patch also
adds missing 'gles guard' for enums (added in f1563e6392).
Tested via arb_texture_view.*_gles3 tests and individual app
utilizing texture view with ETC2.
Signed-off-by: Tapani Pälli
---
https://bugs.freedesktop.org/show_bug.cgi?id=91039
--- Comment #2 from Axel Davy ---
To my knowledge we've been handling function stack realignment in nine for a
few years now, and thus I assume this issue is solved. Do you confirm ?
--
You are receiving this mail because:
88 matches
Mail list logo