linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Nick Piggin <npiggin@suse.de>
To: Alexey Starikovskiy <aystarik@gmail.com>
Cc: Christoph Lameter <cl@linux-foundation.org>,
	Pekka Enberg <penberg@cs.helsinki.fi>,
	Linux Memory Management List <linux-mm@kvack.org>,
	linux-acpi@vger.kernel.org, lenb@kernel.org
Subject: Re: [patch][rfc] acpi: do not use kmem caches
Date: Mon, 1 Dec 2008 18:12:19 +0100	[thread overview]
Message-ID: <20081201171219.GI10790@wotan.suse.de> (raw)
In-Reply-To: <49341915.5000900@gmail.com>

On Mon, Dec 01, 2008 at 08:04:21PM +0300, Alexey Starikovskiy wrote:
> Nick Piggin wrote:
> >Hmm.
> >Acpi-Operand        2641   2773     64   59    1 : tunables  120   60    8 
> >: slabdata     47     47     0
> >Acpi-ParseExt          0      0     64   59    1 : tunables  120   60    8 
> >: slabdata      0      0     0
> >Acpi-Parse             0      0     40   92    1 : tunables  120   60    8 
> >: slabdata      0      0     0
> >Acpi-State             0      0     80   48    1 : tunables  120   60    8 
> >: slabdata      0      0     0
> >Acpi-Namespace      1711   1792     32  112    1 : tunables  120   60    8 
> >: slabdata     16     16     0
> >
> >  
> >Looks different for my thinkpad.
> >  
> Probably this is SLUB vs. SLAB thing Pecca was talking about...

Sizes should not be bigger with SLUB. Although if you have SLUB debugging
turned on then maybe the size gets padded with redzones, but in that
configuration you don't expect memory saving anyway because padding bloats
things up.


> And, probably you run at 32-bit? This is part of my .config:

No, 64 bit.


> --------------------------------------------
> CONFIG_SLUB_DEBUG=y
> # CONFIG_SLAB is not set
> CONFIG_SLUB=y
> # CONFIG_SLOB is not set
> -------------------------------------------
> 
> With your patch you would be able to save 64*(2773 - 2641) + 32 * 
> (1792-1711)= 8448 + 2592 = 11040 bytes of memory, less than 3 pages?

You don't account the cost of the kmem cache. Or fragmentation that
can be caused with extra kmem caches.  I guess neither is any problem
with SLOB, which is used by tiny systems...

 
> 2856 * (96-72) = 68544 bytes and 170 * (64-48) = 2720 bytes, so you will be 
> wasting 5 times more memory in 64 bit case.

With debugging on, in which case you're wasting memory anyway.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2008-12-01 17:12 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-01  8:31 Nick Piggin
2008-12-01 11:18 ` Pekka Enberg
2008-12-01 12:00   ` Nick Piggin
2008-12-01 13:12     ` Alexey Starikovskiy
2008-12-01 13:36       ` Nick Piggin
2008-12-01 14:14         ` Alexey Starikovskiy
2008-12-01 16:32           ` Nick Piggin
2008-12-01 17:18           ` Christoph Hellwig
2008-12-01 17:32             ` Alexey Starikovskiy
2008-12-01 13:37       ` Pekka Enberg
2008-12-01 14:02         ` Alexey Starikovskiy
2008-12-01 16:14           ` Nick Piggin
2008-12-01 16:45             ` Alexey Starikovskiy
2008-12-01 16:58               ` Nick Piggin
2008-12-01 17:20               ` Moore, Robert
2008-12-01 17:30                 ` Andi Kleen
2008-12-01 17:32                   ` Moore, Robert
2008-12-01 17:20               ` Christoph Hellwig
2008-12-01 17:49                 ` Alexey Starikovskiy
2008-12-01 17:53                 ` Len Brown
2008-12-01 18:10                   ` Nick Piggin
2008-12-31 22:04                     ` Len Brown
2009-01-05  4:14                       ` Nick Piggin
2009-01-05  5:43                         ` Skywing
2009-01-05  6:55                           ` Nick Piggin
2008-12-01 14:32         ` Christoph Lameter
2008-12-01 14:48           ` Alexey Starikovskiy
2008-12-01 16:20             ` Nick Piggin
2008-12-01 17:04               ` Alexey Starikovskiy
2008-12-01 17:12                 ` Nick Piggin [this message]
2008-12-01 17:25                   ` Pekka Enberg
2008-12-01 17:32                     ` Pekka Enberg
2008-12-01 17:36                       ` Alexey Starikovskiy
2008-12-01 17:48                         ` Pekka Enberg
2008-12-01 18:09                         ` Christoph Lameter
2008-12-01 17:43                   ` Alexey Starikovskiy
2008-12-01 17:31 ` Len Brown

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=20081201171219.GI10790@wotan.suse.de \
    --to=npiggin@suse.de \
    --cc=aystarik@gmail.com \
    --cc=cl@linux-foundation.org \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=penberg@cs.helsinki.fi \
    /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