[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110121132011.GA15066@ghostprotocols.net>
Date: Fri, 21 Jan 2011 11:20:11 -0200
From: Arnaldo Carvalho de Melo <acme@...stprotocols.net>
To: Denis Kirjanov <dkirjanov@...nel.org>, Ingo Molnar <mingo@...e.hu>,
linux-kernel@...r.kernel.org, a.p.zijlstra@...llo.nl,
paulus@...ba.org
Subject: Re: [PATCH] perftools: Fix build error
Em Fri, Jan 21, 2011 at 04:57:21PM +0800, Han Pingtian escreveu:
> On Thu, Jan 20, 2011 at 03:22:18PM -0200, Arnaldo Carvalho de Melo wrote:
> > Em Thu, Jan 20, 2011 at 06:44:38PM +0300, Denis Kirjanov escreveu:
> > > On 01/20/2011 02:47 PM, Ingo Molnar wrote:
> > > > Wouldnt %Lu solve this in a cleaner way?
> > > I think that cleaner way is to use PRIu64.
> > Fuglier, definetely, but indeed, more portable, as I noticed when trying
> > to build the tools on a RHEL4/Fedora3 machine, as there are people using
> > such old beasts with modern kernels.
> > I'll take the patch for perf/urgent, as it prevents building in .38,
> > please consider going thru tools/perf and patching the other %Ld and
> > %Lu instances :-)
> There are also some %Lx in code, shall we change them with PRIx64?
Right, one patch doing the conversion for the whole tools/perf tree
would be better.
Han, if you can after the patch make sure it builds in an RHEL4 machine,
that would be great,
Thanks,
- Arnaldo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists