On 19.04.2017 00:01, Ilia Mirkin wrote:
On Tue, Apr 18, 2017 at 5:26 PM, Nicolai Hähnle wrote:
On 18.04.2017 21:49, Dave Airlie wrote:
On 19 April 2017 at 05:30, Samuel Pitoiset
wrote:
On 04/18/2017 08:14 PM, Nicolai Hähnle wrote:
On 11.04.2017 18:48, Samuel Pitoiset wrote:
Bindles
On 19.04.2017 02:37, Timothy Arceri wrote:
V2: typo know -> known
V3: add security check (Suggested by Nicolai)
Thanks for the update.
Patches 1 & 2:
Reviewed-by: Nicolai Hähnle
---
docs/envvars.html | 3 +++
src/mesa/main/context.c | 6 ++
2 files changed, 9 insertions(+)
diff
Patches 1-3:
Reviewed-by: Nicolai Hähnle
On 18.04.2017 20:46, Emil Velikov wrote:
From: Emil Velikov
If VDPAU is installed in the non-default location, we'll fail to find
the headers and error at build time.
../../src/gallium/include/state_tracker/vdpau_dmabuf.h:37:25: fatal error:
vdpau/v
Thanks! R-b and pushed.
On 15.04.2017 00:53, Dieter Nützel wrote:
Tested-by: Dieter Nützel
On Turks XT (6670)
Dieter
Am 13.04.2017 22:56, schrieb Constantine Kharlamov:
Ported from radeonsi. Testing with Kane&Lynch2 shows ≈1k skipped
updates per
frame on average.
No piglit changes with tes
On Tuesday, April 18, 2017 9:56:14 AM PDT Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> Engine discovery uAPI allows userspace to probe for engine
> configuration and features without needing to maintain the
> internal PCI id based database.
I don't understand why I would want to query the ex
https://bugs.freedesktop.org/show_bug.cgi?id=100708
Michel Dänzer changed:
What|Removed |Added
QA Contact|dri-devel@lists.freedesktop |mesa-dev@lists.freedesktop.
Reviewed-by: Jason Ekstrand
On Tue, Apr 18, 2017 at 4:31 PM, Nanley Chery wrote:
> The section of the PRM mentioned in the code comment above this table
> says that this format supports the render target write message. Internal
> documentation says that this format also supports alpha blending.
On Tue, Apr 18, 2017 at 4:31 PM, Nanley Chery wrote:
> Signed-off-by: Nanley Chery
> ---
> src/intel/isl/isl.c | 5 -
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c
> index f89f351c15..09dafcbab6 100644
> --- a/src/intel/isl/isl.c
Reviewed-by: Jason Ekstrand wrote:
> This field has no users.
>
> Signed-off-by: Nanley Chery
> ---
> src/intel/vulkan/anv_private.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_
> private.h
> index 7d079000b0..dbe1e0475e 100644
>
On Tue, Apr 18, 2017 at 4:41 PM, Nanley Chery wrote:
> According to the Linux kernel sources, the ioctl in anv_gem_mmap() will
> not fail if the size isn't a multiple of 4KB.
>
I was fairly sure that wasn't the case... I'll have to do some looking.
> Signed-off-by: Nanley Chery
> ---
> src/i
Hi Tapani,
You seem to have been one of the last people to touch this code. Do you
think you could take a look at this patch?
Thanks,
Tim
On 13/04/17 12:01, Timothy Arceri wrote:
From: Timothy Arceri
Currently we were only making sure types were the same within a
single stage. This looks t
This fixes the build after:
commit 399ebd2a84a133bd2ca3da388a059fd3bafe33f5
Author: Dave Airlie
Date: Wed Apr 19 06:18:23 2017 +1000
radv/meta: add common shader vertex generation function
Signed-off-by: Mike Lothian
---
src/amd/vulkan/radv_meta.h | 2 +-
1 file changed, 1 insertion(+),
This fixes the build after:
commit 224cf2906a8f38ce47411afc93a223ac0e41795f
Author: Dave Airlie
Date: Mon Apr 17 13:01:52 2017 +1000
radv/ac: add initial pre-pass for shader info gathering
Signed-off-by: Mike Lothian
---
src/amd/common/ac_shader_info.c | 2 +-
1 file changed, 1 insertio
On 19/04/17 11:39, Michel Dänzer wrote:
On 18/04/17 07:35 PM, Nicolai Hähnle wrote:
On 18.04.2017 02:09, Timothy Arceri wrote:
V2: typo know -> known
---
docs/envvars.html | 3 +++
src/mesa/main/context.c | 3 +++
2 files changed, 6 insertions(+)
diff --git a/docs/envvars.html b/docs/
On 18/04/17 07:35 PM, Nicolai Hähnle wrote:
> On 18.04.2017 02:09, Timothy Arceri wrote:
>> V2: typo know -> known
>> ---
>> docs/envvars.html | 3 +++
>> src/mesa/main/context.c | 3 +++
>> 2 files changed, 6 insertions(+)
>>
>> diff --git a/docs/envvars.html b/docs/envvars.html
>> index 65
Anuj Phogat writes:
> From: Ben Widawsky
>
> V2: Squash the changes in one patch and rebased on master (Anuj).
>
> Signed-off-by: Ben Widawsky
> Signed-off-by: Anuj Phogat
> ---
> src/intel/common/gen_l3_config.c | 43
> ++--
> 1 file changed, 37 insertion
On 19/04/17 11:12, Manolova, Plamena wrote:
Thank you for reviewing Timothy!
On Mon, Apr 17, 2017 at 7:10 PM, Timothy Arceri mailto:tarc...@itsqueeze.com>> wrote:
On 18/04/17 11:25, Plamena Manolova wrote:
This extension provides new GLSL built-in functions
beginInvocatio
Thank you for reviewing Nicolai!
On Tue, Apr 18, 2017 at 4:17 AM, Nicolai Hähnle wrote:
> On 18.04.2017 03:25, Plamena Manolova wrote:
>
>> This extension provides new GLSL built-in functions
>> beginInvocationInterlockARB() and endInvocationInterlockARB()
>> that delimit a critical section of f
Thank you for reviewing Timothy!
On Mon, Apr 17, 2017 at 7:10 PM, Timothy Arceri
wrote:
>
>
> On 18/04/17 11:25, Plamena Manolova wrote:
>
>> This extension provides new GLSL built-in functions
>> beginInvocationInterlockARB() and endInvocationInterlockARB()
>> that delimit a critical section of
On 18/04/17 09:52 PM, Ilia Mirkin wrote:
> On Tue, Apr 18, 2017 at 2:21 AM, Michel Dänzer wrote:
>> On 18/04/17 01:02 PM, Ilia Mirkin wrote:
>>> On Tue, Apr 18, 2017 at 12:00 AM, Ilia Mirkin wrote:
val_bool and val_int are in a union. val_bool gets the first byte, which
happens to work
From: Tvrtko Ursulin
Building on top of the previous patch which exported the concept
of engine classes and instances, we can also use this instead of
the current awkward engine selection uAPI.
This is primarily interesting for the VCS engine selection which
is a) currently done via disjoint set
From: Tvrtko Ursulin
Engine discovery uAPI allows userspace to probe for engine
configuration and features without needing to maintain the
internal PCI id based database.
This enables removal of code duplications across userspace
components.
Probing is done via the new DRM_IOCTL_I915_GEM_ENGINE
From: Tvrtko Ursulin
Inspired by the recent introduction of the engine class and instance concept,
and a chat with Chris Wilson about a potential unification of PCI id based
device discovery across multiple userspace components, I have cooked up two
patches to gather some opinions on whether this
We also move _mesa_update_array_format() into the caller.
This gets these functions ready for KHR_no_error support.
V2: Updated function comment as suggested by Brian.
Reviewed-by: Nicolai Hähnle
---
src/mesa/main/varray.c | 71 --
1 file changed
V2: move MESA_VERBOSE checks back into the common code path.
Reviewed-by: Nicolai Hähnle
---
src/mesa/vbo/vbo_exec_array.c | 96 ---
1 file changed, 73 insertions(+), 23 deletions(-)
diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array
The only caller we don't update is update_arrays(), we leave that to the
following commit.
Reviewed-by: Nicolai Hähnle
---
src/mesa/main/varray.c | 136 ++---
1 file changed, 73 insertions(+), 63 deletions(-)
diff --git a/src/mesa/main/varray.c b/src/
V2: restore lost comment, add static to validate_uniform(),
simplify array offset logic.
Reviewed-by: Nicolai Hähnle
---
src/mesa/main/uniform_query.cpp | 73 -
1 file changed, 50 insertions(+), 23 deletions(-)
diff --git a/src/mesa/main/uniform_query
Reviewed-by: Nicolai Hähnle
---
src/mesa/main/varray.c | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c
index 7c46cb3..6da9d84 100644
--- a/src/mesa/main/varray.c
+++ b/src/mesa/main/varray.c
@@ -1783,29 +1783,
V2: add missing FLUSH_CURRENT() to no_error path
Reviewed-by: Nicolai Hähnle
---
src/mesa/vbo/vbo_exec_array.c | 49 ++-
1 file changed, 39 insertions(+), 10 deletions(-)
diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c
index 945
These checks do not generate any errors. Move them so we can add
KHR_no_error support and still make sure we do these checks.
Reviewed-by: Nicolai Hähnle
---
src/mesa/main/api_validate.c | 43 +
src/mesa/vbo/vbo_exec_array.c | 106 ++
2 f
Reviewed-by: Nicolai Hähnle
---
src/mesa/main/varray.c | 352 +
1 file changed, 236 insertions(+), 116 deletions(-)
diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c
index be5482d..7c46cb3 100644
--- a/src/mesa/main/varray.c
+++ b/src/me
V2: typo know -> known
V3: add security check (Suggested by Nicolai)
---
docs/envvars.html | 3 +++
src/mesa/main/context.c | 6 ++
2 files changed, 9 insertions(+)
diff --git a/docs/envvars.html b/docs/envvars.html
index 6537365..a064f56 100644
--- a/docs/envvars.html
+++ b/docs/envvar
This will be used for adding KHR_no_error support.
Reviewed-by: Nicolai Hähnle
---
src/mesa/main/varray.c | 55 +-
1 file changed, 41 insertions(+), 14 deletions(-)
diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c
index 233dc0d..65734d
Reviewed-by: Nicolai Hähnle
---
src/mesa/main/context.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/mesa/main/context.h b/src/mesa/main/context.h
index 9704a96..ccb5463 100644
--- a/src/mesa/main/context.h
+++ b/src/mesa/main/context.h
@@ -317,20 +317,27 @@ _mesa_is_gles31(cons
This will help us split array validation from array update.
V2: add const to ctx param
Reviewed-by: Nicolai Hähnle
---
src/mesa/main/varray.c | 31 ---
1 file changed, 20 insertions(+), 11 deletions(-)
diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c
ind
V3:
- add security check to patch 2
- include patch 1 which didn't get sent out in V2
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
---
src/mesa/main/extensions_table.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa/main/extensions_table.h
index d11cb0f..dc735c5 100644
--- a/src/mesa/main/extensions_table.h
+++ b/src/mesa/main/extensions_table.h
@@ -296,20 +296,21 @@ EXT(IBM_tex
Reviewed-by: Nicolai Hähnle
---
src/mesa/main/getstring.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/mesa/main/getstring.c b/src/mesa/main/getstring.c
index 6e90511..5da405d 100644
--- a/src/mesa/main/getstring.c
+++ b/src/mesa/main/getstring.c
@@ -297,17 +297,28 @@ inva
Rob Clark writes:
> Signed-off-by: Rob Clark
I don't know too much about compute/ssbo, but the logic seems to
basically match the VS/FS cases.
Reviewed-by: Eric Anholt
signature.asc
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lis
On Mon, Apr 17, 2017 at 11:57 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> We want to expose more descriptor sets to the applications,
> but currently we have a 1:1 mapping between shader descriptor
> sets and 2 user sgprs, limiting us to 4 per stage. This commit
> check if we don't have enough
v2: Handle out of pool memory error.
v3: Actually use VK_ERROR_OUT_OF_POOL_MEMORY_KHR for the error condition.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_descriptor_set.c | 69 +++-
src/amd/vulkan/radv_private.h| 4 +++
2 files changed, 57 i
Nice cleanup.
Reviewed-by: Bas Nieuwenhuizen
On Wed, Apr 19, 2017 at 1:27 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> Some of the shaders could just generate the vertex data in the
> shader, so add helpers to allow us to move to doing that.
>
> Signed-off-by: Dave Airlie
> ---
> src/amd/v
This field has no users.
Signed-off-by: Nanley Chery
---
src/intel/vulkan/anv_private.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h
index 7d079000b0..dbe1e0475e 100644
--- a/src/intel/vulkan/anv_private.h
+++ b/src/intel/vulk
According to the Linux kernel sources, the ioctl in anv_gem_mmap() will
not fail if the size isn't a multiple of 4KB.
Signed-off-by: Nanley Chery
---
src/intel/vulkan/anv_image.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vul
The section of the PRM mentioned in the code comment above this table
says that this format supports the render target write message. Internal
documentation says that this format also supports alpha blending. As a
side effect, this allows CCS_D buffers to be created for images with
this format.
Si
Signed-off-by: Nanley Chery
---
src/intel/isl/isl.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c
index f89f351c15..09dafcbab6 100644
--- a/src/intel/isl/isl.c
+++ b/src/intel/isl/isl.c
@@ -1573,8 +1573,11 @@ isl_surf_get_ccs_sur
With this we don't have any operations on a pool with non-freeable
descriptors left that have O(#descriptors) complexity.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_descriptor_set.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_descrip
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_descriptor_set.c | 16
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git a/src/amd/vulkan/radv_descriptor_set.c
b/src/amd/vulkan/radv_descriptor_set.c
index 48cb8c2a30a..26f89152d52 100644
--- a/src/amd/vulkan/r
In cases where it is used it is always 1.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_descriptor_set.c | 21 ++---
src/amd/vulkan/radv_descriptor_set.h | 1 -
src/amd/vulkan/radv_private.h| 1 -
3 files changed, 10 insertions(+), 13 deletions(-)
diff --git
v2: Handle out of pool memory error.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_descriptor_set.c | 69 +++-
src/amd/vulkan/radv_private.h| 4 +++
2 files changed, 57 insertions(+), 16 deletions(-)
diff --git a/src/amd/vulkan/radv_descriptor
From: Dave Airlie
For depth clears we have to pass the depth in the 2nd
component, we can use push constants for some of this
later to drop the vertex buffer completely
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta.c | 9 --
src/amd/vulkan/radv_meta.h | 2 +-
src/am
From: Dave Airlie
Generate the position vertices.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_blit2d.c | 35 +++
1 file changed, 7 insertions(+), 28 deletions(-)
diff --git a/src/amd/vulkan/radv_meta_blit2d.c
b/src/amd/vulkan/radv_meta_blit2d.c
ind
From: Dave Airlie
The vs vertex generate and fs noop shaders are used in a few places,
so refactor them out.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta.c| 35 ++
src/amd/vulkan/radv_meta.h| 2 ++
src/amd/vulkan/radv_meta_decompr
From: Dave Airlie
This generates the position info using the vertex shader.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_blit.c | 67 ++---
1 file changed, 16 insertions(+), 51 deletions(-)
diff --git a/src/amd/vulkan/radv_meta_blit.c b/src/amd/v
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_resolve.c | 74 --
1 file changed, 7 insertions(+), 67 deletions(-)
diff --git a/src/amd/vulkan/radv_meta_resolve.c
b/src/amd/vulkan/radv_meta_resolve.c
index 52f7246..70dcc30 100644
From: Dave Airlie
Some of the shaders could just generate the vertex data in the
shader, so add helpers to allow us to move to doing that.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta.c | 39 +++
src/amd/vulkan/radv_meta.h | 7 +++
2 files ch
From: Dave Airlie
Instead of passing in the same 1.0, -1.0 combinations via
vertex buffers, we can just use vertex id to have the vertex
shader build them. This function introduces the generator
code needed, later patches will use this.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta.c
From: Dave Airlie
This removes the vertex buffer, and just generates the values
in the shader.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_decompress.c | 75 +++
1 file changed, 6 insertions(+), 69 deletions(-)
diff --git a/src/amd/vulkan/radv_meta_
From: Dave Airlie
Avoids having to setup vertex buffers.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_fast_clear.c | 74 +++
1 file changed, 6 insertions(+), 68 deletions(-)
diff --git a/src/amd/vulkan/radv_meta_fast_clear.c
b/src/amd/vulkan/radv_me
On 18/04/17 20:48, Nicolai Hähnle wrote:
On 18.04.2017 02:09, Timothy Arceri wrote:
V2: some bug fixes and suggestions addressed as suggested by Brian and
Nicolai.
This series adds some initial support for the KHR_no_error.
For now it can only be enabled with the MESA_NO_ERROR environmen
Thanks for making the changes I requested.
Reviewed-by: Dylan Baker
Quoting Rafael Antognolli (2017-04-18 14:50:20)
> If the 'dwords' dict is empty, max(dwords.keys()) throws an exception.
> This case could happen when we have an instruction that is only an array
> of other structs, with variabl
2017-04-18 8:56 GMT+02:00 Andres Gomez :
> Mauro, is it OK that I mention you as a reference for Android in the
> releasing doc?
>
Looking at my, somehow limited, coding skills, I think that my role would
be defined better
as an active android-x86 debugging contributor/build tester, with
experien
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/freedreno_context.h | 14 +++-
src/gallium/drivers/freedreno/freedreno_draw.c| 47
src/gallium/drivers/freedreno/freedreno_gmem.c| 7 ++
src/gallium/drivers/freedreno/freedreno_gmem.h| 1 +
src/gallium/drivers/f
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/disasm.h | 1 +
.../drivers/freedreno/ir3/ir3_compiler_nir.c | 81 +-
src/gallium/drivers/freedreno/ir3/ir3_shader.c | 76 +++-
src/gallium/drivers/freedreno/ir3/ir3_shader.h
Hidden under FD_MESA_DEBUG=compute flag for now. In particular because
we completely lie about having image support.
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/Makefile.sources | 2 +
src/gallium/drivers/freedreno/a5xx/fd5_compute.c | 175 +++
src/gallium
Sync to the same files from freedreno.git to correct decoding of ldgb/
stgb instructions.
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/a2xx/disasm-a2xx.c | 2 +-
src/gallium/drivers/freedreno/disasm.h | 3 +
src/gallium/drivers/freedreno/ir3/disasm-a3xx.c | 83
TODO cwabbott pointed out a write-after-read hazzard, which effects both
this and arrays. A write needs to depend on *all* reads since the last
write, not just the last read.
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/ir3/ir3.c| 74 +++-
src/gallium/drivers/free
To simplify things for now, since all the gfx shader stages share a
single SSBO state block, only advertise SSBO support for fragment shader
(and compute when we have that). We could possibly use a fixed-
partitioning of the SSBO index space to support SSBOs on other stages
without having to resor
The generation-independent support for binding shader buffer objects.
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/freedreno_context.h | 8
src/gallium/drivers/freedreno/freedreno_draw.c | 6 +++
src/gallium/drivers/freedreno/freedreno_resource.c | 9
src/gallium/
Signed-off-by: Rob Clark
---
src/mesa/state_tracker/st_glsl_to_nir.cpp | 14 ++
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 3 ++-
src/mesa/state_tracker/st_program.c| 13 +
src/mesa/state_tracker/st_program.h| 3 +++
4 files changed, 32 insertions(+),
This is equivalent to what mesa/st does in glsl_to_tgsi. For most hw
there isn't a particularly good reason to treat these differently.
Signed-off-by: Rob Clark
---
src/compiler/Makefile.sources| 1 +
src/compiler/nir/nir.h | 1 +
src/compiler/nir/nir_l
Here is an early version of basic compute shader for a5xx. I've already
pushed a bunch of cleanup/prep work for this series. In the later driver
specific parts of this series I am still fixing a couple things. And
I completely lie about image support (required for ARB_compute_shader),
so it is h
On Tue, Apr 18, 2017 at 5:26 PM, Nicolai Hähnle wrote:
> On 18.04.2017 21:49, Dave Airlie wrote:
>>
>> On 19 April 2017 at 05:30, Samuel Pitoiset
>> wrote:
>>>
>>>
>>>
>>> On 04/18/2017 08:14 PM, Nicolai Hähnle wrote:
On 11.04.2017 18:48, Samuel Pitoiset wrote:
>
>
> Bi
'start' parameter from Group.emit_pack_function() is useless.
Signed-off-by: Rafael Antognolli
---
src/intel/genxml/gen_pack_header.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/intel/genxml/gen_pack_header.py
b/src/intel/genxml/gen_pack_header.py
index 2a70945.
We need to emit BLEND_STATE, which size is 1 + 2 * nr_draw_buffers
dwords (on gen8+), but the BLEND_STATE struct length is always 17. By
marking it size 1, which is actually the size of the struct minus the
BLEND_STATE_ENTRY's, we can emit a BLEND_STATE of variable number of
entries.
For gen6 and
If the 'dwords' dict is empty, max(dwords.keys()) throws an exception.
This case could happen when we have an instruction that is only an array
of other structs, with variable length.
v2:
- Add another clause for empty dwords and make it work with python 3
(Dylan)
- Set the length to 0 if
On 18.04.2017 21:49, Dave Airlie wrote:
On 19 April 2017 at 05:30, Samuel Pitoiset wrote:
On 04/18/2017 08:14 PM, Nicolai Hähnle wrote:
On 11.04.2017 18:48, Samuel Pitoiset wrote:
Bindless sampler/image types are really different from the existing
sampler/image types. They are considered
On Tue, Apr 18, 2017 at 05:56:15PM +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> Building on top of the previous patch which exported the concept
> of engine classes and instances, we can also use this instead of
> the current awkward engine selection uAPI.
>
> This is primarily intere
On Tue, Apr 18, 2017 at 05:56:14PM +0100, Tvrtko Ursulin wrote:
> +enum drm_i915_gem_engine_class {
> + DRM_I915_ENGINE_CLASS_OTHER = 0,
> + DRM_I915_ENGINE_CLASS_RENDER = 1,
> + DRM_I915_ENGINE_CLASS_COPY = 2,
> + DRM_I915_ENGINE_CLASS_VIDEO_DECODE = 3,
> + DRM_I915_ENGINE_CLAS
From: Dave Airlie
In practice this will probably just drop draw id in a few places.
v2: just do draw_id for now. (Bas)
it might be possible to do something more if we need it in the
future. (nha)
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 26 ++
On 19 April 2017 at 05:30, Samuel Pitoiset wrote:
>
>
> On 04/18/2017 08:14 PM, Nicolai Hähnle wrote:
>>
>> On 11.04.2017 18:48, Samuel Pitoiset wrote:
>>>
>>> Bindless sampler/image types are really different from the existing
>>> sampler/image types. They are considered 64-bit unsigned integers,
Emil Velikov writes:
> Without it the compiler will barf at us with dozens of errors like
> the following.
>
> In file included from llvm/codegen/bitcode.cpp:34:0:
> ./llvm/codegen.hpp:52:22: error: ‘<::’ cannot begin a template-argument list
> [-fpermissive]
> ./llvm/codegen.hpp:52:22: note: ‘<
On 04/18/2017 08:10 PM, Nicolai Hähnle wrote:
On 11.04.2017 18:48, Samuel Pitoiset wrote:
The ARB_bindless_texture spec says:
"Modify Section 5.4.1, Conversion and Scalar Constructors, p. 60"
(add the following constructors:)
// In the following four constructors, the low 32 bi
On 04/18/2017 08:14 PM, Nicolai Hähnle wrote:
On 11.04.2017 18:48, Samuel Pitoiset wrote:
Bindless sampler/image types are really different from the existing
sampler/image types. They are considered 64-bit unsigned integers,
they can be declared as temporary, shader inputs/outputs and are
non-
From: Emil Velikov
Resolves build issues like the following:
src/gallium/winsys/sw/dri/dri_sw_winsys.c:203:31: error: pointer of type ‘void
*’ used in arithmetic [-Werror=pointer-arith]
data = dri_sw_dt->data + (dri_sw_dt->stride * box->y) + box->x * blsize;
Without it the compiler will barf at us with dozens of errors like
the following.
In file included from llvm/codegen/bitcode.cpp:34:0:
./llvm/codegen.hpp:52:22: error: ‘<::’ cannot begin a template-argument list
[-fpermissive]
./llvm/codegen.hpp:52:22: note: ‘<:’ is an alternate spelling for ‘[’.
Fixes: 1ac40173c2a ("configure.ac: simplify EGL requirements for drivers
dependent on EGL")
Signed-off-by: Emil Velikov
---
configure.ac | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index e1c60d842b5..957d15df8ca 100644
--- a/configure.ac
+++
From: Emil Velikov
If VDPAU is installed in the non-default location, we'll fail to find
the headers and error at build time.
../../src/gallium/include/state_tracker/vdpau_dmabuf.h:37:25: fatal error:
vdpau/vdpau.h: No such file or directory
#include
^
Fixes: faba96b
Before this commit, when a group with count="0" is found, only one field
is added to the struct representing the instruction. This causes only
one entry to be printed by aubinator, for variable length groups.
With this commit we "detect" that there's a variable length group
(count="0") and store t
On Fri, Apr 14, 2017 at 12:47 PM, Eric Anholt wrote:
> Android.mk was setting the flag across the entire driver, so we didn't
> have non-NEON versions getting built. This was going to be a problem with
> the next commit, when I start auto-detecting NEON support and use the
> non-NEON version when
On 11.04.2017 18:48, Samuel Pitoiset wrote:
This also adds the extension to the standalone GLSL compiler.
Signed-off-by: Samuel Pitoiset
Patches 1 & 2:
Reviewed-by: Nicolai Hähnle
---
src/compiler/glsl/glsl_parser_extras.cpp | 1 +
src/compiler/glsl/glsl_parser_extras.h | 7 ++
On 18.04.2017 18:57, Marek Olšák wrote:
We also have asynchronous compilation for optimized shaders (linking
separate shaders etc.) using the same threaded queue as the initial
compilations, and we can't let apps suppress the optimized ones.
Good point. I think if the app says it doesn't want p
On 11.04.2017 18:48, Samuel Pitoiset wrote:
Bindless sampler/image types are really different from the existing
sampler/image types. They are considered 64-bit unsigned integers,
they can be declared as temporary, shader inputs/outputs and are
non-opaque types.
For these reasons, it looks more c
On 11.04.2017 18:48, Samuel Pitoiset wrote:
The ARB_bindless_texture spec says:
"Modify Section 5.4.1, Conversion and Scalar Constructors, p. 60"
(add the following constructors:)
// In the following four constructors, the low 32 bits of the sampler
// type correspond to the
On 11.04.2017 18:48, Samuel Pitoiset wrote:
The ARB_bindless_texture spec says:
"If both bindless_sampler and bound_sampler, or bindless_image
and bound_image, are declared at global scope in any
compilation unit, a link- time error will be generated."
Signed-off-by: Samuel Pitoiset
We also have asynchronous compilation for optimized shaders (linking
separate shaders etc.) using the same threaded queue as the initial
compilations, and we can't let apps suppress the optimized ones.
Marek
On Apr 18, 2017 1:08 PM, "Nicolai Hähnle" wrote:
> On 17.04.2017 14:07, Edward O'Callag
For 1-3:
Reviewed-by: Marek Olšák
Marek
On Apr 18, 2017 3:35 PM, "Nicolai Hähnle" wrote:
> From: Nicolai Hähnle
>
> When any count[i] is negative, we must skip all draws.
>
> Moving to vbo makes the subsequent change easier.
>
> v2:
> - provide the function in all contexts, including GLES
>
https://bugs.freedesktop.org/show_bug.cgi?id=100668
--- Comment #2 from Brian Paul ---
Hi Charles,
I compiled your test program and ran it with Mesa in a Windows VM and
wglChoosePixelFormatARB failed to find a pixel format. But I found the exact
same failure when I ran it with NVIDIA's OpenGL d
That should be fine. If nothing else, I can help route questions to
appropriate folks. =)
Reviewed-by: Jeremy Sequoia
> On Apr 18, 2017, at 02:34, Andres Gomez wrote:
>
> Jeremy, is it OK that I mention you as a reference for MacOSX in the
> releasing doc?
>
> On Mon, 2017-03-27 at 21:48 +0
On 18 April 2017 at 16:16, Andres Gomez wrote:
> On Mon, 2017-04-10 at 12:15 +0200, Juan A. Suarez Romero wrote:
>> On Fri, 2017-04-07 at 19:38 +0100, Emil Velikov wrote:
>> > On 7 April 2017 at 12:30, Juan A. Suarez Romero
>> > wrote:
>> > > Both scripts does not use a file with the commits to
1 - 100 of 143 matches
Mail list logo
