Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752759AbcLBFS1 (ORCPT ); Fri, 2 Dec 2016 00:18:27 -0500 Received: from mail1.windriver.com ([147.11.146.13]:63658 "EHLO mail1.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752144AbcLBFS0 (ORCPT ); Fri, 2 Dec 2016 00:18:26 -0500 Message-ID: <58410402.506@windriver.com> Date: Fri, 2 Dec 2016 13:17:54 +0800 From: yjin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 MIME-Version: 1.0 To: Pan Xinhui , Balbir Singh CC: Michael Ellerman , Benjamin Herrenschmidt , Paul Mackerras , , , "open list:LINUX FOR POWERPC (32-BIT AND 64-BIT)" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] powerpc: cputime: fix a compile warning References: <1479704219-21403-1-git-send-email-yanjiang.jin@windriver.com> <874m2n7yrf.fsf@concordia.ellerman.id.au> <5840FA25.3010009@windriver.com> <546ba7c8-999c-b393-b4b3-ee8602b724fd@linux.vnet.ibm.com> In-Reply-To: <546ba7c8-999c-b393-b4b3-ee8602b724fd@linux.vnet.ibm.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2884 Lines: 85 On 2016年12月02日 13:15, Pan Xinhui wrote: > > > 在 2016/12/2 12:35, yjin 写道: >> >> On 2016年12月02日 12:22, Balbir Singh wrote: >>> On Fri, Dec 2, 2016 at 3:15 PM, Michael Ellerman >>> wrote: >>>> yanjiang.jin@windriver.com writes: >>>> >>>>> diff --git a/arch/powerpc/include/asm/cputime.h >>>>> b/arch/powerpc/include/asm/cputime.h >>>>> index 4f60db0..4423e97 100644 >>>>> --- a/arch/powerpc/include/asm/cputime.h >>>>> +++ b/arch/powerpc/include/asm/cputime.h >>>>> @@ -228,7 +228,8 @@ static inline cputime_t >>>>> clock_t_to_cputime(const unsigned long clk) >>>>> return (__force cputime_t) ct; >>>>> } >>>>> >>>>> -#define cputime64_to_clock_t(ct) cputime_to_clock_t((cputime_t)(ct)) >>>>> +#define cputime64_to_clock_t(ct) \ >>>>> + (__force u64)(cputime_to_clock_t((cputime_t)(ct))) >>>> Given the name of the function is "cputime64 to clock_t", surely we >>>> should be returning a clock_t ? >>> Please fix it in cpuacct.c Also check out git commit >>> 527b0a76f41d062381adbb55c8eb61e32cb0bfc9 >>> sched/cpuacct: Avoid %lld seq_printf warning >> >> Hi Balbir, >> >> Where can I find this commit? >> > hello, > it is in next tree. :) Got it. Thanks! Regards! Yanjiang > > commit 527b0a76f41d062381adbb55c8eb61e32cb0bfc9 > Author: Martin Schwidefsky > Date: Fri Nov 11 15:27:49 2016 +0100 > > sched/cpuacct: Avoid %lld seq_printf warning > For s390 kernel builds I keep getting this warning: > kernel/sched/cpuacct.c: In function 'cpuacct_stats_show': > kernel/sched/cpuacct.c:298:25: warning: format '%lld' expects > argument of type 'long long int', but argument 4 has type 'clock_t > {aka long int}' [-Wformat=] > seq_printf(sf, "%s %lld\n", > Silence the warning by adding an explicit cast. > Signed-off-by: Martin Schwidefsky > Signed-off-by: Peter Zijlstra (Intel) > Cc: Linus Torvalds > Cc: Peter Zijlstra > Cc: Thomas Gleixner > Link: > http://lkml.kernel.org/r/20161111142749.6545-1-schwidefsky@de.ibm.com > Signed-off-by: Ingo Molnar > > diff --git a/kernel/sched/cpuacct.c b/kernel/sched/cpuacct.c > index bc0b309c..9add206 100644 > --- a/kernel/sched/cpuacct.c > +++ b/kernel/sched/cpuacct.c > @@ -297,7 +297,7 @@ static int cpuacct_stats_show(struct seq_file *sf, > void *v) > for (stat = 0; stat < CPUACCT_STAT_NSTATS; stat++) { > seq_printf(sf, "%s %lld\n", > cpuacct_stat_desc[stat], > - cputime64_to_clock_t(val[stat])); > + (long long)cputime64_to_clock_t(val[stat])); > } > > return 0; > >> Thanks! >> Yanjiang >>> >>> Balbir >> >