All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v1] OpenCSD: Support for Arm CoreSight decode lib
@ 2020-03-04 12:28 Leo Yan
  2020-03-04 16:05 ` Scott Murray
  0 siblings, 1 reply; 8+ messages in thread
From: Leo Yan @ 2020-03-04 12:28 UTC (permalink / raw)
  To: Khem Raj, openembeded-devel, Mike Leach, Mathieu Poirier

This recipe is to support OpenCSD, which is an open source
CoreSight trace decode library and utility.

Signed-off-by: Leo Yan <leo.yan@linaro.org>
---
 .../recipes-devtools/opencsd/opencsd_git.bb   | 28 +++++++++++++++++++
 1 file changed, 28 insertions(+)
 create mode 100644 meta-oe/recipes-devtools/opencsd/opencsd_git.bb

diff --git a/meta-oe/recipes-devtools/opencsd/opencsd_git.bb b/meta-oe/recipes-devtools/opencsd/opencsd_git.bb
new file mode 100644
index 000000000..0c3b4837d
--- /dev/null
+++ b/meta-oe/recipes-devtools/opencsd/opencsd_git.bb
@@ -0,0 +1,28 @@
+SUMMARY = "OpenCSD - An open source CoreSight(tm) Trace Decode library"
+HOMEPAGE = "https://github.com/Linaro/OpenCSD"
+LICENSE = "BSD-3-Clause"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=ad8cb685eb324d2fa2530b985a43f3e5"
+
+SRC_URI = "git://github.com/Linaro/OpenCSD;protocol=http;branch=master"
+SRCREV = "03c194117971e4ad0598df29395757ced2e6e9bd"
+
+S = "${WORKDIR}/git"
+
+COMPATIBLE_HOST = "(x86_64.*|aarch64.*)-linux"
+
+EXTRA_OEMAKE = "ARCH='${TARGET_ARCH}' \
+                CROSS_COMPILE='${TARGET_SYS}-' \
+                CC='${CC}' \
+                CXX='${CXX}' \
+                LINKER='${CXX}' \
+                LINUX64=1 \
+                DEBUG=1 \
+                "
+
+do_compile() {
+    ( cd ${S}/decoder/build/linux; oe_runmake ${EXTRA_OEMAKE}; cd - )
+}
+
+do_install() {
+    ( cd ${S}/decoder/build/linux; oe_runmake PREFIX=${D}/usr install; cd - )
+}
-- 
2.17.1



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

* Re: [PATCH v1] OpenCSD: Support for Arm CoreSight decode lib
  2020-03-04 12:28 [PATCH v1] OpenCSD: Support for Arm CoreSight decode lib Leo Yan
@ 2020-03-04 16:05 ` Scott Murray
  2020-03-04 16:36   ` Philip Balister
  2020-03-04 19:19   ` Khem Raj
  0 siblings, 2 replies; 8+ messages in thread
From: Scott Murray @ 2020-03-04 16:05 UTC (permalink / raw)
  To: Leo Yan; +Cc: Mike Leach, Mathieu Poirier, openembeded-devel

On Wed, 4 Mar 2020, Leo Yan wrote:

> This recipe is to support OpenCSD, which is an open source
> CoreSight trace decode library and utility.

Just a thought, this seems like the kind of thing that might make sense
for the new meta-arm layer that Jon has been working up?

> Signed-off-by: Leo Yan <leo.yan@linaro.org>
> ---
>  .../recipes-devtools/opencsd/opencsd_git.bb   | 28 +++++++++++++++++++
>  1 file changed, 28 insertions(+)
>  create mode 100644 meta-oe/recipes-devtools/opencsd/opencsd_git.bb
>
> diff --git a/meta-oe/recipes-devtools/opencsd/opencsd_git.bb b/meta-oe/recipes-devtools/opencsd/opencsd_git.bb
> new file mode 100644
> index 000000000..0c3b4837d
> --- /dev/null
> +++ b/meta-oe/recipes-devtools/opencsd/opencsd_git.bb
> @@ -0,0 +1,28 @@
> +SUMMARY = "OpenCSD - An open source CoreSight(tm) Trace Decode library"
> +HOMEPAGE = "https://github.com/Linaro/OpenCSD"
> +LICENSE = "BSD-3-Clause"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=ad8cb685eb324d2fa2530b985a43f3e5"
> +
> +SRC_URI = "git://github.com/Linaro/OpenCSD;protocol=http;branch=master"
> +SRCREV = "03c194117971e4ad0598df29395757ced2e6e9bd"
> +
> +S = "${WORKDIR}/git"
> +
> +COMPATIBLE_HOST = "(x86_64.*|aarch64.*)-linux"
> +
> +EXTRA_OEMAKE = "ARCH='${TARGET_ARCH}' \
> +                CROSS_COMPILE='${TARGET_SYS}-' \
> +                CC='${CC}' \
> +                CXX='${CXX}' \
> +                LINKER='${CXX}' \
> +                LINUX64=1 \
> +                DEBUG=1 \
> +                "
> +
> +do_compile() {
> +    ( cd ${S}/decoder/build/linux; oe_runmake ${EXTRA_OEMAKE}; cd - )
> +}
> +
> +do_install() {
> +    ( cd ${S}/decoder/build/linux; oe_runmake PREFIX=${D}/usr install; cd - )
> +}
>


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

