All of lore.kernel.org
 help / color / mirror / Atom feed
* [master] weston: Add SoC performance monitoring utilities
@ 2017-03-13 18:31 Karthik Ramanan
  2017-03-14 21:52 ` Denys Dmytriyenko
  2017-03-14 22:22 ` Denys Dmytriyenko
  0 siblings, 2 replies; 6+ messages in thread
From: Karthik Ramanan @ 2017-03-13 18:31 UTC (permalink / raw)
  To: meta-arago

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



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [master] weston: Add SoC performance monitoring utilities
  2017-03-13 18:31 [master] weston: Add SoC performance monitoring utilities Karthik Ramanan
@ 2017-03-14 21:52 ` Denys Dmytriyenko
  2017-03-14 22:22 ` Denys Dmytriyenko
  1 sibling, 0 replies; 6+ messages in thread
From: Denys Dmytriyenko @ 2017-03-14 21:52 UTC (permalink / raw)
  To: Karthik Ramanan; +Cc: meta-arago

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [master] weston: Add SoC performance monitoring utilities
  2017-03-13 18:31 [master] weston: Add SoC performance monitoring utilities Karthik Ramanan
  2017-03-14 21:52 ` Denys Dmytriyenko
@ 2017-03-14 22:22 ` Denys Dmytriyenko
  2017-03-15  2:40   ` Karthik Ramanan
  1 sibling, 1 reply; 6+ messages in thread
From: Denys Dmytriyenko @ 2017-03-14 22:22 UTC (permalink / raw)
  To: Karthik Ramanan; +Cc: meta-arago

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [master] weston: Add SoC performance monitoring utilities
  2017-03-14 22:22 ` Denys Dmytriyenko
@ 2017-03-15  2:40   ` Karthik Ramanan
  2017-03-15 15:26     ` Denys Dmytriyenko
  0 siblings, 1 reply; 6+ messages in thread
From: Karthik Ramanan @ 2017-03-15  2:40 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: meta-arago

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.



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [master] weston: Add SoC performance monitoring utilities
  2017-03-15  2:40   ` Karthik Ramanan
@ 2017-03-15 15:26     ` Denys Dmytriyenko
  2017-03-16  6:15       ` Karthik Ramanan
  0 siblings, 1 reply; 6+ messages in thread
From: Denys Dmytriyenko @ 2017-03-15 15:26 UTC (permalink / raw)
  To: Karthik Ramanan; +Cc: meta-arago

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [master] weston: Add SoC performance monitoring utilities
  2017-03-15 15:26     ` Denys Dmytriyenko
@ 2017-03-16  6:15       ` Karthik Ramanan
  0 siblings, 0 replies; 6+ messages in thread
From: Karthik Ramanan @ 2017-03-16  6:15 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: meta-arago



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.



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-03-16  6:15 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-13 18:31 [master] weston: Add SoC performance monitoring utilities Karthik Ramanan
2017-03-14 21:52 ` Denys Dmytriyenko
2017-03-14 22:22 ` Denys Dmytriyenko
2017-03-15  2:40   ` Karthik Ramanan
2017-03-15 15:26     ` Denys Dmytriyenko
2017-03-16  6:15       ` Karthik Ramanan

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.