On 9/28/21 5:32 PM, Alexander Kanavin wrote:
**[Please note: This e-mail is from an EXTERNAL e-mail address]

How did you run the test? I cherry-picked the proposed fix into my queue, and alas, the fail from ptest-runner is still occurring sporadically.


# ./test.sh
Running tests:
EVPORT
Skipping test
KQUEUE
Skipping test
EPOLL
test-eof: OKAY
test-closed: OKAY
test-weof: OKAY
test-time: OKAY
test-changelist: OKAY
test-fdleak: OKAY
test-dumpevents: OKAY (output not checked)
regress: OKAY
regress_debug: OKAY
DEVPOLL
Skipping test
POLL
test-eof: OKAY
test-closed: OKAY
test-weof: OKAY
test-time: OKAY
test-changelist: OKAY
test-fdleak: OKAY
test-dumpevents: OKAY (output not checked)
regress:
FAIL ../libevent-2.1.12-stable/test/regress_util.c:1478: assert(diff.tv_sec == 0): 1 vs 0util/monotonic_prc_fallback:
[monotonic_prc_fallback FAILED]

[RETRYING monotonic_prc_fallback (3)]
OKAY
regress_debug: OKAY
SELECT
test-eof: OKAY
test-closed: [warn] event_base_new_with_config: no event mechanism available
OKAY
test-weof: OKAY
test-time: OKAY
test-changelist: OKAY
test-fdleak: OKAY
test-dumpevents: OKAY (output not checked)
regress: OKAY
regress_debug: OKAY
WIN32
Skipping test

BTW, let us use https://github.com/libevent/libevent/issues/1193 to track the issue.

Thanks,


Alex

On Tue, 28 Sept 2021 at 11:08, Yu, Mingli <[email protected] <mailto:[email protected]>> wrote:



    On 9/28/21 1:15 AM, Alexandre Belloni wrote:
     > [Please note: This e-mail is from an EXTERNAL e-mail address]
     >
     > Hello,
     >
     > On 22/09/2021 16:35:06+0800, Yu, Mingli wrote:
     >> From: Mingli Yu <[email protected]
    <mailto:[email protected]>>
     >>
     >> Backport a patch to fix the below ptest failure:
     >>   # ./run-ptest
     >>   Running tests:
     >>   EVPORT
     >>   Skipping test
     >>   KQUEUE
     >>   Skipping test
     >>   EPOLL
     >>   test-eof: OKAY
     >>   test-dumpevents: OKAY (output not checked)
     >>   regress:
     >>    FAIL ../libevent-2.1.12-stable/test/regress_util.c:1478:
    assert(diff.tv_sec == 0): 1 vs 0 util/monotonic_prc_fallback:
     >>    [monotonic_prc_fallback FAILED]
     >>   1/312 TESTS FAILED. (33 skipped)
     >>   FAILED
     >>   regress_debug:
     >>    FAIL ../libevent-2.1.12-stable/test/regress_util.c:1478:
    assert(diff.tv_sec == 0): 1 vs 0 util/monotonic_prc_fallback:
     >>    [monotonic_prc_fallback FAILED]
     >>   1/312 TESTS FAILED. (33 skipped)
     >>   FAILED
     >>   [snip]
     >>
     >
     > Thanks, I did apply this patch on top of my kirkstone-next branch
    for a
     > while and unfortunately, this didn't solve the issue on the
     > autobuilders. What I know for now is that the time difference in the
     > loop is not the issue and what is failing is actually the setup
    of the
     > build, i.e, the evutil_configure_monotonic_time_() call fails in
     > test_evutil_monotonic_prc(). I still dodn't find why as this should
     > simply return 0 under Linux. We are looking at thet a bit more in
    depth
     > this week but we may end up disabling that test.

    Interesting, the patch did silence the failure in my env.

    Thanks,

     >
     >
     >> Signed-off-by: Mingli Yu <[email protected]
    <mailto:[email protected]>>
     >> ---
     >>   ...-monotonic_prc_fallback-as-retriable.patch | 30
    +++++++++++++++++++
     >>   .../libevent/libevent_2.1.12.bb
<https://urldefense.com/v3/__http://libevent_2.1.12.bb__;!!AjveYdw8EvQ!OV1jBmLZjnKzcZVfasjKZH6gL-LzB-G3K8HKJ3DSiZ0whmoaFuYLbnuTgZhJYova8kU$>              |  1 +
     >>   2 files changed, 31 insertions(+)
     >>   create mode 100644
    
meta/recipes-support/libevent/libevent/0001-test-mark-util-monotonic_prc_fallback-as-retriable.patch
     >>
     >> diff --git
    
a/meta/recipes-support/libevent/libevent/0001-test-mark-util-monotonic_prc_fallback-as-retriable.patch
    
b/meta/recipes-support/libevent/libevent/0001-test-mark-util-monotonic_prc_fallback-as-retriable.patch
     >> new file mode 100644
     >> index 0000000000..754c446a67
     >> --- /dev/null
     >> +++
    
b/meta/recipes-support/libevent/libevent/0001-test-mark-util-monotonic_prc_fallback-as-retriable.patch
     >> @@ -0,0 +1,30 @@
     >> +From 8fc3fb57ac1b1b0f338506cb86e3adac29d684d7 Mon Sep 17
    00:00:00 2001
     >> +From: Azat Khuzhin <[email protected] <mailto:[email protected]>>
     >> +Date: Wed, 22 Sep 2021 15:14:01 +0800
     >> +Subject: [PATCH] test: mark util/monotonic_prc_fallback as
    retriable
     >> +
     >> +Refs: #1193
     >> +
     >> +Upstream-Status: Backport
    
