On Fri, Mar 13, 2020 at 05:23:19PM +0200, Nikolay Borisov wrote: > btrfs_ioctl_snap_create_transid no longer takes a transid argument, so > remove it and rename the function to __btrfs_ioctl_snap_create to > reflect it's an internal, worker function. Hm, this use of an underscored version does not look all great to me. You're right that it's a worker, so it could get a completely new name. The functions directly handling the ioctls have _ioctl_ in their name but in this case I don't think it's necessary and gives us more freedom to come up with a better name. I don't have a suggestion for now.
