Hi folks,
do we still need to support pre-c99 compilers or could we drop
that compat stuff ?
--mtx
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Tuesday, August 2, 2016 5:03:32 PM PDT Haixia Shi wrote:
> We need to include mt->offset in the calculation of src pointer because its
> value may be non-zero, for example in a cubemap texture.
>
> Signed-off-by: Haixia Shi
> Cc: Jason Ekstrand
> Cc: kenneth.w.grau...@intel.com
> Cc: Chad Ver
On Wed, Aug 3, 2016 at 5:12 AM, Rob Herring wrote:
> On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote:
>> Make the code at least compile when being built without drm_gralloc
>> headers.
>>
>> v2: Replaced #ifdefs with stubs for gralloc_drm_get_gem_handle()
>> and GRALLOC_MODULE_PERFORM_GET_D
Hi Rob,
On Wed, Aug 3, 2016 at 2:32 AM, Rob Herring wrote:
> On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote:
>> From: Nicolas Boichat
>>
>> Existing image loader code supports creating images only for window
>> surfaces. Moreover droid_create_surface() passes wrong surface type to
>> dri2_ge
Hi folks,
is there still a real need for having non-shared glapi ?
Otherwise, should we remove it ?
--mtx
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Iago Toral Quiroga writes:
> ---
> src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 8 +---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp
> b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp
> index cf35f2e..fde7b60 100644
> --- a/src/me
Seems reasonable to be too. Rb if you'd like
On Aug 2, 2016 8:25 PM, "Timothy Arceri"
wrote:
> Reviewed-by: Timothy Arceri
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_
Iago Toral Quiroga writes:
> ---
> src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 18 ++
> 1 file changed, 18 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp
> b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp
> index 1525a3d..4014020 100644
> --- a/src/mesa/d
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Iago Toral Quiroga writes:
> ---
> src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp
> b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp
> index 7b8e30d..65fa057 100644
> --- a/src/mesa/dri
Iago Toral Quiroga writes:
> From: Connor Abbott
>
Reviewed-by: Francisco Jerez
> ---
> src/mesa/drivers/dri/i965/brw_vec4.cpp | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp
> b/src/mesa/drivers/dri/i965/brw_vec4.cpp
> index 162b481..bf6701
Iago Toral Quiroga writes:
> From: Connor Abbott
>
Reviewed-by: Francisco Jerez
> ---
> src/mesa/drivers/dri/i965/brw_reg.h | 6 ++
> 1 file changed, 6 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_reg.h
> b/src/mesa/drivers/dri/i965/brw_reg.h
> index 38cf8e3..e4c3e7a 1006
Iago Toral Quiroga writes:
> Add asserts so we remember to address this when we enable 64-bit
> integer support, as suggested by Connor and Jason.
Reviewed-by: Francisco Jerez
> ---
> src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 70
> ++
> 1 file changed, 52 inser
Reviewed-by: Francisco Jerez
Iago Toral Quiroga writes:
> ---
> src/mesa/drivers/dri/i965/brw_disasm.c | 5 -
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c
> b/src/mesa/drivers/dri/i965/brw_disasm.c
> index d74d5d5..c8bdeab 100644
Reviewed-by: Francisco Jerez
Iago Toral Quiroga writes:
> ---
> src/mesa/drivers/dri/i965/brw_vec4.h | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_vec4.h
> b/src/mesa/drivers/dri/i965/brw_vec4.h
> index 3043147..afcf31e 100644
> --- a/src/mesa/dri
On 02.08.2016 23:01, Leo Liu wrote:
> On 08/01/2016 10:20 PM, Michel Dänzer wrote:
>> On 28.07.2016 18:34, Michel Dänzer wrote:
>>> Running XTS (the X test suite) on Xephyr using glamor running on Xorg,
>>> I stumbled upon an issue:
>>>
>>> * XTS causes Xephyr to keep resetting and starting new ser
On Tue, Aug 2, 2016 at 5:28 PM, Timothy Arceri
wrote:
> Currently we only print the swizzle if the components are not
> used in order. This means x is not printed when it is the only
> component used. By checking if the last component is used we
> can print the swizzle for this case.
I agree. I'v
Currently we only print the swizzle if the components are not
used in order. This means x is not printed when it is the only
component used. By checking if the last component is used we
can print the swizzle for this case.
---
src/compiler/nir/nir_print.c | 10 +-
1 file changed, 9 inserti
On 08/02/2016 05:03 PM, Haixia Shi wrote:
We need to include mt->offset in the calculation of src pointer because its
value may be non-zero, for example in a cubemap texture.
Signed-off-by: Haixia Shi
Cc: Jason Ekstrand
Cc: kenneth.w.grau...@intel.com
Cc: Chad Versace
Change-Id: I461ad5b2046
We need to include mt->offset in the calculation of src pointer because its
value may be non-zero, for example in a cubemap texture.
Signed-off-by: Haixia Shi
Cc: Jason Ekstrand
Cc: kenneth.w.grau...@intel.com
Cc: Chad Versace
Change-Id: I461ad5b204626d5a1c45611fc6b63735dcf29f63
---
src/mesa/
On 08/02/2016 04:35 PM, Chad Versace wrote:
On 08/02/2016 04:17 PM, Haixia Shi wrote:
We need to include mt->offset in the calculation of src pointer
because its
value may be non-zero, for example in a cubemap texture.
Signed-off-by: Haixia Shi
Cc: Jason Ekstrand
Cc: kenneth.w.grau...@intel.c
On 08/02/2016 04:17 PM, Haixia Shi wrote:
We need to include mt->offset in the calculation of src pointer because its
value may be non-zero, for example in a cubemap texture.
Signed-off-by: Haixia Shi
Cc: Jason Ekstrand
Cc: kenneth.w.grau...@intel.com
Cc: Chad Versace
Change-Id: I461ad5b2046
On Tue, Aug 2, 2016 at 9:23 PM, Nicolai Hähnle wrote:
>
>
> On 02.08.2016 21:06, Marek Olšák wrote:
>>
>> On Tue, Aug 2, 2016 at 3:40 PM, Nicolai Hähnle wrote:
>>>
>>> On 31.07.2016 22:29, Marek Olšák wrote:
Inspired by LLVM, I created a small test suite that looks like "lit"
We need to include mt->offset in the calculation of src pointer because its
value may be non-zero, for example in a cubemap texture.
Signed-off-by: Haixia Shi
Cc: Jason Ekstrand
Cc: kenneth.w.grau...@intel.com
Cc: Chad Versace
Change-Id: I461ad5b204626d5a1c45611fc6b63735dcf29f63
---
src/mesa/
On 08/02/2016 04:07 AM, Tomasz Figa wrote:
This patch splits current dri2_create_image_android_native_buffer() into
main entry point and two additional functions, one for creating an image
from flink name and one for handling prime FDs using the generic DMA-buf
path. This makes the code cleaner a
On 08/02/2016 04:07 AM, Tomasz Figa wrote:
Drivers can request different set of buffers depending on the buffer
mask they pass to the get_buffers callback. This patch makes
droid_image_get_buffers() respect this mask.
v2: Return error only in case of real error condition and ignore requests
Bas pointed out some things that mean this patch (and the next) probably
need to be reworked. The tests were probably not thorough enough.
Nicolai
On 02.08.2016 21:41, Nicolai Hähnle wrote:
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_state_draw.c | 23 ++-
1
On 08/02/2016 04:07 AM, Tomasz Figa wrote:
Fix compilation warnings due to unused variables left after some earlier
code changes.
Signed-off-by: Tomasz Figa
---
src/egl/drivers/dri2/platform_android.c | 3 ---
1 file changed, 3 deletions(-)
Reviewed-by: Chad Versace
___
On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote:
> Make the code at least compile when being built without drm_gralloc
> headers.
>
> v2: Replaced #ifdefs with stubs for gralloc_drm_get_gem_handle()
> and GRALLOC_MODULE_PERFORM_GET_DRM_FD.
> Removed explicit render node probing code.
>
>
From: Nicolai Hähnle
Prefer to use DRAW_(INDEX)_INDIRECT_MULTI when available in the firmware.
Versions for SI and CI already added as provided by the firmware team, but
keep in mind that they won't currently be used since the radeon kernel module
has no interface to query the firmware version.
From: Nicolai Hähnle
This enables GL_ARB_indirect_parameters as well as a properly accelerated
implementation of GL_ARB_multi_draw_indirect.
Enabling the feature requires a sufficiently uptodate firmware -- those have
already been released a long time ago, although this does mean that the
featur
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_state_draw.c | 25 +
1 file changed, 9 insertions(+), 16 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
b/src/gallium/drivers/radeonsi/si_state_draw.c
index 24fa682..afb9bbf 100644
--- a/sr
On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote:
> This patch adds support for YV12 pixel format to the Android platform
> backend. Only creating EGL images is supported, it is not added to the
> list of available visuals.
>
> v2: Use const array defined just for YV12 instead of trying to be ove
On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote:
> There are DRI_IMAGE_FOURCC macros, for which there are no corresponding
> DRI_IMAGE_FORMAT macros. To support such formats we need to make the
> lookup function take the native format directly. As a side effect, it
> simplifies all existing call
On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote:
> This patch splits current dri2_create_image_android_native_buffer() into
> main entry point and two additional functions, one for creating an image
> from flink name and one for handling prime FDs using the generic DMA-buf
> path. This makes the
On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote:
> Drivers can request different set of buffers depending on the buffer
> mask they pass to the get_buffers callback. This patch makes
> droid_image_get_buffers() respect this mask.
>
> v2: Return error only in case of real error condition and igno
On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote:
> Fix compilation warnings due to unused variables left after some earlier
> code changes.
>
> Signed-off-by: Tomasz Figa
Tested-by: Rob Herring
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.or
From: Nicolai Hähnle
The radeon kernel module doesn't have the firmware query interface, so the
corresponding values will remain 0.
---
src/gallium/drivers/radeon/r600_pipe_common.c | 3 +++
src/gallium/drivers/radeon/radeon_winsys.h| 3 +++
src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c |
From: Nicolai Hähnle
This allows better code sharing for indirect draw calls.
---
src/gallium/drivers/radeonsi/si_state_draw.c | 76
1 file changed, 31 insertions(+), 45 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
b/src/gallium/drivers/ra
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_state_draw.c | 21 -
1 file changed, 12 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
b/src/gallium/drivers/radeonsi/si_state_draw.c
index afb9bbf..6b8ab9f 100644
--- a/src/ga
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_state_draw.c | 23 ++-
1 file changed, 18 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
b/src/gallium/drivers/radeonsi/si_state_draw.c
index a918f85..46dd5c7 100644
--- a/src/
Thanks!
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 02.08.2016 21:06, Marek Olšák wrote:
On Tue, Aug 2, 2016 at 3:40 PM, Nicolai Hähnle wrote:
On 31.07.2016 22:29, Marek Olšák wrote:
Inspired by LLVM, I created a small test suite that looks like "lit"
tests, except that the input is GLSL and it verifies asm.
The motivation was to verify
On Tue, Aug 2, 2016 at 3:40 PM, Nicolai Hähnle wrote:
> On 31.07.2016 22:29, Marek Olšák wrote:
>>
>> Inspired by LLVM, I created a small test suite that looks like "lit"
>> tests, except that the input is GLSL and it verifies asm.
>>
>> The motivation was to verify that we generate optimal code w
swr rasterizer contains numerous data transfers between vectors
and ordinary C types. Fixing for strict aliasing will take time.
---
src/gallium/drivers/swr/Makefile.am | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/swr/Makefile.am
b/src/gallium/drivers/swr/Makefile.am
i
On Tue, Aug 2, 2016 at 5:15 PM, Nicolai Hähnle wrote:
> On 29.07.2016 23:42, Marek Olšák wrote:
>>
>> This series ensures that memory usage of gfx IBs is below the kernel-
>> exposed memory limits in most cases.
>>
>> It's not possible to prevent CS ioctl failures completely because it
>> depends
On Aug 2, 2016 12:36 PM, "Matt Turner" wrote:
>
> On Mon, Aug 1, 2016 at 8:59 PM, Kenneth Graunke
wrote:
> > Previously, for every input, we moved the dispatch mask to the flag
> > register, then emitted two predicated PLN instructions, one with
> > centroid barycentric coordinates (for normal pi
> On Aug 2, 2016, at 1:00 PM, Matt Turner wrote:
>
> On Tue, Aug 2, 2016 at 10:53 AM, Tim Rowley
> wrote:
>> swr rasterizer contains numerous data transfers between vectors
>> and ordinary C types. Fixing for strict aliasing will take time.
>
> Oh, sorry! I forgot about swr.
>
>> ---
>> con
On Tue, Aug 2, 2016 at 10:53 AM, Tim Rowley wrote:
> swr rasterizer contains numerous data transfers between vectors
> and ordinary C types. Fixing for strict aliasing will take time.
Oh, sorry! I forgot about swr.
> ---
> configure.ac| 7 +++
> src/gallium/drivers/
On Sunday, July 31, 2016 12:22:40 PM PDT Francisco Jerez wrote:
> Kenneth Graunke writes:
>
> > On Friday, July 22, 2016 8:59:03 PM PDT Francisco Jerez wrote:
> >> The problem with the current approach is that driver output locations
> >> are represented as a linear offset within the nir_outputs
Needed to successfully link llvmpipe or swr when using shared llvm libs.
---
configure.ac | 5 +
1 file changed, 5 insertions(+)
diff --git a/configure.ac b/configure.ac
index fb4a12a..edbc95b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2497,6 +2497,11 @@ dnl in LLVM_LIBS.
if test "x$
swr rasterizer contains numerous data transfers between vectors
and ordinary C types. Fixing for strict aliasing will take time.
---
configure.ac| 7 +++
src/gallium/drivers/swr/Makefile.am | 1 +
2 files changed, 8 insertions(+)
diff --git a/configure.ac b/configure.
On Mon, Aug 1, 2016 at 8:59 PM, Kenneth Graunke wrote:
> Previously, for every input, we moved the dispatch mask to the flag
> register, then emitted two predicated PLN instructions, one with
> centroid barycentric coordinates (for normal pixels), and one with
> pixel barycentric coordinates (for
On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote:
> From: Nicolas Boichat
>
> Existing image loader code supports creating images only for window
> surfaces. Moreover droid_create_surface() passes wrong surface type to
> dri2_get_dri_config(), resulting in incorrect configs being returned for
>
Am 02.08.2016 um 14:57 schrieb Alex Deucher:
On Tue, Aug 2, 2016 at 4:55 AM, Marek Olšák wrote:
On Tue, Aug 2, 2016 at 3:13 AM, Michel Dänzer wrote:
On 01.08.2016 16:35, Michel Dänzer wrote:
On 30.07.2016 06:42, Marek Olšák wrote:
From: Marek Olšák
This is controversial, but I don't see a
This fixes 104 of the new image_clearing and copy_and_blit Vulkan CTS
tests.
Signed-off-by: Jason Ekstrand
Cc: "12.0"
---
src/intel/vulkan/anv_meta_blit2d.c | 210 -
1 file changed, 208 insertions(+), 2 deletions(-)
diff --git a/src/intel/vulkan/anv_meta_bli
Signed-off-by: Jasosn Ekstrand
Cc: "12.0"
---
src/intel/vulkan/anv_meta_blit2d.c | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/src/intel/vulkan/anv_meta_blit2d.c
b/src/intel/vulkan/anv_meta_blit2d.c
index 9d4c2fc..30bc6ed 100644
--- a/src/intel/vulkan/anv_
On Tue, 2016-08-02 at 17:04 +0100, Eric Engestrom wrote:
> On Tue, Aug 02, 2016 at 12:20:54PM +0300, Andres Gomez wrote:
[snip]
> > I'm not sure I'm understanding what you mean.
> >
> > If you mean to remove the #define and add the value as and additional
> > element to the enum, the existence o
This field was added in 2d94601582e4f0fcaf8c ("Add DRI3+Present loader"), which
was a v2. Guessing it was used in v1 and Keith forgot to remove it in v2.
Signed-off-by: Eric Engestrom
---
Fair warning: This patch will conflict with Jan Ziak's "[PATCH] dri: use 'bool'
instead of 'int' for boolean
On Tue, Aug 02, 2016 at 05:32:16PM +0200, Jan Ziak wrote:
> Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com>
Much better with the s/0/false/;s/1/true/ :)
Reviewed-by: Eric Engestrom
> ---
> src/egl/drivers/dri2/egl_dri2.h | 19 ++-
> src/egl/
On Mon, Aug 01, 2016 at 03:43:17PM +0200, ⚛ wrote:
> On Mon, Aug 1, 2016 at 2:46 PM, Eric Engestrom
> wrote:
> > On Sun, Jul 31, 2016 at 05:49:02PM +0200, Jan Ziak wrote:
> >> Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com>
> >
> > This is a good change, and with a coup
Signed-off-by: Andres Gomez
---
.dir-locals.el | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/.dir-locals.el b/.dir-locals.el
index 4b53931..9c89a63 100644
--- a/.dir-locals.el
+++ b/.dir-locals.el
@@ -1,4 +1,5 @@
-((prog-mode
+((nil . ((show-trailing-whitespace . t)
On Tue, Aug 02, 2016 at 12:20:54PM +0300, Andres Gomez wrote:
> On Mon, 2016-08-01 at 14:02 +0100, Eric Engestrom wrote:
> > On Sun, Jul 31, 2016 at 07:07:34PM +0300, Andres Gomez wrote:
>
> [snip]
>
> > > diff --git a/src/compiler/glsl/ast.h b/src/compiler/glsl/ast.h
> > > index 157895d..89f531c
On 08/02/2016 08:40 AM, Jan Ziak wrote:
Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com>
---
src/mesa/vbo/vbo_exec_array.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c
in
On 08/02/2016 08:45 AM, ⚛ wrote:
On Tue, Aug 2, 2016 at 4:13 PM, Brian Paul wrote:
In bufferobj.c, it looks like we're just using %d or %s and casting the
arguments to int. But in other places, we're using %ld and casting to long.
That's not going to accurately report 64-bit values, but I gues
Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com>
---
src/egl/drivers/dri2/egl_dri2.h | 19 ++-
src/egl/drivers/dri2/platform_drm.c | 6 +++---
src/egl/drivers/dri2/platform_wayland.c | 20 ++--
src/egl/drivers/dri2/platf
On 29.07.2016 23:42, Marek Olšák wrote:
This series ensures that memory usage of gfx IBs is below the kernel-
exposed memory limits in most cases.
It's not possible to prevent CS ioctl failures completely because it
depends on whether continuous free space for all buffers is available,
i.e. it d
On Tue, Aug 2, 2016 at 4:13 PM, Brian Paul wrote:
> In bufferobj.c, it looks like we're just using %d or %s and casting the
> arguments to int. But in other places, we're using %ld and casting to long.
> That's not going to accurately report 64-bit values, but I guess that hasn't
> been a concern
On 02.08.2016 10:55, Marek Olšák wrote:
On Tue, Aug 2, 2016 at 3:13 AM, Michel Dänzer wrote:
On 01.08.2016 16:35, Michel Dänzer wrote:
On 30.07.2016 06:42, Marek Olšák wrote:
From: Marek Olšák
This is controversial, but I don't see a better way out of this.
Tonga has 2 GB of VRAM and 2 GB
Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com>
---
src/mesa/vbo/vbo_exec_array.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c
index f371890..ae6a3da 100644
--- a/src/mesa/v
On 29.07.2016 23:42, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 2 +-
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
b/src/gallium/wins
Patches 1-7:
Reviewed-by: Nicolai Hähnle
On 02.08.2016 12:27, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 30 +--
src/gallium/winsys/radeon/drm/radeon_drm_cs.h | 1 +
2 files changed, 29 insertions(+), 2 deletions(-)
dif
On 02.08.2016 12:27, Marek Olšák wrote:
From: Marek Olšák
+23% Bioshock Infinite performance.
---
src/gallium/drivers/radeon/r600_pipe_common.c | 45 ++-
1 file changed, 44 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c
b/src/g
On 08/01/2016 10:20 PM, Michel Dänzer wrote:
On 28.07.2016 18:34, Michel Dänzer wrote:
Running XTS (the X test suite) on Xephyr using glamor running on Xorg,
I stumbled upon an issue:
* XTS causes Xephyr to keep resetting and starting new server generations
* glamor creates a new GLX context
On 08/01/2016 04:35 PM, Jan Ziak wrote:
Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com>
---
src/mesa/vbo/vbo_exec_array.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c
index 90f1
Reviewed-by: Nicolai Hähnle
On 02.08.2016 12:34, Marek Olšák wrote:
From: Marek Olšák
Small decrease in draw call overhead.
---
src/gallium/drivers/radeonsi/si_pipe.h | 1 +
src/gallium/drivers/radeonsi/si_state.c | 6 ++
src/gallium/drivers/radeonsi/si_state_draw.c
Patch 2 & 3:
Reviewed-by: Nicolai Hähnle
On 30.07.2016 17:22, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_debug.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_debug.c
b/src/gallium/drivers/radeonsi/si_debug.c
index e030
Obviously, the R-b applies to this rebased series as well.
Nicolai
On 01.08.2016 23:55, Marek Olšák wrote:
From: Marek Olšák
v2: rebase on top of Brian's commit
---
src/gallium/drivers/radeonsi/si_debug.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_
On 01.08.2016 15:21, Jan Ziak wrote:
Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com>
---
src/egl/drivers/dri2/egl_dri2.h | 19 ++-
src/gallium/auxiliary/vl/vl_winsys_dri3.c | 3 ++-
src/glx/dri3_priv.h | 5 +++--
src/l
Reviewed-by: Nicolai Hähnle
On 31.07.2016 22:57, Marek Olšák wrote:
From: Marek Olšák
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97140
---
src/gallium/drivers/ddebug/dd_draw.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/drivers/ddebug/dd_draw.c
b/src/gal
The series is
Reviewed-by: Nicolai Hähnle
On 31.07.2016 22:34, Marek Olšák wrote:
From: Marek Olšák
LLVM doesn't use it.
---
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_s
On 31.07.2016 22:29, Marek Olšák wrote:
Inspired by LLVM, I created a small test suite that looks like "lit" tests,
except that the input is GLSL and it verifies asm.
The motivation was to verify that we generate optimal code when starting from
GLSL.
[PATCH 1/3] radeonsi: add environment vari
On Aug 2, 2016 2:46 AM, "Timothy Arceri"
wrote:
>
> Looks like a copy and paste error from f752effa087
Agreed. Rb
> Cc: Jason Ekstrand
> ---
> src/compiler/nir/nir_validate.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/compiler/nir/nir_validate.c
b/src/comp
On Tue, Aug 2, 2016 at 4:55 AM, Marek Olšák wrote:
> On Tue, Aug 2, 2016 at 3:13 AM, Michel Dänzer wrote:
>> On 01.08.2016 16:35, Michel Dänzer wrote:
>>> On 30.07.2016 06:42, Marek Olšák wrote:
From: Marek Olšák
This is controversial, but I don't see a better way out of this.
>>>
There are DRI_IMAGE_FOURCC macros, for which there are no corresponding
DRI_IMAGE_FORMAT macros. To support such formats we need to make the
lookup function take the native format directly. As a side effect, it
simplifies all existing calls to this function, because they all called
get_format() fir
This patch splits current dri2_create_image_android_native_buffer() into
main entry point and two additional functions, one for creating an image
from flink name and one for handling prime FDs using the generic DMA-buf
path. This makes the code cleaner and also prepares for disabling flink
path mor
Fix compilation warnings due to unused variables left after some earlier
code changes.
Signed-off-by: Tomasz Figa
---
src/egl/drivers/dri2/platform_android.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/src/egl/drivers/dri2/platform_android.c
b/src/egl/drivers/dri2/platform_android.c
i
From: Nicolas Boichat
Existing image loader code supports creating images only for window
surfaces. Moreover droid_create_surface() passes wrong surface type to
dri2_get_dri_config(), resulting in incorrect configs being returned for
pbuffers. This patch fixes these issues.
In addition, the conf
This patch adds support for YV12 pixel format to the Android platform
backend. Only creating EGL images is supported, it is not added to the
list of available visuals.
v2: Use const array defined just for YV12 instead of trying to be overly
generic.
Signed-off-by: Tomasz Figa
Signed-off-by:
Make the code at least compile when being built without drm_gralloc
headers.
v2: Replaced #ifdefs with stubs for gralloc_drm_get_gem_handle()
and GRALLOC_MODULE_PERFORM_GET_DRM_FD.
Removed explicit render node probing code.
Signed-off-by: Tomasz Figa
---
src/egl/Android.mk
Drivers can request different set of buffers depending on the buffer
mask they pass to the get_buffers callback. This patch makes
droid_image_get_buffers() respect this mask.
v2: Return error only in case of real error condition and ignore requests
of unavailable buffers.
Signed-off-by: Tomas
Hi,
This series is a collection of various fixes and extensions we came up
with during our attempt to use Mesa for Android.
Fixes included in this series:
- fixed multiple issues with handling pbuffers in the backend,
- made the implementation of DRI image loader .getBuffers callback
conform
Currently the code creates a display target struct with refcount field
initialized to 1 and then the caller again increments it, leading to
a leaked reference. Let's remove the unnecessary increment.
v2: Split from one big patch into four fixing one thing at a time.
Signed-off-by: Tomasz Figa
CC
Current prime import code in kms-dri Gallium winsys backend has several
issues:
- It leaks imported buffers, because of duplicate reference count
increment at import time.
- It does not check whether an already imported (or local) buffer is not
being imported, which is problematic because p
drmPrimeHandleToFD() will return the same GEM handle every time the same
buffer is imported, even from a different prime FD. Since GEM handles
are not reference counted, we need to make sure that each GEM handle is
referenced only by one display target struct, by looking it up in
kms_sw->bo_list fi
Currently kms_sw_displaytarget_add_from_prime() allocates the struct and
fills in only some of the fields, resulting in a half-baked struct that
needs to be further completed by the caller. To make this a bit more
consistent, pass width, height and stride to this function and fill in
everything the
As a preparation to use the lookup in more than once place, move the
code that looks up given KMS/GEM handle to a separate function. This
change should not introduce any functional changes.
v2: Split into separate patch.
Move lookup code into separate function.
Signed-off-by: Tomasz Figa
CC:
From: Marek Olšák
Small decrease in draw call overhead.
---
src/gallium/drivers/radeonsi/si_pipe.h | 1 +
src/gallium/drivers/radeonsi/si_state.c | 6 ++
src/gallium/drivers/radeonsi/si_state_draw.c| 20 +---
src/gallium/drivers/radeonsi/si_state_shader
On Tue, Aug 2, 2016 at 12:27 PM, Marek Olšák wrote:
> Hi,
>
> Most interested people already know what this is, so I'm just gonna share the
> performance numbers here.
>
> This series makes Bioshock Infinite 23% faster. Along with the CPU overhead
> optimizations that are in the master branch al
From: Marek Olšák
---
src/gallium/drivers/radeon/radeon_winsys.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/drivers/radeon/radeon_winsys.h
b/src/gallium/drivers/radeon/radeon_winsys.h
index 3be4738..e7787d3 100644
--- a/src/gallium/drivers/radeon/radeon_winsys.h
+++
1 - 100 of 115 matches
Mail list logo
