From: travis@sgi.com
To: Andrew Morton <akpm@linux-foundation.org>,
Andi Kleen <ak@suse.de>,
mingo@elte.hu
Cc: Christoph Lameter <clameter@sgi.com>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: [PATCH 8/8] percpu: Add debug of invalid per_cpu usage rc8-mm1-fixup
Date: Mon, 21 Jan 2008 12:27:55 -0800 [thread overview]
Message-ID: <20080121202748.713745000@sgi.com> (raw)
In-Reply-To: 20080121202747.593568000@sgi.com
[-- Attachment #1: debug-percpu --]
[-- Type: text/plain, Size: 2126 bytes --]
Provide a means to trap usages of per_cpu variables before
the per_cpu_areas are setup. Define CONFIG_DEBUG_PER_CPU to activate.
Based on: 2.6.24-rc8-mm1
Signed-off-by: Mike Travis <travis@sgi.com>
---
rc8-mm1-fixup:
- rebased from 2.6.24-rc6-mm1 to 2.6.24-rc8-mm1
(removed changes that are in the git-x86.patch)
---
include/asm-generic/percpu.h | 11 ++++++++++-
lib/Kconfig.debug | 12 ++++++++++++
2 files changed, 22 insertions(+), 1 deletion(-)
--- a/include/asm-generic/percpu.h
+++ b/include/asm-generic/percpu.h
@@ -47,12 +47,21 @@ extern unsigned long __per_cpu_offset[NR
#endif
/*
- * A percpu variable may point to a discarded reghions. The following are
+ * A percpu variable may point to a discarded regions. The following are
* established ways to produce a usable pointer from the percpu variable
* offset.
*/
+#ifdef CONFIG_DEBUG_PER_CPU
+#define per_cpu(var, cpu) (*({ \
+ if (!per_cpu_offset(cpu)) { \
+ printk("KERN_NOTICE per_cpu(%s,%d): not available!\n", #var, (int)cpu); \
+ dump_stack(); \
+ } \
+ SHIFT_PERCPU_PTR(&per_cpu_var(var), per_cpu_offset(cpu));}))
+#else
#define per_cpu(var, cpu) \
(*SHIFT_PERCPU_PTR(&per_cpu_var(var), per_cpu_offset(cpu)))
+#endif
#define __get_cpu_var(var) \
(*SHIFT_PERCPU_PTR(&per_cpu_var(var), my_cpu_offset))
#define __raw_get_cpu_var(var) \
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -610,6 +610,18 @@ config PROVIDE_OHCI1394_DMA_INIT
See Documentation/debugging-via-ohci1394.txt for more information.
+config DEBUG_PER_CPU
+ bool "Debug per_cpu usage"
+ depends on DEBUG_KERNEL
+ depends on SMP
+ default n
+ help
+ Say Y here to add code that verifies the per_cpu area is
+ setup before accessing a per_cpu variable. It does add a
+ significant amount of code to kernel memory.
+
+ If unsure, say N.
+
source "samples/Kconfig"
source "lib/Kconfig.kgdb"
--
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
prev parent reply other threads:[~2008-01-21 20:27 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-21 20:27 [PATCH 0/8] percpu: Per cpu code simplification rc8-mm1-fixup travis
2008-01-21 20:27 ` [PATCH 1/8] Modules: Fold percpu_modcopy into module.c rc8-mm1-fixup travis
2008-01-21 20:27 ` [PATCH 2/8] percpu: Change Kconfig to HAVE_SETUP_PER_CPU_AREA rc8-mm1-fixup travis
2008-01-21 20:27 ` [PATCH 3/8] pecpu: Fix size of percpu_data.ptrs array rc8-mm1-fixup travis
2008-01-21 20:27 ` [PATCH 4/8] Sparc64: Use generic percpu fixup rc8-mm1-fixup travis
2008-01-21 20:27 ` [PATCH 5/8] ia64: " travis
2008-01-21 20:27 ` [PATCH 6/8] Powerpc: Use generic per cpu rc8-mm1-fixup travis
2008-01-21 20:27 ` [PATCH 7/8] s390: Use generic percpu rc8-mm1-fixup travis
2008-01-21 20:27 ` travis [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080121202748.713745000@sgi.com \
--to=travis@sgi.com \
--cc=ak@suse.de \
--cc=akpm@linux-foundation.org \
--cc=clameter@sgi.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mingo@elte.hu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).