On Tue, Dec 23, 2014 at 08:26, Mark Kettenis wrote: >> Date: Mon, 22 Dec 2014 21:19:40 -0500 >> From: Ted Unangst <[email protected]> >> >> @@ -1723,11 +1706,9 @@ uvm_swap_io(struct vm_page **pps, int st >> * now allocate a buf for the i/o. >> * [make sure we don't put the pagedaemon to sleep...] >> */ >> - s = splbio(); >> pflag = (async || curproc == uvm.pagedaemon_proc) ? PR_NOWAIT : >> PR_WAITOK; >> - bp = pool_get(&bufpool, pflag); >> - splx(s); >> + bp = pool_get(&bufpool, pflag | PR_ZERO); > > Why are you adding PR_ZERO here?
I don't trust the other fields to be correctly set. The pool_get in vfs_bio.c always does PR_ZERO.
