From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752878AbeCPORZ (ORCPT ); Fri, 16 Mar 2018 10:17:25 -0400 Received: from mail.kernel.org ([198.145.29.99]:45386 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752317AbeCPORX (ORCPT ); Fri, 16 Mar 2018 10:17:23 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 3117E20855 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=frederic@kernel.org Date: Fri, 16 Mar 2018 15:17:20 +0100 From: Frederic Weisbecker To: "Rafael J. Wysocki" Cc: "Rafael J. Wysocki" , Peter Zijlstra , Linux PM , Frederic Weisbecker , Thomas Gleixner , Paul McKenney , Thomas Ilsche , Doug Smythies , Rik van Riel , Aubrey Li , Mike Galbraith , LKML Subject: Re: [RFT][PATCH v4 3/7] sched: idle: Do not stop the tick before cpuidle_idle_call() Message-ID: <20180316141719.GB20981@lerouge> References: <2352117.3UUoYAu18A@aspire.rjw.lan> <3550231.xNMJN5JcLx@aspire.rjw.lan> <20180315181858.GA14983@lerouge> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Mar 15, 2018 at 10:12:57PM +0100, Rafael J. Wysocki wrote: > On Thu, Mar 15, 2018 at 9:41 PM, Rafael J. Wysocki wrote: > > On Thu, Mar 15, 2018 at 7:19 PM, Frederic Weisbecker > > wrote: > >> On Mon, Mar 12, 2018 at 10:53:25AM +0100, Rafael J. Wysocki wrote: > >>> From: Rafael J. Wysocki > >>> > >>> Make cpuidle_idle_call() decide whether or not to stop the tick. > >>> > >>> First, the cpuidle_enter_s2idle() path deals with the tick (and with > >>> the entire timekeeping for that matter) by itself and it doesn't need > >>> the tick to be stopped beforehand. > >> > >> Not sure you meant timekeeping either :) > > > > Yeah, I meant nohz. > > Well, not really. :-) > > It is the entire timekeeping this time, as it can be suspended > entirely in that code path. Fair point. Thanks!