From: Mike Snitzer <snitzer@redhat.com>
To: Tomohiro Kusumi <kusumi.tomohiro@gmail.com>
Cc: dm-devel@redhat.com, mpatocka@redhat.com
Subject: Re: [PATCH 6/6] dm-stripe: Fix error message
Date: Wed, 28 Oct 2015 18:00:57 -0400 [thread overview]
Message-ID: <20151028220055.GA22827@redhat.com> (raw)
In-Reply-To: <CAHndrBn06ih=6atGBcC7TjoFEWd0y=9WPu6GBygFE06y6Q=j7w@mail.gmail.com>
On Wed, Oct 28 2015 at 4:40pm -0400,
Tomohiro Kusumi <kusumi.tomohiro@gmail.com> wrote:
> hi, thanks for your reply.
>
> I understand what you mean, but that's the reason.
> I happened to use something like below via dmsetup
>
> --table '0 100 striped 2 4 /dev/xxx 0 /dev/yyy 0'
>
> and hit this message which is not obvious as long as it says "Target
> length".
> Users have no idea 50/4 was what was really done in kernel space.
You're allowing a very literal look at the code to cloud your perception
of the error and concluding that it is misleading/incorrect.
Just because the code does math in a certain order doesn't change the
fact that the error message is still correct (if/when it is reported).
next prev parent reply other threads:[~2015-10-28 22:00 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-27 19:38 [PATCH 1/6] dm-delay: Use DM_MAPIO_XXX macros instead of 0/1 (resend) Tomohiro Kusumi
2015-10-27 19:38 ` [PATCH 2/6] dm-delay: Update documentation for offsets (resend) Tomohiro Kusumi
2015-10-27 19:38 ` [PATCH 3/6] dm-delay: Capitalize error message consistently Tomohiro Kusumi
2015-10-27 19:38 ` [PATCH 4/6] dm-linear: Improve error message consistency Tomohiro Kusumi
2015-10-27 19:38 ` [PATCH 5/6] dm-switch: Remove unnecessary condition in conditional Tomohiro Kusumi
2015-10-27 23:03 ` Mikulas Patocka
2015-10-27 19:39 ` [PATCH 6/6] dm-stripe: Fix error message Tomohiro Kusumi
2015-10-28 19:51 ` Mike Snitzer
2015-10-28 20:40 ` Tomohiro Kusumi
2015-10-28 22:00 ` Mike Snitzer [this message]
2015-10-29 11:36 ` Tomohiro Kusumi
2015-10-27 23:04 ` [PATCH 1/6] dm-delay: Use DM_MAPIO_XXX macros instead of 0/1 (resend) Mikulas Patocka
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151028220055.GA22827@redhat.com \
--to=snitzer@redhat.com \
--cc=dm-devel@redhat.com \
--cc=kusumi.tomohiro@gmail.com \
--cc=mpatocka@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.