xfs
[Top] [All Lists]

Re: [PATCH 1/3] quota: Add support for ->get_nextdqblk() for VFS quota

To: Jan Kara <jack@xxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx
Subject: Re: [PATCH 1/3] quota: Add support for ->get_nextdqblk() for VFS quota
From: Eric Sandeen <sandeen@xxxxxxxxxxx>
Date: Fri, 19 Feb 2016 11:31:58 -0600
Cc: ocfs2-devel@xxxxxxxxxxxxxx, Mark Fasheh <mfasheh@xxxxxxxx>, Joel Becker <jlbec@xxxxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1454596087-6814-2-git-send-email-jack@xxxxxxx>
References: <1454596087-6814-1-git-send-email-jack@xxxxxxx> <1454596087-6814-2-git-send-email-jack@xxxxxxx>
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:38.0) Gecko/20100101 Thunderbird/38.6.0

On 2/4/16 8:28 AM, Jan Kara wrote:
> Add infrastructure for supporting get_nextdqblk() callback for VFS
> quotas. Translate the operation into a callback to appropriate
> filesystem and consequently to quota format callback.
> 
> Signed-off-by: Jan Kara <jack@xxxxxxx>
> ---
>  fs/ext4/super.c          |  1 +
>  fs/quota/dquot.c         | 39 +++++++++++++++++++++++++++++++++++++++
>  fs/reiserfs/super.c      |  1 +
>  include/linux/quota.h    |  3 +++
>  include/linux/quotaops.h |  3 +++
>  5 files changed, 47 insertions(+)
> 
> diff --git a/fs/ext4/super.c b/fs/ext4/super.c
> index f1b56ff01208..51649f442bf6 100644
> --- a/fs/ext4/super.c
> +++ b/fs/ext4/super.c
> @@ -1100,6 +1100,7 @@ static const struct dquot_operations 
> ext4_quota_operations = {
>       .write_info     = ext4_write_info,
>       .alloc_dquot    = dquot_alloc,
>       .destroy_dquot  = dquot_destroy,
> +     .get_next_id    = dquot_get_next_id,
>  };

Hi Jan, doesn't this also need to set:


+       .get_nextdqblk  = dquot_get_next_dqblk,

in ext4_qctl_sysfile_operations?

Thanks,
-Eric

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