xfs
[Top] [All Lists]

Re: [PATCH 0/17 v3] quota: Unify VFS and XFS quota interfaces

To: Jan Kara <jack@xxxxxxx>
Subject: Re: [PATCH 0/17 v3] quota: Unify VFS and XFS quota interfaces
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Thu, 22 Jan 2015 08:38:26 +1100
Cc: linux-fsdevel@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, cluster-devel@xxxxxxxxxx, ocfs2-devel@xxxxxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1421412471-4747-1-git-send-email-jack@xxxxxxx>
References: <1421412471-4747-1-git-send-email-jack@xxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Fri, Jan 16, 2015 at 01:47:34PM +0100, Jan Kara wrote:
>   Hello,
> 
>   this is another iteration of patches to unify VFS and XFS quota interfaces 
> so
> that XFS quotactls work for filesystems with VFS quotas and vice versa.  This
> is useful so that userspace doesn't have to care that much about which
> filesystem it is using at least when using basic quota functionality. In
> particular we should be able to reuse project quota tests in xfstests for 
> ext4.
> 
> The patches are based on quota cleanup series [1] and XFS cleanup series [2]
> I've sent previously and based on 3.19-rc4 + for-next branch of XFS tree.  I
> could further split this series into per-quotactl series but frankly I don't
> think it will help reviewers in any way (the patches in the current series are
> already ordered by quotactl).

Hi Jan,

Do you have a this series in a git tree somewhere so it's easy to
merge and test?

> All the comments to the first version of the series are hopefully addressed,
> except for Dave's request to reduce number of copying of dquot information -
> I have benchmarked that reducing number of copies from 3 to 2 brings just 2%
> improvement in speed in my test setup and getting quota information isn't IMHO
> so performance critical that it would be worth the complications of the code.

I think the numbers address my concern adequately ;)

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>