linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [RFC 0/4] Replace mmap_sem by a range lock
@ 2017-04-19 12:18 Laurent Dufour
  2017-04-19 12:18 ` [RFC 1/4] Add additional range parameter to GUP() and handle_page_fault() Laurent Dufour
                   ` (5 more replies)
  0 siblings, 6 replies; 15+ messages in thread
From: Laurent Dufour @ 2017-04-19 12:18 UTC (permalink / raw)
  To: linux-mm
  Cc: Davidlohr Bueso, akpm, Jan Kara, Kirill A . Shutemov,
	Michal Hocko, Peter Zijlstra, Mel Gorman, haren, aneesh.kumar,
	khandual, Paul.McKenney, linux-kernel

Following the series pushed by Davidlohr Bueso based on the Jan Kara's
work [1] which introduces range locks, this series implements the
first step of the attempt to replace the mmap_sem by a range lock.

While this series simply replaces the mmap_sem by a full range lock,
the final goal is to introduce finer grain locking to allow better
multi-thread performance in regards to the process's memory layout
changes.

This series is currently supports x86 and PowerPc architectures only.
Some drivers are also impacted to allow build and basic test on few
platforms but a lot of additional works is required to complete the
job for all the supported architectures.

The goal of this series is to check that no major performance hit
happens for mono threaded process, as we could hope major improvements
in the multi-threaded case once the finer grain locking is
implemented.

I didn't do massive performance checking yet, but building a full
kernel on a 80 threaded Power node, doesn't show performance hits. The
build time is 11m56.701s on a vanilla kernel and 12m4.679s when range
lock is used.

The next steps will attempt to implement finer grain locking but
specific locking would certainly be required to protect mm data like
the VMA cache.

This series applies on top of 4.11-rc7.

The first patch introduce a new parameter to some memory service which
need to release the lock, since range lock require the range to
specify, caller has to know about it.

The second patch is removing some assert which were based on lock
service which are not provided by the range lock API. While some of
these checks will not be valid with range locks, some may be reviewed.

The third patch is replacing nest locking operation on mmap_sem to
simple locking operation has the nest lock check is not yet provided
by the range lock API.

The latest patch is doing the job of replacing the mmap_sem by a range
lock.

[1] "locking: Introduce range reader/writer lock"
http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1371431.html

