From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kara Subject: Re: [PATCH 2/8] dquot: cleanup inode allocation / freeing routines Date: Wed, 24 Feb 2010 23:39:54 +0100 Message-ID: <20100224223954.GC3097@quack.suse.cz> References: <20100220115101.469826792@bombadil.infradead.org> <20100220115212.577547106@bombadil.infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: jack@suse.cz, linux-fsdevel@vger.kernel.org To: Christoph Hellwig Return-path: Received: from cantor.suse.de ([195.135.220.2]:35563 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758426Ab0BXWjq (ORCPT ); Wed, 24 Feb 2010 17:39:46 -0500 Content-Disposition: inline In-Reply-To: <20100220115212.577547106@bombadil.infradead.org> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Sat 20-02-10 06:51:03, Christoph Hellwig wrote: > Get rid of the alloc_inode and free_inode dquot operations - they are > always called from the filesystem and if a filesystem really needs > their own (which none currently does) it can just call into it's > own routine directly. > > Also get rid of the vfs_dq_alloc/vfs_dq_free wrappers and always > call the lowlevel dquot_alloc_inode / dqout_free_inode routines > directly, which now lose the number argument which is always 1. > > Signed-off-by: Christoph Hellwig Looks good. Merged. Honza -- Jan Kara SUSE Labs, CR