[Mesa-dev] [Bug 99055] Games hang / freeze completely

2016-12-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99055 --- Comment #13 from Michel Dänzer --- Note that it's not really necessary to remove anything from the Steam directory; one can force using the system libraries via the LD_PRELOAD environment variable, e.g.

Re: [Mesa-dev] [PATCH] Mesa: Return GL error INVALID_OPERATION in case invalid format/type

2016-12-13 Thread Xu, Randy
Thanks, Ilia Yes, they are similar and just for different API. commit 19f13b2096a9fdd986e5a12d4e9d8b0d6a4632f3 Author: Kenneth Graunke Date: Mon Mar 7 16:43:35 2016 -0800 mesa: Fix error code for GetFramebufferAttachmentParameter in ES 3.0+. Thanks, Randy

Re: [Mesa-dev] [PATCH] Mesa: Return GL error INVALID_OPERATION in case invalid format/type

2016-12-13 Thread Xu, Randy
Thanks, Matt I just create a freedesktop issue Bug 99076 - dEQP-GLES3.functional.negative_api.texture#teximage3d fails due to wrong Error code I distinguish GLES an GL as there is below comments in code, the _mesa_error_check_format_and_type function is not only used by glTexImage3D. /*

[Mesa-dev] [Bug 99076] dEQP-GLES3.functional.negative_api.texture#teximage3d fails due to wrong Error code

2016-12-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99076 Bug ID: 99076 Summary: dEQP-GLES3.functional.negative_api.texture#teximage3d fails due to wrong Error code Product: Mesa Version: git Hardware: x86-64 (AMD64)

[Mesa-dev] [Bug 99076] dEQP-GLES3.functional.negative_api.texture#teximage3d fails due to wrong Error code

2016-12-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99076 Randy changed: What|Removed |Added Blocks||94448 Referenced Bugs:

Re: [Mesa-dev] [PATCH] Mesa: Return GL error INVALID_OPERATION in case invalid format/type

2016-12-13 Thread Ilia Mirkin
On Tue, Dec 13, 2016 at 8:56 PM, Matt Turner wrote: > On Tue, Dec 13, 2016 at 4:28 AM, Randy Xu wrote: >> From: "Xu,Randy" > > Please configure your name properly: > > git config --global user.name "Randy Xu" > >> >> Refer to GLES3.2

Re: [Mesa-dev] [PATCH] Mesa: Return GL error INVALID_OPERATION in case invalid format/type

2016-12-13 Thread Matt Turner
On Tue, Dec 13, 2016 at 4:28 AM, Randy Xu wrote: > From: "Xu,Randy" Please configure your name properly: git config --global user.name "Randy Xu" > > Refer to GLES3.2 spec in 8.5 > Textures with a base internal format of DEPTH_COMPONENT, DEPTH_- >

Re: [Mesa-dev] [Mesa-maintainers] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Igor Gnatenko
On Tue, Dec 13, 2016 at 2:29 PM, Emil Velikov wrote: > On 13 December 2016 at 11:08, Eric Engestrom > wrote: >> On Thursday, 2016-12-08 19:21:46 +, Emil Velikov wrote: >>> From: Emil Velikov >>> >>> Since day

[Mesa-dev] [PATCH] Mesa: Return GL error INVALID_OPERATION in case invalid format/type

2016-12-13 Thread Randy Xu
From: "Xu,Randy" Refer to GLES3.2 spec in 8.5 Textures with a base internal format of DEPTH_COMPONENT, DEPTH_- STENCIL or STENCIL_INDEX are supported by texture image specification commands only if target is TEXTURE_2D, TEXTURE_2D_MULTISAMPLE, TEXTURE_2D_ARRAY,

[Mesa-dev] [PATCH] Mesa: Return GL error INVALID_OPERATION in case invalid format/type

2016-12-13 Thread nuc
From: "Xu,Randy" Refer to GLES3.2 spec in 8.5 Textures with a base internal format of DEPTH_COMPONENT, DEPTH_- STENCIL or STENCIL_INDEX are supported by texture image specification commands only if target is TEXTURE_2D, TEXTURE_2D_MULTISAMPLE, TEXTURE_2D_ARRAY,

