From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ozlabs.org (ozlabs.org [IPv6:2401:3900:2:1::2]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3t8RX85Xy7zDvHw for ; Thu, 3 Nov 2016 12:02:32 +1100 (AEDT) Date: Thu, 3 Nov 2016 11:50:38 +1100 From: Paul Mackerras To: Stanislaw Gruszka Cc: linux-kernel@vger.kernel.org, Ingo Molnar , Peter Zijlstra , Frederic Weisbecker , Benjamin Herrenschmidt , Michael Neuling , linuxppc-dev@lists.ozlabs.org, Martin Schwidefsky , Heiko Carstens , linux-s390@vger.kernel.org Subject: Re: [PATCH 1/4] cputime/powerpc: remove cputime_last_delta global variable Message-ID: <20161103005038.GB4764@fergus.ozlabs.ibm.com> References: <1477917389-11341-1-git-send-email-sgruszka@redhat.com> <1477917389-11341-2-git-send-email-sgruszka@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1477917389-11341-2-git-send-email-sgruszka@redhat.com> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, Oct 31, 2016 at 01:36:26PM +0100, Stanislaw Gruszka wrote: > Since commit: > > cf9efce0ce313 ("powerpc: Account time using timebase rather than PURR") > > cputime_last_delta is not initialized to other value than 0, hence it's > not used except zero check and cputime_to_scaled() just returns > the argument. > > Signed-off-by: Stanislaw Gruszka Yes, I should have removed them in that commit. We don't want to do any scaling in the places where cputime_to_scaled() is used. Acked-by: Paul Mackerras