From: Andreas Dilger <adilger@turbolinux.com>
To: Andrew Morton <andrewm@uow.edu.au>
Cc: "Stephen C. Tweedie" <sct@redhat.com>,
Manas Garg <mls@chakpak.net>,
linux-kernel@vger.kernel.org
Subject: Re: O_TRUNC problem on a full filesystem
Date: Thu, 24 May 2001 11:24:10 -0600 (MDT) [thread overview]
Message-ID: <200105241724.f4OHOAhQ014259@webber.adilger.int> (raw)
In-Reply-To: <3B0CF068.A6ADA562@uow.edu.au> "from Andrew Morton at May 24, 2001 09:28:40 pm"
Andrew writes:
> "Stephen C. Tweedie" wrote:
> > On Wed, May 23, 2001 at 07:55:48PM +1000, Andrew Morton wrote:
> > > When you truncated your file, the blocks remained preallocated
> > > on behalf of the file, and were hence considered "used". For
> > > some reason, a subsequent attempt to allocate blocks for the
> > > same file failed to use that file's preallocated blocks.
> >
> > Nope. ext2_truncate() calls ext2_discard_prealloc() to fix this up.
> > Both 2.2 and 2.4 do this correctly.
>
> But the problem goes away when you disable EXT2_PREALLOCATE.
> I tested it.
Are you sure that a truncated file will re-use the same truncated blocks,
but not the preallocated ones? I can imagine not re-using all of the data
blocks within a single transaction, but it would be odd if the preallocated
blocks are treated differently.
How have you done the ext3 preallocation code? One way to do it would be
to only mark the blocks as used in the in-memory copy of the block bitmap
and not write that to disk (we keep 2 copies of the block bitmap, IIRC).
That way you don't need to do anything fancy at recovery time.
Did you ever benchmark ext2 with and without preallocation to see if it
made any difference? No point in doing extra work if there is no benefit.
Cheers, Andreas
--
Andreas Dilger \ "If a man ate a pound of pasta and a pound of antipasto,
\ would they cancel out, leaving him still hungry?"
http://www-mddsp.enel.ucalgary.ca/People/adilger/ -- Dogbert
next prev parent reply other threads:[~2001-05-24 17:24 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-05-23 6:13 Manas Garg
2001-05-23 9:16 ` OT: " Helge Hafting
2001-05-23 9:55 ` Andrew Morton
2001-05-24 11:16 ` Stephen C. Tweedie
2001-05-24 11:28 ` Andrew Morton
2001-05-24 17:24 ` Andreas Dilger [this message]
2001-05-24 18:15 ` Stephen C. Tweedie
2001-05-24 20:26 ` Andreas Dilger
2001-05-25 0:24 ` Andrew Morton
2001-05-25 9:42 ` Stephen C. Tweedie
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=200105241724.f4OHOAhQ014259@webber.adilger.int \
--to=adilger@turbolinux.com \
--cc=andrewm@uow.edu.au \
--cc=linux-kernel@vger.kernel.org \
--cc=mls@chakpak.net \
--cc=sct@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®