* Re: [PATCH] perf_event_open.2: Update description for mmap_data bit [not found] ` <1384291959-5329-1-git-send-email-dsahern-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> @ 2013-11-13 17:59 ` Vince Weaver [not found] ` <alpine.DEB.2.10.1311131254380.348-6xBS8L8d439fDsnSvq7Uq4Se7xf15W0s1dQoKJhdanU@public.gmane.org> 0 siblings, 1 reply; 3+ messages in thread From: Vince Weaver @ 2013-11-13 17:59 UTC (permalink / raw) To: David Ahern Cc: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w, Arnaldo Carvalho de Melo, linux-man-u79uwXL29TY76Z2rM5mHXA Here's an updated version of the patch that expands the "mmap" definition as well as that of "mmap_data". Also some manpage related formatting improvements from the original patch. I'm not really sure how signed-off-by's and other tags work when modifying a patch like this so sorry if I mixed that up. Signed-off-by: Vince Weaver <vincent.weaver-e7X0jjDqjFGHXe+LvDLADg@public.gmane.org> Cc: David Ahern <dsahern-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Cc: Arnaldo Carvalho de Melo <acme-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> Link: https://lkml.org/lkml/2013/11/11/505 --- diff --git a/man2/perf_event_open.2 b/man2/perf_event_open.2 index 1889689..5ba1a42 100644 --- a/man2/perf_event_open.2 +++ b/man2/perf_event_open.2 @@ -805,7 +805,16 @@ If set, don't count when the CPU is idle. .IR "mmap" The .I mmap -bit enables recording of exec mmap events. +bit enables generation of +.I PERF_RECORD_MMAP +samples for every +.BR mmap (2) +call that has +.I PROT_EXEC +set. +This allows tools to notice new executable code being mapped into +a program (dynamic shared libraries for example) +so that addresses can be mapped back to the original code. .TP .IR "comm" The @@ -884,8 +893,14 @@ See also .IR "mmap_data" " (Since Linux 2.6.36)" The counterpart of the .I mmap -field, but enables including data mmap events -in the ring-buffer. +field. +This enables generation of +.I PERF_RECORD_MMAP +samples for +.BR mmap (2) +calls that do not have +.I PROT_EXEC +set (for example data and SysV shared memory). .TP .IR "sample_id_all" " (Since Linux 2.6.38)" If set, then TID, TIME, ID, STREAM_ID, and CPU can -- To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 3+ messages in thread
[parent not found: <alpine.DEB.2.10.1311131254380.348-6xBS8L8d439fDsnSvq7Uq4Se7xf15W0s1dQoKJhdanU@public.gmane.org>]
* Re: [PATCH] perf_event_open.2: Update description for mmap_data bit [not found] ` <alpine.DEB.2.10.1311131254380.348-6xBS8L8d439fDsnSvq7Uq4Se7xf15W0s1dQoKJhdanU@public.gmane.org> @ 2013-11-15 16:36 ` David Ahern 2013-12-12 3:39 ` Michael Kerrisk (man-pages) 1 sibling, 0 replies; 3+ messages in thread From: David Ahern @ 2013-11-15 16:36 UTC (permalink / raw) To: Vince Weaver Cc: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w, Arnaldo Carvalho de Melo, linux-man-u79uwXL29TY76Z2rM5mHXA On 11/13/13, 10:59 AM, Vince Weaver wrote: > > Here's an updated version of the patch that expands the "mmap" definition > as well as that of "mmap_data". Also some manpage related formatting > improvements from the original patch. > > I'm not really sure how signed-off-by's and other tags work when modifying > a patch like this so sorry if I mixed that up. > I just want the documentation updated as we find holes. I'm fine with your change. David -- To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] perf_event_open.2: Update description for mmap_data bit [not found] ` <alpine.DEB.2.10.1311131254380.348-6xBS8L8d439fDsnSvq7Uq4Se7xf15W0s1dQoKJhdanU@public.gmane.org> 2013-11-15 16:36 ` David Ahern @ 2013-12-12 3:39 ` Michael Kerrisk (man-pages) 1 sibling, 0 replies; 3+ messages in thread From: Michael Kerrisk (man-pages) @ 2013-12-12 3:39 UTC (permalink / raw) To: Vince Weaver, David Ahern Cc: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w, Arnaldo Carvalho de Melo, linux-man-u79uwXL29TY76Z2rM5mHXA On 11/14/13 06:59, Vince Weaver wrote: > > Here's an updated version of the patch that expands the "mmap" definition > as well as that of "mmap_data". Also some manpage related formatting > improvements from the original patch. > > I'm not really sure how signed-off-by's and other tags work when modifying > a patch like this so sorry if I mixed that up. > > Signed-off-by: Vince Weaver <vincent.weaver-e7X0jjDqjFGHXe+LvDLADg@public.gmane.org> > Cc: David Ahern <dsahern-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > Cc: Arnaldo Carvalho de Melo <acme-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> > Link: https://lkml.org/lkml/2013/11/11/505 Thanks Vince (and David). I've applied, and added an "Acked-by:" from David, since he said he was fine with your revision. Cheers, Michael > --- > > diff --git a/man2/perf_event_open.2 b/man2/perf_event_open.2 > index 1889689..5ba1a42 100644 > --- a/man2/perf_event_open.2 > +++ b/man2/perf_event_open.2 > @@ -805,7 +805,16 @@ If set, don't count when the CPU is idle. > .IR "mmap" > The > .I mmap > -bit enables recording of exec mmap events. > +bit enables generation of > +.I PERF_RECORD_MMAP > +samples for every > +.BR mmap (2) > +call that has > +.I PROT_EXEC > +set. > +This allows tools to notice new executable code being mapped into > +a program (dynamic shared libraries for example) > +so that addresses can be mapped back to the original code. > .TP > .IR "comm" > The > @@ -884,8 +893,14 @@ See also > .IR "mmap_data" " (Since Linux 2.6.36)" > The counterpart of the > .I mmap > -field, but enables including data mmap events > -in the ring-buffer. > +field. > +This enables generation of > +.I PERF_RECORD_MMAP > +samples for > +.BR mmap (2) > +calls that do not have > +.I PROT_EXEC > +set (for example data and SysV shared memory). > .TP > .IR "sample_id_all" " (Since Linux 2.6.38)" > If set, then TID, TIME, ID, STREAM_ID, and CPU can > -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ -- To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-12-12 3:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1384291959-5329-1-git-send-email-dsahern@gmail.com>
[not found] ` <1384291959-5329-1-git-send-email-dsahern-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2013-11-13 17:59 ` [PATCH] perf_event_open.2: Update description for mmap_data bit Vince Weaver
[not found] ` <alpine.DEB.2.10.1311131254380.348-6xBS8L8d439fDsnSvq7Uq4Se7xf15W0s1dQoKJhdanU@public.gmane.org>
2013-11-15 16:36 ` David Ahern
2013-12-12 3:39 ` Michael Kerrisk (man-pages)
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).