All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Weinberger <richard@nod.at>
To: Jan Kiszka <jan.kiszka@siemens.com>
Cc: xenomai <xenomai@xenomai.org>
Subject: Re: [PATCH v2 0/9] Revive alchemy, pSOS and VxWorks tests
Date: Thu, 14 Apr 2022 17:13:41 +0200 (CEST)	[thread overview]
Message-ID: <1588685029.254441.1649949221432.JavaMail.zimbra@nod.at> (raw)
In-Reply-To: <7d2e38dc-7386-e703-2e09-572874c5e162@siemens.com>

----- Ursprüngliche Mail -----
> Von: "Jan Kiszka" <jan.kiszka@siemens.com>
>>   task5 fails:
>> 	[9] at task-5.c:79
>> 	[1] at task-5.c:23
>> 	[10] at task-5.c:87
>> 	[3] at task-5.c:40
>> 	[11] at task-5.c:95
>> 	[4] at task-5.c:45
>> 	[5] at task-5.c:50
>> 	[6] at task-5.c:55
>> 	[2] at task-5.c:28
>> 	[7] at task-5.c:60
>> 	   0"003.160| BUG in __traceobj_check_abort(): [FGND] wrong return status:
>> 		      task-5.c:63 => EINVAL (want OK)


This failure is a little trickier.

Line 62 is:
ret = rt_task_set_priority(&t_bgnd, info.prio + 1);
ret is EINVAL, that's why the assert in line 63 fails.
It fails because t_bgnd has already terminated.

This concurs also with the above marker [2].
[2] is reached when t_bgnd is done.

The foreground task does:
        ret = rt_task_inquire(NULL, &info);
        traceobj_assert(&trobj, ret == 0 && info.prio == 21);

        traceobj_mark(&trobj, 6);

        ret = rt_task_set_priority(&t_bgnd, info.prio);
        traceobj_check(&trobj, ret, 0);

        traceobj_mark(&trobj, 7);

        ret = rt_task_set_priority(&t_bgnd, info.prio + 1);
        traceobj_check(&trobj, ret, 0);

        traceobj_mark(&trobj, 8);

So it asks for it's own priority, it must be 21, that's okay.
Then it raises the priority of t_bgnd from 20 to 21
and assumes that no scheduling happens. But this seems to fail.

Did the nucleus CPU scheduler guarantee that giving another task
the same priority of the calling task will favour the caller?
Now the gifted task seems to win.

Thanks,
//richard


  parent reply	other threads:[~2022-04-14 15:13 UTC|newest]

