On 9/21/23 21:21, Olivier Dion via lttng-dev wrote:
On Thu, 21 Sep 2023, Ondřej Surý via lttng-dev <lttng-dev@lists.lttng.org>
wrote:
[...]
It fails with:
rculfhash.c:1189:2: error: address argument to atomic operation must be a
pointer to integer ('typeof (node_next)' (aka 'struct cds_lfht_node **')
invalid)
uatomic_or_mo(node_next, REMOVED_FLAG, CMM_RELEASE);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../include/urcu/uatomic/builtins-generic.h:123:10: note: expanded from macro
'uatomic_or_mo'
(void) __atomic_or_fetch(cmm_cast_volatile(addr), mask, \
^ ~~~~~~~~~~~~~~~~~~~~~~~
rculfhash.c:1440:3: error: address argument to atomic operation must be a
pointer to integer ('typeof (fini_bucket_next)' (aka 'struct cds_lfht_node **')
invalid)
uatomic_or(fini_bucket_next, REMOVED_FLAG);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../include/urcu/uatomic/builtins-generic.h:130:2: note: expanded from macro
'uatomic_or'
uatomic_or_mo(addr, mask, CMM_RELAXED)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../include/urcu/uatomic/builtins-generic.h:123:10: note: expanded from macro
'uatomic_or_mo'
(void) __atomic_or_fetch(cmm_cast_volatile(addr), mask, \
^ ~~~~~~~~~~~~~~~~~~~~~~~
Eh I thought we fixed that. Clang is very strict about these things.
You can apply the following
<https://review.lttng.org/c/userspace-rcu/+/10911/1>. That ought to fix
the issue until we merge the patch.
Fix merged into liburcu master, thanks!
Mathieu
--
Mathieu Desnoyers
EfficiOS Inc.
https://www.efficios.com
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev