This is:
Reviewed-by: Andres Gomez
On Wed, 2017-05-03 at 11:16 +0200, Samuel Pitoiset wrote:
> It doesn't make sense to prefix them with 'image' because
> they are called "Memory Qualifiers" and they can be applied
> to members of storage buffer blocks.
>
> Signed-off-by:
On Mon, May 01, 2017 at 06:43:03PM -0700, Rafael Antognolli wrote:
> From: Kenneth Graunke
>
> v3:
>- Drop aub parameter (Ken)
>
> Signed-off-by: Kenneth Graunke
Reviewed-by: Topi Pohjolainen
> ---
>
On Thu, May 04, 2017 at 05:49:08AM +0300, Pohjolainen, Topi wrote:
> On Wed, May 03, 2017 at 05:08:12PM -0700, Rafael Antognolli wrote:
> > On Wed, May 03, 2017 at 09:10:10PM +0300, Pohjolainen, Topi wrote:
> > >
> > > In the subject: s/ge5/gen5/
> > >
> > > But don't we need to squash this into
This is something the original decoder did, but I didn't bother with
until now. I recently had to debug an Ironlake issue, and wanted to
inspect VS_STATE. So, now it's back.
The other packets in the switch statement are all Gen6/7+, where we
use offsets from dynamic state base address, so we
On Mon, May 01, 2017 at 06:43:06PM -0700, Rafael Antognolli wrote:
> From: Kenneth Graunke
>
> This emits 3DSTATE_WM_DEPTH_STENCIL on Gen8+ or DEPTH_STENCIL_STATE
> (and the relevant pointer packets) on Gen6-7.5 from a single function.
>
> v3:
>- Watch for
On 05/01/2017 10:03 AM, Brian Paul wrote:
On 05/01/2017 08:32 AM, Brian Paul wrote:
On 04/28/2017 05:12 PM, Marek Olšák wrote:
Hi,
This series shrinks various gallium structures and removes
set_index_buffer in order to decrease CPU overhead.
PART 1: Performance results
All testing below
On 04/05/17 13:31, Dave Airlie wrote:
+/* The ARB_separate_shader_object spec says:
+ *
+ * "The executable code for an individual shader stage is taken from
+ * the current program for that stage. If there is a current program
+ * object established by UseProgram, that program is
> +/* The ARB_separate_shader_object spec says:
> + *
> + * "The executable code for an individual shader stage is taken from
> + * the current program for that stage. If there is a current program
> + * object established by UseProgram, that program is considered current
> + *
From: Dave Airlie
This adds a path to allow compute resolves to be used
for subpass resolves.
This isn't used yet, but will be later.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_resolve_cs.c | 93 +++
From: Dave Airlie
This drops the resolve workarounds that change an image
tiling mode behinds it's back, this is horrible and breaks
the image_view->image relationship. Remove all this.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_cmd_buffer.c |
From: Dave Airlie
This will allow to add a subpass compute resolve path.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_resolve_cs.c | 141 +++---
1 file changed, 78 insertions(+), 63 deletions(-)
diff --git
From: Dave Airlie
In order to resolve into DCC enabled dests we need to use
the fragment shader. This reuses the code from the compute
path and implements a resolve path in vertex/fragment shader.
This code isn't used until later.
Signed-off-by: Dave Airlie
From: Dave Airlie
If we are resolving into an srgb dest, we need to convert
to linear so the store does the conversion back.
This should fix some wierdness seen when we subresolves
hit the compute path.
Signed-off-by: Dave Airlie
---
radv has had a hack in place to change some image details (tiling_mode)
to enable hw resolver to work, however this isn't at all how Vulkan
should work. Coupled with the compute resolve path hanging in some situations,
I decided to rework the lot.
This firstly refactors compute resolve to
From: Dave Airlie
There are 3 resolve paths, the fastest being the hw resolver
but it has restriction on tile modes and can't do subresolves,
the compute resolver is next speed wise, but can't handle DCC
destinations, the fragment resolver handles that case.
This will end up
From: Dave Airlie
I want to reuse the same code for the fragment shader
version of the resolve shaders.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta.c| 127 ++
src/amd/vulkan/radv_meta.h
On Wednesday, May 3, 2017 7:19:24 PM PDT Matt Turner wrote:
> On Wed, May 3, 2017 at 6:03 PM, Kenneth Graunke wrote:
> > On Tuesday, May 2, 2017 11:02:58 AM PDT Rafael Antognolli wrote:
> >> Since the enum is in the same header now, we can use it as the type of
> >> the
On Monday, May 1, 2017 6:42:48 PM PDT Rafael Antognolli wrote:
> The main difference for this one is that it includes the changes based on the
> review by Kenneth.
>
> Current version here:
> https://github.com/rantogno/mesa/commits/review/genxml-v03
I've pushed patches 1-33.
signature.asc
On Monday, May 1, 2017 6:43:22 PM PDT Rafael Antognolli wrote:
> The following states are ported on this patch:
>- gen6_gs_push_constants
>- gen6_vs_push_constants
>- gen6_wm_push_constants
>- gen7_tes_push_constants
>
> v2:
>- Use helper to setup brw_address (Kristian)
> v3:
On Wed, May 03, 2017 at 05:11:45PM -0700, Rafael Antognolli wrote:
> On Wed, May 03, 2017 at 08:28:24PM +0300, Pohjolainen, Topi wrote:
> > On Mon, May 01, 2017 at 06:43:02PM -0700, Rafael Antognolli wrote:
> > > These macros are defined in brw_defines.h, which contains a lot of
> > > macros that
On Wed, May 03, 2017 at 05:08:12PM -0700, Rafael Antognolli wrote:
> On Wed, May 03, 2017 at 09:10:10PM +0300, Pohjolainen, Topi wrote:
> >
> > In the subject: s/ge5/gen5/
> >
> > But don't we need to squash this into the previous patch? Alone that patch
> > won't even link, right?
>
> Yes, you
When running shader-db with intel_stub and recent Mesa, context creation
fails when making a logical hardware context. In this case, we call
intelDestroyContext(), which gets here and tries to unmap the cache BO.
But there isn't one - we haven't made it yet. So we try to unmap a
NULL pointer,
Recent Mesa has started detecting a hardware context ID of zero (which
is bogus - it's the default context ID) and failing GL context creation.
Previously it saw that the ioctl returned success and happily proceeded.
Don't bother assigning a real ID, but do assign a non-zero value.
---
Haswell requires a relatively new command parser version to enable
GL 4.5, and we'd like to compile those shaders. Since we don't actually
execute anything, it doesn't really matter. Claim version 9, the
current version.
---
intel_stub.c | 3 +++
1 file changed, 3 insertions(+)
diff --git
Newer Mesa requires this. It's irrelevant for us; pretend to have it.
---
intel_stub.c | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/intel_stub.c b/intel_stub.c
index 4917656..7f85678 100644
--- a/intel_stub.c
+++ b/intel_stub.c
@@ -192,11 +192,14 @@ ioctl(int
On Wed, May 3, 2017 at 5:56 PM, Pierre Moreau wrote:
>
> * As there is no upstream version of LLVM which can produce SPIR-V out of
> OpenCL code, clCreateProgramWithSource will refuse to work if the target IR
> is SPIR-V, for now.
hopefully this[1] will eventually
On Wed, May 3, 2017 at 6:03 PM, Kenneth Graunke wrote:
> On Tuesday, May 2, 2017 11:02:58 AM PDT Rafael Antognolli wrote:
>> Since the enum is in the same header now, we can use it as the type of
>> the field.
>>
>> Signed-off-by: Rafael Antognolli
https://bugs.freedesktop.org/show_bug.cgi?id=100613
--- Comment #20 from Ray Strode ---
Hi,
> - if i leave vector_justify to FALSE, but change attachment 130980
> to use fetch_width instead of format_desc->block.bits then all the
> 3 component sshort tests and half float
---
src/mapi/glapi/gen/gl_API.xml | 2 +-
src/mesa/main/shaderapi.c | 65 +++
src/mesa/main/shaderapi.h | 2 ++
3 files changed, 44 insertions(+), 25 deletions(-)
diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml
index
---
src/mapi/glapi/gen/ARB_separate_shader_objects.xml | 2 +-
src/mesa/main/pipelineobj.c| 21 +
src/mesa/main/pipelineobj.h| 3 +++
3 files changed, 25 insertions(+), 1 deletion(-)
diff --git
---
src/mapi/glapi/gen/ARB_separate_shader_objects.xml | 2 +-
src/mesa/main/pipelineobj.c| 26 ++
src/mesa/main/pipelineobj.h| 2 ++
3 files changed, 29 insertions(+), 1 deletion(-)
diff --git
This will be used to create a KHR_no_error version of
glUseProgramStages().
---
src/mesa/main/pipelineobj.c | 71 +
1 file changed, 39 insertions(+), 32 deletions(-)
diff --git a/src/mesa/main/pipelineobj.c b/src/mesa/main/pipelineobj.c
index
Makes the code easier to follow
---
src/mesa/main/shaderapi.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
index b764fee..eb75a3e 100644
--- a/src/mesa/main/shaderapi.c
+++ b/src/mesa/main/shaderapi.c
@@ -1846,21
I can't think of any reason you would want this in a release
build.
---
src/mesa/main/shaderapi.c | 14 ++
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
index c41f006..b764fee 100644
--- a/src/mesa/main/shaderapi.c
---
src/mapi/glapi/gen/ARB_separate_shader_objects.xml | 2 +-
src/mesa/main/pipelineobj.c| 18 ++
src/mesa/main/pipelineobj.h| 2 ++
3 files changed, 21 insertions(+), 1 deletion(-)
diff --git
On Wednesday, May 3, 2017 5:53:35 PM PDT Rafael Antognolli wrote:
> The hardware docs are wrong, but the length used in the xml is also
> wrong.
>
> Signed-off-by: Rafael Antognolli
> ---
> src/intel/genxml/gen5.xml | 6 +++---
> 1 file changed, 3 insertions(+), 3
On 04/05/17 10:43, Ilia Mirkin wrote:
I looked into removing these passes a while back. A little detail here
is that DCE has to get run at least a little bit for correctness
reasons -- some (lazy) glsl -> tgsi
Is there anyway to improve this? I was actually looking at this a little
yesterday
On Tuesday, May 2, 2017 11:02:58 AM PDT Rafael Antognolli wrote:
> Since the enum is in the same header now, we can use it as the type of
> the field.
>
> Signed-off-by: Rafael Antognolli
> ---
>
> PS: We can merge this with the previous patch too if that's better.
The hardware docs are wrong, but the length used in the xml is also
wrong.
Signed-off-by: Rafael Antognolli
---
src/intel/genxml/gen5.xml | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/intel/genxml/gen5.xml b/src/intel/genxml/gen5.xml
From: Dave Airlie
There is no reason to advertise transfer ability for formats we can't
use for anything else. This stops some CTS tests hitting internal
error for 64-bit types when they see the transfer flags.
Signed-off-by: Dave Airlie
---
I looked into removing these passes a while back. A little detail here
is that DCE has to get run at least a little bit for correctness
reasons -- some (lazy) glsl -> tgsi code uses the dead_mask with the
assumption that DCE is run later.
Another observation was that while, indeed, those passes
This and the tgsi copy_propagation pass are very slow, I'd really like
it if we both didn't require the pass for things to work and also didn't
make it any slower. perf is showing these 2 passes at ~11% during Deus
Ex start-up (cold cache).
Can we not just add a DCE call to the glsl
On Wed, May 03, 2017 at 08:28:24PM +0300, Pohjolainen, Topi wrote:
> On Mon, May 01, 2017 at 06:43:02PM -0700, Rafael Antognolli wrote:
> > These macros are defined in brw_defines.h, which contains a lot of
> > macros that conflict with autogenerated code from genxml. But we need to
> > use them
On Wed, May 03, 2017 at 09:10:10PM +0300, Pohjolainen, Topi wrote:
>
> In the subject: s/ge5/gen5/
>
> But don't we need to squash this into the previous patch? Alone that patch
> won't even link, right?
Yes, you are right, it should be squashed.
> On Mon, May 01, 2017 at 06:43:05PM -0700,
On Wed, May 03, 2017 at 12:22:13PM +0300, Topi Pohjolainen wrote:
> Patches 1-17 are revision that
>
> - rework hiz on gen6 to use on-demand offset calculator allowing
> one to drop dependency to miptree structure and
> - rework all auxiliary surfaces to be created against isl directly.
Samuel Iglesias Gonsálvez writes:
> Reorder the uniforms to load first the dvec4-aligned variables
> in the push constant buffer and then push the vec4-aligned ones.
>
> This fixes a bug were the dvec3/4 might be loaded one part on a GRF and
> the rest in next GRF, so the
Thanks. This patch has no regressions.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
From: Chad Versace
Add a new bool, _EGLSurface::Lost, and check it in eglMakeCurrent and
eglSwapBuffers. The EGL 1.5 spec says that those functions emit errors
when the native surface is no longer valid.
This patch just updates core EGL. No driver sets _EGLSurface::Lost yet.
From: Chad Versace
This ensures that future calls to eglSwapBuffers and eglMakeCurrent emit
an error.
This patch is part of a series for fixing
android.hardware.camera2.cts.RobustnessTest#testAbandonRepeatingRequestSurface
on Chrome OS x86 devices.
Cc: Tomasz Figa
From: Chad Versace
That is, call ANativeWindow::cancelBuffer in droid_destroy_surface().
This should prevent application deadlock when the app destroys the
EGLSurface after EGL has acquired a buffer from SurfaceFlinger
(ANativeWindow::dequeueBuffer) but before EGL has
Reviewed-by: Rafael Antognolli
On Fri, Apr 28, 2017 at 05:04:05PM -0700, Kenneth Graunke wrote:
> The Ironlake documentation is terrible, so it's unclear whether or not
> this field exists there. It definitely doesn't exist on Sandybridge
> and later. It definitely
Ok, I must say I'm a bit impressed. This series is a bit nuts but you
pulled it off with style. I don't think any hardware designers ever
intended for resolves to be done this way but it seems to work out really
well. Good work! I've sent a variety of comments throughout the series
but no
On Thu, Apr 27, 2017 at 11:32 AM, Nanley Chery
wrote:
> There's no image layout to represent a full-RT-cleared color attachment.
> That's one reason we can end up with redundant resolves. Testing has
> shown that such resolves can measurably hurt performance and that
>
This is how VC4 stores 5551 textures, which we need to support for
GL_OES_required_internalformat.
v2: Extend commit message, fix svga driver build, add BE ordering from
Roland.
---
src/gallium/auxiliary/util/u_format.csv | 2 ++
src/gallium/drivers/svga/svga_format.c | 2 ++
Roland Scheidegger writes:
> Am 02.05.2017 um 21:33 schrieb Eric Anholt:
>> ---
>>
>> Do I have the swizzles right here? It's a bit complicated because I
>> have a reswizzle in vc4, so I may have just massaged things to work
>> out in my case. I tried a lot of combinations
Fixes DEQP's scoping.invalid.redeclare_function_fragment/vertex.
v2: Fix accidental rejection of prototype+decl.
Reviewed-by: Samuel Pitoiset
---
Thanks for testing! Here's a new version with that fixed. Piglit
series covering this regression and more, to follow.
Looks great.
For the series, Reviewed-by: Charmaine Lee
From: Brian Paul
Sent: Wednesday, May 3, 2017 12:49:40 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee; Neha Bhende
Subject: [PATCH 2/2] wglgears:
On Wed, May 3, 2017 at 6:04 PM, Nicolai Hähnle wrote:
> On 01.05.2017 14:53, Marek Olšák wrote:
>>
>> From: Marek Olšák
>>
>> This is the best place to do it. Now drivers without u_vbuf don't have to
>> do it.
>> ---
>>
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/core/kernel.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/kernel.cpp
index 4716705323..328323b6b0 100644
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/api/program.cpp | 35 ---
1 file changed, 25 insertions(+), 10 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/api/program.cpp | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/gallium/state_trackers/clover/api/program.cpp
index
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/api/program.cpp | 29 ++-
src/gallium/state_trackers/clover/core/program.cpp | 57 --
src/gallium/state_trackers/clover/core/program.hpp | 14 ++
3 files changed, 95
OpenCL 2.1 gives the ability to query for a program’s IL, which is
returned as a pointer.
Signed-off-by: Pierre Moreau
---
.../state_trackers/clover/core/property.hpp| 39 ++
1 file changed, 39 insertions(+)
diff --git
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/core/program.cpp | 19 ++-
1 file changed, 14 insertions(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/program.cpp
Creating a program using clCreateProgramWithSource to SPIR-V requires a
non-upstreamed version of LLVM and clang, therefore it is currently not
supported.
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/core/program.cpp | 4
1 file changed, 4
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/core/device.cpp | 11 +++
src/gallium/state_trackers/clover/core/device.hpp | 3 +++
2 files changed, 14 insertions(+)
diff --git a/src/gallium/state_trackers/clover/core/device.cpp
Signed-off-by: Pierre Moreau
---
src/gallium/state_trackers/clover/Makefile.am | 10 +-
src/gallium/state_trackers/clover/Makefile.sources | 4 +
.../state_trackers/clover/spirv/invocation.cpp | 481 +
Signed-off-by: Pierre Moreau
---
.../state_trackers/clover/spirv/spirv.hpp11| 952 +
1 file changed, 952 insertions(+)
create mode 100644 src/gallium/state_trackers/clover/spirv/spirv.hpp11
diff --git
Signed-off-by: Pierre Moreau
---
include/CL/cl.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/CL/cl.h b/include/CL/cl.h
index 44d7aedc3e..cc8d7ddf60 100644
--- a/include/CL/cl.h
+++ b/include/CL/cl.h
@@ -455,6 +455,7 @@ typedef struct _cl_buffer_region {
Hello everyone,
I have been working on converting SPIR-V to NVIR in order to run OpenCL kernels
on Nouveau, and I would like to submit the first part of that work for review.
Pieces from the SPIR-V to NVIR conversion work will be submitted once I have
cleaned it up and this series has progressed
Signed-off-by: Pierre Moreau
---
configure.ac | 16
1 file changed, 16 insertions(+)
diff --git a/configure.ac b/configure.ac
index ba042791ad..602aeb279d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2064,6 +2064,11 @@ AC_ARG_WITH([clang-libdir],
Signed-off-by: Pierre Moreau
---
src/gallium/auxiliary/Makefile.am | 1 +
src/gallium/auxiliary/Makefile.sources| 4 ++
src/gallium/auxiliary/spirv/spirv_utils.c | 75 +++
src/gallium/auxiliary/spirv/spirv_utils.h | 86
Signed-off-by: Pierre Moreau
---
include/CL/cl.h | 6 ++
include/CL/cl_platform.h | 1 +
2 files changed, 7 insertions(+)
diff --git a/include/CL/cl.h b/include/CL/cl.h
index 316565d6e4..44d7aedc3e 100644
--- a/include/CL/cl.h
+++ b/include/CL/cl.h
@@ -757,6
Signed-off-by: Pierre Moreau
---
src/gallium/auxiliary/Makefile.sources |4 +-
src/gallium/auxiliary/spirv/spirv_linker.c | 1324
src/gallium/auxiliary/spirv/spirv_linker.h | 67 ++
3 files changed, 1394 insertions(+), 1 deletion(-)
Signed-off-by: Pierre Moreau
---
src/gallium/include/pipe/p_defines.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/include/pipe/p_defines.h
b/src/gallium/include/pipe/p_defines.h
index ce2cfd1d88..71991383c2 100644
---
Besides parsing all the opcodes until reaching the EOF character, there
is no way to compute the size of a SPIR-V binary. Therefore, it is
easier to pass it along the SPIR-V binary in pipe_compute_state.
Signed-off-by: Pierre Moreau
---
src/gallium/include/pipe/p_state.h
Looks good to me. For the series,
Reviewed-by: Neha Bhende
Regards,
Neha
From: Brian Paul
Sent: Wednesday, May 3, 2017 12:49:40 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee; Neha Bhende
Subject: [PATCH 2/2]
On Thu, Apr 27, 2017 at 11:32 AM, Nanley Chery
wrote:
> Signed-off-by: Nanley Chery
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 93 ++
>
> 1 file changed, 93 insertions(+)
>
> diff --git
On 03/05/17 19:14, Andres Gomez wrote:
v2: Brian Paul as main contact point and Jose Fonseca as
fallback (Vinson, Jose)
Signed-off-by: Andres Gomez
Cc: Emil Velikov
Cc: Vinson Lee
Cc: Brian Paul
Cc:
Ping
On Fri, Apr 28, 2017 at 11:42 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> Cc: 17.1
> ---
> src/amd/common/gfx9d.h | 4
> src/gallium/drivers/radeonsi/si_state.c | 21 ++---
Only reported with -v option for now.
---
src/wgl/wglinfo.c | 21 +++--
1 file changed, 19 insertions(+), 2 deletions(-)
diff --git a/src/wgl/wglinfo.c b/src/wgl/wglinfo.c
index f81978b..faccbb1 100644
--- a/src/wgl/wglinfo.c
+++ b/src/wgl/wglinfo.c
@@ -44,6 +44,7 @@
static
Specify WGL_DRAW_TO_WINDOW_ARB and WGL_COLOR_BITS_ARB.
Improve some comments, per Charmaine.
---
src/wgl/wglgears.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/src/wgl/wglgears.c b/src/wgl/wglgears.c
index 7d43822..d673143 100644
--- a/src/wgl/wglgears.c
+++
On Wed, May 3, 2017 at 12:04 PM, Chad Versace
wrote:
> On Wed 03 May 2017, Jason Ekstrand wrote:
> > On Tue, May 2, 2017 at 5:15 PM, Chad Versace <[1]
> chadvers...@chromium.org>
> > wrote:
> >
> > On Fri 14 Apr 2017, Jason Ekstrand wrote:
> > > This
Adam Jackson writes:
> If the gbm_create_device() call here actually did fail, any subsequent
> eglTerminate on the display would segfault.
#1 and 2 are:
Reviewed-by: Eric Anholt
signature.asc
Description: PGP signature
Adam Jackson writes:
> From: Jonny Lamb
>
> This is a rebase/squash/rewrite of a series Jonny had sent long ago. The
> major change is implementing this in terms of the drmDevice API. Both
> relevant piglits go from skip to pass on i965.
Just some
Android O moves to LLVM 3.9 and also has some differences in header
dependencies as LLVM has moved to blueprint files. It seems libLLVMCore
was only needed for header dependencies, so we can drop that for O.
Signed-off-by: Rob Herring
---
Android.mk | 5 -
1 file changed, 4
Including libgcc breaks on Android O (master). This doesn't appear to be
needed any more as both Android M and N have also been built w/o libgcc.
Signed-off-by: Rob Herring
Reviewed-by: Chih-Wei Huang
---
src/gallium/targets/dri/Android.mk | 4
1
Currently, building with "mmma external/mesa3d" which builds all targets
and dependencies is broken for targets that require LLVM. This is due to
the build settings depending on MESA_ENABLE_LLVM. Instead of using a
conditional in the global Android.common.mk, make all the components that
need LLVM
Add libelf as a library dependency rather than explicitly listing its
include paths. This should work for Android M and later which have the
necessary exported directories in libelf.
Signed-off-by: Rob Herring
---
Android.common.mk | 2 --
src/gallium/targets/dri/Android.mk contains lots of conditional for
individual drivers. Let's move these details into the individual driver
makefiles.
In the process, align the make driver conditionals with automake
(i.e. HAVE_GALLIUM_*).
Signed-off-by: Rob Herring
---
Add a driver string "all" so that if BOARD_GPU_DRIVERS is set to "all",
all the drivers are enabled in the build. This makes build testing all
drivers easier to maintain.
Signed-off-by: Rob Herring
---
Android.mk | 5 +
1 file changed, 5 insertions(+)
diff --git
Mesa no longer supports LLVM 3.5 for any targets we support.
Android-x86 adds support for llvmpipe which could work, but android-x86
for L is using mesa 11.0 anyway.
Dropping this support enables clean-up of libelf dependencies.
Signed-off-by: Rob Herring
Reviewed-by: Chih-Wei
It is not necessary to filter driver and winsys directories based on the
list of enabled drivers. Selecting the included driver libraries or not is
sufficient to control what is built.
Signed-off-by: Rob Herring
Reviewed-by: Chih-Wei Huang
---
Android.mk
Building libmesa_amd_common fails with:
external/mesa/src/amd/common/ac_shader_info.c:23:10: fatal error: 'nir/nir.h'
file not found
^
external/mesa/src/compiler/nir/nir.h:48:10: fatal error: 'nir_opcodes.h' file
not found
^
libmesa_amd_common now depends on libmesa_nir, so
A build of only swrast is broken as the Android EGL now depends on
libdrm as does GBM. While we could make EGL conditionally depend on
libdrm, we probably want to enable kms_dri winsys as well and that will
need libdrm enabled. So just always enable libdrm and simplify the
Android makefiles a bit.
This is a series of clean-ups and fixes to the Android build files.
Overall, it removes a lot of the if conditions that aren't necessary for
normal builds and also enables "mmma external/mesa3d" to build
everything for easier build testing.
This fixes LLVM enabled builds on master though that
Explicitly including libcxx includes is not necessary at least on
Android M and later. It appears that libc++ was made the default in
commit "Make libc++ the default STL." in Android build system post L.
However, if L support is still needed, using "LOCAL_CXX_STL=libc++" is
the preferred way.
From: Mauro Rossi
Inspired by Chih-Wei Huang and Zhen Wu similar patches
Linking against llvm with both static and shared may be avoided,
provided that libLLVM shared library for device supports
whole static R600/AMDGPU libraries, necessary for radeonsi/amdgpu.
Add exported include paths rather than explicitly adding the includes
in each user of the common AMD libs.
Signed-off-by: Rob Herring
Reviewed-by: Chih-Wei Huang
---
src/amd/Android.addrlib.mk | 6 ++
src/amd/Android.common.mk
Andres Gomez writes:
> v2: left code style/formatting corrections out.
>
> Signed-off-by: Andres Gomez
Reviewed-by: Eric Anholt
signature.asc
Description: PGP signature
___
mesa-dev mailing
1 - 100 of 238 matches
Mail list logo