On 9/16/20 2:18 PM, Dumitru Ceara wrote: [...]
> >> + hmap_remove(&lfrr->ref_lflow_table, &lrln->rlfn->node); >> + ref_lflow_node_destroy(lrln->rlfn); > > As mentioned on v1 (at least for me) it's really hard to keep track of > what's going on due to the variable/field names, e.g., "lrln->rlfn". I'd > really like it if we had somewhat more explicit names. > This can be done as a follow up patch. I guess the goal is to fix the crash as soon as possible. Refactoring should come afterwards. Thanks, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