Re: [Mesa-dev] [PATCH v3 2/3] svga: Fix a strict-aliasing violation in shader dumper

2016-12-13 Thread Charmaine Lee
Change looks fine to me. Thanks. Reviewed-by: Charmaine Lee From: Roland Scheidegger Sent: Tuesday, December 13, 2016 10:38 AM To: Edward O'Callaghan; mesa-dev@lists.freedesktop.org Cc: Brian Paul; Charmaine Lee Subject: Re:

[Mesa-dev] [PATCH 84/84 v4] st/nine: Implement new buffer upload path

2016-12-13 Thread Axel Davy
This new buffer upload path enables to lock faster than the normal path when using DISCARD/NOOVERWRITE. v2: Diverse cleanups and fixes. v3: Fix allocation size for 'lone' buffers and add more debug info. v4: Rewrite of the path to handle when DISCARD/NOOVERWRITE is not used anymore. The resource

Re: [Mesa-dev] [PATCH] egl: Fix crashes in eglCreate*Surface()

2016-12-13 Thread Mark Janes
Reviewed-by: Mark Janes Chad Versace writes: > Don't dereference a null EGLDisplay. > > Fixes tests > dEQP-EGL.functional.negative_api.create_pbuffer_surface > dEQP-EGL.functional.negative_api.create_pixmap_surface > > Fixes:

Re: [Mesa-dev] Documenting with Sphinx

2016-12-13 Thread Rob Clark
On Tue, Dec 13, 2016 at 5:47 PM, Eric Anholt wrote: > Jason Ekstrand writes: > >> Hey All, >> I don't figure this will be terribly controversial (I'm about to be wrong, >> aren't I?) but how do people feel about switching our "primary" >> documentation

Re: [Mesa-dev] Documenting with Sphinx

2016-12-13 Thread Kenneth Graunke
On Tuesday, December 13, 2016 2:47:27 PM PST Eric Anholt wrote: > Jason Ekstrand writes: > > > Hey All, > > I don't figure this will be terribly controversial (I'm about to be wrong, > > aren't I?) but how do people feel about switching our "primary" > > documentation

Re: [Mesa-dev] Documenting with Sphinx

2016-12-13 Thread Jason Ekstrand
On Tue, Dec 13, 2016 at 2:47 PM, Eric Anholt wrote: > Jason Ekstrand writes: > > > Hey All, > > I don't figure this will be terribly controversial (I'm about to be > wrong, > > aren't I?) but how do people feel about switching our "primary" > >

[Mesa-dev] [PATCH] st/nine: Dirty MANAGED buffers at Lock time

2016-12-13 Thread Axel Davy
Tests suggest MANAGED buffers are made dirty at Lock time, not at Unlock time. Signed-off-by: Axel Davy --- Dirtying MANAGED buffers at Unlock was a requirement of our previous implementation, since we were uploading all dirty MANAGED buffers as soon as a draw call was made.

Re: [Mesa-dev] Documenting with Sphinx

2016-12-13 Thread Dylan Baker
Quoting Jason Ekstrand (2016-12-13 12:46:17) > Hey All, > I don't figure this will be terribly controversial (I'm about to be wrong, > aren't I?) but how do people feel about switching our "primary" documentation > focus, as far as we have one, to sphinx?  Right now, Gallium uses sphinx for >

[Mesa-dev] [PATCH v3 6/7] i965: Add support for xfb overflow query on conditional render.

2016-12-13 Thread Rafael Antognolli
Enable the use of a transform feedback overflow query with glBeginConditionalRender. The render commands will only execute if the query is true (i.e. if there was an overflow). Use ARB_conditional_render_inverted to change this behavior. Signed-off-by: Rafael Antognolli

[Mesa-dev] [PATCH v3 1/7] i965: Enable predicate support on gen >= 8.

2016-12-13 Thread Rafael Antognolli
Predication needs cmd parser only on gen7. For newer platforms, it should be available without it. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/intel_extensions.c | 1 + 1 file changed, 1 insertion(+) diff --git

