On Wed, Nov 28, 2007 at 08:02:01PM -0800, Christoph Lameter wrote: > On Thu, 29 Nov 2007, David Chinner wrote: > > > > 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, > > > > > > Reiserfs: Wrong type of inline function > > Signed-off-by: Christoph Lameter <clameter@xxxxxxx> > > --- > fs/reiserfs/inode.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > Index: mm/fs/reiserfs/inode.c > =================================================================== > --- mm.orig/fs/reiserfs/inode.c 2007-11-28 19:59:41.083133259 -0800 > +++ mm/fs/reiserfs/inode.c 2007-11-28 20:00:23.317882809 -0800 > @@ -2006,7 +2006,7 @@ static int grab_tail_page(struct inode * > unsigned long pos = 0; > unsigned long start = 0; > unsigned long blocksize = p_s_inode->i_sb->s_blocksize; > - unsigned long offset = page_cache_index(p_s_inode->i_mapping, > + unsigned long offset = page_cache_offset(p_s_inode->i_mapping, > p_s_inode->i_size); > struct buffer_head *bh; > struct buffer_head *head; Ok. 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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html