longhuan-cisco

Results 27 comments of longhuan-cisco

@prgeor please review (couldn't add to reviewers list directly, thus add here)

> Can you please also test restarting xcvrd to ensure that the table gets deleted and updated accordingly Yea Attached restart test log in description.

@prgeor @mihirpat1 I took care of the existing comments, can you please review?

Added pm table related change, to use this PR to track changes for all three tables: status/dom_sensor/pm. (Since they all touch the similar flow in xcvrd, and some is dependent...

As discussed, I tested the change from this PR, `host_tx_ready` gets retained properly after warm-reboot and link stays up (especially for those CMIS modules). @mihirpat1 @prgeor Could you please continue...

@ jaganbal-a @prgeor @mihirpat1 Please take look at the PR.

@Junchao-Mellanox Unfortunately, I don't think we have resources/cycles on this task for (at least) this quarter, no certainty at this point.

@prgeor @mihirpat1 Could you please review?

@prgeor basic test on 8111 with breakout: ``` root@sonic:/home/cisco# show plat sum Platform: x86_64-8111_32eh_o-r0 HwSKU: Cisco-8111-O64 ASIC: cisco-8000 ASIC Count: 1 Serial Number: FLM2632096P Model Number: 8111-32EH-O Hardware Revision: 1.0...

@mihirpat1 tested on 8101-O32 with 400G AOC: ``` root@sonic:/home/cisco# show plat sum Platform: x86_64-8101_32fh_o-r0 HwSKU: Cisco-8101-O32 ASIC: cisco-8000 ASIC Count: 1 Serial Number: FLM251102C6 Model Number: 8101-32FH-O Hardware Revision: 0.41...