On 11/7/24 12:22, Lorenzo Bianconi wrote:
>> Also, should we instead add a new module?  E.g., ecmp-next-hop-monitor.c.?
> it is just few lines of code, does it worth to do so?

Maybe not, but the problem I have now is that the variable definition is
far away from the function and it took me a while to realize that we
don't empty the map between runs.

I'm thinking that if they'd are in a separate module it would be more
obvious.

But I'll leave it up to you.

Thanks,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to