From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kara Subject: Re: [PATCH 3/4] vfs: Make __fsync_super() a static function (version 2) Date: Thu, 23 Apr 2009 22:46:33 +0200 Message-ID: <20090423204633.GB31584@duck.suse.cz> References: <1240498045-14288-1-git-send-email-jack@suse.cz> <1240498045-14288-4-git-send-email-jack@suse.cz> <20090423165205.GA4083@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: LKML , Trond Myklebust , linux-fsdevel@vger.kernel.org, Andrew Morton To: Christoph Hellwig Return-path: Received: from cantor2.suse.de ([195.135.220.15]:58056 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755069AbZDWUqf (ORCPT ); Thu, 23 Apr 2009 16:46:35 -0400 Content-Disposition: inline In-Reply-To: <20090423165205.GA4083@infradead.org> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Thu 23-04-09 12:52:05, Christoph Hellwig wrote: > On Thu, Apr 23, 2009 at 04:47:24PM +0200, Jan Kara wrote: > > -void __fsync_super(struct super_block *sb) > > +static int __fsync_super(struct super_block *sb) > > { > > sync_inodes_sb(sb, 0); > > vfs_dq_sync(sb); > > @@ -284,8 +284,7 @@ void __fsync_super(struct super_block *sb) > > I think you're missing a return somewhere :) Argh, forgot to commit this compile fix ;). Honza -- Jan Kara SUSE Labs, CR