lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200514224558.GZ2869@paulmck-ThinkPad-P72>
Date:   Thu, 14 May 2020 15:45:58 -0700
From:   "Paul E. McKenney" <paulmck@...nel.org>
To:     Akira Yokosawa <akiyks@...il.com>
Cc:     Boqun Feng <boqun.feng@...il.com>,
        Joel Fernandes <joel@...lfernandes.org>,
        LKML <linux-kernel@...r.kernel.org>,
        Vineeth Remanan Pillai <vpillai@...italocean.com>,
        Jonathan Corbet <corbet@....net>,
        Alan Stern <stern@...land.harvard.edu>,
        Andrea Parri <parri.andrea@...il.com>,
        Will Deacon <will@...nel.org>,
        Peter Zijlstra <peterz@...radead.org>,
        Nicholas Piggin <npiggin@...il.com>,
        David Howells <dhowells@...hat.com>,
        Jade Alglave <j.alglave@....ac.uk>,
        Luc Maranget <luc.maranget@...ia.fr>,
        Daniel Lustig <dlustig@...dia.com>,
        "open list:DOCUMENTATION" <linux-doc@...r.kernel.org>
Subject: Re: [PATCH RESEND 3/4] Documentation/litmus-tests: Merge atomic's
 README into top-level one

On Fri, May 15, 2020 at 07:03:33AM +0900, Akira Yokosawa wrote:
> On Thu, 14 May 2020 10:16:56 -0700, Paul E. McKenney wrote:
> > On Thu, May 14, 2020 at 08:46:18AM +0800, Boqun Feng wrote:
> >> On Wed, May 13, 2020 at 06:39:03AM +0900, Akira Yokosawa wrote:
> >>> From 96fa6680e3b990633ecbb6d11acf03a161b790bd Mon Sep 17 00:00:00 2001
> >>> From: Akira Yokosawa <akiyks@...il.com>
> >>> Date: Sun, 10 May 2020 15:12:57 +0900
> >>> Subject: [PATCH RESEND 3/4] Documentation/litmus-tests: Merge atomic's README into top-level one
> >>>
> >>> Where Documentation/litmus-tests/README lists RCU litmus tests,
> >>> Documentation/litmus-tests/atomic/README lists atomic litmus tests.
> >>> For symmetry, merge the latter into former, with some context
> >>> adjustment in the introduction.
> >>>
> >>> Signed-off-by: Akira Yokosawa <akiyks@...il.com>
> >>> Acked-by: Andrea Parri <parri.andrea@...il.com>
> >>> Acked-by: Joel Fernandes (Google) <joel@...lfernandes.org>
> >>
> >> Acked-by: Boqun Feng <boqun.feng@...il.com>
> >>
> >> Thanks!
> > 
> > Applied, and thank you all!
> > 
> > I rebased, cancelling the revert with the original, resulting in an
> > updated lkmm branch on -rcu.  There was one minor conflict, so could
> > one of you please check to make sure that I resolved things appropriately?
> 
> One thing I noticed.
> 
> Commit b2998782ded4 ("Documentation/litmus-tests: Clarify about the RCU
> pre-initialization test")'s change log says:
> 
>     Since this test returned to tools/memory-model/, make sure that it is
>                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>     at least referenced from Documentation/litmus-tests/'s README.
> 
> Because of the rebase, this needs amendment as well as the title.
> 
> Something like
> 
>     Documentation/litumus-tests: Cite a relevant litmus test in tools/memory-model
> 
>     For ease of finding the RCU related litmus test under
>     tools/memory-model/, add an entry in README.
> 
> ?

Good catch, and yes, I will update that on the next rebase.

Any other things in need of adjustment?

							Thanx, Paul

>     Thanks, Akira
> 
> > 
> > 							Thanx, Paul
> > 
> >> Regards,
> >> Boqun
> >>
> >>> ---
> >>>  Documentation/litmus-tests/README        | 19 +++++++++++++++++++
> >>>  Documentation/litmus-tests/atomic/README | 16 ----------------
> >>>  2 files changed, 19 insertions(+), 16 deletions(-)
> >>>  delete mode 100644 Documentation/litmus-tests/atomic/README
> >>>
> >>> diff --git a/Documentation/litmus-tests/README b/Documentation/litmus-tests/README
> >>> index c4307ea9f996..ac0b270b456c 100644
> >>> --- a/Documentation/litmus-tests/README
> >>> +++ b/Documentation/litmus-tests/README
> >>> @@ -2,6 +2,25 @@
> >>>  LITMUS TESTS
> >>>  ============
> >>>  
> >>> +Each subdirectory contains litmus tests that are typical to describe the
> >>> +semantics of respective kernel APIs.
> >>> +For more information about how to "run" a litmus test or how to generate
> >>> +a kernel test module based on a litmus test, please see
> >>> +tools/memory-model/README.
> >>> +
> >>> +
> >>> +atomic (/atomic derectory)
> >>> +--------------------------
> >>> +
> >>> +Atomic-RMW+mb__after_atomic-is-stronger-than-acquire.litmus
> >>> +    Test that an atomic RMW followed by a smp_mb__after_atomic() is
> >>> +    stronger than a normal acquire: both the read and write parts of
> >>> +    the RMW are ordered before the subsequential memory accesses.
> >>> +
> >>> +Atomic-RMW-ops-are-atomic-WRT-atomic_set.litmus
> >>> +    Test that atomic_set() cannot break the atomicity of atomic RMWs.
> >>> +
> >>> +
> >>>  RCU (/rcu directory)
> >>>  --------------------
> >>>  
> >>> diff --git a/Documentation/litmus-tests/atomic/README b/Documentation/litmus-tests/atomic/README
> >>> deleted file mode 100644
> >>> index 714cf93816ea..000000000000
> >>> --- a/Documentation/litmus-tests/atomic/README
> >>> +++ /dev/null
> >>> @@ -1,16 +0,0 @@
> >>> -This directory contains litmus tests that are typical to describe the semantics
> >>> -of our atomic APIs. For more information about how to "run" a litmus test or
> >>> -how to generate a kernel test module based on a litmus test, please see
> >>> -tools/memory-model/README.
> >>> -
> >>> -============
> >>> -LITMUS TESTS
> >>> -============
> >>> -
> >>> -Atomic-RMW+mb__after_atomic-is-stronger-than-acquire
> >>> -	Test that an atomic RMW followed by a smp_mb__after_atomic() is
> >>> -	stronger than a normal acquire: both the read and write parts of
> >>> -	the RMW are ordered before the subsequential memory accesses.
> >>> -
> >>> -Atomic-RMW-ops-are-atomic-WRT-atomic_set.litmus
> >>> -	Test that atomic_set() cannot break the atomicity of atomic RMWs.
> >>> -- 
> >>> 2.17.1
> >>>
> >>>

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