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