[issue4111] Add DTrace probes

Changes by Mark Wielaard : -- nosy: +mjw ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/

[issue4111] Add DTrace probes

Changes by Dave Malcolm : -- nosy: +dmalcolm ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.

[issue4111] Add DTrace probes

Changes by Jim Baker : -- nosy: +jbaker ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/m

[issue4111] Add DTrace probes

Robert Kern added the comment: Ah, I misread the Apple function-return probe code. Its extra argument is the type name of the return object or "error" if an exception was raised, not the returned object itself. Could be useful. -- ___ Python tracker

[issue4111] Add DTrace probes

Robert Kern added the comment: James McIlree from Apple has informed me on dtrace-discuss that ustack helpers cannot currently be built on OS X. Bummer. -- ___ Python tracker ___

[issue4111] Add DTrace probes

Robert Kern added the comment: We could probably merge Apple's and Sun's probes without too much trouble. Apple simply extended function-entry to include the argcount in addition to Sun's (filename, funcname, lineno) arguments. We could use Apple's probe while retaining compatibility with Sun-or

[issue4111] Add DTrace probes

Skip Montanaro added the comment: Robert> Is there any interest in my expanding the list of probes? Yes. Jeff Garrett (a guy I work with) added some more DTrace probes to a 2.4 source tree at work. I mentioned them in an earlier message. I'll check with him at work tomorrow and see about ma

[issue4111] Add DTrace probes

Jesús Cea Avión added the comment: > Is there any interest in my expanding the list of probes? Definitively!!!. -- ___ Python tracker ___ __

[issue4111] Add DTrace probes

Robert Kern added the comment: Is there any interest in my expanding the list of probes? Ruby has quite a few more than function-entry and function-return, to give some examples of what is possible: http://dev.joyent.com/projects/ruby-dtrace/wiki/Ruby+DTrace+probes+and+arguments I think that a

[issue4111] Add DTrace probes

Robert Kern added the comment: Hmm, wait a second. Never mind. The Solaris patches don't have ceval.o on the line for compiling phelper.o, either. If dtrace needs to resolve the symbol PyEval_EvalFrameEx in an object file, how does it know to look in ceval.o for the phelper.d? WITH_DTRACE is de

[issue4111] Add DTrace probes

Robert Kern added the comment: Ah, duh, of course. The problem here with PyEval_EvalFrameEx is that I don't have ceval.o on the command line *at all* since OS X's dtrace doesn't support -G. It doesn't appear to accept ceval.o with -h, either, so I suppose that adding the ustack helper may simply

[issue4111] Add DTrace probes

Ted Leung added the comment: On Apr 22, 2009, at 1:24 PM, Skip Montanaro wrote: > > Skip Montanaro added the comment: > > Sorry, I've been away from this issue. I was sort of hoping the Sun > and > Apple folks would just work things out amongst themselves and > present us > with a fait ac

[issue4111] Add DTrace probes

John Levon added the comment: Yes, my mistake. I noticed this typo in the original patch: 207 +PyObject * 208 +PyEval_EvalFrameexEx(PyFrameObject *f, int throwflag)) Can you: - verify that HAVE_DTRACE is indeed defined for ceval.c - do an nm on ceval.o and look which Eval* functions

[issue4111] Add DTrace probes

Robert Kern added the comment: Skip> Perhaps not quite on-topic for this tracker item, but it bugs me that the Skip> mere compilation of a D script requires root privileges. It doesn't. "dtrace -G" and "dtrace -h" (the only "mere compilation" that dtrace does) run without root privileges. That

[issue4111] Add DTrace probes

Skip Montanaro added the comment: Sorry, I've been away from this issue. I was sort of hoping the Sun and Apple folks would just work things out amongst themselves and present us with a fait accompli. ;-) I'll try to mess around with this a little. Robert> $ sudo dtrace ... Perhaps not qu

[issue4111] Add DTrace probes

Robert Kern added the comment: John, -Z does not appear to help: $ sudo dtrace -Z -n 'pid$target::PyEval_EvalFrameEx:entry' -c python dtrace: description 'pid$target::PyEval_EvalFrameEx:entry' matched 0 probes I'm not sure how that would help. If I'm reading the man page correctly, that jus

[issue4111] Add DTrace probes

John Levon added the comment: Robert, I have no idea how Mac OS does pstack helpers without generating object files, sorry. > no simply pid$target:a.out:: probes available. Hmm. Try adding -Z to see if that helps. > /Users/rkern/hg/Python-2.5.4/Include/phelper.d: line 110: relocation > remain

[issue4111] Add DTrace probes

