[Mesa-dev] [Bug 95215] ARB_shading_language_include is not implemented

2018-05-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95215 Timothy Arceri changed: What|Removed |Added Resolution|--- |WONTFIX

[Mesa-dev] [PATCH] mesa: add glUniform*ui{v} support to display lists

2018-05-09 Thread Timothy Arceri
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

[Mesa-dev] [PATCH] radv: disable DCC for shareable images on GFX9+

2018-05-09 Thread Andres Rodriguez
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

[Mesa-dev] [PATCH] virgl: enable vertex streams when glsl level is high enough.

2018-05-09 Thread Dave Airlie
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

Re: [Mesa-dev] [PATCH] virgl: Update virgl_hw.h

2018-05-09 Thread Dave Airlie
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

[Mesa-dev] [PATCH] mesa: fix glGetInteger/Float/etc queries for vertex arrays attribs

2018-05-09 Thread Brian Paul
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

[Mesa-dev] [Bug 106450] glGetIntegerv return wrong value in some cases

2018-05-09 Thread bugzilla-daemon
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

[Mesa-dev] [Bug 62612] dispatch_sanity test lumps GL4.0+ functions into 4.3

2018-05-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=62612 Matt Turner changed: What|Removed |Added Resolution|--- |WONTFIX

[Mesa-dev] [Bug 106450] glGetIntegerv return wrong value in some cases

2018-05-09 Thread bugzilla-daemon
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

Re: [Mesa-dev] [PATCH 3/5] st/mesa: add support for ARB_sample_locations

2018-05-09 Thread Marek Olšák
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

Re: [Mesa-dev] [PATCH] r600: fix constant buffer bounds.

2018-05-09 Thread Roland Scheidegger
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 >

Re: [Mesa-dev] [PATCH] radv: move handling nosisched option in a better place

2018-05-09 Thread Bas Nieuwenhuizen
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

Re: [Mesa-dev] [PATCH v2 2/2] winsys/radeon: Destroy fd_hash table when the last winsys is removed.

2018-05-09 Thread Marek Olšák
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

[Mesa-dev] [PATCH] tgsi/scan: add hw atomic to the list of memory accessing files

2018-05-09 Thread Dave Airlie
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

[Mesa-dev] [PATCH] configure.ac: rework -latomic check

2018-05-09 Thread Matt Turner
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

Re: [Mesa-dev] [PATCH] src/intel/Makefile.vulkan.am: add missing MKDIR_GEN

2018-05-09 Thread Matt Turner
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

[Mesa-dev] [Bug 106445] No Threading

2018-05-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106445 Timothy Arceri changed: What|Removed |Added Resolution|--- |INVALID

Re: [Mesa-dev] [PATCH 2/2] st/nir: use NIR for asm programs

2018-05-09 Thread Timothy Arceri
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

Re: [Mesa-dev] [PATCH] i965, anv: Set the CS stall bit on the ISP disable PIPE_CONTROL

2018-05-09 Thread Lionel Landwerlin
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

[Mesa-dev] [PATCH] r600: fix constant buffer bounds.

2018-05-09 Thread Dave Airlie
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,

Re: [Mesa-dev] [PATCH] glx/dri: Take an extra reference on our own GLX drawables

2018-05-09 Thread Mike Lothian
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

[Mesa-dev] [PATCH] i965, anv: Set the CS stall bit on the ISP disable PIPE_CONTROL

2018-05-09 Thread Jason Ekstrand
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

[Mesa-dev] [PATCH] radv: move handling nosisched option in a better place

2018-05-09 Thread Samuel Pitoiset
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

Re: [Mesa-dev] [PATCH] radv: assorted typo fixes

2018-05-09 Thread Samuel Pitoiset
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 +++---

Re: [Mesa-dev] [Mesa-stable] [PATCH] mesa: fix error handling in get_framebuffer_parameteriv

2018-05-09 Thread Ian Romanick
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 >

[Mesa-dev] [PATCH] radv: assorted typo fixes

2018-05-09 Thread Grazvydas Ignotas
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 |

[Mesa-dev] [PATCH] src/intel/Makefile.vulkan.am: add missing MKDIR_GEN

2018-05-09 Thread Ross Burton
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:

Re: [Mesa-dev] [PATCH] glx/dri: Take an extra reference on our own GLX drawables

2018-05-09 Thread Mike Lothian
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

[Mesa-dev] [PATCH v2 1/1] eg/compute: Use reference counting to handle compute memory pool.

2018-05-09 Thread Jan Vesely
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

