diff mbox series

[meta-arago,kirkstone] meta-arago-distro: add meta-clang dependency and llvmpipe

Message ID 20230822012914.3184998-2-rs@ti.com
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,kirkstone] meta-arago-distro: add meta-clang dependency and llvmpipe | expand

Commit Message

Randolph Sapp Aug. 22, 2023, 1:29 a.m. UTC
From: Randolph Sapp <rs@ti.com>

Devices that must use software rendering should use llvmpipe over the
default softpipe. There are significant performance improvements
associated with this change for devices like AM62A which lack a GPU.

Signed-off-by: Randolph Sapp <rs@ti.com>
---
 meta-arago-distro/conf/distro/arago.conf | 4 ++++
 meta-arago-distro/conf/layer.conf        | 1 +
 2 files changed, 5 insertions(+)

Comments

Ryan Eatmon Aug. 22, 2023, 8:37 p.m. UTC | #1
On 8/21/2023 8:29 PM, rs@ti.com wrote:
> From: Randolph Sapp <rs@ti.com>
> 
> Devices that must use software rendering should use llvmpipe over the
> default softpipe. There are significant performance improvements
> associated with this change for devices like AM62A which lack a GPU.
> 
> Signed-off-by: Randolph Sapp <rs@ti.com>
> ---
>   meta-arago-distro/conf/distro/arago.conf | 4 ++++
>   meta-arago-distro/conf/layer.conf        | 1 +
>   2 files changed, 5 insertions(+)
> 
> diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-distro/conf/distro/arago.conf
> index 48f053bb..ffa10cfa 100644
> --- a/meta-arago-distro/conf/distro/arago.conf
> +++ b/meta-arago-distro/conf/distro/arago.conf
> @@ -102,6 +102,10 @@ PACKAGECONFIG:append:pn-piglit = " \
>   PACKAGECONFIG:pn-opengl-es-cts = "${@bb.utils.contains("DISTRO_FEATURES", "wayland", "wayland", "surfaceless", d)}"
>   PACKAGECONFIG:pn-vulkan-cts = "${@bb.utils.contains("DISTRO_FEATURES", "wayland", "wayland", "surfaceless", d)}"
>   
> +# Use llvmpipe for better performance with software rendering
> +PACKAGECONFIG:append:pn-mesa = "gallium-llvm"
> +PACKAGECONFIG:append:pn-mesa-pvr = "gallium-llvm"

In addition to the test that I'm running on omapl138-lcdk, the above 
append lines need spaces at the beginning of the string.  I'll let you 
know the results of the test for a v2 of this patch.

> +
>   # Enable encryption support in open62541
>   PACKAGECONFIG:append:pn-open62541 = " encrypt"
>   
> diff --git a/meta-arago-distro/conf/layer.conf b/meta-arago-distro/conf/layer.conf
> index 5de46fc9..02d5beaa 100644
> --- a/meta-arago-distro/conf/layer.conf
> +++ b/meta-arago-distro/conf/layer.conf
> @@ -19,6 +19,7 @@ LAYERDEPENDS_meta-arago-distro = " \
>       meta-python \
>       qt5-layer \
>       meta-arago-extras \
> +    clang-layer \
>   "
>   
>   LAYERRECOMMENDS_meta-arago-distro = " \
diff mbox series

Patch

diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-distro/conf/distro/arago.conf
index 48f053bb..ffa10cfa 100644
--- a/meta-arago-distro/conf/distro/arago.conf
+++ b/meta-arago-distro/conf/distro/arago.conf
@@ -102,6 +102,10 @@  PACKAGECONFIG:append:pn-piglit = " \
 PACKAGECONFIG:pn-opengl-es-cts = "${@bb.utils.contains("DISTRO_FEATURES", "wayland", "wayland", "surfaceless", d)}"
 PACKAGECONFIG:pn-vulkan-cts = "${@bb.utils.contains("DISTRO_FEATURES", "wayland", "wayland", "surfaceless", d)}"
 
+# Use llvmpipe for better performance with software rendering
+PACKAGECONFIG:append:pn-mesa = "gallium-llvm"
+PACKAGECONFIG:append:pn-mesa-pvr = "gallium-llvm"
+
 # Enable encryption support in open62541
 PACKAGECONFIG:append:pn-open62541 = " encrypt"
 
diff --git a/meta-arago-distro/conf/layer.conf b/meta-arago-distro/conf/layer.conf
index 5de46fc9..02d5beaa 100644
--- a/meta-arago-distro/conf/layer.conf
+++ b/meta-arago-distro/conf/layer.conf
@@ -19,6 +19,7 @@  LAYERDEPENDS_meta-arago-distro = " \
     meta-python \
     qt5-layer \
     meta-arago-extras \
+    clang-layer \
 "
 
 LAYERRECOMMENDS_meta-arago-distro = " \