On Sun, Sep 13, 2020 at 10:40 PM Namhyung Kim <namhy...@kernel.org> wrote: > > On Mon, Sep 14, 2020 at 6:03 AM Jiri Olsa <jo...@kernel.org> wrote: > > > > Add new version of mmap event. The MMAP3 record is an > > augmented version of MMAP2, it adds build id value to > > identify the exact binary object behind memory map: > > > > struct { > > struct perf_event_header header; > > > > u32 pid, tid; > > u64 addr; > > u64 len; > > u64 pgoff; > > u32 maj; > > u32 min; > > u64 ino; > > u64 ino_generation; > > u32 prot, flags; > > u32 reserved;
I guess we need reserved _after_ buildid, no? > > u8 buildid[20]; > > Do we need maj, min, ino, ino_generation for mmap3 event? > I think they are to compare binaries, then we can do it with > build-id (and I think it'd be better).. +1 we shouldn't need maj, min, etc. Thanks, Song [...]