V7 is up, sorrty about that. ________________________________ From: Simon Horman <simon.hor...@netronome.com> Sent: Thursday, December 19, 2019 10:27 PM To: Paul Blakey <pa...@mellanox.com> Cc: Roi Dayan <r...@mellanox.com>; Oz Shlomo <o...@mellanox.com>; Marcelo Ricardo Leitner <mleit...@redhat.com>; Justin Pettit <jpet...@nicira.com>; Ilya Maximets <i.maxim...@ovn.org>; Ben Pfaff <b...@ovn.org>; d...@openvswitch.org <d...@openvswitch.org> Subject: Re: [ovs-dev][ovs-dev][PATCH v6 07/10] netdev-offload-tc: Add recirculation support via tc chains
On Thu, Dec 19, 2019 at 04:57:37PM +0200, Paul Blakey wrote: > Each recirculation id will create a tc chain, and we translate > the recirculation action to a tc goto chain action. > > We check for kernel support for this by probing OvS Datapath for the > tc recirc id sharing feature. If supported, we can offload rules > that match on recirc_id, and recirculation action safely. > > Signed-off-by: Paul Blakey <pa...@mellanox.com> > Reviewed-by: Roi Dayan <r...@mellanox.com> ... > @@ -1427,6 +1443,11 @@ netdev_tc_flow_put(struct netdev *netdev, struct match > *match, > } > } > > + if ((chain || recirc_act) && !info.recirc_id_shared_with_tc) { Hi Paul, Travis is a reporting an error on this line, pehaps 'info.' should be 'info->' ? https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftravis-ci.org%2Fhorms2%2Fovs%2Fjobs%2F627292953%23L1391&data=02%7C01%7Cpaulb%40mellanox.com%7Ce05baf3967c74c2b745008d784c1d26c%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C637123840305692270&sdata=FeEL7bcYJLovSBecMLjOaIjsE8Dkf4k0b019vpkvr5Y%3D&reserved=0 > + VLOG_ERR_RL(&error_rl, "flow_put: recirc_id sharing not supported"); > + return EOPNOTSUPP; > + } > + > if (get_ufid_tc_mapping(ufid, &id) == 0) { > VLOG_DBG_RL(&rl, "updating old handle: %d prio: %d", > id.handle, id.prio); > ... _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev