linux-stable.git/kernel/kprobes.c, branch linux-2.6.20.y

[PATCH] kprobes: replace magic numbers with enum

2007-01-31T00:01:35+00:00

Replace the magic numbers with an enum, and gets rid of a warning on the
specific architectures (ex.  powerpc) on which the compiler considers
'char' as 'unsigned char'.

Signed-off-by: Masami Hiramatsu 
Cc: Prasanna S Panchamukhi 
Cc: Ananth N Mavinakayanahalli 
Cc: Anil S Keshavamurthy 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[PATCH] kprobes: enable booster on the preemptible kernel

2006-12-07T16:39:38+00:00

When we are unregistering a kprobe-booster, we can't release its
instruction buffer immediately on the preemptive kernel, because some
processes might be preempted on the buffer.  The freeze_processes() and
thaw_processes() functions can clean most of processes up from the buffer.
There are still some non-frozen threads who have the PF_NOFREEZE flag.  If
those threads are sleeping (not preempted) at the known place outside the
buffer, we can ensure safety of freeing.

However, the processing of this check routine takes a long time.  So, this
patch introduces the garbage collection mechanism of insn_slot.  It also
introduces the "dirty" flag to free_insn_slot because of efficiency.

The "clean" instruction slots (dirty flag is cleared) are released
immediately.  But the "dirty" slots which are used by boosted kprobes, are
marked as garbages.  collect_garbage_slots() will be invoked to release
"dirty" slots if there are more than INSNS_PER_PAGE garbage slots or if
there are no unused slots.

Cc: "Keshavamurthy, Anil S" 
Cc: Ananth N Mavinakayanahalli 
Cc: "bibo,mao" 
Cc: Prasanna S Panchamukhi 
Cc: Yumiko Sugita 
Cc: Satoshi Oshima 
Cc: Hideo Aoki 
Signed-off-by: Masami Hiramatsu 
Acked-by: Ingo Molnar 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[PATCH] kretprobe spinlock deadlock patch

2006-10-02T14:57:16+00:00

kprobe_flush_task() possibly calls kfree function during holding
kretprobe_lock spinlock, if kfree function is probed by kretprobe that will
incur spinlock deadlock.  This patch moves kfree function out scope of
kretprobe_lock.

Signed-off-by: bibo, mao 
Signed-off-by: Ananth N Mavinakayanahalli 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[PATCH] kprobe whitespace cleanup

2006-10-02T14:57:16+00:00

Whitespace is used to indent, this patch cleans up these sentences by
kernel coding style.

Signed-off-by: bibo, mao 
Signed-off-by: Ananth N Mavinakayanahalli 
Cc: "Luck, Tony" 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[PATCH] Kprobes: Make kprobe modules more portable

2006-10-02T14:57:16+00:00

In an effort to make kprobe modules more portable, here is a patch that:

o Introduces the "symbol_name" field to struct kprobe.
  The symbol->address resolution now happens in the kernel in an
  architecture agnostic manner. 64-bit powerpc users no longer have
  to specify the ".symbols"
o Introduces the "offset" field to struct kprobe to allow a user to
  specify an offset into a symbol.
o The legacy mechanism of specifying the kprobe.addr is still supported.
  However, if both the kprobe.addr and kprobe.symbol_name are specified,
  probe registration fails with an -EINVAL.
o The symbol resolution code uses kallsyms_lookup_name(). So
  CONFIG_KPROBES now depends on CONFIG_KALLSYMS
o Apparantly kprobe modules were the only legitimate out-of-tree user of
  the kallsyms_lookup_name() EXPORT. Now that the symbol resolution
  happens in-kernel, remove the EXPORT as suggested by Christoph Hellwig
o Modify tcp_probe.c that uses the kprobe interface so as to make it
  work on multiple platforms (in its earlier form, the code wouldn't
  work, say, on powerpc)

Signed-off-by: Ananth N Mavinakayanahalli 
Signed-off-by: Prasanna S Panchamukhi 
Signed-off-by: Christoph Hellwig 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[PATCH] IA64: kprobe invalidate icache of jump buffer

2006-07-31T20:28:38+00:00

Kprobe inserts breakpoint instruction in probepoint and then jumps to
instruction slot when breakpoint is hit, the instruction slot icache must
be consistent with dcache.  Here is the patch which invalidates instruction
slot icache area.

Without this patch, in some machines there will be fault when executing
instruction slot where icache content is inconsistent with dcache.

Signed-off-by: bibo,mao 
Acked-by: "Luck, Tony" 
Acked-by: Keshavamurthy Anil S 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[PATCH] Notify page fault call chain

2006-06-26T16:58:22+00:00

With this patch Kprobes now registers for page fault notifications only when
their is an active probe registered.  Once all the active probes are
unregistered their is no need to be notified of page faults and kprobes
unregisters itself from the page fault notifications.  Hence we will have ZERO
side effects when no probes are active.

Signed-off-by: Anil S Keshavamurthy 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[PATCH] Kprobes registers for notify page fault

2006-06-26T16:58:22+00:00

Kprobes now registers for page fault notifications.

Signed-off-by: Anil S Keshavamurthy 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[PATCH] Kprobe: multi kprobe posthandler for booster

2006-06-26T16:58:22+00:00

If there are multi kprobes on the same probepoint, there will be one extra
aggr_kprobe on the head of kprobe list.  The aggr_kprobe has
aggr_post_handler/aggr_break_handler whether the other kprobe
post_hander/break_handler is NULL or not.  This patch modifies this, only
when there is one or more kprobe in the list whose post_handler is not
NULL, post_handler of aggr_kprobe will be set as aggr_post_handler.

[soshima@redhat.com: !CONFIG_PREEMPT fix]
Signed-off-by: bibo, mao 
Cc: Masami Hiramatsu 
Cc: Ananth N Mavinakayanahalli 
Cc: "Keshavamurthy, Anil S" 
Cc: Prasanna S Panchamukhi 
Cc: Jim Keniston 
Cc: Yumiko Sugita 
Cc: Hideo Aoki 
Signed-off-by: Satoshi Oshima 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[PATCH] kprobes: NULL out non-relevant fields in struct kretprobe

2006-04-20T14:54:03+00:00

In cases where a struct kretprobe's *_handler fields are non-NULL, it is
possible to cause a system crash, due to the possibility of calls ending up
in zombie functions.  Documentation clearly states that unused *_handlers
should be set to NULL, but kprobe users sometimes fail to do so.

Fix it by setting the non-relevant fields of the struct kretprobe to NULL.

Signed-off-by: Ananth N Mavinakayanahalli 
Acked-by: Jim Keniston 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds