Cc: Andrew Morton <akpm@...>, Andreas Dilger <adilger@...>, Amit K. Arora <aarora@...>, <linux-fsdevel@...>, <linux-kernel@...>, <linux-ext4@...>, <xfs@...>, <suparna@...>
On Mon, May 07, 2007 at 05:41:39PM -0700, Mingming Cao wrote:
Checking against the fs free blocks is a good idea, since it will
prevent the obvious error case where someone tries to preallocate 10GB
when there is only 2GB left. But it won't help if there are multiple
processes trying to allocate blocks the same time. On the other hand,
that case is probably relatively rare, and in that case, the
filesystem was probably going to be left completely full in any case.
On Mon, May 07, 2007 at 05:15:41PM -0700, Andrew Morton wrote:
Actually, the kernel could do it, in that could simply release all
unitialized extents back to the system. The problem is distinguishing
between the unitialized extents that had just been newly added, versus
the ones that had there from before. (On the other hand, if the
filesystem was completely full, releasing unitialized blocks wouldn't
be the worse thing in the world to do, although releasing previously
fallocated blocks probably does violate the princple of least
surprise, even if it's what the user would have wanted.)
On Mon, May 07, 2007 at 05:41:39PM -0700, Mingming Cao wrote:
Could you really use a single reservation window? When the filesystem
is almost full, the free extents are likely going to be scattered all
over the disk. The general principle of grabbing all of the extents
and keeping them in an in-memory data structure, and only adding them
to the extent tree would work, though; I'm just not sure we could do
it using the existing reservation window code, since it only supports
a single reservation window per file, yes?
- Ted
-
