diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2015-06-11 14:46:44 +0200 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2015-06-19 00:09:56 +0200 |
| commit | c04dca02bc73096435a5c36efd5ccb2171edcbe1 (patch) | |
| tree | 6875551d63882ac926b68a964c9629f45d0c4f7d /kernel/time/tick-broadcast-hrtimer.c | |
| parent | selftest: Timers: Avoid signal deadlock in leap-a-day (diff) | |
| download | linux-c04dca02bc73096435a5c36efd5ccb2171edcbe1.tar.xz linux-c04dca02bc73096435a5c36efd5ccb2171edcbe1.zip | |
hrtimer: Remove HRTIMER_STATE_MIGRATE
I do not understand HRTIMER_STATE_MIGRATE. Unless I am totally
confused it looks buggy and simply unneeded.
migrate_hrtimer_list() sets it to keep hrtimer_active() == T, but this
is not enough: this can fool, say, hrtimer_is_queued() in
dequeue_signal().
Can't migrate_hrtimer_list() simply use HRTIMER_STATE_ENQUEUED?
This fixes the race and we can kill STATE_MIGRATE.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: ktkhai@parallels.com
Cc: rostedt@goodmis.org
Cc: juri.lelli@gmail.com
Cc: pang.xunlei@linaro.org
Cc: wanpeng.li@linux.intel.com
Cc: umgwanakikbuti@gmail.com
Link: http://lkml.kernel.org/r/20150611124743.072387650@infradead.org
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'kernel/time/tick-broadcast-hrtimer.c')
0 files changed, 0 insertions, 0 deletions