Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Dell-Debian9]: Dell Platform Management Controller module not loaded #2006

Closed
chitra-raghavan opened this issue Aug 31, 2018 · 4 comments
Closed
Assignees

Comments

@chitra-raghavan
Copy link
Contributor

Description

Platform Management Controller module not loaded in Dell hardware inDebian9

Steps to reproduce the issue:

  1. show environment

Describe the results you received:

root@sonic-s6100-01:~# show environment
coretemp-isa-0000
Adapter: ISA adapter
Core 0:       +21.0 C  (high = +98.0 C, crit = +98.0 C)
Core 1:       +21.0 C  (high = +98.0 C, crit = +98.0 C)
Core 2:       +21.0 C  (high = +98.0 C, crit = +98.0 C)
Core 3:       +22.0 C  (high = +98.0 C, crit = +98.0 C)

ERROR:root:S6100 Platform Management Controller module not loaded !

Onboard Temperature Sensors:
  CPU:                             /sys/devices/platform/SMF.512/hwmon/hwmon1/temp1_input not found !
Traceback (most recent call last):
  File "/usr/local/bin/platform_sensors.py", line 100, in <module>
    print_temperature_sensors()
  File "/usr/local/bin/platform_sensors.py", line 58, in print_temperature_sensors
    int(get_pmc_register('temp1_input'))/1000, 'C'
ValueError: invalid literal for int() with base 10: 'ERR'

root@sonic-s6100-01:~# lsmod | grep dell
root@sonic-s6100-01:~#

Output of show version:

root@sonic-s6100-01:~# show ver
SONiC Software Version: SONiC.master.0-dirty-20180830.090951
Distribution: Debian 9.5
Kernel: 4.9.0-7-amd64
Build commit: 3f37b96
Build date: Thu Aug 30 09:13:57 UTC 2018
Built by: lonelyfire@ubuntu

Docker images:
REPOSITORY TAG IMAGE ID SIZE
docker-fpm-quagga latest 6ad833ee54dd 264.1 MB
docker-fpm-quagga master.0-dirty-20180830.090951 6ad833ee54dd 264.1 MB
docker-lldp-sv2 latest c03d0fc2d426 270.8 MB
docker-lldp-sv2 master.0-dirty-20180830.090951 c03d0fc2d426 270.8 MB
docker-snmp-sv2 latest 7b463619af86 291.2 MB
docker-snmp-sv2 master.0-dirty-20180830.090951 7b463619af86 291.2 MB
docker-orchagent-brcm latest 25f2951795a8 257.6 MB
docker-orchagent-brcm master.0-dirty-20180830.090951 25f2951795a8 257.6 MB
docker-platform-monitor latest 6416fe799722 283.3 MB
docker-platform-monitor master.0-dirty-20180830.090951 6416fe799722 283.3 MB
docker-teamd latest cb3ea79e20cf 257.3 MB
docker-teamd master.0-dirty-20180830.090951 cb3ea79e20cf 257.3 MB
docker-syncd-brcm latest fa4ccc644bcf 336.4 MB
docker-syncd-brcm master.0-dirty-20180830.090951 fa4ccc644bcf 336.4 MB
docker-database latest eac364fc5f11 252.3 MB
docker-database master.0-dirty-20180830.090951 eac364fc5f11 252.3 MB
docker-dhcp-relay latest ab2d6bb2f6bc 253.7 MB
docker-dhcp-relay master.0-dirty-20180830.090951 ab2d6bb2f6bc 253.7 MB
docker-router-advertiser latest 1c9d6dd0feb8 250 MB
docker-router-advertiser master.0-dirty-20180830.090951 1c9d6dd0feb8 250 MB

@xinliu-seattle
Copy link
Contributor

I am contacting Dell to take a look at this issue.

@lguohan
Copy link
Collaborator

lguohan commented Sep 11, 2018

addressed in #2026

@lguohan
Copy link
Collaborator

lguohan commented Sep 12, 2018

