[ovs-dev] [PATCH v5 0/2] flow dumps containing igmp match cannot be restored

Salvatore Daniele sdaniele at redhat.com
Thu Nov 4 19:39:44 UTC 2021


match_format() prints the keyword "igmp" for flows with the field
"ip,nw_proto=2". ofp_parse_protocol does not accept this value, or
the values igmp_type or igmp_code.

This results in flow dump restoration failing when the ovs-save script
is used by "ovs-ctl restart" on a dump of flows containing this match.
However, removing the "igmp" keyword entirely could break existing
scripts in stable branches.

The first patch addresses this issue by providing a workaround within
ovs-save to preserve the 'igmp' keywords while allowing flows to be
restored. This change would be backported to all stable branches.

The second patch removes the 'igmp' outputs entirely, replacing it with
'ip,nw_proto=2'. This has been added to NEWS, and would be applied
in master branch only.

While it might make sense to eventually remove the ovs-save workaround,
it remains in this second patch to ensure flows can be restored when
upgrading ovs-vswitchd from older versions.

v5
- Handle igmp_type/code in workaround and remove them from match.c in
  2/2
v4
- Include the ovs-save workaround in both patch 1/2 and 2/2 to address 
  upgrade case discussed in v3
v3
- Fixed typos to be inline with DCO policy
- Rebased ontop of latest master
v2
- Address comments made on v1 with regard to a work around for stable 
  branches

Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1982743

Signed-off-by: Salvatore Daniele <sdaniele at redhat.com>
Signed-off-by: Adrian Moreno <amorenoz at redhat.com>
Co-authored-by: Adrian Moreno <amorenoz at redhat.com>

Adrian Moreno (1):
  Match: Do not print "igmp" match keyword

Salvatore Daniele (1):
  ovs-save: Save igmp flows in ofp_parse syntax

 NEWS               | 2 ++
 lib/match.c        | 6 ------
 tests/ovs-ofctl.at | 6 ++++++
 utilities/ovs-save | 5 ++++-
 4 files changed, 12 insertions(+), 7 deletions(-)

-- 
2.31.1



More information about the dev mailing list