Re: [PATCH 07/25] mm/csky: Use mm_fault_accounting()

From: Linus Torvalds
Date: Wed Jun 17 2020 - 13:53:48 EST


On Wed, Jun 17, 2020 at 8:49 AM Peter Xu <peterx@xxxxxxxxxx> wrote:
>
> I don't think it's a must, but mmap_sem should not be required at least by
> observing current code. E.g., do_user_addr_fault() of x86 does the accounting
> without mmap_sem even before this series.

All the accounting should be per-thread and not need any locking.

Which is why a remote GUP should never account to the remote mm - not
only isn't there an unambiguous thread to account to (an mm can share
many threads), but it would require locking not just for the remote
update, but for all normal page faults.

Linus