From: Philippe Gerum <rpm@xenomai.org>
To: Jan Kiszka <jan.kiszka@siemens.com>, xenomai@xenomai.org
Subject: Re: [PATCH 5/8] testsuite/smokey: reset main()'s SCHED_OTHER policy before each test
Date: Mon, 3 Jun 2019 13:51:57 +0200 [thread overview]
Message-ID: <f016feb5-303f-9fa9-77b1-180f7f3b45a3@xenomai.org> (raw)
In-Reply-To: <3bec3f3f-509a-69ff-bac1-aee18494bf90@siemens.com>
On 6/3/19 1:09 PM, Jan Kiszka wrote:
> On 01.06.19 17:14, Philippe Gerum wrote:
>> Signed-off-by: Philippe Gerum <rpm@xenomai.org>
>> ---
>> testsuite/smokey/main.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/testsuite/smokey/main.c b/testsuite/smokey/main.c
>> index 12321dfc5..5702e825a 100644
>> --- a/testsuite/smokey/main.c
>> +++ b/testsuite/smokey/main.c
>> @@ -22,6 +22,7 @@
>> int main(int argc, char *const argv[])
>> {
>> + struct sched_param param = { .sched_priority = 0 };
>> struct smokey_test *t;
>> int ret, fails = 0;
>> @@ -29,6 +30,7 @@ int main(int argc, char *const argv[])
>> return 0;
>> for_each_smokey_test(t) {
>> + pthread_setschedparam(pthread_self(), SCHED_OTHER, ¶m);
>> ret = t->run(t, argc, argv);
>> if (ret) {
>> if (ret == -ENOSYS) {
>>
>
> Is that a cosmetic cleanup, or did we see tests failing or doing the
> wrong thing? Just to clarify the importance of this fix, also for stable
> (a few word in the commit log would have done that trick for me ;) ).
>
https://www.xenomai.org/pipermail/xenomai/2019-May/040925.html
--
Philippe.
next prev parent reply other threads:[~2019-06-03 11:51 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-01 15:14 [PATCH 0/8] assorted fixes, ARM port to kernel 4.19 Philippe Gerum
2019-06-01 15:14 ` [PATCH 1/8] lib/cobalt: init: fix misleading message Philippe Gerum
2019-06-01 15:14 ` [PATCH 2/8] cobalt/arm: fix context switching code for 4.19+ Philippe Gerum
2019-06-01 15:14 ` [PATCH 3/8] drivers/udd: fix spurious select() read events Philippe Gerum
2019-06-01 15:14 ` [PATCH 4/8] testsuite/smokey: sched-quota: allow up to 1.5% deviation from plan Philippe Gerum
2019-06-01 15:14 ` [PATCH 5/8] testsuite/smokey: reset main()'s SCHED_OTHER policy before each test Philippe Gerum
2019-06-03 11:09 ` Jan Kiszka
2019-06-03 11:51 ` Philippe Gerum [this message]
2019-06-03 12:29 ` Jan Kiszka
2019-06-01 15:15 ` [PATCH 6/8] cobalt/thread: send backtracing request only when XNWARN is set Philippe Gerum
2019-06-01 15:15 ` [PATCH 7/8] cobalt/thread: fix error status (-EINTR) of xnthread_killall() Philippe Gerum
2019-06-01 15:15 ` [PATCH 8/8] lib/cobalt: kickstart backtrace() from safe context Philippe Gerum
2019-06-03 15:00 ` [PATCH 0/8] assorted fixes, ARM port to kernel 4.19 Jan Kiszka
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=f016feb5-303f-9fa9-77b1-180f7f3b45a3@xenomai.org \
--to=rpm@xenomai.org \
--cc=jan.kiszka@siemens.com \
--cc=xenomai@xenomai.org \
/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.