[Mesa-dev] [PATCH v3 4/7] i965: add plumbing for ARB_transform_feedback_overflow_query.

2016-12-13 Thread Rafael Antognolli
When querying for transform feedback overflow on one or all of the streams, store information about number of generated and written primitives. Then check whether generated == written. v2: - use only SO_PRIM_STORAGE_NEEDED, do not fallback to CL_INVOCATION_COUNT. (Kenneth)

[Mesa-dev] [PATCH v3 5/7] i965: Add support for xfb overflow on query buffer objects.

2016-12-13 Thread Rafael Antognolli
Enable getting the results of a transform feedback overflow query with a buffer object. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/hsw_queryobj.c | 108 +++ 1 file changed, 108 insertions(+) diff --git

[Mesa-dev] [PATCH v3 7/7] i965/gen6+: Enable arb_transform_feedback_overflow_query.

2016-12-13 Thread Rafael Antognolli
This extension adds new query types which can be used to detect overflow of transform feedback buffers. The new query types are also accepted by conditional rendering commands. v3: - s/gen7+/gen6+/ in the relnotes (Jordan Justen) Signed-off-by: Rafael Antognolli

[Mesa-dev] [PATCH v3 3/7] mesa: Track transform feedback overflow query objects.

2016-12-13 Thread Rafael Antognolli
Also update checks on conditional rendering. Signed-off-by: Rafael Antognolli --- src/mesa/main/condrender.c | 4 +++- src/mesa/main/queryobj.c| 21 + src/mesa/state_tracker/st_cb_queryobj.c | 6 ++ 3 files

[Mesa-dev] [PATCH v3 2/7] mesa: Add types for ARB_transform_feedback_oveflow_query.

2016-12-13 Thread Rafael Antognolli
Add some basic types and storage for the queries of this extension. v2: - update date of extension (Kenneth) Signed-off-by: Rafael Antognolli --- src/mesa/main/extensions_table.h | 1 + src/mesa/main/mtypes.h | 5 + 2 files changed, 6

[Mesa-dev] [PATCH v3 0/7] Add support for ARB_transform_feedback_overflow_query.

2016-12-13 Thread Rafael Antognolli
This patch series implements the ARB_transform_feedback_overflow_query extension for i965. The series is available on github here: https://github.com/rantogno/mesa/tree/review/overflow_query-v03 There are also piglit tests available for it here:

[Mesa-dev] [PATCH] st/nine: SYSTEMMEM ignores DISCARD.

2016-12-13 Thread Axel Davy
Tests show SYSTEMMEM should ignore DISCARD. Prevents game bugs with following patches reimplementing DISCARD. Halo is affected. Signed-off-by: Axel Davy --- This patch prevents a regression on radeonsi for Halo (and probably other games) with "st/nine: Implement Fast path for

Re: [Mesa-dev] Documenting with Sphinx

2016-12-13 Thread Eric Anholt
Jason Ekstrand writes: > Hey All, > I don't figure this will be terribly controversial (I'm about to be wrong, > aren't I?) but how do people feel about switching our "primary" > documentation focus, as far as we have one, to sphinx? Right now, Gallium > uses sphinx for

Re: [Mesa-dev] [PATCH 4/6] nir: Add a local variable-based copy propagation pass

2016-12-13 Thread Jason Ekstrand
On Tue, Dec 13, 2016 at 9:43 AM, Eric Anholt wrote: > Jason Ekstrand writes: > > > diff --git a/src/compiler/nir/nir_opt_copy_prop_vars.c > b/src/compiler/nir/nir_opt_copy_prop_vars.c > > new file mode 100644 > > index 000..728e476 > > --- /dev/null >

[Mesa-dev] [PATCH] egl: Fix crashes in eglCreate*Surface()

