]> git.baikalelectronics.ru Git - kernel.git/commit
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)
commitc48d8c0e9f20c6aadd16ee5abc81cb3c8fc1d5f8
tree8bec19b1d0a549562dc63251a94b1e1ddbb790b0
parentad930e77771ffa11bf5f9ba0f05e4db073f8671d
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