@chitra-raghavan , can you confirm if the issue is addressed or not.

@chitra-raghavan
Copy link
Contributor Author

The platform modules doesnt get loaded when the image is loaded for first time. It gets loaded only after reboot.

root@sonic:~# show environment
coretemp-isa-0000
Adapter: ISA adapter
Core 0:       +21.0 C  (high = +98.0 C, crit = +98.0 C)
Core 1:       +21.0 C  (high = +98.0 C, crit = +98.0 C)
Core 2:       +25.0 C  (high = +98.0 C, crit = +98.0 C)
Core 3:       +24.0 C  (high = +98.0 C, crit = +98.0 C)

ERROR:root:S6100 Platform Management Controller module not loaded !
Traceback (most recent call last):
  File "/usr/local/bin/platform_sensors.py", line 100, in <module>
    print_temperature_sensors()
  File "/usr/local/bin/platform_sensors.py", line 58, in print_temperature_sensors
    int(get_pmc_register('temp1_input'))/1000, 'C'
ValueError
: invalid literal for int() with base 10: 'ERR'
Onboard Temperature Sensors:
  CPU:                             /sys/devices/platform/SMF.512/hwmon/hwmon1/temp1_input not found !
root@sonic:~#
root@sonic:~# show ver
SONiC Software Version: SONiC.HEAD.709-5d23a43
Distribution: Debian 9.5
Kernel: 4.9.0-7-amd64
Build commit: 5d23a43
Build date: Wed Sep 12 14:11:17 UTC 2018
Built by: johnar@jenkins-worker-3

Docker images:
REPOSITORY                 TAG                 IMAGE ID            SIZE
docker-syncd-brcm          HEAD.709-5d23a43    b49d02995620        336.8 MB
docker-syncd-brcm          latest              b49d02995620        336.8 MB
docker-orchagent-brcm      HEAD.709-5d23a43    80a6e8dbcfc5        258.1 MB
docker-orchagent-brcm      latest              80a6e8dbcfc5        258.1 MB
docker-lldp-sv2            HEAD.709-5d23a43    f57ccc6afe6d        271.4 MB
docker-lldp-sv2            latest              f57ccc6afe6d        271.4 MB
docker-dhcp-relay          HEAD.709-5d23a43    18accd42e7fb        254 MB
docker-dhcp-relay          latest              18accd42e7fb        254 MB
docker-database            HEAD.709-5d23a43    3fae32211ef2        252.7 MB
docker-database            latest              3fae32211ef2        252.7 MB
docker-teamd               HEAD.709-5d23a43    103d6a5bb6b4        257.8 MB
docker-teamd               latest              103d6a5bb6b4        257.8 MB
docker-snmp-sv2            HEAD.709-5d23a43    4e55eb4c2dd2        291.2 MB
docker-snmp-sv2            latest              4e55eb4c2dd2        291.2 MB
docker-router-advertiser   HEAD.709-5d23a43    8509c84a90c0        250 MB
docker-router-advertiser   latest              8509c84a90c0        250 MB
docker-platform-monitor    HEAD.709-5d23a43    3b39477b5703        283.5 MB
docker-platform-monitor    latest              3b39477b5703        283.5 MB
docker-fpm-quagga          HEAD.709-5d23a43    b682c6b001c3        264.3 MB
docker-fpm-quagga          latest              b682c6b001c3        264.3 MB

root@sonic:~# lsmod | grep dell
root@sonic:~#
root@sonic:~# reboot
Stopping syncd...
requested COLD shutdown
[  OK  ] Stopped LLDP container.
[  OK  ] Stopped Update minigraph and set configuration based on minigraph.
...
...
...
root@sonic:~#
root@sonic:~# lsmod | grep dell
dell_s6100_lpc         53248  0
dell_s6100_iom_cpld    16384  0
dell_ich               20480  0
root@sonic:~#
root@sonic:~#
root@sonic:~#

