From: Peter Korsgaard <peter@korsgaard.com>
To: Julien Olivain <ju.o@free.fr>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 1/1] support/testing: improve weston test reliablity
Date: Sun, 08 Dec 2024 08:01:46 +0100 [thread overview]
Message-ID: <875xnu3bfp.fsf@dell.be.48ers.dk> (raw)
In-Reply-To: <20241207150131.1055326-1-ju.o@free.fr> (Julien Olivain's message of "Sat, 7 Dec 2024 16:01:31 +0100")
>>>>> "Julien" == Julien Olivain <ju.o@free.fr> writes:
> The weston runtime test is unreliable, depending on the execution speed
> of its runner. Example of failure is [1], and success is [2]. This
> commit improves the test in several ways, to make it more robust to
> execution speed variations:
> - The command started in background (weston, weston-simple-egl) are
> now started in a subshell. This suppresses the job control messages
> when they are stopped. Those messages could interfere with the parsing
> of the output;
> - Wait time are moved outside the emulator;
> - The kernel argument vt.global_cursor_default=0 is added, to make sure
> cursors are globally disabled, since the test use vkms display CRCs;
> - The memory of the emulator is increased to 512M. This test uses cpio
> initramfs, and the filesystem size increased;
> - The vkms driver emulates a "vsync" event, but can generate a warning
> when the system is too slow. This warning is printed on the console by
> the klogd daemon in its default configuration. This commit adds the
> overlay file /etc/default/klogd to limit only kernel emergency
> messages to be printed on the console. This change fixes the failure
> seen in [1] ;
> - Some sleep time values were adjusted to run on a "performant" idle
> host. Those values are not suitable in all situations. In the
> meantime a generic retry mechanism is added in the test infra, this
> commit adds a retry logic inspired from the test_flutter runtime
> test.
> Fixes: [1]
> [1] https://gitlab.com/buildroot.org/buildroot/-/jobs/8562483474
> [2] https://gitlab.com/buildroot.org/buildroot/-/jobs/8435236652
> Signed-off-by: Julien Olivain <ju.o@free.fr>
Committed, thanks.
--
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2024-12-08 7:01 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-07 15:01 [Buildroot] [PATCH 1/1] support/testing: improve weston test reliablity Julien Olivain
2024-12-08 7:01 ` Peter Korsgaard [this message]
2024-12-08 14:48 ` Peter Korsgaard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=875xnu3bfp.fsf@dell.be.48ers.dk \
--to=peter@korsgaard.com \
--cc=buildroot@buildroot.org \
--cc=ju.o@free.fr \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.