Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Wed, 2 Jan 2019 05:27:53 -0800
From:   "Paul E. McKenney" <paulmck@linux.ibm.com>
To:     Junchang Wang <junchangwang@gmail.com>
Cc:     dave@stgolabs.net, josh@joshtriplett.org,
        Jonathan Corbet <corbet@lwn.net>, rostedt@goodmis.org,
        Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
        jiangshanlai@gmail.com, joel@joelfernandes.org,
        linux-kernel <linux-kernel@vger.kernel.org>,
        linux-doc@vger.kernel.org
Subject: Re: [PATCH 2/2] torture.txt: update the list of supported torture
 types
Reply-To: paulmck@linux.ibm.com
References: <1546351400-16389-1-git-send-email-junchangwang@gmail.com>
 <1546351400-16389-3-git-send-email-junchangwang@gmail.com>
 <20190101175510.GW4170@linux.ibm.com>
 <CABoNC82px+072c2eKWcpjMyJX-m8Dh4=L0W6Wxw5Q+HeXoK2Pg@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CABoNC82px+072c2eKWcpjMyJX-m8Dh4=L0W6Wxw5Q+HeXoK2Pg@mail.gmail.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
Message-Id: <20190102132753.GZ4170@linux.ibm.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, Jan 02, 2019 at 11:48:00AM +0800, Junchang Wang wrote:
> On Wed, Jan 2, 2019 at 1:54 AM Paul E. McKenney <paulmck@linux.ibm.com> wrote:
> >
> > On Tue, Jan 01, 2019 at 10:03:20PM +0800, Junchang Wang wrote:
> > > Torture types "rcu_bh" and "sched" were removed in commit "c770c82a23".
> > > The name of torture type "rcu_busted" was changed to "busted" in commit
> > > "b3c983142d". Two other types, "srcud" and "busted_srcud" were added in
> > > commits "ca1d51ed98" and "2397d072f7", respectively. This patch updates
> > > torture.txt accordingly.
> > >
> > > Signed-off-by: Junchang Wang <junchangwang@gmail.com>
> >
> > Given that this section of this file duplicates the rcutorture kernel boot
> > parameters called out in Documentation/admin-guide/kernel-parameters.txt,
> > perhaps the best way forward is to instead remove this information from
> > Documentation/RCU/torture.txt in favor of a reference to the information
> > (with search string) in Documentation/admin-guide/kernel-parameters.txt.
> >
> > Thoughts?
> 
> Hi Paul,
> 
> Thanks for the comment; indeed the section "MODULE PARAMETERS" in
> torture.txt duplicates the corresponding sections in
> kernel-parameters.txt. I can resubmit a patch that (1) removes the
> section "MODULE PARAMETERS" in torture.txt, and (2) add a single
> sentence saying "Module parameters are detailed in
> Documentation/admin-guide/kernel-parameters.txt". Does that sound
> good?

A patch that did that would be quite welcome!

							Thanx, Paul

> Thanks,
> --Junchang
> 
> >
> >                                                         Thanx, Paul
> >
> > > ---
> > >  Documentation/RCU/torture.txt | 15 +++++++--------
> > >  1 file changed, 7 insertions(+), 8 deletions(-)
> > >
> > > diff --git a/Documentation/RCU/torture.txt b/Documentation/RCU/torture.txt
> > > index 55918b5..6395f0b 100644
> > > --- a/Documentation/RCU/torture.txt
> > > +++ b/Documentation/RCU/torture.txt
> > > @@ -155,20 +155,19 @@ torture_type    The type of RCU to test, with string values as follows:
> > >                       along with expedited, synchronous, and polling
> > >                       variants.
> > >
> > > -             "rcu_bh": rcu_read_lock_bh(), rcu_read_unlock_bh(), and
> > > -                     call_rcu_bh(), along with expedited and synchronous
> > > -                     variants.
> > > -
> > > -             "rcu_busted": This tests an intentionally incorrect version
> > > +             "busted": This tests an intentionally incorrect version
> > >                       of RCU in order to help test rcutorture itself.
> > >
> > >               "srcu": srcu_read_lock(), srcu_read_unlock() and
> > >                       call_srcu(), along with expedited and
> > >                       synchronous variants.
> > >
> > > -             "sched": preempt_disable(), preempt_enable(), and
> > > -                     call_rcu_sched(), along with expedited,
> > > -                     synchronous, and polling variants.
> > > +             "srcud": A second form of srcu that dynamically allocates
> > > +                     and frees srcu_struct to test srcu cleanup code path.
> > > +
> > > +             "busted_srcud": This torture type verifies rcutorture's ability
> > > +                     to detect extensions of RCU read-side critical sections
> > > +                     that are not handled.
> > >
> > >               "tasks": voluntary context switch and call_rcu_tasks(),
> > >                       along with expedited and synchronous variants.
> > > --
> > > 2.7.4
> > >
> >
>