This is the mail archive of the
systemtap@sources.redhat.com
mailing list for the systemtap project.
Re: [RFC] Multiple kprobes at an address redux (take3)
- From: Maneesh Soni <maneesh at in dot ibm dot com>
- To: Ananth N Mavinakayanahalli <amavin at redhat dot com>
- Cc: suparna at in dot ibm dot com, SystemTAP <systemtap at sources dot redhat dot com>
- Date: Wed, 13 Apr 2005 10:16:50 +0530
- Subject: Re: [RFC] Multiple kprobes at an address redux (take3)
- References: <425AD0D3.80703@redhat.com> <20050412035650.GA3749@in.ibm.com> <425C1E67.50600@redhat.com>
- Reply-to: maneesh at in dot ibm dot com
On Tue, Apr 12, 2005 at 03:15:51PM -0400, Ananth N Mavinakayanahalli wrote:
> Suparna Bhattacharya wrote:
>
> Hello Suparna,
>
> >It would be nice to summarize all the requirements upfront, before
> >moving on to the design.
> >
> >One observation I have is that a register_single_kprobe equivalent
> >is still useful for those who might be setting up a kprobe for debugging
> >or fault injection purposes -- or rather maybe something like
> >register_exclusive_kprobe, because one might not want to be surprised by
> >having multiple handlers clicking away in such a situation.
>
> Yes, this is something we have to be congnizant of. So, here is what I
> propose:
>
> We have a "coexist" field in struct kprobe which is default 0. This
> means that the kprobe cannot tolerate other kprobes at the same address.
> A non zero value would indicate the kprobe can be one of many at the
> address.
>
Ananth,
At this time I think "coexist" will be an overkill as we now know that
theoriticaly there can be same problems even with kporbes on adjacent
instruction as with coexisting kprobes.
[..]
> >
> >Nomenclature wise - both aggr_probe and multi-probe are names that
> >sound more like a set of different probes, not multiple handlers
> >for the same probe as is really intended. Ideas for better names ?
>
> "manager kprobe"? Don't know what would be a good name, really :)
>
IMO, register_kprobe_single() for low-level uses and register_kprobe() for
normal uses are ok.
Thanks
Maneesh
--
Maneesh Soni
Linux Technology Center,
IBM India Software Labs,
Bangalore, India
email: maneesh@in.ibm.com
Phone: 91-80-25044990