Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751665AbZL1Oa4 (ORCPT ); Mon, 28 Dec 2009 09:30:56 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751415AbZL1Oa4 (ORCPT ); Mon, 28 Dec 2009 09:30:56 -0500 Received: from bombadil.infradead.org ([18.85.46.34]:50849 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751414AbZL1Oaz (ORCPT ); Mon, 28 Dec 2009 09:30:55 -0500 Subject: Re: [RFC 8/12][PATCH] SCHED_DEADLINE: wait next instance syscall added. From: Peter Zijlstra To: Raistlin Cc: linux-kernel , michael trimarchi , Fabio Checconi , Ingo Molnar , Thomas Gleixner , Dhaval Giani , Johan Eker , "p.faure" , Chris Friesen , Steven Rostedt , Henrik Austad , Frederic Weisbecker , Darren Hart , Sven-Thorsten Dietrich , Bjoern Brandenburg , Tommaso Cucinotta , "giuseppe.lipari" , Juri Lelli In-Reply-To: <1255707898.6228.463.camel@Palantir> References: <1255707324.6228.448.camel@Palantir> <1255707898.6228.463.camel@Palantir> Content-Type: text/plain; charset="UTF-8" Date: Mon, 28 Dec 2009 15:30:01 +0100 Message-ID: <1262010601.7135.113.camel@laptop> Mime-Version: 1.0 X-Mailer: Evolution 2.28.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1110 Lines: 22 On Fri, 2009-10-16 at 17:44 +0200, Raistlin wrote: > This commit introduces another new SCHED_DEADLINE related syscall. It is > called sched_wait_interval() and it has close-to-clock_nanosleep semantic. > > However, for SCHED_DEADLINE tasks, it should be the call with which each > job closes its current instance. In fact, in this case, the task is put to > sleep and, when it wakes up, the scheduler is informed that a new job > arrived, saving the overhead that usually comes with a task activation > to enforce maximum task bandwidth. The changelog suggests (and a very brief looks seems to confirm) that this code could be much smaller by using hrtimer_nanosleep(). The implementation as presented seems to only call ->wait_interval() when the timer arms, which seems like a bug, we should always call it, regardless of whether we're on a period boundary. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/