De-emphasize {smp_,}read_barrier_depends
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest21 patches 53 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/21 | doc: READ_ONCE() now implies smp_barrier_depends()Documentation/ | +6 | 6 replies | |
| 2/21 | mn10300: READ_ONCE() now implies smp_read_barrier_depends()arch/ | +5 | 0 replies | |
| 3/21 | drivers/net/ethernet/qlogic/qed: Fix __qed_spq_block() orderingdrivers/ | +1 | 0 replies | |
| 4/21 | fs/dcache: Use release-acquire for name/length updatefs/ | +3 | 0 replies | |
| 5/21 | percpu: READ_ONCE() now implies smp_read_barrier_depends()include/ lib/ | +7 | 0 replies | |
| 6/21 | rcu: Adjust read-side accessor comments for READ_ONCE()include/ | +11 | 0 replies | |
| 7/21 | rtnetlink: Update now-misleading smp_read_barrier_depends() commentinclude/ | +1 | 0 replies | |
| 8/21 | seqlock: Remove now-redundant smp_read_barrier_depends()include/ | +1 | 0 replies | |
| 9/21 | uprobes: Remove now-redundant smp_read_barrier_depends()kernel/ | +6 | 0 replies | |
| 10/21 | locking: Remove smp_read_barrier_depends() from queued_spin_lock_slowpath()kernel/ | +5 | 0 replies | |
| 11/21 | tracepoint: Remove smp_read_barrier_depends() from commentkernel/ | +4 | 0 replies | |
| 12/21 | lib/assoc_array: Remove smp_read_barrier_depends()lib/ | +12 | 0 replies | |
| 13/21 | mm/ksm: Remove now-redundant smp_read_barrier_depends()mm/ | +1 | 0 replies | |
| 14/21 | netfilter: Remove now-redundant smp_read_barrier_depends()net/ net/ net/ | +3 | 0 replies | |
| 15/21 | keyring: Remove now-redundant smp_read_barrier_depends()security/ | +0 | 2 replies | |
| 16/21 | drivers/infiniband: Remove now-redundant smp_read_barrier_depends()drivers/ drivers/ drivers/ +9 more | +1 | 4 replies | |
| 17/21 | doc: De-emphasize smp_read_barrier_dependsDocumentation/ Documentation/ Documentation/ +2 more | +9 | 0 replies | |
| 18/21 | genetlink: Remove smp_read_barrier_depends() from commentinclude/ | +1 | 0 replies | |
| 19/21 | netlink: Remove smp_read_barrier_depends() from commentinclude/ | +1 | 0 replies | |
| 20/21 | checkpatch: Add warnings for {smp_,}read_barrier_depends()scripts/ | +6 | 5 replies | |
| 21/21 | drivers/vhost: Remove now-redundant read_barrier_depends()drivers/ | +1 | 36 replies |
Reviews trailers per patch
5 of 21 patches have a Reviewed-by or Acked-by.
| Patch | Steven Rostedt (VMware) | Mathieu Desnoyers | Tejun Heo | James Morris | Joe Perches | Jason Gunthorpe |
|---|---|---|---|---|---|---|
| 1/21 | ||||||
| 2/21 | ||||||
| 3/21 | ||||||
| 4/21 | ||||||
| 5/21 | A-b | |||||
| 6/21 | ||||||
| 7/21 | ||||||
| 8/21 | ||||||
| 9/21 | ||||||
| 10/21 | ||||||
| 11/21 | A-b | A-b | ||||
| 12/21 | ||||||
| 13/21 | ||||||
| 14/21 | ||||||
| 15/21 | R-b | |||||
| 16/21 | A-b | |||||
| 17/21 | ||||||
| 18/21 | ||||||
| 19/21 | ||||||
| 20/21 | A-b | |||||
| 21/21 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 6 reviewers.
Cover letter [PATCH tip/core/rcu 0/21] De-emphasize {smp_,}read_barrier_depends
Hello!
Now that READ_ONCE() includes smp_read_barrier_depends(), almost nothing else needs to, the exceptions being DEC Alpha architecture-specific code and of course READ_ONCE() itself. This series therefore removes smp_read_barrier_depends() and read_barrier_depends() from elsewhere, as they no longer have any effect. Note that this patch series also prohibits use of InfiniBand on DEC Alpha: (1) A poll of DEC Alpha users revealed no use of InfiniBand and (2) It is not clear that InfiniBand's memory ordering correctly handles DEC Alpha, even with the smp_read_barrier_depends() invocations.
Note also that patch 4 moves to release-acquire ordering to ensure that the name length is properly synchronized. An alternative approach would be to place the length in with the name, so that dependency ordering would cover both the length and the name, but doing so seemed a bit intrusive for this series.
Thanx, Paul
------------------------------------------------------------------------
Documentation/RCU/Design/Requirements/Requirements.html | 3 - Documentation/RCU/rcu_dereference.txt | 6 -- Documentation/RCU/whatisRCU.txt | 3 - Documentation/circular-buffers.txt | 3 - Documentation/memory-barriers.txt | 18 ++++--- arch/mn10300/kernel/mn10300-serial.c | 7 ++- drivers/dma/ioat/dma.c | 2 drivers/infiniband/Kconfig | 1 drivers/infiniband/hw/hfi1/rc.c | 4 - drivers/infiniband/hw/hfi1/ruc.c | 1 drivers/infiniband/hw/hfi1/sdma.c | 1 drivers/infiniband/hw/hfi1/uc.c | 2 drivers/infiniband/hw/hfi1/ud.c | 2 drivers/infiniband/hw/qib/qib_rc.c | 3 - drivers/infiniband/hw/qib/qib_ruc.c | 1 drivers/infiniband/hw/qib/qib_uc.c | 2 drivers/infiniband/hw/qib/qib_ud.c | 2 drivers/infiniband/sw/rdmavt/qp.c | 1 drivers/net/ethernet/qlogic/qed/qed_spq.c | 4 - drivers/vhost/vhost.c | 7 --- fs/dcache.c | 10 +--- include/linux/genetlink.h | 3 - include/linux/netfilter/nfnetlink.h | 3 - include/linux/percpu-refcount.h | 6 +- include/linux/rcupdate.h | 23 ++++----- include/linux/rtnetlink.h | 3 - include/linux/seqlock.h | 3 - kernel/events/uprobes.c | 12 ++--- kernel/locking/qspinlock.c | 12 ++--- kernel/tracepoint.c | 9 +-- lib/assoc_array.c | 37 +++++----------- lib/percpu-refcount.c | 8 +-- mm/ksm.c | 9 --- net/ipv4/netfilter/arp_tables.c | 7 --- net/ipv4/netfilter/ip_tables.c | 7 --- net/ipv6/netfilter/ip6_tables.c | 7 --- scripts/checkpatch.pl | 6 ++ security/keys/keyring.c | 7 --- 38 files changed, 85 insertions(+), 160 deletions(-)