> > +static inline bool > > +pmd_perf_metrics_enabled(const struct dp_netdev_pmd_thread *pmd) > > +{ > > + /* If stores and reads of 64-bit integers are not atomic, the > > + * full PMD performance metrics are not available as locked > > + * access to 64 bit integers would be prohibitively expensive. */ > > + if (sizeof(uint64_t) > sizeof(void *)) { > > + return false; > > + } > > I think you need to use ATOMIC_LLONG_LOCK_FREE or ATOMIC_ALWAYS_LOCK_FREE_8B > macro here instead of manual pointer checking.
I was looking for such a macro available through the ovs_atomic.h header but didn't find one. Will try the ones you propose. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev