Revert "Btrfs: don't delay inode ref updates during log, replay"
authorJeff Mahoney <jeffm@suse.com>
Sat, 3 Dec 2016 03:21:55 +0000 (22:21 -0500)
committerWilly Tarreau <w@1wt.eu>
Mon, 19 Jun 2017 18:28:14 +0000 (20:28 +0200)
commit0ee88216358c4a1821022763cb84d2b0550bf1e2
tree5cd2672ec44b5f745a66da757d5a68a01a47e0ec
parenta07ea939d7e0406e97739c18e2db6b402eb04cdc
Revert "Btrfs: don't delay inode ref updates during log, replay"

commit 081fafddc3ff1e86e36024b0177c08e340b19a12 upstream.

This reverts commit 644d10716875b24388680925d6c7502420987bfe, upstream
commit 6f8960541b1eb6054a642da48daae2320fddba93.

The original patch for mainline, 6f8960541b1 (Btrfs: don't delay
inode ref updates during log replay) lists 1d52c78afbb (Btrfs: try
not to ENOSPC on log replay) as the only pre-3.18 dependency, but it
also depends on 67de11769bd (Btrfs: introduce the delayed inode ref
deletion for the single link inode), which was introduced in 3.14
and isn't in 3.12.y.

The -stable commit added the check to btrfs_delayed_update_inode,
which may look similar to btrfs_delayed_delete_inode_ref, but it's
only superficial.  The tops of both functions handle typical
delayed node boilerplate.  The upshot is that the patch is harmless
since the caller already checks to see if we're doing log recovery,
so we're not breaking anything.  It should be reverted because it
makes it appear as if this issue was fixed for users who did
backport 67de11769bd, when it is not.

Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Willy Tarreau <w@1wt.eu>
fs/btrfs/delayed-inode.c