Messages in this thread | | | Date | Mon, 7 Jan 2002 04:58:57 +0100 | From | Andrea Arcangeli <> | Subject | Re: [patch] truncate fixes |
| |
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/
| |