On Mon, Dec 10, 2007 at 03:23:50PM +0530, Ananth N Mavinakayanahalli wrote: > From: Ananth N Mavinakayanahalli <[EMAIL PROTECTED]> > > This patch builds samples/kprobes/kretprobe_example.c only on archs that > support kretprobes. > > > Signed-off-by: Ananth N Mavinakayanahalli <[EMAIL PROTECTED]> > --- > samples/kprobes/Makefile | 7 +++++-- > 1 files changed, 5 insertions(+), 2 deletions(-) > > Index: linux-2.6.24-rc4/samples/kprobes/Makefile > =================================================================== > --- linux-2.6.24-rc4.orig/samples/kprobes/Makefile > +++ linux-2.6.24-rc4/samples/kprobes/Makefile > @@ -1,5 +1,8 @@ > # builds the kprobes example kernel modules; > # then to use one (as root): insmod <module_name.ko> > > -obj-$(CONFIG_SAMPLE_KPROBES) += kprobe_example.o jprobe_example.o \ > - kretprobe_example.o > +obj-$(CONFIG_SAMPLE_KPROBES) += kprobe_example.o jprobe_example.o > + > +ifeq ($(CONFIG_HAVE_KRETPROBES),y) > +obj-$(CONFIG_SAMPLE_KPROBES) += kretprobe_example.o > +endif
It would be nice to push this configuration dependency to Kconfig. Something like: config SAMPLE_KRETPROBES default y depends on SAMPLE_KPROBES And then in the Makefile have: obj-$(CONFIG_SAMPLE_KRETPROBES) += kretprobe_example.o Sam -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/