Robert Kern added the comment: This is on an Intel Core 2 Duo running OS X 10.5.6. __i386 is defined. -- ___ Python tracker ___ ___ Py

[issue4111] Add DTrace probes

Laszlo (Laca) Peter added the comment: Look at these lines at the beginning of phelper.d: #if defined(__i386) #definestartframe PyEval_EvalFrameEx #defineendframe PyEval_EvalCodeEx #elif defined(__amd64) #definePyEval_EvalFrameEx PyEval_EvalFrameExReal #definesta

[issue4111] Add DTrace probes

Robert Kern added the comment: Got a bit farther. Adding this stanza to the top of phelper.d gets past the issues in the headers: #ifdef __APPLE__ #define _SYS_TIME_H_ #define _SYS_SELECT_H_ #define __MATH_H__ #define _OS__OSBYTEORDER_H #define _FD_SET #define __GNUC_VA_LIST #endif /* __APPLE__

[issue4111] Add DTrace probes

Robert Kern added the comment: Skip, it doesn't appear that the ustack helper is getting incorporated into the OS X build anywhere. This rule is obviously wrong (compiling the wrong input file with the wrong flags): Include/phelper.h: $(srcdir)/Include/phelper.d dtrace -o $@ $(DFLAGS) -

[issue4111] Add DTrace probes

Changes by Dan Villiom Podlaski Christiansen : -- nosy: +danchr nosy_count: 12.0 -> 13.0 ___ Python tracker ___ ___ Python-bugs-list ma

[issue4111] Add DTrace probes

Changes by Chris Miles : -- nosy: +chrismiles ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue4111] Add DTrace probes

Changes by Joe Ranieri : -- nosy: +sirg3 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/

[issue4111] Add DTrace probes

John Levon added the comment: I haven't seen "shorter than expected" message before, sounds like a Mac OS X specific thing. As for never getting any probes out, this is where it gets fun. Debugging this is very tricky indeed: it involves you dropping into the kernel debugger and looking at the

[issue4111] Add DTrace probes

Changes by Jesús Cea Avión : -- nosy: +jcea ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.o

[issue4111] Add DTrace probes

Ted Leung added the comment: I didn't run auto(re)conf. After I did that, all was well. However, the ustack provider doesn't appear to be working correctly. I tried running the py_profile.d from the DTrace toolkit, and it doesn't show any stack traces, and when the script starts up it

[issue4111] Add DTrace probes

Skip Montanaro added the comment: Ted> I tried building this on my Mac and got this; Forgive me if I'm preaching to the choir here. Did you run autoconf or autoreconf after applying the patch? If not, @DTRACEOBJS@ would not be a substitutable string. It's fairly common (at least in the Pytho

[issue4111] Add DTrace probes

Ted Leung added the comment: I tried building this on my Mac and got this; ar cr libpython2.7.a Python/_warnings.o Python/Python-ast.o Python/ asdl.o Python/ast.o Python/bltinmodule.o Python/ceval.o Python/ compile.o Python/codecs.o Python/errors.o Python/frozen.o Python/ frozenmain.o Python

[issue4111] Add DTrace probes

Changes by John Levon : -- nosy: +movement ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.or

[issue4111] Add DTrace probes

Skip Montanaro added the comment: Here's a patch against the current trunk (2.7) which compiles on my Mac. It adds a --with-dtrace configure option. The code checks to see if the -G option is understood by the dtrace command. If so, dtrace support is added Sun-style. If not we do things Appl

[issue4111] Add DTrace probes

Changes by Brett Cannon : -- nosy: -brett.cannon ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue4111] Add DTrace probes

Skip Montanaro added the comment: Laca> Please see here for discussion about the -G flag on OS X: Laca> http://markmail.org/message/4nheqnexjr2o6mcx Laca> If I read it correctly, on OS X, you will need to use -h instead Laca> of -G and it won't emit an object file (dtrace.o) so you

[issue4111] Add DTrace probes

Laszlo (Laca) Peter added the comment: Please see here for discussion about the -G flag on OS X: http://markmail.org/message/4nheqnexjr2o6mcx If I read it correctly, on OS X, you will need to use -h instead of -G and it won't emit an object file (dtrace.o) so you will not need to link it. Unfo

[issue4111] Add DTrace probes

Skip Montanaro added the comment: me> I get an error later running dtrace which I have yet to investigate. Apple's dtrace program doesn't support the -G flag. When I remove it from Makefile.pre.in and rebuild I get an error about privileges: dtrace -o Python/dtrace.o -C -s ./Python/pytho

[issue4111] Add DTrace probes

