From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751644AbcBWMtZ (ORCPT ); Tue, 23 Feb 2016 07:49:25 -0500 Received: from bombadil.infradead.org ([198.137.202.9]:36354 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751238AbcBWMtY (ORCPT ); Tue, 23 Feb 2016 07:49:24 -0500 Date: Tue, 23 Feb 2016 13:49:15 +0100 From: Peter Zijlstra To: Steven Rostedt Cc: linux-kernel@vger.kernel.org, Ingo Molnar , Thomas Gleixner , Juri Lelli , Clark Williams , Andrew Morton Subject: Re: [PATCH 4/4] tracing: Add __print_ns_to_secs() and __print_ns_without_secs() helpers Message-ID: <20160223124915.GQ6357@twins.programming.kicks-ass.net> References: <20160222212649.485719441@goodmis.org> <20160222212825.900322488@goodmis.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160222212825.900322488@goodmis.org> User-Agent: Mutt/1.5.21 (2012-12-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 22, 2016 at 04:26:53PM -0500, Steven Rostedt wrote: > From: "Steven Rostedt (Red Hat)" > > To have nanosecond output displayed in a more human readable format, its > nicer to convert it to a seconds format (XXX.YYYYYYYYY). The problem is that > to do so, the numbers must be divided by NSEC_PER_SEC, and moded too. But as > these numbers are 64 bit, this can not be done simply with '/' and '%' > operators, but must use do_div() instead. Would not div_[us]64_rem() make more sense? It would typically result in just the one division, instead of two.