On Fri, Aug 25, 2017 at 11:32:21AM +0300, Nikolay Borisov wrote: > > > On 24.08.2017 17:43, Nikolay Borisov wrote: > > > > > > On 22.08.2017 23:00, jo...@toxicpanda.com wrote: > >> From: Josef Bacik <jba...@fb.com> > >> > >> Nikolay reported that generic/273 was failing currently with ENOSPC. > >> Turns out this is because we get to the point where the outstanding > >> reservations are greater than the pinned space on the fs. This is a > >> mistake, previously we used the current reservation amount in > >> may_commit_transaction, not the entire outstanding reservation amount. > >> Fix this to find the minimum byte size needed to make progress in > >> flushing, and pass that into may_commit_transaction. From there we can > >> make a smarter decision on whether to commit the transaction or not. > >> This fixes the failure in generic/273. > >> > >> Reported-by: Nikolay Borisov <nbori...@suse.com> > >> Signed-off-by: Josef Bacik <jba...@fb.com> > > > > Reviewed-and-tested-by: Nikolay Borisov <nbori...@suse.com> > > > For this commit we might also add: > > Fixes: 957780eb2788 ("Btrfs: introduce ticketed enospc infrastructure") > Cc: sta...@vger.kernel.org # 4.8
JFI, tags added plus some additional explanation from N. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html