On 01.11.2011 12:07, David Sterba wrote:
> 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.

Though it would still be much nicer to get a proper error message to the
user directly.

> 
> 
> david
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to