From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Gleixner Subject: Re: [PATCH v7 02/11] task_isolation: add initial support Date: Thu, 1 Oct 2015 14:18:42 +0200 (CEST) Message-ID: References: <1443453446-7827-1-git-send-email-cmetcalf@ezchip.com> <1443453446-7827-3-git-send-email-cmetcalf@ezchip.com> <20151001121414.GB3432@lerouge> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Return-path: In-Reply-To: <20151001121414.GB3432@lerouge> Sender: linux-kernel-owner@vger.kernel.org To: Frederic Weisbecker Cc: Chris Metcalf , Gilad Ben Yossef , Steven Rostedt , Ingo Molnar , Peter Zijlstra , Andrew Morton , Rik van Riel , Tejun Heo , "Paul E. McKenney" , Christoph Lameter , Viresh Kumar , Catalin Marinas , Will Deacon , Andy Lutomirski , linux-doc@vger.kernel.org, linux-api@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-api@vger.kernel.org On Thu, 1 Oct 2015, Frederic Weisbecker wrote: > On Mon, Sep 28, 2015 at 11:17:17AM -0400, Chris Metcalf wrote: > > + > > + while (READ_ONCE(dev->next_event.tv64) != KTIME_MAX) { > > You should add a function in tick-sched.c to get the next tick. This > is supposed to be a private field. Just to make it clear. Neither the above nor a similar check in tick-sched.c is going to happen. This busy waiting is just horrible. Get your act together and solve the problems at the root and do not inflict your quick and dirty 'solutions' on us. Thanks, tglx