@lguohan lguohan closed this as completed Oct 5, 2018
dgsudharsan added a commit to dgsudharsan/sonic-buildimage that referenced this issue Nov 4, 2021
Including the below commits to update swss submodule
8448a60 [vs tests]Migrating sonic-swss tests to use hwsku instead of fakeplatform (sonic-net#1978)
faa26db Fix random failure in PR/CI build. (sonic-net#2006)
e03edb6 Allow interface type value none (sonic-net#1991)
71b9650 [orchagent] Fix group name of port-buffer-drop in flexcounterorch.cpp (sonic-net#1967)
facdef5 [VS test] Skip flaky virtual chassis test (sonic-net#2004)
8261c1f [pytest]: Increase timeout when checking services (sonic-net#2000)
67278be [teammgrd]: Handle LAGs cleanup gracefully on Warm/Fast reboot. (sonic-net#1934)
e92c1df Enable FEC statistics collection for Ethernet ports (sonic-net#1994)
9f30ca1 VxLAN Tunnel Counters and Rates implementation (sonic-net#1859)

Signed-off-by: Sudharsan Dhamal Gopalarathnam <[email protected]>
prsunny pushed a commit that referenced this issue Nov 5, 2021
Including the below commits to update swss submodule
8448a60 [vs tests]Migrating sonic-swss tests to use hwsku instead of fakeplatform (#1978)
faa26db Fix random failure in PR/CI build. (#2006)
e03edb6 Allow interface type value none (#1991)
71b9650 [orchagent] Fix group name of port-buffer-drop in flexcounterorch.cpp (#1967)
facdef5 [VS test] Skip flaky virtual chassis test (#2004)
8261c1f [pytest]: Increase timeout when checking services (#2000)
67278be [teammgrd]: Handle LAGs cleanup gracefully on Warm/Fast reboot. (#1934)
e92c1df Enable FEC statistics collection for Ethernet ports (#1994)
9f30ca1 VxLAN Tunnel Counters and Rates implementation (#1859)

Signed-off-by: Sudharsan Dhamal Gopalarathnam <[email protected]>
stepanblyschak added a commit to stepanblyschak/sonic-buildimage that referenced this issue Nov 11, 2021
```
5f8ebfa (HEAD, origin/master, origin/HEAD, master) [AclOrch] move ACL counters to flex counter infrastructure (sonic-net#1943)
8119ec0 [bfdorch] Orchagent support hardware BFD (sonic-net#1883)
15074ac [sonic-swss]:enable unconfiguring PFC on last TC on a port (sonic-net#1962)
05c7c05 [Mux orch] set default as standby, change mux orch priority (sonic-net#2010)
fe5b2a9 [pytest]: Ignore errors deleting host ifs (sonic-net#2005)
70da9af [ci]: use native arm64 and armhf pool (sonic-net#2013)
e14a071 [qos] Add EXP to TC map support (sonic-net#1954)
c91a7f2 [switchorch] Implement VXLAN src port range feature  (sonic-net#1959)
b20f0f4 Gcov for swss daemon (sonic-net#1737)
01c243a [CRM][MPLS] Fix the mpls nexthop CRM attribute (sonic-net#2008)
8448a60 [vs tests]Migrating sonic-swss tests to use hwsku instead of fakeplatform (sonic-net#1978)
faa26db Fix random failure in PR/CI build. (sonic-net#2006)
e03edb6 Allow interface type value none (sonic-net#1991)
71b9650 [orchagent] Fix group name of port-buffer-drop in flexcounterorch.cpp (sonic-net#1967)
facdef5 [VS test] Skip flaky virtual chassis test (sonic-net#2004)
8261c1f [pytest]: Increase timeout when checking services (sonic-net#2000)
67278be [teammgrd]: Handle LAGs cleanup gracefully on Warm/Fast reboot. (sonic-net#1934)
e92c1df Enable FEC statistics collection for Ethernet ports (sonic-net#1994)
9f30ca1 VxLAN Tunnel Counters and Rates implementation (sonic-net#1859)
ac3103a Add missing neighbor resolution for MPLS route programming (sonic-net#1968)
bfba0ad [vlanmgr]Fix for STATE_DB port check logic (sonic-net#1980)
9ef2ba4 [vlanmgr]: Update VLAN removal code to work with 5.10 kernel and newer iproute2 versions (sonic-net#1970)
41fb26c [Mux orch] Handle setting unknown mux state (sonic-net#1984)
ac09bde [azp]: Increase timeout for VS tests (sonic-net#1988)
da8a43e [pytest]: Check if appl DB exists before deleting (sonic-net#1983)
553d75a [tunnel decap] Change tunnel orch order (sonic-net#1977)
7444e96 [macsecmgr]: Add rekey period in macsec mgr (sonic-net#1958)
d95823d [Buffermgr]Graceful handling of buffer model change (sonic-net#1956)
b0aa6a0 EVPN VxLAN enhancement to support P2MP tunnel based programming for Layer2 extension (sonic-net#1858)
85bdf54 Fix the option missing in kernel config issue (sonic-net#1973)
6b15584 Orchagent validates mirror session queue parameter against maximum value from SAI (sonic-net#1957)
fc9ffb9 [copp] Add ISIS, LDP and micro-BFD trap types to CoPP manager (sonic-net#1890)
452cbc1 [macsecorch]: Add IPG adjusting for MACsec gearbox model (sonic-net#1925)
```

Signed-off-by: Stepan Blyschak <[email protected]>
qiluo-msft added a commit that referenced this issue Dec 31, 2021
Includes below commits
```
38b616ba 2021-12-09 | [LGTM] lgtm to use 202106 branches of swss-common and sairedis (#2074) [Stephen Sun]
c68bad07 2021-12-07 | Fix random failure in PR/CI build. #2006 [Shilong Liu]
ba17675b 2021-08-23 | [ci]: fix artifacts download from swss-common and sairedis (#1882) [Guohan Lu]
```
vivekrnv pushed a commit to vivekrnv/sonic-buildimage that referenced this issue Jan 14, 2022
fa23bfc [Barefoot] Added CLI to list/set P4 profile (sonic-net#1951)
fc0ecb6 [GCU] Disallowing DeleteInsteadOfReplaceMoveExtender from generating delete whole config move (sonic-net#2006)
dd8c8fe [GCU] Show default option for '--format' (sonic-net#2003)

Signed-off-by: Vivek Reddy Karri <[email protected]>
judyjoseph added a commit that referenced this issue Jan 17, 2022
dd71848 [GCU] Show default option for '--format' (#2003)
f296e76 [GCU] Disallowing DeleteInsteadOfReplaceMoveExtender from generating delete whole config move (#2006)
731d643 [flow counter] Fix issue: should not compare str with int (#2001)
e628f01 Support CLI for buffer queue configuration (#1965)
585fd40 Fix show ip bgp nei command rw required issue (#2011)
theasianpianist pushed a commit to theasianpianist/sonic-buildimage that referenced this issue Feb 5, 2022
*step to clean work space, to avoid random failure in azure pipeline.
taras-keryk pushed a commit to taras-keryk/sonic-buildimage that referenced this issue Apr 28, 2022
…delete whole config move (sonic-net#2006)

#### What I did
Not generating delete whole config move because it is not allowed by JsonPatch library and it is not possible for ConfigDb to be equal to NULL.

This solves the first problem in issue sonic-net/sonic-utilities#2000 (comment)

#### How I did it
- Verified UpperLevelMoveExtender does not generate delete whole config move
- Modified DeleteInsteadOfReplaceMoveExtender to not generate delete whole config move
- DeleteRefsMoveExtender cannot generate delete whole config move, because it deletes the referrer leafs

#### How to verify it
unit-test

#### Previous command output (if the output of a command-line utility has changed)

#### New command output (if the output of a command-line utility has changed)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants