summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLakshmi Yadlapati <lakshmiy@us.ibm.com>2023-04-21 00:53:59 +0300
committerLakshmi Yadlapati <lakshmiy@us.ibm.com>2023-06-30 23:01:05 +0300
commita5409991398b6b220564dd083e37225b1baf0509 (patch)
tree71fb2f3a564476b7b1e2f385a9d39d9099b33e7d
parentcf3b484e14fe9754c1812d9871d77b5b1ac9f620 (diff)
downloadbmcweb-a5409991398b6b220564dd083e37225b1baf0509.tar.xz
Add PCIe Slot information to PCIeDevice schema
To align with Redfish's transition from using the PCIeSlot schema to including the Slot within the PCIeDevice schema, this commit adds PCIe Slot information to the PCIeDevice schema. The corresponding PCIe Slot is retrieved using the 'contained_by' association, which establishes a link from the PCIeDevice object to the PCIeSlot object. If there is no PCIeSlot associated with the PCIeDevice, the Slot properties will not be returned. Directed associations, ‘containing’ and ‘contained_by’, are used to establish a link between PCIeDevice and PCIeSlot. The 'containing' association establishes a link from a PCIeSlot to the PCIeDevice it contains, while the 'contained_by' association establishes a link from a PCIeDevice to the PCIeSlot that contains it. Additionally, this commit refactors the PCIeDevice schema to improve its structure, readability, and adherence to best practices. Redfish commit: https://redfish.dmtf.org/schemas/v1/PCIeDevice.v1_11_0.json Tested: Validator Passed ''' curl -k https://$bmc/redfish/v1/Systems/system/PCIeDevices/pcie_card7 { "@odata.id": "/redfish/v1/Systems/system/PCIeDevices/pcie_card7", "@odata.type": "#PCIeDevice.v1_9_0.PCIeDevice", "Id": "pcie_card7", "Name": "PCIe Device", "PCIeFunctions": { "@odata.id": "/redfish/v1/Systems/system/PCIeDevices/pcie_card7/PCIeFunctions" }, "PCIeInterface": { "LanesInUse": -1 }, "Slot": { "Lanes": 0, "SlotType": "FullLength" }, "Status": { "Health": "OK", "State": "Absent" } } PCIeDevice with no association: curl -k https://$bmc/redfish/v1/Systems/system/PCIeDevices/pcie_card0 { "@odata.id": "/redfish/v1/Systems/system/PCIeDevices/pcie_card0", "@odata.type": "#PCIeDevice.v1_9_0.PCIeDevice", "Id": "pcie_card0", "Name": "PCIe Device", "PCIeFunctions": { "@odata.id": "/redfish/v1/Systems/system/PCIeDevices/pcie_card0/PCIeFunctions" }, "PCIeInterface": { "LanesInUse": -1 }, "Status": { "Health": "OK", "State": "Absent" } } ''' Change-Id: I15ac33be0035721f44c60fded795092896bce9bd Signed-off-by: Lakshmi Yadlapati <lakshmiy@us.ibm.com> Signed-off-by: Ed Tanous <edtanous@google.com>
-rw-r--r--Redfish.md4
-rw-r--r--redfish-core/lib/pcie.hpp198
2 files changed, 178 insertions, 24 deletions
diff --git a/Redfish.md b/Redfish.md
index 347048dd5f..a7b3445d82 100644
--- a/Redfish.md
+++ b/Redfish.md
@@ -863,6 +863,10 @@ other.
- LanesInUse
- PCIeType
- SerialNumber
+- Slot
+ - Lanes
+ - PCIeType
+ - SlotType
- SparePartNumber
- Status
diff --git a/redfish-core/lib/pcie.hpp b/redfish-core/lib/pcie.hpp
index 82e1a5f1a6..e10ba64643 100644
--- a/redfish-core/lib/pcie.hpp
+++ b/redfish-core/lib/pcie.hpp
@@ -35,6 +35,8 @@ namespace redfish
static constexpr const char* inventoryPath = "/xyz/openbmc_project/inventory";
static constexpr std::array<std::string_view, 1> pcieDeviceInterface = {
"xyz.openbmc_project.Inventory.Item.PCIeDevice"};
+static constexpr std::array<std::string_view, 1> pcieSlotInterface = {
+ "xyz.openbmc_project.Inventory.Item.PCIeSlot"};
static inline void handlePCIeDevicePath(
const std::string& pcieDeviceId,
@@ -146,6 +148,148 @@ inline void requestRoutesSystemPCIeDeviceCollection(App& app)
std::bind_front(handlePCIeDeviceCollectionGet, std::ref(app)));
}
+inline void addPCIeSlotProperties(
+ crow::Response& res, const boost::system::error_code& ec,
+ const dbus::utility::DBusPropertiesMap& pcieSlotProperties)
+{
+ if (ec)
+ {
+ BMCWEB_LOG_ERROR << "DBUS response error for getAllProperties"
+ << ec.value();
+ messages::internalError(res);
+ return;
+ }
+ std::string generation;
+ size_t lanes = 0;
+ std::string slotType;
+
+ bool success = sdbusplus::unpackPropertiesNoThrow(
+ dbus_utils::UnpackErrorPrinter(), pcieSlotProperties, "Generation",
+ generation, "Lanes", lanes, "SlotType", slotType);
+
+ if (!success)
+ {
+ messages::internalError(res);
+ return;
+ }
+
+ std::optional<pcie_device::PCIeTypes> pcieType =
+ pcie_util::redfishPcieGenerationFromDbus(generation);
+ if (!pcieType)
+ {
+ BMCWEB_LOG_WARNING << "Unknown PCIeType: " << generation;
+ }
+ else
+ {
+ if (*pcieType == pcie_device::PCIeTypes::Invalid)
+ {
+ BMCWEB_LOG_ERROR << "Invalid PCIeType: " << generation;
+ messages::internalError(res);
+ return;
+ }
+ res.jsonValue["Slot"]["PCIeType"] = *pcieType;
+ }
+
+ res.jsonValue["Slot"]["Lanes"] = lanes;
+
+ std::optional<pcie_slots::SlotTypes> redfishSlotType =
+ pcie_util::dbusSlotTypeToRf(slotType);
+ if (!redfishSlotType)
+ {
+ BMCWEB_LOG_WARNING << "Unknown PCIeSlot Type: " << slotType;
+ }
+ else
+ {
+ if (*redfishSlotType == pcie_slots::SlotTypes::Invalid)
+ {
+ BMCWEB_LOG_ERROR << "Invalid PCIeSlot type: " << slotType;
+ messages::internalError(res);
+ return;
+ }
+ res.jsonValue["Slot"]["SlotType"] = *redfishSlotType;
+ }
+}
+
+inline void getPCIeDeviceSlotPath(
+ const std::string& pcieDevicePath,
+ const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
+ std::function<void(const std::string& pcieDeviceSlot)>&& callback)
+{
+ std::string associationPath = pcieDevicePath + "/contained_by";
+ dbus::utility::getAssociatedSubTreePaths(
+ associationPath, sdbusplus::message::object_path(inventoryPath), 0,
+ pcieSlotInterface,
+ [callback, asyncResp, pcieDevicePath](
+ const boost::system::error_code& ec,
+ const dbus::utility::MapperGetSubTreePathsResponse& endpoints) {
+ if (ec)
+ {
+ if (ec.value() == EBADR)
+ {
+ // Missing association is not an error
+ return;
+ }
+ BMCWEB_LOG_ERROR
+ << "DBUS response error for getAssociatedSubTreePaths "
+ << ec.value();
+ messages::internalError(asyncResp->res);
+ return;
+ }
+ if (endpoints.size() > 1)
+ {
+ BMCWEB_LOG_ERROR
+ << "PCIeDevice is associated with more than one PCIeSlot: "
+ << endpoints.size();
+ messages::internalError(asyncResp->res);
+ return;
+ }
+ if (endpoints.empty())
+ {
+ // If the device doesn't have an association, return without PCIe
+ // Slot properties
+ BMCWEB_LOG_DEBUG << "PCIeDevice is not associated with PCIeSlot";
+ return;
+ }
+ callback(endpoints[0]);
+ });
+}
+
+inline void
+ afterGetDbusObject(const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
+ const std::string& pcieDeviceSlot,
+ const boost::system::error_code& ec,
+ const dbus::utility::MapperGetObject& object)
+{
+ if (ec || object.empty())
+ {
+ BMCWEB_LOG_ERROR << "DBUS response error for getDbusObject "
+ << ec.value();
+ messages::internalError(asyncResp->res);
+ return;
+ }
+ sdbusplus::asio::getAllProperties(
+ *crow::connections::systemBus, object.begin()->first, pcieDeviceSlot,
+ "xyz.openbmc_project.Inventory.Item.PCIeSlot",
+ [asyncResp](
+ const boost::system::error_code& ec2,
+ const dbus::utility::DBusPropertiesMap& pcieSlotProperties) {
+ addPCIeSlotProperties(asyncResp->res, ec2, pcieSlotProperties);
+ });
+}
+
+inline void afterGetPCIeDeviceSlotPath(
+ const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
+ const std::string& pcieDeviceSlot)
+{
+ dbus::utility::getDbusObject(
+ pcieDeviceSlot, pcieSlotInterface,
+ [asyncResp,
+ pcieDeviceSlot](const boost::system::error_code& ec,
+ const dbus::utility::MapperGetObject& object) {
+ afterGetDbusObject(asyncResp, pcieDeviceSlot, ec, object);
+ });
+}
+
inline void
getPCIeDeviceHealth(const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
const std::string& pcieDevicePath,
@@ -181,7 +325,7 @@ inline void
sdbusplus::asio::getProperty<bool>(
*crow::connections::systemBus, service, pcieDevicePath,
"xyz.openbmc_project.Inventory.Item", "Present",
- [asyncResp](const boost::system::error_code& ec, const bool value) {
+ [asyncResp](const boost::system::error_code& ec, bool value) {
if (ec)
{
if (ec.value() != EBADR)
@@ -265,7 +409,8 @@ inline void
}
inline void addPCIeDeviceProperties(
- crow::Response& resp, const std::string& pcieDeviceId,
+ const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
+ const std::string& pcieDeviceId,
const dbus::utility::DBusPropertiesMap& pcieDevProperties)
{
const std::string* deviceType = nullptr;
@@ -279,13 +424,13 @@ inline void addPCIeDeviceProperties(
if (!success)
{
- messages::internalError(resp);
+ messages::internalError(asyncResp->res);
return;
}
if (deviceType != nullptr && !deviceType->empty())
{
- resp.jsonValue["PCIeInterface"]["DeviceType"] = *deviceType;
+ asyncResp->res.jsonValue["PCIeInterface"]["DeviceType"] = *deviceType;
}
if (generationInUse != nullptr)
@@ -304,10 +449,10 @@ inline void addPCIeDeviceProperties(
{
BMCWEB_LOG_ERROR << "Invalid PCIe Device Generation: "
<< *generationInUse;
- messages::internalError(resp);
+ messages::internalError(asyncResp->res);
return;
}
- resp.jsonValue["PCIeInterface"]["PCIeType"] =
+ asyncResp->res.jsonValue["PCIeInterface"]["PCIeType"] =
*redfishGenerationInUse;
}
}
@@ -316,12 +461,13 @@ inline void addPCIeDeviceProperties(
// left as off if it is a default value.
if (lanesInUse != nullptr && *lanesInUse != 0)
{
- resp.jsonValue["PCIeInterface"]["LanesInUse"] = *lanesInUse;
+ asyncResp->res.jsonValue["PCIeInterface"]["LanesInUse"] = *lanesInUse;
}
- resp.jsonValue["PCIeFunctions"]["@odata.id"] = boost::urls::format(
- "/redfish/v1/Systems/system/PCIeDevices/{}/PCIeFunctions",
- pcieDeviceId);
+ asyncResp->res.jsonValue["PCIeFunctions"]["@odata.id"] =
+ boost::urls::format(
+ "/redfish/v1/Systems/system/PCIeDevices/{}/PCIeFunctions",
+ pcieDeviceId);
}
inline void getPCIeDeviceProperties(
@@ -365,6 +511,23 @@ inline void addPCIeDeviceCommonProperties(
asyncResp->res.jsonValue["Status"]["Health"] = "OK";
}
+inline void afterGetValidPcieDevicePath(
+ const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
+ const std::string& pcieDeviceId, const std::string& pcieDevicePath,
+ const std::string& service)
+{
+ addPCIeDeviceCommonProperties(asyncResp, pcieDeviceId);
+ getPCIeDeviceAsset(asyncResp, pcieDevicePath, service);
+ getPCIeDeviceState(asyncResp, pcieDevicePath, service);
+ getPCIeDeviceHealth(asyncResp, pcieDevicePath, service);
+ getPCIeDeviceProperties(
+ asyncResp, pcieDevicePath, service,
+ std::bind_front(addPCIeDeviceProperties, asyncResp, pcieDeviceId));
+ getPCIeDeviceSlotPath(
+ pcieDevicePath, asyncResp,
+ std::bind_front(afterGetPCIeDeviceSlotPath, asyncResp));
+}
+
inline void
handlePCIeDeviceGet(App& app, const crow::Request& req,
const std::shared_ptr<bmcweb::AsyncResp>& asyncResp,
@@ -391,20 +554,7 @@ inline void
getValidPCIeDevicePath(
pcieDeviceId, asyncResp,
- [asyncResp, pcieDeviceId](const std::string& pcieDevicePath,
- const std::string& service) {
- addPCIeDeviceCommonProperties(asyncResp, pcieDeviceId);
- getPCIeDeviceAsset(asyncResp, pcieDevicePath, service);
- getPCIeDeviceState(asyncResp, pcieDevicePath, service);
- getPCIeDeviceHealth(asyncResp, pcieDevicePath, service);
- getPCIeDeviceProperties(
- asyncResp, pcieDevicePath, service,
- [asyncResp, pcieDeviceId](
- const dbus::utility::DBusPropertiesMap& pcieDevProperties) {
- addPCIeDeviceProperties(asyncResp->res, pcieDeviceId,
- pcieDevProperties);
- });
- });
+ std::bind_front(afterGetValidPcieDevicePath, asyncResp, pcieDeviceId));
}
inline void requestRoutesSystemPCIeDevice(App& app)