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

Added new Platform APIs and modified APIs for supporting reboot on a SmartSwitch #501

Merged
merged 2 commits into from
Sep 27, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 30 additions & 1 deletion sonic_platform_base/module_base.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,8 @@ class ModuleBase(device_base.DeviceBase):
MODULE_REBOOT_CPU_COMPLEX = "CPU"
# Module reboot type to reboot FPGA complex
MODULE_REBOOT_FPGA_COMPLEX = "FPGA"
# Module reboot type to reboot DPU
MODULE_REBOOT_DPU = "DPU"

def __init__(self):
# List of ComponentBase-derived objects representing all components
Expand Down Expand Up @@ -166,7 +168,9 @@ def reboot(self, reboot_type):
Args:
reboot_type: A string, the type of reboot requested from one of the
predefined reboot types: MODULE_REBOOT_DEFAULT, MODULE_REBOOT_CPU_COMPLEX,
or MODULE_REBOOT_FPGA_COMPLEX
MODULE_REBOOT_FPGA_COMPLEX or MODULE_REBOOT_DPU

MODULE_REBOOT_DPU is only applicable for smartswitch chassis.

Returns:
bool: True if the request has been issued successfully, False if not
Expand Down Expand Up @@ -258,6 +262,31 @@ def get_state_info(self):
"""
raise NotImplementedError

def get_bus_info(self, module_name):
"""
Retrieves the bus information for the specified by "module_name" on a SmartSwitch.
vvolam marked this conversation as resolved.
Show resolved Hide resolved

Returns:
Returns the PCI bus information in BDF format like "[DDDD:]BB:SS:F"
"""
raise NotImplementedError

def pci_detach(self, module_name):
"""
Detaches the DPU PCI device specified by "module_name" on a SmartSwitch.

Returns: True once the PCI is successfully detached.
vvolam marked this conversation as resolved.
Show resolved Hide resolved
"""
raise NotImplementedError

def pci_reattach(self, module_name):
"""
Rescans and reconnects the DPU PCI device specified by "module_name" on a SmartSwitch.

Returns: True once the PCI is successfully reconnected.
vvolam marked this conversation as resolved.
Show resolved Hide resolved
"""
raise NotImplementedError

##############################################
# Component methods
##############################################
Expand Down
3 changes: 3 additions & 0 deletions tests/module_base_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@ def test_module_base(self):
[module.get_dpu_id],
[module.get_reboot_cause],
[module.get_state_info],
[module.get_bus_info, "module_name"],
[module.pci_detach, "module_name"],
[module.pci_reattach, "module_name"],
]

for method in not_implemented_methods:
Expand Down
Loading