For the series:
Reviewed-by: Edward O'Callaghan
Thanks, the PBO stuff fixes a problem I had with my video stream
application also.
On 11/10/2016 02:01 AM, Nicolai Hähnle wrote:
> Hi,
>
> this is a series of assorted patches that together fix
> GL45-CTS.gtf32.GL3Tests.packed_pixels.* on radeons
I've pushed this one.
___
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=97879
Jani Kärkkäinen changed:
What|Removed |Added
CC||jani.karkkai...@gmail.com
--
You are
https://bugs.freedesktop.org/show_bug.cgi?id=97879
--- Comment #44 from Jani Kärkkäinen ---
I'm quite sure I'm getting this as well. I'm however on a HD6850, so that'd be
the r600 driver I believe?
Anything I can do to help pinpoint the culprit?
Some specs:
OpenGL renderer string: Gallium 0.4
https://bugs.freedesktop.org/show_bug.cgi?id=94979
Markus Wick changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=98691
Bug ID: 98691
Summary: egl.h:55: error: redefinition of typedef ‘EGLDisplay’
Product: Mesa
Version: 13.0
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
On Fri, Nov 11, 2016 at 10:13 AM, Emil Velikov
wrote:
> Hello list,
>
> The candidate for the Mesa 12.0.4 is now available. Currently we have:
> - 51 queued
> - 18 nominated (outstanding)
> - and 0 (self-)rejected patches
>
> As one can imagine huge hunk of the commits are fixes in the Vulkan
On 10 November 2016 at 20:42, Emil Velikov wrote:
> On 10 November 2016 at 20:18, Kyriazis, George
> wrote:
>> Emil,
>>
>> I've followed the approach of trying to localize the swr changes to where
>> they belong in order to avoid "polluting" upper-level files. If you're
>> talking about modify
On Fri, Nov 11, 2016 at 1:39 PM, Ilia Mirkin wrote:
> On Fri, Nov 11, 2016 at 10:23 AM, Jason Ekstrand
> wrote:
> > On Nov 11, 2016 6:16 AM, "Ilia Mirkin" wrote:
> >>
> >> I haven't checked spirv for the specific naming, but note that prim ID
> can
> >> be an output in gs.
> >
> > You're right.
On Thu, Nov 10, 2016 at 10:24 AM, Ben Widawsky wrote:
> On 16-11-10 10:20:13, Ben Widawsky wrote:
>>
>> From: Ben Widawsky
>>
>> Signed-off-by: Ben Widawsky
>> ---
>> include/pci_ids/i965_pci_ids.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/pci_ids/i965_pc
On Fri, Nov 11, 2016 at 8:10 PM, Emil Velikov wrote:
> On 11 November 2016 at 18:36, Laurent Carlier wrote:
>> Le vendredi 11 novembre 2016, 18:13:25 CET Emil Velikov a écrit :
>>> Hello list,
>>>
>>> The candidate for the Mesa 12.0.4 is now available. Currently we have:
>>> - 51 queued
>>> - 1
On Fri, Nov 11, 2016 at 10:35 PM, Daniel Scharrer wrote:
> On 2016-11-11 22:24, Bas Nieuwenhuizen wrote:
>> Maybe also add an assert in ac_add_function_attr that the attr is a
>> single flag? Something like
>>
>> assert(attr && util_is_power_of_two(attr));
>
> There is already a warning printed fo
On Fri, Nov 11, 2016 at 10:23 AM, Jason Ekstrand wrote:
> On Nov 11, 2016 6:16 AM, "Ilia Mirkin" wrote:
>>
>> I haven't checked spirv for the specific naming, but note that prim ID can
>> be an output in gs.
>
> You're right. I'm not 100% site how we handle that today. We probably just
> pass i
On 2016-11-11 22:24, Bas Nieuwenhuizen wrote:
> Maybe also add an assert in ac_add_function_attr that the attr is a
> single flag? Something like
>
> assert(attr && util_is_power_of_two(attr));
There is already a warning printed for unknown attributes in
ac_attr_to_llvm_attr / attr_to_str that ca
Maybe also add an assert in ac_add_function_attr that the attr is a
single flag? Something like
assert(attr && util_is_power_of_two(attr));
Either way, this patch is
Reviewed-by: Bas Nieuwenhuizen
On Fri, Nov 11, 2016 at 9:36 PM, Daniel Scharrer wrote:
> This fixes a NULL pointer dereference f
Reviewed-by: Bruce Cherniak
> On Nov 9, 2016, at 9:18 PM, Tim Rowley wrote:
>
> Highlights include more 8x2 render work, fix for alphatest, and
> archrast updates.
>
> Tim Rowley (14):
> swr: [rasterizer jitter] code style fix
> swr: [rasterizer core] 16-wide tile store nearly completed
>
This fixes a NULL pointer dereference for intrinsics with more than
one function attribute introduced in commit 2fdaf38.
The fix is ported from the lp_build_intrinsic changes in commit 8bdd52c.
---
I'm a bit unsure about the index change from 0 to -1, but the gallium code does
this as well and wit
https://bugs.freedesktop.org/show_bug.cgi?id=97422
Karol Herbst changed:
What|Removed |Added
Summary|function call to function |trying to call a number as
On Thu, Nov 10, 2016 at 10:28 AM, Ben Widawsky
wrote:
>
> From: Ben Widawsky
>
> Signed-off-by: Ben Widawsky
> ---
> intel/intel_chipset.h | 13 ++---
> 1 file changed, 10 insertions(+), 3 deletions(-)
>
> diff --git a/intel/intel_chipset.h b/intel/intel_chipset.h
> index 514f659..41fc0
https://bugs.freedesktop.org/show_bug.cgi?id=97422
--- Comment #3 from Karol Herbst ---
Created attachment 127920
--> https://bugs.freedesktop.org/attachment.cgi?id=127920&action=edit
more simple example
seems like this crashes mesa already
--
You are receiving this mail because:
You are the
On 11 November 2016 at 18:36, Laurent Carlier wrote:
> Le vendredi 11 novembre 2016, 18:13:25 CET Emil Velikov a écrit :
>> Hello list,
>>
>> The candidate for the Mesa 12.0.4 is now available. Currently we have:
>> - 51 queued
>> - 18 nominated (outstanding)
>> - and 0 (self-)rejected patches
Le vendredi 11 novembre 2016, 18:13:25 CET Emil Velikov a écrit :
> Hello list,
>
> The candidate for the Mesa 12.0.4 is now available. Currently we have:
> - 51 queued
> - 18 nominated (outstanding)
> - and 0 (self-)rejected patches
>
> As one can imagine huge hunk of the commits are fixes in
On Fri, Nov 11, 2016 at 10:40 AM, Lionel Landwerlin
wrote:
> Signed-off-by: Lionel Landwerlin
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98480
> ---
> src/compiler/glsl/glsl_parser.yy | 5 +--
> src/compiler/glsl/glsl_parser_extras.cpp | 58
> +
https://bugs.freedesktop.org/show_bug.cgi?id=97879
--- Comment #43 from Steven ---
I also experience the issue. If someone can explain or point me to how I can
profile it I can also try to provide some api traces.
I'm running
Mesa 12.0.3
OpenGL renderer string: Gallium 0.4 on AMD HAWAII (DRM 2
Hello list,
The candidate for the Mesa 12.0.4 is now available. Currently we have:
- 51 queued
- 18 nominated (outstanding)
- and 0 (self-)rejected patches
As one can imagine huge hunk of the commits are fixes in the Vulkan drivers.
Up-to 30% performance increase using the Intel ANV Vulkan dr
On 11 November 2016 at 17:19, Daniel Stone wrote:
> Hi,
>
> On 11 November 2016 at 16:45, Emil Velikov wrote:
>> @@ -174,14 +172,24 @@ dri2_wl_create_surface(_EGLDriver *drv, _EGLDisplay
>> *disp,
>> config = dri2_get_dri_config(dri2_conf, EGL_WINDOW_BIT,
>>
https://bugs.freedesktop.org/show_bug.cgi?id=98681
Brian Paul changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
On 11 November 2016 at 17:12, Daniel Stone wrote:
> Hi,
>
> On 11 November 2016 at 16:44, Emil Velikov wrote:
>> As described in commit 690ead4a135 ("egl/wayland-egl: Fix for segfault
>> in dri2_wl_destroy_surface.") if we attempt to destroy a EGL surface
>> attached to already destroyed Wayland
Hi,
On 11 November 2016 at 16:27, Emil Velikov wrote:
> The function returns "void" rather than int. We could rework that, yet
> again there will be no benefit since all the callers have no use of it.
>
> Fixes: 9ca6711faa0 ("Revert "wayland: Block for the frame callback in
> get_back_bo not dri2
Hi,
On 11 November 2016 at 16:45, Emil Velikov wrote:
> @@ -174,14 +172,24 @@ dri2_wl_create_surface(_EGLDriver *drv, _EGLDisplay
> *disp,
> config = dri2_get_dri_config(dri2_conf, EGL_WINDOW_BIT,
> dri2_surf->base.GLColorspace);
>
> - dri2_surf->dri_drawab
Hi,
On 11 November 2016 at 16:44, Emil Velikov wrote:
> As described in commit 690ead4a135 ("egl/wayland-egl: Fix for segfault
> in dri2_wl_destroy_surface.") if we attempt to destroy a EGL surface
> attached to already destroyed Wayland window we'll get a segfault.
s/destroy/resize/ ... ? The p
On 11/11/16 16:18, Brian Paul wrote:
Otherwise, inttypes.h may not define the macro for C++ on MinGW.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98681
---
src/compiler/glsl/ir_builder_print_visitor.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/compiler/glsl/ir_builder_
On 11/11/2016 09:43 AM, Emil Velikov wrote:
From: Emil Velikov
Signed-off-by: Emil Velikov
---
src/mesa/main/context.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index 80bee16..bd4551e 100644
--- a/src/mesa/mai
On 11/11/2016 09:43 AM, Emil Velikov wrote:
From: Emil Velikov
As seen a couple of lines above - there's no way for the assert to
trigger.
Signed-off-by: Emil Velikov
---
src/mesa/main/framebuffer.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/main/framebuffer.c b/src/mesa/
From: Emil Velikov
Rather than having two almost identical codepaths (one for HW/wl_drm and
another for SW/wl_shm), just factorise and reuse in both places.
Signed-off-by: Emil Velikov
---
Another fine example of useless duplication that we have throughout ;-\
---
src/egl/drivers/dri2/platform
From: Emil Velikov
As described in commit 690ead4a135 ("egl/wayland-egl: Fix for segfault
in dri2_wl_destroy_surface.") if we attempt to destroy a EGL surface
attached to already destroyed Wayland window we'll get a segfault.
Cc: "12.0 13.0"
Signed-off-by: Emil Velikov
---
src/egl/drivers/dri
From: Emil Velikov
If we fail to create the drawable there's not much point in attampting
to destroy it.
Signed-off-by: Emil Velikov
---
src/egl/drivers/dri2/platform_wayland.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/egl/drivers/dri2/platform_wayland.c
b/src
From: Emil Velikov
As seen a couple of lines above - there's no way for the assert to
trigger.
Signed-off-by: Emil Velikov
---
src/mesa/main/framebuffer.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/main/framebuffer.c b/src/mesa/main/framebuffer.c
index 9c7f0c9..0a4cbef 100644
From: Emil Velikov
Signed-off-by: Emil Velikov
---
src/mesa/main/context.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index 80bee16..bd4551e 100644
--- a/src/mesa/main/context.c
+++ b/src/mesa/main/context.c
@@ -16
From: Emil Velikov
Identical throughout dri2, dri3 and drisw. Next patch will add more
common code, so rather than duplicating it factor out the function.
Note: this also sets eglError on failure. Something that's quite
inconsistent throughout the codebase.
Signed-off-by: Emil Velikov
---
src
From: Emil Velikov
Just fetch and store it once, rather than doing the
xcb_setup_roots_iterator + get_xcb_screen dance five times.
Signed-off-by: Emil Velikov
---
These two patches are an example of the duplication we have within each
loader (be that egl, gbm or glx)_and how we can fold it up
From: Emil Velikov
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/vl/vl_winsys_dri.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri.c
b/src/gallium/auxiliary/vl/vl_winsys_dri.c
index 9ecc216..0b2c939 100644
--- a/src/ga
From: Emil Velikov
The function returns "void" rather than int. We could rework that, yet
again there will be no benefit since all the callers have no use of it.
Fixes: 9ca6711faa0 ("Revert "wayland: Block for the frame callback in
get_back_bo not dri2_swap_buffers"")
Cc: Daniel Stone
Signed-of
On 11 November 2016 at 16:18, Brian Paul wrote:
> Otherwise, inttypes.h may not define the macro for C++ on MinGW.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98681
> ---
> src/compiler/glsl/ir_builder_print_visitor.cpp | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/c
From: Boyuan Zhang
The gop_size in rate control is the budget window for internal rate
control calculation, and shouldn't always equal to idr period. Define
a coefficient to let budget window contains a number of idr period for
proper rate control calculation. Adjust the number of i/p frame rema
From: Boyuan Zhang
When using dual instance and rate control, driver needs to submit jobs either
in dual submissions or 2 consecutive single submissions to keep the pattern
constant for rate control
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=98005
Signed-off-by: Boyuan Zhang
---
src/
Otherwise, inttypes.h may not define the macro for C++ on MinGW.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98681
---
src/compiler/glsl/ir_builder_print_visitor.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/compiler/glsl/ir_builder_print_visitor.cpp
b/src/compiler/glsl/
On 11/11/2016 09:03 AM, Jose Fonseca wrote:
On 11/11/16 15:48, Brian Paul wrote:
inttypes.h doesn't always define the PRIx64 macro when compiling C++
code with MinGW. For that case, simply define PRIx64 to be "lx".
This generates a format/argument type mismatch warning. And using
"llx" generat
On 11/11/16 15:48, Brian Paul wrote:
inttypes.h doesn't always define the PRIx64 macro when compiling C++
code with MinGW. For that case, simply define PRIx64 to be "lx".
This generates a format/argument type mismatch warning. And using
"llx" generates other warnings. Just live with it.
https
On Fri, Nov 11, 2016 at 02:00:26PM +0100, Laurent Carlier wrote:
Reviewed-by: Tom Stellard
> pretty trivial fix
> ---
> src/gallium/state_trackers/clover/llvm/codegen/bitcode.cpp | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/src/gallium/state_trackers/clover/llvm/codegen/bitcod
inttypes.h doesn't always define the PRIx64 macro when compiling C++
code with MinGW. For that case, simply define PRIx64 to be "lx".
This generates a format/argument type mismatch warning. And using
"llx" generates other warnings. Just live with it.
https://bugs.freedesktop.org/show_bug.cgi?id
Hi,
There is some interest in having broader image format support in GLES 3.1
(mostly for R8/RG8). Ilia pointed out that NV_image_formats could be the
extension we need to enable this. So here are a couple of patches to add the
extension to Mesa and enable it on the i965 driver.
An associated pig
Signed-off-by: Lionel Landwerlin
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98480
---
src/mesa/drivers/dri/i965/intel_extensions.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 6
Signed-off-by: Lionel Landwerlin
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98480
---
src/compiler/glsl/glsl_parser.yy | 5 +--
src/compiler/glsl/glsl_parser_extras.cpp | 58
src/compiler/glsl/glsl_parser_extras.h | 4 +++
src/mesa/main/ext
https://bugs.freedesktop.org/show_bug.cgi?id=98681
--- Comment #5 from Brian Paul ---
(In reply to Roland Scheidegger from comment #4)
> (In reply to Brian Paul from comment #3)
> > Actually, this is still failing with our Jenkins build. Hmm...
>
> I've got some feeling this is due this being i
On Nov 11, 2016 6:16 AM, "Ilia Mirkin" wrote:
>
> I haven't checked spirv for the specific naming, but note that prim ID
can be an output in gs.
You're right. I'm not 100% site how we handle that today. We probably
just pass it through as a varying. It looks like the code here needs to be
a bi
https://bugs.freedesktop.org/show_bug.cgi?id=98681
--- Comment #4 from Roland Scheidegger ---
(In reply to Brian Paul from comment #3)
> Actually, this is still failing with our Jenkins build. Hmm...
I've got some feeling this is due this being in c++ code?
--
You are receiving this mail beca
For the series:
Reviewed-by: Marek Olšák
Marek
On Wed, Nov 9, 2016 at 4:01 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> I'm also sending out a piglit test, gl-2.0/vertexattribpointer-size-3,
> which exposes this corner case.
> ---
> src/gallium/drivers/radeonsi/si_descriptors.c | 26
https://bugs.freedesktop.org/show_bug.cgi?id=98681
Brian Paul changed:
What|Removed |Added
Resolution|FIXED |---
Status|RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=98681
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
I haven't checked spirv for the specific naming, but note that prim ID can
be an output in gs.
On Nov 11, 2016 1:34 AM, "Jason Ekstrand" wrote:
> Before, we were treating it as an output which is just completely bogus.
> We could go for an input but a system value makes more sense.
>
> Cc: "13.0
pretty trivial fix
---
src/gallium/state_trackers/clover/llvm/codegen/bitcode.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/state_trackers/clover/llvm/codegen/bitcode.cpp
b/src/gallium/state_trackers/clover/llvm/codegen/bitcode.cpp
index 108f8d5..8e89a49 100644
--- a/sr
https://bugs.freedesktop.org/show_bug.cgi?id=97230
--- Comment #11 from Eero Tamminen ---
(In reply to f.mach4 from comment #8)
> Created attachment 127912 [details]
> GDB full trace when MATLAB has hanged
Ok, so everything is in pthread_cond_wait(), except couple of threads that:
* wait accepti
https://bugs.freedesktop.org/show_bug.cgi?id=98681
Vinson Lee changed:
What|Removed |Added
CC||i...@freedesktop.org,
|
Do not evaluate spill costs for registers that were already marked as
no_spill.
---
src/mesa/drivers/dri/i965/brw_vec4_reg_allocate.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_reg_allocate.cpp
b/src/mesa/drivers/dri/i965/brw_vec4_
https://bugs.freedesktop.org/show_bug.cgi?id=98681
Bug ID: 98681
Summary: ir_builder_print_visitor.cpp:401:67: error: expected
')' before 'PRIx64'
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linu
66 matches
Mail list logo
