This is the mail archive of the systemtap@sources.redhat.com mailing list for the systemtap project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: runtime committed to cvs

On Wed, 2005-03-09 at 20:33 -0500, Frank Ch. Eigler wrote:
> Hi -
> 
> > [...]
> > > OOM handling should be left to the caller.
> > 
> > That gets ugly fast.  I thought it better to have global handling of OOM
> > situations.  Really we only have two options, record an OOM situation
> > was seen and either continue (with loss of data) or exit.
> > [...]
> 
> Yes, but deciding between those alternatives, safely cleaning up
> whatever the caller was doing, and especially contemplating something
> as drastic as actually triggering a module unload, are all best left
> to the layer above this one.

No reason I can't support both.  I'll just change the _stp_map_set_xxx
funcs to return an error code instead of void. 

Martin
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]