On Tue, Oct 11, 2016 at 10:26:32PM +0300, Topi Pohjolainen wrote:
> The entire series is mostly bookkeeping changes. It switches to track
> color and state per slice instead of per miptree (such as we already
> do for depth and hiz).
>
> Patch 22 is still work in progress. We have all the details
Specifically, at least for now, we don't want to deal with the fact that
channel sizes for fp64 instructions are twice the size, so prevent
coalescing from instructions with a different type size.
Also, we should check that if we are coalescing a register from another
MOV we should be writing the
Am 13.10.2016 um 00:52 schrieb Mark Thompson:
The encoder attributes are needed for a user of the encoder to be
able to configure it sensibly without internal knowledge.
Reviewed-by: Christian König for the whole
series.
Do you have commit access?
Regards,
Christian.
---
src/gallium/st
On 13.10.2016 05:45, Michel Dänzer wrote:
On 12/10/16 08:52 PM, Nicolai Hähnle wrote:
On 12.10.2016 11:31, Michel Dänzer wrote:
diff --git a/src/mesa/state_tracker/st_cb_syncobj.c
b/src/mesa/state_tracker/st_cb_syncobj.c
index 123925a..de01880 100644
--- a/src/mesa/state_tracker/st_cb_syncobj.
On 13.10.2016 08:34, Edward O'Callaghan wrote:
Signed-off-by: Edward O'Callaghan
I think i965 can get the same treatment. And the mesamatrix people need
to check their parsing code for OpenGL 4.4 ;)
Cheers,
Nicolai
---
docs/features.txt | 18 +-
1 file changed, 9 insertio
Hi Michel,
On 13.10.2016 08:42, Michel Dänzer wrote:
On 13/10/16 01:50 AM, Nicolai Hähnle wrote:
Module: Mesa
Branch: master
Commit: f5f3cadca3809952288e3726ed5fde22090dc61d
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=f5f3cadca3809952288e3726ed5fde22090dc61d
Author: Nicolai Hähnl
Thanks Emil!
Reviewed-by: Tapani Pälli
On 10/12/2016 08:49 PM, Emil Velikov wrote:
This commit effectively reverts c10dcb2ce837922c6ee4e191e6d6202098a5ee10
and fixes the typedef redefinition which inspired it.
In order to prevent requiring X packages at build time earlier commit
forward decla
On 13/10/16 04:26 PM, Nicolai Hähnle wrote:
> Hi Michel,
>
> On 13.10.2016 08:42, Michel Dänzer wrote:
>> On 13/10/16 01:50 AM, Nicolai Hähnle wrote:
>>> Module: Mesa
>>> Branch: master
>>> Commit: f5f3cadca3809952288e3726ed5fde22090dc61d
>>> URL:
>>> http://cgit.freedesktop.org/mesa/mesa/commi
On 10/13/2016 06:24 PM, Nicolai Hähnle wrote:
> On 13.10.2016 08:34, Edward O'Callaghan wrote:
>> Signed-off-by: Edward O'Callaghan
>
> I think i965 can get the same treatment. And the mesamatrix people need
OK, I just was not sure about 'i965/hsw+' vs. 'i965' as I am not
familiar with all the
Signed-off-by: Edward O'Callaghan
---
docs/features.txt | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/docs/features.txt b/docs/features.txt
index ec2634f..a5c8750 100644
--- a/docs/features.txt
+++ b/docs/features.txt
@@ -206,19 +206,19 @@ GL 4.4, GLSL 4.4
https://bugs.freedesktop.org/show_bug.cgi?id=98223
Alejandro Piñeiro (freenode IRC: apinheiro) changed:
What|Removed |Added
CC||apinhe...@i
On 12.10.2016 19:02, Steven Toth wrote:
https://bugs.freedesktop.org/show_bug.cgi?id=98169
Really a two part bug.
1. The recent extensions to the HUB framework were tested exclusively
with glxgears/demo/head. None of these tools exercise the
free_query_data() code path. When these codepaths wer
Currently radeonsi doesn't advertise GLSL 4.40 and "GL 4.4, GLSL 4.40 --
all DONE" means the driver actually advertises GL 4.4 and GLSL 4.40.
So as long as radeonsi doesn't enable GLSL >= 4.40 it's not "all DONE".
Fixes: 789119d21 ("st/mesa: enable ARB_enhanced_layouts and turn the cap on")
Signe
nak, neither radeonsi nor i965 advertise GLSL 4.50.
Nicolai hasn't pushed the patch to enable GLSL 4.50 [1].
I'm not sure what's the plan for nouveau is [2].
See also https://patchwork.freedesktop.org/patch/115441/
Thanks,
Andreas
[1] https://patchwork.freedesktop.org/patch/114577/
[2] https://p
This will allow use to stop copying values between structs and
will also simplify handling handling these values in the shader cache.
---
src/compiler/nir/nir.h | 98 +---
src/compiler/shader_info.h | 123
src/mesa
Having these fields directly in nir_shader makes things difficult for
the shader cache. This series creates a shared shader info and makes a
start at the large amount of clean-ups this will enable.
I decided to send this before getting too carried away to make sure
people are happy with this chang
---
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 +-
src/mesa/drivers/dri/i965/brw_tcs.c | 32 ++-
src/mesa/drivers/dri/i965/brw_tcs_
---
src/compiler/shader_info.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/compiler/shader_info.h b/src/compiler/shader_info.h
index 68800a1..8266fc3 100644
--- a/src/compiler/shader_info.h
+++ b/src/compiler/shader_info.h
@@ -117,6 +117,13 @@ typedef struct shader_info {
---
src/compiler/shader_info.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/compiler/shader_info.h b/src/compiler/shader_info.h
index 8266fc3..502c7c8 100644
--- a/src/compiler/shader_info.h
+++ b/src/compiler/shader_info.h
@@ -78,6 +78,9 @@ typedef struct shader_info {
/**
---
src/compiler/shader_info.h | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/compiler/shader_info.h b/src/compiler/shader_info.h
index 502c7c8..66e06ad 100644
--- a/src/compiler/shader_info.h
+++ b/src/compiler/shader_info.h
@@ -114,6 +114,16 @@ typedef struct shader_info {
And use this field as the source for shader info in the nir_shader
this will allow us to set some of these fields from GLSL directly.
It will also simplify restoring from shader cache and allow the
removal of duplicate fields from GLSL.
---
src/compiler/glsl/glsl_to_nir.cpp | 3 ++-
src/mesa/main
Reviewed-by: Lionel Landwerlin
On 12/10/16 19:05, Emil Velikov wrote:
From: Emil Velikov
Use the shorthand make variable(s) as elsewhere in the build.
Signed-off-by: Emil Velikov
---
src/intel/Makefile.genxml.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/inte
On 10/13/2016 10:31 AM, Andreas Boll wrote:
nak, neither radeonsi nor i965 advertise GLSL 4.50.
Nicolai hasn't pushed the patch to enable GLSL 4.50 [1].
I'm not sure what's the plan for nouveau is [2].
The plan is to not change docs/features.txt for nvc0 (not yet). :)
See also https://patc
On 13/10/16 08:20, Christian König wrote:
> Am 13.10.2016 um 00:52 schrieb Mark Thompson:
>> The encoder attributes are needed for a user of the encoder to be
>> able to configure it sensibly without internal knowledge.
>
> Reviewed-by: Christian König for the whole series.
>
> Do you have commi
https://bugs.freedesktop.org/show_bug.cgi?id=98172
Michel Dänzer changed:
What|Removed |Added
Attachment #127237|0 |1
is obsolete|
https://bugs.freedesktop.org/show_bug.cgi?id=98172
Michel Dänzer changed:
What|Removed |Added
Attachment #127204|0 |1
is obsolete|
Hi Steven,
On 12 October 2016 at 18:02, Steven Toth wrote:
> https://bugs.freedesktop.org/show_bug.cgi?id=98169
>
Just a general comment/suggestion:
Don't include bug# in the commit summary. Having it as Bugzilla:
https... towards the end of the description. Aside: if you know the
commit that is
On Thu, Oct 13, 2016 at 2:09 AM, Roland Scheidegger wrote:
> Am 12.10.2016 um 15:29 schrieb Marek Olšák:
>> From: Marek Olšák
>>
>> For performance tuning in drivers. It filters out window system
>> framebuffers and OpenGL renderbuffers.
>>
>> radeonsi will use this to guess whether a depth buffe
On 13 October 2016 at 04:07, Michel Dänzer wrote:
> On 13/10/16 03:37 AM, Tobias Droste wrote:
>> Am Mittwoch, 12. Oktober 2016, 11:53:50 CEST schrieb Emil Velikov:
+LLVM_VERSION_REQUIRED_OPENCL=3.6.0
+LLVM_VERSION_REQUIRED_R600=3.6.0
+LLVM_VERSION_REQUIRED_RADEONSI=3.6.0
>>>
>
On 13.10.2016 10:20, Andreas Boll wrote:
Currently radeonsi doesn't advertise GLSL 4.40 and "GL 4.4, GLSL 4.40 --
all DONE" means the driver actually advertises GL 4.4 and GLSL 4.40.
So as long as radeonsi doesn't enable GLSL >= 4.40 it's not "all DONE".
Fixes: 789119d21 ("st/mesa: enable ARB_e
Alternatively you could expose GLSL 440 until the bugs are squished for 450
On Thu, 13 Oct 2016 at 12:39 Nicolai Hähnle wrote:
> On 13.10.2016 10:20, Andreas Boll wrote:
> > Currently radeonsi doesn't advertise GLSL 4.40 and "GL 4.4, GLSL 4.40 --
> > all DONE" means the driver actually advertise
I forgot to reply here on the list, I've just been talking about this
with Tapani face to face.
My series rebased and fixed on top of mesa master branch from yesterday
is here
https://github.com/juhapekka/juha_mesaexperimentals/tree/jenkins
Tapani was already taking rebased patches from above
On 12 October 2016 at 18:59, Axel Davy wrote:
> On 12/10/2016 19:15, Emil Velikov wrote:
>>
>> On 11 October 2016 at 19:54, Axel Davy wrote:
>>>
>>> On 11/10/2016 20:31, Emil Velikov wrote:
- udev = udev_new();
- if (!udev)
- goto prime_clean;
+ if (drmGetDe
On Oct 13, 2016 2:10 PM, "Mike Lothian" wrote:
>
> Alternatively you could expose GLSL 440 until the bugs are squished for
450
We can't expose GL 4.4 and 4.5 without approval from Khronos. I don't know
if that also applies to GLSL, but I'm assuming that it does. Until then,
we're stuck with GL 4.
From: Nicolai Hähnle
This fixes GL45-CTS.compute_shader.fp64-case3.
Cc: mesa-sta...@lists.freedesktop.org
---
src/gallium/drivers/radeonsi/si_shader.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi
From: Nicolai Hähnle
Remove the split between direct and indirect.
---
src/gallium/drivers/radeonsi/si_shader.c | 39 +---
1 file changed, 11 insertions(+), 28 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shade
Hello Nicolai
>> On inspection, this was due to the 'unigine' application
>> created the HUD contexts twice (two screens?). The
>> extensions had never been tested in that configuration,
>> and the free_query_data() calls were thus trigger.
>
>
> The HUD is actually created per-context. Two genera
On 10/12/2016 08:42 PM, Ilia Mirkin wrote:
src2 was being given the wrong modifier, and we were not properly
managing the modifier on the SHL source either.
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 12 +---
1 file changed, 5 insertions
On Thu, Oct 13, 2016 at 9:53 AM, Samuel Pitoiset
wrote:
>
>
> On 10/12/2016 08:42 PM, Ilia Mirkin wrote:
>>
>> src2 was being given the wrong modifier, and we were not properly
>> managing the modifier on the SHL source either.
>>
>> Signed-off-by: Ilia Mirkin
>> ---
>> src/gallium/drivers/nouve
On 10/13/2016 03:56 PM, Ilia Mirkin wrote:
On Thu, Oct 13, 2016 at 9:53 AM, Samuel Pitoiset
wrote:
On 10/12/2016 08:42 PM, Ilia Mirkin wrote:
src2 was being given the wrong modifier, and we were not properly
managing the modifier on the SHL source either.
Signed-off-by: Ilia Mirkin
---
Am 13.10.2016 um 11:33 schrieb Marek Olšák:
> On Thu, Oct 13, 2016 at 2:09 AM, Roland Scheidegger
> wrote:
>> Am 12.10.2016 um 15:29 schrieb Marek Olšák:
>>> From: Marek Olšák
>>>
>>> For performance tuning in drivers. It filters out window system
>>> framebuffers and OpenGL renderbuffers.
>>>
>
If you recompile llvm and mesa each time, are there any downsides of using
the shared libs?
I think there were some issues with some programs / games that had clashing
symbols but I'm not sure if that's still an issue
On Thu, 13 Oct 2016 at 11:15 Emil Velikov wrote:
> On 13 October 2016 at 04:0
On 10/13/2016 10:39 PM, Nicolai Hähnle wrote:
> On 13.10.2016 10:20, Andreas Boll wrote:
>> Currently radeonsi doesn't advertise GLSL 4.40 and "GL 4.4, GLSL 4.40 --
>> all DONE" means the driver actually advertises GL 4.4 and GLSL 4.40.
>>
>> So as long as radeonsi doesn't enable GLSL >= 4.40 it'
https://bugs.freedesktop.org/show_bug.cgi?id=98169
Steven Toth changed:
What|Removed |Added
Attachment #127222|0 |1
is obsolete|
https://bugs.freedesktop.org/show_bug.cgi?id=98223
--- Comment #5 from Randy ---
We can use the patch (in
https://lists.freedesktop.org/archives/mesa-dev/2016-August/124910.html) to
workaround the name issue, while some cases still fail due to wrong property,
like ARRAY_SIZE
in TargetInterface
On 10/13/2016 04:20 PM, Juha-Pekka Heikkila wrote:
I forgot to reply here on the list, I've just been talking about this
with Tapani face to face.
My series rebased and fixed on top of mesa master branch from
yesterday is here
https://github.com/juhapekka/juha_mesaexperimentals/tree/jenkins
https://bugs.freedesktop.org/show_bug.cgi?id=98169
--- Comment #30 from Steven Toth ---
Christoph, based on feedback from the mailing list, I made some changes to use
the MESA internal reference counting mechanisms which makes sensor and other
cleanup a little more robust.
The patch tests fine f
Reviewed-by: Tim Rowley
mailto:timothy.o.row...@intel.com>>
On Oct 12, 2016, at 4:13 PM, Eric Engestrom
mailto:e...@engestrom.ch>> wrote:
Fixes: b3bd8bb611bb465d2e5e ("swr: [rasterizer core] add support
for "RAW" surface format")
CovID: 1373647
Signed-off-by: Eric Engestrom mailto:e...@en
From: Nicolai Hähnle
Properly handle the case where there is a gap in the assigned output locations,
e.g. a fragment shader writes to color buffer 2 but not to color buffers 0 & 1.
Fixes
GL45-CTS.gtf33.GL3Tests.explicit_attrib_location.explicit_attrib_location_pipeline.
---
src/mesa/state_trac
From: Nicolai Hähnle
When more than one atomic counter buffer is in use, UniformStorage[n].opaque
is set up to contain indices that are contiguous across all used buffers.
This appears to be used by i965 via NIR, but for TGSI we do not treat atomic
counter buffers as opaque, so using the data in
From: Nicolai Hähnle
The corresponding opcodes for integers need to be treated the same as F2D.
Fixes GL45-CTS.gpu_shader_fp64.conversions.
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_t
From: Nicolai Hähnle
See the comment in the code for an explanation. This fixes
GL45-CTS.buffer_storage.map_persistent_draw.
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 29 -
1 file changed, 28 insertions(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_
Anyone else have opinions on this? Should st/mesa be adding
PIPE_BIND_RENDER_TARGET for all textures of the internal formats
listed in table 8.12? (Which would, in this case, force RGBA8 to be
used since RGBA4 is non-renderable.) Or is it perfectly legal for a
format to be not renderable when used
From: Nicolai Hähnle
---
src/compiler/glsl/ir_print_visitor.cpp | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/compiler/glsl/ir_print_visitor.cpp
b/src/compiler/glsl/ir_print_visitor.cpp
index c238c16..efb728b 100644
--- a/src/compiler/glsl/ir_print_visitor.cpp
---
src/amd/common/ac_nir_to_llvm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index e6ff7c8..9c764c7 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_nir_to_llvm.c
@@ -2373,8 +2373,8
https://bugs.freedesktop.org/show_bug.cgi?id=98172
--- Comment #8 from shinji.suz...@gmail.com ---
Comment on attachment 127267
--> https://bugs.freedesktop.org/attachment.cgi?id=127267
Lock the shared state mutex and work with a local reference of so->fence
Review of attachment 127267:
---
https://bugs.freedesktop.org/show_bug.cgi?id=98172
--- Comment #9 from shinji.suz...@gmail.com ---
oops. I'm getting confused. Concurrent programmingis hard.
> screen->fence_reference(screen, &so->fence, NULL);
will not be executed in arbitrary order but serially due to the mutex locking.
Still th
These were changed in LLVM r284024.
---
src/amd/common/ac_nir_to_llvm.c | 131
1 file changed, 107 insertions(+), 24 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 9c764c7..4fba7d3 100644
--- a/src/amd/com
On 13.10.2016 17:15, Ilia Mirkin wrote:
Anyone else have opinions on this? Should st/mesa be adding
PIPE_BIND_RENDER_TARGET for all textures of the internal formats
listed in table 8.12? (Which would, in this case, force RGBA8 to be
used since RGBA4 is non-renderable.) Or is it perfectly legal fo
https://bugs.freedesktop.org/show_bug.cgi?id=98172
--- Comment #10 from shinji.suz...@gmail.com ---
>I'm not confortable about sreen->fence_ference() being called concurrently.
I'm not comfortable about sreen->fence_finish() being called concurrently.
--
You are receiving this mail because:
You
https://bugs.freedesktop.org/show_bug.cgi?id=98169
--- Comment #31 from Christoph Haag ---
Yes, your new patch works. No crashing, all panes are visible and show the
correct values.
--
You are receiving this mail because:
You are the QA Contact for the bug.__
https://bugs.freedesktop.org/show_bug.cgi?id=98172
--- Comment #11 from shinji.suz...@gmail.com ---
Sorry I've made too many mistake in writing. (Should have gone to bed before
writing.) I'll rewrite whole post below.
I'm afraid execution of st_fence_sync() can still race.
Thread-A can run upto
On 12.10.2016 15:54, Marek Olšák wrote:
From: Marek Olšák
so that decompress blits aren't needed and depth texturing needs less
memory bandwidth.
Z16 and Z24 are promoted to Z32_FLOAT by the driver, because TC-compatible
HTILE only supports Z32_FLOAT. This doubles memory footprint for Z16.
The
Reviewed-by: Nicolai Hähnle
On 12.10.2016 15:29, Marek Olšák wrote:
From: Marek Olšák
For performance tuning in drivers. It filters out window system
framebuffers and OpenGL renderbuffers.
radeonsi will use this to guess whether a depth buffer will be read
by a shader. There is no guarantee
https://bugs.freedesktop.org/show_bug.cgi?id=98223
--- Comment #6 from Alejandro Piñeiro (freenode IRC: apinheiro)
---
(In reply to Randy from comment #5)
> We can use the patch (in
> https://lists.freedesktop.org/archives/mesa-dev/2016-August/124910.html) to
> workaround the name issue, while s
The series is
Reviewed-by: Nicolai Hähnle
On 12.10.2016 23:19, Marek Olšák wrote:
From: Marek Olšák
The table was copied from the Vulkan driver. The comment lines are as long
as the table for cosmetic reasons.
---
src/gallium/drivers/radeonsi/si_shader.h| 1 -
src/gallium/drivers/r
On Thu, Oct 13, 2016 at 5:35 PM, Nicolai Hähnle wrote:
> On 13.10.2016 17:15, Ilia Mirkin wrote:
>>
>> Anyone else have opinions on this? Should st/mesa be adding
>> PIPE_BIND_RENDER_TARGET for all textures of the internal formats
>> listed in table 8.12? (Which would, in this case, force RGBA8 to
On Fri 23 Sep 2016, Chad Versace wrote:
> diff --git a/docs/specs/EGL_MESA_platform_surfaceless.txt
> b/docs/specs/EGL_MESA_platform_surfaceless.txt
> new file mode 100644
> index 000..b700370
> --- /dev/null
> +++ b/docs/specs/EGL_MESA_platform_surfaceless.txt
> @@ -0,0 +1,108 @@
> +Name St
Mark Thompson wrote:
On 13/10/16 08:20, Christian König wrote:
Am 13.10.2016 um 00:52 schrieb Mark Thompson:
The encoder attributes are needed for a user of the encoder to be
able to configure it sensibly without internal knowledge.
Reviewed-by: Christian König for the whole series.
Do you
On Thu 13 Oct 2016, Chad Versace wrote:
> On Fri 23 Sep 2016, Chad Versace wrote:
>
> > diff --git a/docs/specs/EGL_MESA_platform_surfaceless.txt
> > b/docs/specs/EGL_MESA_platform_surfaceless.txt
> > new file mode 100644
> > index 000..b700370
> > --- /dev/null
> > +++ b/docs/specs/EGL_MESA_
From: Marek Olšák
Vulkan doesn't set these fields even though it doesn't use HiS.
HiS is disabled by programming DB_SRESULTS_COMPARE_STATEn to 0.
---
src/gallium/drivers/radeonsi/si_state.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_sta
For the series:
Reviewed-by: Marek Olšák
Marek
On Thu, Oct 13, 2016 at 3:44 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Remove the split between direct and indirect.
> ---
> src/gallium/drivers/radeonsi/si_shader.c | 39
> +---
> 1 file changed, 11 inser
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index b2d7699..e6edd90 100644
--- a/src/gallium/drivers/radeonsi
Dear Tom,
just FYI: this fails to apply on top of master
(761388a0eb586b1dcaec063ee561056ed132dc1a). git am chokes on the
visit_image_store() hunk for me. Attached is a "refreshed" version, which
applies for me. I hope I didn't butcher anything inadvertently.
Cheers,
Kai
Tom Stellard wrote on 13
---
src/amd/common/ac_nir_to_llvm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index e6ff7c8..9c764c7 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_nir_to_llvm.c
@@ -2373,8 +2373,8
Round two of the patchset, incorporating feedback
from nhaeh...@gmail.com
The fixes in this set address bugfix #68169, HUD crashing
when testing with unigine (heaven).
The bug also manifested itself as a lack of data in
HUD charts when multiple instanced were created and
destroyed in a specific o
On Thu, Oct 13, 2016 at 07:20:30PM +0200, Kai Wasserbäch wrote:
> Dear Tom,
> just FYI: this fails to apply on top of master
> (761388a0eb586b1dcaec063ee561056ed132dc1a). git am chokes on the
> visit_image_store() hunk for me. Attached is a "refreshed" version, which
> applies for me. I hope I didn
These were changed in LLVM r284024.
v2:
- Only use float types for vdata of llvm.amdgcn.image.store. LLVM doesn't
support integer types for this intrinsic.
---
src/amd/common/ac_nir_to_llvm.c | 133
1 file changed, 108 insertions(+), 25 deletions(-)
On 10/07/2016 12:05 PM, Serge Martin wrote:
> On Thursday 06 October 2016 16:26:21 Vedran Miletić wrote:
>> CLOVER_CL_VERSION_OVERRIDE allows overriding default OpenCL version
>> supported by Clover, analogous to MESA_GL_VERSION_OVERRIDE for OpenGL.
>> CLOVER_CL_C_VERSION_OVERRIDE allows overridng
On 10/13/2016 07:45 PM, Vedran Miletić wrote:
> On 10/07/2016 12:05 PM, Serge Martin wrote:
>> On Thursday 06 October 2016 16:26:21 Vedran Miletić wrote:
>>> PACKAGE_VERSION #ifdef MESA_GIT_SHA1
>>> " (" MESA_GIT_SHA1 ")"
>>> #endif
>>> diff --git a/src/gallium/state_track
Disregard this, the mbox file only contained the second patch. Sorry for the
noise.
Kai Wasserbäch wrote on 13.10.2016 19:20:
> Dear Tom,
> just FYI: this fails to apply on top of master
> (761388a0eb586b1dcaec063ee561056ed132dc1a). git am chokes on the
> visit_image_store() hunk for me. Attached
On Thursday, 13 October 2016, Mike Lothian > wrote:
> If you recompile llvm and mesa each time, are there any downsides of using
> the shared libs?
>
> How many devs rebuild package X when one of its dependencies has a bug fix
release. And how many think it's a good idea ;-)
Just some food for tho
On Thursday, 13 October 2016, Emil Velikov wrote:
> On 12 October 2016 at 18:59, Axel Davy >
> wrote:
> > On 12/10/2016 19:15, Emil Velikov wrote:
> >>
> >> On 11 October 2016 at 19:54, Axel Davy >
> wrote:
> >>>
> >>> On 11/10/2016 20:31, Emil Velikov wrote:
>
> - udev = udev_new(
Hi all,
On Tue, Oct 11, 2016 at 01:47:37PM -0400, Robert Foss wrote:
>
>
> On 2016-10-11 01:37 PM, Chad Versace wrote:
> > On Tue 11 Oct 2016, Emil Velikov wrote:
> > > On 10 October 2016 at 20:41, Rob Clark wrote:
> > > > On Mon, Oct 10, 2016 at 1:43 PM, Chad Versace
> > > > wrote:
> >
> >
When !use_ib_bos, we can't easily chain ibs one to another. If the
required cs size grows over 1Mi - 8 dwords just fail the cs so that we
won't assert-fail in radv_amdgpu_winsys_cs_submit later on.
---
Please, push the patch after it has been reviewed.
src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.
On Thu, Oct 13, 2016 at 4:37 PM, Rafael Antognolli
wrote:
> Hi all,
>
> On Tue, Oct 11, 2016 at 01:47:37PM -0400, Robert Foss wrote:
>>
>>
>> On 2016-10-11 01:37 PM, Chad Versace wrote:
>> > On Tue 11 Oct 2016, Emil Velikov wrote:
>> > > On 10 October 2016 at 20:41, Rob Clark wrote:
>> > > > On M
Mesa's set of supported platform extensions depends on the autoconf
option --with-egl-platforms=foo,bar,baz. If --with-egl-platforms lacks
foo, then eglGetPlatformDisplay(EGL_PLATFORM_FOO, ...) unconditonally
fails.
So, if --with-egl-platforms lacks foo, then remove
EGL_VENDOR_platform_foo from th
v2:
- Assign enum values.
- Define interactions with EGL_EXT_platform_base and EGL 1.4.
- Add Gurchetan as contributor, as he implemented the pbuffer support.
---
docs/specs/EGL_MESA_platform_surfaceless.txt | 120 +++
1 file changed, 120 insertions(+)
create m
Mesa's EGL has supported Chrome OS's "surfaceless" platform for many
months, but the behavior of that platform has never been documented.
Here's my attempt to fix that.
I've already committed the draft extension spec into the private Khronos
registry. After these patches land in master, I plan to
It was the lone file in src/egl/docs. Move it to where the other specs
live, in $MESA_TOP/docs/specs.
---
.../EGL_MESA_screen_surface => docs/specs/OLD/EGL_MESA_screen_surface.txt | 0
1 file changed, 0 insertions(+), 0 deletions(-)
rename src/egl/docs/EGL_MESA_screen_surface =>
docs/specs/OLD/E
Years ago Khronos replaced the registry's spec files with newfangled XML
files. Update the reference in doc/specs/enum.txt accordingly.
---
docs/specs/enums.txt | 16 ++--
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/docs/specs/enums.txt b/docs/specs/enums.txt
index
---
include/EGL/eglmesaext.h | 5 +
src/egl/main/eglapi.c | 42 +-
src/egl/main/egldisplay.c | 21 +
src/egl/main/egldisplay.h | 5 +
src/egl/main/eglglobals.c | 3 +++
5 files changed, 75 insertions(+), 1 deletion(-)
dif
Document the EGL enum ranges for Mesa and those values allocated by the
following extensions:
EGL_MESA_drm_image
EGL_MESA_platform_gbm
EGL_MESA_platform_surfaceless
EGL_WL_bind_wayland_display
---
docs/specs/enums.txt | 27 ++-
1 file changed, 26 insertions
For the series:
Reviewed-by: Marek Olšák
Marek
On Thu, Oct 13, 2016 at 5:13 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> The corresponding opcodes for integers need to be treated the same as F2D.
>
> Fixes GL45-CTS.gpu_shader_fp64.conversions.
> ---
> src/mesa/state_tracker/st_glsl_t
https://bugs.freedesktop.org/show_bug.cgi?id=98222
--- Comment #1 from Christoph Haag ---
https://cgit.freedesktop.org/mesa/mesa/commit/?id=5c66d46d6ace0603227e4b919b132416444949e7
should fix it.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee fo
On 2016-10-13 04:37 PM, Rafael Antognolli wrote:
Hi all,
On Tue, Oct 11, 2016 at 01:47:37PM -0400, Robert Foss wrote:
On 2016-10-11 01:37 PM, Chad Versace wrote:
On Tue 11 Oct 2016, Emil Velikov wrote:
On 10 October 2016 at 20:41, Rob Clark wrote:
On Mon, Oct 10, 2016 at 1:43 PM, Chad V
CLOVER_CL_VERSION_OVERRIDE allows overriding default OpenCL version
supported by Clover, analogous to MESA_GL_VERSION_OVERRIDE for OpenGL.
CLOVER_CL_C_VERSION_OVERRIDE allows overridng default OpenCL C version.
v2:
- move version getters to version.hpp, simplify
- set -cl-std= to CLOVER_CL_C_V
On 13/10/16 07:14 PM, Emil Velikov wrote:
> On 13 October 2016 at 04:07, Michel Dänzer wrote:
>> On 13/10/16 03:37 AM, Tobias Droste wrote:
>>> Am Mittwoch, 12. Oktober 2016, 11:53:50 CEST schrieb Emil Velikov:
>
> +LLVM_VERSION_REQUIRED_OPENCL=3.6.0
> +LLVM_VERSION_REQUIRED_R600=3.6.0
On 13/10/16 11:16 PM, Mike Lothian wrote:
> If you recompile llvm and mesa each time, are there any downsides of
> using the shared libs?
No particular downsides come to mind, rather upsides:
* Generally no requirement to rebuild Mesa after changes to LLVM.
* Both LLVM and Mesa take significantly
On 14/10/16 09:54 AM, Michel Dänzer wrote:
> On 13/10/16 11:16 PM, Mike Lothian wrote:
>> I think there were some issues with some programs / games that had
>> clashing symbols but I'm not sure if that's still an issue
>
> I'm only aware of such issues with the Steam runtime, which are arguably
>
1 - 100 of 130 matches
Mail list logo
