Re: [PATCH] perf: fix two warning in bench/numa

2013-10-29 Thread Wei Yang
Dear maintainer, Is this one accepted or droped? Or I missed the right person? On Sun, Sep 22, 2013 at 04:49:24PM +0800, Wei Yang wrote: >There two warnings in bench/numa, when buiding this on 32-bit machine. > >The warning output is attached: > >bench/numa.c:1113:20: error: comparison between s

Re: [PATCH] perf: fix two warning in bench/numa

2013-09-25 Thread Wei Yang
On Wed, Sep 25, 2013 at 01:26:29PM +0200, Ingo Molnar wrote: > >* Wei Yang wrote: > >> Hi, all >> >> Any comments on this one? >> >> These two warnings will break the compile of perf under 32-bit machine. > >fix looks good to me. Thanks for your comments :-) > >> >- if (diff

Re: [PATCH] perf: fix two warning in bench/numa

2013-09-25 Thread Ingo Molnar
* Wei Yang wrote: > Hi, all > > Any comments on this one? > > These two warnings will break the compile of perf under 32-bit machine. fix looks good to me. > >-if (diff.tv_sec >= g->p.nr_secs) { > >+if (diff.tv_sec >= (time_t)g->p.nr_secs) { > >-

Re: [PATCH] perf: fix two warning in bench/numa

2013-09-25 Thread Wei Yang
Hi, all Any comments on this one? These two warnings will break the compile of perf under 32-bit machine. On Sun, Sep 22, 2013 at 04:49:24PM +0800, Wei Yang wrote: >There two warnings in bench/numa, when buiding this on 32-bit machine. > >The warning output is attached: > >bench/numa.c:1113:20

[PATCH] perf: fix two warning in bench/numa

2013-09-22 Thread Wei Yang
There two warnings in bench/numa, when buiding this on 32-bit machine. The warning output is attached: bench/numa.c:1113:20: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] bench/numa.c:1161:6: error: format ‘%lx’ expects argument of t'long unsigned int