On Wed, Jan 11, 2023 at 9:34 AM Eli Britstein <[email protected]> wrote: > >> >- I was doing a quick check on dpdk offload in OVS. > >> >Unless I missed something, full offloading is broken because of the > >> >use of mixed ingress and transfer attributes. > >> >Unfortunately, the issue on ingress attribute is still present with > >> >this series applied. > >> >Do you have a fix for this too? > >> @@ -2242,7 +2242,7 @@ > >> struct nlattr *nl_actions, > >> size_t actions_len) { > >> - const struct rte_flow_attr flow_attr = { .ingress = 1, .transfer = 1 > >> }; > >> + const struct rte_flow_attr flow_attr = { .transfer = 1 }; > >> struct flow_actions actions = { > >> .actions = NULL, > >> .cnt = 0, > > > >Well, that's what I did as a hack, and it seemed enough. > >Should this simple fix go as is, do we need Ivan series as well? > Ivan's series is orthogonal to this. > This line fix should be part of the adaptations to DPDK 22.11.
Ok, I posted a patch. Please review. Thanks. -- David Marchand _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
