Re: [PATCH 0/2] tools/memory-model: remove ACCESS_ONCE()

From: Andrea Parri <andrea.parri@amarulasolutions.com>
To: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
	linux-kernel@vger.kernel.org, akiyks@gmail.com,
	boqun.feng@gmail.com, dhowells@redhat.com, j.alglave@ucl.ac.uk,
	luc.maranget@inria.fr, npiggin@gmail.com, peterz@infradead.org,
	stern@rowland.harvard.edu, will.deacon@arm.com
Subject: Re: [PATCH 0/2] tools/memory-model: remove ACCESS_ONCE()
Date: Thu, 28 Jun 2018 18:51:11 +0200	[thread overview]
Message-ID: <20180628165111.GA4013@andrea> (raw)
In-Reply-To: <20180628152140.GK3593@linux.vnet.ibm.com>

> 1bc179880fba docs: atomic_ops: Describe atomic_set as a write operation
> 
> 	The above patches need at least one additional Acked-by
> 	or Reviewed-by.  If any of you gets a chance, please do
> 	look them over.

Glad this came out. ;-)

No objection to the patch: feel free to add my Reviewed-by: tag.

(BTW, atomic_set() would be better mapped to WRITE_ONCE()... in fact, to
 be fair, some archs do it the __asm__ __volatile__() way).

I do however have some suggestions concerning "the process":  searching
LKML for the patch and the related discussion, I could only find:

  [PATCH] docs: atomic_ops: atomic_set is a write (not read) operation

and I realize that none of the person Cc:-ed in this thread, except you,
were Cc:-ed in that discussion (in compliance with get_maintainer.pl).

My suggestions:

  1) Merge the file touched by that patch into (the recently created):

        Documentation/atomic_t.txt

     (FWIW, queued in my TODO list).

  2) Add the entry:

	F: Documentation/atomic_t.txt

     to the "ATOMIC INFRASTRUCTURE" subsystem in the MAINTAINERS file so
     that developers can easily find (the intended?) reviewers for their
     patch. (Of course, this will need ACK from the ATOMIC people).

  Andrea