Acked-by: Paul Blakey <pa...@mellanox.com>

On 27/11/2017 02:26, Ben Pfaff wrote:
Open vSwitch enables the GCC 7+ option that warns about fall-through
switch statements.  This commit fixes newly introduced warnings.

CC: Paul Blakey <pa...@mellanox.com>
Fixes: d6118e628988 ("netdev-tc-offloads: Verify csum flags on dump from tc")
Signed-off-by: Ben Pfaff <b...@ovn.org>
---
  lib/tc.c | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/lib/tc.c b/lib/tc.c
index d3a031237ab1..bfe20ce1a3bd 100644
--- a/lib/tc.c
+++ b/lib/tc.c
@@ -1250,6 +1250,7 @@ csum_update_flag(struct tc_flower *flower,
      switch (htype) {
      case TCA_PEDIT_KEY_EX_HDR_TYPE_IP4:
          flower->csum_update_flags |= TCA_CSUM_UPDATE_FLAG_IPV4HDR;
+        /* Fall through. */
      case TCA_PEDIT_KEY_EX_HDR_TYPE_IP6:
      case TCA_PEDIT_KEY_EX_HDR_TYPE_TCP:
      case TCA_PEDIT_KEY_EX_HDR_TYPE_UDP:
@@ -1269,6 +1270,7 @@ csum_update_flag(struct tc_flower *flower,
                           flower->key.ip_proto);
              break;
          }
+        /* Fall through. */
      case TCA_PEDIT_KEY_EX_HDR_TYPE_ETH:
          return 0; /* success */
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to