This changes the requeue handling of normal posix timers, instead of a counter
it has now three states: 0 - the timer is idle or running normally, 1 - signal
is queued and callback is pending, 2 - callback is late and timer is stopped
and must be restarted.
This also means if signal callback is still pending, we start the timer but
keep the signal pending, so when the timer expires, it won't touch the signal
structure, which may be read by another cpu at this time to finally deliver the
signal.
This allows to keep the timer running and allows later to keep the common path
smaller (especially most of schedule_next_timer()).
Signed-off-by: Roman Zippel <[email protected]>
---
kernel/posix-timers.c | 50 +++++++++++++++++++++++++-------------------------
1 file changed, 25 insertions(+), 25 deletions(-)
Index: linux-2.6-mm/kernel/posix-timers.c
===================================================================
--- linux-2.6-mm.orig/kernel/posix-timers.c 2005-11-28 22:31:00.000000000 +0100
+++ linux-2.6-mm/kernel/posix-timers.c 2005-11-28 22:31:03.000000000 +0100
@@ -373,7 +373,6 @@ static int schedule_next_timer(struct k_
}
timr->it_overrun_last = timr->it_overrun;
timr->it_overrun = -1;
- ++timr->it_requeue_pending;
return 1;
}
@@ -394,18 +393,24 @@ void do_schedule_next_timer(struct sigin
unsigned long flags;
timr = lock_timer(info->si_tid, &flags);
+ if (!timr)
+ return;
- if (!timr || timr->it_requeue_pending != info->si_sys_private)
- goto exit;
-
- if (timr->it_clock < 0) /* CPU clock */
+ if (timr->it_clock < 0) {
+ /* CPU clock */
+ if (timr->it_requeue_pending != info->si_sys_private)
+ goto exit;
posix_cpu_timer_schedule(timr);
- else if (schedule_next_timer(timr))
- ptimer_start(&timr->it.real.timer);
+ } else {
+ BUG_ON(!timr->it_requeue_pending);
+ if (timr->it_requeue_pending > 1 &&
+ schedule_next_timer(timr))
+ ptimer_start(&timr->it.real.timer);
+ timr->it_requeue_pending = 0;
+ }
info->si_overrun = timr->it_overrun_last;
exit:
- if (timr)
- unlock_timer(timr, flags);
+ unlock_timer(timr, flags);
}
int posix_timer_event(struct k_itimer *timr,int si_private)
@@ -492,21 +497,16 @@ static int posix_timer_fn(struct ptimer
}
if (do_notify) {
- int si_private=0;
-
- if (timr->it.real.incr)
- si_private = ++timr->it_requeue_pending;
- else {
+ if (!timr->it.real.incr)
remove_from_abslist(timr);
- }
-
- if (posix_timer_event(timr, si_private))
- /*
- * signal was not sent because of sig_ignor
- * we will not get a call back to restart it AND
- * it should be restarted.
- */
+ if (!timr->it_requeue_pending) {
+ if (!posix_timer_event(timr, 1))
+ timr->it_requeue_pending = 1;
do_restart = schedule_next_timer(timr);
+ } else {
+ timr->it_requeue_pending = 2;
+ timr->it_overrun++;
+ }
}
unlock_timer(timr, flags); /* hold thru abs lock to keep irq off */
@@ -792,7 +792,7 @@ common_timer_get(struct k_itimer *timr,
posix_time_before(&timr->it.real.timer, &now))
timr->it.real.timer.expires = expires = 0;
if (expires) {
- if (timr->it_requeue_pending & REQUEUE_PENDING ||
+ if (timr->it_requeue_pending > 1 ||
(timr->it_sigev_notify & ~SIGEV_THREAD_ID) == SIGEV_NONE) {
posix_bump_timer(timr, now);
expires = timr->it.real.timer.expires;
@@ -972,8 +972,8 @@ common_timer_set(struct k_itimer *timr,
remove_from_abslist(timr);
- timr->it_requeue_pending = (timr->it_requeue_pending + 2) &
- ~REQUEUE_PENDING;
+ if (timr->it_requeue_pending)
+ timr->it_requeue_pending = 1;
timr->it_overrun_last = 0;
timr->it_overrun = -1;
/*
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
