Minor bug fix

dhanasekar-arista · dhanasekar-arista · commit 29a6bc4d81de · 2025-12-04T14:44:37.000Z
diff --git a/orchagent/portsorch.cpp b/orchagent/portsorch.cpp
@@ -3812,9 +3812,9 @@ bool PortsOrch::initPort(const PortConfig &port)
                 }
                 if (flex_counters_orch->getPortAttrCountersState())
                 {
-                    if (p.m_type == Port::Type::PHY && supportsPortAttr(p.m_port_id, p.m_alias.c_str()))
+                    if (p.m_type == Port::Type::PHY && !m_supported_phy_attrs.empty())
                     {
-                        if (!m_supported_phy_attrs.empty())
+                        if (supportsPortAttr(p.m_port_id, p.m_alias.c_str()))
                         {
                             auto port_attr_stats = generateCounterStats(m_supported_phy_attrs, sai_serialize_port_attr);
                             port_attr_manager.setCounterIdList(p.m_port_id,

Original file line number	Diff line number	Diff line change
`@@ -3812,9 +3812,9 @@ bool PortsOrch::initPort(const PortConfig &port)`
`3812`	`3812`	`}`
`3813`	`3813`	`if (flex_counters_orch->getPortAttrCountersState())`
`3814`	`3814`	`{`
`3815`		`- if (p.m_type == Port::Type::PHY && supportsPortAttr(p.m_port_id, p.m_alias.c_str()))`
	`3815`	`+ if (p.m_type == Port::Type::PHY && !m_supported_phy_attrs.empty())`
`3816`	`3816`	`{`
`3817`		`- if (!m_supported_phy_attrs.empty())`
	`3817`	`+ if (supportsPortAttr(p.m_port_id, p.m_alias.c_str()))`
`3818`	`3818`	`{`
`3819`	`3819`	`auto port_attr_stats = generateCounterStats(m_supported_phy_attrs, sai_serialize_port_attr);`
`3820`	`3820`	`port_attr_manager.setCounterIdList(p.m_port_id,`