[Mesa-dev] [PATCH v2 2/2] winsys/radeon: Destroy fd_hash table when the last winsys is removed.

2018-05-09 Thread 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

[Mesa-dev] [PATCH 1/2] gallium/auxiliary: Add helper function to count the number of entries in hash table

2018-05-09 Thread Jan Vesely
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

Re: [Mesa-dev] [PATCH v2 1/2] i965: require post sync operation prior to ISP disable

2018-05-09 Thread Lionel Landwerlin
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

[Mesa-dev] [Bug 106283] Shader replacements works only for limited use cases

2018-05-09 Thread bugzilla-daemon
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

Re: [Mesa-dev] [PATCH 9.5/17] squash! i965/miptree: Unify aux buffer allocation

2018-05-09 Thread Jason Ekstrand
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

[Mesa-dev] [Bug 100960] Special block from Minecraft mod rendered out of place

2018-05-09 Thread bugzilla-daemon
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.

Re: [Mesa-dev] gcc bug / crash in ast_type_qualifier::validate_in_qualifier()?

2018-05-09 Thread Kenneth Graunke
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

[Mesa-dev] [PATCH 9.5/17] squash! i965/miptree: Unify aux buffer allocation

2018-05-09 Thread Nanley Chery
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

Re: [Mesa-dev] [PATCH 2/2] loader_dri3: Variant 2: Wait for pending swaps to complete before drawable_fini.

2018-05-09 Thread Adam Jackson
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

Re: [Mesa-dev] [PATCH 29/29] intel/isl: Allow CCS_E on more formats

2018-05-09 Thread Pohjolainen, Topi
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 |

Re: [Mesa-dev] [PATCH 2/5] gallium: add support for programmable sample locations

2018-05-09 Thread Marek Olšák
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

Re: [Mesa-dev] [PATCH 00/17] i965: Avoid a redundant color buffer fast-clear

2018-05-09 Thread Nanley Chery
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

Re: [Mesa-dev] [PATCH 09/17] i965/miptree: Unify aux buffer allocation

2018-05-09 Thread Nanley Chery
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. > > --- > >

Re: [Mesa-dev] [PATCH 2/2] st/nir: use NIR for asm programs

2018-05-09 Thread Eric Anholt
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

Re: [Mesa-dev] [PATCH 1/8] util: Add a virtual memory allocator

2018-05-09 Thread Jordan Justen
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

Re: [Mesa-dev] [PATCH 26/29] intel/blorp: Add support for more format bitcasting

2018-05-09 Thread Pohjolainen, Topi
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

Re: [Mesa-dev] [PATCH] glx/dri: Take an extra reference on our own GLX drawables

2018-05-09 Thread Michel Dänzer
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

Re: [Mesa-dev] [PATCH] i965: silence unused variable

2018-05-09 Thread Caio Marcelo de Oliveira Filho
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.") > --- >

[Mesa-dev] [PATCH] spirv: Fix InterpolateAt* instructions for vecs with dynamic index

2018-05-09 Thread Neil Roberts
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

[Mesa-dev] [PATCH] i965: silence unused variable

2018-05-09 Thread Lionel Landwerlin
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

[Mesa-dev] [Bug 106450] glGetIntegerv return wrong value in some cases

2018-05-09 Thread bugzilla-daemon
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

Re: [Mesa-dev] [PATCH 1/8] util: Add a virtual memory allocator

2018-05-09 Thread Scott D Phillips
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

[Mesa-dev] [PATCH] virgl: Add support for passing GL_ANY_SAMPLES_PASSED_CONSERVATIVE

2018-05-09 Thread Gert Wollny
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

Re: [Mesa-dev] [PATCH v2 1/2] i965: require post sync operation prior to ISP disable

2018-05-09 Thread Lionel Landwerlin
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

Re: [Mesa-dev] [PATCH v2 1/2] i965: require post sync operation prior to ISP disable

2018-05-09 Thread 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: >

[Mesa-dev] [Bug 106450] glGetIntegerv return wrong value in some cases

2018-05-09 Thread bugzilla-daemon
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

Re: [Mesa-dev] [PATCH] radeon/vce: add firmware support for ver 53 and up

2018-05-09 Thread Leo Liu
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 ---

Re: [Mesa-dev] [PATCH] intel: devinfo: silence coverity warning

2018-05-09 Thread Rafael Antognolli
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 > --- >

[Mesa-dev] [PATCH] intel: devinfo: silence coverity warning

2018-05-09 Thread 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

[Mesa-dev] [Bug 106157] [Tracker] Mesa 18.1 release tracker

2018-05-09 Thread bugzilla-daemon
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

