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 S6000: wrong lane mapping in port_config.ini #2276

Closed
vivekvijaykumar opened this issue Nov 19, 2018 · 5 comments
Closed

Dell S6000: wrong lane mapping in port_config.ini #2276

vivekvijaykumar opened this issue Nov 19, 2018 · 5 comments

Comments

@vivekvijaykumar
Copy link

vivekvijaykumar commented Nov 19, 2018

Description

admin@sonic:~$ show platform sum
Platform: x86_64-dell_s6000_s1220-r0
HwSKU: Force10-S6000
ASIC: broadcom

I am noticing that the lane maps for the first 6 ports seemed to be swapped based on LLDP neighbors and also by manually checking it by doing single port shut/no shut test.

current lane map is as follows -

Ethernet0 29,30,31,32 40G
Ethernet4 25,26,27,28 40G
Ethernet8 37,38,39,40 40G
Ethernet12 33,34,35,36 40G
Ethernet16 41,42,43,44 40G
Ethernet20 45,46,47,48 40G

I believe based on LLDP and manual shut/no shut tests the lane maps should be -

Ethernet0 25,26,27,28
Ethernet4 29,30,31,32
Ethernet8 33,34,35,36
Ethernet12 37,38,39,40
Ethernet16 45,46,47,48
Ethernet20 41,42,43,44

Could we verify that the current lane map for this platform on all ports is valid ?

Steps to reproduce the issue:

  1. Install latest sonic on Dell_S6000 <x86_64-dell_s6000_s1220-r0>
  2. Wait for all ports to be up with a neighboring 40G platform
  3. Do individual shut/no shut and see the peer ports that flap. The wiring will show as though interfaces are cross wired

Describe the results you received:

Describe the results you expected:

Additional information you deem important (e.g. issue happens only occasionally):

```

root@sonic:/home/admin# cat /host/machine.conf
onie_version=3.20.1.3
onie_vendor_id=674
onie_platform=x86_64-dell_s6000_s1220-r0
onie_machine=dell_s6000_s1220
onie_machine_rev=0
onie_arch=x86_64
onie_config_version=1
onie_build_date="2015-05-03T16:20-0700"
onie_partition_type=gpt
onie_kernel_version=3.2.35
```

**Attach debug file `sudo generate_dump`:**

```
(paste your output here)
```
@vivekvijaykumar vivekvijaykumar changed the title Dell S6000 lane wrong lane maps in port_config.ini Dell S6000: wrong lane mapping in port_config.ini Nov 19, 2018
@vivekvijaykumar
Copy link
Author

@vharish02 Can we please get someone from Dell to take a look?

@lguohan
Copy link
Collaborator

lguohan commented Nov 20, 2018

you must be using s6000-on. the one in the buildimage is for s6000. they are the same hardware, but dell have different naming convention on the front panel port. in fact, all ports are swapped.

also s6000 have different eeprom format than s6000-on, that is why decode-syseeprom is not working.

@vivekvijaykumar
Copy link
Author

@lguohan , thanks for the update . I see on the sonic wiki for support hw that Dell S600-ON is listed as supported. It also says Dell S6100-ON is supported . Since the SONIC buildimage is building for S6000, could we update the wiki so it avoids confusion (given that the onie_platform name is also identical between S6000 and S6000-ON)

@Anandaraj-Maharajan
Copy link
Collaborator

@vivekvijaykumar We are looking in to it. We will make necessary modifications for S6000-ON and raise a PR soon. Update from normal FTOS to SONiC in normal is not a straight forward step.

@yxieca
Copy link
Contributor

yxieca commented Sep 19, 2019

Information has been provided by Guohan's comments.

@yxieca yxieca closed this as completed Sep 19, 2019
liat-grozovik pushed a commit that referenced this issue Jul 28, 2022
feeac84 [counters] Keep counters cache in a single directory (#2232)
cd7909e Updated the CLI file for Radius and Linux kernel commands (#2276)
cd6898b Update db_migrator to support `PORT_QOS_MAP|global` (#2205)
8f4fc62 Revert "Revert "[scripts/fast-reboot] Shutdown remaining containers through systemd (#2133)" (#2161)" (#2184)
27667cf Fix test for pfcwd_sw_enable in db_migrator_test (#2253)
e1866e3 [MultiAsic] sudo reboot command doesn't gracefully stop Asic syncd# on multiasic platform (#2258)
9e3ba82 Change db_migrator major version on master branch from version 2 to 3 (#2272)
65346ce Microsoft mandatory file (#2177)
308e25f portconfig option to configure Tx power and laser frequency of ZR transceiver module (#2197)
e49b1e8 [vnet_route_check] Align DB data parse logic with format used by swsscommon API (#2268)

Signed-off-by: Stepan Blyschak <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants