Messages in this thread | | | Date | Mon, 13 Sep 2010 09:33:48 -0700 | From | Randy Dunlap <> | Subject | Re: [PATCH -next,-tip ] kprobes: Fix Kconfig dependency |
| |
On 09/13/10 03:25, Masami Hiramatsu wrote: > Fix Kconfig dependency among Kprobes, optprobe and kallsyms. > > Kprobes uses kallsyms_lookup for finding target function and checking > instruction boundary, thus CONFIG_KPROBES should select CONFIG_KALLSYMS. > > Optprobe is an optional feature which is supported on x86 arch, and > it also uses kallsyms_lookup for checking instructions in the target > function. Since KALLSYMS_ALL just adds symbols of kernel variables, > it doesn't need to select KALLSYMS_ALL. > > Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> > Cc: Felipe Contreras <felipe.contreras@gmail.com> > Cc: Randy Dunlap <randy.dunlap@oracle.com>, > Cc: linux-kernel@vger.kernel.org, > Cc: linux-next@vger.kernel.org, > Cc: Ananth N Mavinakayanahalli <ananth@in.ibm.com>, > Cc: Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com>, > Cc: "David S. Miller" <davem@davemloft.net>, > Cc: akpm <akpm@linux-foundation.org> > --- > > arch/Kconfig | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/Kconfig b/arch/Kconfig > index 4877a8c..fe48fc7 100644 > --- a/arch/Kconfig > +++ b/arch/Kconfig > @@ -32,8 +32,9 @@ config HAVE_OPROFILE > > config KPROBES > bool "Kprobes" > - depends on KALLSYMS && MODULES > + depends on MODULES > depends on HAVE_KPROBES > + select KALLSYMS > help > Kprobes allows you to trap at almost any kernel address and > execute a callback function. register_kprobe() establishes > @@ -45,7 +46,6 @@ config OPTPROBES > def_bool y > depends on KPROBES && HAVE_OPTPROBES > depends on !PREEMPT > - select KALLSYMS_ALL > > config HAVE_EFFICIENT_UNALIGNED_ACCESS > bool >
Fixes the kconfig dependency problem. Thanks.
Acked-by: Randy Dunlap <randy.dunlap@oracle.com>
-- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***
| |