[oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.5

2019-07-18 Thread Paolo Valente
Signed-off-by: Paolo Valente 
---
 meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
index 741142c50..8c49b4674 100644
--- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
+++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
@@ -2,8 +2,8 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
 
-SRCREV = "8d22ab8a4cc1484622c47ac9c5944fb9a61a5c0f"
-PV = "3.4"
+SRCREV = "3b9eef2259a70f412ab02fbe6b3a4d313d09c38a"
+PV = "3.5"
 SRC_URI = "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
 
 S = "${WORKDIR}/git"
-- 
2.20.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH V2] s-suite: push SRCREV to version 3.4

2019-06-04 Thread Paolo Valente
Hi,
anyone had time to look into this? Is the patch ok now?

Thanks,
Paolo

> Il giorno 30 mag 2019, alle ore 15:21, Paolo Valente 
>  ha scritto:
> 
> Signed-off-by: Paolo Valente 
> ---
> meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
> b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
> index 706a55e11..741142c50 100644
> --- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
> +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
> @@ -2,8 +2,8 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
> LICENSE = "GPLv2"
> LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
> 
> -SRCREV = "262227ec9de4be1ab41d9918d32ceb773089f1dd"
> -PV = "3.2+git${SRCPV}"
> +SRCREV = "8d22ab8a4cc1484622c47ac9c5944fb9a61a5c0f"
> +PV = "3.4"
> SRC_URI = "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
> 
> S = "${WORKDIR}/git"
> --
> 2.20.1
> 



signature.asc
Description: Message signed with OpenPGP
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.4

2019-05-30 Thread Paolo Valente


> Il giorno 30 mag 2019, alle ore 14:20, Adrian Bunk  ha 
> scritto:
> 
> On Thu, May 30, 2019 at 02:15:21PM +0200, Paolo Valente wrote:
>> 
>> 
>>> Il giorno 30 mag 2019, alle ore 13:14, Adrian Bunk  ha 
>>> scritto:
>>> 
>>> On Thu, May 30, 2019 at 12:25:39PM +0200, Paolo Valente wrote:
>>>> Signed-off-by: Paolo Valente 
>>>> ---
>>>> meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 4 ++--
>>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>> 
>>>> diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
>>>> b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>> index 706a55e11..db1945dd0 100644
>>>> --- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>> +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>> @@ -2,8 +2,8 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
>>>> LICENSE = "GPLv2"
>>>> LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
>>>> 
>>>> -SRCREV = "262227ec9de4be1ab41d9918d32ceb773089f1dd"
>>>> -PV = "3.2+git${SRCPV}"
>>>> +SRCREV = "8d22ab8a4cc1484622c47ac9c5944fb9a61a5c0f"
>>>> +PV = "3.4+git${SRCPV}"
>>>> SRC_URI = 
>>>> "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
>>>> ...
>>> 
>>> The problem was already there before, but the upgrade gives the
>>> opportunity to get rid of the +git${SRCPV}, which is incorrect
>>> when exactly the release is packaged.
>>> 
>> 
>> Sorry,
>> you pointed this out already, but then I thought I had solved it.  Now
>> I also understand why the previous push-up to 3.3 was not it.
>> 
>> And now, I'm sorry, but I think I'd better to ask for your help
>> explicitly: how should I fix this exactly?
> 
> Just remove the +git${SRCPV} from the PV line:
> 
> -SRCREV = "262227ec9de4be1ab41d9918d32ceb773089f1dd"
> -PV = "3.2+git${SRCPV}"
> +SRCREV = "8d22ab8a4cc1484622c47ac9c5944fb9a61a5c0f"
> +PV = "3.4"
> 

Ok, making and sending a V2.

Thank you,
Paolo

>> Thanks for your patience,
>> Paolo
> 
> 
> cu
> Adrian
> 
> --
> 
>   "Is there not promise of rain?" Ling Tan asked suddenly out
>of the darkness. There had been need of rain for many days.
>   "Only a promise," Lao Er said.
>   Pearl S. Buck - Dragon Seed



signature.asc
Description: Message signed with OpenPGP
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [meta-oe][PATCH V2] s-suite: push SRCREV to version 3.4

2019-05-30 Thread Paolo Valente
Signed-off-by: Paolo Valente 
---
 meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
index 706a55e11..741142c50 100644
--- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
+++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
@@ -2,8 +2,8 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
 
-SRCREV = "262227ec9de4be1ab41d9918d32ceb773089f1dd"
-PV = "3.2+git${SRCPV}"
+SRCREV = "8d22ab8a4cc1484622c47ac9c5944fb9a61a5c0f"
+PV = "3.4"
 SRC_URI = "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
 
 S = "${WORKDIR}/git"
-- 
2.20.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.4

2019-05-30 Thread Paolo Valente


