Re: [patch] truncate fixes

From: Andrea Arcangeli (andrea@suse.de)
Date: Sun Jan 06 2002 - 22:58:57 EST


On Sun, Jan 06, 2002 at 07:11:45PM -0800, Andrew Morton wrote:
> Andrea Arcangeli wrote:
> >
> > I prefer my fix that simply recalls the ->truncate callback if -ENOSPC
> > is returned by prepare_write. vmtruncate seems way overkill,
>
> Actually, vmtruncate will trim the page off the inode as well as the
> blocks from the file. I don't think there's necessarily a problem
> with having a page wholly outside i_size, but..

agreed, a truncate_inode_pages before ->truncate is needed, or after an
extension the old bh could be still there mapped to random in the
leftover page. But still the whole vmtruncate looks overkill, there
cannot be vm mappings in the way of our truncate because in
prepare_write we worked after the i_size if truncate will later do
something. pages in the middle if i_size cannot be mapped either, so if
it was a partial write that still couldn't be mapped.

Andrea
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Mon Jan 07 2002 - 21:00:33 EST