Re: [yocto] [PATCH 1/1] add scheedtool-dl for testing sched_deadline use jiri's github as a main repo

2013-03-16 Thread Insop Song
On Sat, Mar 16, 2013 at 10:01 PM, Khem Raj  wrote:
>
> On Mar 15, 2013, at 4:47 AM, Insop Song  wrote:
>
>> +SRCREV = "${AUTOREV}"
>
> IMO find a good revision and send that one here. you can use AUTOREV 
> internally for your
> development thats all fine


Hi Khem,

Thank you for the review.
I've updated that already as Bruce suggested the same as well in the
latest patch update.

Here is the update patch for your reference.

Thank you.

Insop

+From 041b2b65889c44c47f90313c5bf9ce61effa96df Mon Sep 17 00:00:00 2001
+From: Insop Song 
+Date: Sat, 9 Mar 2013 01:23:24 -0800
+Subject: [PATCH] set "--with-deadline" as a default to go around yocto build
+ error on deadline enabled
+
+---
+ configure.ac  |2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/configure.ac  b/configure.ac
+index a5e2990..64e57e4 100644
+--- a/configure.ac
 b/configure.ac
+@@ -34,7 +34,7 @@ AC_ARG_WITH([deadline],
+   [AS_HELP_STRING([--with-deadline],
+  [Add support for SCHED_DEADLINE])],
+   [],
+-  [with_deadline=no])
++  [with_deadline=yes])
+
+ AS_IF([test "x$with_deadline" != xno],
+   [AC_DEFINE([DLSCHED], [1], [Define if you have SCHED_DEADLINE support])
+--
+1.7.9.5
+
diff --git a/recipes-tools/rt-app/rt-app.bb  b/recipes-tools/rt-app/rt-app.bb
new file mode 100644
index 000..acce437
--- /dev/null
+++ b/recipes-tools/rt-app/rt-app.bb
@@ -0,0 +1,28 @@
+DESCRIPTION = "rt-app is a test application that starts multiple
periodic threads in order to simulate a real-time periodic load"
+SECTION = "devel"
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM =
"file://COPYING;endline=339;md5=e43fc16fccd8519fba405f0a0ff6e8a3"
+PR = "r1"
+
+SRCREV = "e81cbe9f76ee298feded17bb7455d4deb2fb32af"
+
+SRC_URI = "git://github.com/gbagnoli/rt-app.git \
+   file://0001-set-with-deadline-as-a-default-to-go-around-yocto-bu.patch \
+   "
+
+S = "${WORKDIR}/git"
+
+inherit autotools gettext pkgconfig
+
+do_configure_prepend() {
+   autoreconf --install --verbose --force
+}
+
+do_install() {
+   install -d ${D}${bindir}
+   install -m 0755 src/rt-app ${D}${bindir}
+}
+
+FILES_{PN} = "${bindir}/rt-app"
+
+PARALLEL_MAKE = ""
diff --git a/recipes-tools/schedtool-dl/schedtool-dl.bb
b/recipes-tools/schedtool-dl/schedtool-dl.bb
new file mode 100644
index 000..5592175
--- /dev/null
+++ b/recipes-tools/schedtool-dl/schedtool-dl.bb
@@ -0,0 +1,23 @@
+DESCRIPTION = "schedtool-dl (scheduler test tool) for deadline scheduler"
+SECTION = "devel"
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM =
"file://Makefile;endline=55;md5=e4b4e8ed9c2132e1d727a1bb5e3bd984"
+PR = "r1"
+
+SRC_URI = "git://github.com/jlelli/schedtool-dl.git;protocol=git"
+SRCREV = "cd4e3f2f5a33c3266e25e447e8efbacb3b8c88da"
+
+S = "${WORKDIR}/git"
+
+do_compile() {
+   oe_runmake
+}
+
+do_install() {
+   install -d ${D}${bindir}
+   install -m 0755 schedtool ${D}${bindir}
+}
+
+FILES_{PN} = "${bindir}/schedtool"
+
+PARALLEL_MAKE = ""
___
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto


Re: [yocto] [PATCH 1/1] add scheedtool-dl for testing sched_deadline use jiri's github as a main repo

2013-03-16 Thread Khem Raj

On Mar 15, 2013, at 4:47 AM, Insop Song  wrote:

> +SRCREV = "${AUTOREV}"

IMO find a good revision and send that one here. you can use AUTOREV internally 
for your
development thats all fine
___
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto


Re: [yocto] [PATCH 1/1] add scheedtool-dl for testing sched_deadline use jiri's github as a main repo

2013-03-16 Thread Insop Song
On Fri, Mar 15, 2013 at 7:46 AM, Bruce Ashfield
 wrote:
> On 13-03-15 04:47 AM, Insop Song wrote:
>>
>> ---
>>   recipes-extended/images/core-image-realtime.bb |1 +
>>   recipes-tools/schedtool-dl/schedtool-dl.bb |   23
>> +++
>>   2 files changed, 24 insertions(+)
>>   create mode 100644 recipes-tools/schedtool-dl/schedtool-dl.bb
>>
>> diff --git a/recipes-extended/images/core-image-realtime.bb
>> b/recipes-extended/images/core-image-realtime.bb
>> index 624aacf..386b16b 100644
>> --- a/recipes-extended/images/core-image-realtime.bb
>> +++ b/recipes-extended/images/core-image-realtime.bb
>> @@ -8,5 +8,6 @@ EXTRA_IMAGE_FEATURES = "tools-debug debug-tweaks
>> tools-profile dbg-pkgs"
>>   IMAGE_INSTALL = "\
>> ${CORE_IMAGE_BASE_INSTALL} \
>> rt-app \
>> +   schedtool-dl \
>> libgcc \
>> "
>
>
> This part of the patch is still a problem, so I'll drop  it on merge.
> We still need to rationalize why we can't extend the oe-core
> rt images:
>
> ./recipes-rt/images/core-image-rt.bb
> ./recipes-rt/images/core-image-rt-sdk.bb
>
> For this addition. Until we work through that, you can carry a local
> patch, or add the tools back via your local.conf adding it to IMAGE_INSTALL.
>
>

I put the "libgcc" because rt-app complained during finish regarding
pthread and libgcc during my previous meta-realtime/meta-dl, which was
using core-image-minimal.

Since, I updated my image bases this time, I might not need it.
I will test without "libgcc" and will update the patch.

I've updated my poky tree to the latest, and I am having trouble due
to "neard" package failure (I post a question on this).
That's why it's taking long to verify and update the patch.


>> diff --git a/recipes-tools/schedtool-dl/schedtool-dl.bb
>> b/recipes-tools/schedtool-dl/schedtool-dl.bb
>> new file mode 100644
>> index 000..6b298f1
>> --- /dev/null
>> +++ b/recipes-tools/schedtool-dl/schedtool-dl.bb
>> @@ -0,0 +1,23 @@
>> +DESCRIPTION = "schedtool-dl (scheduler test tool) for deadline scheduler"
>> +SECTION = "devel"
>> +LICENSE = "GPLv2"
>> +LIC_FILES_CHKSUM =
>> "file://Makefile;endline=55;md5=e4b4e8ed9c2132e1d727a1bb5e3bd984"
>> +PR = "r1"
>> +
>> +SRC_URI = "git://github.com/jlelli/schedtool-dl.git;protocol=git"
>> +SRCREV = "${AUTOREV}"
>
>
> We also want to lock down the SRCREV vs autorev. I'd expect that going
> forward,
> you can maintain/update this once we've merged it, so when a SRCREV
> is known to be good, you can send update patches with the new values
> (just like I do with linux-yocto in oe-core). But in the general case,
> we want to have consistent and reproducible builds and to do that,
> we need a well defined SRCREV.
>

updated and testing is in-progress.


> So one more version of the series and we should be good (you can
> drop the image addition as part of that) .. sorry for all the
> iterations, the first merges always take the longest.
>

I appreciate your through review and will send out the revision soon
as I am done testing.

Insop
___
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto


Re: [yocto] [PATCH 1/1] add scheedtool-dl for testing sched_deadline use jiri's github as a main repo

2013-03-15 Thread Bruce Ashfield

On 13-03-15 04:47 AM, Insop Song wrote:

---
  recipes-extended/images/core-image-realtime.bb |1 +
  recipes-tools/schedtool-dl/schedtool-dl.bb |   23 +++
  2 files changed, 24 insertions(+)
  create mode 100644 recipes-tools/schedtool-dl/schedtool-dl.bb

diff --git a/recipes-extended/images/core-image-realtime.bb 
b/recipes-extended/images/core-image-realtime.bb
index 624aacf..386b16b 100644
--- a/recipes-extended/images/core-image-realtime.bb
+++ b/recipes-extended/images/core-image-realtime.bb
@@ -8,5 +8,6 @@ EXTRA_IMAGE_FEATURES = "tools-debug debug-tweaks tools-profile 
dbg-pkgs"
  IMAGE_INSTALL = "\
${CORE_IMAGE_BASE_INSTALL} \
rt-app \
+   schedtool-dl \
libgcc \
"


This part of the patch is still a problem, so I'll drop  it on merge.
We still need to rationalize why we can't extend the oe-core
rt images:

./recipes-rt/images/core-image-rt.bb
./recipes-rt/images/core-image-rt-sdk.bb

For this addition. Until we work through that, you can carry a local
patch, or add the tools back via your local.conf adding it to IMAGE_INSTALL.


diff --git a/recipes-tools/schedtool-dl/schedtool-dl.bb 
b/recipes-tools/schedtool-dl/schedtool-dl.bb
new file mode 100644
index 000..6b298f1
--- /dev/null
+++ b/recipes-tools/schedtool-dl/schedtool-dl.bb
@@ -0,0 +1,23 @@
+DESCRIPTION = "schedtool-dl (scheduler test tool) for deadline scheduler"
+SECTION = "devel"
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM = 
"file://Makefile;endline=55;md5=e4b4e8ed9c2132e1d727a1bb5e3bd984"
+PR = "r1"
+
+SRC_URI = "git://github.com/jlelli/schedtool-dl.git;protocol=git"
+SRCREV = "${AUTOREV}"


We also want to lock down the SRCREV vs autorev. I'd expect that going 
forward,

you can maintain/update this once we've merged it, so when a SRCREV
is known to be good, you can send update patches with the new values
(just like I do with linux-yocto in oe-core). But in the general case,
we want to have consistent and reproducible builds and to do that,
we need a well defined SRCREV.

So one more version of the series and we should be good (you can
drop the image addition as part of that) .. sorry for all the
iterations, the first merges always take the longest.

Cheers,

Bruce


+
+S = "${WORKDIR}/git"
+
+do_compile() {
+   oe_runmake
+}
+
+do_install() {
+   install -d ${D}${bindir}
+   install -m 0755 schedtool ${D}${bindir}
+}
+
+FILES_{PN} = "${bindir}/schedtool"
+
+PARALLEL_MAKE = ""



___
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto


[yocto] [PATCH 1/1] add scheedtool-dl for testing sched_deadline use jiri's github as a main repo

2013-03-15 Thread Insop Song
---
 recipes-extended/images/core-image-realtime.bb |1 +
 recipes-tools/schedtool-dl/schedtool-dl.bb |   23 +++
 2 files changed, 24 insertions(+)
 create mode 100644 recipes-tools/schedtool-dl/schedtool-dl.bb

diff --git a/recipes-extended/images/core-image-realtime.bb 
b/recipes-extended/images/core-image-realtime.bb
index 624aacf..386b16b 100644
--- a/recipes-extended/images/core-image-realtime.bb
+++ b/recipes-extended/images/core-image-realtime.bb
@@ -8,5 +8,6 @@ EXTRA_IMAGE_FEATURES = "tools-debug debug-tweaks tools-profile 
dbg-pkgs"
 IMAGE_INSTALL = "\
${CORE_IMAGE_BASE_INSTALL} \
rt-app \
+   schedtool-dl \
libgcc \
"
diff --git a/recipes-tools/schedtool-dl/schedtool-dl.bb 
b/recipes-tools/schedtool-dl/schedtool-dl.bb
new file mode 100644
index 000..6b298f1
--- /dev/null
+++ b/recipes-tools/schedtool-dl/schedtool-dl.bb
@@ -0,0 +1,23 @@
+DESCRIPTION = "schedtool-dl (scheduler test tool) for deadline scheduler"
+SECTION = "devel"
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM = 
"file://Makefile;endline=55;md5=e4b4e8ed9c2132e1d727a1bb5e3bd984"
+PR = "r1"
+
+SRC_URI = "git://github.com/jlelli/schedtool-dl.git;protocol=git"
+SRCREV = "${AUTOREV}"
+
+S = "${WORKDIR}/git"
+
+do_compile() {
+   oe_runmake
+}
+
+do_install() {
+   install -d ${D}${bindir}
+   install -m 0755 schedtool ${D}${bindir}
+}
+
+FILES_{PN} = "${bindir}/schedtool" 
+
+PARALLEL_MAKE = ""
-- 
1.7.9.5

___
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto