cvs commit: src/sys/kern kern_slaballoc.c
Joerg Sonnenberger
joerg at britannica.bec.de
Mon Mar 28 10:04:59 PST 2005
On Mon, Mar 28, 2005 at 09:48:40AM -0800, Matthew Dillon wrote:
>
> :
> :On Mon, Mar 28, 2005 at 05:13:33PM -0000, Andreas Hauser wrote:
> :> > Remove redundant panic.
> :>
> :> Joerg,
> :>
> :> can you elaborate a bit more ?
> :
> :This simply removed some dead code. The conditional was always false,
> :because it was already checked a few lines before.
> :
> :Joerg
>
> Actually not quite true. The first condition requires that both bits
> be zero. The second requires only that one bit be 0... which means
> that you can't remove the second one. You might be able to remove
> the first one.
I should sleep more. Matt, what do you think about truely separating the
M_NULLOK and M_RNOWAIT flags? Currently M_RNOWAIT implies M_NULLOK and
since the flags are created by sys/malloc.h anyway, we could
include M_NULLOK in the M_RNOWAIT cases.
Joerg
More information about the Commits
mailing list