diff options
| author | Marri Devender Rao <devenrao@in.ibm.com> | 2017-11-07 04:58:14 -0600 |
|---|---|---|
| committer | Brad Bishop <bradleyb@fuzziesquirrel.com> | 2017-11-16 20:35:43 +0000 |
| commit | 6088558429fbe5f723783b4220b6ac1c9096cde9 (patch) | |
| tree | 3fb71a2014e4e65f3070711c3219442da042d308 /evdevpp | |
| parent | 67967f9a3f8e779d897a66cc6d559783911b2c14 (diff) | |
| download | phosphor-fan-presence-6088558429fbe5f723783b4220b6ac1c9096cde9.tar.gz phosphor-fan-presence-6088558429fbe5f723783b4220b6ac1c9096cde9.zip | |
Fix up InternalFailure to include metadata
Scope is to add missing logs for InternalFailure errors
Change-Id: I12c958127c1303fba0057914682e651457a0e547
Signed-off-by: Marri Devender Rao <devenrao@in.ibm.com>
Diffstat (limited to 'evdevpp')
| -rw-r--r-- | evdevpp/evdev.hpp | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/evdevpp/evdev.hpp b/evdevpp/evdev.hpp index 1861982..b0ba06e 100644 --- a/evdevpp/evdev.hpp +++ b/evdevpp/evdev.hpp @@ -36,6 +36,7 @@ using EvDev = std::unique_ptr<libevdev, EvDevDeleter>; } // namespace details +using namespace phosphor::logging; /** @class EvDev * @brief Provides C++ bindings to the libevdev C API. */ @@ -72,7 +73,10 @@ class EvDev evdev.get(), type, code, &val); if (!rc) { - phosphor::logging::elog<InternalFailure>(); + log<level::ERR>("Error in call to libevdev_fetch_event_value", + entry("TYPE=%d", type), + entry("CODE=%d", code)); + elog<InternalFailure>(); } return val; @@ -88,7 +92,9 @@ class EvDev evdev.get(), LIBEVDEV_READ_FLAG_NORMAL, &ev); if (rc < 0) { - phosphor::logging::elog<InternalFailure>(); + log<level::ERR>("Error in call to libevdev_next_event", + entry("RC=%d", rc)); + elog<InternalFailure>(); } if (ev.type == EV_SYN && ev.code == SYN_REPORT) @@ -118,7 +124,10 @@ inline auto newFromFD(int fd) if (rc) { - phosphor::logging::elog<InternalFailure>(); + log<level::ERR>("Error in call to libevdev_new_from_fd", + entry("RC=%d", rc), + entry("FD=%d", fd)); + elog<InternalFailure>(); } return EvDev(dev); |