> Il giorno 30 mag 2019, alle ore 13:14, Adrian Bunk  ha 
> scritto:
> 
> On Thu, May 30, 2019 at 12:25:39PM +0200, Paolo Valente wrote:
>> Signed-off-by: Paolo Valente 
>> ---
>> meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
>> b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>> index 706a55e11..db1945dd0 100644
>> --- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>> +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>> @@ -2,8 +2,8 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
>> LICENSE = "GPLv2"
>> LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
>> 
>> -SRCREV = "262227ec9de4be1ab41d9918d32ceb773089f1dd"
>> -PV = "3.2+git${SRCPV}"
>> +SRCREV = "8d22ab8a4cc1484622c47ac9c5944fb9a61a5c0f"
>> +PV = "3.4+git${SRCPV}"
>> SRC_URI = 
>> "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
>> ...
> 
> The problem was already there before, but the upgrade gives the
> opportunity to get rid of the +git${SRCPV}, which is incorrect
> when exactly the release is packaged.
> 

Sorry,
you pointed this out already, but then I thought I had solved it.  Now
I also understand why the previous push-up to 3.3 was not it.

And now, I'm sorry, but I think I'd better to ask for your help
explicitly: how should I fix this exactly?

Thanks for your patience,
Paolo

> cu
> Adrian
> 
> --
> 
>   "Is there not promise of rain?" Ling Tan asked suddenly out
>of the darkness. There had been need of rain for many days.
>   "Only a promise," Lao Er said.
>   Pearl S. Buck - Dragon Seed



signature.asc
Description: Message signed with OpenPGP
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.4

2019-05-30 Thread Paolo Valente
Signed-off-by: Paolo Valente 
---
 meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
index 706a55e11..db1945dd0 100644
--- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
+++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
@@ -2,8 +2,8 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
 
-SRCREV = "262227ec9de4be1ab41d9918d32ceb773089f1dd"
-PV = "3.2+git${SRCPV}"
+SRCREV = "8d22ab8a4cc1484622c47ac9c5944fb9a61a5c0f"
+PV = "3.4+git${SRCPV}"
 SRC_URI = "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
 
 S = "${WORKDIR}/git"
-- 
2.20.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [meta-oe][PATCH V2] s-suite: push SRCREV to version 3.2

2019-04-16 Thread Paolo Valente
Signed-off-by: Paolo Valente 
---
 meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
index 47d21d36c..706a55e11 100644
--- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
+++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
@@ -2,8 +2,8 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
 
-SRCREV = "d05006865f68829fa7603bdb92bd51649f5ba1b6"
-PV = "0.0+git${SRCPV}"
+SRCREV = "262227ec9de4be1ab41d9918d32ceb773089f1dd"
+PV = "3.2+git${SRCPV}"
 SRC_URI = "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
 
 S = "${WORKDIR}/git"
-- 
2.20.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.2

2019-04-15 Thread Paolo Valente
Signed-off-by: Paolo Valente 
---
 meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
index 47d21d36c..78d299e3b 100644
--- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
+++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
@@ -2,7 +2,7 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
 
-SRCREV = "d05006865f68829fa7603bdb92bd51649f5ba1b6"
+SRCREV = "262227ec9de4be1ab41d9918d32ceb773089f1dd"
 PV = "0.0+git${SRCPV}"
 SRC_URI = "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
 
-- 
2.20.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.1

2019-04-03 Thread Paolo Valente
Hi,
have you guys had time to look into this?

Thanks,
Paolo

> Il giorno 28 mar 2019, alle ore 23:04, Paolo Valente 
>  ha scritto:
> 
> Signed-off-by: Paolo Valente 
> ---
> meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
> b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
> index 8bd0afc49..47d21d36c 100644
> --- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
> +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
> @@ -2,7 +2,7 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
> LICENSE = "GPLv2"
> LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
> 
> -SRCREV = "e2727f70ed79b5461689c062a4a66a87f1adb4ce"
> +SRCREV = "d05006865f68829fa7603bdb92bd51649f5ba1b6"
> PV = "0.0+git${SRCPV}"
> SRC_URI = "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
> 
> -- 
> 2.20.1
> 

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.1

2019-03-28 Thread Paolo Valente
Signed-off-by: Paolo Valente 
---
 meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
index 8bd0afc49..47d21d36c 100644
--- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
+++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
@@ -2,7 +2,7 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
 
-SRCREV = "e2727f70ed79b5461689c062a4a66a87f1adb4ce"
+SRCREV = "d05006865f68829fa7603bdb92bd51649f5ba1b6"
 PV = "0.0+git${SRCPV}"
 SRC_URI = "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
 
-- 
2.20.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.0

2018-12-15 Thread Paolo Valente
Yep, sorry. I've just sent a V2 which should hopefully work.

Thanks for your patience,
Paolo

