On 7/26/21 10:14 AM, Eli Britstein wrote: > This patch-set improves/fixes offloads transitions behavior. > > Patch #1 avoids flushing PMD offloads unnecessarily. > Patch #2 fixes a flow modifications bug. > Patch #3 fixes a race condition with flow modifications. > Patch #3 improves debuggability of flow modifications. > > v2-v1: > - Rebase. > v3-v2: > - Added patch #2 for fixes another flow modification scenario. > - Changed log to a separated function. > > GitHub Actions: > v1: https://github.com/elibritstein/OVS/actions/runs/769805954 > - This run has encountered some internal GitHub problems. > - A previous good run, with the same code, only changed commit > messages since: > https://github.com/elibritstein/OVS/actions/runs/766660787 > v2: https://github.com/elibritstein/OVS/actions/runs/1023045302 > v3: https://github.com/elibritstein/OVS/actions/runs/1066584199 > > Eli Britstein (4): > dpif-netdev: Do not flush PMD offloads on reload > dpif-netdev: Fix flow modification after failure > dpif-netdev: Fix offloads of modified flows > dpif-netdev: Log flow modification in debug level > > lib/dpif-netdev.c | 144 ++++++++++++++++++++++++---------------------- > 1 file changed, 74 insertions(+), 70 deletions(-) >
Thanks! I renamed the function in the patch #4 from 'dump_*' to 'log_*' to avoid confusion with flow dump functions and applied the patch set. I added Ack from David to patch #1 since it's the same as in v2. Patches #2 and #3 backported down to 2.13. Technically, I think, we can consider patch #1 as a bug fix too and backport it down to 2.15. Let me know if that's needed. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
