From: Alex Tomas <alex@clusterfs.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: "Theodore Ts'o" <tytso@mit.edu>, Alex Tomas <alex@clusterfs.com>,
"linux-ext4@vger.kernel.org" <linux-ext4@vger.kernel.org>
Subject: Re: booked-page-flag.patch
Date: Thu, 15 Feb 2007 17:03:45 +0300 [thread overview]
Message-ID: <m3odnv4iwu.fsf@bzzz.home.net> (raw)
In-Reply-To: <20070215004504.023a90f0.akpm@linux-foundation.org> (Andrew Morton's message of "Thu\, 15 Feb 2007 00\:45\:04 -0800")
>>>>> Andrew Morton (AM) writes:
AM> Sorry, we're seriously, seriously, seriously short on flags in the page
AM> struct and this patch is going to need one heck of a good case for it to be
AM> acceptable.
AM> Even then, we should put a lot of effort into finding some way of avoiding
AM> adding that page flag. One option might be to add a new radix-tree tag,
AM> and defining it as "for filesytem usage". Or use PG_checked (which should
AM> be renamed to to PG_fs_misc) (if that doesn't conflict with ext4's existing
AM> use of PG_checked). Or use !PageMappedToDisk()?
there is a difference between being mapped and "booked".
the latter means that page isn't allocated yet, but
space is reserved (including metadata) and we're sure
we'll be able to allocate space when the page is being
flushed.
AM> These patches seem to have a number of issues - we should get them properly
AM> commented and properly changelogged then get them on the wire for decent
AM> review before investing too much in them, please.
I've been reworking the delayed allocation patch to move
part of it into VFS. I'll try to comment things better
this time.
thanks, Alex
next prev parent reply other threads:[~2007-02-15 21:11 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-15 8:45 booked-page-flag.patch Andrew Morton
2007-02-15 14:03 ` Alex Tomas [this message]
2007-02-15 17:18 ` booked-page-flag.patch Eric Sandeen
2007-02-15 17:30 ` booked-page-flag.patch Alex Tomas
2007-02-15 20:56 ` booked-page-flag.patch Andrew Morton
2007-02-15 21:07 ` booked-page-flag.patch Alex Tomas
2007-02-15 23:23 ` booked-page-flag.patch Andrew Morton
2007-02-16 7:30 ` booked-page-flag.patch Alex Tomas
2007-02-16 7:46 ` booked-page-flag.patch Andrew Morton
2007-02-16 7:56 ` booked-page-flag.patch Alex Tomas
2007-02-16 8:06 ` booked-page-flag.patch Andrew Morton
2007-02-16 12:14 ` booked-page-flag.patch Andreas Dilger
2007-02-16 16:02 ` booked-page-flag.patch Dave Kleikamp
2007-02-16 15:03 ` booked-page-flag.patch Theodore Tso
2007-02-15 20:21 ` booked-page-flag.patch Andrew Morton
2007-02-16 12:17 ` booked-page-flag.patch Andreas Dilger
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=m3odnv4iwu.fsf@bzzz.home.net \
--to=alex@clusterfs.com \
--cc=akpm@linux-foundation.org \
--cc=linux-ext4@vger.kernel.org \
--cc=tytso@mit.edu \
/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.