[PATCH] xfs: fix the comment explaining xfs_trans_dqlockedjoin

[PATCH] xfs: fix the comment explaining xfs_trans_dqlockedjoin

[Christoph Hellwig]

Signed-off-by: Christoph Hellwig <hch@lst.de>

diff --git a/fs/xfs/xfs_trans_dquot.c b/fs/xfs/xfs_trans_dquot.c
index cd2a10e..4117286 100644
--- a/fs/xfs/xfs_trans_dquot.c
+++ b/fs/xfs/xfs_trans_dquot.c
@@ -295,8 +295,8 @@ xfs_trans_mod_dquot(
 /*
  * Given an array of dqtrx structures, lock all the dquots associated and join
  * them to the transaction, provided they have been modified.  We know that the
- * highest number of dquots of one type - usr, grp OR prj - involved in a
- * transaction is 2 so we don't need to make this very generic.
+ * highest number of dquots of one type - usr, grp and prj - involved in a
+ * transaction is 3 so we don't need to make this very generic.
  */
 STATIC void
 xfs_trans_dqlockedjoin(

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

Re: [PATCH] xfs: fix the comment explaining xfs_trans_dqlockedjoin

[Carlos Maiolino]

Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>

On Wed, Nov 06, 2013 at 03:45:36AM -0800, Christoph Hellwig wrote:
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> 
> diff --git a/fs/xfs/xfs_trans_dquot.c b/fs/xfs/xfs_trans_dquot.c
> index cd2a10e..4117286 100644
> --- a/fs/xfs/xfs_trans_dquot.c
> +++ b/fs/xfs/xfs_trans_dquot.c
> @@ -295,8 +295,8 @@ xfs_trans_mod_dquot(
>  /*
>   * Given an array of dqtrx structures, lock all the dquots associated and join
>   * them to the transaction, provided they have been modified.  We know that the
> - * highest number of dquots of one type - usr, grp OR prj - involved in a
> - * transaction is 2 so we don't need to make this very generic.
> + * highest number of dquots of one type - usr, grp and prj - involved in a
> + * transaction is 3 so we don't need to make this very generic.
>   */
>  STATIC void
>  xfs_trans_dqlockedjoin(
> 
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs

-- 
Carlos

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

Re: [PATCH] xfs: fix the comment explaining xfs_trans_dqlockedjoin

[Christoph Hellwig]

Not the most important patch ever, but I think it should still be picked
up :)

On Wed, Nov 06, 2013 at 03:45:36AM -0800, Christoph Hellwig wrote:
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> 
> diff --git a/fs/xfs/xfs_trans_dquot.c b/fs/xfs/xfs_trans_dquot.c
> index cd2a10e..4117286 100644
> --- a/fs/xfs/xfs_trans_dquot.c
> +++ b/fs/xfs/xfs_trans_dquot.c
> @@ -295,8 +295,8 @@ xfs_trans_mod_dquot(
>  /*
>   * Given an array of dqtrx structures, lock all the dquots associated and join
>   * them to the transaction, provided they have been modified.  We know that the
> - * highest number of dquots of one type - usr, grp OR prj - involved in a
> - * transaction is 2 so we don't need to make this very generic.
> + * highest number of dquots of one type - usr, grp and prj - involved in a
> + * transaction is 3 so we don't need to make this very generic.
>   */
>  STATIC void
>  xfs_trans_dqlockedjoin(
> 
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs
---end quoted text---

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

Re: [PATCH] xfs: fix the comment explaining xfs_trans_dqlockedjoin

[Ben Myers]

On Fri, Nov 15, 2013 at 09:34:15AM -0800, Christoph Hellwig wrote:
> Not the most important patch ever, but I think it should still be picked
> up :)

Sure.  We'll get it in once the merge window closes.

-Ben

> On Wed, Nov 06, 2013 at 03:45:36AM -0800, Christoph Hellwig wrote:
> > Signed-off-by: Christoph Hellwig <hch@lst.de>
> > 
> > diff --git a/fs/xfs/xfs_trans_dquot.c b/fs/xfs/xfs_trans_dquot.c
> > index cd2a10e..4117286 100644
> > --- a/fs/xfs/xfs_trans_dquot.c
> > +++ b/fs/xfs/xfs_trans_dquot.c
> > @@ -295,8 +295,8 @@ xfs_trans_mod_dquot(
> >  /*
> >   * Given an array of dqtrx structures, lock all the dquots associated and join
> >   * them to the transaction, provided they have been modified.  We know that the
> > - * highest number of dquots of one type - usr, grp OR prj - involved in a
> > - * transaction is 2 so we don't need to make this very generic.
> > + * highest number of dquots of one type - usr, grp and prj - involved in a
> > + * transaction is 3 so we don't need to make this very generic.
> >   */
> >  STATIC void
> >  xfs_trans_dqlockedjoin(
> > 
> > _______________________________________________
> > xfs mailing list
> > xfs@oss.sgi.com
> > http://oss.sgi.com/mailman/listinfo/xfs
> ---end quoted text---
> 
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

Re: [PATCH] xfs: fix the comment explaining xfs_trans_dqlockedjoin

[Ben Myers]

On Wed, Nov 06, 2013 at 03:45:36AM -0800, Christoph Hellwig wrote:
> Signed-off-by: Christoph Hellwig <hch@lst.de>

Applied this.

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs