119789719d
* [Mellanox]Check dmi file permission before access (#11309) Signed-off-by: Sudharsan Dhamal Gopalarathnam sudharsand@nvidia.com Why I did it During the system boot up when 'show platform status' or 'show version' command is executed before STATE_DB CHASSIS_INFO table is populated, the show will try to fallback to use the platform API. The DMI file in mellanox platforms require root permission for access. So if the show commands are executed as admin or any other user, the following error log will appear in the syslog Jun 28 17:21:25.612123 sonic ERR show: Fail to decode DMI /sys/firmware/dmi/entries/2-0/raw due to PermissionError(13, 'Permission denied') How I did it Check the file permission before accessing it. How to verify it Added UT to verify. Manually verified if the error log is not thrown. |
||
|---|---|---|
| .. | ||
| barefoot | ||
| broadcom | ||
| cavium | ||
| centec | ||
| centec-arm64 | ||
| checkout | ||
| components | ||
| generic | ||
| innovium | ||
| marvell | ||
| marvell-arm64 | ||
| marvell-armhf | ||
| mellanox | ||
| nephos | ||
| p4 | ||
| pddf | ||
| template | ||
| vs | ||