These two GLES 3.2 entry points were being defined in the category of
the ARB_ES3_2_compatibility and KHR_blend_equation_advanced extensions
respectively instead of in the ES3.2 category. Defining them in the
ES3.2 category makes sure that the gl_procs.py generator emits
declarations in the
This reverts commit 85e9bbc14d93fa7166c9ae075ee7ae29a8313e3f. The
previous commit should help with the scons build failure caused by the
original commit.
---
src/mapi/glapi/gen/static_data.py | 12
1 file changed, 12 insertions(+)
diff --git a/src/mapi/glapi/gen/static_data.py
The state tracker tries to attach the info to the wrong shader. This is
easy enough to protect against.
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/nv50/nv50_program.c | 3 +++
src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 7 +--
2 files changed, 8
From: Dave Airlie
I can't see this being used anywhere.
Signed-off-by: Dave Airlie
---
src/intel/vulkan/anv_private.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h
index
https://bugs.freedesktop.org/show_bug.cgi?id=98172
--- Comment #28 from Suzuki, Shinji ---
Yes. I agree with you that we can do without per-sync-object if we
allow all waiters enter fence_finish() freely.
With that said, per-sync-object mutex has another benefit of
Ilia Mirkin writes:
> Why does it care where those functions are defined? I thought it was
> all one big happy namespace, with the categories just there for
> general amusement. Could you shed some light on what the actual
> situation is?
>
Heh, I won't pretend to
With ARB_gpu_shader5, texture offsets can be any source, including TEMPs
and IN's. Make sure to process them as regular sources so that we pick
up masks, etc.
This should fix some CTS tests that feed offsets directly to
textureGatherOffset, and we were not picking up the input use, thus not
Why does it care where those functions are defined? I thought it was
all one big happy namespace, with the categories just there for
general amusement. Could you shed some light on what the actual
situation is?
On Tue, Oct 18, 2016 at 11:48 PM, Francisco Jerez wrote:
>
rb
On Tue, Oct 18, 2016 at 8:36 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> I can't see this being used anywhere.
>
> Signed-off-by: Dave Airlie
> ---
> src/intel/vulkan/anv_private.h | 2 --
> 1 file changed, 2 deletions(-)
>
>
https://bugs.freedesktop.org/show_bug.cgi?id=98172
--- Comment #24 from Marek Olšák ---
Hm. Probably none. The patch looks good.
BTW, the patch won't apply on master, because the fence_finish function is
slightly different there.
--
You are receiving this mail because:
You
On 11.10.2016 20:31, Emil Velikov wrote:
From: Emil Velikov
Currently mesa has three code paths in the loader - libudev, manual
sysfs and drm ioctl one.
Considering the issues we had with libudev - strip those down in favour
of the libdrm drm device API. The latter
Both are:
Reviewed-by: Nicolai Hähnle
On 17.10.2016 22:03, Marek Olšák wrote:
From: Marek Olšák
---
si-report.py | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/si-report.py b/si-report.py
index 144a229..620b3bd
Reviewed-by: Nicolai Hähnle
On 17.10.2016 15:40, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_util.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_util.h
On 17.10.2016 15:39, Marek Olšák wrote:
From: Marek Olšák
I noticed that glsl_to_tgsi_instruction is too huge.
sizeof(glsl_to_tgsi_instruction): 752 -> 464 (-38%)
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 71 +-
1 file changed, 40
https://bugs.freedesktop.org/show_bug.cgi?id=98271
--- Comment #14 from John ---
> John, please double check that you are actually correctly installing VDPAU.
> E.g. add something like "while(1);" into the VDPAU driver create function or
> something like this.
https://bugs.freedesktop.org/show_bug.cgi?id=98275
--- Comment #4 from Andy Furniss ---
(In reply to lesserbrute from comment #0)
> libva info: Trying to open /usr/lib/x86_64-linux-gnu/dri/gallium_drv_video.so
If obiaf uses recent mesa then the driver name was changed to
https://bugs.freedesktop.org/show_bug.cgi?id=98172
--- Comment #25 from Suzuki, Shinji ---
Thank you for your comments.
Patches attached here by me are against 11.2.
I'll base on maser when posting diff to the mailing list.
--
You are receiving this mail because:
You
On 14.10.2016 20:21, Emil Velikov wrote:
From: Emil Velikov
Currently not everyone has libudev and with follow-up patches we'll
completely remove the divergent codepaths.
Use the libdrm drm device API to construct the required ID_PATH_TAG-like
string, to preserve
On 10/18/2016 06:22 AM, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
Cc: mesa-sta...@lists.freedesktop.org
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On 17.10.2016 15:39, Marek Olšák wrote:
From: Marek Olšák
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 31 ++
1 file changed, 19 insertions(+), 12 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
Hi Christian,
I pushed it before already. I can try to add a git notes if you want.
In any case thx for the review.
Cheers
Julien
On 18 October 2016 at 10:09, Christian König
wrote:
> Am 17.10.2016 um 21:40 schrieb Julien Isorce:
>
>
>
> On Monday, 17 October 2016,
On Tue, 2016-10-18 at 00:08 -0700, Kenneth Graunke wrote:
> On Tuesday, October 18, 2016 5:12:16 PM PDT Timothy Arceri wrote:
> >
> > These need to be bitfields for use with gallium.
> > ---
> > src/compiler/glsl/glsl_to_nir.cpp | 2 +-
> > src/compiler/shader_info.h| 12 +---
>
boyuan.zh...@amd.com wrote:
From: Boyuan Zhang
cabac is only supported in the h264 main and higher profiles
So shouldn't there be code allows it if the user space doesn't
set baseline?
I don't know how in gstreamer as it seems to try to use b-frames
if you use other
On 10/18/2016 10:50 AM, Samuel Pitoiset wrote:
On 10/18/2016 05:53 AM, Ilia Mirkin wrote:
The indirect handle has to come right after the coordinates, so if there
was a sample/bias/depth compare/offset, everything would end up being
shifted by one argument position.
Signed-off-by: Ilia
On 10/18/2016 12:33 PM, Samuel Pitoiset wrote:
On 10/18/2016 10:50 AM, Samuel Pitoiset wrote:
On 10/18/2016 05:53 AM, Ilia Mirkin wrote:
The indirect handle has to come right after the coordinates, so if there
was a sample/bias/depth compare/offset, everything would end up being
shifted
An assert is currently raised, preventing decompression of a texture image into
a GL_TEXTURE_3D target. I have not found any spec wording that would explain
this, or implementation detail that would prevent it. And in any case, the
driver should not cause a crash upon user input arguments.
Fixes
Reviewed-by: Nicolai Hähnle
On 17.10.2016 19:50, Marek Olšák wrote:
From: Marek Olšák
v2: remove goto, cosmetic changes
Tested-by: Edmondo Tommasina (v1)
---
src/util/ralloc.c | 353
Patches 5-12, 14 are
Reviewed-by: Nicolai Hähnle
Those inline-explosion tests have been bugging me as well, but
realistically speaking, the compiler needs to be at least robust enough
to just not blow up and instead refuse compilation before any work on
proper
Am 18.10.2016 um 11:19 schrieb Andy Furniss:
boyuan.zh...@amd.com wrote:
From: Boyuan Zhang
cabac is only supported in the h264 main and higher profiles
So shouldn't there be code allows it if the user space doesn't
set baseline?
I don't know how in gstreamer as it
Christian König wrote:
Am 18.10.2016 um 11:19 schrieb Andy Furniss:
boyuan.zh...@amd.com wrote:
From: Boyuan Zhang
cabac is only supported in the h264 main and higher profiles
So shouldn't there be code allows it if the user space doesn't
set baseline?
I don't know
On 11.10.2016 20:31, Emil Velikov wrote:
Hi all,
I've been 'threatening' to do this for a while, so here it is:
Remove the three separate code paths that we have in the loader giving
is a since cleanup and consistent/predictable runtime behaviour across
the board. Not to mention some 100 lines
https://bugs.freedesktop.org/show_bug.cgi?id=98271
--- Comment #13 from Christian König ---
John, please double check that you are actually correctly installing VDPAU.
E.g. add something like "while(1);" into the VDPAU driver create function or
something like this.
Am 17.10.2016 um 21:40 schrieb Julien Isorce:
On Monday, 17 October 2016, Mark Thompson > wrote:
On 17/10/16 17:33, Julien Isorce wrote:
> As specified in va.h, default value should be set on attributes
> not present in the input list.
Hi,
On 18.10.2016 01:07, Jan Ziak wrote:
This patch replaces the ir_variable_refcount_entry's linked-list
with an array-list.
The array-list has local storage which does not require ANY additional
allocations if the list has small number of elements. The size of this
storage is configurable
Makes sense as a cleanup. At some point it would make sense to look into
sharing some stuff with radv instead. There's probably not a huge amount
because of the NIR/TGSI split, but still.
Patches 1 & 4:
Acked-by: Nicolai Hähnle
Patches 2, 3, 5, 6:
Reviewed-by:
https://bugs.freedesktop.org/show_bug.cgi?id=98275
--- Comment #5 from Andy Furniss ---
Another random observation = If you are only concerned with
decode/display/deinterlace it's currently best to use vdpau anyway.
There are bugs with vaapi that don't affect vdpau.
--
After the changes in comit 5b2675093e863a52, we moved this check to the
linker, but the spec expects this to be checked at compile-time. There are
dEQP tests that expect an error at compile time and the spec seems to confirm
that expectation:
"Except for the last declared member of a shader
Hi Michael,
thanks for the suggestions about code formatting. I formatted the
whole fast_list.h file in the Netbeans editor and uploaded a new
revision of the patch.
Jan
On Tue, Oct 18, 2016 at 1:42 PM, Michael Schellenberger Costa
wrote:
> Hi Jan,
>
> On
Andy Furniss wrote:
Christian König wrote:
Am 18.10.2016 um 11:19 schrieb Andy Furniss:
boyuan.zh...@amd.com wrote:
From: Boyuan Zhang
cabac is only supported in the h264 main and higher profiles
So shouldn't there be code allows it if the user space doesn't
set
On Tue, Oct 18, 2016 at 3:12 PM, Nicolai Hähnle wrote:
> On 18.10.2016 15:07, Jan Ziak wrote:
>>
>> On Tue Oct 18 09:29:59 UTC 2016, Eero Tamminen wrote:
>>>
>>> On 18.10.2016 01:07, Jan Ziak wrote:
- The total number of executed instructions goes down from 64.184 to
On 18/10/16 02:38, srol...@vmware.com wrote:
From: Roland Scheidegger
For the texturing packs, things looked pretty terrible. For every
lerp, we were repacking the values, and while those look sort of cheap
with 128bit, with 256bit we end up with 2 of them instead of just 1
This patch replaces the ir_variable_refcount_entry's linked-list
with an array-list.
The array-list has local storage which does not require ANY additional
allocations if the list has small number of elements. The size of this
storage is configurable for each variable.
Benchmark results for
On 17 October 2016 at 23:07, Jan Ziak <0xe2.0x9a.0...@gmail.com> wrote:
> - exec_list assign_list;
> + arraylist assign_list;
>
Just an FYI - when people started using C++ for glsl there was an
agreement that templating is not to be used.
Not sure if it still holds true,
Hi,
On 18.10.2016 16:25, Jan Ziak wrote:
On Tue, Oct 18, 2016 at 3:12 PM, Nicolai Hähnle wrote:
On 18.10.2016 15:07, Jan Ziak wrote:
On Tue Oct 18 09:29:59 UTC 2016, Eero Tamminen wrote:
On 18.10.2016 01:07, Jan Ziak wrote:
- The total number of executed instructions
On Tue Oct 18 09:29:59 UTC 2016, Eero Tamminen wrote:
> On 18.10.2016 01:07, Jan Ziak wrote:
>> - The total number of executed instructions goes down from 64.184 to 63.797
>> giga-instructions when Mesa is compiled with "gcc -O0 ..."
>
>Please don't do performance related decisions based on data
On Tue, Oct 18, 2016 at 3:32 PM, Emil Velikov wrote:
> On 17 October 2016 at 23:07, Jan Ziak <0xe2.0x9a.0...@gmail.com> wrote:
>
>> - exec_list assign_list;
>> + arraylist assign_list;
>>
> Just an FYI - when people started using C++ for glsl there
On 18.10.2016 15:07, Jan Ziak wrote:
On Tue Oct 18 09:29:59 UTC 2016, Eero Tamminen wrote:
On 18.10.2016 01:07, Jan Ziak wrote:
- The total number of executed instructions goes down from 64.184 to 63.797
giga-instructions when Mesa is compiled with "gcc -O0 ..."
Please don't do performance
https://bugs.freedesktop.org/show_bug.cgi?id=98271
--- Comment #16 from John ---
I actually had to use -vo vdpau when I tried against mesa 11.0 (somehow mpv
didn't work with ogl-hq on that version) so I know it is problematic.
But the bug you link is still interesting
https://bugs.freedesktop.org/show_bug.cgi?id=98271
--- Comment #15 from Andy Furniss ---
(In reply to John from comment #10)
> Correct, I have an mpv config with:
>
> hwdec=vdpau
> hwdec-codecs=all
> vo=opengl-hq
>
> The rest shouldn't matter I believe.
If your mpv is not
https://bugs.freedesktop.org/show_bug.cgi?id=98271
--- Comment #18 from John ---
> amdgpu does not support UVD or VCE on SI parts yet.
oh, I should have verified in dmesg.
Sorry about that.
What should I try next?
--
You are receiving this mail because:
You are the
From: Deepak Sharma
This allows building radeonsi driver for surfaceless platform.
If we get any other requirements, we'll fix them in the function.
Removing the function isn't a solution.
v2: cosmetic changes - Marek
Signed-off-by: Deepak Sharma
https://bugs.freedesktop.org/show_bug.cgi?id=98132
--- Comment #3 from Iago Toral ---
(In reply to Iago Toral from comment #2)
> I have sent a couple of patches for review that fix these tests:
>
> https://lists.freedesktop.org/archives/mesa-dev/2016-October/131942.html
This
https://bugs.freedesktop.org/show_bug.cgi?id=98271
--- Comment #17 from Alex Deucher ---
amdgpu does not support UVD or VCE on SI parts yet.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the
Am 18.10.2016 um 15:42 schrieb Andy Furniss:
Andy Furniss wrote:
Christian König wrote:
Am 18.10.2016 um 11:19 schrieb Andy Furniss:
boyuan.zh...@amd.com wrote:
From: Boyuan Zhang
cabac is only supported in the h264 main and higher profiles
So shouldn't there be
https://bugs.freedesktop.org/show_bug.cgi?id=92198
Emil Velikov changed:
What|Removed |Added
Resolution|--- |NOTABUG
From: Marek Olšák
These constant value VS PARAM exports:
- 0,0,0,0
- 0,0,0,1
- 1,1,1,0
- 1,1,1,1
can be loaded into PS inputs using the DEFAULT_VAL field, and the VS exports
can be removed from the IR to save export & parameter memory.
After LLVM optimizations, analyze the
Am 17.10.2016 um 22:31 schrieb Alex Deucher:
On Mon, Oct 17, 2016 at 4:16 PM, wrote:
From: Boyuan Zhang
During dual instance encoding submission, if the second encode task and first
encode task have no reference dependency, e.g. p following with
On 10/18/2016 05:53 AM, Ilia Mirkin wrote:
The indirect handle has to come right after the coordinates, so if there
was a sample/bias/depth compare/offset, everything would end up being
shifted by one argument position.
Signed-off-by: Ilia Mirkin
Cc:
On 17 October 2016 at 14:44, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/gallium/drivers/radeonsi/si_pipe.c | 2 +-
> src/gallium/drivers/radeonsi/si_shader.c | 96 ++---
>
Since commit 63c5d5c6c46c8472ee7a8241a0f80f13d79cb8cd, the surfaceless
platform has allowed creation of pbuffer surfaces. But the vtable entry
for eglSwapBuffers has remained NULL.
Discovered by running a little pbuffer test.
Cc: Gurchetan Singh
---
On 18.10.2016 18:01, Emil Velikov wrote:
On 18 October 2016 at 09:49, Nicolai Hähnle wrote:
On 14.10.2016 20:21, Emil Velikov wrote:
From: Emil Velikov
Currently not everyone has libudev and with follow-up patches we'll
completely remove the
On 18 October 2016 at 09:49, Nicolai Hähnle wrote:
> On 14.10.2016 20:21, Emil Velikov wrote:
>>
>> From: Emil Velikov
>>
>> Currently not everyone has libudev and with follow-up patches we'll
>> completely remove the divergent codepaths.
>>
>> Use
From: Nicolai Hähnle
Fixes a regression introduced by commit 777dcf81b.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98307
--
Using std::sort here is quite a bit C++-ier than most parts of Mesa.
I used it because the standard C library is being its usual lame
For the series:
Reviewed-by: Marek Olšák
Marek
On Mon, Oct 17, 2016 at 7:25 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Use a full writemask in this case. This is relevant e.g. when a function
> has an inout argument
On Mon, Oct 17, 2016 at 11:12 PM, Timothy Arceri <
timothy.arc...@collabora.com> wrote:
> And pass in a pointer to the shader info in gl_program for ARB programs.
> ---
> src/amd/vulkan/radv_meta_blit.c | 12
> src/amd/vulkan/radv_meta_blit2d.c | 12
On 18 October 2016 at 09:57, Nicolai Hähnle wrote:
> On 11.10.2016 20:31, Emil Velikov wrote:
>>
>> Hi all,
>>
>> I've been 'threatening' to do this for a while, so here it is:
>>
>> Remove the three separate code paths that we have in the loader giving
>> is a since cleanup
On Tue, Oct 18, 2016 at 3:55 PM, Eero Tamminen
wrote:
> Hi,
>
> On 18.10.2016 16:25, Jan Ziak wrote:
>>
>> On Tue, Oct 18, 2016 at 3:12 PM, Nicolai Hähnle
>> wrote:
>>>
>>> On 18.10.2016 15:07, Jan Ziak wrote:
On Tue Oct 18 09:29:59 UTC
On Tue, Oct 18, 2016 at 6:28 PM, Emil Velikov wrote:
> On 17 October 2016 at 14:44, Marek Olšák wrote:
>> From: Marek Olšák
>>
>> ---
>> src/gallium/drivers/radeonsi/si_pipe.c | 2 +-
>>
Use the 'texture' local var in more places.
Rename 'pFormat' to 'viewFormat'.
---
src/gallium/drivers/svga/svga_state_sampler.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_state_sampler.c
https://bugs.freedesktop.org/show_bug.cgi?id=98310
Bug ID: 98310
Summary: GLES context returned by OpenGL API
Product: Mesa
Version: git
Hardware: Other
OS: All
Status: NEW
Severity: major
https://bugs.freedesktop.org/show_bug.cgi?id=98310
--- Comment #2 from Chad Versace ---
Related bug on the real libepoxy bugtracker:
https://github.com/anholt/libepoxy/issues/25
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA
https://bugs.freedesktop.org/show_bug.cgi?id=92198
ajax at nwnk dot net changed:
What|Removed |Added
Product|epoxy |Mesa
QA
https://bugs.freedesktop.org/show_bug.cgi?id=98271
--- Comment #19 from Christian König ---
(In reply to John from comment #18)
> What should I try next?
Installing an older kernel, see if that works with 12.0 mesa.
If yes we have narrowed it down to the kernel, if not
On 17 October 2016 at 08:56, Michel Dänzer wrote:
> On 14/10/16 07:02 PM, Emil Velikov wrote:
>> On 14 October 2016 at 09:45, Michel Dänzer wrote:
>>> On 14/10/16 05:14 PM, Emil Velikov wrote:
On 14 October 2016 at 01:45, Michel Dänzer
https://bugs.freedesktop.org/show_bug.cgi?id=98308
Bug ID: 98308
Summary: llvmpipe crashes with glxgears
Product: Mesa
Version: git
Hardware: Other
OS: All
Status: NEW
Severity: normal
Priority:
https://bugs.freedesktop.org/show_bug.cgi?id=98310
ajax at nwnk dot net changed:
What|Removed |Added
Resolution|--- |NOTOURBUG
On Tue, Oct 18, 2016 at 8:14 AM, Jason Ekstrand
wrote:
> I want to make a few comments on how this series is structured. This is
> not the way I would have done it and I think the way you structured it
> makes it substantially less rebasable than it could be and a bit
On 17 October 2016 at 19:07, Marek Olšák wrote:
> On Mon, Oct 17, 2016 at 7:10 PM, Emil Velikov
> wrote:
>> On 17 October 2016 at 17:47, Marek Olšák wrote:
>>> From: Deepak Sharma
>>>
>>> This allows building
I want to make a few comments on how this series is structured. This is
not the way I would have done it and I think the way you structured it
makes it substantially less rebasable than it could be and a bit harder to
review. The way *I* would have done this would be something like the
On Mon, Oct 17, 2016 at 11:12 PM, Timothy Arceri <
timothy.arc...@collabora.com> wrote:
> ---
> src/mesa/drivers/dri/i965/brw_context.h | 6 ++---
> src/mesa/drivers/dri/i965/brw_draw.c | 2 +-
> src/mesa/drivers/dri/i965/brw_program.c | 2 +-
>
On 18 October 2016 at 00:58, Jonathan Gray wrote:
> On Mon, Oct 17, 2016 at 05:34:02PM +0100, Emil Velikov wrote:
>> On 17 October 2016 at 16:39, Eric Engestrom
>> wrote:
>> > On Monday, 2016-10-17 22:53:20 +1100, Jonathan Gray wrote:
>> >> On Mon, Oct
Hi Dave,
On 17 October 2016 at 03:07, Dave Airlie wrote:
> From: Dave Airlie
>
> This moves all the alloc/free in anv to the generic helpers.
>
> Signed-off-by: Dave Airlie
> ---
> src/intel/vulkan/anv_batch_chain.c| 40
On 17 October 2016 at 05:24, Dave Airlie wrote:
> From: Dave Airlie
>
> This allows the API and the internals to be split, and the
> internals shared.
> ---
> src/intel/vulkan/anv_wsi_x11.c | 33 -
> 1 file changed, 24
On Tuesday, October 18, 2016 8:28:02 AM PDT Jason Ekstrand wrote:
> On Tue, Oct 18, 2016 at 8:14 AM, Jason Ekstrand
> wrote:
>
> > I want to make a few comments on how this series is structured. This is
> > not the way I would have done it and I think the way you
Reviewed-by: Charmaine Lee
From: Brian Paul
Sent: Tuesday, October 18, 2016 9:36 AM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee
Subject: [PATCH] svga: minor code improvements in
On 17 October 2016 at 03:07, Dave Airlie wrote:
> From: Dave Airlie
>
> Signed-off-by: Dave Airlie
Seems like a typo from the development stage - anv_wsi_inter_a_face
10 and 11 are independent so feel free to land whenever possible.
On Tue, Oct 18, 2016 at 7:12 PM, Jan Ziak <0xe2.0x9a.0...@gmail.com> wrote:
>> Regarding C++ templates, the compiler doesn't use them. If u_vector
>> (Dave Airlie?) provides the same functionality as your array, I
>> suggest we use u_vector instead.
>
> Let me repeat what you just wrote, because
Perf stat results for shader-db:
This is measured on an AMD Kaveri CPU.
gcc-6.2.0 -fno-omit-frame-pointer -g -O2
Unpatched:
$ cd shader-db
$ ../run-upstream perfstat-u --repeat=5 -- ./run -1 shaders >/dev/null
Performance counter stats for './run -1 shaders' (5 runs):
> Regarding C++ templates, the compiler doesn't use them. If u_vector
> (Dave Airlie?) provides the same functionality as your array, I
> suggest we use u_vector instead.
Let me repeat what you just wrote, because it is unbelievable: You are
advising the use of non-templated collection types in
On 09/29/2016 01:55 PM, Anutex wrote:
> I tried to debug this issue with changing the condition to check only bad
> magic and Error.
> And the test passed.
>
> Though i am not sure what is the correct behaviour if we are in this
> condition.
> May be we should make some other condition if the
On Tue, Oct 18, 2016 at 8:28 AM, Jason Ekstrand
wrote:
> On Tue, Oct 18, 2016 at 8:14 AM, Jason Ekstrand
> wrote:
>
>> I want to make a few comments on how this series is structured. This is
>> not the way I would have done it and I think the way you
THANK YOU! I've been wanting to see this happen for a long time.
On Sun, Oct 16, 2016 at 7:07 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> Although the vulkan drivers include mesa macros.h, for
> radv I'd like to move away from that.
>
>
We already talked on IRC about putting vk_alloc.h in src/util. Assuming
that's done, the series is
Acked-by: Jason Ekstrand
Please make sure you do a fairly complete (fedora config?) build test. I
don't want those MIN/MAX macros to cause problems.
--Jason
On Sun, Oct
On Tue, Oct 18, 2016 at 8:04 PM, Marek Olšák wrote:
> On Tue, Oct 18, 2016 at 7:12 PM, Jan Ziak <0xe2.0x9a.0...@gmail.com>
> wrote:
> >> Regarding C++ templates, the compiler doesn't use them. If u_vector
> >> (Dave Airlie?) provides the same functionality as your array, I
> >>
Hi Dave,
Thanks for doing this. It'll be great to get an Ack from the Intel
devs, on the idea.
Afaics with 22/22 in place you can drop the vk_alloc2/vk_free2
functions since they are no longer used.
Just an extra (small) suggestion below:
On 17 October 2016 at 05:24, Dave Airlie
Reviewed-by: Marek Olšák
Marek
On Tue, Oct 18, 2016 at 6:06 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Fixes a regression introduced by commit 777dcf81b.
>
> Bugzilla:
Typo in the summary - s/uneeded/unneeded/
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Tue, Oct 18, 2016 at 9:43 AM, Chad Versace wrote:
> Since commit 63c5d5c6c46c8472ee7a8241a0f80f13d79cb8cd, the surfaceless
> platform has allowed creation of pbuffer surfaces. But the vtable entry
> for eglSwapBuffers has remained NULL.
>
> Discovered by running a
From: Marek Olšák
v2: rebased
---
src/mesa/state_tracker/st_atom_blend.c | 3 ++-
src/mesa/state_tracker/st_atom_depth.c | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_blend.c
b/src/mesa/state_tracker/st_atom_blend.c
Reviewed-by: Brian Paul
On 10/18/2016 11:48 AM, Marek Olšák wrote:
From: Marek Olšák
v2: rebased
---
src/mesa/state_tracker/st_atom_blend.c | 3 ++-
src/mesa/state_tracker/st_atom_depth.c | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
1 - 100 of 193 matches
Mail list logo