Re: [PATCH 2/8] xfs: include reservations in quota reporting

Ben Myers
2012-02-29T19:37:52Z

On Sun, Feb 19, 2012 at 09:28:17PM -0500, Christoph Hellwig wrote:
> Report all quota usage including the currently pending reservations. This
> avoids the need to flush delalloc space before gathering quota information,
> and matches quota enforcement, which already takes the reservations into
> account.
>
> This fixes xfstests 270.
>
> Reviewed-by: Dave Chinner <dchinner@redhat.com>
> Signed-off-by: Christoph Hellwig <hch@lst.de>

I'm a little concerned about large speculative delay block reservations
with this code, but that remains to be seen.

Looks good.

Reviewed-by: Ben Myers <bpm@sgi.com>