* Thomas Gleixner <[email protected]> [060225 08:16]:
> > +#ifdef CONFIG_NO_IDLE_HZ
> > +/**
> > + * hrtimer_get_next - get next hrtimer to expire
> > + *
> > + * @bases: ktimer base array
> > + */
> > +static inline struct hrtimer *hrtimer_get_next(struct hrtimer_base *bases)
> > +{
> > + unsigned long flags;
> > + struct hrtimer *timer = NULL;
> > + int i;
> > +
> > + for (i = 0; i < MAX_HRTIMER_BASES; i++) {
> > + struct hrtimer_base *base;
> > + struct hrtimer *cur;
> > +
> > + base = &bases[i];
> > + spin_lock_irqsave(&base->lock, flags);
> > + cur = rb_entry(base->first, struct hrtimer, node);
> > + spin_unlock_irqrestore(&base->lock, flags);
> > +
> > + if (cur == NULL)
> > + continue;
> > +
> > + if (timer == NULL || cur->expires.tv64 < timer->expires.tv64)
> > + timer = cur;
> > + }
> > +
> > + return timer;
> > +}
>
> This is racy on SMP. nanosleep hrtimers are on the stack and can go away
> due to a signal. posix timers can be removed and destroyed on another
> CPU.
This should be fixed. But just as a note, we can tolerate some removed
timer values values as it would be just an extra timer interrupt.
> Also the expires fields of CLOCK_MONOTONIC and CLOCK_REALTIME based
> tiemrs can not really be compared. Expiry value is absolute time of the
> respective base clock.
OK. How can we get the first event in nanoseconds easily?
> > +/**
> > + * ktime_to_jiffies - converts ktime to jiffies
> > + *
> > + * @event: ktime event to be converted to jiffies
> > + *
> > + * Caller must take care of xtime locking.
> > + */
> > +static inline unsigned long ktime_to_jiffies(const ktime_t event)
> > +{
> > + ktime_t now, delta;
> > +
> > + now = timespec_to_ktime(xtime);
> > + delta = ktime_sub(event, now);
> > +
> > + return jiffies + (((delta.tv64 * NSEC_CONVERSION) >>
> > + (NSEC_JIFFIE_SC - SEC_JIFFIE_SC)) >> SEC_JIFFIE_SC);
> > +}
>
> Only CLOCK_REALTIME based timers are based on xtime. For CLOCK_MONOTONIC
> based timers this calculation is off by wall_to_monotonic.
Needs to be fixed then.
> > +/**
> > + * hrtimer_next_jiffie - get next hrtimer event in jiffies
> > + *
> > + * Called from next_timer_interrupt() to get the next hrtimer event.
> > + * Eventually we should change next_timer_interrupt() to return
> > + * results in nanoseconds instead of jiffies. Caller must host xtime_lock.
>
> S390 does not hold xtime lock when calling next_timer_interrupt !
Good point. Needs to be fixed.
> I look for a sane solution.
We really should fix this for 2.6.16. Then maybe after 2.6.16 we can
convert next_timer_interrupt() to return nanosecond instead of jiffies.
And then we can get rid of jiffies again in the hrtimer code :)
Tony
-
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]