> Il giorno 13 dic 2018, alle ore 22:33, Khem Raj  ha 
> scritto:
> 
> btw. it fails on mips/musl
> http://errors.yoctoproject.org/Errors/Details/211648/
> On Thu, Dec 13, 2018 at 12:59 PM Khem Raj  wrote:
>> 
>> On Thu, Dec 13, 2018 at 11:13 AM Paolo Valente  
>> wrote:
>>> 
>>> 
>>> 
>>>> Il giorno 13 dic 2018, alle ore 18:24, Khem Raj  ha 
>>>> scritto:
>>>> 
>>>> On Thu, Dec 13, 2018 at 9:20 AM Paolo Valente  
>>>> wrote:
>>>>> 
>>>>> 
>>>>> 
>>>>>> Il giorno 13 dic 2018, alle ore 18:18, Khem Raj  ha 
>>>>>> scritto:
>>>>>> 
>>>>>> On Thu, Dec 13, 2018 at 9:16 AM Paolo Valente  
>>>>>> wrote:
>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>>>> Il giorno 13 dic 2018, alle ore 18:11, Khem Raj  
>>>>>>>> ha scritto:
>>>>>>>> 
>>>>>>>> On Thu, Dec 13, 2018 at 7:33 AM Paolo Valente 
>>>>>>>>  wrote:
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>>> Il giorno 13 dic 2018, alle ore 16:27, Khem Raj  
>>>>>>>>>> ha scritto:
>>>>>>>>>> 
>>>>>>>>>> On Thu, Dec 13, 2018 at 7:16 AM Paolo Valente 
>>>>>>>>>>  wrote:
>>>>>>>>>>> 
>>>>>>>>>>> Signed-off-by: Paolo Valente 
>>>>>>>>>>> ---
>>>>>>>>>>> meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 5 +++--
>>>>>>>>>>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>>>>>>>>>> 
>>>>>>>>>>> diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
>>>>>>>>>>> b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>>>>>>> index 77d172814..166f3000a 100644
>>>>>>>>>>> --- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>>>>>>> +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>>>>>>> @@ -2,7 +2,7 @@ SUMMARY = "Small collection of benchmarks for 
>>>>>>>>>>> storage I/O"
>>>>>>>>>>> LICENSE = "GPLv2"
>>>>>>>>>>> LIC_FILES_CHKSUM = 
>>>>>>>>>>> "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
>>>>>>>>>>> 
>>>>>>>>>>> -SRCREV = "79698f645bfb28d0d966484ddad3a1efb562246d"
>>>>>>>>>>> +SRCREV = "e2727f70ed79b5461689c062a4a66a87f1adb4ce"
>>>>>>>>>>> PV = "0.0+git${SRCPV}"
>>>>>>>>>>> SRC_URI = 
>>>>>>>>>>> "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
>>>>>>>>>>> 
>>>>>>>>>>> @@ -21,7 +21,8 @@ do_install() {
>>>>>>>>>>> install -m0755 ${S}/def_config_params.sh ${D}/opt/S-suite
>>>>>>>>>>> }
>>>>>>>>>>> 
>>>>>>>>>>> -RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio 
>>>>>>>>>>> libaio-dev sysstat"
>>>>>>>>>>> +RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio 
>>>>>>>>>>> libaio-dev sysstat \
>>>>>>>>>>> + psmisc git"
>>>>>>>>>> 
>>>>>>>>>> why these new rdeps
>>>>>>>>> 
>>>>>>>>> psmisc is for killall, git is for ... git :)
>>>>>>>> 
>>>>>>>> I think busybox provides killall too.
>>>>>>> 
>>>>>>> Ok, I'll remove this dep.
>>>>>>> 
>>>>>>>>> 
>>>>>>>>> Some weeks ago, I bumped into this missing dependencies that made the
>>>>>>>>> automatic execution of a test fail on a system without either killall
>>>>>>>>> or git.
>>>>>>>> 
>>>>>>>> OK. so I wonder if we dont have killall enabled in oe-core by default 
>>>>>>>> in
>>>>>>>> busybox ? or does it need ps-misc one specifically ?
>>>>>>>> 
>>>>>>>> git is interesting, does it clone some sources during executing tests ?
>>>>>>>> 
>>>>>>> In kernel-task benchmarks a mainline Linux is cloned.  But maybe such
>>>>>>> benchmarks do not make sense in oe.  Let me know, and I'll remove this
>>>>>>> dep too.
>>>>>> 
>>>>>> they are fine. We just need to understand that they are well meaning 
>>>>>> changes.
>>>>>> 
>>>>> 
>>>>> So, is the patch ok, or shall I remove the psmisc dep?
>>>> 
>>>> I think it will be good to understand why killall went missing without
>>>> ps-misc package. As such patch is ok. Since I think killall should always
>>>> be there, either from ps-misc or a backfill from busybox.
>>>> 
>>> 
>>> Sorry, I created a lot of confusion.
>>> 
>>> 1) killall *was not* missing in any of the oe rootfs that I tested
>>> (rootfs made by Anders, in CC).  killall was missing on a non-oe
>>> remote machine, made available for my tests by a company.  So, I added
>>> that dependency (in my S suite) to make it work on that machine.  For
>>> this reason, I thought of adding this dependency to this recipe.  But,
>>> if it is not needed, I'll remove it.
>> 
>> ok lets remove it
>> 
>>> 
>>> 2) git is needed only if you want to run kernel-development tests.
>>> 
>> 
>> lets keep it.
>> 
>>> That said, just tell me whether the patch is ok, or tell me what
>>> dependencies to remove, and I'll make a V2.
>> 
>> yes please.
>> 
>>> 
>>> Thanks,
>>> Paolo
>>> 
>>>>> 
>>>>> Thanks,
>>>>> Paolo
>>> 

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [meta-oe][PATCH V2] s-suite: push SRCREV to version 3.0

