From: Christopher Lameter <cl@linux.com>
To: Roman Gushchin <guro@fb.com>
Cc: "Tobin C. Harding" <tobin@kernel.org>,
Andrew Morton <akpm@linux-foundation.org>,
Pekka Enberg <penberg@cs.helsinki.fi>,
Matthew Wilcox <willy@infradead.org>,
Tycho Andersen <tycho@tycho.ws>,
"linux-mm@kvack.org" <linux-mm@kvack.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [RFC 04/15] slub: Enable Slab Movable Objects (SMO)
Date: Tue, 12 Mar 2019 04:39:36 +0000 [thread overview]
Message-ID: <010001697032e074-f9658e7a-595f-4804-a7a0-fd4220ee8473-000000@email.amazonses.com> (raw)
In-Reply-To: <20190311224842.GC7915@tower.DHCP.thefacebook.com>
On Mon, 11 Mar 2019, Roman Gushchin wrote:
> > +static inline void *alloc_scratch(struct kmem_cache *s)
> > +{
> > + unsigned int size = oo_objects(s->max);
> > +
> > + return kmalloc(size * sizeof(void *) +
> > + BITS_TO_LONGS(size) * sizeof(unsigned long),
> > + GFP_KERNEL);
>
> I wonder how big this allocation can be?
> Given that the reason for migration is probably highly fragmented memory,
> we probably don't want to have a high-order allocation here. So maybe
> kvmalloc()?
The smallest object size is 8 bytes which is one word which would be
places in an order 0 page. So it comes out to about a page again.
Larger allocation orders are possible if the slab pages itself can have
larger orders of course. If you set the min_order to the huge page order
then we can have similar sized orders for the allocation of the scratch
space. However, that is not a problem since the allocations for the slab
pages itself are also already of that same order.
next prev parent reply other threads:[~2019-03-12 4:39 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-08 4:14 [RFC 00/15] mm: Implement " Tobin C. Harding
2019-03-08 4:14 ` [RFC 01/15] slub: Create sysfs field /sys/slab/<cache>/ops Tobin C. Harding
2019-03-11 21:23 ` Roman Gushchin
2019-03-12 1:16 ` Tobin C. Harding
2019-03-08 4:14 ` [RFC 02/15] slub: Add isolate() and migrate() methods Tobin C. Harding
2019-03-08 15:28 ` Tycho Andersen
2019-03-08 16:15 ` Christopher Lameter
2019-03-08 16:22 ` Tycho Andersen
2019-03-08 19:53 ` Tobin C. Harding
2019-03-08 20:08 ` Tycho Andersen
2019-03-11 21:51 ` Roman Gushchin
2019-03-12 1:08 ` Tobin C. Harding
2019-03-12 4:35 ` Christopher Lameter
2019-03-12 18:47 ` Roman Gushchin
2019-03-08 4:14 ` [RFC 03/15] tools/vm/slabinfo: Add support for -C and -F options Tobin C. Harding
2019-03-11 21:54 ` Roman Gushchin
2019-03-12 1:20 ` Tobin C. Harding
2019-03-08 4:14 ` [RFC 04/15] slub: Enable Slab Movable Objects (SMO) Tobin C. Harding
2019-03-11 22:48 ` Roman Gushchin
2019-03-12 1:47 ` Tobin C. Harding
2019-03-12 18:00 ` Roman Gushchin
2019-03-12 4:39 ` Christopher Lameter [this message]
2019-03-08 4:14 ` [RFC 05/15] slub: Sort slab cache list Tobin C. Harding
2019-03-08 4:14 ` [RFC 06/15] tools/vm/slabinfo: Add remote node defrag ratio output Tobin C. Harding
2019-03-08 4:14 ` [RFC 07/15] slub: Add defrag_used_ratio field and sysfs support Tobin C. Harding
2019-03-08 16:01 ` Tycho Andersen
2019-03-11 6:04 ` Tobin C. Harding
2019-03-08 4:14 ` [RFC 08/15] tools/vm/slabinfo: Add defrag_used_ratio output Tobin C. Harding
2019-03-08 4:14 ` [RFC 09/15] slub: Enable slab defragmentation using SMO Tobin C. Harding
2019-03-11 23:35 ` Roman Gushchin
2019-03-12 1:49 ` Tobin C. Harding
2019-03-08 4:14 ` [RFC 10/15] tools/testing/slab: Add object migration test module Tobin C. Harding
2019-03-08 4:14 ` [RFC 11/15] tools/testing/slab: Add object migration test suite Tobin C. Harding
2019-03-08 4:14 ` [RFC 12/15] xarray: Implement migration function for objects Tobin C. Harding
2019-03-12 0:16 ` Roman Gushchin
2019-03-12 1:54 ` Tobin C. Harding
2019-03-08 4:14 ` [RFC 13/15] tools/testing/slab: Add XArray movable objects tests Tobin C. Harding
2019-03-08 4:14 ` [RFC 14/15] slub: Enable move _all_ objects to node Tobin C. Harding
2019-03-08 4:14 ` [RFC 15/15] slub: Enable balancing slab objects across nodes Tobin C. Harding
2019-03-12 0:09 ` [RFC 00/15] mm: Implement Slab Movable Objects (SMO) Roman Gushchin
2019-03-12 1:48 ` Tobin C. Harding
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=010001697032e074-f9658e7a-595f-4804-a7a0-fd4220ee8473-000000@email.amazonses.com \
--to=cl@linux.com \
--cc=akpm@linux-foundation.org \
--cc=guro@fb.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=penberg@cs.helsinki.fi \
--cc=tobin@kernel.org \
--cc=tycho@tycho.ws \
--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