2016-12-13 Thread Chad Versace
Don't dereference a null EGLDisplay. Fixes tests dEQP-EGL.functional.negative_api.create_pbuffer_surface dEQP-EGL.functional.negative_api.create_pixmap_surface Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=99038 --- src/egl/main/eglapi.c | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [Mesa-dev] [PATCH v2 000/103] i965 Haswell ARB_gpu_shader_fp64 / OpenGL 4.0

2016-12-13 Thread Francisco Jerez
Samuel Iglesias Gonsálvez writes: > On Tue, 2016-12-13 at 09:01 +0100, Samuel Iglesias Gonsálvez wrote: >> > [...] >> > i965/vec4/nir: implement double comparisons >> > >> >Trivial: A newline before the if() would be nice. >> > >> >I have a memory of Curro

Re: [Mesa-dev] [PATCH 1/9] i965/gen6+: Invalidate constant cache on brw_emit_mi_flush().

2016-12-13 Thread Kenneth Graunke
On Tuesday, December 13, 2016 1:10:22 PM PST Francisco Jerez wrote: > Kenneth Graunke writes: > > > On Friday, December 9, 2016 11:03:24 AM PST Francisco Jerez wrote: > >> In order to make sure that the constant cache is coherent with > >> previous rendering when we start

Re: [Mesa-dev] [PATCH 6/9] i965/fs: Fetch one cacheline of pull constants at a time.

2016-12-13 Thread Francisco Jerez
Kenneth Graunke writes: > On Friday, December 9, 2016 11:03:29 AM PST Francisco Jerez wrote: >> Asking the DC for less than one cacheline (4 owords) of data for >> uniform pull constants is suboptimal because the DC cannot request >> less than that from L3, resulting in

Re: [Mesa-dev] [PATCH 1/9] i965/gen6+: Invalidate constant cache on brw_emit_mi_flush().

2016-12-13 Thread Francisco Jerez
Kenneth Graunke writes: > On Friday, December 9, 2016 11:03:24 AM PST Francisco Jerez wrote: >> In order to make sure that the constant cache is coherent with >> previous rendering when we start using it for pull constant loads. >> --- >>

[Mesa-dev] Documenting with Sphinx

2016-12-13 Thread Jason Ekstrand
Hey All, I don't figure this will be terribly controversial (I'm about to be wrong, aren't I?) but how do people feel about switching our "primary" documentation focus, as far as we have one, to sphinx? Right now, Gallium uses sphinx for documenting all of it's "public" interfaces. A while ago,

Re: [Mesa-dev] [PATCH 1/2] i965: Make DCE set null destinations on messages with side effects.

2016-12-13 Thread Francisco Jerez
Kenneth Graunke writes: > (Co-authored by Matt Turner.) > > Image atomics, for example, return a value - but the shader may not > want to use it. We assigned a useless VGRF destination. This seemed > harmless, but it can actually be quite harmful. The register allocator

Re: [Mesa-dev] [Mesa-announce] Mesa 12.0.5 release candidate

2016-12-13 Thread Marek Olšák
On Tue, Dec 13, 2016 at 7:10 PM, Emil Velikov wrote: > On 6 December 2016 at 13:27, Emil Velikov wrote: > >>> Can we do another release that includes the CSO patch and the other 2 >>> patches? >>> >> If we get at least a couple other [driver]

Re: [Mesa-dev] [PATCH mesa] egl/x11: cleanup init code

2016-12-13 Thread Eric Engestrom
On Thursday, 2016-12-08 20:17:21 +, Emil Velikov wrote: > On 8 December 2016 at 00:30, Eric Engestrom wrote: > > No functional change, just rewriting it in an easier-to-understand way. > > > I might be mislead by the diff, but it seems that there's functionality > change.

Re: [Mesa-dev] [PATCH v3 2/3] svga: Fix a strict-aliasing violation in shader dumper

2016-12-13 Thread Roland Scheidegger
adding CC: Charmaine Am 12.12.2016 um 06:23 schrieb Edward O'Callaghan: > Brian/Roland ping? > > On 12/07/2016 10:30 AM, Edward O'Callaghan wrote: >> As per the C spec, it is illegal to alias pointers to different >> types. This results in undefined behaviour after optimization >> passes,

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Ilia Mirkin
On Tue, Dec 13, 2016 at 1:26 PM, Emil Velikov wrote: > On 13 December 2016 at 18:24, Ilia Mirkin wrote: >> On Tue, Dec 13, 2016 at 1:19 PM, Emil Velikov >> wrote: >>> On 13 December 2016 at 18:06, Ilia Mirkin