* Re: [PATCH v1] OpenCSD: Support for Arm CoreSight decode lib
  2020-03-04 16:05 ` Scott Murray
@ 2020-03-04 16:36   ` Philip Balister
  2020-03-04 17:02     ` Josef Holzmayr
  2020-03-04 19:19   ` Khem Raj
  1 sibling, 1 reply; 8+ messages in thread
From: Philip Balister @ 2020-03-04 16:36 UTC (permalink / raw)
  To: Scott Murray, Leo Yan; +Cc: openembeded-devel, Mathieu Poirier, Mike Leach

On 3/4/20 11:05 AM, Scott Murray wrote:
> On Wed, 4 Mar 2020, Leo Yan wrote:
> 
>> This recipe is to support OpenCSD, which is an open source
>> CoreSight trace decode library and utility.
> 
> Just a thought, this seems like the kind of thing that might make sense
> for the new meta-arm layer that Jon has been working up?

I had the exact thought and am really glad I read this before saying the
same thing!

Philip

> 
>> Signed-off-by: Leo Yan <leo.yan@linaro.org>
>> ---
>>  .../recipes-devtools/opencsd/opencsd_git.bb   | 28 +++++++++++++++++++
>>  1 file changed, 28 insertions(+)
>>  create mode 100644 meta-oe/recipes-devtools/opencsd/opencsd_git.bb
>>
>> diff --git a/meta-oe/recipes-devtools/opencsd/opencsd_git.bb b/meta-oe/recipes-devtools/opencsd/opencsd_git.bb
>> new file mode 100644
>> index 000000000..0c3b4837d
>> --- /dev/null
>> +++ b/meta-oe/recipes-devtools/opencsd/opencsd_git.bb
>> @@ -0,0 +1,28 @@
>> +SUMMARY = "OpenCSD - An open source CoreSight(tm) Trace Decode library"
>> +HOMEPAGE = "https://github.com/Linaro/OpenCSD"
>> +LICENSE = "BSD-3-Clause"
>> +LIC_FILES_CHKSUM = "file://LICENSE;md5=ad8cb685eb324d2fa2530b985a43f3e5"
>> +
>> +SRC_URI = "git://github.com/Linaro/OpenCSD;protocol=http;branch=master"
>> +SRCREV = "03c194117971e4ad0598df29395757ced2e6e9bd"
>> +
>> +S = "${WORKDIR}/git"
>> +
>> +COMPATIBLE_HOST = "(x86_64.*|aarch64.*)-linux"
>> +
>> +EXTRA_OEMAKE = "ARCH='${TARGET_ARCH}' \
>> +                CROSS_COMPILE='${TARGET_SYS}-' \
>> +                CC='${CC}' \
>> +                CXX='${CXX}' \
>> +                LINKER='${CXX}' \
>> +                LINUX64=1 \
>> +                DEBUG=1 \
>> +                "
>> +
>> +do_compile() {
>> +    ( cd ${S}/decoder/build/linux; oe_runmake ${EXTRA_OEMAKE}; cd - )
>> +}
>> +
>> +do_install() {
>> +    ( cd ${S}/decoder/build/linux; oe_runmake PREFIX=${D}/usr install; cd - )
>> +}
>>


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

* Re: [PATCH v1] OpenCSD: Support for Arm CoreSight decode lib
  2020-03-04 16:36   ` Philip Balister
@ 2020-03-04 17:02     ` Josef Holzmayr
  0 siblings, 0 replies; 8+ messages in thread
From: Josef Holzmayr @ 2020-03-04 17:02 UTC (permalink / raw)
  To: Philip Balister; +Cc: Mike Leach, openembeded-devel, Mathieu Poirier

