Re: [PATCH] hrtimer: make epoll_wait() use the hrtimer rangefeature

From: Davide Libenzi
Date: Thu Aug 26 2010 - 19:23:07 EST


On Fri, 27 Aug 2010, Thomas Gleixner wrote:

> On Thu, 26 Aug 2010, Davide Libenzi wrote:
> > On Sun, 8 Aug 2010, Shawn Bohrer wrote:
> >
> > > This make epoll use hrtimers for the timeout value which prevents
> > > epoll_wait() from timing out up to a millisecond early.
> > >
> > > This mirrors the behavior of select() and poll().
> >
> > I saw this now, since I got notifications from Andrew's patch machine.
> > Is this really needed?
>
> Yes, as it is the last user space interface which is jiffies based.

What? You killed jiffies?! :)
Alright then, the patch itself, besides the different bahavior on timeout
greater/equal to EP_MAX_MSTIMEO (which is not documented, hence does not
exist), looks OK to me.



- Davide


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/