linux-next: build warning after merge of the ext3 tree

[Stephen Rothwell]

Hi Jan,

After merging the ext3 tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

fs/gfs2/quota.c:1581: warning: initialization from incompatible pointer type

Introduced by commit ec304a27d7750ba666ad2d446434ddd47fa93c40 ("quota:
move code from sync_quota_sb into vfs_quota_sync"). From the commit
message, I expect that this has been thought about.
--
Cheers,
Stephen Rothwell s...@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[Jan Kara]

Hi Stephen,

On Wed 24-02-10 19:05:20, Stephen Rothwell wrote:
> After merging the ext3 tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> fs/gfs2/quota.c:1581: warning: initialization from incompatible pointer type
>
> Introduced by commit ec304a27d7750ba666ad2d446434ddd47fa93c40 ("quota:
> move code from sync_quota_sb into vfs_quota_sync"). From the commit
> message, I expect that this has been thought about.

Thanks for the notification. It seems Christoph missed to convert
gfs2_quota_sync(). I've now fixed it in a minimalistic way (just added
the wait parameter to the function).
Steven (Whitehouse), if GFS2 wants to do something more clever, please
let me know (or fix up in your tree).

Thanks
Honza
--
Jan Kara <ja...@suse.cz>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majo...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

[Jan Kara]

On Wed 24-02-10 11:58:11, Jan Kara wrote:
> Hi Stephen,
>
> On Wed 24-02-10 19:05:20, Stephen Rothwell wrote:
> > After merging the ext3 tree, today's linux-next build (x86_64
> > allmodconfig) produced this warning:
> >
> > fs/gfs2/quota.c:1581: warning: initialization from incompatible pointer type
> >
> > Introduced by commit ec304a27d7750ba666ad2d446434ddd47fa93c40 ("quota:
> > move code from sync_quota_sb into vfs_quota_sync"). From the commit
> > message, I expect that this has been thought about.
> Thanks for the notification. It seems Christoph missed to convert
> gfs2_quota_sync(). I've now fixed it in a minimalistic way (just added
> the wait parameter to the function).
> Steven (Whitehouse), if GFS2 wants to do something more clever, please
> let me know (or fix up in your tree).

And I had to fixup a few internal callers in GFS2 as well. So Steven, you
might want to have a look at the patch (commit
ef9eb614d974943af3c8eab8ea78a30d5132428c in my linux-fs-2.6 tree).