On Wed, Mar 04, 2020 at 11:36:38AM -0500, Philip Balister wrote:
> On 3/4/20 11:05 AM, Scott Murray wrote:
> > On Wed, 4 Mar 2020, Leo Yan wrote:
> > 
> >> This recipe is to support OpenCSD, which is an open source
> >> CoreSight trace decode library and utility.
> > 
> > Just a thought, this seems like the kind of thing that might make sense
> > for the new meta-arm layer that Jon has been working up?
> 
> I had the exact thought and am really glad I read this before saying the
> same thing!

+1

> 
> Philip
> 
> > 
> >> Signed-off-by: Leo Yan <leo.yan@linaro.org>
> >> ---
> >>  .../recipes-devtools/opencsd/opencsd_git.bb   | 28 +++++++++++++++++++
> >>  1 file changed, 28 insertions(+)
> >>  create mode 100644 meta-oe/recipes-devtools/opencsd/opencsd_git.bb
> >>
> >> diff --git a/meta-oe/recipes-devtools/opencsd/opencsd_git.bb b/meta-oe/recipes-devtools/opencsd/opencsd_git.bb
> >> new file mode 100644
> >> index 000000000..0c3b4837d
> >> --- /dev/null
> >> +++ b/meta-oe/recipes-devtools/opencsd/opencsd_git.bb
> >> @@ -0,0 +1,28 @@
> >> +SUMMARY = "OpenCSD - An open source CoreSight(tm) Trace Decode library"
> >> +HOMEPAGE = "https://github.com/Linaro/OpenCSD"
> >> +LICENSE = "BSD-3-Clause"
> >> +LIC_FILES_CHKSUM = "file://LICENSE;md5=ad8cb685eb324d2fa2530b985a43f3e5"
> >> +
> >> +SRC_URI = "git://github.com/Linaro/OpenCSD;protocol=http;branch=master"
> >> +SRCREV = "03c194117971e4ad0598df29395757ced2e6e9bd"
> >> +
> >> +S = "${WORKDIR}/git"
> >> +
> >> +COMPATIBLE_HOST = "(x86_64.*|aarch64.*)-linux"
> >> +
> >> +EXTRA_OEMAKE = "ARCH='${TARGET_ARCH}' \
> >> +                CROSS_COMPILE='${TARGET_SYS}-' \
> >> +                CC='${CC}' \
> >> +                CXX='${CXX}' \
> >> +                LINKER='${CXX}' \
> >> +                LINUX64=1 \
> >> +                DEBUG=1 \
> >> +                "
> >> +
> >> +do_compile() {
> >> +    ( cd ${S}/decoder/build/linux; oe_runmake ${EXTRA_OEMAKE}; cd - )
> >> +}
> >> +
> >> +do_install() {
> >> +    ( cd ${S}/decoder/build/linux; oe_runmake PREFIX=${D}/usr install; cd - )
> >> +}
> >>
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
———————————————
Josef Holzmayr
Software Developer Embedded Systems

Tel: +49 8444 9204-48
Fax: +49 8444 9204-50

R-S-I Elektrotechnik GmbH & Co. KG
Woelkestrasse 11
D-85301 Schweitenkirchen
www.rsi-elektrotechnik.de
———————————————
Amtsgericht Ingolstadt – GmbH: HRB 191328 – KG: HRA 170393
Geschäftsführer: Dr.-Ing. Michael Sorg, Dipl.-Ing. Franz Sorg
Ust-IdNr: DE 128592548 

_____________________________________________________________
Amtsgericht Ingolstadt - GmbH: HRB 191328 - KG: HRA 170363
Geschäftsführer: Dr.-Ing. Michael Sorg, Dipl.-Ing. Franz Sorg
USt-IdNr.: DE 128592548



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

* Re: [PATCH v1] OpenCSD: Support for Arm CoreSight decode lib
  2020-03-04 16:05 ` Scott Murray
  2020-03-04 16:36   ` Philip Balister
@ 2020-03-04 19:19   ` Khem Raj
  2020-03-05  1:43     ` Leo Yan
  1 sibling, 1 reply; 8+ messages in thread
From: Khem Raj @ 2020-03-04 19:19 UTC (permalink / raw)
  To: Scott Murray, Leo Yan; +Cc: openembeded-devel, Mathieu Poirier, Mike Leach



On 3/4/20 8:05 AM, Scott Murray wrote:
> On Wed, 4 Mar 2020, Leo Yan wrote:
> 
>> This recipe is to support OpenCSD, which is an open source
>> CoreSight trace decode library and utility.
> 
> Just a thought, this seems like the kind of thing that might make sense
> for the new meta-arm layer that Jon has been working up?
> 

it also says x86_64 in COMPATIBLE_HOST, while coresight seems to be arm
specific is this not supposed to be so ?

