On Fri, Jul 20, 2018 at 07:37:46PM +0300, Nikolay Borisov wrote: > Here are a bunch of patches which cleanup extraneous fs_info parameters to > function which already take a structure that holds a reference to the > fs_info. > > Except for patches 4 and 5, everything else is correct - due to those > functions > always taking a transaction. 4 and 5 in turn reference the fs_info from > struct btrfs_device. Inspecting the callers I managed to convince myself that > those function are always called with well-formed btrfs_device i.e one which > has its fs_info member initialised. Reviewers might want to pay extra > attention to that but otherwise they are trivial.
4 and 5 look good to me, a device without a valid fs_info has a short timespan and should not appear anywhere besides the helpers that set up fs_devices etc. Series added to misc-next, thanks. -- 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