On Wed, Nov 28, 2007 at 05:11:09PM -0800, Christoph Lameter wrote:
quoted text
> @@ -2000,11 +2001,13 @@ static int grab_tail_page(struct inode *
>  	/* we want the page with the last byte in the file,
>  	 ** not the page that will hold the next byte for appending
>  	 */
> -	unsigned long index = (p_s_inode->i_size - 1) >> PAGE_CACHE_SHIFT;
> +	unsigned long index = page_cache_index(p_s_inode->i_mapping,
> +						p_s_inode->i_size - 1);
>  	unsigned long pos = 0;
>  	unsigned long start = 0;
>  	unsigned long blocksize = p_s_inode->i_sb->s_blocksize;
> -	unsigned long offset = (p_s_inode->i_size) & (PAGE_CACHE_SIZE - 1);
> +	unsigned long offset = page_cache_index(p_s_inode->i_mapping,
> +							p_s_inode->i_size);

	unsigned long offset = page_cache_offset(p_s_inode->i_mapping,

Cheers,

Dave.
-- 
Dave Chinner
Principal Engineer
SGI Australian Software Group
-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html