Hi
I think you are right, @name here is unneeded..
You can give a patch for that.^_^
Wang
> |The code is pasted below for convenience of reference, but in the function
> to
> create a qgruop, it taks a 4th parameter (char * name). I assume this is the
> name
> of the path to limit, however, i don't see where its used anywhere in the
> function.
>
> -Kevin Brandstatter
>
> int btrfs_create_qgroup(struct btrfs_trans_handle *trans,
> struct btrfs_fs_info *fs_info, u64 qgroupid, *char**
> *****name**)*
> {
> struct btrfs_root *quota_root;
> struct btrfs_qgroup *qgroup;
> int ret = 0;
>
> mutex_lock(&fs_info->qgroup_ioctl_lock);
> quota_root = fs_info->quota_root;
> if (!quota_root) {
> ret = -EINVAL;
> goto out;
> }
> qgroup = find_qgroup_rb(fs_info, qgroupid);
> if (qgroup) {
> ret = -EEXIST;
> goto out;
> }
>
> ret = add_qgroup_item(trans, quota_root, qgroupid);
> if (ret)
> goto out;
>
> spin_lock(&fs_info->qgroup_lock);
> qgroup = add_qgroup_rb(fs_info, qgroupid);
> spin_unlock(&fs_info->qgroup_lock);
>
> if (IS_ERR(qgroup))
> ret = PTR_ERR(qgroup);
> out:
> mutex_unlock(&fs_info->qgroup_ioctl_lock);
> return ret;
> }|
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
