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.
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
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.
/*
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)
https://bugs.freedesktop.org/show_bug.cgi?id=99076
Randy changed:
What|Removed |Added
Blocks||94448
Referenced Bugs:
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
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_-
>
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
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,
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,
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:
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
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:
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
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
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"
> >
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.
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
>
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
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
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)
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
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
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
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
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:
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
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
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
>
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
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
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
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
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.
>> ---
>>
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,
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
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]
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.
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,
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
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
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
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
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
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
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
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
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
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
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,
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
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.
>
>
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 @@
> +/*
> + *
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
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
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
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
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
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
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
>>>
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
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
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
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
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
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.
>>
>>
https://bugs.freedesktop.org/show_bug.cgi?id=99055
Eero Tamminen changed:
What|Removed |Added
Status|REOPENED|RESOLVED
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
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
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
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
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
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
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
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]
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
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
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
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
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
80 matches
Mail list logo