On Fri, Nov 13, 2020 at 12:17:04PM +0100, Theo Buehler wrote: > On Wed, Nov 04, 2020 at 11:04:12AM -0300, Martin Pieuchot wrote: > > Diff below introduces a helper that looks for existing mapping. The > > value returned by this lookup function determines if there's an anon > > at the faulting address which tells us if we're dealign with a fault > > of type 1 or 2. > > > > This small refactoring is part of the current work to separate the code > > handling faults of type 1 and 2. The end goal being to move the type 1 > > faults handling out of the KERNEL_LOCK(). > > > > The function name is taken from NetBSD to not introduce more difference > > than there's already. > > > > ok? > > ok tb. > > I've been running the three diffs for two days and this went through two > 'make release' >
Same here. NetBSD's uvm fault handler is a lot more readable than ours, so heading in that direction seems like a pretty good idea, ok jmatthew@
