On 29.01.2016 21:36, Daniel Stone wrote:
> On 29 January 2016 at 03:44, Michel Dänzer wrote:
>> It still sounds like significant work (particularly for somebody like me
>> who isn't very familiar with Wayland details yet). It should be done by
>> somebody who cares about the difference you're desc
On 31.01.2016 19:30, Emil Velikov wrote:
> On 25 January 2016 at 11:24, Michel Dänzer wrote:
>> On 23.01.2016 02:14, Marek Olšák wrote:
>>> On Fri, Jan 22, 2016 at 6:02 PM, Emil Velikov
>>> wrote:
On 22 January 2016 at 16:50, Marek Olšák wrote:
> On Fri, Jan 22, 2016 at 5:32 PM, Emil V
On 30.01.2016 06:18, Marek Olšák wrote:
> From: Marek Olšák
Patches 1, 2 & 4 are
Reviewed-by: Michel Dänzer
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
___
On 30.01.2016 06:18, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/gallium/drivers/radeon/r600_pipe_common.c | 3 ++-
> src/gallium/drivers/radeon/radeon_winsys.h| 1 -
> src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 15 ---
> src/gallium/winsys/radeon/drm/
On 30.01.2016 06:18, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c
> b/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c
> inde
Am 01.02.2016 um 02:00 schrieb srol...@vmware.com:
> From: Roland Scheidegger
>
> If the tri is fully inside a scissor edge (or rather, we just use the
> bounding box of the tri for the comparison), then we can drop these
> additional scissor "planes" early. We do not even need to allocate
> spac
On Sun, Jan 31, 2016 at 7:29 PM, Dave Airlie wrote:
> On 30 January 2016 at 07:00, Ilia Mirkin wrote:
>> We use this logic to detect live ranges and then do plain renaming
>> across the whole codebase. As such, to prevent WaW hazards, we have to
>> treat a write as if it were also a read.
>>
>> F
From: Roland Scheidegger
If the tri is fully inside a scissor edge (or rather, we just use the
bounding box of the tri for the comparison), then we can drop these
additional scissor "planes" early. We do not even need to allocate
space for them in the tri.
The math actually appears to be slightly
From: Roland Scheidegger
Just slightly simpler assembly.
---
src/gallium/drivers/llvmpipe/lp_setup_tri.c | 11 +--
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/llvmpipe/lp_setup_tri.c
b/src/gallium/drivers/llvmpipe/lp_setup_tri.c
index 907129d..1e3a7
From: Roland Scheidegger
When we switched to 64bit rasterization, we could no longer use straight
aligned loads for loading the plane data. However, what the code actually
does for loading 3 planes, is 12 scalar loads + 9 unpacks, and then there's
another 8 unpacks for the transpose we need (!).
On 30 January 2016 at 07:00, Ilia Mirkin wrote:
> We use this logic to detect live ranges and then do plain renaming
> across the whole codebase. As such, to prevent WaW hazards, we have to
> treat a write as if it were also a read.
>
> For example, the following sequence was observed before this
Hi
Am 30/01/2016 um 16:50 schrieb Marek Olšák:
> From: Marek Olšák
>
> ---
> src/mesa/state_tracker/st_atom_shader.c | 18 +---
> src/mesa/state_tracker/st_program.c | 164
> +---
> src/mesa/state_tracker/st_program.h | 17 +---
> 3 files changed, 28 inser
On Sun, Jan 31, 2016 at 5:24 PM, Rob Herring wrote:
> On Fri, Jan 29, 2016 at 4:37 PM, Ilia Mirkin wrote:
>> On Fri, Jan 29, 2016 at 5:18 PM, Rob Herring wrote:
>>> On Fri, Jan 29, 2016 at 3:46 PM, Ilia Mirkin wrote:
On Fri, Jan 29, 2016 at 4:41 PM, Rob Herring wrote:
> On Fri, Jan 29
On Fri, Jan 29, 2016 at 4:37 PM, Ilia Mirkin wrote:
> On Fri, Jan 29, 2016 at 5:18 PM, Rob Herring wrote:
>> On Fri, Jan 29, 2016 at 3:46 PM, Ilia Mirkin wrote:
>>> On Fri, Jan 29, 2016 at 4:41 PM, Rob Herring wrote:
On Fri, Jan 29, 2016 at 3:05 PM, Ilia Mirkin wrote:
> Both for this
On Sun, Jan 31, 2016 at 4:23 AM, Emil Velikov wrote:
> On 29 January 2016 at 20:52, Rob Herring wrote:
>> Android builds with -Wunused-parameter enabled which results in spewing
>> lots of warnings. Disable it so more meaningful warnings are more visible.
>>
> Would be great if one day we resolve
ping? I haven't heard anything other than a suggestion that GLES2 and
GLES3 should be merged (which is something that ought to be done
separately from this change)...
On Fri, Jan 22, 2016 at 1:46 PM, Ilia Mirkin wrote:
> On Thu, Jan 21, 2016 at 11:58 AM, Ilia Mirkin wrote:
>> The EXT spec has be
On Sun, Jan 31, 2016 at 4:17 AM, Emil Velikov wrote:
> On 29 January 2016 at 20:52, Rob Herring wrote:
>> Define the DEFAULT_DRIVER_DIR for arm64 builds.
>>
>> Cc: Emil Velikov
> Reviewed-by: Emil Velikov
>
>> Cc: Chih-Wei Huang
>> Signed-off-by: Rob Herring
>> ---
>> src/egl/Android.mk | 1
On Sun, Jan 31, 2016 at 3:32 PM, Ilia Mirkin wrote:
> This is supposed to return GL_TRUE or GL_FALSE, not the full counter
> value.
>
> Signed-off-by: Ilia Mirkin
> ---
Errr... hm. Just realized many drivers will fail at this. Oh well,
I'll drop it for now.
> src/mesa/state_tracker/st_cb_query
On Sun, Jan 31, 2016 at 3:36 PM, Dave Airlie wrote:
> On 1 February 2016 at 06:34, Ilia Mirkin wrote:
>> On Sun, Jan 31, 2016 at 3:26 PM, Dave Airlie wrote:
>>> On 31 January 2016 at 23:39, Ilia Mirkin wrote:
On Sun, Jan 31, 2016 at 2:37 AM, Dave Airlie wrote:
>> + for (i = 0; i < p
On 1 February 2016 at 06:34, Ilia Mirkin wrote:
> On Sun, Jan 31, 2016 at 3:26 PM, Dave Airlie wrote:
>> On 31 January 2016 at 23:39, Ilia Mirkin wrote:
>>> On Sun, Jan 31, 2016 at 2:37 AM, Dave Airlie wrote:
> + for (i = 0; i < program->shader->NumImages; i++) {
> + if (program-
On Sun, Jan 31, 2016 at 3:26 PM, Dave Airlie wrote:
> On 31 January 2016 at 23:39, Ilia Mirkin wrote:
>> On Sun, Jan 31, 2016 at 2:37 AM, Dave Airlie wrote:
+ for (i = 0; i < program->shader->NumImages; i++) {
+ if (program->images_used & (1 << i)) {
+ t->images[i]
From: Rafal Mielniczuk
Add QueryBuffer and initialise it to NullBufferObj on start
Signed-off-by: Rafal Mielniczuk
[imirkin: also release QueryBuffer on free]
Signed-off-by: Ilia Mirkin
---
src/mesa/main/bufferobj.c | 14 ++
src/mesa/main/mtypes.h| 2 ++
2 files changed, 16 i
This is supposed to return GL_TRUE or GL_FALSE, not the full counter
value.
Signed-off-by: Ilia Mirkin
---
src/mesa/state_tracker/st_cb_queryobj.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_cb_queryobj.c
b/src/mesa/state_tracker/st_cb_queryob
Signed-off-by: Ilia Mirkin
---
docs/GL3.txt | 2 +-
docs/relnotes/11.2.0.html | 1 +
src/gallium/drivers/nouveau/nvc0/mme/com9097.mme | 50 ++
src/gallium/drivers/nouveau/nvc0/mme/com9097.mme.h | 34 +++
src/galliu
Signed-off-by: Ilia Mirkin
---
src/mesa/state_tracker/st_cb_bufferobjects.c | 3 +
src/mesa/state_tracker/st_cb_queryobj.c | 97 ++-
src/mesa/state_tracker/st_cb_texturebarrier.c | 3 +
src/mesa/state_tracker/st_extensions.c| 1 +
4 files changed, 103 ins
Signed-off-by: Ilia Mirkin
---
src/gallium/docs/source/context.rst | 5 +
src/gallium/include/pipe/p_context.h | 22 ++
src/gallium/include/pipe/p_defines.h | 10 ++
3 files changed, 37 insertions(+)
diff --git a/src/gallium/docs/source/context.rst
b/src/galliu
From: Rafal Mielniczuk
Signed-off-by: Rafal Mielniczuk
[imirkin: move definition to gl_API.xml as it is very short]
Signed-off-by: Ilia Mirkin
---
src/mapi/glapi/gen/gl_API.xml | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/
Forwards query result writes to drivers.
Signed-off-by: Ilia Mirkin
---
src/mesa/main/queryobj.c | 280 ++-
1 file changed, 133 insertions(+), 147 deletions(-)
diff --git a/src/mesa/main/queryobj.c b/src/mesa/main/queryobj.c
index 9836685..92d8f9a 100
Signed-off-by: Ilia Mirkin
---
src/gallium/docs/source/screen.rst | 2 ++
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
src/gallium/drivers/ilo/ilo_screen.c | 1 +
src/gallium/drivers/llvmpipe/lp_screen.c | 1 +
src/gallium/drivers/nouveau/nv30/nv30_scr
From: Rafal Mielniczuk
Signed-off-by: Rafal Mielniczuk
[imirkin: add string to extensions.c]
Signed-off-by: Ilia Mirkin
---
src/mesa/main/extensions_table.h | 1 +
src/mesa/main/mtypes.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa
Signed-off-by: Ilia Mirkin
---
src/mesa/main/dd.h | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/mesa/main/dd.h b/src/mesa/main/dd.h
index d4378e5..cfe0a82 100644
--- a/src/mesa/main/dd.h
+++ b/src/mesa/main/dd.h
@@ -726,6 +726,15 @@ struct dd_function_table {
void (*EndQuer
From: Rafal Mielniczuk
Signed-off-by: Rafal Mielniczuk
[imirkin: move to GL/GL_CORE section]
Signed-off-by: Ilia Mirkin
---
src/mesa/main/get.c | 5 +
src/mesa/main/get_hash_params.py | 3 +++
2 files changed, 8 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/g
I believe I've addressed all the feedback from last time (except
Marek's suggestion to use clear_buffer instead of an inline write for
QUERY_TARGET support as I don't think that's an improvement), as well
as adding an nvc0 implementation. It passes the test that Glenn wrote
a while back:
http://pa
On 31 January 2016 at 23:39, Ilia Mirkin wrote:
> On Sun, Jan 31, 2016 at 2:37 AM, Dave Airlie wrote:
>>> + for (i = 0; i < program->shader->NumImages; i++) {
>>> + if (program->images_used & (1 << i)) {
>>> + t->images[i] = ureg_DECL_image(ureg, i,
>>> +
From: Rob Clark
Signed-off-by: Rob Clark
---
src/compiler/Makefile.sources| 1 +
src/compiler/nir/nir.h | 12 +
src/compiler/nir/nir_lower_wpos_ytransform.c | 317 +++
3 files changed, 330 insertions(+)
create mode 100644 src/com
From: Rob Clark
Signed-off-by: Rob Clark
---
src/compiler/Makefile.sources | 1 +
src/compiler/nir/nir.h | 13 ++
src/compiler/nir/nir_builder.h | 2 +-
src/compiler/nir/nir_lower_drawpixels.c | 252
4 files changed, 267
From: Rob Clark
NIR equivalent of tgsi_emulate
Signed-off-by: Rob Clark
---
src/gallium/auxiliary/Makefile.sources | 2 +
src/gallium/auxiliary/nir/nir_emulate.c | 177
src/gallium/auxiliary/nir/nir_emulate.h | 34 ++
3 files changed, 213 insertions(+)
From: Rob Clark
Signed-off-by: Rob Clark
---
src/compiler/nir/nir.h | 2 +
src/mesa/Makefile.sources | 1 +
src/mesa/state_tracker/st_glsl_to_nir.cpp | 407 +
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 40 ++-
src/mesa/sta
From: Rob Clark
We'll need this for a nir pass to lower builtin-uniform access.
Signed-off-by: Rob Clark
---
src/compiler/glsl/builtin_variables.cpp | 24 +++-
src/compiler/glsl/ir.h | 3 +++
2 files changed, 18 insertions(+), 9 deletions(-)
diff --git a/
From: Rob Clark
Signed-off-by: Rob Clark
---
src/mesa/Makefile.sources | 2 +
src/mesa/state_tracker/st_nir.h | 28
src/mesa/state_tracker/st_nir_lower_builtin.c | 207 ++
3 files changed, 237 insertions(+)
create mode 100644 sr
The goal is to allow the pipe driver to request something other than
TGSI, but detect whether what is getting is TGSI vs what it requested.
The pipe drivers will always have to support TGSI (and convert that into
whatever it is that they prefer), but in some cases we should be able to
skip the TGSI
From: Rob Clark
Signed-off-by: Rob Clark
---
src/compiler/Makefile.sources | 1 +
src/compiler/nir/nir.h | 7 ++
src/compiler/nir/nir_lower_bitmap.c | 166
3 files changed, 174 insertions(+)
create mode 100644 src/compiler/nir/nir_low
From: Rob Clark
Signed-off-by: Rob Clark
---
src/gallium/include/pipe/p_defines.h | 1 +
src/gallium/include/pipe/p_state.h | 6 ++
2 files changed, 7 insertions(+)
diff --git a/src/gallium/include/pipe/p_defines.h
b/src/gallium/include/pipe/p_defines.h
index 3a1c691..1365d66 100644
---
From: Rob Clark
Currently the vec4 version of this is duplicated in brw and mesa/st
(although the mesa/st version handles 64b types). There is also a
scalar version floating around in brw. We should consolidate all of
these.
Signed-off-by: Rob Clark
---
src/compiler/nir_types.cpp
From: Rob Clark
With TGSI, the ir_variable::data.location gets fixed up to be a stage
local location (rather than program global). In this case we need to
skip the UniformStorage[location] lookup.
Signed-off-by: Rob Clark
---
src/compiler/nir/nir_lower_samplers.c | 23 ---
From: Rob Clark
Ok, v3 of something I've been working on here and there for a while..
Since the last time I've sent this patchset, I've added support for
lowering drawpix/bitmap in NIR, as well as a NIR equiv to tgsi_emulate.
In addition, I've added a pass to lower builtin uniforms, for reasons
On Sun, Jan 31, 2016 at 6:18 AM, Emil Velikov wrote:
> Hi Rob,
>
> On 30 January 2016 at 00:36, Rob Herring wrote:
>> It is necessary to share the screen between mesa and gralloc to
>> properly ref count resources. This implements a hash lookup on
>> the file description to re-use an already crea
Look after _mesa_meta_setup_sampler() return value.
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/drivers/common/meta_blit.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/common/meta_blit.c
b/src/mesa/drivers/common/meta_blit.c
index 5d80f7d..7e4c6cb 100644
--- a/src/
On Sun, Jan 31, 2016 at 1:25 AM, Edward O'Callaghan
wrote:
> Handle the case of ARB_framebuffer_no_attachment.
> Also, kill off a dead debug printf() call while we are here.
> ---
> src/mesa/state_tracker/st_atom_framebuffer.c | 8 +---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> di
On Sun, Jan 31, 2016 at 1:25 AM, Edward O'Callaghan
wrote:
> Here we store the number of samples and layers directly in the
> pipe_framebuffer_state so that in the case of
> ARB_framebuffer_no_attachment we may make use of them directly.
>
> Further, we adjust various gallium/auxiliary helper func
Reviewed-by: Ilia Mirkin
On Sun, Jan 31, 2016 at 1:25 AM, Edward O'Callaghan
wrote:
> Change references to gl_framebuffer::Width, Height, MaxNumLayers
> and Visual::samples to use the _mesa_geometric_ convenience functions
> for those places where the geometry of the gl_framebuffer is needed.
>
This seems odd. When would this not just be MaxArrayTextureLayers? IMO
a plain enable cap, like PIPE_CAP_FRAMEBUFFER_NO_ATTACHMENTS makes
more sense here.
On Sun, Jan 31, 2016 at 1:25 AM, Edward O'Callaghan
wrote:
> diff --git a/src/mesa/state_tracker/st_extensions.c
> b/src/mesa/state_tracker/s
On Sun, Jan 31, 2016 at 2:37 AM, Dave Airlie wrote:
>> + for (i = 0; i < program->shader->NumImages; i++) {
>> + if (program->images_used & (1 << i)) {
>> + t->images[i] = ureg_DECL_image(ureg, i,
>> +program->image_targets[i],
>> +
Hi Rob,
On 30 January 2016 at 00:36, Rob Herring wrote:
> It is necessary to share the screen between mesa and gralloc to
> properly ref count resources. This implements a hash lookup on
> the file description to re-use an already created screen. This is
> a similar implementation as freedreno an
On 25 January 2016 at 11:24, Michel Dänzer wrote:
> On 23.01.2016 02:14, Marek Olšák wrote:
>> On Fri, Jan 22, 2016 at 6:02 PM, Emil Velikov
>> wrote:
>>> On 22 January 2016 at 16:50, Marek Olšák wrote:
On Fri, Jan 22, 2016 at 5:32 PM, Emil Velikov
wrote:
> On 22 January 2016 at
On 30 January 2016 at 23:03, Rob Clark wrote:
> On Sat, Jan 30, 2016 at 3:38 PM, Rob Herring wrote:
>> On Sat, Jan 30, 2016 at 2:06 PM, Rob Clark wrote:
>>> From: Rob Clark
>>>
>>> We seem to end up w/ duplication between compiler/Makefile.sources and
>>> compiler/glsl/Makefile.sources. The la
On 29 January 2016 at 20:52, Rob Herring wrote:
> Enabling swrast on Android causes a link error because vtest is missing.
>
> Signed-off-by: Rob Herring
Reviewed-by: Emil Velikov
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://l
On 29 January 2016 at 20:52, Rob Herring wrote:
> Android builds with -Wunused-parameter enabled which results in spewing
> lots of warnings. Disable it so more meaningful warnings are more visible.
>
Would be great if one day we resolve them and even toggle -Wextra.
Although I might be alone thin
On 29 January 2016 at 20:52, Rob Herring wrote:
> Define the DEFAULT_DRIVER_DIR for arm64 builds.
>
> Cc: Emil Velikov
Reviewed-by: Emil Velikov
> Cc: Chih-Wei Huang
> Signed-off-by: Rob Herring
> ---
> src/egl/Android.mk | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/egl/Andro
On 29 January 2016 at 20:52, Rob Herring wrote:
> At least in Android M, using LOCAL_CC does not work for secondary arch
> when doing dual arch builds. Use LOCAL_CLFAGS and LOCAL_CONLYFLAGS
> instead.
>
Not really Android M specific afaict. You can get issues even with
Android 4.{3,4} as long as y
On 29 January 2016 at 20:52, Rob Herring wrote:
> Builds with gallium enabled fail on x86 with linker error:
>
> external/mesa3d/src/mesa/vbo/vbo_exec_array.c:127: error: undefined reference
> to '_mesa_uint_array_min_max'
>
> Fix this by adding the missing file sse_minmax.c.
>
> Signed-off-by: R
Hi Rob,
On 29 January 2016 at 20:52, Rob Herring wrote:
> Commits a39a8fbbaa12 ("nir: move to compiler/") and eb63640c1d38
> ("glsl: move to compiler/") broke Android builds. Fix them.
>
> Built with i915, i965, freedreno, r300g, r600g, and virgl enabled.
>
> Cc: Emil Velikov
> Signed-off-by: Ro
62 matches
Mail list logo
