On 2/16/22 15:27, Adrian Moreno wrote: > Rewrite hmap's loops using multi-variable helpers. > > For SAFE loops, use the LONG version of the multi-variable macros to > keep backwards compatibility. > > Signed-off-by: Adrian Moreno <amore...@redhat.com> > ---
A couple tiny nits, with those addressed: Acked-by: Dumitru Ceara <dce...@redhat.com> > diff --git a/lib/ovs-numa.h b/lib/ovs-numa.h > index ecc251a7f..83bd10cca 100644 > --- a/lib/ovs-numa.h > +++ b/lib/ovs-numa.h > @@ -68,9 +68,9 @@ void ovs_numa_dump_destroy(struct ovs_numa_dump *); > int ovs_numa_thread_setaffinity_core(unsigned core_id); > > #define FOR_EACH_CORE_ON_DUMP(ITER, DUMP) \ > - HMAP_FOR_EACH((ITER), hmap_node, &(DUMP)->cores) > + HMAP_FOR_EACH (ITER, hmap_node, &(DUMP)->cores) Nit: kind of unrelated. > > #define FOR_EACH_NUMA_ON_DUMP(ITER, DUMP) \ > - HMAP_FOR_EACH((ITER), hmap_node, &(DUMP)->numas) > + HMAP_FOR_EACH (ITER, hmap_node, &(DUMP)->numas) Nit: here too. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev