On Fri, Apr 12, 2019 at 04:02:53PM +0800, Anand Jain wrote: > In an attempt to stream line the property and extended attribute set here > are the few cleanup patches. > > 1/6 to 3/6 are mostly non functional cleanups (except for the conversion > to non static function in 3/6) and can be merged together. > 4/6 removes the readonly root check in btrfs_setxattr() more details in > the change log. > 5/6 as now we have btrfs_setxattr() and btrfs_setxattr_trans() for the > threads with transaction and without transaction respectively, so this > patch uses them. > 6/6 as 5/6 as diverted the threads with transaction to btrfs_setxattr(), > now btrfs_setxattr_trans() can drop the trans arg. > > Anand Jain (6): > btrfs: rename btrfs_setxattr to btrfs_setxattr_trans > btrfs: rename do_setxattr to btrfs_setxattr > btrfs: declare btrfs_setxattr as a non static function > btrfs: remove redundant readonly root check in btrfs_setxattr_trans > btrfs: split thread with trans to use btrfs_setxattr > btrfs: cleanup btrfs_setxattr_trans drop trans arg
Looks good to me, thanks. The result is very close to what the previous patchset did. Patchset will go to for-next soon.
