The additional requested comment was not included in commit 273a4fce951a. This patch adds the missing comment.
Related commit: 273a4fce951a ("netdev-offload-tc: Only install recirc flows if the parent is present.") Signed-off-by: Eelco Chaudron <echau...@redhat.com> --- lib/netdev-offload-tc.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/lib/netdev-offload-tc.c b/lib/netdev-offload-tc.c index 985bb80a3..804885285 100644 --- a/lib/netdev-offload-tc.c +++ b/lib/netdev-offload-tc.c @@ -2699,6 +2699,10 @@ netdev_tc_flow_put(struct netdev *netdev, struct match *match, struct tc_action *action = flower.actions; for (int i = 0; i < flower.action_count; i++, action++) { + /* If action->chain is zero, this is not a true "goto" + * (recirculation) action, but rather a drop action. + * Since it does not involve recirculation handling, + * it should be ignored. */ if (action->type == TC_ACT_GOTO && action->chain) { chain_goto = action->chain; ovs_mutex_lock(&used_chains_mutex); -- 2.47.1 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev