On Thu, Mar 10, 2016 at 12:22:15PM +0800, Anand Jain wrote: > This patch will log return value of add/del_qgroup_relation() and pass the > err code of btrfs_run_qgroups to the btrfs_std_error(). > > Signed-off-by: Anand Jain <anand.jain@xxxxxxxxxx> > --- > v2: fix the forgotten git commit amend, to take in compile fail, sorry > > fs/btrfs/ioctl.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c > index f704d1c..f43a104 100644 > --- a/fs/btrfs/ioctl.c > +++ b/fs/btrfs/ioctl.c > @@ -4803,10 +4803,15 @@ static long btrfs_ioctl_qgroup_assign(struct file *file, void __user *arg) > sa->src, sa->dst); > } > > + if (ret) > + btrfs_err(root->fs_info, > + "add/del qgroup relation failed, assign %llu ret %d", > + sa->assign, ret); It's just logging an error, but it should really fail, without a message. The error code will tell what went wrong. Assigning existing qgroups is eg. EEXIST, if either doesn't exist then ENOENT. There's a FIXME a few lines above that should be addressed. I'd like to avoid the message in this case because it should be handled by the ioctl caller, otherwise it could be just flooding the system log. > + > /* update qgroup status and info */ > err = btrfs_run_qgroups(trans, root->fs_info); > if (err < 0) > - btrfs_std_error(root->fs_info, ret, > + btrfs_std_error(root->fs_info, err, > "failed to update qgroup status and info\n"); > err = btrfs_end_transaction(trans, root); > if (err && !ret) -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
