[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250407030306.411977-1-bhe@redhat.com>
Date: Mon, 7 Apr 2025 11:03:03 +0800
From: Baoquan He <bhe@...hat.com>
To: linux-mm@...ck.org
Cc: akpm@...ux-foundation.org,
osalvador@...e.de,
david@...hat.com,
mingo@...nel.org,
yanjun.zhu@...ux.dev,
linux-kernel@...r.kernel.org,
Baoquan He <bhe@...hat.com>
Subject: [PATCH v3 0/3] mm/gup: Minor fix, cleanup and improvements
These are made when I explore codes in mm/gup.c.
v2->v3:
=========
- In patch 1, change to use local variable 'start' as a loop cursor
in fault_in_readable() and fault_in_writeable() so that they have the
consistent code style with fault_in_safe_writeable(). Doing this can
avoid ugly long line caused by kinds of type cast (const char __user *).
Thanks to David who suggested two ways including this one, I found this
is better when changing code.
- In patch2, changes to add VM_WARN_ON_ONCE() for both below chekcing
because all external users of __get_user_pages() have done the
checking in is_valid_gup_args() before calling __get_user_pages().
Just adding these VM_WARN_ON_ONCE() in case, e.g internal setting
wrong flags in kernel code. Thanks to David for suggesting this.
- (gup_flags & (FOLL_PIN | FOLL_GET)) == (FOLL_PIN | FOLL_GET))
- !!pages != !!(gup_flags & (FOLL_GET | FOLL_PIN))
- Drop
- the old patch 3 of v2 because it's from misunderstanding;
- the old patch 4, 5 of v2 because they have been merged into x86
tip tree by Ingo;
- the old patch 7 of v2 because it has been coverred in another
thread. Thanks to David for telling the thread link.
- In patch 3, add reviewing tags from Oscar and David.
v1->v2:
==========
- In patch 1, I carelessly copied the fault_in_readable() as
fault_in_writeable(). Thanks to Yanjun for pointing it out.
- In patch 2, I copied the code in follow_page_pte() to
__get_user_pages() directly w/o adjustment which is done but not
merged to patch. That failed testing taken by lkp test robot, thanks
for reporting.
Baoquan He (3):
mm/gup: fix wrongly calculated returned value in
fault_in_safe_writeable()
mm/gup: remove unneeded checking in follow_page_pte()
mm/gup: remove gup_fast_pgd_leaf() and clean up the relevant codes
mm/gup.c | 100 ++++++++++++++++---------------------------------------
1 file changed, 28 insertions(+), 72 deletions(-)
--
2.41.0
Powered by blists - more mailing lists