Laurent Dufour (4):
  Add additional range parameter to GUP() and handle_page_fault()
  Deactivate mmap_sem assert
  Remove nest locking operation with mmap_sem
  Change mmap_sem to range lock

 arch/powerpc/kernel/vdso.c                         |  8 ++-
 arch/powerpc/kvm/book3s_64_mmu_hv.c                |  6 +-
 arch/powerpc/kvm/book3s_64_mmu_radix.c             |  6 +-
 arch/powerpc/kvm/book3s_64_vio.c                   |  6 +-
 arch/powerpc/kvm/book3s_hv.c                       |  8 ++-
 arch/powerpc/kvm/e500_mmu_host.c                   |  7 ++-
 arch/powerpc/mm/copro_fault.c                      |  8 ++-
 arch/powerpc/mm/fault.c                            | 12 ++--
 arch/powerpc/mm/mmu_context_iommu.c                |  6 +-
 arch/powerpc/mm/subpage-prot.c                     | 16 ++++--
 arch/powerpc/oprofile/cell/spu_task_sync.c         |  8 ++-
 arch/powerpc/platforms/cell/spufs/file.c           |  4 +-
 arch/x86/entry/vdso/vma.c                          | 14 +++--
 arch/x86/events/core.c                             |  1 -
 arch/x86/kernel/tboot.c                            |  2 +-
 arch/x86/kernel/vm86_32.c                          |  6 +-
 arch/x86/mm/fault.c                                | 39 +++++++------
 arch/x86/mm/mpx.c                                  | 20 ++++---
 drivers/android/binder.c                           |  8 ++-
 drivers/firmware/efi/arm-runtime.c                 |  2 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c             |  9 ++-
 drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c            |  8 ++-
 drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c             |  8 ++-
 drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c            |  2 +-
 drivers/gpu/drm/amd/amdkfd/kfd_events.c            |  6 +-
 drivers/gpu/drm/amd/amdkfd/kfd_process.c           |  6 +-
 drivers/gpu/drm/etnaviv/etnaviv_gem.c              |  9 ++-
 drivers/gpu/drm/i915/i915_gem.c                    |  6 +-
 drivers/gpu/drm/i915/i915_gem_userptr.c            | 14 +++--
 drivers/gpu/drm/radeon/radeon_cs.c                 |  9 ++-
 drivers/gpu/drm/radeon/radeon_gem.c                |  8 ++-
 drivers/gpu/drm/radeon/radeon_mn.c                 |  8 ++-
 drivers/gpu/drm/radeon/radeon_ttm.c                |  2 +-
 drivers/gpu/drm/ttm/ttm_bo_vm.c                    |  6 +-
 drivers/gpu/drm/via/via_dmablit.c                  |  6 +-
 drivers/infiniband/core/umem.c                     | 22 +++++---
 drivers/infiniband/core/umem_odp.c                 |  8 ++-
 drivers/infiniband/hw/hfi1/user_pages.c            | 18 ++++--
 drivers/infiniband/hw/mlx4/main.c                  |  6 +-
 drivers/infiniband/hw/mlx5/main.c                  |  6 +-
 drivers/infiniband/hw/mthca/mthca_memfree.c        |  3 +-
 drivers/infiniband/hw/qib/qib_user_pages.c         | 18 ++++--
 drivers/infiniband/hw/usnic/usnic_uiom.c           | 22 +++++---
 drivers/iommu/amd_iommu_v2.c                       |  8 ++-
 drivers/iommu/intel-svm.c                          |  8 ++-
 drivers/media/v4l2-core/videobuf-core.c            |  9 ++-
 drivers/media/v4l2-core/videobuf-dma-contig.c      |  6 +-
 drivers/media/v4l2-core/videobuf-dma-sg.c          |  8 ++-
 drivers/misc/cxl/fault.c                           |  6 +-
 drivers/misc/mic/scif/scif_rma.c                   | 19 ++++---
 drivers/oprofile/buffer_sync.c                     | 14 +++--
 drivers/staging/lustre/lustre/llite/llite_mmap.c   |  4 +-
 drivers/staging/lustre/lustre/llite/vvp_io.c       |  6 +-
 .../interface/vchiq_arm/vchiq_2835_arm.c           |  7 ++-
 .../vc04_services/interface/vchiq_arm/vchiq_arm.c  |  6 +-
 drivers/vfio/vfio_iommu_spapr_tce.c                | 13 +++--
 drivers/vfio/vfio_iommu_type1.c                    | 24 +++++---
 drivers/virt/fsl_hypervisor.c                      |  6 +-
 drivers/xen/gntdev.c                               |  6 +-
 drivers/xen/privcmd.c                              | 14 +++--
 fs/aio.c                                           |  7 ++-
 fs/coredump.c                                      |  6 +-
 fs/exec.c                                          | 26 ++++++---
 fs/proc/base.c                                     | 38 ++++++++-----
 fs/proc/internal.h                                 |  1 +
 fs/proc/task_mmu.c                                 | 30 ++++++----
 fs/proc/task_nommu.c                               | 27 +++++----
 fs/userfaultfd.c                                   | 33 +++++------
 include/linux/huge_mm.h                            |  2 -
 include/linux/hugetlb.h                            |  4 +-
 include/linux/mm.h                                 | 21 ++++---
 include/linux/mm_types.h                           |  3 +-
 include/linux/pagemap.h                            |  8 ++-
 include/linux/userfaultfd_k.h                      |  6 +-
 ipc/shm.c                                          | 13 +++--
 kernel/acct.c                                      |  6 +-
 kernel/events/core.c                               |  6 +-
 kernel/events/uprobes.c                            | 28 ++++++----
 kernel/exit.c                                      | 10 ++--
 kernel/fork.c                                      | 21 ++++---
 kernel/futex.c                                     |  8 ++-
 kernel/sched/fair.c                                |  7 ++-
 kernel/sys.c                                       | 31 ++++++++---
 kernel/trace/trace_output.c                        |  6 +-
 mm/filemap.c                                       |  9 +--
 mm/frame_vector.c                                  |  9 ++-
 mm/gup.c                                           | 65 ++++++++++++----------
 mm/hugetlb.c                                       |  3 +-
 mm/init-mm.c                                       |  2 +-
 mm/internal.h                                      |  3 +-
 mm/khugepaged.c                                    | 59 ++++++++++++--------
 mm/ksm.c                                           | 48 ++++++++++------
 mm/madvise.c                                       | 38 +++++++------
 mm/memcontrol.c                                    | 14 +++--
 mm/memory.c                                        | 43 +++++++-------
 mm/mempolicy.c                                     | 32 +++++++----
 mm/migrate.c                                       | 12 ++--
 mm/mincore.c                                       |  6 +-
 mm/mlock.c                                         | 25 ++++++---
 mm/mmap.c                                          | 51 +++++++++++------
 mm/mmu_notifier.c                                  |  6 +-
 mm/mprotect.c                                      | 21 ++++---
 mm/mremap.c                                        |  6 +-
 mm/msync.c                                         | 10 ++--
 mm/nommu.c                                         | 31 +++++++----
 mm/oom_kill.c                                      |  9 ++-
 mm/pagewalk.c                                      |  3 -
 mm/process_vm_access.c                             |  9 ++-
 mm/shmem.c                                         |  3 +-
 mm/swapfile.c                                      |  8 ++-
 mm/userfaultfd.c                                   | 25 +++++----
 mm/util.c                                          | 15 +++--
 security/tomoyo/domain.c                           |  2 +-
 virt/kvm/async_pf.c                                |  8 ++-
 virt/kvm/kvm_main.c                                | 31 +++++++----
 115 files changed, 960 insertions(+), 555 deletions(-)

-- 
2.7.4

^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2017-04-24 15:48 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-19 12:18 [RFC 0/4] Replace mmap_sem by a range lock Laurent Dufour
2017-04-19 12:18 ` [RFC 1/4] Add additional range parameter to GUP() and handle_page_fault() Laurent Dufour
2017-04-19 12:18 ` [RFC 2/4] Deactivate mmap_sem assert Laurent Dufour
2017-04-19 12:30   ` Peter Zijlstra
2017-04-19 13:45     ` Laurent Dufour
2017-04-19 13:54       ` Peter Zijlstra
2017-04-19 12:18 ` [RFC 3/4] Remove nest locking operation with mmap_sem Laurent Dufour
2017-04-19 12:18 ` [RFC 4/4] Change mmap_sem to range lock Laurent Dufour
2017-04-20 13:33 ` [RFC 0/4] Replace mmap_sem by a " Matthew Wilcox
2017-04-20 14:39   ` Laurent Dufour
2017-04-20 14:28 ` [RFC 4/4] Change mmap_sem to " Laurent Dufour
2017-04-20 14:37   ` Peter Zijlstra
2017-04-20 14:42     ` Laurent Dufour
2017-04-20 23:36   ` Andi Kleen
2017-04-24 15:47     ` Laurent Dufour

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).