OSDN Git Service

xfs: don't rely on ->total in xfs_alloc_space_available
authorChristoph Hellwig <hch@lst.de>
Mon, 9 Jan 2017 21:39:35 +0000 (13:39 -0800)
committerDarrick J. Wong <darrick.wong@oracle.com>
Mon, 9 Jan 2017 21:45:01 +0000 (13:45 -0800)
commit12ef830198b0d71668eb9b59f9ba69d32951a48a
tree8bec19b1d0a549562dc63251a94b1e1ddbb790b0
parent54fee133ad59c87ab01dd84ab3e9397134b32acb
xfs: don't rely on ->total in xfs_alloc_space_available

->total is a bit of an odd parameter passed down to the low-level
allocator all the way from the high-level callers.  It's supposed to
contain the maximum number of blocks to be allocated for the whole
transaction [1].

But in xfs_iomap_write_allocate we only convert existing delayed
allocations and thus only have a minimal block reservation for the
current transaction, so xfs_alloc_space_available can't use it for
the allocation decisions.  Use the maximum of args->total and the
calculated block requirement to make a decision.  We probably should
get rid of args->total eventually and instead apply ->minleft more
broadly, but that will require some extensive changes all over.

[1] which creates lots of confusion as most callers don't decrement it
once doing a first allocation.  But that's for a separate series.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
fs/xfs/libxfs/xfs_alloc.c