Skip Montanaro added the comment: After applying the patch and reconfiguring I get compilation errors in Python/ceval.c. I suspect it's because there is a new header file, Python/python.h. That's probably found by the #include directive in favor of Include/Python.h because of the Mac's case-in

[issue4111] Add DTrace probes

Ted Leung added the comment: On Jan 22, 2009, at 1:35 PM, Skip Montanaro wrote: > > Skip Montanaro added the comment: > > So I completely dropped the ball on this. It appears we have some > folks from Sun and Brett surmised that Ronald Oussoren would be the > likely person to do the heavy lif

[issue4111] Add DTrace probes

Skip Montanaro added the comment: So I completely dropped the ball on this. It appears we have some folks from Sun and Brett surmised that Ronald Oussoren would be the likely person to do the heavy lifting on the Apple side of things. Ronald, I've made you nosy. I will try to get the Solaris p

[issue4111] Add DTrace probes

Changes by Raymond Hettinger : -- nosy: +rhettinger ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

[issue4111] Add DTrace probes

Changes by Robert Kern : -- nosy: +robert.kern ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pytho

[issue4111] Add DTrace probes

Laszlo (Laca) Peter <[EMAIL PROTECTED]> added the comment: I'm the python package maintainer at Sun. We would really like to get the dtrace probes upstream, let me know how I can help. -- nosy: +laca ___ Python tracker <[EMAIL PROTECTED]>

[issue4111] Add DTrace probes

Ted Leung <[EMAIL PROTECTED]> added the comment: And courtesy of Philip Jenvey, here I am. I would *really* like to work to help make this happen. Laszlo Peter at Sun has been doing the ports of Python on Solaris, but we are not up to 2.6 just yet. I'm attaching a pointer to his patches again

[issue4111] Add DTrace probes

Brett Cannon <[EMAIL PROTECTED]> added the comment: On Thu, Nov 13, 2008 at 08:05, Skip Montanaro <[EMAIL PROTECTED]> wrote: > > Skip Montanaro <[EMAIL PROTECTED]> added the comment: > > Brett> http://www.opensource.apple.com/darwinsource/10.5.5/python-30.1.2/ >... >Brett> > http://cvs.o

[issue4111] Add DTrace probes

Skip Montanaro <[EMAIL PROTECTED]> added the comment: Brett> http://www.opensource.apple.com/darwinsource/10.5.5/python-30.1.2/ ... Brett> http://cvs.opensolaris.org/source/xref//jds/spec-files/trunk/patches/Python-07-dtrace.diff Thanks for the pointers. I'll work on getting a uniform

[issue4111] Add DTrace probes

Brett Hoerner <[EMAIL PROTECTED]> added the comment: On Wed, Nov 12, 2008 at 9:31 PM, Skip Montanaro <[EMAIL PROTECTED]> wrote: > I see the reference to Apple in your original post, but can't find anything > related to dtrace & python starting from the URL you gave. Do you have > something more

[issue4111] Add DTrace probes

Skip Montanaro <[EMAIL PROTECTED]> added the comment: Brett> They have released the changes, that's what my patch (attached to Brett> the issue) is based on. I see the reference to Apple in your original post, but can't find anything related to dtrace & python starting from the URL you gave.

[issue4111] Add DTrace probes

Brett Hoerner <[EMAIL PROTECTED]> added the comment: They have released the changes, that's what my patch (attached to the issue) is based on. It's working, it just needs to be cleaned up (it will fail, I believe, for people on systems without DTrace - as I said I'm not very familiar with Makefi

[issue4111] Add DTrace probes

Skip Montanaro <[EMAIL PROTECTED]> added the comment: It appears that Apple has dtracified their Python exeutable in Leopard. Any chance that they can be persuaded to release a patch? I'm working on a patch (based on some work from a friend at work), but it seems like Apple already has a horse i

[issue4111] Add DTrace probes

Changes by Alexander Belopolsky <[EMAIL PROTECTED]>: -- nosy: +belopolsky ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-lis

[issue4111] Add DTrace probes

Changes by Skip Montanaro <[EMAIL PROTECTED]>: -- nosy: +skip.montanaro ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-list

[issue4111] Add DTrace probes

Brett Cannon <[EMAIL PROTECTED]> added the comment: At one point we were told Apple would try to backport their dtrace instrumentation. I don't know what the status of that is, but it obviously has not happened. -- nosy: +brett.cannon ___ Python track

[issue4111] Add DTrace probes

New submission from Brett Hoerner <[EMAIL PROTECTED]>: It would be great if the main Python distribution supported DTrace on (Open)Solaris / FreeBSD / OS X. I've attached a patch against 2.6 that instruments function calls. It's from the ed scripts included in Apple's Python distribution: http: