diff options
author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-09-20 17:03:16 -0700 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-11-23 10:37:35 -0800 |
commit | 1658d35ead5d8dd76f2b2d6ad0e32c08d123faa2 (patch) | |
tree | b237b079d8c057cb77e54f7e21baec00196b448f /include/linux/rculist.h | |
parent | 1c97be677f72b3c338312aecd36d8fff20322f32 (diff) | |
download | lwn-1658d35ead5d8dd76f2b2d6ad0e32c08d123faa2.tar.gz lwn-1658d35ead5d8dd76f2b2d6ad0e32c08d123faa2.zip |
list: Use READ_ONCE() when testing for empty lists
Most of the list-empty-check macros (list_empty(), hlist_empty(),
hlist_bl_empty(), hlist_nulls_empty(), and hlist_nulls_empty()) use
an unadorned load to check the list header. Given that these macros
are sometimes invoked without the protection of a lock, this is
not sufficient. This commit therefore adds READ_ONCE() calls to
them. This commit does not touch llist_empty() because it already
has the needed ACCESS_ONCE().
Reported-by: Dmitry Vyukov <dvyukov@google.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Diffstat (limited to 'include/linux/rculist.h')
0 files changed, 0 insertions, 0 deletions