cvs commit: src/sys/netinet sctp_timer.c sctp_timer.h
rrs at lakerest.net
Fri Aug 29 09:40:02 UTC 2008
MY bad for not catching it earlier.. I forgot to
build LINT (which I usually do) especially when
I am fixing something for a outside user (they were
having panics and I realized all panics that are
recoverable SHOULD be under INVARANTS, panic
I think was caused by another change I put in
strict_sacks really needs to be on by default and
only turned off at interops...)
On Aug 28, 2008, at 5:51 PM, Giorgos Keramidas wrote:
> On Thu, 28 Aug 2008 20:31:24 +0000 (UTC), Randall Stewart <rrs at freebsd.org
> > wrote:
>> rrs 2008-08-28 20:31:24 UTC
>> FreeBSD src repository
>> Modified files:
>> sys/netinet sctp_timer.c sctp_timer.h
>> SVN rev 182405 on 2008-08-28 20:31:24Z by rrs
>> ok, non static the function and put in the .h so
>> when we do INVARANT compile the compiler will not
>> dis the function that is not used. Hmm maybe I should have
>> made it ifndef INVARIANTs..
> gcc should be smart enough to throw away the non-static function
> if SCTP is built into the kernel (and not as a module) *and*
> there are no external references from other parts of the source.
> So this should work fine.
> The ifndef INVARIANTS version is probably something like the
> second patch I sent earlier, also at
> but I didn't really 'like' having a static function inside a
> large #ifndef INVARIANTS block...
> Thanks for fixing this so fast :-)
More information about the cvs-all