Re: [PATCH v2] XFS: Let the broken fiemap work in query mode.

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Wed, Apr 28, 2010 at 11:00:25AM +0800, Tao Ma wrote:
> -     bm.bmv_count = fieinfo->fi_extents_max + 1;
> +     bm.bmv_count = !fieinfo->fi_extents_max ? MAXEXTNUM :
> +                                     fieinfo->fi_extents_max + 1;
> +     bm.bmv_count = MIN(bm.bmv_count,
> +                        (__s32)(PAGE_SIZE * 16 / sizeof(struct getbmapx)));

I would use min_t here instead of the case, but otherwise the patch
looks good to me,


Reviewed-by: Christoph Hellwig <hch@xxxxxx>