Skip to content

[Juniper][QFX5210] Updating preemphasis values for supported optics#3686

Merged
lguohan merged 19 commits intosonic-net:masterfrom
ciju-juniper:master
Nov 5, 2019
Merged

[Juniper][QFX5210] Updating preemphasis values for supported optics#3686
lguohan merged 19 commits intosonic-net:masterfrom
ciju-juniper:master

Conversation

@ciju-juniper
Copy link
Contributor

This patch set adds the following changes:

  • Preemphasis values for various optics supported in qfx5210
  • Updating the platform README

This switch has 64 QSFP28 (40G/100G) ports, 2 SFP+ (1G/10G) ports
on Broadcom Tomahawk II chipset. CPU used in QFX5210-64C-S is
Intel Broadwell-DE. The machine has Redundant and hot-swappable
Power Supply (1+1) and also has Redundant and hot swappable fans (3+1).

Signed-off-by: Ciju Rajan K <[email protected]>
Signed-off-by: Ciju Rajan K <[email protected]>
This commit implements the following changes
 - Moving to optoe driver for sfp management
 - Removing the old sfp driver
 - Updating the port-config.ini to add the index field
 - Correction in sfputil.py to incorporate optoe driver
 - Platform support for 'poweroff' command

Signed-off-by: Ciju Rajan K <[email protected]>
Orchagent is crashing on the latest SONiC images. Issue #458
is raised to track the problem. Dynamic port breakout
commit 6f40933 introduced
this regression. One of the recommendation that we received
was to disable the loopback and mgmt ports in bcm config.
It was helpful in fixing the issue but not the ideal solution.
It works on qfx5210 as the loopback and management configurations
are not applicable.

While the orchagent crash is being debugged, we need this commit
to fix the issue in qfx5210 platform. This patch can be reversed
once the correct fix for the orchagent is identified.

Signed-off-by: Ciju Rajan K <[email protected]>
This patch does the following:
 - Storing eeprom contents including the vendor extensions
   into a text file
 - Implement the get routines for reading various fields
   from eeprom

Signed-off-by: Ciju Rajan K <[email protected]>
This patch implements the following:
 - Added logging facility in juniper_qfx5210_util file
 - Changed the path of juniper_qfx5210_monitor.log to /var/log
 - Invoke "poweroff" command when temperature reaches fire threshold
 - Display warning messages in the console before system shutdown

Signed-off-by: Ciju Rajan K <[email protected]>
There are primary and secondary bios in qfx5210 platform.
There is a problem which prevents the OS booting from the
secondary bios when the OS was installed using primary bios.
Secondary bios fails to detect the UEFI partition. Right now
the workaround is to have a folder structure /EFI/BOOT/BOOT64x.efi

Signed-off-by: Ciju Rajan K <[email protected]>
There is a cpld swizzle logic which will reboot qfx5210
platform if the swizzle logic was not disabled in five minutes.

Signed-off-by: Ciju Rajan K <[email protected]>
This patch set implements the following:
 - Adding the cpld soft reset as the platform reboot handler
 - Fixing the cpld access issues

Signed-off-by: Ciju Rajan K <[email protected]>
Added the documentation for the platform poweroff/reboot

Signed-off-by: Ciju Rajan K <[email protected]>
This patch adds the preemphasis values for the various supported
optics for qfx5210 platform

Signed-off-by: Ciju Rajan K <[email protected]>
@ciju-juniper
Copy link
Contributor Author

@lguohan Please merge this patch set.

@lguohan lguohan merged commit a4c35b7 into sonic-net:master Nov 5, 2019
yejianquan pushed a commit that referenced this pull request Jun 20, 2025
…lly (#22927)

Why I did it
src/sonic-swss

* 3e39104 - (HEAD -> master, origin/master, origin/HEAD) add support to handle ha notifications (#3659) (7 hours ago) [Jing Zhang]
* 3423a49 - Improve orchagent to enable Dash ZMQ by feature flag. (#3619) (9 hours ago) [Hua Liu]
* f38f6e4 - Orchagent SAI error handling improvements (#3587) (30 hours ago) [prabhataravind]
* e653cdd - SRv6: set TTL decap mode to PIPE (#3686) (7 days ago) [Yakiv Huryk]
* fd0562a - [FC] Fix the cache to handle change in stats for the same object type (#3654) (8 days ago) [Vivek]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants