From: Josh Stone <jistone@redhat.com>
Date: Fri, 3 Aug 2012 01:56:54 +0000 (-0700)
Subject: stapdyn: TODO updates
X-Git-Tag: release-2.0~215
X-Git-Url: https://sourceware.org/git/?a=commitdiff_plain;h=923fc591b1dcf4de193079eb6c868359e2077d2d;p=systemtap.git

stapdyn: TODO updates

- Build integration is in good shape, removing
- Runtime is largely done, but some cleanup left (PR14179)
- Breaking up the tapsets is pretty much done (PR14180)
- New: really need a deref() mechanism
- New: need to rework stapdu_target
---

diff --git a/stapdyn/TODO b/stapdyn/TODO
index e6e599cfe..3456eedfd 100644
--- a/stapdyn/TODO
+++ b/stapdyn/TODO
@@ -1,16 +1,8 @@
 === TODO items for SystemTap+Dyninst ====
 
-* Integrate with the rest of SystemTap build.
-- Add configury, --with-dyninst or similar.
-- Add stapdyn/* to Makefiles.
-(These are partially there, but should be made more general
- and reviewed for good autoconf style.)
-
-* Reorganize the SystemTap runtime to better support different targets.
-- Leave the runtime/ base for generic shared code.
-- Move kernel-specific stuff to runtime/linux/ or runtime/kernel/.
-- Move dyninst-specific stuff to runtime/dyninst/
-- Select sub-runtimes with e.g. #ifdef __KERNEL__
+* Finish splitting the runtime (PR14179)
+- A lot of kernel-specific stuff is moved to runtime/linux/
+- More needs to be moved, and implemented in runtime/dyninst/ as needed.
 
 * Restrict the set of available probe types.
 - Currently the "bad" types will just lead to pass-4 errors.  We're operating
@@ -31,10 +23,6 @@
 - Will allow for other two-way commands, maybe removing the need for
   stuff like stp_dummy_init/exit.
 
-* Refactor tapset/.
-- Use a scheme like what is proposed for runtime/ ?
-- Nearly-similar embedded-c can use #ifdef __KERNEL__ alternations.
-
 * Translator generalization, reducing kernel-isms.
 - Generated code should take a more general flavor, that can be implemented on
   either kernel or dyninst runtime interfaces, rather than kernel-flavored code
@@ -61,4 +49,16 @@
 - Needs to check that the remote has stapdyn at all.
 - Probably needs a new stapio command akin to 'run', e.g. 'dyn'.
 
+* Need a deref implementation for memory protection!
+- Not sure how to do this, maybe just a SIGSEGV handler?
+
+* Make stapdu_target more flexible
+- Current fixed ABI is too limited IMO
+- JSON is an idea, but perhaps overkill.
+- Simpler idea: just a list of key=value strings, where each probe=TYPE starts
+  a new target.  Something like:
+    probe=uprobe\0path=/bin/foo\0offset=0x42\0
+    probe=uprobe\0path=/bin/bar\0offset=0x84\0
+  - I like keeping it textual, but want to avoid too much verbosity too...
+
 * Enumerate more TODO tasks.