<div dir="ltr"><div>hi, thanks for your reply.</div><div><br></div><div>I understand what you mean, but that's the reason.</div><div>I happened to use something like below via dmsetup</div><div><br></div><div>--table '0 100 striped 2 4 /dev/xxx 0 /dev/yyy 0'</div><div><br></div><div>and hit this message which is not obvious as long as it says "Target length".</div><div>Users have no idea 50/4 was what was really done in kernel space.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-10-29 4:51 GMT+09:00 Mike Snitzer <span dir="ltr"><<a href="mailto:snitzer@redhat.com" target="_blank">snitzer@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Oct 27 2015 at  3:39pm -0400,<br>
Tomohiro Kusumi <<a href="mailto:kusumi.tomohiro@gmail.com">kusumi.tomohiro@gmail.com</a>> wrote:<br>
<br>
> The meaning of "not divisible by chunk size" has changed after<br>
> d793e684, so the error message here should probably be using<br>
> "Stripe length" since tmp_len is result of width/stripes, but<br>
> not the whole target device size.<br>
><br>
> Signed-off-by: Tomohiro Kusumi <<a href="mailto:kusumi.tomohiro@gmail.com">kusumi.tomohiro@gmail.com</a>><br>
> ---<br>
>  drivers/md/dm-stripe.c |    2 +-<br>
>  1 files changed, 1 insertions(+), 1 deletions(-)<br>
><br>
> diff --git a/drivers/md/dm-stripe.c b/drivers/md/dm-stripe.c<br>
> index 797ddb9..85cb09f 100644<br>
> --- a/drivers/md/dm-stripe.c<br>
> +++ b/drivers/md/dm-stripe.c<br>
> @@ -127,7 +127,7 @@ static int stripe_ctr(struct dm_target *ti, unsigned int argc, char **argv)<br>
><br>
>       tmp_len = width;<br>
>       if (sector_div(tmp_len, chunk_size)) {<br>
> -             ti->error = "Target length not divisible by "<br>
> +             ti->error = "Stripe length not divisible by "<br>
>                   "chunk size";<br>
>               return -EINVAL;<br>
>       }<br>
<br>
</span>There is no need for this change.  The user-facing error is more<br>
meaningful.  The only input that is being checked here is the specified<br>
target length.<br>
<br>
Dropping this patch.<br>
</blockquote></div><br></div>