From: "Petr Cervenka" <grugh@centrum.cz>
To: "Philippe Gerum" <rpm@xenomai.org>,
"Gilles Chanteperdrix" <gilles.chanteperdrix@xenomai.org>
Cc: Xenomai <xenomai@xenomai.org>
Subject: Re: [Xenomai] non-blocking rt_task_suspend(NULL)
Date: Wed, 07 May 2014 15:13:10 +0200 [thread overview]
Message-ID: <20140507151310.0F980ADD@centrum.cz> (raw)
In-Reply-To: <5368DC2C.5000505@xenomai.org>
> Od: Philippe Gerum <rpm@xenomai.org>
>>
>> Here it is. It's full of modules and other perhaps not so real-time
>> settings, because it was derived from Kubuntu kernel config file.
>>
>
>Thanks. The issue seems to happen as a result of a relax -> harden transition racing with a signal receipt. In order to help me ruling out some assumptions, could you please apply the patch below, and confirm that no job control is involved (SIGCONT/SIGSTOP) in your application?
>
>Knowing whether any of the two warnings added by this patch is issued when the bug happens would also help solving the issue, TIA.
>
>diff --git a/ksrc/nucleus/pod.c b/ksrc/nucleus/pod.c
>index 0a2ee19..9fb797f 100644
>--- a/ksrc/nucleus/pod.c
>+++ b/ksrc/nucleus/pod.c
>@@ -1379,15 +1379,17 @@ void xnpod_suspend_thread(xnthread_t *thread, xnflags_t mask,
> * context, to collect and act upon the pending Linux
> * signal.
> */
>- if ((mask & XNRELAX) == 0 &&
>- xnthread_test_info(thread, XNKICKED)) {
>- if (wchan) {
>- thread->wchan = wchan;
>- xnsynch_forget_sleeper(thread);
>+ if (xnthread_test_info(thread, XNKICKED)) {
>+ if ((mask & XNRELAX) == 0) {
>+ if (wchan) {
>+ thread->wchan = wchan;
>+ xnsynch_forget_sleeper(thread);
>+ }
>+ xnthread_clear_info(thread, XNRMID | XNTIMEO);
>+ xnthread_set_info(thread, XNBREAK);
>+ goto unlock_and_exit;
> }
>- xnthread_clear_info(thread, XNRMID | XNTIMEO);
>- xnthread_set_info(thread, XNBREAK);
>- goto unlock_and_exit;
>+ WARN_ON(1);
> }
> #endif /* CONFIG_XENO_OPT_PERVASIVE */
>
>diff --git a/ksrc/nucleus/shadow.c b/ksrc/nucleus/shadow.c
>index 38c1423..fc592a2 100644
>--- a/ksrc/nucleus/shadow.c
>+++ b/ksrc/nucleus/shadow.c
>@@ -2696,6 +2696,8 @@ static inline void do_sigwake_event(struct task_struct *p)
> }
> }
>
>+ WARN_ON(!signal_pending(p));
>+
> /*
> * If a relaxed thread is getting a signal while running, we
> * force it out of RPI, so that it won't keep a boosted
>--
Finally, I was able to catch the warning(s). I got 38 of these before task ended:
[ 1109.336726] ------------[ cut here ]------------
[ 1109.336737] WARNING: at kernel/xenomai/nucleus/pod.c:1392 xnpod_suspend_thread+0x17d/0x5a0()
[ 1109.336746] Hardware name: X7SBA
[ 1109.336755] Modules linked in: fr01_rtdm(O) netconsole configfs igb dca e1000e e1000 r8169 rt_e1000(O) rt_r8169(O) rtpacket(O) rtnet(O) coretemp psmouse microcode serio_raw shpchp lpc_ich i3200_edac video edac_core floppy
[ 1109.336908] Pid: 786, comm: ASYNC_TASK_1869 Tainted: G O 3.5.7-debug #38
[ 1109.336917] Call Trace:
[ 1109.336933] [<ffffffff8103cdcf>] warn_slowpath_common+0x7f/0xc0
[ 1109.336949] [<ffffffff8103ce2a>] warn_slowpath_null+0x1a/0x20
[ 1109.336965] [<ffffffff8110b08d>] xnpod_suspend_thread+0x17d/0x5a0
[ 1109.336981] [<ffffffff81117054>] xnshadow_relax+0xf4/0x250
[ 1109.336997] [<ffffffff810cc73c>] ? __ipipe_restore_head+0x7c/0x100
[ 1109.337013] [<ffffffff811181db>] xnshadow_harden+0x30b/0x340
[ 1109.337029] [<ffffffff811183e0>] losyscall_event+0xb0/0x2f0
[ 1109.337045] [<ffffffff810d0829>] ipipe_syscall_hook+0x89/0xd0
[ 1109.337061] [<ffffffff810cbc68>] __ipipe_notify_syscall+0x158/0x340
[ 1109.337076] [<ffffffff8102167a>] __ipipe_syscall_root+0x4a/0x1f0
[ 1109.337092] [<ffffffff81367baa>] __ipipe_syscall_root_thunk+0x35/0x67
[ 1109.337108] [<ffffffff81682217>] ? system_call_after_swapgs+0x54/0x6d
[ 1109.337117] ---[ end trace 9fc5fa66a7479311 ]---
The trace log is in the attachment.
Petr
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: freeze14.txt
URL: <http://www.xenomai.org/pipermail/xenomai/attachments/20140507/37a07957/attachment.txt>
next prev parent reply other threads:[~2014-05-07 13:13 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-02 12:13 [Xenomai] non-blocking rt_task_suspend(NULL) Petr Cervenka
2014-05-02 12:30 ` Gilles Chanteperdrix
2014-05-02 13:16 ` Philippe Gerum
2014-05-06 8:17 ` Petr Cervenka
2014-05-06 8:39 ` Philippe Gerum
2014-05-06 8:56 ` Philippe Gerum
2014-05-06 9:29 ` Petr Cervenka
2014-05-06 12:57 ` Philippe Gerum
2014-05-07 13:13 ` Petr Cervenka [this message]
2014-05-08 15:53 ` Philippe Gerum
2014-05-12 12:37 ` Petr Cervenka
2014-05-12 13:09 ` Philippe Gerum
2014-05-20 12:27 ` Petr Cervenka
2014-05-20 12:54 ` Philippe Gerum
-- strict thread matches above, loose matches on Subject: below --
2014-04-16 16:02 Petr Cervenka
2014-04-16 16:17 ` Gilles Chanteperdrix
2014-04-18 8:51 ` Petr Cervenka
2014-04-22 17:20 ` Gilles Chanteperdrix
2014-04-24 15:06 ` Petr Cervenka
2014-04-24 17:53 ` Gilles Chanteperdrix
2014-04-25 8:38 ` Petr Cervenka
2014-04-16 14:20 Petr Cervenka
2014-04-16 14:28 ` Gilles Chanteperdrix
2014-04-15 12:42 Petr Cervenka
2014-04-16 9:08 ` Gilles Chanteperdrix
2014-04-16 12:22 ` Petr Cervenka
2014-04-16 12:26 ` Gilles Chanteperdrix
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=20140507151310.0F980ADD@centrum.cz \
--to=grugh@centrum.cz \
--cc=gilles.chanteperdrix@xenomai.org \
--cc=rpm@xenomai.org \
--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.