linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
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


             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