Em Thu, Jun 25, 2015 at 10:38:41AM +0200, Jiri Olsa escreveu: > On Tue, Jun 23, 2015 at 06:47:26PM -0300, Arnaldo Carvalho de Melo wrote: > > From: Jiri Olsa <jo...@kernel.org> > > > > We need to store command names with the pid. Changing map entries to be > > a struct holding pid. Process name is coming in shortly. > > > > Signed-off-by: Jiri Olsa <jo...@kernel.org> > > Cc: Adrian Hunter <adrian.hun...@intel.com> > > Cc: Andi Kleen <a...@linux.intel.com> > > Cc: David Ahern <dsah...@gmail.com> > > Cc: Namhyung Kim <namhy...@kernel.org> > > Cc: Peter Zijlstra <a.p.zijls...@chello.nl> > > Cc: Stephane Eranian <eran...@google.com> > > Link: > > http://lkml.kernel.org/r/1435012588-9007-2-git-send-email-jo...@kernel.org > > [ Split providing the set/get accessors from transforming the entries > > structs ] > > the comment in [] is wrong.. > > and you missed following hunk: > > --- > static struct thread_map *thread_map__realloc(struct thread_map *map, int nr) > { > - size_t size = sizeof(*map) + sizeof(pid_t) * nr; > + size_t size = sizeof(*map) + sizeof(struct thread_map_data) * nr; > --- > > shouldn't be fatal, it probably has the same or biggger size ATM > I'll include the fix in the next version
Yeah, sorry about that, should have noticed that, but as you said, code wise it is harmless. - Arnaldo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
