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

Re: [dm-devel] [PATCH v2 for-3.14] dm cache: add block sizes and total cache blocks to status output



On Thu, Jan 16 2014 at 12:18pm -0500,
Brassow Jonathan <jbrassow 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 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)


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