csky: fix up lock_mm_and_find_vma() conversion
authorLinus Torvalds <torvalds@linux-foundation.org>
Fri, 30 Jun 2023 06:34:29 +0000 (23:34 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 30 Jun 2023 06:34:29 +0000 (23:34 -0700)
As already mentioned in my merge message for the 'expand-stack' branch,
we have something like 24 different versions of the page fault path for
all our different architectures, all just _slightly_ different due to
various historical reasons (usually related to exactly when they
branched off the original i386 version, and the details of the other
architectures they had in their history).

And a few of them had some silly mistake in the conversion.

Most of the architectures call the faulting address 'address' in the
fault path.  But not all.  Some just call it 'addr'.  And if you end up
doing a bit too much copy-and-paste, you end up with the wrong version
in the places that do it differently.

In this case it was csky.

Fixes: a050ba1e7422 ("mm/fault: convert remaining simple cases to lock_mm_and_find_vma()")
Reported-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/csky/mm/fault.c

index ae9781b7d92ea5e65554d42d6d9d6ba2ac5838d6..a885518ce1dd285671e1e06d548cd91aa7f855f5 100644 (file)
@@ -237,7 +237,7 @@ asmlinkage void do_page_fault(struct pt_regs *regs)
        if (is_write(regs))
                flags |= FAULT_FLAG_WRITE;
 retry:
-       vma = lock_mm_and_find_vma(mm, address, regs);
+       vma = lock_mm_and_find_vma(mm, addr, regs);
        if (unlikely(!vma)) {
                bad_area_nosemaphore(regs, mm, code, addr);
                return;