public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@redhat.com>
To: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Cc: Keshavamurthy@bambi.jf.intel.com,
	Anil S <anil.s.keshavamurthy@intel.com>,
	        Ananth N Mavinakayanahalli <ananth@in.ibm.com>,
	        Prasanna S Panchamukhi <prasanna@in.ibm.com>,
	        SystemTAP <systemtap@sources.redhat.com>,
	        Satoshi Oshima <soshima@redhat.com>,
	Hideo Aoki <haoki@redhat.com>,
	        Yumiko Sugita <yumiko.sugita.yf@hitachi.com>
Subject: Re: [PATCH 2/5][djprobe] djprobe core patch
Date: Mon, 30 Oct 2006 14:11:00 -0000	[thread overview]
Message-ID: <1162217443.15286.14.camel@earth> (raw)
In-Reply-To: <454606E0.3050703@hitachi.com>

On Mon, 2006-10-30 at 23:06 +0900, Masami Hiramatsu wrote:
> 1) I thought that should be waste of memory. As you can see,
> the djprobe structure doesn't have its kprobe nor instruction
> buffer, because it is just an interface structure. Instead of
> that, it has an pointer to the djprobe_instance structure which
> is true instance of the djprobe. This "instance" has the kprobe
> structure and instruction buffer. So, I could keep the djprobe
> structure small.
> 2) The other reason is from difference of the usage between djprobe
> and kprobe. Kprobe users have to ensure that the target address
> points to the 1st byte of instruction. On the other hand, djprobe
> users additionally have to count the length of the target
> instructions, and ensure those instructions are relocatable.
> 
> Therefore, I provided those special interfaces.
> 
> If you never mind that the jump-based-kprobe needs two
> kprobe structures per one probe point (one is for the interface,
> another is for the instance -- for deferred releasing), I can
> integrate these interfaces. 

I dont think memory overhead is a big issue - API compatibility is
important and it will also speed up the adoption of djprobes.

could the size restriction of djprobes be relaxed by it doing a
deassembly of the instruction? (only to figure out the size of the
instruction)

basically, i think the best model would be to make djprobes a
transparent speedup of kprobes.

	Ingo

  reply	other threads:[~2006-10-30 14:11 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-16 13:14 [RFC][PATCH][kprobe] enabling booster on the preemptible kernel, take 2 Masami Hiramatsu
2006-10-19  9:00 ` [PATCH 1/5][djprobe] generalize the length of the instruction slots Masami Hiramatsu
2006-10-19  9:03 ` [PATCH 2/5][djprobe] djprobe core patch Masami Hiramatsu
2006-10-27 23:34   ` Keshavamurthy, Anil S
2006-10-30 14:07     ` Masami Hiramatsu
2006-10-30 14:11       ` Ingo Molnar [this message]
2006-10-19  9:03 ` [PATCH 4/5][djprobe] djprobe for i386 architecture code Masami Hiramatsu
2006-10-19  9:03 ` [PATCH 3/5][djprobe] export set_jmp_op() for sharing Masami Hiramatsu
2006-10-19  9:04 ` [PATCH 5/5][djprobe] delayed invoking commit_djprobes() Masami Hiramatsu
2006-10-19  9:04 ` [RFC][djprobe] djprobe examples Masami Hiramatsu
2006-10-30  6:37 ` [RFC][PATCH][kprobe] enabling booster on the preemptible kernel, take 2 bibo,mao
2006-10-30 14:07   ` Masami Hiramatsu
2006-10-31  9:14     ` bibo,mao
2006-10-31 13:47       ` Masami Hiramatsu
2006-10-31 13:49         ` Ingo Molnar
2006-10-31 14:13         ` Ingo Molnar
2006-10-31 16:39           ` Masami Hiramatsu

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=1162217443.15286.14.camel@earth \
    --to=mingo@redhat.com \
    --cc=Keshavamurthy@bambi.jf.intel.com \
    --cc=ananth@in.ibm.com \
    --cc=anil.s.keshavamurthy@intel.com \
    --cc=haoki@redhat.com \
    --cc=masami.hiramatsu.pt@hitachi.com \
    --cc=prasanna@in.ibm.com \
    --cc=soshima@redhat.com \
    --cc=systemtap@sources.redhat.com \
    --cc=yumiko.sugita.yf@hitachi.com \
    /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 read-only IMAP folder(s) and NNTP newsgroup(s).