Patches 1 & 2:
Reviewed-by: Nicolai Hähnle
On 03.01.2017 20:17, Marek Olšák wrote:
From: Marek Olšák
This has no effect because no code uses those members with ranged decls.
---
src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
di
Interesting, this suggests a bug in the LLVM WQM stuff. Anyway, it makes
sense for now.
Reviewed-by: Nicolai Hähnle
On 05.01.2017 13:58, Marek Olšák wrote:
From: Marek Olšák
v2: do it properly
---
src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c | 15 +--
1 file changed, 13
On 07.01.2017 09:02, Kenneth Graunke wrote:
It feels weird using GL_* enums in a Vulkan driver.
Signed-off-by: Kenneth Graunke
---
src/compiler/glsl/ast.h | 2 +-
src/compiler/glsl/glsl_parser.yy | 8
src/compiler/glsl/glsl_parser_extras.cpp | 2 +-
src/com
vtn_ssa_value() can produce variable loads, and the cursor might
be after a return statement, causing nir_builder assert failures
about not inserting instructions after a jump.
This fixes:
dEQP-VK.spirv_assembly.instruction.graphics.barrier.in_if
dEQP-VK.spirv_assembly.instruction.graphics.barrier
Signed-off-by: Samuel Iglesias Gonsálvez
---
docs/relnotes/13.1.0.html | 1 +
1 file changed, 1 insertion(+)
diff --git a/docs/relnotes/13.1.0.html b/docs/relnotes/13.1.0.html
index be1ab0d76ef..662f0ceb070 100644
--- a/docs/relnotes/13.1.0.html
+++ b/docs/relnotes/13.1.0.html
@@ -52,6 +52,7 @@
On Mon, Jan 9, 2017 at 9:40 AM, Nicolai Hähnle wrote:
> Interesting, this suggests a bug in the LLVM WQM stuff. Anyway, it makes
> sense for now.
Or more likely: v_interp is executed after KILL, which breaks WQM.
Marek
___
mesa-dev mailing list
mesa-de
On Thu, 2017-01-05 at 12:46 -0800, Jason Ekstrand wrote:
> With the image_ms_array line removed (I don't believe we support
> that), 20-22 are
>
> Reviewed-by: Jason Ekstrand
>
I have applied all the suggestions and pushed the patches to master.
Thanks a lot for the review!
Sam
__
On Mon, Jan 9, 2017 at 7:25 AM, Michel Dänzer wrote:
> On 09/01/17 03:13 PM, Michel Dänzer wrote:
>> On 07/01/17 11:46 PM, Marek Olšák wrote:
>>> From: Marek Olšák
>>>
>>> ~/.drirc is created by the driconf tool (GPL license) and it overrides
>>> system drirc settings and can't be changed by Mesa
Reviewed-by: Marek Olšák
Marek
On Mon, Jan 9, 2017 at 6:13 AM, Timothy Arceri
wrote:
> We only need to set it when linking was successful and the program
> being linked is currently active.
>
> The programs_in_use mask is just used as a flag for now but in
> a following patch we will use it to
On 09.01.2017 11:58, Marek Olšák wrote:
On Mon, Jan 9, 2017 at 7:25 AM, Michel Dänzer wrote:
On 09/01/17 03:13 PM, Michel Dänzer wrote:
On 07/01/17 11:46 PM, Marek Olšák wrote:
From: Marek Olšák
~/.drirc is created by the driconf tool (GPL license) and it overrides
system drirc settings and
Am 09.01.2017 um 11:58 schrieb Marek Olšák:
On Mon, Jan 9, 2017 at 7:25 AM, Michel Dänzer wrote:
On 09/01/17 03:13 PM, Michel Dänzer wrote:
On 07/01/17 11:46 PM, Marek Olšák wrote:
From: Marek Olšák
~/.drirc is created by the driconf tool (GPL license) and it overrides
system drirc settings
On 09.01.2017 11:40, Marek Olšák wrote:
On Mon, Jan 9, 2017 at 9:40 AM, Nicolai Hähnle wrote:
Interesting, this suggests a bug in the LLVM WQM stuff. Anyway, it makes
sense for now.
Or more likely: v_interp is executed after KILL, which breaks WQM.
That's only relevant for derivative calcul
Reviewed-by: Edward O'Callaghan
On 01/09/2017 04:38 AM, Grazvydas Ignotas wrote:
> Fixes crashes when both glx-tls and asm are enabled on x32.
>
> Cc: mesa-sta...@lists.freedesktop.org
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94512
> Signed-off-by: Grazvydas Ignotas
> ---
> no c
Could be a patch like this useful for power users?
Load custom drirc only when specified in env variable.
Patch untested, not even compiled.
edmondo (on mobile phone)
diff --git a/docs/envvars.html b/docs/envvars.html
index cf57ca5..a404ced 100644
--- a/docs/envvars.html
+++ b/docs/envvars.h
On Mon, Jan 9, 2017 at 12:09 PM, Nicolai Hähnle wrote:
> On 09.01.2017 11:40, Marek Olšák wrote:
>>
>> On Mon, Jan 9, 2017 at 9:40 AM, Nicolai Hähnle wrote:
>>>
>>> Interesting, this suggests a bug in the LLVM WQM stuff. Anyway, it makes
>>> sense for now.
>>
>>
>> Or more likely: v_interp is exe
On Mon, Jan 9, 2017 at 12:16 PM, Edmondo Tommasina
wrote:
> Could be a patch like this useful for power users?
>
> Load custom drirc only when specified in env variable.
Yes, that sounds like a good alternative.
Marek
>
> Patch untested, not even compiled.
>
> edmondo (on mobile phone)
>
> diff
On 09.01.2017 12:28, Marek Olšák wrote:
On Mon, Jan 9, 2017 at 12:09 PM, Nicolai Hähnle wrote:
On 09.01.2017 11:40, Marek Olšák wrote:
On Mon, Jan 9, 2017 at 9:40 AM, Nicolai Hähnle wrote:
Interesting, this suggests a bug in the LLVM WQM stuff. Anyway, it makes
sense for now.
Or more li
On Mon, Jan 9, 2017 at 12:58 PM, Nicolai Hähnle wrote:
> On 09.01.2017 12:28, Marek Olšák wrote:
>>
>> On Mon, Jan 9, 2017 at 12:09 PM, Nicolai Hähnle
>> wrote:
>>>
>>> On 09.01.2017 11:40, Marek Olšák wrote:
On Mon, Jan 9, 2017 at 9:40 AM, Nicolai Hähnle
wrote:
>
>
>
https://bugs.freedesktop.org/show_bug.cgi?id=99319
--- Comment #4 from Vladislav Egorov ---
(In reply to Thomas Helland from comment #2)
> A small update: It appears we are spending all of the time walking in a
> for-loop at the bottom of context_cleanup:
>
> for (i = 0; i < ARRAY_SIZE(cs->buffe
Geometry and Tessellation stages do handle this as a system value instead.
Fixes:
dEQP-VK.geometry.basic.primitive_id
---
src/compiler/spirv/vtn_variables.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/compiler/spirv/vtn_variables.c
b/src/compiler/spirv/vtn_variables
Thanks for your review, Rob. Using kms-dri would mean writing a new gralloc
to basically the same thing as
gralloc.default and moving to grm_gralloc seems to be a bigger task meant
for another day. But I agree we would
want to avoid introducing dependency. Would extending drisw loader func and
limi
2016-11-23 21:20 GMT+08:00 Boyan Ding :
> 2016-11-07 20:27 GMT+08:00 Boyan Ding :
>> 2016-11-05 3:23 GMT+08:00 Matt Turner :
>>> On Sun, Oct 30, 2016 at 11:45 PM, Boyan Ding wrote:
According to OpenGL Shading Language 4.50 spec, Section 8.7 "Vector
Relational Functions", functions of thi
To match the usage at other places.
Signed-off-by: Boyan Ding
---
src/compiler/spirv/vtn_variables.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/compiler/spirv/vtn_variables.c
b/src/compiler/spirv/vtn_variables.c
index e3845365bd..26a8ca369b 100644
--- a/src/co
Unused since 0a7691ee (mesa: Enable enums for OES_viewport_array).
Silence a warning of unused variable.
Signed-off-by: Boyan Ding
---
src/mesa/main/get.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index 10b40d405d..f0bb041fc3 100644
--- a/src/me
---
src/compiler/spirv/spirv_to_nir.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/compiler/spirv/spirv_to_nir.c
b/src/compiler/spirv/spirv_to_nir.c
index befee1a..2205072 100644
--- a/src/compiler/spirv/spirv_to_nir.c
+++ b/src/compiler/spirv/spirv_to_nir.c
@@ -559,7 +
On Sun, 2017-01-08 at 21:26 -0800, Kenneth Graunke wrote:
> We need to:
> - handle the extra array level for per-vertex varyings
> - handle the patch qualifier correctly
> - assign varying locations
>
> Signed-off-by: Kenneth Graunke
> ---
> src/compiler/spirv/vtn_private.h | 1 +
> src/compi
On Jan 9, 2017 1:48 AM, "Kenneth Graunke" wrote:
vtn_ssa_value() can produce variable loads, and the cursor might
be after a return statement, causing nir_builder assert failures
about not inserting instructions after a jump.
This fixes:
dEQP-VK.spirv_assembly.instruction.graphics.barrier.in_if
On Monday, January 9, 2017 9:47:36 AM PST Nicolai Hähnle wrote:
> On 07.01.2017 09:02, Kenneth Graunke wrote:
> > It feels weird using GL_* enums in a Vulkan driver.
> >
> > Signed-off-by: Kenneth Graunke
> > ---
> > src/compiler/glsl/ast.h | 2 +-
> > src/compiler/glsl/glsl_par
Patches 1-5 are:
Reviewed-by: Iago Toral Quiroga
On Sun, 2017-01-08 at 21:26 -0800, Kenneth Graunke wrote:
> Signed-off-by: Kenneth Graunke
> ---
> src/compiler/spirv/spirv_to_nir.c | 28 ++--
> 1 file changed, 22 insertions(+), 6 deletions(-)
>
> One totally bogus thin
On 9 December 2016 at 05:58, Tapani Pälli wrote:
> When 'evict_random_item' attempts to remove cache content to make more
> space, it may try to remove from new cache directory it just created
> which only has '.tmp' content at this phase and not proper cache files,
> this will cause eviction to f
On 9 January 2017 at 15:36, Emil Velikov wrote:
> On 9 December 2016 at 05:58, Tapani Pälli wrote:
>> When 'evict_random_item' attempts to remove cache content to make more
>> space, it may try to remove from new cache directory it just created
>> which only has '.tmp' content at this phase and n
On 16 December 2016 at 17:25, Gustaw Smolarczyk wrote:
> 2016-12-16 18:12 GMT+01:00 Gustaw Smolarczyk :
>> 2016-12-16 17:57 GMT+01:00 Emil Velikov :
>>> On 16 December 2016 at 15:27, Gustaw Smolarczyk
>>> wrote:
2016-12-16 14:50 GMT+01:00 Emil Velikov :
>
> On 5 October 2016 at 23:1
On Mon, Jan 9, 2017 at 4:50 AM, Iago Toral Quiroga
wrote:
> Geometry and Tessellation stages do handle this as a system value instead.
>
> Fixes:
> dEQP-VK.geometry.basic.primitive_id
> ---
> src/compiler/spirv/vtn_variables.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --
Also, we should probably consider converting it over to a system value one
of these days. That would make more sense from a theoretical model
perspective. Not sure if it makes sense in any drivers though.
On Mon, Jan 9, 2017 at 8:03 AM, Jason Ekstrand wrote:
> On Mon, Jan 9, 2017 at 4:50 AM, I
I hate decorations... 1 and 2 are
Reviewed-by: Jason Ekstrand
On Mon, Jan 9, 2017 at 7:16 AM, Iago Toral wrote:
> On Sun, 2017-01-08 at 21:26 -0800, Kenneth Graunke wrote:
> > We need to:
> > - handle the extra array level for per-vertex varyings
> > - handle the patch qualifier correctly
> >
On Sun, Jan 8, 2017 at 9:38 PM, Dave Airlie wrote:
> On 9 January 2017 at 15:26, Kenneth Graunke wrote:
> > This is all implemented now.
>
> Do we want to key these off the nir_spirv_supported_extensions thing I
> added?
>
I think we do unless Dave thinks he can get tess support landed for radv
On Sun, Jan 8, 2017 at 9:26 PM, Kenneth Graunke
wrote:
> Signed-off-by: Kenneth Graunke
> ---
> src/intel/vulkan/anv_pipeline.c | 9 +++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/src/intel/vulkan/anv_pipeline.c b/src/intel/vulkan/anv_
> pipeline.c
> index 17491e34
On Sun, Jan 8, 2017 at 9:26 PM, Kenneth Graunke
wrote:
> Signed-off-by: Kenneth Graunke
> ---
> src/intel/vulkan/anv_private.h | 2 +
> src/intel/vulkan/genX_pipeline.c | 87 ++
> ++
> src/intel/vulkan/genX_state.c| 4 --
> 3 files changed, 89 inserti
For similar compute-related things (workgroup sizes) we handle this
directly in nir_lower_system_values.
On Sun, Jan 8, 2017 at 9:39 PM, Dave Airlie wrote:
> On 9 January 2017 at 15:26, Kenneth Graunke wrote:
> > In Vulkan, we always have both the TCS and TES available in the same
> > pipeline,
On Mon, Jan 9, 2017 at 8:40 AM, Jason Ekstrand wrote:
> For similar compute-related things (workgroup sizes) we handle this
> directly in nir_lower_system_values.
>
Doing so would probably require adding a vertices_in thing to tes... I
don't know how strongly I feel about doing it one way vs. an
On Sun, Jan 8, 2017 at 9:26 PM, Kenneth Graunke
wrote:
> Signed-off-by: Kenneth Graunke
> ---
> src/intel/vulkan/anv_pipeline.c | 156 ++
> +-
> 1 file changed, 154 insertions(+), 2 deletions(-)
>
> There's one majorly bogus thing here: caching is totally dis
On Mon, Jan 9, 2017 at 6:34 AM, Boyan Ding wrote:
> 2016-11-23 21:20 GMT+08:00 Boyan Ding :
>> 2016-11-07 20:27 GMT+08:00 Boyan Ding :
>>> 2016-11-05 3:23 GMT+08:00 Matt Turner :
On Sun, Oct 30, 2016 at 11:45 PM, Boyan Ding
wrote:
> According to OpenGL Shading Language 4.50 spec, S
On Sun, Jan 8, 2017 at 9:26 PM, Kenneth Graunke
wrote:
> Signed-off-by: Kenneth Graunke
> ---
> src/compiler/spirv/spirv_to_nir.c | 28 ++--
> 1 file changed, 22 insertions(+), 6 deletions(-)
>
> One totally bogus thing here: we set CCW backwards. In all three driver
>
woohoo
Reviewed-by: Jason Ekstrand
On Mon, Jan 9, 2017 at 2:38 AM, Samuel Iglesias Gonsálvez <
sigles...@igalia.com> wrote:
> Signed-off-by: Samuel Iglesias Gonsálvez
> ---
> docs/relnotes/13.1.0.html | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/docs/relnotes/13.1.0.html b/docs/re
Hi,
This series implements the support for Haswell 64bit vertex attributes. With it,
we can enable OpenGL 4.2 in Haswell.
This work covers partially https://bugs.freedesktop.org/show_bug.cgi?id=94442.
These patches are available in our repository for testing. You can clone it
using the following
From: Alejandro Piñeiro
---
src/mesa/drivers/dri/i965/brw_vec4.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp
b/src/mesa/drivers/dri/i965/brw_vec4.cpp
index 5ddbe58..f80162d 100644
--- a/src/mesa/drivers/dri/i965/brw_vec4.cpp
+++ b
From: Alejandro Piñeiro
gen < 8 doesn't support *64*PASSTHRU formats when emitting
vertices. So in order to provide the equivalent functionality, we need
to downsize the format to equivalent *32*FLOAT, and in some cases
(R64G64B64 and R64G64B64A64) submit two 3DSTATE_VERTEX_ELEMENTS for
each vert
From: Alejandro Piñeiro
Although gen7 doesn't include surface types as a valid conversion format,
we return it, as it reflects what we want to achieve, even if we need
to workaround it on gen < 8.
---
src/mesa/drivers/dri/i965/brw_draw_upload.c | 8 ++--
1 file changed, 6 insertions(+), 2 de
For dvec3 and dvec4 types, a single GRF do not have enough space to
allocate two inputs from two different vertices (SIMD4x2).
So the GRF only contains first two components for the two vertices, and
the next GRF has the remaining components.
We want to put all the components for the same vertex i
From: Alejandro Piñeiro
Doubles need extra space, so we would need to do a remapping for vec4
too in order to take that into account. We reuse the already
existing remap_vs_attrs, but passing is_scalar, so they could
remap accordingly.
Signed-off-by: Alejandro Piñeiro
Signed-off-by: Juan A. Sua
From: Alejandro Piñeiro
Doubles needs more that one slot per attribute. So when filling the
attribute_map we check if it is a double in order to allocate one
extra register.
Signed-off-by: Alejandro Piñeiro
Signed-off-by: Juan A. Suarez Romero
---
src/mesa/drivers/dri/i965/brw_vec4.cpp | 13 +
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
Signed-off-by: Juan A. Suarez Romero
---
src/mesa/drivers/dri/i965/intel_extensions.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/d
From: Alejandro Piñeiro
---
src/mesa/drivers/dri/i965/intel_extensions.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 915797a..ac877cc 100644
--- a/src/mesa/drivers/dri/i9
From: Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez
Signed-off-by: Juan A. Suarez Romero
---
src/mesa/drivers/dri/i965/intel_extensions.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dr
Those not supporting 64 bit input vertex attributes will have the
dual_slot value as false.
---
src/mesa/drivers/dri/i965/brw_draw_upload.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_draw_upload.c
b/src/mesa/drivers/dri/i965/brw_draw_upload
GL_ARB_vertex_attrib_64bit was the last piece missing.
---
src/mesa/drivers/dri/i965/intel_extensions.c | 2 +-
src/mesa/drivers/dri/i965/intel_screen.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i
On Monday, January 9, 2017 8:26:34 AM PST Jason Ekstrand wrote:
> On Sun, Jan 8, 2017 at 9:26 PM, Kenneth Graunke
> wrote:
>
> > Signed-off-by: Kenneth Graunke
> > ---
> > src/intel/vulkan/anv_pipeline.c | 9 +++--
> > 1 file changed, 7 insertions(+), 2 deletions(-)
> >
> > diff --git a/src
Define a new MESA_CUSTOM_DRIRC environment variable to load
a drirc custom file, instead of blindly using the hardcoded
${HOME}/.drirc file created the driconf tool.
Some more discussion about the driconf issues can be found
in reading following mesa-dev thread:
https://lists.freedesktop.org/archi
On Mon, Jan 9, 2017 at 12:37 PM, Marek Olšák wrote:
> On Mon, Jan 9, 2017 at 12:16 PM, Edmondo Tommasina
> wrote:
>> Could be a patch like this useful for power users?
>>
>> Load custom drirc only when specified in env variable.
>
> Yes, that sounds like a good alternative.
Patch sent to the mai
On Mon, Jan 9, 2017 at 10:07 AM, Kenneth Graunke
wrote:
> On Monday, January 9, 2017 8:26:34 AM PST Jason Ekstrand wrote:
> > On Sun, Jan 8, 2017 at 9:26 PM, Kenneth Graunke
> > wrote:
> >
> > > Signed-off-by: Kenneth Graunke
> > > ---
> > > src/intel/vulkan/anv_pipeline.c | 9 +++--
> > >
On 28 July 2016 at 13:45, Marek Olšák wrote:
> On Fri, Jul 22, 2016 at 6:22 PM, Rob Herring wrote:
>> In order to prevent multiple pipe_screens being created in the same
>> process, lookup of the DRM FD and reference counting of the pipe_screen
>> are needed. Several implementations of this exist
On 22 July 2016 at 19:01, Rob Herring wrote:
> On Fri, Jul 22, 2016 at 11:46 AM, Ilia Mirkin wrote:
>> On Fri, Jul 22, 2016 at 12:22 PM, Rob Herring wrote:
>>> In order to prevent multiple pipe_screens being created in the same
>>> process, lookup of the DRM FD and reference counting of the pipe
Assuming it still builds on newer GCC,
Acked-by: Jason Ekstrand
On Sun, Jan 8, 2017 at 9:26 AM, Vinson Lee wrote:
> Fix this build error with GCC 4.4.7.
>
> CC nir/nir_opt_copy_prop_vars.lo
> nir/nir_opt_copy_prop_vars.c: In function ‘copy_prop_vars_block’:
> nir/nir_opt_copy_prop_vars.c
Acked-by: Marek Olšák
Marek
On Fri, Jan 6, 2017 at 10:42 AM, Samuel Pitoiset
wrote:
> D3D always computes the absolute value while GLSL says that the
> result of inversesqrt() is undefined if x <= 0 (and undefined if
> x < 0 for sqrt()). But some apps rely on this specific behaviour
> which is
On 22 July 2016 at 17:22, Rob Herring wrote:
> Use the common pipe_screen ref counting and fd hashing functions. The
> mutex can be dropped as the pipe loader protects the create_screen()
> calls.
>
> Signed-off-by: Rob Herring
> Cc: "Marek Olšák"
> Cc: Ilia Mirkin
> ---
> src/gallium/drivers/
On 29 July 2016 at 20:08, Marek Olšák wrote:
> On Fri, Jul 29, 2016 at 8:01 PM, Rob Herring wrote:
>> On Fri, Jul 29, 2016 at 12:51 PM, Marek Olšák wrote:
>>> The fd table and reference counting in the winsys is required by the
>>> GL-VDPAU interop.
>>>
>>> radeon_drm_winsys_create and amdgpu_wi
Hi Rob,
On 22 July 2016 at 17:22, Rob Herring wrote:
> Another version of common pipe_screen reference counting. Please help
> test on AMD and Nouveau as those are the more complicated ones and I
> don't have h/w to test.
>
> Changes in v4:
> - Move fd dup() back into driver winsys create screen
Hi Christian,
There's a few nitpicks. Note that neither is a blocker so feel free to
send patches in the week(s) to come.
On 23 December 2016 at 22:04, Christian Gmeiner
wrote:
> --- a/configure.ac
> +++ b/configure.ac
> @@ -76,6 +76,7 @@ LIBDRM_NVVIEUX_REQUIRED=2.4.66
> LIBDRM_NOUVEAU_REQUIRE
On Thu, Dec 1, 2016 at 2:09 PM, Ben Widawsky
wrote:
> From: Ben Widawsky
>
> This will be used by clients that need to know the number of planes
> allocated for them on behalf of the GL or other API. The best current
> example of this is when an extra "plane" is allocated to store
> compression
Reviewed-by: Bas Nieuwenhuizen
On Fri, Dec 30, 2016 at 6:22 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This just adds the nir->llvm support, enabling
> the extension causes some failures on llvm 3.9 at least,
> but this code seems fine.
>
> NIR passes the sampler in src[1].x, and we LLVM/SI
Am 07.01.2017 um 19:11 schrieb Marek Olšák:
> On Sat, Jan 7, 2017 at 5:23 PM, Roland Scheidegger wrote:
>> Am 07.01.2017 um 15:51 schrieb Marek Olšák:
>>> From: Marek Olšák
>>>
>>> This fixes broken depth texturing after:
>>>
>>> commit 22639a6e19f95902aef23474ad672bf489231ea7
>>> Author: Timothy
Am 06.01.2017 um 10:42 schrieb Samuel Pitoiset:
> D3D always computes the absolute value while GLSL says that the
That should probably say "d3d9" - it is completely wrong for d3d10 and
later (which have it to be defined as a guaranteed NaN).
(Otherwise, I'm still not quite convinced it's the right
The foreach loop was called both in the else case
and right after. The indentation seems to indicate
that the extra call was from a previous version
with an else section with out curly brackets.
---
Spotted due to the wrong indentation. The patch is untested
as I do not have the neccesary hardware.
On Thu, Jan 05, 2017 at 11:22:51AM -0800, Kenneth Graunke wrote:
> On Tuesday, December 13, 2016 2:50:58 PM PST Rafael Antognolli wrote:
> > Enable the use of a transform feedback overflow query with
> > glBeginConditionalRender. The render commands will only execute if the
> > query is true (i.e.
Ugh, will update the patch with these changes.
Thanks!
Rafael
On Thu, Jan 05, 2017 at 11:17:51AM -0800, Kenneth Graunke wrote:
> On Tuesday, December 13, 2016 2:50:57 PM PST Rafael Antognolli wrote:
> > Enable getting the results of a transform feedback overflow query with a
> > buffer object.
>
On Thu, Jan 05, 2017 at 11:29:39AM -0800, Kenneth Graunke wrote:
> Predication needs cmd parser only on gen7. For newer platforms, it
> should be available without it.
>
> Signed-off-by: Rafael Antognolli
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/intel_screen.c | 2 +-
>
Matt Turner writes:
> On 01/05, Samuel Iglesias Gonsálvez wrote:
>>From: Iago Toral Quiroga
>>
>>4-wide DF operations where NibCtrl applies require and execsize of 8
>>in IvyBridge/Valleyview.
>
> Wow, the documentation is bad in this area. The QtrCtrl description in
> IVB's Vol4 Part3 explicitl
On Wed, Jan 4, 2017 at 8:43 PM, Ben Widawsky wrote:
> v2: Make the error return be -1 instead of 0 because I think 0 is
> actually valid.
>
> Cc: Daniel Stone
> Signed-off-by: Ben Widawsky
> ---
> src/gbm/backends/dri/gbm_dri.c | 28
> src/gbm/gbm-symbols-check
Samuel Iglesias Gonsálvez writes:
> From: "Juan A. Suarez Romero"
>
> In IVB/VLV, for instructions dealing with DF, execsize will be
> duplicated in the final code.
>
> So take this in account when checking if instructions should be split.
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp | 13 +
On Mon, Jan 2, 2017 at 6:36 PM, Ben Widawsky wrote:
> Signed-off-by: Ben Widawsky
> Reviewed-by: Eric Engestrom
> Acked-by: Daniel Stone
> ---
> src/gbm/backends/dri/gbm_dri.c | 26 +-
> src/gbm/gbm-symbols-check | 1 +
> src/gbm/main/gbm.c | 15 +
Samuel Iglesias Gonsálvez writes:
> From: "Juan A. Suarez Romero"
>
> The execution data size is the biggest type size of any instruction
> operand.
>
> We will use it to know if the instruction deals with DF, because in Ivy
> we need to duplicate the execution size and regioning parameters.
> -
On Mon, Jan 9, 2017 at 3:50 PM, Jason Ekstrand wrote:
> On Mon, Jan 2, 2017 at 6:36 PM, Ben Widawsky wrote:
>
>> Signed-off-by: Ben Widawsky
>> Reviewed-by: Eric Engestrom
>> Acked-by: Daniel Stone
>> ---
>> src/gbm/backends/dri/gbm_dri.c | 26 +-
>> src/gbm/gbm-symbo
Samuel Iglesias Gonsálvez writes:
> From: Iago Toral Quiroga
>
> It seems to use 1 channel por DF, just like later hardware. The docs say
> things
> like:
>
> "Each DF operand uses a pair of channels and all masking and swizzling
> should be adjusted appropriately."
>
> "In Align16, all regio
In parse_identifier, it doesn't stop copying '*pcur' untill
encounter the NULL. As the 'ret' has a fixed-size buffer, if
the '*pcur' has a long string, there will be a buffer overflow.
This patch avoid this.
Signed-off-by: Li Qiang
---
src/gallium/auxiliary/tgsi/tgsi_text.c | 6 +++---
1 file ch
In parse_identifier, it doesn't stop copying '*pcur'
untill encounter the NULL. As the 'ret' has a fixed-size
buffer, if the '*pcur' has a long string, there will be
a buffer overflow. This patch avoid this.
Signed-off-by: Li Qiang
---
src/gallium/auxiliary/tgsi/tgsi_text.c | 9 ++---
1 file
Somehow I didn't actually get the original e-mail so I'm replying via
git-send-email...
On 01/02, Ben Widawsky wrote:
> Modifiers will be obtains or guessed by the client and passed in during
> image creation/import.
>
> This requires bumping the DRIimage version.
>
> As of this patch, the modif
---
src/gallium/state_trackers/dri/dri_screen.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/dri/dri_screen.c
b/src/gallium/state_trackers/dri/dri_screen.c
index aa0ad09..a950f52 100644
--- a/src/gallium/state_trackers/dri/dri_screen.c
+++ b/src/g
Hi,
I'm sending a series of 12 patches for android,
comprising fixes for build errors, LLVMInitializeAMDGPU* declarations,
Android 7 fixes and a (small) i915 patch for feature parity with i965.
Tested with nougat-x86 and marshmallow-x86
Mauro
Changelog:
[building errors/trailing whitespaces]
LLVMInitializeAMDGPU* functions need to be explicitly declared
and mesa expects them via header,
but LLVM needs to be instructed to invoke its own LLVM_TARGET(AMDGPU) macro,
or the functions will not be available.
A new llvm cflag (-DFORCE_BUILD_AMDGPU) serves this purpose,
the same mechanism is
Fixes building error due to dependency on nir generated headers
---
src/mesa/Android.libmesa_st_mesa.mk | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/Android.libmesa_st_mesa.mk
b/src/mesa/Android.libmesa_st_mesa.mk
index 3905ddc..92df4ad 100644
--- a/src/mesa/And
LLVMInitializeAMDGPU* functions need to be explicitly declared
and mesa expects them via header,
but LLVM needs to be instructed to invoke its own LLVM_TARGET(AMDGPU) macro,
or the functions will not be available.
A new llvm cflag (-DFORCE_BUILD_AMDGPU) serves this purpose,
the same mechanism is
Fixes building errors due to multiple definition of ffs and ffsll
as they are provided by bionic in kitkat,
in later versions these definitions do no harm.
---
Android.common.mk | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Android.common.mk b/Android.common.mk
index 80cf572..1745b08 10064
Fixes a series of libz related building errors:
target SharedLib: gallium_dri_32
(out/target/prod...SHARED_LIBRARIES/gallium_dri_intermediates/LINKED/gallium_dri.so)
external/elfutils/libelf/elf_compress.c:117: error: undefined reference to
'deflateInit_'
...
external/elfutils/libelf/elf_compress
Removed as unnecessary
---
src/gallium/targets/dri/Android.mk | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/gallium/targets/dri/Android.mk
b/src/gallium/targets/dri/Android.mk
index 1564d9d..9067b32 100644
--- a/src/gallium/targets/dri/Android.mk
+++ b/src/gallium/targets/dri/Android.m
This is similar to commit 7420c9dab4aaf87e6b840410226c296c4668a48f
but for the i915 driver, as neeeded in android-x86.
In version v2, as per i965 implementation,
the extension GL_OES_EGL_image_external is not enabled by default.
The by-the-book way to enable the extension is mesa environment vari
Conditional libLLVMCore static library dependency is added,
for the case when MESA_ENABLE_LLVM is true
Fixes the following building error with Android 7.0:
In file included from
external/mesa/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp:62:
...
external/llvm/include/llvm/IR/Attributes.h:68:14: f
Integration of rules that set correct HAVE_LLVM value for Android 7.0
---
Android.common.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Android.common.mk b/Android.common.mk
index 1745b08..36a44bb 100644
--- a/Android.common.mk
+++ b/Android.common.mk
@@ -80,7 +80,7 @@ endi
LLVMInitializeAMDGPU* functions need to be explicitly declared
and mesa expects them via header,
but LLVM needs to be instructed to invoke its own LLVM_TARGET(AMDGPU) macro,
or the functions will not be available.
A new llvm cflag (-DFORCE_BUILD_AMDGPU) serves this purpose,
the same mechanism is
LOCAL_C_INCLUDES need to include correct path of libelf,
which had exceptions for Android 5.0 (i.e external/elfutils/0.153/libelf)
and Android 6.0 (i.e. external/elfutils/src/libelf)
In Android 7.0 the path is again external/elfutils/libelf
Fixes the following building error:
external/mesa/src/ga
On Mon, Jan 2, 2017 at 6:37 PM, Ben Widawsky wrote:
> The idea behind modifiers like this is that the user of GBM will have
> some mechanism to query what properties the hardware supports for its BO
> or surface. This information is directly passed in (and stored) so that
> the DRI implementation
1 - 100 of 135 matches
Mail list logo
