Message ID | 20190420114900.15472-1-anand.jain@oracle.com (mailing list archive) |
---|---|
Headers | show |
Series | cleanup btrfs_ioctl_setflags and btrfs_set_prop | expand |
On Sat, Apr 20, 2019 at 07:48:50PM +0800, Anand Jain wrote: > A set of pathes to clean up btrfs_ioctl_setflags(), btrfs_set_prop_trans() > and btrfs_set_prop(). So as to avoidi duplicate code in > btrfs_ioctl_setflags() and killing of btrfs_set_prop_trans(). > > Anand Jain (10): > btrfs: refactor btrfs_set_props to validate externally > btrfs: export btrfs_set_prop > btrfs: start transaction in btrfs_ioctl_setflags() > btrfs: drop useless inode i_flags copy and restore > btrfs: make a local copy of btrfs_inode flags before update > btrfs: drop old_fsflags in btrfs_ioctl_setflags > btrfs: drop local copy of inode i_mode > btrfs: start transaction in xattr_handler_set_prop > btrfs: delete unused function btrfs_set_prop_trans > btrfs: btrfs_set_prop trans is never null Looks good to me, I'll add it to misc-next as it's only a cleanup series, now that we're getting close to 5.2 code freeze.
On 24/4/19 11:14 PM, David Sterba wrote: > On Sat, Apr 20, 2019 at 07:48:50PM +0800, Anand Jain wrote: >> A set of pathes to clean up btrfs_ioctl_setflags(), btrfs_set_prop_trans() >> and btrfs_set_prop(). So as to avoidi duplicate code in >> btrfs_ioctl_setflags() and killing of btrfs_set_prop_trans(). >> >> Anand Jain (10): >> btrfs: refactor btrfs_set_props to validate externally >> btrfs: export btrfs_set_prop >> btrfs: start transaction in btrfs_ioctl_setflags() >> btrfs: drop useless inode i_flags copy and restore >> btrfs: make a local copy of btrfs_inode flags before update >> btrfs: drop old_fsflags in btrfs_ioctl_setflags >> btrfs: drop local copy of inode i_mode >> btrfs: start transaction in xattr_handler_set_prop >> btrfs: delete unused function btrfs_set_prop_trans >> btrfs: btrfs_set_prop trans is never null > > Looks good to me, I'll add it to misc-next as it's only a cleanup > series, now that we're getting close to 5.2 code freeze. > Thanks. I thought I will take a look, looks like its not yet pushed? Which is the most updated? repo gitlab/github? -Anand
On Fri, Apr 26, 2019 at 02:20:08PM +0800, Anand Jain wrote: > On 24/4/19 11:14 PM, David Sterba wrote: > > On Sat, Apr 20, 2019 at 07:48:50PM +0800, Anand Jain wrote: > >> A set of pathes to clean up btrfs_ioctl_setflags(), btrfs_set_prop_trans() > >> and btrfs_set_prop(). So as to avoidi duplicate code in > >> btrfs_ioctl_setflags() and killing of btrfs_set_prop_trans(). > >> > >> Anand Jain (10): > >> btrfs: refactor btrfs_set_props to validate externally > >> btrfs: export btrfs_set_prop > >> btrfs: start transaction in btrfs_ioctl_setflags() > >> btrfs: drop useless inode i_flags copy and restore > >> btrfs: make a local copy of btrfs_inode flags before update > >> btrfs: drop old_fsflags in btrfs_ioctl_setflags > >> btrfs: drop local copy of inode i_mode > >> btrfs: start transaction in xattr_handler_set_prop > >> btrfs: delete unused function btrfs_set_prop_trans > >> btrfs: btrfs_set_prop trans is never null > > > > Looks good to me, I'll add it to misc-next as it's only a cleanup > > series, now that we're getting close to 5.2 code freeze. > > > Thanks. > I thought I will take a look, looks like its not yet pushed? > Which is the most updated? repo gitlab/github? It's been in for-next until yesterday and I've merged it to misc-next after that so it'll be out today.