[https://github.com/libevent/libevent/commit/04fcd7c6df158bb65261867de4b9ec8439696934
    
<https://urldefense.com/v3/__https://github.com/libevent/libevent/commit/04fcd7c6df158bb65261867de4b9ec8439696934__;!!AjveYdw8EvQ!OV1jBmLZjnKzcZVfasjKZH6gL-LzB-G3K8HKJ3DSiZ0whmoaFuYLbnuTgZhJv4-tI38$>]
     >> +
     >> +Signed-off-by: Mingli Yu <[email protected]
    <mailto:[email protected]>>
     >> +---
     >> + test/regress_util.c | 2 +-
     >> + 1 file changed, 1 insertion(+), 1 deletion(-)
     >> +
     >> +diff --git a/test/regress_util.c b/test/regress_util.c
     >> +index 45caa27..a9e80db 100644
     >> +--- a/test/regress_util.c
     >> ++++ b/test/regress_util.c
     >> +@@ -1672,7 +1672,7 @@ struct testcase_t util_testcases[] = {
     >> +     { "monotonic_res_fallback", test_evutil_monotonic_res,
    TT_OFF_BY_DEFAULT, &basic_setup, (void*)"fallback" },
     >> +     { "monotonic_prc", test_evutil_monotonic_prc, 0,
    &basic_setup, (void*)"" },
     >> +     { "monotonic_prc_precise", test_evutil_monotonic_prc,
    TT_RETRIABLE, &basic_setup, (void*)"precise" },
     >> +-    { "monotonic_prc_fallback", test_evutil_monotonic_prc, 0,
    &basic_setup, (void*)"fallback" },
     >> ++    { "monotonic_prc_fallback", test_evutil_monotonic_prc,
    TT_RETRIABLE, &basic_setup, (void*)"fallback" },
     >> +     { "date_rfc1123", test_evutil_date_rfc1123, 0, NULL, NULL },
     >> +     { "evutil_v4addr_is_local", test_evutil_v4addr_is_local,
    0, NULL, NULL },
     >> +     { "evutil_v6addr_is_local", test_evutil_v6addr_is_local,
    0, NULL, NULL },
     >> +--
     >> +2.17.1
     >> +
     >> diff --git a/meta/recipes-support/libevent/libevent_2.1.12.bb
    
<https://urldefense.com/v3/__http://libevent_2.1.12.bb__;!!AjveYdw8EvQ!OV1jBmLZjnKzcZVfasjKZH6gL-LzB-G3K8HKJ3DSiZ0whmoaFuYLbnuTgZhJYova8kU$>
    b/meta/recipes-support/libevent/libevent_2.1.12.bb
    
<https://urldefense.com/v3/__http://libevent_2.1.12.bb__;!!AjveYdw8EvQ!OV1jBmLZjnKzcZVfasjKZH6gL-LzB-G3K8HKJ3DSiZ0whmoaFuYLbnuTgZhJYova8kU$>
     >> index 4b419eab22..3f29c0cffe 100644
     >> --- a/meta/recipes-support/libevent/libevent_2.1.12.bb
    
<https://urldefense.com/v3/__http://libevent_2.1.12.bb__;!!AjveYdw8EvQ!OV1jBmLZjnKzcZVfasjKZH6gL-LzB-G3K8HKJ3DSiZ0whmoaFuYLbnuTgZhJYova8kU$>
     >> +++ b/meta/recipes-support/libevent/libevent_2.1.12.bb
    
<https://urldefense.com/v3/__http://libevent_2.1.12.bb__;!!AjveYdw8EvQ!OV1jBmLZjnKzcZVfasjKZH6gL-LzB-G3K8HKJ3DSiZ0whmoaFuYLbnuTgZhJYova8kU$>
     >> @@ -16,6 +16,7 @@ SRC_URI =
    "https://github.com/libevent/libevent/releases/download/release-${PV}-
    
<https://urldefense.com/v3/__https://github.com/libevent/libevent/releases/download/release-$*7BPV*7D-__;JSU!!AjveYdw8EvQ!OV1jBmLZjnKzcZVfasjKZH6gL-LzB-G3K8HKJ3DSiZ0whmoaFuYLbnuTgZhJfM99jEA$>
     >>              file://run-ptest \
>> file://0001-test-regress_dns.c-patch-out-tests-that-require-a-wo.patch \ >> file://0002-test-regress.h-Increase-default-timeval-tolerance-50.patch \ >> +  file://0001-test-mark-util-monotonic_prc_fallback-as-retriable.patch \
     >>              "
     >>
     >>   SRC_URI[sha256sum] =
    "92e6de1be9ec176428fd2367677e61ceffc2ee1cb119035037a27d346b0403bb"
     >> --
     >> 2.32.0
     >>
     >
     >>
     >>
     >>
     >
     >
     > --
     > Alexandre Belloni, co-owner and COO, Bootlin
     > Embedded Linux and Kernel engineering
     > https://bootlin.com
    
<https://urldefense.com/v3/__https://bootlin.com__;!!AjveYdw8EvQ!OV1jBmLZjnKzcZVfasjKZH6gL-LzB-G3K8HKJ3DSiZ0whmoaFuYLbnuTgZhJbAyvLVM$>
     >

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156436): 
https://lists.openembedded.org/g/openembedded-core/message/156436
Mute This Topic: https://lists.openembedded.org/mt/85785430/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to