Re: [Mesa-dev] [PATCH 17/28] auxiliary/vl: attempt to fix X11-less builds

2016-12-13 Thread Nayan Deshmukh
On Fri, Dec 9, 2016 at 1:30 AM, Emil Velikov wrote: > From: Emil Velikov > > Use the proper guard around the include and use void * instead of > Display *. > Some part of the patch is missing. Regards, Nayan > Signed-off-by: Emil Velikov

Re: [Mesa-dev] [PATCH 24/28] configure: error out if building OMX w/o supported platform

2016-12-13 Thread Nayan Deshmukh
Reviewed-by: Nayan Deshmukh On Tue, Dec 13, 2016 at 11:58 PM, Nayan Deshmukh wrote: > Reviewed-by: Nayan Deshmukh > > On Fri, Dec 9, 2016 at 1:30 AM, Emil Velikov wrote: >> From: Emil

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Emil Velikov
On 13 December 2016 at 18:24, Ilia Mirkin wrote: > On Tue, Dec 13, 2016 at 1:19 PM, Emil Velikov > wrote: >> On 13 December 2016 at 18:06, Ilia Mirkin wrote: >>> On Tue, Dec 13, 2016 at 12:33 PM, Emil Velikov

Re: [Mesa-dev] [PATCH 26/28] configure: error out if building XVMC w/o supported platform

2016-12-13 Thread Nayan Deshmukh
Half of this patch went in 25, with that fixed both are Reviewed-by: Nayan Deshmukh On Fri, Dec 9, 2016 at 1:30 AM, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Ilia Mirkin
On Tue, Dec 13, 2016 at 1:19 PM, Emil Velikov wrote: > On 13 December 2016 at 18:06, Ilia Mirkin wrote: >> On Tue, Dec 13, 2016 at 12:33 PM, Emil Velikov >> wrote: >>> A workaround which just make to mind: >>> - keep

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Emil Velikov
On 13 December 2016 at 18:06, Ilia Mirkin wrote: > On Tue, Dec 13, 2016 at 12:33 PM, Emil Velikov > wrote: >> On 13 December 2016 at 16:35, Ilia Mirkin wrote: >>> On Tue, Dec 13, 2016 at 11:30 AM, Emil Velikov

Re: [Mesa-dev] [PATCH 25/28] configure: error out if building VDPAU w/o supported platform

2016-12-13 Thread Nayan Deshmukh
On Fri, Dec 9, 2016 at 1:30 AM, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov > --- > configure.ac | 17 - > 1 file changed, 16 insertions(+), 1 deletion(-) > > diff

Re: [Mesa-dev] [Mesa-announce] Mesa 12.0.5 release candidate

2016-12-13 Thread Emil Velikov
On 6 December 2016 at 13:27, Emil Velikov wrote: >> Can we do another release that includes the CSO patch and the other 2 >> patches? >> > If we get at least a couple other [driver] developers how are in > favour, sure thing. > My suggestion/proposal still stands. If

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Ilia Mirkin
On Tue, Dec 13, 2016 at 12:33 PM, Emil Velikov wrote: > On 13 December 2016 at 16:35, Ilia Mirkin wrote: >> On Tue, Dec 13, 2016 at 11:30 AM, Emil Velikov >> wrote: >>> On 13 December 2016 at 16:19, Ilia Mirkin

Re: [Mesa-dev] [PATCH 27/28] configure: trivial whitespace cleanup

2016-12-13 Thread Nayan Deshmukh
Reviewed-by: Nayan Deshmukh On Fri, Dec 9, 2016 at 1:30 AM, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov > --- > configure.ac | 6 +++--- > 1 file changed,

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/2] i965/mt: Disable aux surfaces after making miptree shareable

