On Mon, Jun 20, 2011 at 3:43 PM, Miod Vallat <m...@online.fr> wrote: >> uvm_extern.h is the header people should be consuming. change a few uvm.h >> to uvm_extern.h. > > This is not what, say, kern_fork.c does. If you include <uvm.h>, you > don't need any other <uvm_whatever.h>, and definitely not > <uvm/uvm_extern.h>.
kern_fork.c was relying on vnode.h to get uvm.h, so it's added. I was undecided on whether to include both or not, I think there's already several examples of that in the tree. I went with both as a hint that uvm.h was intentional, but can easily remove the uvm_extern.h includes. >> also, simplify and move the bufcache adjustment code from sysctl to the >> vfs_bio.c where it fits nicer. > > Separate diff please. OK, this happened because it's another file that needed dma_constraint.
