diff options
author | Deepak Kodihalli <dkodihal@in.ibm.com> | 2017-07-21 07:07:09 -0500 |
---|---|---|
committer | Deepak Kodihalli <dkodihal@in.ibm.com> | 2017-07-31 22:48:26 -0500 |
commit | 18aa044eace292d2d58406e06e3ca4a98fbfaaea (patch) | |
tree | c0b58725a4ef134e2d99910955fa2e9090526867 | |
parent | 6c8d51baee01211c671567881dec27c327ae2c86 (diff) | |
download | phosphor-host-ipmid-18aa044eace292d2d58406e06e3ca4a98fbfaaea.tar.gz phosphor-host-ipmid-18aa044eace292d2d58406e06e3ca4a98fbfaaea.zip |
Add API to retrieve settings
Add API to retrieve and cache settings objects of interest.
Change-Id: I8afd58b5b3e9a691e4d3fa6bfba2ba5c92cd2568
Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
-rw-r--r-- | Makefile.am | 19 | ||||
-rw-r--r-- | configure.ac | 2 | ||||
-rw-r--r-- | settings.cpp | 82 | ||||
-rw-r--r-- | settings.hpp | 56 |
4 files changed, 156 insertions, 3 deletions
diff --git a/Makefile.am b/Makefile.am index 0379037..8dd3fc2 100644 --- a/Makefile.am +++ b/Makefile.am @@ -4,8 +4,11 @@ sbin_PROGRAMS = \ ipmid ipmid_SOURCES = \ - ipmid.cpp + ipmid.cpp \ + settings.cpp + nodist_ipmid_SOURCES = ipmiwhitelist.cpp + BUILT_SOURCES = \ ipmiwhitelist.cpp \ sensor-gen.cpp \ @@ -15,8 +18,18 @@ BUILT_SOURCES = \ CLEANFILES = $(BUILT_SOURCES) #TODO - Make this path a configure option (bitbake parameter) -ipmid_CPPFLAGS = -DHOST_IPMI_LIB_PATH=\"/usr/lib/host-ipmid/\" $(PHOSPHOR_LOGGING_CFLAGS) -ipmid_LDFLAGS = $(SYSTEMD_LIBS) $(libmapper_LIBS) $(LIBADD_DLOPEN) $(PHOSPHOR_LOGGING_LIBS) -export-dynamic +ipmid_CPPFLAGS = \ + -DHOST_IPMI_LIB_PATH=\"/usr/lib/host-ipmid/\" \ + $(PHOSPHOR_LOGGING_CFLAGS) \ + $(PHOSPHOR_DBUS_INTERFACES_CFLAGS) + +ipmid_LDFLAGS = \ + $(SYSTEMD_LIBS) \ + $(libmapper_LIBS) \ + $(LIBADD_DLOPEN) \ + $(PHOSPHOR_LOGGING_LIBS) \ + $(PHOSPHOR_DBUS_INTERFACES_LIBS) \ + -export-dynamic # TODO: Rather than use -export-dynamic, we should use -export-symbol to have a # selective list of symbols. diff --git a/configure.ac b/configure.ac index 7efb842..9c909ea 100644 --- a/configure.ac +++ b/configure.ac @@ -26,6 +26,8 @@ AC_CHECK_LIB([mapper], [mapper_get_service], ,[AC_MSG_ERROR([Could not find libm PKG_CHECK_MODULES([SYSTEMD], [libsystemd >= 221], [], [AC_MSG_ERROR(["systemd required and not found"])]) PKG_CHECK_MODULES([PHOSPHOR_LOGGING], [phosphor-logging],, [AC_MSG_ERROR([Could not find phosphor-logging...openbmc/phosphor-logging package required])]) PKG_CHECK_MODULES([PHOSPHOR_DBUS_INTERFACES], [phosphor-dbus-interfaces],, [AC_MSG_ERROR([Could not find phosphor-dbus-interfaces...openbmc/phosphor-dbus-interfaces package required])]) +PKG_CHECK_MODULES([SDBUSPLUS], [sdbusplus],,\ + AC_MSG_ERROR(["Requires sdbusplus package."])) AS_IF([test "x$enable_softoff" != "xno"], # Check for sdbusplus diff --git a/settings.cpp b/settings.cpp new file mode 100644 index 0000000..cb86d99 --- /dev/null +++ b/settings.cpp @@ -0,0 +1,82 @@ +#include <phosphor-logging/elog-errors.hpp> +#include <phosphor-logging/log.hpp> +#include "xyz/openbmc_project/Common/error.hpp" +#include "settings.hpp" + +namespace settings +{ + +using namespace phosphor::logging; +using namespace sdbusplus::xyz::openbmc_project::Common::Error; + +constexpr auto mapperService = "xyz.openbmc_project.ObjectMapper"; +constexpr auto mapperPath = "/xyz/openbmc_project/object_mapper"; +constexpr auto mapperIntf = "xyz.openbmc_project.ObjectMapper"; + +Objects::Objects(sdbusplus::bus::bus& bus, + const std::vector<Interface>& filter): + bus(bus) +{ + auto depth = 0; + + auto mapperCall = bus.new_method_call(mapperService, + mapperPath, + mapperIntf, + "GetSubTree"); + mapperCall.append(root); + mapperCall.append(depth); + mapperCall.append(filter); + auto response = bus.call(mapperCall); + if (response.is_method_error()) + { + log<level::ERR>("Error in mapper GetSubTree"); + elog<InternalFailure>(); + } + + using Interfaces = std::vector<Interface>; + using MapperResponse = std::map<Path, std::map<Service, Interfaces>>; + MapperResponse result; + response.read(result); + if (result.empty()) + { + log<level::ERR>("Invalid response from mapper"); + elog<InternalFailure>(); + } + + for (auto& iter : result) + { + const auto& path = iter.first; + auto& interface = iter.second.begin()->second[0]; + map.emplace(std::move(interface), path); + } +} + +Service Objects::service(const Path& path, const Interface& interface) const +{ + using Interfaces = std::vector<Interface>; + auto mapperCall = bus.new_method_call(mapperService, + mapperPath, + mapperIntf, + "GetObject"); + mapperCall.append(path); + mapperCall.append(Interfaces({interface})); + + auto response = bus.call(mapperCall); + if (response.is_method_error()) + { + log<level::ERR>("Error in mapper GetObject"); + elog<InternalFailure>(); + } + + std::map<Service, Interfaces> result; + response.read(result); + if (result.empty()) + { + log<level::ERR>("Invalid response from mapper"); + elog<InternalFailure>(); + } + + return result.begin()->first; +} + +} // namespace settings diff --git a/settings.hpp b/settings.hpp new file mode 100644 index 0000000..60ee4d4 --- /dev/null +++ b/settings.hpp @@ -0,0 +1,56 @@ +#pragma once + +#include <string> +#include <sdbusplus/bus.hpp> + +namespace settings +{ + +using Path = std::string; +using Service = std::string; +using Interface = std::string; + +constexpr auto root = "/"; + +/** @class Objects + * @brief Fetch paths of settings d-bus objects of interest, upon construction + */ +struct Objects +{ + public: + /** @brief Constructor - fetch settings objects + * + * @param[in] bus - The Dbus bus object + * @param[in] filter - A vector of settings interfaces the caller is + * interested in. + */ + Objects(sdbusplus::bus::bus& bus, const std::vector<Interface>& filter); + Objects(const Objects&) = default; + Objects& operator=(const Objects&) = default; + Objects(Objects&&) = delete; + Objects& operator=(Objects&&) = delete; + ~Objects() = default; + + /** @brief Fetch d-bus service, given a path and an interface. The + * service can't be cached because mapper returns unique + * service names. + * + * @param[in] path - The Dbus object + * @param[in] interface - The Dbus interface + * + * @return std::string - the dbus service + */ + Service service(const Path& path, const Interface& interface) const; + + // TODO openbmc/openbmc#2058 - This will break when multiple settings, + // or in general multiple objects implement a single setting interface. + // For instance this will break for a 2-blade server, because we'd have + // 2 sets of settings objects. Need to revisit and fix this. + /** @brief map of settings objects */ + std::map<Interface, Path> map; + + /** @brief The Dbus bus object */ + sdbusplus::bus::bus& bus; +}; + +} // namespace settings |