Re: Portmasters alert: was => Re: uvm physseg - put the lid on it.

[Cherry G. Mathew <cherry%zyx.in@localhost>]

>>>>> "Cherry" == Cherry G Mathew <cherry%zyx.in@localhost> writes:

    Cherry> Hi Matt,
>>>>> "matt" == Matt Thomas <matt%3am-software.com@localhost> writes:


[...]

    matt> Don't add another uvm_* call to all ports.  It's just another things to do wrong.
    matt> Instead add a uvm_md_init and make it call uvm_setpagesize and uvm_physseg_init.
    matt> Thus if something new gets added we don't have to mung all the ports again.

    Cherry> This turned out to be a very excellent suggestion - it actually caught a
    Cherry> couple of places in MD where uvm_physseg_init() was in the wrong place.

Committed via these:
http://mail-index.netbsd.org/source-changes/2016/12/22/msg080088.html
http://mail-index.netbsd.org/source-changes/2016/12/22/msg080094.html


So far there's been only code reorg - would be really nice to know if
things are ok so far.

The real "switch" is in this updated patch now:
http://ftp.netbsd.org/pub/NetBSD/misc/cherry/uvm_physmem/src.diff

Please note that the above patches are version controlled via:
http://ftp.netbsd.org/pub/NetBSD/misc/cherry/uvm_physmem/RCS

in case you wanted to have a look at prior versions.

-- 
~cherry