Re: uvm_fault_lower refactoring

2021-02-23 Thread Mark Kettenis
> Date: Tue, 23 Feb 2021 10:07:43 +0100 > From: Martin Pieuchot > > On 23/02/21(Tue) 00:24, Mark Kettenis wrote: > > > Date: Mon, 22 Feb 2021 10:10:21 +0100 > > > From: Martin Pieuchot > > > > > > On 16/02/21(Tue) 11:20, Martin Pieuchot wrote: > > > > Start by moving `pgo_fault' handler

Re: uvm_fault_lower refactoring

2021-02-23 Thread Martin Pieuchot
On 23/02/21(Tue) 00:24, Mark Kettenis wrote: > > Date: Mon, 22 Feb 2021 10:10:21 +0100 > > From: Martin Pieuchot > > > > On 16/02/21(Tue) 11:20, Martin Pieuchot wrote: > > > Start by moving `pgo_fault' handler outside of uvm_fault_lower(). > > > > > > If a page has a backing object that prefer

Re: uvm_fault_lower refactoring

2021-02-22 Thread Mark Kettenis
> Date: Mon, 22 Feb 2021 10:10:21 +0100 > From: Martin Pieuchot > > On 16/02/21(Tue) 11:20, Martin Pieuchot wrote: > > Start by moving `pgo_fault' handler outside of uvm_fault_lower(). > > > > If a page has a backing object that prefer to handler to fault itself > > the locking will be

Re: uvm_fault_lower refactoring

2021-02-22 Thread Chris Cappuccio
Martin Pieuchot [m...@openbsd.org] wrote: > On 16/02/21(Tue) 11:20, Martin Pieuchot wrote: > > Start by moving `pgo_fault' handler outside of uvm_fault_lower(). > > > > If a page has a backing object that prefer to handler to fault itself > > the locking will be different, so keep it under

Re: uvm_fault_lower refactoring

2021-02-22 Thread Martin Pieuchot
On 16/02/21(Tue) 11:20, Martin Pieuchot wrote: > Start by moving `pgo_fault' handler outside of uvm_fault_lower(). > > If a page has a backing object that prefer to handler to fault itself > the locking will be different, so keep it under KERNEL_LOCK() for the > moment and make it separate from

uvm_fault_lower refactoring

2021-02-16 Thread Martin Pieuchot
Start by moving `pgo_fault' handler outside of uvm_fault_lower(). If a page has a backing object that prefer to handler to fault itself the locking will be different, so keep it under KERNEL_LOCK() for the moment and make it separate from the rest of uvm_fault_lower(). ok? Index: