Skip to content

bmcweb: Route PCIe/DIMM/Storage/CBS by host - 2x1P - #78

Open
ashok2ku wants to merge 1 commit into
integ_sp8from
RHI_2x1P_Support
Open

bmcweb: Route PCIe/DIMM/Storage/CBS by host - 2x1P#78
ashok2ku wants to merge 1 commit into
integ_sp8from
RHI_2x1P_Support

Conversation

@ashok2ku

@ashok2ku ashok2ku commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

In 2x1P the iodevices-inventory "PcieData" service runs one instance per host, so the service name and object path are host-specific. Add amd_host_inventory.hpp and use it in the Bios, Memory, PCIe and Storage handlers:
host0 : xyz.openbmc_project.PCIe .../inventory/PCIe
hostN : xyz.openbmc_project.PCIe.hostN .../inventory/hostN/PCIe
Host resolution: explicit ?HostNumber=N (1..2) wins; else fall back to
the source USB vNIC so BIOS pushes (which can't set ?HostNumber) reach
the right instance:
usb0 192.168.31.x -> P0 -> host0 (2P) / host1 (2x1P)
usb1 192.168.32.x -> P1 -> host2
Absent/invalid/non-vNIC -> host0, so 2P behaviour is unchanged. Mode is
read from /etc/amd/hosts.d/host0.conf.

Tested: On Morocco

In 2x1P the iodevices-inventory "PcieData" service runs one instance per
host, so the service name and object path are host-specific. Add
amd_host_inventory.hpp and use it in the Bios, Memory, PCIe and Storage
handlers:
  host0 : xyz.openbmc_project.PCIe        .../inventory/PCIe
  hostN : xyz.openbmc_project.PCIe.hostN  .../inventory/hostN/PCIe
Host resolution: explicit ?HostNumber=N (1..2) wins; else fall back to
the source USB vNIC so BIOS pushes (which can't set ?HostNumber) reach
the right instance:
  usb0 192.168.31.x -> P0 -> host0 (2P) / host1 (2x1P)
  usb1 192.168.32.x -> P1 -> host2
Absent/invalid/non-vNIC -> host0, so 2P behaviour is unchanged. Mode is
read from /etc/amd/hosts.d/host0.conf.

Tested: On Morocco

Signed-off-by: Ashok Kumar <ashok2ku@amd.com>
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.

1 participant