From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH 1/2] quota: merge __dquot_transfer into dquot_transfer Date: Mon, 8 Mar 2010 12:13:08 -0500 Message-ID: <20100308171308.GA28427@infradead.org> References: <20100306131753.GA28278@infradead.org> <20100308171133.GF5644@quack.suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Christoph Hellwig , linux-fsdevel@vger.kernel.org, dmonakhov@openvz.org To: Jan Kara Return-path: Received: from bombadil.infradead.org ([18.85.46.34]:47276 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752565Ab0CHRNK (ORCPT ); Mon, 8 Mar 2010 12:13:10 -0500 Content-Disposition: inline In-Reply-To: <20100308171133.GF5644@quack.suse.cz> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Mon, Mar 08, 2010 at 06:11:34PM +0100, Jan Kara wrote: > On Sat 06-03-10 08:17:53, Christoph Hellwig wrote: > > Signed-off-by: Christoph Hellwig > Christoph, I see that you have removed the 'mask' argument of > dquot_transfer. Actually, this has been added a short time ago as > a preparation for the project_id patches of Dmitry. Since his patches > seem to be getting in a good shape, I'd prefer not to make his life > harder and leave __dquot_transfer as is for now so that he can hook > into it. Dmitri will have to add code to grab the project ID somewhere. His patch made that easier when the two functions were split over an indirect call boundary, but now that this is gone it's not needed anymore. One line still needs to be added, and that's in the ne dquot_transfer.