Skip to content

Commit 276b9ae

Browse files
authored
Re-enable support for new SAI API sai_query_attribute_capability() (sonic-net#651)
Previously the code to support query SAI attribute capabilities were commented out due to discovered missing BRCM Vendor SAI API missing in their 3.7.5.1 debian package that caused linking error during image build. See sonic-net#645 for details on the SAI attribute capability query support. Since BRCM has provided the patch for the missing API stub code and included them in BRCM SAI 3.7.5.1-2 (see sonic-net#5101), I am raising this PR to re-enable the calling to Vendor SAI API sai_query_attribute_capability(). I have tested this with the new BRCM SAI 3.7.5.1-2 debians and ensured that the linking issue was fixed by that patch.
1 parent 6938e40 commit 276b9ae

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

syncd/VendorSai.cpp

-4
Original file line numberDiff line numberDiff line change
@@ -951,15 +951,11 @@ sai_status_t VendorSai::queryAttributeCapability(
951951
SWSS_LOG_ENTER();
952952
VENDOR_CHECK_API_INITIALIZED();
953953

954-
#if 0 // Wait until BRCM fix their SAI issue then enable this
955954
return sai_query_attribute_capability(
956955
switchId,
957956
objectType,
958957
attrId,
959958
capability);
960-
#else
961-
return SAI_STATUS_NOT_IMPLEMENTED;
962-
#endif
963959
}
964960

965961
sai_status_t VendorSai::queryAattributeEnumValuesCapability(

0 commit comments

Comments
 (0)