>> Signed-off-by: Leo Yan <leo.yan@linaro.org>
>> ---
>>   .../recipes-devtools/opencsd/opencsd_git.bb   | 28 +++++++++++++++++++
>>   1 file changed, 28 insertions(+)
>>   create mode 100644 meta-oe/recipes-devtools/opencsd/opencsd_git.bb
>>
>> diff --git a/meta-oe/recipes-devtools/opencsd/opencsd_git.bb b/meta-oe/recipes-devtools/opencsd/opencsd_git.bb
>> new file mode 100644
>> index 000000000..0c3b4837d
>> --- /dev/null
>> +++ b/meta-oe/recipes-devtools/opencsd/opencsd_git.bb
>> @@ -0,0 +1,28 @@
>> +SUMMARY = "OpenCSD - An open source CoreSight(tm) Trace Decode library"
>> +HOMEPAGE = "https://github.com/Linaro/OpenCSD"
>> +LICENSE = "BSD-3-Clause"
>> +LIC_FILES_CHKSUM = "file://LICENSE;md5=ad8cb685eb324d2fa2530b985a43f3e5"
>> +
>> +SRC_URI = "git://github.com/Linaro/OpenCSD;protocol=http;branch=master"
>> +SRCREV = "03c194117971e4ad0598df29395757ced2e6e9bd"
>> +
>> +S = "${WORKDIR}/git"
>> +
>> +COMPATIBLE_HOST = "(x86_64.*|aarch64.*)-linux"
>> +
>> +EXTRA_OEMAKE = "ARCH='${TARGET_ARCH}' \
>> +                CROSS_COMPILE='${TARGET_SYS}-' \
>> +                CC='${CC}' \
>> +                CXX='${CXX}' \
>> +                LINKER='${CXX}' \
>> +                LINUX64=1 \
>> +                DEBUG=1 \
>> +                "
>> +
>> +do_compile() {
>> +    ( cd ${S}/decoder/build/linux; oe_runmake ${EXTRA_OEMAKE}; cd - )
>> +}
>> +
>> +do_install() {
>> +    ( cd ${S}/decoder/build/linux; oe_runmake PREFIX=${D}/usr install; cd - )
>> +}
>>


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

* Re: [PATCH v1] OpenCSD: Support for Arm CoreSight decode lib
  2020-03-04 19:19   ` Khem Raj
@ 2020-03-05  1:43     ` Leo Yan
  2020-03-05  1:49       ` Khem Raj
  0 siblings, 1 reply; 8+ messages in thread
From: Leo Yan @ 2020-03-05  1:43 UTC (permalink / raw)
  To: Khem Raj; +Cc: openembeded-devel, Mathieu Poirier, Mike Leach

Hi all,

On Wed, Mar 04, 2020 at 11:19:37AM -0800, Khem Raj wrote:
> 
> 
> On 3/4/20 8:05 AM, Scott Murray wrote:
> > On Wed, 4 Mar 2020, Leo Yan wrote:
> > 
> > > This recipe is to support OpenCSD, which is an open source
> > > CoreSight trace decode library and utility.
> > 
> > Just a thought, this seems like the kind of thing that might make sense
> > for the new meta-arm layer that Jon has been working up?

Thanks for the suggestion, Scott.

To be honest, I didn't know meta-arm layer before; seems to me it makes
sense to integrate this recipe into meta-arm rather than
meta-openembedded.  Will explain more for Khem's question.

> it also says x86_64 in COMPATIBLE_HOST, while coresight seems to be arm
> specific is this not supposed to be so ?

OpenCSD lib is dedicated to be used for Arm CoreSight IP, but it can
be built both for x86_64's and Arm64's distro.

The main usage case is to enable OpenCSD lib for Arm64 building, and the
users can decode the trace data with Perf/OpenCSD in the same distro.
Though I set COMPATIBLE_HOST for x86_64, it would be seldom to do cross
analysis (e.g. capture trace data in OE/Arm64 and copy the perf data to
OE/x86_64 for analysis).

For this reason, it's good to maintain this recipe in meta-arm.  If no
objection, I will prepare patch for meta-arm.

Thanks,
Leo


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

* Re: [PATCH v1] OpenCSD: Support for Arm CoreSight decode lib
  2020-03-05  1:43     ` Leo Yan
@ 2020-03-05  1:49       ` Khem Raj
  2020-03-05  2:00         ` Leo Yan
  0 siblings, 1 reply; 8+ messages in thread
From: Khem Raj @ 2020-03-05  1:49 UTC (permalink / raw)
  To: Leo Yan; +Cc: openembeded-devel, Mathieu Poirier, Mike Leach

