[master] weston: Add SoC performance monitoring utilities

[master] weston: Add SoC performance monitoring utilities

[Karthik Ramanan]

Signed-off-by: Karthik Ramanan <a0393906@ti.com>
---
 .../recipes-graphics/wayland/weston_1.11.0.bbappend              | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
index f8329c8..9b52df0 100644
--- a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
+++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
@@ -2,7 +2,7 @@
 PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
 PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
 
-PR_append = ".arago22"
+PR_append = ".arago25"
 
 FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
 
@@ -17,3 +17,10 @@ SRC_URI += " \
 	file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
 	file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
 "
+
+
+# SoC Performance Monitoring Utility
+SRC_URI += " \
+        file://0001-Add-soc-performance-monitor-utilites.patch \
+"
+
-- 
1.9.1

Re: [master] weston: Add SoC performance monitoring utilities

[Denys Dmytriyenko]

I presume this is also needed for morty...


On Tue, Mar 14, 2017 at 12:01:44AM +0530, Karthik Ramanan wrote:
> Signed-off-by: Karthik Ramanan <a0393906@ti.com>
> ---
>  .../recipes-graphics/wayland/weston_1.11.0.bbappend              | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> index f8329c8..9b52df0 100644
> --- a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> @@ -2,7 +2,7 @@
>  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
>  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
>  
> -PR_append = ".arago22"
> +PR_append = ".arago25"
>  
>  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>  
> @@ -17,3 +17,10 @@ SRC_URI += " \
>  	file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
>  	file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
>  "
> +
> +
> +# SoC Performance Monitoring Utility
> +SRC_URI += " \
> +        file://0001-Add-soc-performance-monitor-utilites.patch \
> +"
> +
> -- 
> 1.9.1
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [master] weston: Add SoC performance monitoring utilities

[Denys Dmytriyenko]

BTW, can this be done in a separate recipe to add additional app clients, 
instead of a patch to Weston? This is not scalable/portable at all...

-- 
Denys


On Tue, Mar 14, 2017 at 12:01:44AM +0530, Karthik Ramanan wrote:
> Signed-off-by: Karthik Ramanan <a0393906@ti.com>
> ---
>  .../recipes-graphics/wayland/weston_1.11.0.bbappend              | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> index f8329c8..9b52df0 100644
> --- a/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.11.0.bbappend
> @@ -2,7 +2,7 @@
>  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
>  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
>  
> -PR_append = ".arago22"
> +PR_append = ".arago25"
>  
>  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>  
> @@ -17,3 +17,10 @@ SRC_URI += " \
>  	file://0001-compositor-drm-fix-hotplug-weston-termination-proble.patch \
>  	file://0001-compositor-drm-support-RGB565-with-pixman-renderer.patch \
>  "
> +
> +
> +# SoC Performance Monitoring Utility
> +SRC_URI += " \
> +        file://0001-Add-soc-performance-monitor-utilites.patch \
> +"
> +
> -- 
> 1.9.1
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [master] weston: Add SoC performance monitoring utilities

[Karthik Ramanan]

Denys,

Yes this is for morty.
May be I missed a notification on this, I'm thinking morty=master? 
Should I mention morty explicitly?


On 15-Mar-17 3:52 AM, Denys Dmytriyenko wrote:
> BTW, can this be done in a separate recipe to add additional app clients,
> instead of a patch to Weston? This is not scalable/portable at all...
I take your point.
For now, lets have this patch in this way. There have been no changes to 
the application for sometime.
If it undergoes additional changes we will explore ways of combining 
them. There is an ongoing effort to consolidate applications so we will 
include this one as well.

Re: [master] weston: Add SoC performance monitoring utilities

[Denys Dmytriyenko]

On Wed, Mar 15, 2017 at 08:10:50AM +0530, Karthik Ramanan wrote:
> Denys,
> 
> Yes this is for morty.
> May be I missed a notification on this, I'm thinking morty=master?
> Should I mention morty explicitly?
> 
> 
> On 15-Mar-17 3:52 AM, Denys Dmytriyenko wrote:
> >BTW, can this be done in a separate recipe to add additional app clients,
> >instead of a patch to Weston? This is not scalable/portable at all...
> I take your point.
> For now, lets have this patch in this way. There have been no
> changes to the application for sometime.
> If it undergoes additional changes we will explore ways of combining
> them. There is an ongoing effort to consolidate applications so we
> will include this one as well.

A bit confused by your statement above...

I was talking about "SoC performance monitoring utilities" added by this patch 
being Weston clients. In other words, they are just apps talking to Weston and 
hence should be in own separate code bases built by separate recipes. Not as a 
patch on top of Weston...

Are you saying if you don't envision any changes to these SoC monitoring apps, 
then you'll keep on carrying them as a Weston patch?

-- 
Denys

Re: [master] weston: Add SoC performance monitoring utilities

[Karthik Ramanan]

On 15-Mar-17 8:56 PM, Denys Dmytriyenko wrote:
> On Wed, Mar 15, 2017 at 08:10:50AM +0530, Karthik Ramanan wrote:
>> Denys,
>>
>> Yes this is for morty.
>> May be I missed a notification on this, I'm thinking morty=master?
>> Should I mention morty explicitly?
>>
>>
>> On 15-Mar-17 3:52 AM, Denys Dmytriyenko wrote:
>>> BTW, can this be done in a separate recipe to add additional app clients,
>>> instead of a patch to Weston? This is not scalable/portable at all...
>> I take your point.
>> For now, lets have this patch in this way. There have been no
>> changes to the application for sometime.
>> If it undergoes additional changes we will explore ways of combining
>> them. There is an ongoing effort to consolidate applications so we
>> will include this one as well.
> A bit confused by your statement above...
>
> I was talking about "SoC performance monitoring utilities" added by this patch
> being Weston clients. In other words, they are just apps talking to Weston and
> hence should be in own separate code bases built by separate recipes. Not as a
> patch on top of Weston...
The "SoC performance monitoring utilities" is a weston application. At 
the time it
was developed, we reused some of the weston application utilities that 
are present
in the weston/clients folder - more specifically window.c and window.h 
utilities.
These are not libraries and not exported by Weston.

So the way it exists today, "SoC performance monitoring utilities" will 
be a weston
client and that it is the reason why it is applied to the Weston 
repository in this
manner.
> Are you saying if you don't envision any changes to these SoC monitoring apps,
> then you'll keep on carrying them as a Weston patch?
For now, yes that's the approach. This patch is an independent change to 
the
weston/clients. So it should apply to any version. But having said that, 
we will
try to explore the possibility of separating it to a different 
application without
this dependency, but that will have to be for later.