From: Linus Torvalds Date: Thu, 29 Jun 2023 19:36:47 +0000 (-0700) Subject: gup: avoid stack expansion warning for known-good case X-Git-Tag: v6.6.17~4544 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=eee9c708cc89b4600c6e6cdda5bc2b8b4dad96cb;p=platform%2Fkernel%2Flinux-rpi.git gup: avoid stack expansion warning for known-good case In commit a425ac5365f6 ("gup: add warning if some caller would seem to want stack expansion") I added a temporary warning to catch any strange GUP users that would be impacted by the fact that GUP no longer extends the stack. But it turns out that the warning is most easily triggered through __access_remote_vm(), that already knows to expand the stack - it just does it *after* calling GUP. So the warning is easy to trigger by just running gdb (or similar) and accessing things remotely under the stack. This just adds a temporary extra "expand stack early" to avoid the warning for the already converted case - not because the warning is bad, but because getting the warning for this known good case would then hide any subsequent warnings for any actually interesting cases. Let's try to remember to revert this change when we remove the warnings. Signed-off-by: Linus Torvalds --- diff --git a/mm/memory.c b/mm/memory.c index d8a9a77..0ae5947 100644 --- a/mm/memory.c +++ b/mm/memory.c @@ -5694,6 +5694,10 @@ int __access_remote_vm(struct mm_struct *mm, unsigned long addr, void *buf, if (mmap_read_lock_killable(mm)) return 0; + /* Avoid triggering the temporary warning in __get_user_pages */ + if (!vma_lookup(mm, addr) && !expand_stack(mm, addr)) + return 0; + /* ignore errors, just check how much was successfully transferred */ while (len) { int bytes, offset;