John Hawkinson <jhawk@MIT.EDU> writes:
> |  > i think you should be careful about not protecting it.   by placing
> |  > a page that has possible active mappings on the inactive queue you are
> |  > violating one of the assumptions that i wrote the rest of the code with
> |  > and it could cause trouble later....
> | 
> | Yah, I'm making a sweep through the code.  You just have to be careful
> | dealing with the PG_CLEAN bit.  Looks like the only place that really
> | needs help is uvm_mk_pcluster().  And it looks like it might not even
> | be *THAT* bad.
> 
> It seems like these assumptions, regardless of whether they are changed,
> should be documented in section 9.

Ditto. These are the sort of algorithms that are so key to the system and
its performance that they should be documented.

Perry