From: "Tobin C. Harding" <tobin@kernel.org>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: "Tobin C. Harding" <tobin@kernel.org>,
Roman Gushchin <guro@fb.com>, Christoph Lameter <cl@linux.com>,
Pekka Enberg <penberg@kernel.org>,
David Rientjes <rientjes@google.com>,
Joonsoo Kim <iamjoonsoo.kim@lge.com>,
Matthew Wilcox <willy@infradead.org>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: [PATCH v2 0/5]] mm: Use slab_list list_head instead of lru
Date: Wed, 13 Mar 2019 16:20:25 +1100 [thread overview]
Message-ID: <20190313052030.13392-1-tobin@kernel.org> (raw)
Currently the slab allocators (ab)use the struct page 'lru' list_head.
We have a list head for slab allocators to use, 'slab_list'.
Clean up all three allocators by using the 'slab_list' list_head instead
of overloading the 'lru' list_head.
Patch 1 - Makes no code changes, adds comments to #endif statements.
Patches 2,3,4 - Do changes as a patch per allocator, tested by building
and booting (in Qemu) after configuring kernel to use
appropriate allocator. Also build and boot with debug
options enabled (for slab and slub). Verify the object
files (before and after the set applied) are the same.
Patch 5 - Removes the now stale comment in the page struct definition.
Changes since v1:
- Verify object files are the same before and after the patch set is
applied (suggested by Matthew).
- Add extra explanation to the commit logs explaining why these changes
are safe to make (suggested by Roman).
- Remove stale comment (thanks Willy).
thanks,
Tobin.
Tobin C. Harding (5):
slub: Add comments to endif pre-processor macros
slub: Use slab_list instead of lru
slab: Use slab_list instead of lru
slob: Use slab_list instead of lru
mm: Remove stale comment from page struct
include/linux/mm_types.h | 2 +-
mm/slab.c | 49 ++++++++++++++++----------------
mm/slob.c | 10 +++----
mm/slub.c | 60 ++++++++++++++++++++--------------------
4 files changed, 61 insertions(+), 60 deletions(-)
--
2.21.0
next reply other threads:[~2019-03-13 5:21 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-13 5:20 Tobin C. Harding [this message]
2019-03-13 5:20 ` [PATCH v2 1/5] slub: Add comments to endif pre-processor macros Tobin C. Harding
2019-03-13 19:00 ` Christopher Lameter
2019-03-13 5:20 ` [PATCH v2 2/5] slub: Use slab_list instead of lru Tobin C. Harding
2019-03-13 19:02 ` Christopher Lameter
2019-03-13 5:20 ` [PATCH v2 3/5] slab: " Tobin C. Harding
2019-03-13 5:20 ` [PATCH v2 4/5] slob: " Tobin C. Harding
2019-03-13 19:05 ` Christopher Lameter
2019-03-14 3:24 ` Tobin C. Harding
2019-03-13 5:20 ` [PATCH v2 5/5] mm: Remove stale comment from page struct Tobin C. Harding
2019-03-13 19:05 ` Christopher Lameter
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=20190313052030.13392-1-tobin@kernel.org \
--to=tobin@kernel.org \
--cc=akpm@linux-foundation.org \
--cc=cl@linux.com \
--cc=guro@fb.com \
--cc=iamjoonsoo.kim@lge.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=penberg@kernel.org \
--cc=rientjes@google.com \
--cc=willy@infradead.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