sparc32: fix lock_mm_and_find_vma() conversion
authorLinus Torvalds <torvalds@linux-foundation.org>
Fri, 30 Jun 2023 03:41:24 +0000 (20:41 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 30 Jun 2023 03:41:24 +0000 (20:41 -0700)
The sparc32 conversion to lock_mm_and_find_vma() in commit a050ba1e7422
("mm/fault: convert remaining simple cases to lock_mm_and_find_vma()")
missed the fact that we didn't actually have a 'regs' pointer available
in the 'force_user_fault()' case.

It's there in the regular page fault path ("do_sparc_fault()"), but not
the window underflow/overflow paths.

Which is all fine - we can just pass in a NULL pointer.  The register
state is only used to avoid deadlock with kernel faults, which is not
the case for any of these register window faults.

Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Fixes: a050ba1e7422 ("mm/fault: convert remaining simple cases to lock_mm_and_find_vma()")
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/sparc/mm/fault_32.c

index a3ccc0267bc20515fcf42437430f0e410c6c2b0f..86a831ebd8c8e30e51058bc09d8c02d2ef2cf71b 100644 (file)
@@ -312,7 +312,7 @@ static void force_user_fault(unsigned long address, int write)
 
        code = SEGV_MAPERR;
 
-       vma = lock_mm_and_find_vma(mm, address, regs);
+       vma = lock_mm_and_find_vma(mm, address, NULL);
        if (!vma)
                goto bad_area_nosemaphore;
        code = SEGV_ACCERR;