On 2018年05月18日 09:00, Misono Tomohiro wrote:
> On 2018/05/17 15:39, Qu Wenruo wrote:
>>
>>
>> On 2018年05月16日 13:49, Tomohiro Misono wrote:
>>> Add new unprivileged ioctl BTRFS_IOC_GET_SUBVOL_INFO which returns
>>> the information of subvolume containing this inode.
>>> (i.e. returns the information in ROOT_ITEM and ROOT_BACKREF.)
>>>
>>> Signed-off-by: Tomohiro Misono <misono.tomohiro@xxxxxxxxxxxxxx>
>>> ---
>>> v4 -> v5
>>> - Update error handling of btrfs_next_leaf() to cover all cases
>>> - Return error if ROOT_BACKREF is not found (except top-level)
>>>
>>> fs/btrfs/ioctl.c | 146 +++++++++++++++++++++++++++++++++++++++++++++
>>> include/uapi/linux/btrfs.h | 51 ++++++++++++++++
>>> 2 files changed, 197 insertions(+)
>>>
>>> diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c
>>> index 48e2ddff32bd..c1c9ae9a937d 100644
>>> --- a/fs/btrfs/ioctl.c
>>> +++ b/fs/btrfs/ioctl.c
>>> @@ -2242,6 +2242,150 @@ static noinline int btrfs_ioctl_ino_lookup(struct file *file,
>>> return ret;
>>> }
>>>
>>> +/* Get the subvolume information in BTRFS_ROOT_ITEM and BTRFS_ROOT_BACKREF */
>>> +static noinline int btrfs_ioctl_get_subvol_info(struct file *file,
>>> + void __user *argp)
>>> +{
>>> + struct btrfs_ioctl_get_subvol_info_args *subvol_info;
>>> + struct btrfs_root *root;
>>> + struct btrfs_path *path;
>>> + struct btrfs_key key;
>>> +
>>> + struct btrfs_root_item root_item;
>>> + struct btrfs_root_ref *rref;
>>> + struct extent_buffer *l;
>>> + int slot;
>>> +
>>> + unsigned long item_off;
>>> + unsigned long item_len;
>>> +
>>> + struct inode *inode;
>>> + int ret;
>>> +
>>> + path = btrfs_alloc_path();
>>> + if (!path)
>>> + return -ENOMEM;
>>> +
>>> + subvol_info = kzalloc(sizeof(*subvol_info), GFP_KERNEL);
>>> + if (!subvol_info) {
>>> + btrfs_free_path(path);
>>> + return -ENOMEM;
>>> + }
>>> +
>>> + inode = file_inode(file);
>>> + root = BTRFS_I(inode)->root->fs_info->tree_root;
>>> +
>>> + key.objectid = BTRFS_I(inode)->root->root_key.objectid;
>>> + key.type = BTRFS_ROOT_ITEM_KEY;
>>> + key.offset = 0;
>>> +
>>> + ret = btrfs_search_slot(NULL, root, &key, path, 0, 0);
>>
>> What about just using btrfs_get_fs_root()?
>>
>> It would save several lines, and have better error handling along with
>> better tree cache.
>
> Here we search ROOT_ITEM of subvolume in root_tree and
> I don't think btrfs_get_fs_root() would save lines, right?
btrfs_root has root_item member.
So just btrfs_get_fs_root(), then use root->root_item.
>>> + if (ret < 0) {
>>> + goto out;
>>> + } else if (ret > 0) {
>>> + u64 objectid = key.objectid;
>>> +
>>> + if (path->slots[0] >= btrfs_header_nritems(path->nodes[0])) {
>>> + ret = btrfs_next_leaf(root, path);
>>> + if (ret < 0) {
>>> + goto out;
>>> + } else if (ret > 0) {
>>> + ret = -ENOENT;
>>> + goto out;
>>> + }
>>> + }
>>> +
>>> + /* If the subvolume is a snapshot, offset is not zero */
>>> + btrfs_item_key_to_cpu(path->nodes[0], &key, path->slots[0]);
>>> + if (key.objectid != objectid ||
>>> + key.type != BTRFS_ROOT_ITEM_KEY) {
>>> + ret = -ENOENT;
>>> + goto out;
>>> + }
>>> + }
These lines can be saved.
Thanks,
Qu
>>> + l = path->nodes[0];
>>> + slot = path->slots[0];
>>> + item_off = btrfs_item_ptr_offset(l, slot);
>>> + item_len = btrfs_item_size_nr(l, slot);
>>> + read_extent_buffer(l, &root_item, item_off, item_len);
>>> +
>>> + subvol_info->id = key.objectid;
>>> +
>>> + subvol_info->generation = btrfs_root_generation(&root_item);
>>> + subvol_info->flags = btrfs_root_flags(&root_item);
>>> +
>>> + memcpy(subvol_info->uuid, root_item.uuid, BTRFS_UUID_SIZE);
>>> + memcpy(subvol_info->parent_uuid, root_item.parent_uuid,
>>> + BTRFS_UUID_SIZE);
>>> + memcpy(subvol_info->received_uuid, root_item.received_uuid,
>>> + BTRFS_UUID_SIZE);
>>> +
>>> + subvol_info->ctransid = btrfs_root_ctransid(&root_item);
>>> + subvol_info->ctime.sec = btrfs_stack_timespec_sec(&root_item.ctime);
>>> + subvol_info->ctime.nsec = btrfs_stack_timespec_nsec(&root_item.ctime);
>>> +
>>> + subvol_info->otransid = btrfs_root_otransid(&root_item);
>>> + subvol_info->otime.sec = btrfs_stack_timespec_sec(&root_item.otime);
>>> + subvol_info->otime.nsec = btrfs_stack_timespec_nsec(&root_item.otime);
>>> +
>>> + subvol_info->stransid = btrfs_root_stransid(&root_item);
>>> + subvol_info->stime.sec = btrfs_stack_timespec_sec(&root_item.stime);
>>> + subvol_info->stime.nsec = btrfs_stack_timespec_nsec(&root_item.stime);
>>> +
>>> + subvol_info->rtransid = btrfs_root_rtransid(&root_item);
>>> + subvol_info->rtime.sec = btrfs_stack_timespec_sec(&root_item.rtime);
>>> + subvol_info->rtime.nsec = btrfs_stack_timespec_nsec(&root_item.rtime);
>>> +
>>> + btrfs_release_path(path);
>>> + if (key.objectid != BTRFS_FS_TREE_OBJECTID) {
>>> + key.type = BTRFS_ROOT_BACKREF_KEY;
>>> + key.offset = 0;
>>> + ret = btrfs_search_slot(NULL, root, &key, path, 0, 0);
>>> + if (ret < 0) {
>>> + goto out;
>>> + } else if (path->slots[0] >=
>>> + btrfs_header_nritems(path->nodes[0])) {
>>> + ret = btrfs_next_leaf(root, path);
>>> + if (ret < 0) {
>>> + goto out;
>>> + } else if (ret > 0) {
>>> + ret = -ENOENT;
>>> + goto out;
>>> + }
>>> + }
>>> +
>>> + l = path->nodes[0];
>>> + slot = path->slots[0];
>>> + btrfs_item_key_to_cpu(l, &key, slot);
>>> + if (key.objectid == subvol_info->id &&
>>> + key.type == BTRFS_ROOT_BACKREF_KEY) {
>>> + subvol_info->parent_id = key.offset;
>>> +
>>> + rref = btrfs_item_ptr(l, slot, struct btrfs_root_ref);
>>> + subvol_info->dirid = btrfs_root_ref_dirid(l, rref);
>>> +
>>> + item_off = btrfs_item_ptr_offset(l, slot)
>>> + + sizeof(struct btrfs_root_ref);
>>> + item_len = btrfs_item_size_nr(l, slot)
>>> + - sizeof(struct btrfs_root_ref);
>>> + read_extent_buffer(l, subvol_info->name,
>>> + item_off, item_len);
>>> + } else {
>>> + ret = -ENOENT;
>>> + goto out;
>>> + }
>>> + }
>>> +
>>> + if (copy_to_user(argp, subvol_info, sizeof(*subvol_info)))
>>> + ret = -EFAULT;
>>> +
>>> +out:
>>> + kzfree(subvol_info);
>>> + btrfs_free_path(path);
>>> + return ret;
>>> +}
>>> +
>>> static noinline int btrfs_ioctl_snap_destroy(struct file *file,
>>> void __user *arg)
>>> {
>>> @@ -5374,6 +5518,8 @@ long btrfs_ioctl(struct file *file, unsigned int
>>> return btrfs_ioctl_get_features(file, argp);
>>> case BTRFS_IOC_SET_FEATURES:
>>> return btrfs_ioctl_set_features(file, argp);
>>> + case BTRFS_IOC_GET_SUBVOL_INFO:
>>> + return btrfs_ioctl_get_subvol_info(file, argp);
>>> }
>>>
>>> return -ENOTTY;
>>> diff --git a/include/uapi/linux/btrfs.h b/include/uapi/linux/btrfs.h
>>> index c8d99b9ca550..02cd1f1994e8 100644
>>> --- a/include/uapi/linux/btrfs.h
>>> +++ b/include/uapi/linux/btrfs.h
>>> @@ -725,6 +725,55 @@ struct btrfs_ioctl_send_args {
>>> __u64 reserved[4]; /* in */
>>> };
>>>
>>> +struct btrfs_ioctl_get_subvol_info_args {
>>> + /* All field is out */
>>> + /* Id of this subvolume */
>>> + __u64 id;
>>> + /* Name of this subvolume, used to get the real name at mount point */
>>> + char name[BTRFS_VOL_NAME_MAX + 1];
>>> + /*
>>> + * Id of the subvolume which contains this subvolume.
>>> + * Zero for top-level subvolume or deleted subvolume
>>> + */
>>> + __u64 parent_id;
>>> + /*
>>> + * Inode number of the directory which contains this subvolume.
>>> + * Zero for top-level subvolume or deleted subvolume
>>> + */
>>> + __u64 dirid;
>>> +
>>> + /* Latest transaction id of this subvolume */
>>> + __u64 generation;
>>> + /* Flags of this subvolume */
>>> + __u64 flags;
>>> +
>>> + /* uuid of this subvolume */
>>> + __u8 uuid[BTRFS_UUID_SIZE];
>>> + /*
>>> + * uuid of the subvolume of which this subvolume is a snapshot.
>>> + * All zero for non-snapshot subvolume
>>> + */
>>> + __u8 parent_uuid[BTRFS_UUID_SIZE];
>>> + /*
>>> + * uuid of the subvolume from which this subvolume is received.
>>> + * All zero for non-received subvolume
>>> + */
>>> + __u8 received_uuid[BTRFS_UUID_SIZE];
>>> +
>>> + /* Transaction id indicates when change/create/send/receive happens */
>>> + __u64 ctransid;
>>> + __u64 otransid;
>>> + __u64 stransid;
>>> + __u64 rtransid;
>>> + /* Time corresponds to c/o/s/rtransid */
>>> + struct btrfs_ioctl_timespec ctime;
>>> + struct btrfs_ioctl_timespec otime;
>>> + struct btrfs_ioctl_timespec stime;
>>> + struct btrfs_ioctl_timespec rtime;
>>> +
>>> + __u64 reserved[8];
>>> +};
>>> +
>>> /* Error codes as returned by the kernel */
>>> enum btrfs_err_code {
>>> BTRFS_ERROR_DEV_RAID1_MIN_NOT_MET = 1,
>>> @@ -843,5 +892,7 @@ enum btrfs_err_code {
>>> struct btrfs_ioctl_vol_args_v2)
>>> #define BTRFS_IOC_LOGICAL_INO_V2 _IOWR(BTRFS_IOCTL_MAGIC, 59, \
>>> struct btrfs_ioctl_logical_ino_args)
>>> +#define BTRFS_IOC_GET_SUBVOL_INFO _IOR(BTRFS_IOCTL_MAGIC, 60, \
>>> + struct btrfs_ioctl_get_subvol_info_args)
>>>
>>> #endif /* _UAPI_LINUX_BTRFS_H */
>>>
>>
>
Attachment:
signature.asc
Description: OpenPGP digital signature
