On Fri, Nov 06, 2020 at 10:12:27AM -0300, Martin Pieuchot wrote: > Diff below moves the logic dealing with faults of case 1A & 1B to its > own function. > > With this, the logic in uvm_fault() now only deals with case 2 and the > various if/else/goto dances can be simplified. > > As for the previous refactoring diffs, the name is taken from NetBSD > but the implementation is left mostly untouched to ease reviews. > > Upcoming locking will assert that the given amap and anon are sharing > the same lock and that it is held in this function. > > ok?
It's really an entirely straightforward refactoring. The split seems natural and the result cleaner (the new comment also helps). ok tb
