[vslib] add missing port attributes for virtual switch (sonic-net#419)

- add default port admin state attribute - add default port mtu attribute Signed-off-by: Stepan Blyschak <[email protected]>
dmytroxshevchuk · Feb 15, 2019 · 2b91013 · 2b91013
1 parent dcc8688
commit 2b91013
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 0 deletions.
diff --git a/tests/aspell.en.pws b/tests/aspell.en.pws
@@ -107,6 +107,7 @@ mlnx
 mpls
 mutex
 mutexes
+MTU
 namespace
 namespaces
 NHG

diff --git a/vslib/src/sai_vs_switch_BCM56850.cpp b/vslib/src/sai_vs_switch_BCM56850.cpp
@@ -169,6 +169,16 @@ static sai_status_t create_ports()
 
         sai_attribute_t attr;
 
+        attr.id = SAI_PORT_ATTR_ADMIN_STATE;
+        attr.value.booldata = false;     /* default admin state is down as defined in SAI */
+
+        CHECK_STATUS(vs_generic_set(SAI_OBJECT_TYPE_PORT, port_id, &attr));
+
+        attr.id = SAI_PORT_ATTR_MTU;
+        attr.value.u32 = 1514;     /* default MTU is 1514 as defined in SAI */
+
+        CHECK_STATUS(vs_generic_set(SAI_OBJECT_TYPE_PORT, port_id, &attr));
+
         attr.id = SAI_PORT_ATTR_SPEED;
         attr.value.u32 = 10 * 1000;
 

diff --git a/vslib/src/sai_vs_switch_MLNX2700.cpp b/vslib/src/sai_vs_switch_MLNX2700.cpp
@@ -166,6 +166,16 @@ static sai_status_t create_ports()
 
         sai_attribute_t attr;
 
+        attr.id = SAI_PORT_ATTR_ADMIN_STATE;
+        attr.value.booldata = false;     /* default admin state is down as defined in SAI */
+
+        CHECK_STATUS(vs_generic_set(SAI_OBJECT_TYPE_PORT, port_id, &attr));
+
+        attr.id = SAI_PORT_ATTR_MTU;
+        attr.value.u32 = 1514;     /* default MTU is 1514 as defined in SAI */
+
+        CHECK_STATUS(vs_generic_set(SAI_OBJECT_TYPE_PORT, port_id, &attr));
+
         attr.id = SAI_PORT_ATTR_SPEED;
         attr.value.u32 = 40 * 1000;     /* TODO from config */