Thread overview: 112+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-13 21:58 [PATCH v2 0/9] Revive alchemy, pSOS and VxWorks tests Richard Weinberger
2022-04-13 21:58 ` [PATCH 1/9] testsuite: Move alchemy tests into testsuite/ Richard Weinberger
2022-04-13 21:58 ` [PATCH 2/9] testsuite: Hook up alchemytests Richard Weinberger
2022-04-13 21:58 ` [PATCH 3/9] testsuite: Add a simple test driver for alchemytests Richard Weinberger
2022-04-14 11:28   ` Jan Kiszka
2022-04-14 11:42     ` Richard Weinberger
2022-04-13 21:58 ` [PATCH 4/9] Remove old alchemy tests Makefile Richard Weinberger
2022-04-13 21:58 ` [PATCH 5/9] alchemytests: Fix gcc warning in buffer-1 Richard Weinberger
2022-04-13 21:58 ` [PATCH 6/9] alchemytests: Fix gcc warning in task-9 Richard Weinberger
2022-04-13 21:58 ` [PATCH 7/9] testsuite: Move pSOS tests into testsuite/ Richard Weinberger
2022-04-13 21:58 ` [PATCH 8/9] testsuite: Add a simple test driver for psostests Richard Weinberger
2022-04-13 21:58 ` [PATCH 9/9] testsuite: Integrate vwworks tests into testsuite/ Richard Weinberger
2022-04-14 11:18 ` [PATCH v2 0/9] Revive alchemy, pSOS and VxWorks tests Jan Kiszka
2022-04-14 11:29   ` Richard Weinberger
2022-04-14 12:58   ` Richard Weinberger
2022-04-14 14:15     ` Bezdeka, Florian
2022-04-14 14:19       ` Richard Weinberger
2022-04-14 15:13   ` Richard Weinberger [this message]
2022-04-14 15:34     ` Jan Kiszka
2022-04-14 15:41       ` Richard Weinberger
2022-04-14 15:43         ` Jan Kiszka
2023-04-06 10:33 ` [PATCH v3 0/11] " Aaron Marcher
2023-04-06 10:33   ` [PATCH 01/11] testsuite: Move alchemy tests into testsuite/ Aaron Marcher
2023-04-06 10:33   ` [PATCH 02/11] testsuite: Hook up alchemytests Aaron Marcher
2023-04-06 10:33   ` [PATCH 03/11] testsuite: Add a simple test driver for alchemytests Aaron Marcher
2023-04-06 10:33   ` [PATCH 04/11] Remove old alchemy tests Makefile Aaron Marcher
2023-04-06 10:33   ` [PATCH 05/11] alchemytests: Fix gcc warning in buffer-1 Aaron Marcher
2023-04-06 10:33   ` [PATCH 06/11] alchemytests: Fix gcc warning in task-9 Aaron Marcher
2023-04-06 10:33   ` [PATCH 07/11] testsuite: Move pSOS tests into testsuite/ Aaron Marcher
2023-04-06 10:33   ` [PATCH 08/11] testsuite: Add a simple test driver for psostests Aaron Marcher
2023-04-06 10:33   ` [PATCH 09/11] testsuite: Integrate vwworks tests into testsuite/ Aaron Marcher
2023-04-06 10:33   ` [PATCH 10/11] alchemytests: check pipe-1 asprintf ret value Aaron Marcher
2023-04-06 10:33   ` [PATCH 11/11] vxworkstests: fix Makefile.am reference Aaron Marcher
2023-04-16  7:33     ` Jan Kiszka
2023-04-16  7:32   ` [PATCH v3 0/11] Revive alchemy, pSOS and VxWorks tests Jan Kiszka
2023-04-16  7:41     ` Jan Kiszka
2023-04-16  9:03       ` Richard Weinberger
2023-05-17  7:30     ` Aaron Marcher
2023-05-17  9:35       ` Jan Kiszka
2023-05-20 14:57         ` Florian Bezdeka
2023-07-20 14:51   ` [PATCH v4 00/10] " Aaron Marcher
2023-07-20 14:51     ` [PATCH 01/10] testsuite: Fully integrate alchemytests into smokey Aaron Marcher
2023-07-20 16:50       ` Jan Kiszka
2023-07-20 14:51     ` [PATCH 02/10] testsuite: Fully integrate psostests " Aaron Marcher
2023-07-20 14:51     ` [PATCH 03/10] testsuite: Fully integrate vwworkstests " Aaron Marcher
2023-07-20 14:51     ` [PATCH 04/10] testsuite: Remove old alchemytests Makefile Aaron Marcher
2023-07-20 14:51     ` [PATCH 05/10] alchemytests: Fix gcc warning in buffer-1 Aaron Marcher
2023-07-20 14:51     ` [PATCH 06/10] alchemytests: Fix gcc warning in task-9 Aaron Marcher
2023-07-20 14:51     ` [PATCH 07/10] alchemytests: Check pipe-1 asprintf ret value Aaron Marcher
2023-07-20 14:51     ` [PATCH 08/10] alchemytests: Fix task-2 Aaron Marcher
2023-07-20 14:51     ` [PATCH 09/10] alchemytests: Fix task-5 logic Aaron Marcher
2023-07-20 14:51     ` [PATCH 10/10] testsuite: disable broken tests Aaron Marcher
2023-07-20 14:54     ` [PATCH v4 00/10] Revive alchemy, pSOS and VxWorks tests Jan Kiszka
2023-07-20 15:01       ` Aaron Marcher
2023-07-24 13:52     ` [PATCH v5 " Aaron Marcher
2023-07-24 13:52       ` [PATCH 01/10] testsuite: Fully integrate alchemytests into smokey Aaron Marcher
2023-08-10  7:07         ` Florian Bezdeka
2023-08-11 17:32           ` Jan Kiszka
2023-08-16 12:43             ` Aaron Marcher
2023-09-14  7:41               ` Jan Kiszka
2023-09-20 13:52                 ` [PATCH 0/5] Improvements for smokey skintests Aaron Marcher
2023-09-20 13:52                   ` [PATCH 1/5] lib: Add smokey_run_extprog as helper for testsuite Aaron Marcher
2023-09-22 15:49                     ` Jan Kiszka
2023-09-22 21:03                       ` Richard Weinberger
2023-09-25 13:03                       ` [PATCH] lib: Drop cpu-affinity=0 from skin-tests helper Aaron Marcher
2023-09-25 15:04                         ` Philippe Gerum
2023-09-25 15:12                         ` [PATCH 0/1 V2] " Aaron Marcher
2023-09-25 15:12                           ` [PATCH 1/1] " Aaron Marcher
2023-09-25 15:42                             ` Jan Kiszka
2023-09-28 17:04                               ` Jan Kiszka
2023-09-28 17:30                                 ` Jan Kiszka
2023-09-29 12:12                               ` Aaron Marcher
2023-09-29 12:34                                 ` Jan Kiszka
2023-09-20 13:52                   ` [PATCH 2/5] testsuite: Merge alchemytests in single smokey case Aaron Marcher
2023-09-20 13:52                   ` [PATCH 3/5] testsuite: Merge psostests in a " Aaron Marcher
2023-09-20 13:52                   ` [PATCH 4/5] testsuite: Merge vxworkstests " Aaron Marcher
2023-09-20 13:52                   ` [PATCH 5/5] testsuite: Drop skin test "drivers" Aaron Marcher
2023-09-22 15:52                   ` [PATCH 0/5] Improvements for smokey skintests Jan Kiszka
2023-09-20 14:00                 ` [PATCH 01/10] testsuite: Fully integrate alchemytests into smokey Aaron Marcher
2023-08-16 12:41           ` Aaron Marcher
2023-08-16 14:19             ` Jan Kiszka
2023-07-24 13:52       ` [PATCH 02/10] testsuite: Fully integrate psostests " Aaron Marcher
2023-07-24 13:52       ` [PATCH 03/10] testsuite: Fully integrate vwworkstests " Aaron Marcher
2023-07-24 13:52       ` [PATCH 04/10] testsuite: Remove old alchemytests Makefile Aaron Marcher
2023-07-24 13:52       ` [PATCH 05/10] alchemytests: Fix gcc warning in buffer-1 Aaron Marcher
2023-07-24 13:52       ` [PATCH 06/10] alchemytests: Fix gcc warning in task-9 Aaron Marcher
2023-07-24 13:52       ` [PATCH 07/10] alchemytests: Check pipe-1 asprintf ret value Aaron Marcher
2023-07-24 13:52       ` [PATCH 08/10] alchemytests: Fix task-2 Aaron Marcher
2023-07-24 14:39         ` Richard Weinberger
2023-07-25  8:29           ` [PATCH v6 00/10] Revive alchemy, pSOS and VxWorks tests Aaron Marcher
2023-07-25  8:29             ` [PATCH 01/10] testsuite: Fully integrate alchemytests into smokey Aaron Marcher
2023-08-09  7:28               ` Jan Kiszka
2023-08-09 11:32               ` [PATCH v7 " Jan Kiszka
2023-07-25  8:29             ` [PATCH 02/10] testsuite: Fully integrate psostests " Aaron Marcher
2023-08-09 11:32               ` [PATCH v7 " Jan Kiszka
2023-07-25  8:29             ` [PATCH 03/10] testsuite: Fully integrate vwworkstests " Aaron Marcher
2023-08-09 11:32               ` [PATCH v7 " Jan Kiszka
2023-08-09 15:02                 ` Jan Kiszka
2023-07-25  8:29             ` [PATCH 04/10] testsuite: Remove old alchemytests Makefile Aaron Marcher
2023-07-25  8:29             ` [PATCH 05/10] alchemytests: Fix gcc warning in buffer-1 Aaron Marcher
2023-07-25  8:29             ` [PATCH 06/10] alchemytests: Fix gcc warning in task-9 Aaron Marcher
2023-07-25  8:29             ` [PATCH 07/10] alchemytests: Check pipe-1 asprintf ret value Aaron Marcher
2023-07-25  8:29             ` [PATCH 08/10] alchemytests: Fix task-2 Aaron Marcher
2023-07-25  8:29             ` [PATCH 09/10] alchemytests: Fix task-5 logic Aaron Marcher
2023-07-25  8:29             ` [PATCH 10/10] testsuite: disable broken tests Aaron Marcher
2023-08-03 15:35             ` [PATCH v6 00/10] Revive alchemy, pSOS and VxWorks tests Florian Bezdeka
2023-08-09  7:36             ` Jan Kiszka
2023-08-09 11:33               ` Jan Kiszka
2023-08-16 12:37                 ` Aaron Marcher
2023-08-09 11:32             ` [PATCH v7 11/10] testsuite/smokey: Hook up new tests Jan Kiszka
2023-07-24 13:52       ` [PATCH 09/10] alchemytests: Fix task-5 logic Aaron Marcher
2023-07-24 13:52       ` [PATCH 10/10] testsuite: disable broken tests Aaron Marcher

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=1588685029.254441.1649949221432.JavaMail.zimbra@nod.at \
    --to=richard@nod.at \
    --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.