From: Goffredo Baroncelli <kreijack@inwind.it>
To: dsterba@suse.cz, "Tomáš Pružina" <pruzinat@gmail.com>,
linux-btrfs@vger.kernel.org
Subject: Re: Fwd: [suggestion] Add verbose notification about inode-cache rebuild to kernel log
Date: Mon, 12 May 2014 20:18:04 +0200 [thread overview]
Message-ID: <5371105C.5030306@inwind.it> (raw)
In-Reply-To: <20140512143946.GD6917@suse.cz>
Hi David,
On 05/12/2014 04:39 PM, David Sterba wrote:
>> Because most modern hardware is 64 bit (with the exception of ARM ?),
>> > could be make sense to allow btrfs to work without inode_cache only on
>> > 64bit, loosing the possibility to be used on 32 bit system.
>> >
>> > Instead when the inode_cache is used, btrfs still be compatible with
>> > both 32bit and 64bit systems.
> There are 32bit machines in use and inode_cache brings some
> compatibility assurance. Just that I would not recommend it to use by
> default.
>
>> > How often a filesystem is moved between a 64bit and 32bit systems ?
> Not so often I'd say, but for example the block device on a 32bit
> machine can be exported via iscsi to a 64bit machine temporarily, then
> the changes made on the 64bit host can make it unusable.
To be clear, I would like to avoid inode_cache on 64bit machine.
In order to avoid the risk to exhaust the inode on 32bit and to be backward compatible with what already exists, we could add a flag to mkfs.btrfs to be only 64bit compatible and avoid inode_cache.
It it would be faster, I am sure that there are uses cases for that.
The major drawback that I see, is that it would be a code path less tested: I think that the 32bit system[*] would disappear soon
Finally I have a question: it is possible to disable inode_cache ? what means the flag "noinode_cache" ? It means disable the inode cache at all, or only avoid to store on disk the inode cache ?
[*] I means 32bit system were it is reasonable that btrfs would run.
BR
G.Baroncelli
--
gpg @keyserver.linux.it: Goffredo Baroncelli (kreijackATinwind.it>
Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5
next prev parent reply other threads:[~2014-05-12 18:15 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CACHxQ2tAU15z_pUEGBuGqYFR9n_fH9BeyA8=jYryRrb3H=zCYA@mail.gmail.com>
2014-05-08 14:22 ` Fwd: [suggestion] Add verbose notification about inode-cache rebuild to kernel log Tomáš Pružina
2014-05-09 14:51 ` David Sterba
2014-05-09 18:12 ` Goffredo Baroncelli
2014-05-12 14:39 ` David Sterba
2014-05-12 18:18 ` Goffredo Baroncelli [this message]
2014-05-13 4:55 ` Duncan
2014-05-13 14:22 ` David Sterba
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=5371105C.5030306@inwind.it \
--to=kreijack@inwind.it \
--cc=dsterba@suse.cz \
--cc=linux-btrfs@vger.kernel.org \
--cc=pruzinat@gmail.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.