Messages in this thread | | | Date | Tue, 29 Jun 2010 00:18:18 -0700 | From | Joel Becker <> | Subject | Re: [PATCH] Revert "writeback: limit write_cache_pages integrity scanning to current EOF" |
| |
On Tue, Jun 29, 2010 at 12:27:57PM +1000, Dave Chinner wrote: > Just to clarify any possible misunderstanding here, xfs_zero_eof() > also does it's IO through the page cache for similar reasons. It's > just the mappings are found via the internal interfaces before the > zeroing is done via the anonymous pagecache_write_begin()/ > pagecache_write_end() functions (in xfs_iozero()) rather than using > the generic block functions.
Mark and I discussed this some earlier this evening. I think we might be able to get away cheaper than does. In ocfs2_write_begin_nolock(), we call ocfs2_expand_nonsparse_inode() in the case of older filesystems that don't allow sparse files. That's where we handle the zeroing from old_i_size to pos for those files. I the exact same place, we could probably just detect we're about to cover the unzeroed allocation and get it there. This needs some more code eval until we're sure, and then the serious testing happens.
Joel
--
"Nothing is wrong with California that a rise in the ocean level wouldn't cure." - Ross MacDonald
Joel Becker Consulting Software Developer Oracle E-mail: joel.becker@oracle.com Phone: (650) 506-8127
| |