Subject: Re: CVS commit: src/sys/kern
To: None <christos@netbsd.org>
From: enami tsugutomo <enami@but-b.or.jp>
List: source-changes
Date: 01/31/2005 07:56:44
> Modified Files:
> 	src/sys/kern: exec_ecoff.c
> 
> Log Message:
> make sure that bss size > 0 before we add a vmcmd for it.

So, why this test needs to be done in the caller?

When UVM is first introduced, vmcmd_map_zero had a test for the length
like vmcmd_map_{page,read}vn, and it is removed later but the log
message only mentions translation of return value (KERN_* -> E*).  And
until recently (the last summer), mapping of size is allowed.  So, I
guess the removal of the test was not intentional.

Nowadays, NEW_VMCMD{,2} is always expanded to the call of new_vmcmd(),
probably we can move the test to there.

enami.