From: "Emilio G. Cota" <c...@braap.org> It's unnecessary because the pointer isn't dereferenced.
Signed-off-by: Emilio G. Cota <c...@braap.org> Message-Id: <20180819091335.22863-3-c...@braap.org> Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> --- include/qemu/rcu_queue.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/qemu/rcu_queue.h b/include/qemu/rcu_queue.h index dd7b3be..6881ea5 100644 --- a/include/qemu/rcu_queue.h +++ b/include/qemu/rcu_queue.h @@ -36,7 +36,7 @@ extern "C" { /* * List access methods. */ -#define QLIST_EMPTY_RCU(head) (atomic_rcu_read(&(head)->lh_first) == NULL) +#define QLIST_EMPTY_RCU(head) (atomic_read(&(head)->lh_first) == NULL) #define QLIST_FIRST_RCU(head) (atomic_rcu_read(&(head)->lh_first)) #define QLIST_NEXT_RCU(elm, field) (atomic_rcu_read(&(elm)->field.le_next)) -- 1.8.3.1