svn commit: r328218 - in head/sys: amd64/amd64 arm/xscale/ixp425 arm64/arm64 cam cam/ctl compat/ndis dev/aacraid dev/advansys dev/ath dev/beri/virtio dev/bnxt dev/bwn dev/ciss dev/cxgbe/crypto dev/...

[Warner Losh]

On Jan 24, 2018 12:24 PM, "Conrad Meyer" <cem at freebsd.org> wrote:

On Wed, Jan 24, 2018 at 10:51 AM, Pedro Giffuni <pfg at freebsd.org> wrote:
> FWIW, I suggested a panic for M_WAITOK and returning NULL for M_NOWAIT,
but
> cem didn't like it because it was inconsistent.
>
> I think the current argument is more about the size/trigger than the
> behavior though.

Yeah.  If an overflow happens in a path, we want to flag it --
regardless of whether that was a M_NOWAIT or M_WAITOK request.


Which is why we should add check overflows for most of the no wait cases.
They should be checked, but not primarily with mallocarray...

Warner