Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/dev/usb



Jason Thorpe <thorpej%me.com@localhost> writes:

>> What does openindiana/illumos  do here?
>
> It is not an error to not use a constructor -<em-dash>- esp considering that the general memory allocator is built on top of pool_cache.
>
> IIRC, the original Solaris behavior was “uninitialized if no
> constructor specified”.  I would not advocate for “always zero-init if
> no constructor” because that could be wasteful.  I’d rather have it be
> an explicit ask when the pool_cache is initialized.

That sounds reasonable.   It seems like asking for zero and a
constructor should KASSERT and if not DIAGNOSTIC be an error.



Home | Main Index | Thread Index | Old Index