lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <1277748484-23882-1-git-send-email-ebmunson@us.ibm.com>
Date:	Mon, 28 Jun 2010 19:08:04 +0100
From:	Eric B Munson <ebmunson@...ibm.com>
To:	mingo@...e.hu
Cc:	a.p.zijlstra@...llo.nl, paulus@...ba.org, acme@...hat.com,
	linux-kernel@...r.kernel.org, linux-mm@...ck.org,
	Eric B Munson <ebmunson@...ibm.com>,
	Anton Blanchard <anton@...ba.org>
Subject: [PATCH] Add munmap events to perf

This patch adds a new software event for munmaps.  It will allows
users to profile changes to address space.  munmaps will be tracked
with mmaps.

Signed-off-by: Eric B Munson <ebmunson@...ibm.com>
Signed-off-by: Anton Blanchard <anton@...ba.org>
---
 include/linux/perf_event.h  |    6 ++++-
 kernel/perf_event.c         |   49 +++++++++++++++++++++++++++++++++++++++---
 mm/mmap.c                   |    2 +
 tools/perf/builtin-record.c |    1 +
 4 files changed, 53 insertions(+), 5 deletions(-)

diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h
index 716f99b..937dd93 100644
--- a/include/linux/perf_event.h
+++ b/include/linux/perf_event.h
@@ -215,8 +215,9 @@ struct perf_event_attr {
 				 */
 				precise_ip     :  2, /* skid constraint       */
 				mmap_data      :  1, /* non-exec mmap data    */
+				munmap         :  1, /* include munmap events */
 
-				__reserved_1   : 46;
+				__reserved_1   : 45;
 
 	union {
 		__u32		wakeup_events;	  /* wakeup every n events */
@@ -341,6 +342,7 @@ enum perf_event_type {
 	 * };
 	 */
 	PERF_RECORD_MMAP			= 1,
+	PERF_RECORD_MUNMAP			= 10,
 
 	/*
 	 * struct {
@@ -969,6 +971,8 @@ perf_sw_event(u32 event_id, u64 nr, int nmi, struct pt_regs *regs, u64 addr)
 }
 
 extern void perf_event_mmap(struct vm_area_struct *vma);
+extern void perf_event_munmap(struct vm_area_struct *vma, unsigned long start,
+				size_t len);
 extern struct perf_guest_info_callbacks *perf_guest_cbs;
 extern int perf_register_guest_info_callbacks(struct perf_guest_info_callbacks *callbacks);
 extern int perf_unregister_guest_info_callbacks(struct perf_guest_info_callbacks *callbacks);
diff --git a/kernel/perf_event.c b/kernel/perf_event.c
index 403d180..2d24e4e 100644
--- a/kernel/perf_event.c
+++ b/kernel/perf_event.c
@@ -46,6 +46,7 @@ static int perf_overcommit __read_mostly = 1;
 
 static atomic_t nr_events __read_mostly;
 static atomic_t nr_mmap_events __read_mostly;
+static atomic_t nr_munmap_events __read_mostly;
 static atomic_t nr_comm_events __read_mostly;
 static atomic_t nr_task_events __read_mostly;
 
@@ -1891,6 +1892,8 @@ static void free_event(struct perf_event *event)
 		atomic_dec(&nr_events);
 		if (event->attr.mmap || event->attr.mmap_data)
 			atomic_dec(&nr_mmap_events);
+		if (event->attr.munmap)
+			atomic_dec(&nr_munmap_events);
 		if (event->attr.comm)
 			atomic_dec(&nr_comm_events);
 		if (event->attr.task)
@@ -3491,7 +3494,8 @@ perf_event_read_event(struct perf_event *event,
 /*
  * task tracking -- fork/exit
  *
- * enabled by: attr.comm | attr.mmap | attr.mmap_data | attr.task
+ * enabled by: attr.comm | attr.mmap | attr.mmap_data | attr.munmap |
+ *	       attr.task
  */
 
 struct perf_task_event {
@@ -3542,7 +3546,7 @@ static int perf_event_task_match(struct perf_event *event)
 		return 0;
 
 	if (event->attr.comm || event->attr.mmap ||
-	    event->attr.mmap_data || event->attr.task)
+	    event->attr.mmap_data || event->attr.munmap || event->attr.task)
 		return 1;
 
 	return 0;
@@ -3583,6 +3587,7 @@ static void perf_event_task(struct task_struct *task,
 
 	if (!atomic_read(&nr_comm_events) &&
 	    !atomic_read(&nr_mmap_events) &&
+	    !atomic_read(&nr_munmap_events) &&
 	    !atomic_read(&nr_task_events))
 		return;
 
@@ -3776,9 +3781,14 @@ static int perf_event_mmap_match(struct perf_event *event,
 	if (event->cpu != -1 && event->cpu != smp_processor_id())
 		return 0;
 
-	if ((!executable && event->attr.mmap_data) ||
-	    (executable && event->attr.mmap))
+	if (mmap_event->event_id.header.type == PERF_RECORD_MMAP) {
+		if ((!executable && event->attr.mmap_data) ||
+		     (executable && event->attr.mmap))
+			return 1;
+	} else if ((mmap_event->event_id.header.type == PERF_RECORD_MUNMAP) &&
+		   event->attr.munmap) {
 		return 1;
+	}
 
 	return 0;
 }
@@ -3896,6 +3906,35 @@ void perf_event_mmap(struct vm_area_struct *vma)
 	perf_event_mmap_event(&mmap_event);
 }
 
+void perf_event_munmap(struct vm_area_struct *vma, unsigned long start,
+		       size_t len)
+{
+	struct perf_mmap_event mmap_event;
+
+	if (!atomic_read(&nr_munmap_events))
+		return;
+
+	mmap_event = (struct perf_mmap_event){
+		.vma	= vma,
+		/* .file_name */
+		/* .file_size */
+		.event_id  = {
+			.header = {
+				.type = PERF_RECORD_MUNMAP,
+				.misc = 0,
+				/* .size */
+			},
+			/* .pid */
+			/* .tid */
+			.start	= start,
+			.len	= len,
+			.pgoff	= 0,
+		},
+	};
+
+	perf_event_mmap_event(&mmap_event);
+}
+
 /*
  * IRQ throttle logging
  */
@@ -4925,6 +4964,8 @@ done:
 		atomic_inc(&nr_events);
 		if (event->attr.mmap || event->attr.mmap_data)
 			atomic_inc(&nr_mmap_events);
+		if (event->attr.munmap)
+			atomic_inc(&nr_munmap_events);
 		if (event->attr.comm)
 			atomic_inc(&nr_comm_events);
 		if (event->attr.task)
diff --git a/mm/mmap.c b/mm/mmap.c
index e38e910..cb03746 100644
--- a/mm/mmap.c
+++ b/mm/mmap.c
@@ -2082,6 +2082,8 @@ int do_munmap(struct mm_struct *mm, unsigned long start, size_t len)
 		}
 	}
 
+	perf_event_munmap(vma, start, end - start);
+
 	/*
 	 * Remove the vma's, and unmap the actual pages
 	 */
diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c
index b938796..b2018be 100644
--- a/tools/perf/builtin-record.c
+++ b/tools/perf/builtin-record.c
@@ -287,6 +287,7 @@ static void create_counter(int counter, int cpu)
 	}
 
 	attr->mmap		= track;
+	attr->munmap		= track;
 	attr->comm		= track;
 	attr->inherit		= !no_inherit;
 	if (target_pid == -1 && target_tid == -1 && !system_wide) {
-- 
1.7.0.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