Re: [PATCH 05/21] Btrfs: add basic restriper infrastructure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 01, 2011 at 11:08:38AM +0100, Arne Jansen wrote:
> > +/*
> > + * Should be called with restripe_mutex held
> > + */
> > +int btrfs_restripe(struct restripe_control *rctl)
> > +{
...
> > +	if (rctl->data.target & BTRFS_BLOCK_GROUP_DUP) {
> > +		printk(KERN_ERR "btrfs: dup for data is not allowed\n");
> > +		ret = -EINVAL;
> > +		goto out;
> > +	}
> 
> It would be good to get these error messages somehow to the user,
> or at least give the user a hint to look in dmesg.

the restriper command ends with EINVAL which is in most cases returned
as a result of the ioctl and progs counterpart will

1117                         fprintf(stderr, "ERROR: error during restriping '%s' "
1118                                 "- %s\n", path, strerror(e));
1119                         return 19;

the hint should go there imho.


david
--
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


[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux