On Mon, Feb 2, 2009 at 9:40 PM, Jan Kara <jack@xxxxxxx> wrote: >> On Wed, Jan 28, 2009 at 10:47 AM, Manish Katiyar <mkatiyar@xxxxxxxxx> wrote: >> > Below patch removes the following warning during compilation : >> > >> > CC [M] fs/ext4/super.o >> > fs/ext4/super.c:949: warning: initialization from incompatible pointer type >> > LD [M] fs/ext4/ext4.o >> >> Sorry, >> >> Tha correct patch is here. >> >> Signed-off-by: Manish Katiyar <mkatiyar@xxxxxxxxx> > Looks fine to me. > Acked-by: Jan Kara <jack@xxxxxxx> Hi Jan, A simple doubt. Patches acked by you will be automatically picked up or will I have to resend it with your ack added ?? Thanks - Manish > > Honza > >> --- >> fs/ext4/ext4.h | 2 +- >> fs/ext4/inode.c | 4 ++-- >> 2 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h >> index 3ab1a41..0164866 100644 >> --- a/fs/ext4/ext4.h >> +++ b/fs/ext4/ext4.h >> @@ -1088,7 +1088,7 @@ extern int ext4_chunk_trans_blocks(struct inode >> *, int nrblocks); >> extern int ext4_block_truncate_page(handle_t *handle, >> struct address_space *mapping, loff_t from); >> extern int ext4_page_mkwrite(struct vm_area_struct *vma, struct page *page); >> -extern unsigned long long ext4_get_reserved_space(struct inode *inode); >> +extern qsize_t ext4_get_reserved_space(struct inode *inode); >> >> /* ioctl.c */ >> extern long ext4_ioctl(struct file *, unsigned int, unsigned long); >> diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c >> index 6a316ec..6be09be 100644 >> --- a/fs/ext4/inode.c >> +++ b/fs/ext4/inode.c >> @@ -973,9 +973,9 @@ out: >> return err; >> } >> >> -unsigned long long ext4_get_reserved_space(struct inode *inode) >> +qsize_t ext4_get_reserved_space(struct inode *inode) >> { >> - unsigned long long total; >> + qsize_t total; >> >> spin_lock(&EXT4_I(inode)->i_block_reservation_lock); >> total = EXT4_I(inode)->i_reserved_data_blocks + >> -- >> 1.5.4.3 >> >> Thanks - >> Manish >> >> > >> > >> > Signed-off-by: Manish Katiyar <mkatiyar@xxxxxxxxx> >> > --- >> > fs/ext4/ext4.h | 2 +- >> > 1 files changed, 1 insertions(+), 1 deletions(-) >> > >> > diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h >> > index 3ab1a41..0164866 100644 >> > --- a/fs/ext4/ext4.h >> > +++ b/fs/ext4/ext4.h >> > @@ -1088,7 +1088,7 @@ extern int ext4_chunk_trans_blocks(struct inode >> > *, int nrblocks); >> > extern int ext4_block_truncate_page(handle_t *handle, >> > struct address_space *mapping, loff_t from); >> > extern int ext4_page_mkwrite(struct vm_area_struct *vma, struct page *page); >> > -extern unsigned long long ext4_get_reserved_space(struct inode *inode); >> > +extern qsize_t ext4_get_reserved_space(struct inode *inode); >> > >> > /* ioctl.c */ >> > extern long ext4_ioctl(struct file *, unsigned int, unsigned long); >> > -- >> > 1.5.4.3 >> > >> > >> > Thanks - >> > Manish >> > >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-ext4" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > Jan Kara <jack@xxxxxxx> > SuSE CR Labs > -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html