From: luca abeni <luca.abeni@unitn.it>
To: Peter Zijlstra <peterz@infradead.org>
Cc: Ingo Molnar <mingo@redhat.com>, Juri Lelli <juri.lelli@gmail.com>,
linux-kernel@vger.kernel.org
Subject: Another SCHED_DEADLINE bug (with bisection and possible fix)
Date: Tue, 30 Dec 2014 00:27:38 +0100 [thread overview]
Message-ID: <20141230002738.6c12db31@utopia> (raw)
Hi all,
when running some experiments on current git master, I noticed a
regression respect to version 3.18 of the kernel: when invoking
sched_setattr() to change the SCHED_DEADLINE parameters of a task that
is already scheduled by SCHED_DEADLINE, it is possible to crash the
system.
The bug can be reproduced with this testcase:
http://disi.unitn.it/~abeni/reclaiming/bug-test.tgz
Uncompress it, enter the "Bug-Test" directory, and type "make test".
After few cycles, my test machine (a laptop with an intel i7 CPU)
becomes unusable, and freezes.
Since I know that 3.18 is not affected by this bug, I tried a bisect,
that pointed to commit 67dfa1b756f250972bde31d65e3f8fde6aeddc5b
(sched/deadline: Implement cancel_dl_timer() to use in
switched_from_dl()).
By looking at that commit, I suspect the problem is that it removes the
following lines from init_dl_task_timer():
- if (hrtimer_active(timer)) {
- hrtimer_try_to_cancel(timer);
- return;
- }
As a result, when changing the parameters of a SCHED_DEADLINE task
init_dl_task_timer() is invoked again, and it can initialize a pending
timer (not sure why, but this seems to be the cause of the freezes I am
seeing).
So, I modified core.c::__setparam_dl() to invoke init_dl_task_timer()
only if the task is not already scheduled by SCHED_DEADLINE...
Basically, I changed
init_dl_task_timer(dl_se);
into
if (p->sched_class != &dl_sched_class) {
init_dl_task_timer(dl_se);
}
I am not sure if this is the correct fix, but with this change the
kernel survives my test script (mentioned above), and arrives to 500
cycles (without my patch, it crashes after 2 or 3 cycles).
What do you think? Is my patch correct, or should I fix the issue in a
different way?
Thanks,
Luca
next reply other threads:[~2014-12-29 23:27 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-29 23:27 luca abeni [this message]
2015-01-04 22:51 ` Another SCHED_DEADLINE bug (with bisection and possible fix) Kirill Tkhai
2015-01-05 15:21 ` Luca Abeni
2015-01-05 23:07 ` Kirill Tkhai
2015-01-07 7:01 ` Luca Abeni
2015-01-07 12:29 ` Kirill Tkhai
2015-01-07 12:45 ` Luca Abeni
2015-01-07 13:04 ` Kirill Tkhai
2015-01-07 13:14 ` Luca Abeni
2015-01-09 11:15 ` Luca Abeni
2015-01-09 11:46 ` Kirill Tkhai
2015-01-13 8:10 ` Juri Lelli
2015-01-13 9:26 ` Kirill Tkhai
2015-01-13 14:04 ` Peter Zijlstra
2015-01-14 12:43 ` Kirill Tkhai
2015-01-15 11:23 ` Luca Abeni
2015-01-15 12:23 ` Peter Zijlstra
2015-01-15 13:35 ` Luca Abeni
2015-01-28 14:08 ` Peter Zijlstra
2015-01-28 14:40 ` Luca Abeni
2015-01-30 10:25 ` Luca Abeni
2015-01-30 10:35 ` Juri Lelli
2015-01-31 9:56 ` Peter Zijlstra
2015-02-02 11:31 ` Juri Lelli
2015-02-02 13:57 ` Luca Abeni
2015-02-04 14:34 ` [tip:sched/core] sched/deadline: Fix deadline parameter modification handling tip-bot for Peter Zijlstra
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=20141230002738.6c12db31@utopia \
--to=luca.abeni@unitn.it \
--cc=juri.lelli@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=peterz@infradead.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.