[PATCH 04/25] xfs: factor extent map manipulations out of xfs_bmapi
Alex Elder
aelder at sgi.com
Fri Sep 2 17:23:11 CDT 2011
On Wed, 2011-08-24 at 02:04 -0400, Christoph Hellwig wrote:
Unfortunately the content of the new xfs_bmapi_update_map()
is as hideous as it was before, but at least it's isolated.
The -1 index values are just awful, even if we do know from
the logic that they're indexing non-negative offsets into the
underlying array. I do think it's good that you encapsulated
this without changing the existing logic, but someday maybe
this function can be improved.
Reviewed-by: Alex Elder <aelder at sgi.com>
You included no signoff. I am going to assume it included
this like the rest of the series:
Signed-off-by: Christoph Hellwig <hch at lst.de>
More information about the xfs
mailing list