syscall/errno tapset checkins
Frank Ch. Eigler
fche@redhat.com
Mon Sep 26 20:41:00 GMT 2005
Hi, Kevin -
It's nice to see this code starting to appear in CVS. Some
suggestions. For errno.stp, you don't need to use relatively
inefficient embedded C like that. Consider instead an all-script
solution:
global _errno_table
probe begin {
_errno_table[1] = "EFOOBAR"
_errno_table[4] = "EBARBAZ"
// and so on
}
function strerror (e) { return _errno_table[e] }
Then any reference to the script function "strerror" will drag in the
initialization code as a side-effect.
Another suggestion. It would be nice to add a ChangeLog entry for
nontrivial commits such as these. It would be nice to add
stapfuncs(5) or stapprobes(5) documentation lines for the contents as
they settle down.
- FChE
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/systemtap/attachments/20050926/ac31a1fa/attachment.sig>
More information about the Systemtap
mailing list