2016-12-13 Thread Anuj Phogat
On Fri, Dec 9, 2016 at 4:32 PM, Chad Versace wrote: > > The entire goal of intel_miptree_make_shareable() is to permanently > disable the miptree's aux surfaces. So set > intel_mipmap_tree:disable_aux_buffers after the function's done with > discarding down the aux

Re: [Mesa-dev] [PATCH 10/28] st/va: move variable declaration to where its used

2016-12-13 Thread Nayan Deshmukh
s/thinker/tinker. with that fixed Reviewed-by: Nayan Deshmukh On Fri, Dec 9, 2016 at 12:51 AM, Emil Velikov wrote: > From: Emil Velikov > > ... and make it const, since we shouldn't thinker with it. > >

Re: [Mesa-dev] [PATCH 4/6] nir: Add a local variable-based copy propagation pass

2016-12-13 Thread Eric Anholt
Jason Ekstrand writes: > diff --git a/src/compiler/nir/nir_opt_copy_prop_vars.c > b/src/compiler/nir/nir_opt_copy_prop_vars.c > new file mode 100644 > index 000..728e476 > --- /dev/null > +++ b/src/compiler/nir/nir_opt_copy_prop_vars.c > @@ -0,0 +1,799 @@ > +/* > + *

Re: [Mesa-dev] [PATCH 13/28] st/va: check if vl_*_screen_create has failed only once

2016-12-13 Thread Nayan Deshmukh
Reviewed-by: Nayan Deshmukh On Tue, Dec 13, 2016 at 11:04 PM, Nayan Deshmukh wrote: > Reviewed-by: Nayan Deshmukh > > > On Fri, Dec 9, 2016 at 1:30 AM, Emil Velikov wrote: >> From: Emil

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Emil Velikov
On 13 December 2016 at 16:35, Ilia Mirkin wrote: > On Tue, Dec 13, 2016 at 11:30 AM, Emil Velikov > wrote: >> On 13 December 2016 at 16:19, Ilia Mirkin wrote: >>> On Tue, Dec 13, 2016 at 11:14 AM, Emil Velikov

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Ilia Mirkin
On Tue, Dec 13, 2016 at 11:30 AM, Emil Velikov wrote: > On 13 December 2016 at 16:19, Ilia Mirkin wrote: >> On Tue, Dec 13, 2016 at 11:14 AM, Emil Velikov >> wrote: >>> On 13 December 2016 at 16:09, Ilia Mirkin

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Emil Velikov
On 13 December 2016 at 16:19, Ilia Mirkin wrote: > On Tue, Dec 13, 2016 at 11:14 AM, Emil Velikov > wrote: >> On 13 December 2016 at 16:09, Ilia Mirkin wrote: >>> On Tue, Dec 13, 2016 at 11:08 AM, Emil Velikov

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Ilia Mirkin
On Tue, Dec 13, 2016 at 11:14 AM, Emil Velikov wrote: > On 13 December 2016 at 16:09, Ilia Mirkin wrote: >> On Tue, Dec 13, 2016 at 11:08 AM, Emil Velikov >> wrote: >>> On 13 December 2016 at 15:25, Ilia Mirkin

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Emil Velikov
On 13 December 2016 at 16:09, Ilia Mirkin wrote: > On Tue, Dec 13, 2016 at 11:08 AM, Emil Velikov > wrote: >> On 13 December 2016 at 15:25, Ilia Mirkin wrote: >>> On Thu, Dec 8, 2016 at 2:21 PM, Emil Velikov

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Ilia Mirkin
On Tue, Dec 13, 2016 at 11:08 AM, Emil Velikov wrote: > On 13 December 2016 at 15:25, Ilia Mirkin wrote: >> On Thu, Dec 8, 2016 at 2:21 PM, Emil Velikov >> wrote: >>> From: Emil Velikov >>>

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Emil Velikov
On 13 December 2016 at 15:25, Ilia Mirkin wrote: > On Thu, Dec 8, 2016 at 2:21 PM, Emil Velikov wrote: >> From: Emil Velikov >> >> Since day 1, Vulkan has depended on --with-egl-platforms to select the >> platforms

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Ilia Mirkin
On Thu, Dec 8, 2016 at 2:21 PM, Emil Velikov wrote: > From: Emil Velikov > > Since day 1, Vulkan has depended on --with-egl-platforms to select the > platforms build. > > With earlier commits, we've attributed for that internally by renaming

Re: [Mesa-dev] [PATCH v3] main: use new driver flag for conservative rasterization state

2016-12-13 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Tue, Dec 13, 2016 at 2:47 PM, Lionel Landwerlin wrote: > Hi Marek, > > Can I have your r-b with this patch? > > Cheers, > > - > Lionel > > > On 12/12/16 12:30, Lionel Landwerlin wrote: >> >> Suggested by

Re: [Mesa-dev] [PATCH v3] main: use new driver flag for conservative rasterization state

2016-12-13 Thread Lionel Landwerlin
Hi Marek, Can I have your r-b with this patch? Cheers, - Lionel On 12/12/16 12:30, Lionel Landwerlin wrote: Suggested by Marek. v2: Use new driver flag (Marek) v3: Fix i965 comments (Lionel) Signed-off-by: Lionel Landwerlin Cc: Ilia Mirkin

Re: [Mesa-dev] [PATCH 09/28] XXX:configure: set HAVE_PLATFORM_foo as applicable

2016-12-13 Thread Emil Velikov
On 13 December 2016 at 11:14, Eric Engestrom wrote: > On Thursday, 2016-12-08 19:21:49 +, Emil Velikov wrote: >> From: Emil Velikov >> >> Signed-off-by: Emil Velikov >> --- >> Should we keep this here and

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Emil Velikov
On 13 December 2016 at 11:08, Eric Engestrom wrote: > On Thursday, 2016-12-08 19:21:46 +, Emil Velikov wrote: >> From: Emil Velikov >> >> Since day 1, Vulkan has depended on --with-egl-platforms to select the >> platforms build. >> >>

[Mesa-dev] [Bug 99055] Games hang / freeze completely

2016-12-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99055 Eero Tamminen changed: What|Removed |Added Status|REOPENED|RESOLVED

Re: [Mesa-dev] [PATCH] radv: drop unused fields in physical device.

2016-12-13 Thread Vedran Miletić
Nice cleanup. Reviewed-by: Vedran Miletić On 12/13/2016 04:58 AM, Dave Airlie wrote: > From: Dave Airlie > > Signed-off-by: Dave Airlie > --- > src/amd/vulkan/radv_private.h | 6 -- > 1 file changed, 6 deletions(-) > > diff

[Mesa-dev] [Bug 98279] [vulkan/radeon] dota2 -vulkan hangs the GPU on R9-390

2016-12-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98279 --- Comment #11 from Roman Serkov --- (In reply to Michel Dänzer from comment #9) > You may be running into a known issue with libxcb in the Steam runtime, see > bug 97174. I've always been running steam with

Re: [Mesa-dev] [PATCH v2 000/103] i965 Haswell ARB_gpu_shader_fp64 / OpenGL 4.0

2016-12-13 Thread Iago Toral
On Tue, 2016-12-13 at 13:22 +0100, Samuel Iglesias Gonsálvez wrote: > On Sun, 2016-12-11 at 15:00 -0800, Matt Turner wrote: > > > > i965/vec4: handle 32 and 64 bit channels in liveness analysis > > > > Please indent the returned multiline expressions in > > var_from_reg() like we do

Re: [Mesa-dev] [PATCH v2 000/103] i965 Haswell ARB_gpu_shader_fp64 / OpenGL 4.0

2016-12-13 Thread Samuel Iglesias Gonsálvez
On Sun, 2016-12-11 at 15:00 -0800, Matt Turner wrote: > i965/vec4: handle 32 and 64 bit channels in liveness analysis > > Please indent the returned multiline expressions in > var_from_reg() like we do elsewhere, so that the second line > begins on the same column as the first

[Mesa-dev] [Bug 98279] [vulkan/radeon] dota2 -vulkan hangs the GPU on R9-390

2016-12-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98279 --- Comment #10 from Roman Serkov --- After trying to run Dota2 with -vulkan on 4.9 release kernel, it now always hangs the gpu, before on rc2-7 i was sometimes able to alt-tab and kill the process, not sure how thats

Re: [Mesa-dev] [PATCH v2 000/103] i965 Haswell ARB_gpu_shader_fp64 / OpenGL 4.0

2016-12-13 Thread Samuel Iglesias Gonsálvez
On Tue, 2016-12-13 at 09:01 +0100, Samuel Iglesias Gonsálvez wrote: > [...] > > i965/vec4/nir: implement double comparisons > > > > Trivial: A newline before the if() would be nice. > > > > I have a memory of Curro telling me that the hardware maps each > > 32-bit chunk in the dst

Re: [Mesa-dev] [PATCH 09/28] XXX:configure: set HAVE_PLATFORM_foo as applicable

2016-12-13 Thread Eric Engestrom
On Thursday, 2016-12-08 19:21:49 +, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov > --- > Should we keep this here and feed the defines to all sources or apply > them locally ? > > If we go with this

Re: [Mesa-dev] [PATCH 06/28] configure: rename --with-{egl-, }platforms

2016-12-13 Thread Eric Engestrom
On Thursday, 2016-12-08 19:21:46 +, Emil Velikov wrote: > From: Emil Velikov > > Since day 1, Vulkan has depended on --with-egl-platforms to select the > platforms build. > > With earlier commits, we've attributed for that internally by renaming > the [internal]

Re: [Mesa-dev] [PATCH v2 2/7] nir/lower_tex: add lowering for texture gradient on cube maps

2016-12-13 Thread Iago Toral
On Mon, 2016-12-12 at 23:14 -0800, Kenneth Graunke wrote: > On Monday, December 12, 2016 2:11:43 PM PST Iago Toral Quiroga wrote: > > > > This is ported from the Intel lowering pass that we use with GLSL > > IR. > > The NIR pass only handles cube maps, not shadow samplers, which are > > also

Re: [Mesa-dev] [PATCH 6/9] i965/fs: Fetch one cacheline of pull constants at a time.

2016-12-13 Thread Kenneth Graunke
On Friday, December 9, 2016 11:03:29 AM PST Francisco Jerez wrote: > Asking the DC for less than one cacheline (4 owords) of data for > uniform pull constants is suboptimal because the DC cannot request > less than that from L3, resulting in wasted bandwidth and unnecessary > message dispatch

Re: [Mesa-dev] [PATCH 3/9] i965/fs: Switch to the constant cache for uniform pull constants.

2016-12-13 Thread Kenneth Graunke
On Friday, December 9, 2016 11:03:26 AM PST Francisco Jerez wrote: > -/* We have to use a message header on Skylake to get SIMD4x2 > - * mode. Reserve space for the register. > -*/ This should reduce the message length of the SENDs, right? It might be worth

Re: [Mesa-dev] [PATCH 2/9] i965: Let the caller of brw_set_dp_write/read_message control the target cache.

2016-12-13 Thread Kenneth Graunke
On Friday, December 9, 2016 11:03:25 AM PST Francisco Jerez wrote: > brw_set_dp_read_message already had a target_cache argument, but its > interpretation was rather convoluted (on Gen6 the render cache was > used if the caller asked for it, otherwise it was ignored using the > sampler cache

Re: [Mesa-dev] [PATCH v2 000/103] i965 Haswell ARB_gpu_shader_fp64 / OpenGL 4.0

2016-12-13 Thread Samuel Iglesias Gonsálvez
Hello Matt! On Mon, 2016-12-05 at 15:21 -0800, Matt Turner wrote: > On 10/11, Iago Toral Quiroga wrote: > > It's been some time since > > ... anyone has reviewed your patches. Sorry. :( > > I'm going to review from your rebased i965-fp64-gen7-scalar-vec4-rc2 > branch. There have probably been