lkml.org 
[lkml]   [2008]   [Sep]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 12/13] hrtimer: create a "timer_slack" field in the task struct
On Mon, 8 Sep 2008 16:15:55 +0200
Pavel Machek <pavel@suse.cz> wrote:

> On Mon 2008-09-08 06:40:02, Arjan van de Ven wrote:
> > On Mon, 8 Sep 2008 15:27:16 +0200
> > Pavel Machek <pavel@suse.cz> wrote:
> > > >
> > > > ... which nobody uses today.
> > > > It's not just new syscalls, it's a new glibc api as well at that
> > > > point.
> > >
> > > ...and new applications, yes. I believe applications should
> > > explicitely enable slacking timers.
> >
> > timers are slacking today, at least for select() and poll(), and
> > are a great deal more so than the defaults in this patchkit.
>
> Ok, so select()/poll() may default to non-zero slack for legacy apps.

that's whats there
>
> > The great advantage of the prctl() approach (which is usable) over
> > new system calls and glibc APIs is that it will get used, because
> > the admin can use it just like he uses the "nice" command, on
> > existing software.
>
> Yes, it is a great advantage, but it feels like a hack. Maybe it is
> better done with LD_PRELOAD or something?

that's not working very well in general, and doesn't work across exec
etc.

>
> I'd certianly want the applications to specify slack themselves in
> like 10 years.

We've been talking to Ulrich to figure out what the right API for this
is (eg how to extend select/poll for this) and I still plan to work on
this, but both Linus and Ulrich seem to be very ademant that it means
only few apps will use it (and I agree with that, just look at how many
apps use linux specific APIs such as sendfile(), linux AIO etc... )


--
If you want to reach me at my work email, use arjan@linux.intel.com
For development, discussion and tips for power savings,
visit http://www.lesswatts.org


\
 
 \ /
  Last update: 2008-09-08 16:25    [W:0.080 / U:4.728 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site