All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCHv2] ltp: remove OOM tests from runtest/mm
@ 2020-07-29 13:09 Matthew
  2020-07-29 13:24 ` [OE-core] " Richard Purdie
  0 siblings, 1 reply; 9+ messages in thread
From: Matthew @ 2020-07-29 13:09 UTC (permalink / raw)
  To: openembedded-core

Fixes [YOCTO #13802]

Remove the OOM tests, since they might cause oeqa ssh
connection lost.

Signed-off-by: Mingde (Matthew) Zeng <matthew.zeng@windriver.com>
---
 ...001-Remove-OOM-tests-from-runtest-mm.patch | 33 +++++++++++++++++++
 meta/recipes-extended/ltp/ltp_20200515.bb     |  1 +
 2 files changed, 34 insertions(+)
 create mode 100644 meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-from-runtest-mm.patch

diff --git a/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-from-runtest-mm.patch b/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-from-runtest-mm.patch
new file mode 100644
index 0000000000..2d8262be78
--- /dev/null
+++ b/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-from-runtest-mm.patch
@@ -0,0 +1,33 @@
+From 373b21dcf7a2879c27cc3dbeac5d83f5d45d22ae Mon Sep 17 00:00:00 2001
+From: "Mingde (Matthew) Zeng" <matthewzmd@gmail.com>
+Date: Wed, 29 Jul 2020 08:47:09 -0400
+Subject: [PATCH] Remove OOM tests from runtest/mm
+
+Disable OOM tests, as they might cause oeqa ssh connection lost
+
+Upstream-Status: Inappropriate [oe-core specific]
+Signed-off-by: Mingde (Matthew) Zeng <matthew.zeng@windriver.com>
+---
+ runtest/mm | 6 ------
+ 1 file changed, 6 deletions(-)
+
+diff --git a/runtest/mm b/runtest/mm
+index 4701a14bd..2f9888ec6 100644
+--- a/runtest/mm
++++ b/runtest/mm
+@@ -75,12 +75,6 @@ ksm06_2 ksm06 -n 10000
+
+ cpuset01 cpuset01
+
+-oom01 oom01
+-oom02 oom02
+-oom03 oom03
+-oom04 oom04
+-oom05 oom05
+-
+ swapping01 swapping01 -i 5
+
+ thp01 thp01 -I 120
+--
+2.27.0
+
diff --git a/meta/recipes-extended/ltp/ltp_20200515.bb b/meta/recipes-extended/ltp/ltp_20200515.bb
index ece3acf0f9..0c7044d044 100644
--- a/meta/recipes-extended/ltp/ltp_20200515.bb
+++ b/meta/recipes-extended/ltp/ltp_20200515.bb
@@ -37,6 +37,7 @@ SRC_URI = "git://github.com/linux-test-project/ltp.git \
            file://0001-ptrace01-Fix-missing-format-string.patch \
            file://0001-sigwaitinfo-Do-not-run-invalid-undefined-test-cases.patch \
            file://0001-syscalls-copy_file_range02-Expect-EFBIG-in-subcase-m.patch \
+           file://0001-Remove-OOM-tests-from-runtest-mm.patch \
            "

 S = "${WORKDIR}/git"
--
2.27.0

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

* Re: [OE-core] [PATCHv2] ltp: remove OOM tests from runtest/mm
  2020-07-29 13:09 [PATCHv2] ltp: remove OOM tests from runtest/mm Matthew
@ 2020-07-29 13:24 ` Richard Purdie
  2020-07-29 13:41   ` Matthew
  2020-08-05 15:52   ` Matthew
  0 siblings, 2 replies; 9+ messages in thread
From: Richard Purdie @ 2020-07-29 13:24 UTC (permalink / raw)
  To: Matthew, openembedded-core