Re: [Mesa-dev] [PATCH 2/5] gallium: add support for programmable sample locations

2018-05-09 Thread Rhys Perry
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

Re: [Mesa-dev] [PATCH 1/2] mesa: changes to expose OES_texture_view extension

2018-05-09 Thread Tapani Pälli
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

[Mesa-dev] [Bug 106452] radv: Early discard in fragment shader breaks derivatives

2018-05-09 Thread bugzilla-daemon
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

Re: [Mesa-dev] [PATCH 1/2] mesa: changes to expose OES_texture_view extension

2018-05-09 Thread Ilia Mirkin
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

Re: [Mesa-dev] [PATCH 2/2] i965: enable OES_texture_view for gen8+

2018-05-09 Thread Tapani Pälli
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

[Mesa-dev] [Bug 106452] radv: Early discard in fragment shader breaks derivatives

2018-05-09 Thread bugzilla-daemon
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

[Mesa-dev] [Bug 106452] radv: Early discard in fragment shader breaks derivatives

2018-05-09 Thread bugzilla-daemon
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

[Mesa-dev] [Bug 106452] radv: Early discard in fragment shader breaks derivatives

2018-05-09 Thread bugzilla-daemon
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

Re: [Mesa-dev] [PATCH] dri3: Only update number of back buffers in loader_dri3_get_buffers

2018-05-09 Thread Michel Dänzer
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

[Mesa-dev] [Bug 106452] radv: Early discard in fragment shader breaks derivatives

2018-05-09 Thread bugzilla-daemon
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

Re: [Mesa-dev] [Mesa-stable] [PATCH] glsl: change ast_type_qualifier bitset size to work around GCC 5.4 bug

2018-05-09 Thread Juan A. Suarez Romero
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

[Mesa-dev] [Bug 106452] radv: Early discard in fragment shader breaks derivatives

2018-05-09 Thread bugzilla-daemon
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

[Mesa-dev] [Bug 106452] radv: Early discard in fragment shader breaks derivatives

2018-05-09 Thread bugzilla-daemon
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

[Mesa-dev] [Bug 106452] radv: Early discard in fragment shader breaks derivatives

2018-05-09 Thread bugzilla-daemon
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

[Mesa-dev] [Bug 106452] radv: Early discard in fragment shader breaks derivatives

2018-05-09 Thread bugzilla-daemon
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:

[Mesa-dev] [Bug 106445] No Threading

2018-05-09 Thread bugzilla-daemon
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

Re: [Mesa-dev] gcc bug / crash in ast_type_qualifier::validate_in_qualifier()?

2018-05-09 Thread Eero Tamminen
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

Re: [Mesa-dev] [PATCH] radv: handle arrays in the fmask descriptor.

2018-05-09 Thread Samuel Pitoiset
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

[Mesa-dev] [Bug 91106] glx: make check fails to build on osx

2018-05-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91106 Jeremy Huddleston Sequoia changed: What|Removed |Added Status|NEEDINFO

Re: [Mesa-dev] [PATCH 2/2] i965: enable OES_texture_view for gen8+

2018-05-09 Thread Kenneth Graunke
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 >

[Mesa-dev] [Bug 106450] glGetIntegerv return wrong value in some cases

2018-05-09 Thread bugzilla-daemon
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

Re: [Mesa-dev] [RFC] Fix attempt for Mesa + X-Server 1.20 + modesetting-ddx hangs on KDE5.

2018-05-09 Thread Adam Jackson
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

[Mesa-dev] [Bug 106445] No Threading

2018-05-09 Thread bugzilla-daemon
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

[Mesa-dev] [Bug 100960] Special block from Minecraft mod rendered out of place

2018-05-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100960 Timothy Arceri changed: What|Removed |Added Status|NEW |NEEDINFO ---

[Mesa-dev] [Bug 106445] No Threading

2018-05-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106445 zacharybin...@hotmail.com changed: What|Removed |Added Resolution|INVALID |---

[Mesa-dev] [Bug 106445] No Threading

2018-05-09 Thread bugzilla-daemon
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

[Mesa-dev] [PATCH 2/2] i965: enable OES_texture_view for gen8+

2018-05-09 Thread Tapani Pälli
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

[Mesa-dev] [PATCH 1/2] mesa: changes to expose OES_texture_view extension

2018-05-09 Thread Tapani Pälli
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 ---

[Mesa-dev] [Bug 91039] [d3d9] D3D9 state tracker crashes due to stack misalignment if Mesa is compiled with -mfpmath=sse

2018-05-09 Thread bugzilla-daemon
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: