Enabled untagged vs default functionality 86/15086/3
authorMike Bly <mbly@ciena.com>
Mon, 24 Sep 2018 17:13:06 +0000 (10:13 -0700)
committerJohn Lo <loj@cisco.com>
Tue, 2 Oct 2018 22:14:25 +0000 (22:14 +0000)
Removed 0-tags attribute for default-sub-if config
Moved default-sub-if check before untagged

Change-Id: I68043445aa2f79846e0743567b9015257fd87f8d
Signed-off-by: Mike Bly <mbly@ciena.com>
src/vnet/ethernet/ethernet.h
src/vnet/ethernet/node.c

index cc44c4e..ece7aa5 100644 (file)
@@ -529,13 +529,13 @@ eth_identify_subint (vnet_hw_interface_t * hi,
   if ((subint->flags & match_flags) == match_flags)
     goto matched;
 
-  // check for untagged interface
-  subint = &main_intf->untagged_subint;
+  // check for default interface
+  subint = &main_intf->default_subint;
   if ((subint->flags & match_flags) == match_flags)
     goto matched;
 
-  // check for default interface
-  subint = &main_intf->default_subint;
+  // check for untagged interface
+  subint = &main_intf->untagged_subint;
   if ((subint->flags & match_flags) == match_flags)
     goto matched;
 
index 383d5d2..3cc501d 100755 (executable)
@@ -850,8 +850,7 @@ ethernet_sw_interface_get_config (vnet_main_t * vnm,
   else if (si->sub.eth.flags.default_sub)
     {
       subint = &main_intf->default_subint;
-      *flags = SUBINT_CONFIG_MATCH_0_TAG |
-       SUBINT_CONFIG_MATCH_1_TAG |
+      *flags = SUBINT_CONFIG_MATCH_1_TAG |
        SUBINT_CONFIG_MATCH_2_TAG | SUBINT_CONFIG_MATCH_3_TAG;
     }
   else if ((si->sub.eth.flags.no_tags) || (si->sub.eth.raw_flags == 0))