[dm-devel] [PATCH v2 for-3.14] dm cache: add block sizes and total cache blocks to status output
Mike Snitzer
snitzer at redhat.com
Thu Jan 16 19:35:33 UTC 2014
On Thu, Jan 16 2014 at 12:18pm -0500,
Brassow Jonathan <jbrassow at redhat.com> wrote:
>
> On Jan 15, 2014, at 10:34 PM, Mike Snitzer wrote:
>
> > On Wed, Jan 15 2014 at 8:16pm -0500,
> > Brassow Jonathan <jbrassow at redhat.com> wrote:
> >
> >> I know I'm pushing my luck...
> >>
> >> Is it too late to ask for just one more thing? Would you also consider printing the policy name in the status (INFO)?
> >> <policy_name> <#policy args> <policy args>
> >>
> >> This is especially useful when you pass in "default" in the CTR, because you cannot get this value from anywhere... right?
> >
> > For the benefit of others, I pushed the change here:
> > http://git.kernel.org/cgit/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=for-next&id=2f20afc4c3f69030204ce55f24bc68971b500b8a
>
> This doesn't quite produce the info we want. If I specify "default"
> in the CTR table, I want the status to show me what the result was
> (i.e. "mq"). It currently seems to be printing "default" back at
> me... which isn't helpful. :)
I updated the commit, see:
http://git.kernel.org/cgit/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=for-next&id=2e68c4e6caad9fdadc1cef8b6cb9569192e8a42b
I've also verified it works (by updating my local
device-mapper-test-suite code, will get these changes to joe soon)
More information about the dm-devel
mailing list