From: Davidlohr Bueso <dave@stgolabs.net>
To: akpm@linux-foundation.org
Cc: mingo@kernel.org, peterz@infradead.org, jack@suse.cz,
torvalds@linux-foundation.org, kirill.shutemov@linux.intel.com,
hch@infradead.org, ldufour@linux.vnet.ibm.com, mhocko@suse.com,
mgorman@techsingularity.net, dave@stgolabs.net,
linux-kernel@vger.kernel.org
Subject: [PATCH -next v4 00/17] rbtree: cache leftmost node internally
Date: Tue, 18 Jul 2017 18:45:46 -0700 [thread overview]
Message-ID: <20170719014603.19029-1-dave@stgolabs.net> (raw)
Changes from v3 (https://lwn.net/Articles/726809/):
- Updated Documentation/rbtree.txt in patch 1.
- Explicitly mentioned the asymmetry wrt rightmost node in patch 1 (hch).
- Most changes in the set is adding more patches: patch 2,3,4,5,6,15,16,17.
- Added Peter's ack.
Changes from v2 (https://lkml.org/lkml/2017/6/8/857):
- Fixed 0day reported crash for drm_mm selftest program. We were
not correctly using the cached version of rbtree with the allocated
nodes.
- Added cfq patch to use internal rbtree caching.
- Added Christian's and Jan's reviews.
Changes from v1 (https://marc.info/?l=linux-kernel&m=149611025616685):
- No longer rfc.
- Removed bogus semimcolon in rb_first_cached()
- Updated missing interval tree user drivers/infiniband/hw/hfi1/
- Removed redundant @cached arg in when erasing a node.
- Added more patches that make use of rb_first_cached(), which I
thought might be worth it: procfs and epoll.
- Cc more people for patch 5, which touches drivers such as infiniband
and gpu. The rest of the changes are pretty covered with the current
Cc'ed maintainers and mm folks.
Hi,
Here's a proposal for extending rbtrees to internally cache the leftmost
node such that we can have fast overlap check optimization for all interval
tree users[1]. The benefits of this series are that:
(i) Unify users that do internal leftmost node caching.
(ii) Optimize all interval tree users.
(iii) Convert at least two new users (epoll and procfs) to the new interface.
(iv) Enable user rightmost node caching, similar to how we deal with the
leftmost case before this patchset.
Patch 1: Layout the rb machinery.
Patch 2,3: Adds an unlikely() prediction to root-most node for insertions
as well as some minor comment enhancements.
Patches 4-6: Improve and extend rbrtee_test.c program.
Patches 7-10: Make use of the internal leftmost node in scheduler, realtime
mutexes and cfq.
Patch 11: Implements fast overlap checks for interval trees.
Patch 12: rocket science.
Patches 13-15: Patches that convert to O(1) rb_first_cached().
Patches 16,17: Apply rightmost node optimization explicitly by two users.
The series has survived booting, kernel builds and pistress workloads.
Applies on top of today's -next.
Thanks!
Davidlohr Bueso (17):
rbtree: cache leftmost node internally
rbtree: optimize root-check during rebalancing loop
rbtree: add some additional comments for rebalancing cases
lib/rbtree_test.c: make input module parameters
lib/rbtree_test.c: add (inorder) traversal test
lib/rbtree_test.c: support rb_root_cached
sched/fair: replace cfs_rq->rb_leftmost
sched/deadline: replace earliest dl and rq leftmost caching
locking/rtmutex: replace top-waiter and pi_waiters leftmost caching
block/cfq: replace cfq_rb_root leftmost caching
lib/interval_tree: fast overlap detection
lib/interval-tree: correct comment wrt generic flavor
procfs: use faster rb_first_cached()
fs/epoll: use faster rb_first_cached()
fs/ext4: use cached rbtrees
mem/memcg: cache rightmost node
block/cfq: cache rightmost rb_node
Documentation/rbtree.txt | 33 +++
block/cfq-iosched.c | 81 +++-----
drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c | 8 +-
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 7 +-
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h | 2 +-
drivers/gpu/drm/drm_mm.c | 19 +-
drivers/gpu/drm/drm_vma_manager.c | 2 +-
drivers/gpu/drm/i915/i915_gem_userptr.c | 6 +-
drivers/gpu/drm/radeon/radeon.h | 2 +-
drivers/gpu/drm/radeon/radeon_mn.c | 8 +-
drivers/gpu/drm/radeon/radeon_vm.c | 7 +-
drivers/infiniband/core/umem_rbtree.c | 4 +-
drivers/infiniband/core/uverbs_cmd.c | 2 +-
drivers/infiniband/hw/hfi1/mmu_rb.c | 10 +-
drivers/infiniband/hw/usnic/usnic_uiom.c | 6 +-
drivers/infiniband/hw/usnic/usnic_uiom.h | 2 +-
.../infiniband/hw/usnic/usnic_uiom_interval_tree.c | 15 +-
.../infiniband/hw/usnic/usnic_uiom_interval_tree.h | 12 +-
drivers/vhost/vhost.c | 2 +-
drivers/vhost/vhost.h | 2 +-
fs/eventpoll.c | 30 +--
fs/ext4/dir.c | 24 ++-
fs/ext4/ext4.h | 4 +-
fs/ext4/mballoc.c | 22 +-
fs/hugetlbfs/inode.c | 6 +-
fs/inode.c | 2 +-
fs/proc/generic.c | 26 +--
fs/proc/internal.h | 2 +-
fs/proc/proc_net.c | 2 +-
fs/proc/root.c | 2 +-
include/drm/drm_mm.h | 2 +-
include/linux/fs.h | 4 +-
include/linux/init_task.h | 5 +-
include/linux/interval_tree.h | 8 +-
include/linux/interval_tree_generic.h | 48 ++++-
include/linux/mm.h | 17 +-
include/linux/rbtree.h | 21 ++
include/linux/rbtree_augmented.h | 33 ++-
include/linux/rmap.h | 4 +-
include/linux/rtmutex.h | 11 +-
include/linux/sched.h | 3 +-
include/rdma/ib_umem_odp.h | 11 +-
include/rdma/ib_verbs.h | 2 +-
kernel/fork.c | 3 +-
kernel/locking/rtmutex-debug.c | 2 +-
kernel/locking/rtmutex.c | 35 +---
kernel/locking/rtmutex_common.h | 12 +-
kernel/sched/deadline.c | 50 ++---
kernel/sched/debug.c | 2 +-
kernel/sched/fair.c | 39 ++--
kernel/sched/sched.h | 9 +-
lib/interval_tree_test.c | 4 +-
lib/rbtree.c | 65 ++++--
lib/rbtree_test.c | 230 +++++++++++++++++----
mm/interval_tree.c | 10 +-
mm/memcontrol.c | 23 ++-
mm/memory.c | 4 +-
mm/mmap.c | 10 +-
mm/rmap.c | 4 +-
59 files changed, 643 insertions(+), 378 deletions(-)
--
2.12.0
next reply other threads:[~2017-07-19 1:46 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-19 1:45 Davidlohr Bueso [this message]
2017-07-19 1:45 ` [PATCH 01/17] " Davidlohr Bueso
2017-07-19 1:45 ` [PATCH 02/17] rbtree: optimize root-check during rebalancing loop Davidlohr Bueso
2017-07-19 1:45 ` [PATCH 03/17] rbtree: add some additional comments for rebalancing cases Davidlohr Bueso
2017-07-19 1:45 ` [PATCH 04/17] lib/rbtree_test.c: make input module parameters Davidlohr Bueso
2017-07-19 1:45 ` [PATCH 05/17] lib/rbtree_test.c: add (inorder) traversal test Davidlohr Bueso
2017-07-19 1:45 ` [PATCH 06/17] lib/rbtree_test.c: support rb_root_cached Davidlohr Bueso
2017-07-19 1:45 ` [PATCH 07/17] sched/fair: replace cfs_rq->rb_leftmost Davidlohr Bueso
2017-07-19 1:45 ` [PATCH 08/17] sched/deadline: replace earliest dl and rq leftmost caching Davidlohr Bueso
2017-07-19 1:45 ` [PATCH 09/17] locking/rtmutex: replace top-waiter and pi_waiters " Davidlohr Bueso
2017-07-19 1:45 ` [PATCH 10/17] block/cfq: replace cfq_rb_root " Davidlohr Bueso
2017-07-19 7:46 ` Jan Kara
2017-07-19 1:45 ` [PATCH 11/17] lib/interval_tree: fast overlap detection Davidlohr Bueso
2017-07-22 17:52 ` Doug Ledford
2017-08-01 17:16 ` Michael S. Tsirkin
2017-07-19 1:45 ` [PATCH 12/17] lib/interval-tree: correct comment wrt generic flavor Davidlohr Bueso
2017-07-19 1:45 ` [PATCH 13/17] procfs: use faster rb_first_cached() Davidlohr Bueso
2017-07-19 1:46 ` [PATCH 14/17] fs/epoll: " Davidlohr Bueso
2017-07-19 1:46 ` [PATCH 15/17] fs/ext4: use cached rbtrees Davidlohr Bueso
2017-07-19 7:40 ` Jan Kara
2017-07-19 22:50 ` Davidlohr Bueso
2017-07-19 1:46 ` [PATCH 16/17] mem/memcg: cache rightmost node Davidlohr Bueso
2017-07-19 7:50 ` Michal Hocko
2017-07-26 21:09 ` Andrew Morton
2017-07-27 7:06 ` Michal Hocko
2017-07-19 1:46 ` [PATCH 17/17] block/cfq: cache rightmost rb_node Davidlohr Bueso
2017-07-19 7:59 ` Jan Kara
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170719014603.19029-1-dave@stgolabs.net \
--to=dave@stgolabs.net \
--cc=akpm@linux-foundation.org \
--cc=hch@infradead.org \
--cc=jack@suse.cz \
--cc=kirill.shutemov@linux.intel.com \
--cc=ldufour@linux.vnet.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mgorman@techsingularity.net \
--cc=mhocko@suse.com \
--cc=mingo@kernel.org \
--cc=peterz@infradead.org \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®