Skip to content

Add negative case for pci_detach() and pci_reattach()#504

Merged
prgeor merged 7 commits intosonic-net:masterfrom
vvolam:ss-reboot
Nov 9, 2024
Merged

Add negative case for pci_detach() and pci_reattach()#504
prgeor merged 7 commits intosonic-net:masterfrom
vvolam:ss-reboot

Conversation

@vvolam
Copy link
Contributor

@vvolam vvolam commented Oct 9, 2024

Address the comments added in #501

Description

  • Add negative for pci_detach() and pci_reattach()
  • Remove module_name from get_bus_info(), pci_reattach() and pci_detach()
  • Add MODULE_TYPE_SMARTSWITCH to reboot API. This reboot_type is used for rebooting DPUs when entire smartswitch undergoes for a reboot.
  • Also fix unittests accordingly.

Motivation and Context

How Has This Been Tested?

Additional Information (Optional)

Also, remove module_name from pci_detach() and get_bus_info()
@vvolam vvolam changed the title Add negative case for pci_attach() and pci_reattach() Add negative case for pci_detach() Nov 4, 2024
@vvolam vvolam requested review from gpunathilell and rameshraghupathy and removed request for gpunathilell November 4, 2024 14:53
@prgeor
Copy link
Collaborator

prgeor commented Nov 5, 2024

@vvolam please add more details in the PR description.

@vvolam vvolam changed the title Add negative case for pci_detach() Add negative case for pci_detach() and pci_reattach() Nov 8, 2024
Copy link
Contributor

@rameshraghupathy rameshraghupathy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok to me

@prgeor prgeor merged commit e79279b into sonic-net:master Nov 9, 2024
@vvolam vvolam deleted the ss-reboot branch November 9, 2024 03:58
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

Successfully merging this pull request may close these issues.

5 participants