On Wed, 2020-07-29 at 09:09 -0400, Matthew wrote:
> Fixes [YOCTO #13802]

Thanks, we can reference the bug but I don't think we can yet claim it
fixes it.

The missing log issue happened again:
https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/908
so I think there are multiple issues at play here.

Is there any way to have the logging logged to disk straight away
rather than stuck internally in oeqa's logging buffers? That way we
could see which tests had run before a hang?

Also, could we make the scp failure non-fatal, maybe a warning so that
when it fails we can look at the rest of the logs?

Cheers,

Richard

> Remove the OOM tests, since they might cause oeqa ssh
> connection lost.
> 
> Signed-off-by: Mingde (Matthew) Zeng <matthew.zeng@windriver.com>
> ---
>  ...001-Remove-OOM-tests-from-runtest-mm.patch | 33
> +++++++++++++++++++
>  meta/recipes-extended/ltp/ltp_20200515.bb     |  1 +
>  2 files changed, 34 insertions(+)
>  create mode 100644 meta/recipes-extended/ltp/ltp/0001-Remove-OOM-
> tests-from-runtest-mm.patch
> 
> diff --git a/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-
> from-runtest-mm.patch b/meta/recipes-extended/ltp/ltp/0001-Remove-
> OOM-tests-from-runtest-mm.patch
> new file mode 100644
> index 0000000000..2d8262be78
> --- /dev/null
> +++ b/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-from-
> runtest-mm.patch
> @@ -0,0 +1,33 @@
> +From 373b21dcf7a2879c27cc3dbeac5d83f5d45d22ae Mon Sep 17 00:00:00
> 2001
> +From: "Mingde (Matthew) Zeng" <matthewzmd@gmail.com>
> +Date: Wed, 29 Jul 2020 08:47:09 -0400
> +Subject: [PATCH] Remove OOM tests from runtest/mm
> +
> +Disable OOM tests, as they might cause oeqa ssh connection lost
> +
> +Upstream-Status: Inappropriate [oe-core specific]
> +Signed-off-by: Mingde (Matthew) Zeng <matthew.zeng@windriver.com>
> +---
> + runtest/mm | 6 ------
> + 1 file changed, 6 deletions(-)
> +
> +diff --git a/runtest/mm b/runtest/mm
> +index 4701a14bd..2f9888ec6 100644
> +--- a/runtest/mm
> ++++ b/runtest/mm
> +@@ -75,12 +75,6 @@ ksm06_2 ksm06 -n 10000
> +
> + cpuset01 cpuset01
> +
> +-oom01 oom01
> +-oom02 oom02
> +-oom03 oom03
> +-oom04 oom04
> +-oom05 oom05
> +-
> + swapping01 swapping01 -i 5
> +
> + thp01 thp01 -I 120
> +--
> +2.27.0
> +
> diff --git a/meta/recipes-extended/ltp/ltp_20200515.bb
> b/meta/recipes-extended/ltp/ltp_20200515.bb
> index ece3acf0f9..0c7044d044 100644
> --- a/meta/recipes-extended/ltp/ltp_20200515.bb
> +++ b/meta/recipes-extended/ltp/ltp_20200515.bb
> @@ -37,6 +37,7 @@ SRC_URI = "git://github.com/linux-test-
> project/ltp.git \
>             file://0001-ptrace01-Fix-missing-format-string.patch \
>             file://0001-sigwaitinfo-Do-not-run-invalid-undefined-
> test-cases.patch \
>             file://0001-syscalls-copy_file_range02-Expect-EFBIG-in-
> subcase-m.patch \
> +           file://0001-Remove-OOM-tests-from-runtest-mm.patch \
>             "
> 
>  S = "${WORKDIR}/git"
> --
> 2.27.0
> 


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

* Re: [OE-core] [PATCHv2] ltp: remove OOM tests from runtest/mm
  2020-07-29 13:24 ` [OE-core] " Richard Purdie
@ 2020-07-29 13:41   ` Matthew
  2020-07-29 13:43     ` Richard Purdie
  2020-08-05 15:52   ` Matthew
  1 sibling, 1 reply; 9+ messages in thread
From: Matthew @ 2020-07-29 13:41 UTC (permalink / raw)
  To: Richard Purdie; +Cc: openembedded-core


Richard Purdie <richard.purdie@linuxfoundation.org> writes:

> On Wed, 2020-07-29 at 09:09 -0400, Matthew wrote:
>> Fixes [YOCTO #13802]
>
> Thanks, we can reference the bug but I don't think we can yet claim it
> fixes it.

Sure, would something like the following work?

Reference [YOCTO #13802]

>
> The missing log issue happened again:
> https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/908
> so I think there are multiple issues at play here.

Can you attach this build's log.do_testimage so I can have a further look?

>
> Is there any way to have the logging logged to disk straight away
> rather than stuck internally in oeqa's logging buffers? That way we
> could see which tests had run before a hang?
>
> Also, could we make the scp failure non-fatal, maybe a warning so that
> when it fails we can look at the rest of the logs?

I'll further investigate this idea.

Matthew

>
> Cheers,
>
> Richard
>
>> Remove the OOM tests, since they might cause oeqa ssh
>> connection lost.
>>
>> Signed-off-by: Mingde (Matthew) Zeng <matthew.zeng@windriver.com>
>> ---
>>  ...001-Remove-OOM-tests-from-runtest-mm.patch | 33
>> +++++++++++++++++++
>>  meta/recipes-extended/ltp/ltp_20200515.bb     |  1 +
>>  2 files changed, 34 insertions(+)
>>  create mode 100644 meta/recipes-extended/ltp/ltp/0001-Remove-OOM-
>> tests-from-runtest-mm.patch
>>
>> diff --git a/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-
>> from-runtest-mm.patch b/meta/recipes-extended/ltp/ltp/0001-Remove-
>> OOM-tests-from-runtest-mm.patch
>> new file mode 100644
>> index 0000000000..2d8262be78
>> --- /dev/null
>> +++ b/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-from-
>> runtest-mm.patch
>> @@ -0,0 +1,33 @@
>> +From 373b21dcf7a2879c27cc3dbeac5d83f5d45d22ae Mon Sep 17 00:00:00
>> 2001
>> +From: "Mingde (Matthew) Zeng" <matthewzmd@gmail.com>
>> +Date: Wed, 29 Jul 2020 08:47:09 -0400
>> +Subject: [PATCH] Remove OOM tests from runtest/mm
>> +
>> +Disable OOM tests, as they might cause oeqa ssh connection lost
>> +
>> +Upstream-Status: Inappropriate [oe-core specific]
>> +Signed-off-by: Mingde (Matthew) Zeng <matthew.zeng@windriver.com>
>> +---
>> + runtest/mm | 6 ------
>> + 1 file changed, 6 deletions(-)
>> +
>> +diff --git a/runtest/mm b/runtest/mm
>> +index 4701a14bd..2f9888ec6 100644
>> +--- a/runtest/mm
>> ++++ b/runtest/mm
>> +@@ -75,12 +75,6 @@ ksm06_2 ksm06 -n 10000
>> +
>> + cpuset01 cpuset01
>> +
>> +-oom01 oom01
>> +-oom02 oom02
>> +-oom03 oom03
>> +-oom04 oom04
>> +-oom05 oom05
>> +-
>> + swapping01 swapping01 -i 5
>> +
>> + thp01 thp01 -I 120
>> +--
>> +2.27.0
>> +
>> diff --git a/meta/recipes-extended/ltp/ltp_20200515.bb
>> b/meta/recipes-extended/ltp/ltp_20200515.bb
>> index ece3acf0f9..0c7044d044 100644
>> --- a/meta/recipes-extended/ltp/ltp_20200515.bb
>> +++ b/meta/recipes-extended/ltp/ltp_20200515.bb
>> @@ -37,6 +37,7 @@ SRC_URI = "git://github.com/linux-test-
>> project/ltp.git \
>>             file://0001-ptrace01-Fix-missing-format-string.patch \
>>             file://0001-sigwaitinfo-Do-not-run-invalid-undefined-
>> test-cases.patch \
>>             file://0001-syscalls-copy_file_range02-Expect-EFBIG-in-
>> subcase-m.patch \
>> +           file://0001-Remove-OOM-tests-from-runtest-mm.patch \
>>             "
>>
>>  S = "${WORKDIR}/git"
>> --
>> 2.27.0
>> 


--
Mingde (Matthew) Zeng

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

* Re: [OE-core] [PATCHv2] ltp: remove OOM tests from runtest/mm
  2020-07-29 13:41   ` Matthew
@ 2020-07-29 13:43     ` Richard Purdie
  2020-07-29 23:23       ` Khem Raj
  0 siblings, 1 reply; 9+ messages in thread
From: Richard Purdie @ 2020-07-29 13:43 UTC (permalink / raw)
  To: Mingde (Matthew) Zeng; +Cc: openembedded-core

On Wed, 2020-07-29 at 09:41 -0400, Mingde (Matthew) Zeng wrote:
> Richard Purdie <richard.purdie@linuxfoundation.org> writes:
> 
> > On Wed, 2020-07-29 at 09:09 -0400, Matthew wrote:
> > > Fixes [YOCTO #13802]
> > 
> > Thanks, we can reference the bug but I don't think we can yet claim
> > it
> > fixes it.
> 
> Sure, would something like the following work?
> 
> Reference [YOCTO #13802]

Simply "[YOCTO #13802]" is fine (I'll tweak the commit message as I
merge).

> > The missing log issue happened again:
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/908
> > so I think there are multiple issues at play here.
> 
> Can you attach this build's log.do_testimage so I can have a further
> look?

Sorry, that build directory has been reused in this case :( Next time
that happens I'll try and spot/grab it.

> > Is there any way to have the logging logged to disk straight away
> > rather than stuck internally in oeqa's logging buffers? That way we
> > could see which tests had run before a hang?
> > 
> > Also, could we make the scp failure non-fatal, maybe a warning so
> > that
> > when it fails we can look at the rest of the logs?
> 
> I'll further investigate this idea.

Thanks!

Cheers,

Richard


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

* Re: [OE-core] [PATCHv2] ltp: remove OOM tests from runtest/mm
  2020-07-29 13:43     ` Richard Purdie
@ 2020-07-29 23:23       ` Khem Raj
  2020-07-30  3:27         ` Matthew
  0 siblings, 1 reply; 9+ messages in thread
From: Khem Raj @ 2020-07-29 23:23 UTC (permalink / raw)
  To: Richard Purdie
  Cc: Mingde (Matthew) Zeng, Patches and discussions about the oe-core layer

on master-next I am also seeing

step1b: WARNING: ltp-20200515-r0 do_patch: QA Issue: Patch log
indicates that patches do not apply cleanly. [patch-fuzz]

I wonder if its related to this as well.

On Wed, Jul 29, 2020 at 6:43 AM Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>
> On Wed, 2020-07-29 at 09:41 -0400, Mingde (Matthew) Zeng wrote:
> > Richard Purdie <richard.purdie@linuxfoundation.org> writes:
> >
> > > On Wed, 2020-07-29 at 09:09 -0400, Matthew wrote:
> > > > Fixes [YOCTO #13802]
> > >
> > > Thanks, we can reference the bug but I don't think we can yet claim
> > > it
> > > fixes it.
> >
> > Sure, would something like the following work?
> >
> > Reference [YOCTO #13802]
>
> Simply "[YOCTO #13802]" is fine (I'll tweak the commit message as I
> merge).
>
> > > The missing log issue happened again:
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/908
> > > so I think there are multiple issues at play here.
> >
> > Can you attach this build's log.do_testimage so I can have a further
> > look?
>
> Sorry, that build directory has been reused in this case :( Next time
> that happens I'll try and spot/grab it.
>
> > > Is there any way to have the logging logged to disk straight away
> > > rather than stuck internally in oeqa's logging buffers? That way we
> > > could see which tests had run before a hang?
> > >
> > > Also, could we make the scp failure non-fatal, maybe a warning so
> > > that
> > > when it fails we can look at the rest of the logs?
> >
> > I'll further investigate this idea.
>
> Thanks!
>
> Cheers,
>
> Richard
>
> 

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

* Re: [OE-core] [PATCHv2] ltp: remove OOM tests from runtest/mm
  2020-07-29 23:23       ` Khem Raj
@ 2020-07-30  3:27         ` Matthew
  2020-07-30  3:34           ` Khem Raj
  0 siblings, 1 reply; 9+ messages in thread
From: Matthew @ 2020-07-30  3:27 UTC (permalink / raw)
  To: Khem Raj; +Cc: Richard Purdie, Patches and discussions about the oe-core layer


Khem Raj <raj.khem@gmail.com> writes:

> on master-next I am also seeing
>
> step1b: WARNING: ltp-20200515-r0 do_patch: QA Issue: Patch log
> indicates that patches do not apply cleanly. [patch-fuzz]
>
> I wonder if its related to this as well.

My bad.

Just a couple days ago ltp master introduced a commit directly effected the line above my change, resulting the diff to look slightly different.

I could send a patch on top of this commit, or is it possible to revert this merged commit altogether and I will send a new patch, to keep the git log clean. Whichever preferred by YP.


Matthew

>
> On Wed, Jul 29, 2020 at 6:43 AM Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
>>
>> On Wed, 2020-07-29 at 09:41 -0400, Mingde (Matthew) Zeng wrote:
>> > Richard Purdie <richard.purdie@linuxfoundation.org> writes:
>> >
>> > > On Wed, 2020-07-29 at 09:09 -0400, Matthew wrote:
>> > > > Fixes [YOCTO #13802]
>> > >
>> > > Thanks, we can reference the bug but I don't think we can yet claim
>> > > it
>> > > fixes it.
>> >
>> > Sure, would something like the following work?
>> >
>> > Reference [YOCTO #13802]
>>
>> Simply "[YOCTO #13802]" is fine (I'll tweak the commit message as I
>> merge).
>>
>> > > The missing log issue happened again:
>> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/908
>> > > so I think there are multiple issues at play here.
>> >
>> > Can you attach this build's log.do_testimage so I can have a further
>> > look?
>>
>> Sorry, that build directory has been reused in this case :( Next time
>> that happens I'll try and spot/grab it.
>>
>> > > Is there any way to have the logging logged to disk straight away
>> > > rather than stuck internally in oeqa's logging buffers? That way we
>> > > could see which tests had run before a hang?
>> > >
>> > > Also, could we make the scp failure non-fatal, maybe a warning so
>> > > that
>> > > when it fails we can look at the rest of the logs?
>> >
>> > I'll further investigate this idea.
>>
>> Thanks!
>>
>> Cheers,
>>
>> Richard
>>
>> 


--
Mingde (Matthew) Zeng

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

* Re: [OE-core] [PATCHv2] ltp: remove OOM tests from runtest/mm
  2020-07-30  3:27         ` Matthew
@ 2020-07-30  3:34           ` Khem Raj
  2020-07-30  3:56             ` Matthew
  0 siblings, 1 reply; 9+ messages in thread
From: Khem Raj @ 2020-07-30  3:34 UTC (permalink / raw)
  To: Mingde (Matthew) Zeng
  Cc: Richard Purdie, Patches and discussions about the oe-core layer

On Wed, Jul 29, 2020 at 8:27 PM Mingde (Matthew) Zeng
<matthew.zeng@windriver.com> wrote:
>
>
> Khem Raj <raj.khem@gmail.com> writes:
>
> > on master-next I am also seeing
> >
> > step1b: WARNING: ltp-20200515-r0 do_patch: QA Issue: Patch log
> > indicates that patches do not apply cleanly. [patch-fuzz]
> >
> > I wonder if its related to this as well.
>
> My bad.
>
> Just a couple days ago ltp master introduced a commit directly effected the line above my change, resulting the diff to look slightly different.
>
> I could send a patch on top of this commit, or is it possible to revert this merged commit altogether and I will send a new patch, to keep the git log clean. Whichever preferred by YP.
>

its not in master yet so please send a v2.

>
> Matthew
>
> >
> > On Wed, Jul 29, 2020 at 6:43 AM Richard Purdie
> > <richard.purdie@linuxfoundation.org> wrote:
> >>
> >> On Wed, 2020-07-29 at 09:41 -0400, Mingde (Matthew) Zeng wrote:
> >> > Richard Purdie <richard.purdie@linuxfoundation.org> writes:
> >> >
> >> > > On Wed, 2020-07-29 at 09:09 -0400, Matthew wrote:
> >> > > > Fixes [YOCTO #13802]
> >> > >
> >> > > Thanks, we can reference the bug but I don't think we can yet claim
> >> > > it
> >> > > fixes it.
> >> >
> >> > Sure, would something like the following work?
> >> >
> >> > Reference [YOCTO #13802]
> >>
> >> Simply "[YOCTO #13802]" is fine (I'll tweak the commit message as I
> >> merge).
> >>
> >> > > The missing log issue happened again:
> >> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/908
> >> > > so I think there are multiple issues at play here.
> >> >
> >> > Can you attach this build's log.do_testimage so I can have a further
> >> > look?
> >>
> >> Sorry, that build directory has been reused in this case :( Next time
> >> that happens I'll try and spot/grab it.
> >>
> >> > > Is there any way to have the logging logged to disk straight away
> >> > > rather than stuck internally in oeqa's logging buffers? That way we
> >> > > could see which tests had run before a hang?
> >> > >
> >> > > Also, could we make the scp failure non-fatal, maybe a warning so
> >> > > that
> >> > > when it fails we can look at the rest of the logs?
> >> >
> >> > I'll further investigate this idea.
> >>
> >> Thanks!
> >>
> >> Cheers,
> >>
> >> Richard
> >>
> >> 
>
>
> --
> Mingde (Matthew) Zeng

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

* Re: [OE-core] [PATCHv2] ltp: remove OOM tests from runtest/mm
  2020-07-30  3:34           ` Khem Raj
@ 2020-07-30  3:56             ` Matthew
  0 siblings, 0 replies; 9+ messages in thread
From: Matthew @ 2020-07-30  3:56 UTC (permalink / raw)
  To: Khem Raj; +Cc: Richard Purdie, Patches and discussions about the oe-core layer


Khem Raj <raj.khem@gmail.com> writes:

> On Wed, Jul 29, 2020 at 8:27 PM Mingde (Matthew) Zeng
> <matthew.zeng@windriver.com> wrote:
>>
>>
>> Khem Raj <raj.khem@gmail.com> writes:
>>
>> > on master-next I am also seeing
>> >
>> > step1b: WARNING: ltp-20200515-r0 do_patch: QA Issue: Patch log
>> > indicates that patches do not apply cleanly. [patch-fuzz]
>> >
>> > I wonder if its related to this as well.
>>
>> My bad.
>>
>> Just a couple days ago ltp master introduced a commit directly effected the line above my change, resulting the diff to look slightly different.
>>
>> I could send a patch on top of this commit, or is it possible to revert this merged commit altogether and I will send a new patch, to keep the git log clean. Whichever preferred by YP.
>>
>
> its not in master yet so please send a v2.

Done.

>
>>
>> Matthew
>>
>> >
>> > On Wed, Jul 29, 2020 at 6:43 AM Richard Purdie
>> > <richard.purdie@linuxfoundation.org> wrote:
>> >>
>> >> On Wed, 2020-07-29 at 09:41 -0400, Mingde (Matthew) Zeng wrote:
>> >> > Richard Purdie <richard.purdie@linuxfoundation.org> writes:
>> >> >
>> >> > > On Wed, 2020-07-29 at 09:09 -0400, Matthew wrote:
>> >> > > > Fixes [YOCTO #13802]
>> >> > >
>> >> > > Thanks, we can reference the bug but I don't think we can yet claim
>> >> > > it
>> >> > > fixes it.
>> >> >
>> >> > Sure, would something like the following work?
>> >> >
>> >> > Reference [YOCTO #13802]
>> >>
>> >> Simply "[YOCTO #13802]" is fine (I'll tweak the commit message as I
>> >> merge).
>> >>
>> >> > > The missing log issue happened again:
>> >> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/908
>> >> > > so I think there are multiple issues at play here.
>> >> >
>> >> > Can you attach this build's log.do_testimage so I can have a further
>> >> > look?
>> >>
>> >> Sorry, that build directory has been reused in this case :( Next time
>> >> that happens I'll try and spot/grab it.
>> >>
>> >> > > Is there any way to have the logging logged to disk straight away
>> >> > > rather than stuck internally in oeqa's logging buffers? That way we
>> >> > > could see which tests had run before a hang?
>> >> > >
>> >> > > Also, could we make the scp failure non-fatal, maybe a warning so
>> >> > > that
>> >> > > when it fails we can look at the rest of the logs?
>> >> >
>> >> > I'll further investigate this idea.
>> >>
>> >> Thanks!
>> >>
>> >> Cheers,
>> >>
>> >> Richard
>> >>
>> >> 
>>
>>
>> --
>> Mingde (Matthew) Zeng


--
Mingde (Matthew) Zeng

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

* Re: [OE-core] [PATCHv2] ltp: remove OOM tests from runtest/mm
  2020-07-29 13:24 ` [OE-core] " Richard Purdie
  2020-07-29 13:41   ` Matthew
@ 2020-08-05 15:52   ` Matthew
  1 sibling, 0 replies; 9+ messages in thread
From: Matthew @ 2020-08-05 15:52 UTC (permalink / raw)
  To: Richard Purdie; +Cc: openembedded-core


Richard Purdie <richard.purdie@linuxfoundation.org> writes:

> On Wed, 2020-07-29 at 09:09 -0400, Matthew wrote:
>> Fixes [YOCTO #13802]
>
> Thanks, we can reference the bug but I don't think we can yet claim it
> fixes it.
>
> The missing log issue happened again:
> https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/908
> so I think there are multiple issues at play here.
>
> Is there any way to have the logging logged to disk straight away
> rather than stuck internally in oeqa's logging buffers? That way we
> could see which tests had run before a hang?

Ltp comes with zoo file already, is it what you're looking for?

>
> Also, could we make the scp failure non-fatal, maybe a warning so that
> when it fails we can look at the rest of the logs?

I submitted a patch that I think should achieve this idea, please have a look.

>
> Cheers,
>
> Richard
>
>> Remove the OOM tests, since they might cause oeqa ssh
>> connection lost.
>>
>> Signed-off-by: Mingde (Matthew) Zeng <matthew.zeng@windriver.com>
>> ---
>>  ...001-Remove-OOM-tests-from-runtest-mm.patch | 33
>> +++++++++++++++++++
>>  meta/recipes-extended/ltp/ltp_20200515.bb     |  1 +
>>  2 files changed, 34 insertions(+)
>>  create mode 100644 meta/recipes-extended/ltp/ltp/0001-Remove-OOM-
>> tests-from-runtest-mm.patch
>>
>> diff --git a/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-
>> from-runtest-mm.patch b/meta/recipes-extended/ltp/ltp/0001-Remove-
>> OOM-tests-from-runtest-mm.patch
>> new file mode 100644
>> index 0000000000..2d8262be78
>> --- /dev/null
>> +++ b/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-from-
>> runtest-mm.patch
>> @@ -0,0 +1,33 @@
>> +From 373b21dcf7a2879c27cc3dbeac5d83f5d45d22ae Mon Sep 17 00:00:00
>> 2001
>> +From: "Mingde (Matthew) Zeng" <matthewzmd@gmail.com>
>> +Date: Wed, 29 Jul 2020 08:47:09 -0400
>> +Subject: [PATCH] Remove OOM tests from runtest/mm
>> +
>> +Disable OOM tests, as they might cause oeqa ssh connection lost
>> +
>> +Upstream-Status: Inappropriate [oe-core specific]
>> +Signed-off-by: Mingde (Matthew) Zeng <matthew.zeng@windriver.com>
>> +---
>> + runtest/mm | 6 ------
>> + 1 file changed, 6 deletions(-)
>> +
>> +diff --git a/runtest/mm b/runtest/mm
>> +index 4701a14bd..2f9888ec6 100644
>> +--- a/runtest/mm
>> ++++ b/runtest/mm
>> +@@ -75,12 +75,6 @@ ksm06_2 ksm06 -n 10000
>> +
>> + cpuset01 cpuset01
>> +
>> +-oom01 oom01
>> +-oom02 oom02
>> +-oom03 oom03
>> +-oom04 oom04
>> +-oom05 oom05
>> +-
>> + swapping01 swapping01 -i 5
>> +
>> + thp01 thp01 -I 120
>> +--
>> +2.27.0
>> +
>> diff --git a/meta/recipes-extended/ltp/ltp_20200515.bb
>> b/meta/recipes-extended/ltp/ltp_20200515.bb
>> index ece3acf0f9..0c7044d044 100644
>> --- a/meta/recipes-extended/ltp/ltp_20200515.bb
>> +++ b/meta/recipes-extended/ltp/ltp_20200515.bb
>> @@ -37,6 +37,7 @@ SRC_URI = "git://github.com/linux-test-
>> project/ltp.git \
>>             file://0001-ptrace01-Fix-missing-format-string.patch \
>>             file://0001-sigwaitinfo-Do-not-run-invalid-undefined-
>> test-cases.patch \
>>             file://0001-syscalls-copy_file_range02-Expect-EFBIG-in-
>> subcase-m.patch \
>> +           file://0001-Remove-OOM-tests-from-runtest-mm.patch \
>>             "
>>
>>  S = "${WORKDIR}/git"
>> --
>> 2.27.0
>> 


--
Mingde (Matthew) Zeng

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

end of thread, other threads:[~2020-08-05 15:52 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-29 13:09 [PATCHv2] ltp: remove OOM tests from runtest/mm Matthew
2020-07-29 13:24 ` [OE-core] " Richard Purdie
2020-07-29 13:41   ` Matthew
2020-07-29 13:43     ` Richard Purdie
2020-07-29 23:23       ` Khem Raj
2020-07-30  3:27         ` Matthew
2020-07-30  3:34           ` Khem Raj
2020-07-30  3:56             ` Matthew
2020-08-05 15:52   ` Matthew

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.