re: CVS commit: src/sys/uvm

[matthew green <mrg%eterna.com.au@localhost>]

"Cherry G. Mathew" writes:
> Module Name:	src
> Committed By:	cherry
> Date:		Thu Dec 22 13:26:25 UTC 2016
> 
> Modified Files:
> 	src/sys/uvm: Makefile uvm.h uvm_extern.h uvm_page.c
> 
> Log Message:
> Use uvm_physseg.h:uvm_page_physload() instead of uvm_extern.h
> 
> For this, include uvm_physseg.h in the build and include tree, make a
> cosmetic modification to the prototype for uvm_page_physload().

why does this need to be installed?

this header seems kernel-only, it should be not installed
and it should not be included if !_KERNEL.

note that <uvm/uvm_extern.h> should work in user or kernel
sanely to include from anywhere outside of sys/uvm itself,
but we shouldn't export things to userland we don't need.


.mrg.