> > +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

Reply via email to