xfs
[Top] [All Lists]

Re: [RFC v2 PATCH 1/5] xfs: Prep work for supporting both pquota and gqu

To: Chandra Seetharaman <sekharan@xxxxxxxxxx>
Subject: Re: [RFC v2 PATCH 1/5] xfs: Prep work for supporting both pquota and gquota.
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Thu, 27 Oct 2011 12:17:24 -0400
Cc: xfs@xxxxxxxxxxx
In-reply-to: <20111026212158.6513.17142.sendpatchset@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
References: <20111026212150.6513.38058.sendpatchset@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20111026212158.6513.17142.sendpatchset@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Wed, Oct 26, 2011 at 04:21:58PM -0500, Chandra Seetharaman wrote:
> Define few macros, inline functions, and change interface in
> preparation for allowing pquota and gquota to be used together.

Looks good, but I'd really prefer to split this up further:

 - one patch for adding xfs_inode_dquot
 - one patch for adding XFS_IS_TYPE_QUOTA_ON
   (which I'd rather make an inline and call xfs_this_quota_on)
 - one for passing the mount structure to xfs_sb_from_disk
 - one for the quota flags changes.

All of these also should have a good changelog, most importantly the flag
changes.  If you can repost just these changes relatively soon please do
so, as I have a huge series for the quota code that should go into the
3.2 tree which I need to rebase anyway, and I'd like to do it on top of
your changes.

> -     xfs_sb_t        *to,
> +     xfs_mount_t     *mp,
>       xfs_dsb_t       *from)
>  {
> +     xfs_sb_t        *to = &mp->m_sb;
> +

For code you touch please always use the struct versions instead of
the typedefs for lines you change anyway or add.

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