While looking at the transaction code and various pattern I came across some problems which this series aim to fix.
Patch 1 just removes some redundant code. Patch 2 adds a missing btrfs_abort_transaction, otherwise we don't properly close out the transaction. I believe this could lead to btrfs_root_item being modified yet we could be missing respective entry in the uuid tree. Patch 3 btrfs_rm_dev_item totally missed aborting the transaction in its failure cases. Nikolay Borisov (3): btrfs: Remove unnecessary btrfs_abort_transaction on transaction commit failure btrfs: Handle failure to add received uuid to uuid tree in _btrfs_ioctl_set_received_subvol btrfs: Fix transaction abort during failure in btrfs_rm_dev_item fs/btrfs/ioctl.c | 9 +-------- fs/btrfs/volumes.c | 6 ++++-- 2 files changed, 5 insertions(+), 10 deletions(-) -- 2.7.4 -- 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