lkml.org 
[lkml]   [2020]   [Oct]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject[RFC PATCH 0/3] hugetlbfs: introduce hinode_rwsem for pmd sharing synchronization
Date
In commit c0d0381ade79, changes were made to use i_mmap_rwsem for pmd
sharing synchronization. This required changes to mm locking order that
are hugetlb specific. Specifically, i_mmap_rwsem must be taken before
the page lock. This is not not a huge issue in hugetlb specific code,
but becomes more problematic in the areas of page migration and memory
failure where generic mm code had to deal with this change to lock
ordering. An ugly routine 'hugetlb_page_mapping_lock_write' was added
to help with these issues.

Recently, Hugh Dickins diagnosed a migration BUG as caused by code
introduced with hugetlb i_mmap_rwsem synchronization [1]. Subsequent
discussion in that thread pointed out additional problems in the code.

Adding a rw_semaphore to the hugetlbfs inode for this type of synchronization
was mentioned. Such an approach is actually 'cleaner' as it can be
inserted in the lock hierarchy where needed. And, there is no issue
with other parts of the mm using this rw_semaphore.

This series adds a rw_semaphore (hinode_rwsem) to the hugetlbfs inode.

The first patch reverts all commits having to deal with the current use
of i_mmap_rwsem for pmd sharing and fault/truncate synchronization. The
revert of 5 commits was combined into a single patch. I am looking for
feedback on this approach. I considered:
- 5 Patches to revert the 5 commits
- Reverting patches depending on c0d0381ade79, then having a patch to
change from i_mmap_rwsem to hinode_rwsem.
To me, a 'clean slate' approach seemed best but I am open to whatever
would be easiest to review.

[1] https://lore.kernel.org/linux-mm/alpine.LSU.2.11.2010071833100.2214@eggly.anvils/

Mike Kravetz (3):
hugetlbfs: revert use of i_mmap_rwsem for pmd sharing and more sync
hugetlbfs: introduce hinode_rwsem for pmd sharing synchronization
huegtlbfs: handle page fault/truncate races

fs/hugetlbfs/inode.c | 72 ++++++------
include/linux/fs.h | 15 ---
include/linux/hugetlb.h | 72 ++++++++++--
mm/hugetlb.c | 243 +++++++++++++++-------------------------
mm/memory-failure.c | 34 ++----
mm/migrate.c | 34 +++---
mm/rmap.c | 17 +--
mm/userfaultfd.c | 14 +--
8 files changed, 220 insertions(+), 281 deletions(-)

--
2.25.4

\
 
 \ /
  Last update: 2020-10-14 01:13    [W:0.080 / U:0.608 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site