On Wed, Mar 4, 2020 at 5:43 PM Leo Yan <leo.yan@linaro.org> wrote:
>
> Hi all,
>
> On Wed, Mar 04, 2020 at 11:19:37AM -0800, Khem Raj wrote:
> >
> >
> > On 3/4/20 8:05 AM, Scott Murray wrote:
> > > On Wed, 4 Mar 2020, Leo Yan wrote:
> > >
> > > > This recipe is to support OpenCSD, which is an open source
> > > > CoreSight trace decode library and utility.
> > >
> > > Just a thought, this seems like the kind of thing that might make sense
> > > for the new meta-arm layer that Jon has been working up?
>
> Thanks for the suggestion, Scott.
>
> To be honest, I didn't know meta-arm layer before; seems to me it makes
> sense to integrate this recipe into meta-arm rather than
> meta-openembedded.  Will explain more for Khem's question.
>
> > it also says x86_64 in COMPATIBLE_HOST, while coresight seems to be arm
> > specific is this not supposed to be so ?
>
> OpenCSD lib is dedicated to be used for Arm CoreSight IP, but it can
> be built both for x86_64's and Arm64's distro.
>
> The main usage case is to enable OpenCSD lib for Arm64 building, and the
> users can decode the trace data with Perf/OpenCSD in the same distro.
> Though I set COMPATIBLE_HOST for x86_64, it would be seldom to do cross
> analysis (e.g. capture trace data in OE/Arm64 and copy the perf data to
> OE/x86_64 for analysis).
>
> For this reason, it's good to maintain this recipe in meta-arm.  If no
> objection, I will prepare patch for meta-arm.

Makes sense in meta-arm

>
> Thanks,
> Leo


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

* Re: [PATCH v1] OpenCSD: Support for Arm CoreSight decode lib
  2020-03-05  1:49       ` Khem Raj
@ 2020-03-05  2:00         ` Leo Yan
  0 siblings, 0 replies; 8+ messages in thread
From: Leo Yan @ 2020-03-05  2:00 UTC (permalink / raw)
  To: Khem Raj; +Cc: openembeded-devel, Mathieu Poirier, Mike Leach

On Wed, Mar 04, 2020 at 05:49:37PM -0800, Khem Raj wrote:
> On Wed, Mar 4, 2020 at 5:43 PM Leo Yan <leo.yan@linaro.org> wrote:
> >
> > Hi all,
> >
> > On Wed, Mar 04, 2020 at 11:19:37AM -0800, Khem Raj wrote:
> > >
> > >
> > > On 3/4/20 8:05 AM, Scott Murray wrote:
> > > > On Wed, 4 Mar 2020, Leo Yan wrote:
> > > >
> > > > > This recipe is to support OpenCSD, which is an open source
> > > > > CoreSight trace decode library and utility.
> > > >
> > > > Just a thought, this seems like the kind of thing that might make sense
> > > > for the new meta-arm layer that Jon has been working up?
> >
> > Thanks for the suggestion, Scott.
> >
> > To be honest, I didn't know meta-arm layer before; seems to me it makes
> > sense to integrate this recipe into meta-arm rather than
> > meta-openembedded.  Will explain more for Khem's question.
> >
> > > it also says x86_64 in COMPATIBLE_HOST, while coresight seems to be arm
> > > specific is this not supposed to be so ?
> >
> > OpenCSD lib is dedicated to be used for Arm CoreSight IP, but it can
> > be built both for x86_64's and Arm64's distro.
> >
> > The main usage case is to enable OpenCSD lib for Arm64 building, and the
> > users can decode the trace data with Perf/OpenCSD in the same distro.
> > Though I set COMPATIBLE_HOST for x86_64, it would be seldom to do cross
> > analysis (e.g. capture trace data in OE/Arm64 and copy the perf data to
> > OE/x86_64 for analysis).
> >
> > For this reason, it's good to maintain this recipe in meta-arm.  If no
> > objection, I will prepare patch for meta-arm.
> 
> Makes sense in meta-arm

Thanks for confirmation, Khem. And thanks all :)  Will do it.

Leo


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

end of thread, other threads:[~2020-03-05  2:00 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-04 12:28 [PATCH v1] OpenCSD: Support for Arm CoreSight decode lib Leo Yan
2020-03-04 16:05 ` Scott Murray
2020-03-04 16:36   ` Philip Balister
2020-03-04 17:02     ` Josef Holzmayr
2020-03-04 19:19   ` Khem Raj
2020-03-05  1:43     ` Leo Yan
2020-03-05  1:49       ` Khem Raj
2020-03-05  2:00         ` Leo Yan

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.