lkml.org 
[lkml]   [2018]   [Jun]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 0/2] tools/memory-model: remove ACCESS_ONCE()
> 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

\
 
 \ /
  Last update: 2018-06-28 18:52    [W:0.117 / U:0.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site