2018-12-15 Thread Paolo Valente
Signed-off-by: Paolo Valente 
---
Changes v1 -> v2:
 - Removed useless psmisc dep
 - Fixed stale install commands

 meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 8 +---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
index 77d172814..8bd0afc49 100644
--- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
+++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
@@ -2,7 +2,7 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"

-SRCREV = "79698f645bfb28d0d966484ddad3a1efb562246d"
+SRCREV = "e2727f70ed79b5461689c062a4a66a87f1adb4ce"
 PV = "0.0+git${SRCPV}"
 SRC_URI = "git://github.com/Algodev-github/S.git;protocol=https;branch=master"

@@ -17,11 +17,13 @@ do_install() {
 install -m0755 -p ${S}/$(basename $i)/* ${D}/opt/S-suite/$(basename $i)
 done

+install -m0755 ${S}/def_config.sh ${D}/opt/S-suite
 install -m0755 ${S}/config_params.sh ${D}/opt/S-suite
-install -m0755 ${S}/def_config_params.sh ${D}/opt/S-suite
+install -m0755 ${S}/process_config.sh ${D}/opt/S-suite
 }

-RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev sysstat"
+RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev sysstat 
\
+ git"

 FILES_${PN} = "/opt/S-suite/"

--
2.16.1
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.0

2018-12-13 Thread Paolo Valente



> Il giorno 13 dic 2018, alle ore 18:24, Khem Raj  ha 
> scritto:
> 
> On Thu, Dec 13, 2018 at 9:20 AM Paolo Valente  
> wrote:
>> 
>> 
>> 
>>> Il giorno 13 dic 2018, alle ore 18:18, Khem Raj  ha 
>>> scritto:
>>> 
>>> On Thu, Dec 13, 2018 at 9:16 AM Paolo Valente  
>>> wrote:
>>>> 
>>>> 
>>>> 
>>>>> Il giorno 13 dic 2018, alle ore 18:11, Khem Raj  ha 
>>>>> scritto:
>>>>> 
>>>>> On Thu, Dec 13, 2018 at 7:33 AM Paolo Valente  
>>>>> wrote:
>>>>>> 
>>>>>> 
>>>>>> 
>>>>>>> Il giorno 13 dic 2018, alle ore 16:27, Khem Raj  ha 
>>>>>>> scritto:
>>>>>>> 
>>>>>>> On Thu, Dec 13, 2018 at 7:16 AM Paolo Valente 
>>>>>>>  wrote:
>>>>>>>> 
>>>>>>>> Signed-off-by: Paolo Valente 
>>>>>>>> ---
>>>>>>>> meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 5 +++--
>>>>>>>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>>>>>>> 
>>>>>>>> diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
>>>>>>>> b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>>>> index 77d172814..166f3000a 100644
>>>>>>>> --- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>>>> +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>>>> @@ -2,7 +2,7 @@ SUMMARY = "Small collection of benchmarks for storage 
>>>>>>>> I/O"
>>>>>>>> LICENSE = "GPLv2"
>>>>>>>> LIC_FILES_CHKSUM = 
>>>>>>>> "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
>>>>>>>> 
>>>>>>>> -SRCREV = "79698f645bfb28d0d966484ddad3a1efb562246d"
>>>>>>>> +SRCREV = "e2727f70ed79b5461689c062a4a66a87f1adb4ce"
>>>>>>>> PV = "0.0+git${SRCPV}"
>>>>>>>> SRC_URI = 
>>>>>>>> "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
>>>>>>>> 
>>>>>>>> @@ -21,7 +21,8 @@ do_install() {
>>>>>>>>  install -m0755 ${S}/def_config_params.sh ${D}/opt/S-suite
>>>>>>>> }
>>>>>>>> 
>>>>>>>> -RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio 
>>>>>>>> libaio-dev sysstat"
>>>>>>>> +RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio 
>>>>>>>> libaio-dev sysstat \
>>>>>>>> + psmisc git"
>>>>>>> 
>>>>>>> why these new rdeps
>>>>>> 
>>>>>> psmisc is for killall, git is for ... git :)
>>>>> 
>>>>> I think busybox provides killall too.
>>>> 
>>>> Ok, I'll remove this dep.
>>>> 
>>>>>> 
>>>>>> Some weeks ago, I bumped into this missing dependencies that made the
>>>>>> automatic execution of a test fail on a system without either killall
>>>>>> or git.
>>>>> 
>>>>> OK. so I wonder if we dont have killall enabled in oe-core by default in
>>>>> busybox ? or does it need ps-misc one specifically ?
>>>>> 
>>>>> git is interesting, does it clone some sources during executing tests ?
>>>>> 
>>>> In kernel-task benchmarks a mainline Linux is cloned.  But maybe such
>>>> benchmarks do not make sense in oe.  Let me know, and I'll remove this
>>>> dep too.
>>> 
>>> they are fine. We just need to understand that they are well meaning 
>>> changes.
>>> 
>> 
>> So, is the patch ok, or shall I remove the psmisc dep?
> 
> I think it will be good to understand why killall went missing without
> ps-misc package. As such patch is ok. Since I think killall should always
> be there, either from ps-misc or a backfill from busybox.
> 

Sorry, I created a lot of confusion.

1) killall *was not* missing in any of the oe rootfs that I tested
(rootfs made by Anders, in CC).  killall was missing on a non-oe
remote machine, made available for my tests by a company.  So, I added
that dependency (in my S suite) to make it work on that machine.  For
this reason, I thought of adding this dependency to this recipe.  But,
if it is not needed, I'll remove it.

2) git is needed only if you want to run kernel-development tests.

That said, just tell me whether the patch is ok, or tell me what
dependencies to remove, and I'll make a V2.

Thanks,
Paolo

>> 
>> Thanks,
>> Paolo

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.0

2018-12-13 Thread Paolo Valente



> Il giorno 13 dic 2018, alle ore 18:18, Khem Raj  ha 
> scritto:
> 
> On Thu, Dec 13, 2018 at 9:16 AM Paolo Valente  
> wrote:
>> 
>> 
>> 
>>> Il giorno 13 dic 2018, alle ore 18:11, Khem Raj  ha 
>>> scritto:
>>> 
>>> On Thu, Dec 13, 2018 at 7:33 AM Paolo Valente  
>>> wrote:
>>>> 
>>>> 
>>>> 
>>>>> Il giorno 13 dic 2018, alle ore 16:27, Khem Raj  ha 
>>>>> scritto:
>>>>> 
>>>>> On Thu, Dec 13, 2018 at 7:16 AM Paolo Valente  
>>>>> wrote:
>>>>>> 
>>>>>> Signed-off-by: Paolo Valente 
>>>>>> ---
>>>>>> meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 5 +++--
>>>>>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>>>>> 
>>>>>> diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
>>>>>> b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>> index 77d172814..166f3000a 100644
>>>>>> --- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>> +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>> @@ -2,7 +2,7 @@ SUMMARY = "Small collection of benchmarks for storage 
>>>>>> I/O"
>>>>>> LICENSE = "GPLv2"
>>>>>> LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
>>>>>> 
>>>>>> -SRCREV = "79698f645bfb28d0d966484ddad3a1efb562246d"
>>>>>> +SRCREV = "e2727f70ed79b5461689c062a4a66a87f1adb4ce"
>>>>>> PV = "0.0+git${SRCPV}"
>>>>>> SRC_URI = 
>>>>>> "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
>>>>>> 
>>>>>> @@ -21,7 +21,8 @@ do_install() {
>>>>>>   install -m0755 ${S}/def_config_params.sh ${D}/opt/S-suite
>>>>>> }
>>>>>> 
>>>>>> -RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev 
>>>>>> sysstat"
>>>>>> +RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev 
>>>>>> sysstat \
>>>>>> + psmisc git"
>>>>> 
>>>>> why these new rdeps
>>>> 
>>>> psmisc is for killall, git is for ... git :)
>>> 
>>> I think busybox provides killall too.
>> 
>> Ok, I'll remove this dep.
>> 
>>>> 
>>>> Some weeks ago, I bumped into this missing dependencies that made the
>>>> automatic execution of a test fail on a system without either killall
>>>> or git.
>>> 
>>> OK. so I wonder if we dont have killall enabled in oe-core by default in
>>> busybox ? or does it need ps-misc one specifically ?
>>> 
>>> git is interesting, does it clone some sources during executing tests ?
>>> 
>> In kernel-task benchmarks a mainline Linux is cloned.  But maybe such
>> benchmarks do not make sense in oe.  Let me know, and I'll remove this
>> dep too.
> 
> they are fine. We just need to understand that they are well meaning changes.
> 

So, is the patch ok, or shall I remove the psmisc dep?

Thanks,
Paolo

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.0

2018-12-13 Thread Paolo Valente



> Il giorno 13 dic 2018, alle ore 18:11, Khem Raj  ha 
> scritto:
> 
> On Thu, Dec 13, 2018 at 7:33 AM Paolo Valente  
> wrote:
>> 
>> 
>> 
>>> Il giorno 13 dic 2018, alle ore 16:27, Khem Raj  ha 
>>> scritto:
>>> 
>>> On Thu, Dec 13, 2018 at 7:16 AM Paolo Valente  
>>> wrote:
>>>> 
>>>> Signed-off-by: Paolo Valente 
>>>> ---
>>>> meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 5 +++--
>>>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>>> 
>>>> diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
>>>> b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>> index 77d172814..166f3000a 100644
>>>> --- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>> +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>> @@ -2,7 +2,7 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
>>>> LICENSE = "GPLv2"
>>>> LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
>>>> 
>>>> -SRCREV = "79698f645bfb28d0d966484ddad3a1efb562246d"
>>>> +SRCREV = "e2727f70ed79b5461689c062a4a66a87f1adb4ce"
>>>> PV = "0.0+git${SRCPV}"
>>>> SRC_URI = 
>>>> "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
>>>> 
>>>> @@ -21,7 +21,8 @@ do_install() {
>>>>install -m0755 ${S}/def_config_params.sh ${D}/opt/S-suite
>>>> }
>>>> 
>>>> -RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev 
>>>> sysstat"
>>>> +RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev 
>>>> sysstat \
>>>> + psmisc git"
>>> 
>>> why these new rdeps
>> 
>> psmisc is for killall, git is for ... git :)
> 
> I think busybox provides killall too.

Ok, I'll remove this dep.

>> 
>> Some weeks ago, I bumped into this missing dependencies that made the
>> automatic execution of a test fail on a system without either killall
>> or git.
> 
> OK. so I wonder if we dont have killall enabled in oe-core by default in
> busybox ? or does it need ps-misc one specifically ?
> 
> git is interesting, does it clone some sources during executing tests ?
> 
In kernel-task benchmarks a mainline Linux is cloned.  But maybe such
benchmarks do not make sense in oe.  Let me know, and I'll remove this
dep too.


Thanks,
Paolo

>> 
>> If these dependencies are certainly useless for oe, then I'll remove them.
>> 
>> Thanks,
>> Paolo
>> 
>>>> 
>>>> FILES_${PN} = "/opt/S-suite/"
>>>> 
>>>> --
>>>> 2.16.1
>>>> 
>>>> --
>>>> ___
>>>> Openembedded-devel mailing list
>>>> Openembedded-devel@lists.openembedded.org
>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>> 

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.0

2018-12-13 Thread Paolo Valente



> Il giorno 13 dic 2018, alle ore 16:27, Khem Raj  ha 
> scritto:
> 
> On Thu, Dec 13, 2018 at 7:16 AM Paolo Valente  
> wrote:
>> 
>> Signed-off-by: Paolo Valente 
>> ---
>> meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 5 +++--
>> 1 file changed, 3 insertions(+), 2 deletions(-)
>> 
>> diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
>> b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>> index 77d172814..166f3000a 100644
>> --- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>> +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>> @@ -2,7 +2,7 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
>> LICENSE = "GPLv2"
>> LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
>> 
>> -SRCREV = "79698f645bfb28d0d966484ddad3a1efb562246d"
>> +SRCREV = "e2727f70ed79b5461689c062a4a66a87f1adb4ce"
>> PV = "0.0+git${SRCPV}"
>> SRC_URI = 
>> "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
>> 
>> @@ -21,7 +21,8 @@ do_install() {
>> install -m0755 ${S}/def_config_params.sh ${D}/opt/S-suite
>> }
>> 
>> -RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev 
>> sysstat"
>> +RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev 
>> sysstat \
>> + psmisc git"
> 
> why these new rdeps

psmisc is for killall, git is for ... git :)

Some weeks ago, I bumped into this missing dependencies that made the
automatic execution of a test fail on a system without either killall
or git.

If these dependencies are certainly useless for oe, then I'll remove them.

Thanks,
Paolo

>> 
>> FILES_${PN} = "/opt/S-suite/"
>> 
>> --
>> 2.16.1
>> 
>> --
>> ___
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [meta-oe][PATCH] s-suite: push SRCREV to version 3.0

2018-12-13 Thread Paolo Valente
Signed-off-by: Paolo Valente 
---
 meta-oe/recipes-benchmark/s-suite/s-suite_git.bb | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
index 77d172814..166f3000a 100644
--- a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
+++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
@@ -2,7 +2,7 @@ SUMMARY = "Small collection of benchmarks for storage I/O"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
 
-SRCREV = "79698f645bfb28d0d966484ddad3a1efb562246d"
+SRCREV = "e2727f70ed79b5461689c062a4a66a87f1adb4ce"
 PV = "0.0+git${SRCPV}"
 SRC_URI = "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
 
@@ -21,7 +21,8 @@ do_install() {
 install -m0755 ${S}/def_config_params.sh ${D}/opt/S-suite
 }
 
-RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev sysstat"
+RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev sysstat 
\
+ psmisc git"
 
 FILES_${PN} = "/opt/S-suite/"
 
-- 
2.16.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] add a storage I/O benchmark test suite

2018-10-30 Thread Paolo Valente



> Il giorno 25 ott 2018, alle ore 20:17, Randy MacLeod 
>  ha scritto:
> 
> On 10/25/18 1:46 PM, Paolo Valente wrote:
>> [RESENDING: FIRST ATTEMPT REJECTED BECAUSE I WAS NOT SUBSCRIBED TO THE LIST]
>>> Il giorno 23 ott 2018, alle ore 15:33, Anders Roxell 
>>>  ha scritto:
>>> 
>>> CCing in the package maintainer.
>>> 
>> Thanks Anders, hi everybody,
>>> On Sun, 21 Oct 2018 at 17:10, Khem Raj  wrote:
>>>> 
>>>> On Sun, Oct 21, 2018 at 12:29 AM Randy MacLeod
>>>>  wrote:
>>>>> 
>>>>> On 10/19/18 10:54 AM, Anders Roxell wrote:
>>>>>> Signed-off-by: Anders Roxell 
>>>>>> ---
>>>>>> .../recipes-benchmark/s-suite/s-suite_git.bb  | 30 +++
>>>>>> 1 file changed, 30 insertions(+)
>>>>>> create mode 100644 meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>> 
>>>>>> diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
>>>>>> b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>> new file mode 100644
>>>>>> index ..77d172814065
>>>>>> --- /dev/null
>>>>>> +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
>>>>>> @@ -0,0 +1,30 @@
>>>>>> +SUMMARY = "Small collection of benchmarks for storage I/O"
>>>>>> +LICENSE = "GPLv2"
>>>>>> +LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
>>>>>> +
>>>>>> +SRCREV = "79698f645bfb28d0d966484ddad3a1efb562246d"
>>>>>> +PV = "0.0+git${SRCPV}"
>>>>>> +SRC_URI = 
>>>>>> "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
>>>>>> +
>>>>>> +S = "${WORKDIR}/git"
>>>>>> +
>>>>>> +# installing in /opt/S-suite since the package has
>>>>>> +# dependencies to the directory structure.
>>>>>> +do_install() {
>>>>>> +install -d ${D}/opt/S-suite
>>>>>> +for i in $(find ${S}/* -type d); do
>>>>>> +install -d ${D}/opt/S-suite/$(basename $i)
>>>>>> +install -m0755 -p ${S}/$(basename $i)/* 
>>>>>> ${D}/opt/S-suite/$(basename $i)
>>>>>> +done
>>>>>> +
>>>>>> +install -m0755 ${S}/config_params.sh ${D}/opt/S-suite
>>>>>> +install -m0755 ${S}/def_config_params.sh ${D}/opt/S-suite
>>>>>> +}
>>>>>> +
>>>>>> +RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev 
>>>>>> sysstat"
>>>>>> +
>>>>>> +FILES_${PN} = "/opt/S-suite/"
>>>>>> +
>>>>>> +# added to INSANE_SKIP since s-suite have an runtime
>>>>>> +# dependency (RDEPENDS) on libaio-dev.
>>>>>> +INSANE_SKIP_${PN} += "dev-deps"
>>>>>> 
>>>>> 
>>>>> Thanks for the submission but from:
>>>>> 
>>>>>   https://github.com/Algodev-github/S
>>>>> 
>>>>>   "These benchmarks have been written just for internal use,
>>>>>and are more a yard than a well-finished work."
>>>>> 
>>>>> 
>>>>> That doesn't lead to a lot of confidence that these suite will
>>>>> be maintained for years.
>> Gosh.  I put that disclaimer in the README when I first wrote it more
>> than 10 years ago.  After then, thousands of changes have been made by
>> me and many other contributors.  Unfortunately, I put the suite on
>> GitHub only recently.  But as you can see from the log, there have
>> been 448 commits in (less than) the last five months.
>> In addition, the S suite is now used in a new Phoronix benchmark and
>> in a new mmtests benchmark.
> 
> Ah, good to know.
> 
>> So, believe me, the S suite ATM is definitely in good shape, and seems
>> to be here to stay.
>> I'll remove that line, right after finishing a non-trivial new commit
>> I'm working on.
>>>>> What's the value to meta-oe users to add
>>>>> this recipe compared to something that has been around for years
>>>>> if not decades such as Bonnie++:
>>>>>   https://layers.openembedded.org/layerindex/recipe/864/
>&

Re: [oe] [meta-oe][PATCH] add a storage I/O benchmark test suite

2018-10-25 Thread Paolo Valente
[RESENDING: FIRST ATTEMPT REJECTED BECAUSE I WAS NOT SUBSCRIBED TO THE LIST]

> Il giorno 23 ott 2018, alle ore 15:33, Anders Roxell 
>  ha scritto:
> 
> CCing in the package maintainer.
> 

Thanks Anders, hi everybody,

> On Sun, 21 Oct 2018 at 17:10, Khem Raj  wrote:
>> 
>> On Sun, Oct 21, 2018 at 12:29 AM Randy MacLeod
>>  wrote:
>>> 
>>> On 10/19/18 10:54 AM, Anders Roxell wrote:
 Signed-off-by: Anders Roxell 
 ---
 .../recipes-benchmark/s-suite/s-suite_git.bb  | 30 +++
 1 file changed, 30 insertions(+)
 create mode 100644 meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
 
 diff --git a/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb 
 b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
 new file mode 100644
 index ..77d172814065
 --- /dev/null
 +++ b/meta-oe/recipes-benchmark/s-suite/s-suite_git.bb
 @@ -0,0 +1,30 @@
 +SUMMARY = "Small collection of benchmarks for storage I/O"
 +LICENSE = "GPLv2"
 +LIC_FILES_CHKSUM = "file://COPYING;md5=b529aaa6a0c50f15d29f89609b5c22f3"
 +
 +SRCREV = "79698f645bfb28d0d966484ddad3a1efb562246d"
 +PV = "0.0+git${SRCPV}"
 +SRC_URI = 
 "git://github.com/Algodev-github/S.git;protocol=https;branch=master"
 +
 +S = "${WORKDIR}/git"
 +
 +# installing in /opt/S-suite since the package has
 +# dependencies to the directory structure.
 +do_install() {
 +install -d ${D}/opt/S-suite
 +for i in $(find ${S}/* -type d); do
 +install -d ${D}/opt/S-suite/$(basename $i)
 +install -m0755 -p ${S}/$(basename $i)/* 
 ${D}/opt/S-suite/$(basename $i)
 +done
 +
 +install -m0755 ${S}/config_params.sh ${D}/opt/S-suite
 +install -m0755 ${S}/def_config_params.sh ${D}/opt/S-suite
 +}
 +
 +RDEPENDS_${PN} = "bash bc coreutils gawk g++ gcc fio libaio libaio-dev 
 sysstat"
 +
 +FILES_${PN} = "/opt/S-suite/"
 +
 +# added to INSANE_SKIP since s-suite have an runtime
 +# dependency (RDEPENDS) on libaio-dev.
 +INSANE_SKIP_${PN} += "dev-deps"
 
>>> 
>>> Thanks for the submission but from:
>>> 
>>>   https://github.com/Algodev-github/S
>>> 
>>>   "These benchmarks have been written just for internal use,
>>>and are more a yard than a well-finished work."
>>> 
>>> 
>>> That doesn't lead to a lot of confidence that these suite will
>>> be maintained for years.

Gosh.  I put that disclaimer in the README when I first wrote it more
than 10 years ago.  After then, thousands of changes have been made by
me and many other contributors.  Unfortunately, I put the suite on
GitHub only recently.  But as you can see from the log, there have
been 448 commits in (less than) the last five months.

In addition, the S suite is now used in a new Phoronix benchmark and
in a new mmtests benchmark.

So, believe me, the S suite ATM is definitely in good shape, and seems
to be here to stay.

I'll remove that line, right after finishing a non-trivial new commit
I'm working on.

>>> What's the value to meta-oe users to add
>>> this recipe compared to something that has been around for years
>>> if not decades such as Bonnie++:
>>>   https://layers.openembedded.org/layerindex/recipe/864/
>>> 
>> 

The added value is that bonnie++ does not perform *any* of the benchmarks in 
the S suite, apart from that on throughput. I have made the S suite, many years 
ago, because I needed to run benchmarks that no suite provided. Here are some 
examples:
- start-up times of real applications under real background workloads
- playback quality (drop rate) of video and audio under real background 
workloads
- speed of code-development tasks (make, git checkout, git merge, git
 grep) under real background workloads, plus measurement of system
 responsiveness while one of these dev tasks is executed
- minimum per-client bandwidth guaranteed to a set of clients doing
 any possible type of I/O
- maximum per-client latency guaranteed to a set of clients doing any
 possible type of I/O

For any doubt, here I am.

Thanks,
Paolo

>> how I read this is that they have not made proper installation
>> targets and its dependencies or letting user know of the dependencies
>> which actually is not a problem with OE since both
>> of these would be taken care of in the recipe with
>> do_install and DEPENDS
> 
> Do you mean DEPENDS or RDEPENDS?
> 
> Cheers,
> Anders

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel