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

From: Ingo Molnar
Date: Wed Sep 25 2013 - 07:26:57 EST



* Wei Yang <weiyang@xxxxxxxxxxxxxxxxxx> 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) {

> >- printf(" #%2d / %2d: %14.2lf nsecs/op [val: %016lx]\n",
> >+ printf(" #%2d / %2d: %14.2lf nsecs/op [val: %016"PRIu64"]\n",

Acked-by: Ingo Molnar <mingo@xxxxxxxxxx>

Thanks,

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/