* + debugging-keep-track-of-page-owners-fix.patch added to -mm tree
@ 2012-12-07 22:26 akpm
0 siblings, 0 replies; 2+ messages in thread
From: akpm @ 2012-12-07 22:26 UTC (permalink / raw)
To: mm-commits; +Cc: dave, apw, cl, lauraa, mel, mingo
The patch titled
Subject: debugging-keep-track-of-page-owners-fix
has been added to the -mm tree. Its filename is
debugging-keep-track-of-page-owners-fix.patch
Before you just go and hit "reply", please:
a) Consider who else should be cc'ed
b) Prefer to cc a suitable mailing list as well
c) Ideally: find the original patch on the mailing list and do a
reply-to-all to that, adding suitable additional cc's
*** Remember to use Documentation/SubmitChecklist when testing your code ***
The -mm tree is included into linux-next and is updated
there every 3-4 working days
------------------------------------------------------
From: Dave Hansen <dave@linux.vnet.ibm.com>
Subject: debugging-keep-track-of-page-owners-fix
Use linux/stacktrace.h rather than hand-coding the stack tracing
Signed-off-by: Dave Hansen <dave@linux.vnet.ibm.com>
Cc: Mel Gorman <mel@csn.ul.ie>
Cc: Andy Whitcroft <apw@shadowen.org>
Cc: Mel Gorman <mel@csn.ul.ie>
Cc: Christoph Lameter <cl@linux-foundation.org>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Laura Abbott <lauraa@codeaurora.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---
include/linux/mm_types.h | 4 +-
include/linux/stacktrace.h | 3 +
kernel/stacktrace.c | 23 ++++++++++++
lib/Kconfig.debug | 1
mm/page_alloc.c | 65 ++++-------------------------------
mm/pageowner.c | 27 ++++----------
mm/vmstat.c | 5 ++
7 files changed, 51 insertions(+), 77 deletions(-)
diff -puN include/linux/mm_types.h~debugging-keep-track-of-page-owners-fix include/linux/mm_types.h
--- a/include/linux/mm_types.h~debugging-keep-track-of-page-owners-fix
+++ a/include/linux/mm_types.h
@@ -8,6 +8,7 @@
#include <linux/spinlock.h>
#include <linux/rbtree.h>
#include <linux/rwsem.h>
+#include <linux/stacktrace.h>
#include <linux/completion.h>
#include <linux/cpumask.h>
#include <linux/page-debug-flags.h>
@@ -180,7 +181,8 @@ struct page {
#ifdef CONFIG_PAGE_OWNER
int order;
unsigned int gfp_mask;
- unsigned long trace[8];
+ struct stack_trace trace;
+ unsigned long trace_entries[8];
#endif
}
/*
diff -puN include/linux/stacktrace.h~debugging-keep-track-of-page-owners-fix include/linux/stacktrace.h
--- a/include/linux/stacktrace.h~debugging-keep-track-of-page-owners-fix
+++ a/include/linux/stacktrace.h
@@ -20,6 +20,8 @@ extern void save_stack_trace_tsk(struct
struct stack_trace *trace);
extern void print_stack_trace(struct stack_trace *trace, int spaces);
+extern int snprint_stack_trace(char *buf, int buf_len,
+ struct stack_trace *trace, int spaces);
#ifdef CONFIG_USER_STACKTRACE_SUPPORT
extern void save_stack_trace_user(struct stack_trace *trace);
@@ -32,6 +34,7 @@ extern void save_stack_trace_user(struct
# define save_stack_trace_tsk(tsk, trace) do { } while (0)
# define save_stack_trace_user(trace) do { } while (0)
# define print_stack_trace(trace, spaces) do { } while (0)
+# define snprint_stack_trace(buf, len, trace, spaces) do { } while (0)
#endif
#endif
diff -puN kernel/stacktrace.c~debugging-keep-track-of-page-owners-fix kernel/stacktrace.c
--- a/kernel/stacktrace.c~debugging-keep-track-of-page-owners-fix
+++ a/kernel/stacktrace.c
@@ -11,6 +11,29 @@
#include <linux/kallsyms.h>
#include <linux/stacktrace.h>
+int snprint_stack_trace(char *buf, int buf_len, struct stack_trace *trace,
+ int spaces)
+{
+ int ret = 0;
+ int i;
+
+ if (WARN_ON(!trace->entries))
+ return 0;
+
+ for (i = 0; i < trace->nr_entries; i++) {
+ unsigned long ip = trace->entries[i];
+ int printed = snprintf(buf, buf_len, "%*c[<%p>] %pS\n",
+ 1 + spaces, ' ',
+ (void *) ip, (void *) ip);
+ buf_len -= printed;
+ ret += printed;
+ buf += printed;
+ }
+
+ return ret;
+}
+EXPORT_SYMBOL_GPL(snprint_stack_trace);
+
void print_stack_trace(struct stack_trace *trace, int spaces)
{
int i;
diff -puN lib/Kconfig.debug~debugging-keep-track-of-page-owners-fix lib/Kconfig.debug
--- a/lib/Kconfig.debug~debugging-keep-track-of-page-owners-fix
+++ a/lib/Kconfig.debug
@@ -103,6 +103,7 @@ config PAGE_OWNER
bool "Track page owner"
depends on DEBUG_KERNEL
select DEBUG_FS
+ select STACKTRACE
help
This keeps track of what call chain is the owner of a page, may
help to find bare alloc_page(s) leaks. Eats a fair amount of memory.
diff -puN mm/page_alloc.c~debugging-keep-track-of-page-owners-fix mm/page_alloc.c
--- a/mm/page_alloc.c~debugging-keep-track-of-page-owners-fix
+++ a/mm/page_alloc.c
@@ -2267,62 +2267,21 @@ __perform_reclaim(gfp_t gfp_mask, unsign
return progress;
}
-#ifdef CONFIG_PAGE_OWNER
-static inline int valid_stack_ptr(struct thread_info *tinfo, void *p)
-{
- return p > (void *)tinfo &&
- p < (void *)tinfo + THREAD_SIZE - 3;
-}
-
-static inline void __stack_trace(struct page *page, unsigned long *stack,
- unsigned long bp)
-{
- int i = 0;
- unsigned long addr;
- struct thread_info *tinfo = (struct thread_info *)
- ((unsigned long)stack & (~(THREAD_SIZE - 1)));
-
- memset(page->trace, 0, sizeof(long) * 8);
-
-#ifdef CONFIG_FRAME_POINTER
- if (bp) {
- while (valid_stack_ptr(tinfo, (void *)bp)) {
- addr = *(unsigned long *)(bp + sizeof(long));
- page->trace[i] = addr;
- if (++i >= 8)
- break;
- bp = *(unsigned long *)bp;
- }
- return;
- }
-#endif /* CONFIG_FRAME_POINTER */
- while (valid_stack_ptr(tinfo, stack)) {
- addr = *stack++;
- if (__kernel_text_address(addr)) {
- page->trace[i] = addr;
- if (++i >= 8)
- break;
- }
- }
-}
-
static void set_page_owner(struct page *page, unsigned int order,
unsigned int gfp_mask)
{
- unsigned long address;
- unsigned long bp = 0;
-#ifdef CONFIG_X86_64
- asm ("movq %%rbp, %0" : "=r" (bp) : );
-#endif
-#ifdef CONFIG_X86_32
- asm ("movl %%ebp, %0" : "=r" (bp) : );
-#endif
+#ifdef CONFIG_PAGE_OWNER
+ struct stack_trace *trace = &page->trace;
+ trace->nr_entries = 0;
+ trace->max_entries = ARRAY_SIZE(page->trace_entries);
+ trace->entries = &page->trace_entries[0];
+ trace->skip = 3;
+ save_stack_trace(&page->trace);
+
page->order = (int) order;
page->gfp_mask = gfp_mask;
- __stack_trace(page, &address, bp);
-}
#endif /* CONFIG_PAGE_OWNER */
-
+}
/* The really slow allocator path where we enter direct reclaim */
static inline struct page *
@@ -2359,10 +2318,8 @@ retry:
goto retry;
}
-#ifdef CONFIG_PAGE_OWNER
if (page)
set_page_owner(page, order, gfp_mask);
-#endif
return page;
}
@@ -2672,10 +2629,8 @@ nopage:
warn_alloc_failed(gfp_mask, order, NULL);
return page;
got_pg:
-#ifdef CONFIG_PAGE_OWNER
if (page)
set_page_owner(page, order, gfp_mask);
-#endif
if (kmemcheck_enabled)
kmemcheck_pagealloc_alloc(page, order, gfp_mask);
@@ -2758,10 +2713,8 @@ out:
memcg_kmem_commit_charge(page, memcg, order);
-#ifdef CONFIG_PAGE_OWNER
if (page)
set_page_owner(page, order, gfp_mask);
-#endif
return page;
}
diff -puN mm/pageowner.c~debugging-keep-track-of-page-owners-fix mm/pageowner.c
--- a/mm/pageowner.c~debugging-keep-track-of-page-owners-fix
+++ a/mm/pageowner.c
@@ -26,12 +26,8 @@ read_page_owner(struct file *file, char
{
unsigned long pfn;
struct page *page;
- char *kbuf, *modname;
- const char *symname;
+ char *kbuf;
int ret = 0;
- char namebuf[128];
- unsigned long offset = 0, symsize;
- int i;
ssize_t num_written = 0;
int blocktype = 0, pagetype = 0;
@@ -67,7 +63,7 @@ read_page_owner(struct file *file, char
pfn);
/* Stop search if page is allocated and has trace info */
- if (page->order >= 0 && page->trace[0]) {
+ if (page->order >= 0 && page->trace.nr_entries) {
//intk("stopped search at pfn: %ld\n", pfn);
break;
}
@@ -126,20 +122,13 @@ read_page_owner(struct file *file, char
num_written = ret;
- for (i = 0; i < 8; i++) {
- if (!page->trace[i])
- break;
- symname = kallsyms_lookup(page->trace[i], &symsize, &offset,
- &modname, namebuf);
- ret = snprintf(kbuf + num_written, count - num_written,
- "[0x%lx] %s+%lu\n",
- page->trace[i], namebuf, offset);
- if (ret >= count - num_written) {
- ret = -ENOMEM;
- goto out;
- }
- num_written += ret;
+ ret = snprint_stack_trace(kbuf + num_written, count - num_written,
+ &page->trace, 0);
+ if (ret >= count - num_written) {
+ ret = -ENOMEM;
+ goto out;
}
+ num_written += ret;
ret = snprintf(kbuf + num_written, count - num_written, "\n");
if (ret >= count - num_written) {
diff -puN mm/vmstat.c~debugging-keep-track-of-page-owners-fix mm/vmstat.c
--- a/mm/vmstat.c~debugging-keep-track-of-page-owners-fix
+++ a/mm/vmstat.c
@@ -985,7 +985,10 @@ static void pagetypeinfo_showmixedcount_
pagetype = allocflags_to_migratetype(page->gfp_mask);
if (pagetype != mtype) {
- count[mtype]++;
+ if (is_migrate_cma(pagetype))
+ count[MIGRATE_MOVABLE]++;
+ else
+ count[mtype]++;
break;
}
_
Patches currently in -mm which might be from dave@linux.vnet.ibm.com are
memory-hotplug-update-mce_bad_pages-when-removing-the-memory.patch
memory-hotplug-update-mce_bad_pages-when-removing-the-memory-fix.patch
memory-hotplug-auto-offline-page_cgroup-when-onlining-memory-block-failed.patch
memory-hotplug-fix-nr_free_pages-mismatch.patch
memory-hotplug-mm-sparsec-clear-the-memory-to-store-struct-page.patch
memory-hotplug-allocate-zones-pcp-before-onlining-pages.patch
mm-cma-skip-watermarks-check-for-already-isolated-blocks-in-split_free_page-fix-fix.patch
mm-provide-more-accurate-estimation-of-pages-occupied-by-memmap.patch
mm-provide-more-accurate-estimation-of-pages-occupied-by-memmap-fix.patch
drop_caches-add-some-documentation-and-info-messsge.patch
debugging-keep-track-of-page-owners-fix.patch
^ permalink raw reply [flat|nested] 2+ messages in thread
* + debugging-keep-track-of-page-owners-fix.patch added to -mm tree
@ 2014-08-25 22:52 akpm
0 siblings, 0 replies; 2+ messages in thread
From: akpm @ 2014-08-25 22:52 UTC (permalink / raw)
To: akpm, mm-commits
The patch titled
Subject: debugging-keep-track-of-page-owners-fix
has been added to the -mm tree. Its filename is
debugging-keep-track-of-page-owners-fix.patch
This patch should soon appear at
http://ozlabs.org/~akpm/mmots/broken-out/debugging-keep-track-of-page-owners-fix.patch
and later at
http://ozlabs.org/~akpm/mmotm/broken-out/debugging-keep-track-of-page-owners-fix.patch
Before you just go and hit "reply", please:
a) Consider who else should be cc'ed
b) Prefer to cc a suitable mailing list as well
c) Ideally: find the original patch on the mailing list and do a
reply-to-all to that, adding suitable additional cc's
*** Remember to use Documentation/SubmitChecklist when testing your code ***
The -mm tree is included into linux-next and is updated
there every 3-4 working days
------------------------------------------------------
From: Andrew Morton <akpm@linux-foundation.org>
Subject: debugging-keep-track-of-page-owners-fix
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---
mm/pageowner.c | 2 +-
mm/vmstat.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff -puN mm/pageowner.c~debugging-keep-track-of-page-owners-fix mm/pageowner.c
--- a/mm/pageowner.c~debugging-keep-track-of-page-owners-fix
+++ a/mm/pageowner.c
@@ -95,7 +95,7 @@ read_page_owner(struct file *file, char
/* Print information relevant to grouping pages by mobility */
blocktype = get_pageblock_migratetype(page);
- pagetype = allocflags_to_migratetype(page->gfp_mask);
+ pagetype = gfpflags_to_migratetype(page->gfp_mask);
ret += snprintf(kbuf+ret, count-ret,
"PFN %lu Block %lu type %d %s "
"Flags %s%s%s%s%s%s%s%s%s%s%s%s\n",
diff -puN mm/vmstat.c~debugging-keep-track-of-page-owners-fix mm/vmstat.c
--- a/mm/vmstat.c~debugging-keep-track-of-page-owners-fix
+++ a/mm/vmstat.c
@@ -1059,7 +1059,7 @@ static void pagetypeinfo_showmixedcount_
if (page->order < 0)
continue;
- pagetype = allocflags_to_migratetype(page->gfp_mask);
+ pagetype = gfpflags_to_migratetype(page->gfp_mask);
if (pagetype != mtype) {
if (is_migrate_cma(pagetype))
count[MIGRATE_MOVABLE]++;
_
Patches currently in -mm which might be from akpm@linux-foundation.org are
origin.patch
i-need-old-gcc.patch
arch-alpha-kernel-systblss-remove-debug-check.patch
maintainers-akpm-maintenance.patch
mem-hotplug-fix-boot-failed-in-case-all-the-nodes-are-hotpluggable-checkpatch-fixes.patch
kernel-posix-timersc-code-clean-up-checkpatch-fixes.patch
input-route-kbd-leds-through-the-generic-leds-layer.patch
fs-ocfs2-stack_userc-fix-typo-in-ocfs2_control_release.patch
o2dlm-fix-null-pointer-dereference-in-o2dlm_blocking_ast_wrapper-checkpatch-fixes.patch
ocfs2-free-inode-when-i_count-becomes-zero.patch
mm.patch
memory-hotplug-add-sysfs-zones_online_to-attribute-fix.patch
memory-hotplug-add-sysfs-zones_online_to-attribute-fix-2.patch
mm-compaction-defer-each-zone-individually-instead-of-preferred-zone-fix.patch
mm-compaction-khugepaged-should-not-give-up-due-to-need_resched-fix.patch
prctl-pr_set_mm-introduce-pr_set_mm_map-operation-v3-fix.patch
mm-introduce-do_shared_fault-and-drop-do_fault-fix-fix.patch
do_shared_fault-check-that-mmap_sem-is-held.patch
checkpatch-fix-spello.patch
fs-proc-task_mmuc-shift-mm_access-from-m_start-to-proc_maps_open-checkpatch-fixes.patch
fs-proc-task_mmuc-simplify-the-vma_stop-logic-checkpatch-fixes.patch
fs-proc-task_nommuc-shift-mm_access-from-m_start-to-proc_maps_open-checkpatch-fixes.patch
linux-next.patch
drivers-gpio-gpio-zevioc-fix-build.patch
mm-replace-remap_file_pages-syscall-with-emulation.patch
watchdog-control-hard-lockup-detection-default-fix.patch
debugging-keep-track-of-page-owners.patch
debugging-keep-track-of-page-owners-fix.patch
journal_add_journal_head-debug.patch
journal_add_journal_head-debug-fix.patch
kernel-forkc-export-kernel_thread-to-modules.patch
mutex-subsystem-synchro-test-module.patch
slab-leaks3-default-y.patch
put_bh-debug.patch
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-08-25 22:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-07 22:26 + debugging-keep-track-of-page-owners-fix.patch added to -mm tree akpm
-- strict thread matches above, loose matches on Subject: below --
2014-08-25 22:52 akpm
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox