From: Nick Piggin <nickpiggin@yahoo.com.au>
To: bidulock@openss7.org
Cc: Theodore Tso <tytso@thunk.org>, linux-kernel@vger.kernel.org
Subject: Re: [RFC] [PATCH 0/8] Inode diet v2
Date: Thu, 22 Jun 2006 01:31:47 +1000 [thread overview]
Message-ID: <44996663.80205@yahoo.com.au> (raw)
In-Reply-To: <20060621084217.B7834@openss7.org>
Brian F. G. Bidulock wrote:
> Theodore,
>
> On Wed, 21 Jun 2006, Theodore Tso wrote:
>
>
>>Unfortunately, since these structures are used by a large amount of
>>kernel code, some of the patches are quite involved, and/or will
>>require a lot of auditing and code review, for "only" 4 or 8 bytes at
>>a time (maybe more on 64-bit platforms). However, since there are
>>many, many copies of struct inode all over the kernel, even a small
>>reduction in size can have a large beneficial result, and as the old
>>Chinese saying goes, a journey of thousand miles begins with a single
>>step....
>
>
> Can you grep inode_cache /proc/slabinfo to see whether you saved any
> memory at all?
>
> You need to save 48 bytes per inode to fit one more into a slab with
> a 32 byte L1 cache slot; 120 bytes per inode, 64 byte L1 cache slot.
That would be interesting, but I don't think that is necessary. You
have different sizes of types and pages in different architectures
and configurations. Even if there were no immediate savings anywhere,
the inode diet still seems like a worthwhile investigation.
--
SUSE Labs, Novell Inc.
Send instant messages to your online friends http://au.messenger.yahoo.com
next prev parent reply other threads:[~2006-06-21 15:31 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-06-21 12:51 [RFC] [PATCH 0/8] Inode diet v2 Theodore Tso
2006-06-21 12:51 ` [RFC] [PATCH 1/8] inode_diet: Replace inode.u.generic_ip with inode.i_private Theodore Tso
2006-06-21 13:23 ` Jan Engelhardt
2006-06-21 14:44 ` Brian F. G. Bidulock
2006-06-21 15:37 ` Theodore Tso
2006-06-21 12:51 ` [RFC] [PATCH 2/8] inode-diet: Move i_pipe into a union Theodore Tso
2006-06-21 12:51 ` [RFC] [PATCH 3/8] inode-diet: Move i_bdev " Theodore Tso
2006-06-21 12:51 ` [RFC] [PATCH 4/8] inode-diet: Move i_cdev " Theodore Tso
2006-06-21 12:51 ` [RFC] [PATCH 5/8] inode-diet: Eliminate i_blksize and use a per-superblock default Theodore Tso
2006-06-21 12:51 ` [RFC] [PATCH 6/8] inode-diet: Move i_cindex from struct inode to struct file Theodore Tso
2006-06-21 12:51 ` [RFC] [PATCH 7/8] inode-diet: Use a union for i_blocks and i_size, i_rdev and i_devices Theodore Tso
2006-06-21 12:51 ` [RFC] [PATCH 8/8] inode-diet: Fix size of i_blkbits, i_version, and i_dnotify_mask Theodore Tso
[not found] ` <200606212335.00176.arnd@arndb.de>
2006-06-22 3:11 ` Theodore Tso
2006-06-21 14:42 ` [RFC] [PATCH 0/8] Inode diet v2 Brian F. G. Bidulock
2006-06-21 15:31 ` Nick Piggin [this message]
2006-06-21 16:06 ` Arjan van de Ven
2006-06-21 16:12 ` Brian F. G. Bidulock
2006-06-21 16:48 ` Theodore Tso
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=44996663.80205@yahoo.com.au \
--to=nickpiggin@yahoo.com.au \
--cc=bidulock@openss7.org \
--cc=linux-kernel@vger.kernel.org \
--cc=tytso@thunk.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