Messages in this thread | | | Date | Mon, 9 Dec 2013 14:35:29 +0100 | From | Jiri Olsa <> | Subject | Re: [PATCH 2/4] perf tools: Fix max mmap_pages |
| |
On Mon, Dec 09, 2013 at 03:18:38PM +0200, Adrian Hunter wrote: > 'SIZE_MAX / page_size' is an upper limit for > the maximum number of mmap pages, not a lower > limit. Change the condition accordingly. > > Signed-off-by: Adrian Hunter <adrian.hunter@intel.com> > --- > tools/perf/util/evlist.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c > index 4d0945c..98ec96b 100644 > --- a/tools/perf/util/evlist.c > +++ b/tools/perf/util/evlist.c > @@ -754,7 +754,7 @@ int perf_evlist__parse_mmap_pages(const struct option *opt, const char *str, > unsigned long max = UINT_MAX; > long pages; > > - if (max < SIZE_MAX / page_size) > + if (max > SIZE_MAX / page_size) > max = SIZE_MAX / page_size;
I agree with the change.. but it looks like the check is superfluos, maybe we want to define max as:
unsigned long max = SIZE_MAX / page_size
?
jirka > > pages = parse_pages_arg(str, 1, max); > -- > 1.7.11.7 >
| |