Author dmalcolm
Recipients dmalcolm, eric.araujo, jcea, loewis, mjw, pitrou
Date 2012-05-11.20:49:53
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1336769395.48.0.610481572826.issue14776@psf.upfronthosting.co.za>
In-reply-to
Content
Thanks Eric, Antoine and Mark.

I'm attaching two new patches: a replacement patch for cpython, and a new patch for the devguide

I've moved the docs to the dev guide, starting a new "Debugging and Instrumentation" section there.

Changes to the cpython patch:
  * fixed a bug in configure.in that was enabling systemtap support even without --with-systemtap (if the devel toolchain was present)
  * I added an initial check to test_systemtap to skip the tests unless Python was configured --with-systemtap
  * pysystemtap.h is not meant to be public, so I've moved the source pysystemtap.d and generated header pysystemtap.h from Include/ to Python/.  I also simplified pysystemtap.d (removed the #pragma lines, since I believe they're DTrace-specific).
  * I've introduced a Python/ceval_systemtap.h private header as suggested by Antoine, moving things in there to simplify the changes to Python/ceval.c

Changes to the devguide docs:
  * removed the ".. impl-detail" as this only seems to work (and be appropriate) in the cpython Doc build, not in devguide.
  * added "eu-readelf -n" example from Mark

The docs refer to the low-level way of doing things (using the markers directly), but don't yet spell out the higher-level way (creating a tapset).  I've left this out of the patches for now to keep the patches simpler.
History
Date User Action Args
2012-05-11 20:49:56dmalcolmsetrecipients: + dmalcolm, loewis, jcea, pitrou, eric.araujo, mjw
2012-05-11 20:49:55dmalcolmsetmessageid: <1336769395.48.0.610481572826.issue14776@psf.upfronthosting.co.za>
2012-05-11 20:49:54dmalcolmlinkissue14776 messages
2012-05-11 20:49:54dmalcolmcreate