- Subject: Re: [PATCH, v7] cgroups: introduce timer slack controller
- From: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
- Date: Fri, 04 Mar 2011 20:03:42 +0100
- Cc: Paul Menage <menage@xxxxxxxxxx>, Li Zefan <lizf@xxxxxxxxxxxxxx>, containers@xxxxxxxxxxxxxxxxxxxxxxxxxx, jacob.jun.pan@xxxxxxxxxxxxxxx, Arjan van de Ven <arjan@xxxxxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, Matt Helsley <matthltc@xxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, linux-api@xxxxxxxxxxxxxxx
- In-reply-to: <1299084001-3916-2-git-send-email-kirill@xxxxxxxxxxxxx>
- References: <1299084001-3916-1-git-send-email-kirill@xxxxxxxxxxxxx> <1299084001-3916-2-git-send-email-kirill@xxxxxxxxxxxxx>
On Wed, 2011-03-02 at 18:40 +0200, Kirill A. Shutsemov wrote:
> - if (ret < current->timer_slack_ns)
> - return current->timer_slack_ns;
> - return ret;
> + return clamp(__estimate_accuracy(&now),
> + get_task_timer_slack(current), LONG_MAX);
That actually makes the code worse, how about:
min(__estimate_accuracy(), get_task_timer_slack()) ?
--
To unsubscribe from this list: send the line "unsubscribe linux-api" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Home]
[Linux USB Devel]
[Video for Linux]
[Linux Audio Users]
[Photo]
[Yosemite News]
[Yosemite Photos]
[Free Online Dating]
[Linux Kernel]
[Linux SCSI]
[XFree86]