Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1380392

Re: [PATCH] locktorture: make verbose writable and control stats print

Show all headers | View raw

On Fri, Apr 15, 2016 at 04:45:32PM -0700, Shi, Yang wrote:
> On 4/15/2016 4:26 PM, Paul E. McKenney wrote:
> >On Fri, Apr 15, 2016 at 01:28:11PM -0700, Yang Shi wrote:
> >>When building locktorture test into kernel image, it keeps printing out
> >>stats information even though there is no lock type specified.
> >>
> >>There is already verbose parameter to control print, but it is read-only,
> >>so it can't be changed at runtime. Make verbose read-write and control
> >>stats print.
> >>
> >>Signed-off-by: Yang Shi <yang.shi@linaro.org>
> >
> >Interesting change!
> >
> >But just out of curiosity, when you boot with locktorture built in,
> >do you specify the shutdown_secs boot parameter?  If so, another
> 
> No, just use the default value, which is 0 for shutdown_secs.
> 
> >approach would be to shutdown immediately upon detecting an error
> >during initialization.
> 
> In my case, it looks there is not error involved.

You said that there is no lock type specified, but that should mean that
the default ("spin_lock") is chosen.  If so, I would expect it to just
do the test, at least if locktorture.torture_runnable has been set.

Either way, the usual way to make locktorture shut up would be to boot
with locktorture.stat_interval=0.

> >If not, I would like to know more about your use case.
> 
> In my test, I just built locktorture test into kernel instead of a
> module then check how it behaves, no specific purpose.
> 
> It sounds like not a normal approach to use it.

Agreed, I do believe that this is a case of "working as designed".

							Thanx, Paul

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] locktorture: make verbose writable and control stats print Yang Shi <yang.shi@linaro.org> - 2016-04-15 23:00 +0200
  Re: [PATCH] locktorture: make verbose writable and control stats  print "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2016-04-16 01:30 +0200
    Re: [PATCH] locktorture: make verbose writable and control stats  print "Shi, Yang" <yang.shi@linaro.org> - 2016-04-16 01:50 +0200
      Re: [PATCH] locktorture: make verbose writable and control stats  print "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2016-04-16 02:10 +0200
        Re: [PATCH] locktorture: make verbose writable and control stats  print "Shi, Yang" <yang.shi@linaro.org> - 2016-04-16 02:20 +0200
          Re: [PATCH] locktorture: make verbose writable and control stats  print "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2016-04-16 02:40 +0200

csiph-web