lib/timerqueue: Rely on rbtree semantics for next timer
authorDavidlohr Bueso <dave@stgolabs.net>
Wed, 24 Jul 2019 15:23:23 +0000 (08:23 -0700)
committerThomas Gleixner <tglx@linutronix.de>
Wed, 24 Jul 2019 15:38:01 +0000 (17:38 +0200)
commit511885d7061eda3eb1faf3f57dcc936ff75863f1
treeaa17e0932d884a2088909c8523d8c3c17e34af06
parentad5e427e0f6b702e52c11d1f7b2b7be3bac7de82
lib/timerqueue: Rely on rbtree semantics for next timer

Simplify the timerqueue code by using cached rbtrees and rely on the tree
leftmost node semantics to get the timer with earliest expiration time.
This is a drop in conversion, and therefore semantics remain untouched.

The runtime overhead of cached rbtrees is be pretty much the same as the
current head->next method, noting that when removing the leftmost node,
a common operation for the timerqueue, the rb_next(leftmost) is O(1) as
well, so the next timer will either be the right node or its parent.
Therefore no extra pointer chasing. Finally, the size of the struct
timerqueue_head remains the same.

Passes several hours of rcutorture.

Signed-off-by: Davidlohr Bueso <dbueso@suse.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lkml.kernel.org/r/20190724152323.bojciei3muvfxalm@linux-r8p5
include/linux/timerqueue.h
lib/timerqueue.c