svn commit: r216134 - in head: share/man/man9 sys/amd64/include
sys/arm/include sys/i386/include sys/ia64/include sys/mips/include
sys/pc98/include sys/powerpc/include sys/sparc64/include
ivoras at freebsd.org
Fri Dec 3 14:27:56 UTC 2010
On 3 December 2010 13:46, John Baldwin <jhb at freebsd.org> wrote:
> On Friday, December 03, 2010 5:16:51 am Bruce Cran wrote:
>> On Fri, 3 Dec 2010 20:45:12 +1100 (EST)
>> Bruce Evans <brde at optusnet.com.au> wrote:
>> > KASSERT() in little inline functions gives a lot of bloat for such an
>> > unlikely error. Stupid callers can still pass any garbage count
>> > except 0.
>> Yes, this catches a specific case that hps raised a few years ago:
>> sending zero-length packets/frames would fail by causing the system to
>> hang. Should we just document the restriction in the man page and not
>> try and prevent it at runtime?
> Documenting it is probably sufficient.
I'd say it depends on if the "specific case that hps raised a few
years ago" sentence part refers to an actual problem; i.e. did it
happen in practice? If yes, leaving KASSERTs looks like the best
More information about the svn-src-all