From d27134787a465bab5e2839bfd259de9662d04638 Mon Sep 17 00:00:00 2001 From: Yurii Lisovskyi Date: Wed, 13 Sep 2023 16:21:00 +0300 Subject: [PATCH] Remove unnecessary adding BPs to default VLAN on init Signed-off-by: Yurii Lisovskyi --- common/sai_npu.py | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/common/sai_npu.py b/common/sai_npu.py index 5a380aeb..761bb241 100644 --- a/common/sai_npu.py +++ b/common/sai_npu.py @@ -70,13 +70,12 @@ def init(self, attr): assert len(self.dot1q_bp_oids) > 0 assert self.dot1q_bp_oids[0].startswith("oid:") - if self.sku_config is None: - # The ports will not be re-created. - # Make sure the bridge ports are added into the default VLAN. - for bp_oid in self.dot1q_bp_oids: - vlan_mbr_oid = self.get_vlan_member(self.default_vlan_oid, bp_oid) - if vlan_mbr_oid == None: - self.create_vlan_member(self.default_vlan_oid, bp_oid, "SAI_VLAN_TAGGING_MODE_UNTAGGED") + # The ports will not be re-created. + # Make sure the bridge ports are added into the default VLAN. + for bp_oid in self.dot1q_bp_oids: + vlan_mbr_oid = self.get_vlan_member(self.default_vlan_oid, bp_oid) + if vlan_mbr_oid == None and self.sku_config is None: + self.create_vlan_member(self.default_vlan_oid, bp_oid, "SAI_VLAN_TAGGING_MODE_UNTAGGED") # Update SKU if self.sku_config is not None: @@ -207,7 +206,8 @@ def set_sku_mode(self, sku): # Remove existing ports num_ports = len(self.dot1q_bp_oids) for idx in range(num_ports): - self.remove_vlan_member(self.default_vlan_oid, self.dot1q_bp_oids[idx]) + if self.get_vlan_member(self.default_vlan_oid, self.dot1q_bp_oids[idx]): + self.remove_vlan_member(self.default_vlan_oid, self.dot1q_bp_oids[idx]) self.remove(self.dot1q_bp_oids[idx]) oid = self.get(self.port_oids[idx], ["SAI_PORT_ATTR_PORT_SERDES_ID"]).oid() if oid != "oid:0x0":