[erlang-patches] DTrace patch, review draft #3

Scott Lystig Fritchie fritchie@REDACTED
Thu Nov 17 08:32:04 CET 2011


Björn Gustavsson <bgustavsson@REDACTED> wrote:

bg> My final comment is that your changes are included in a single
bg> commit, which make it harder to review and will it make it harder to
bg> understand in the future. I would suggest breaking it up into
bg> several commits (at least three): [...]

Björn, I've split the commit from 1 -> 4 pieces.  I've also fixed a few
bugs and have un-done the DTrace-related edits that I'd put in
README.md.  Instead, that text has been moved to a new file,
README.dtrace.md.

I have eliminated all of the OTP test suite failures that I'm aware of,
though the tests were run this afternoon prior to splitting the commit.
It's possible that I un-fixed a bugfix, though I think the odds are low.
I'll be re-running the tests overnight tonight with DTrace unconfigured,
DTrace configured, and DTrace configured + probes enabled(*).

git clone git://github.com/slfritchie/otp.git dtrace-review3

https://github.com/slfritchie/otp/compare/dtrace-review3
https://github.com/slfritchie/otp/compare/dtrace-review3.patch

-Scott

(*) I've encountered a odd/weird/buggy? problem with Solaris 10 where
the Erlang OS process occasionally dumps core due to a SIGTRAP.  I think
it may have to do with a race condition between starting a new process
and a turn-them-all-on probe specification.  If anybody happens to know
(or guess) anything about this, please contact me.



More information about the erlang-patches mailing list