hrtimer: Remove HRTIMER_STATE_MIGRATE
authorOleg Nesterov <[email protected]>
Thu, 11 Jun 2015 12:46:44 +0000 (14:46 +0200)
committerThomas Gleixner <[email protected]>
Thu, 18 Jun 2015 22:09:56 +0000 (00:09 +0200)
commitc04dca02bc73096435a5c36efd5ccb2171edcbe1
tree6875551d63882ac926b68a964c9629f45d0c4f7d
parent51a16c1e887a5975ada27a3ae935a4f2783005da
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 <[email protected]>
Signed-off-by: Peter Zijlstra (Intel) <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Thomas Gleixner <[email protected]>
include/linux/hrtimer.h
kernel/time/hrtimer.c