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 <[email protected]> > > > > 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 <[email protected]> > > Cc: Adrian Hunter <[email protected]> > > Cc: Andi Kleen <[email protected]> > > Cc: David Ahern <[email protected]> > > Cc: Namhyung Kim <[email protected]> > > Cc: Peter Zijlstra <[email protected]> > > Cc: Stephane Eranian <[email protected]> > > Link: > > http://lkml.kernel.org/r/[email protected] > > [ 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
