diff options
author | Patrick Venture <venture@google.com> | 2017-09-20 09:15:33 -0700 |
---|---|---|
committer | Patrick Venture <venture@google.com> | 2017-09-20 09:17:12 -0700 |
commit | 03f84ba1c62099d9669c108b0b90d505847db475 (patch) | |
tree | f1ce238b6c0f857a8288ca0bd74bbfe212cdafff /ipmid.cpp | |
parent | 580af36bd4668723dccedee792350edc5673f417 (diff) | |
download | phosphor-host-ipmid-03f84ba1c62099d9669c108b0b90d505847db475.tar.gz phosphor-host-ipmid-03f84ba1c62099d9669c108b0b90d505847db475.zip |
Move error msg for trying wildcard into debug only
My logs are effectively full of this message, hiding out
other useful messages because we run a lot of OEM traffic. Also,
until the daemon fails to find the command via wildcard, it's not
really an error worth reporting in most cases.
Change-Id: Id6674f3e9a2f237de6bd37d6c9f89931a99d0951
Signed-off-by: Patrick Venture <venture@google.com>
Diffstat (limited to 'ipmid.cpp')
-rw-r--r-- | ipmid.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
@@ -194,8 +194,11 @@ ipmi_ret_t ipmi_netfn_router(ipmi_netfn_t netfn, ipmi_cmd_t cmd, ipmi_request_t auto iter = g_ipmid_router_map.find(std::make_pair(netfn, cmd)); if(iter == g_ipmid_router_map.end()) { + /* By default should only print on failure to find wildcard command. */ +#ifdef __IPMI_DEBUG__ fprintf(stderr, "No registered handlers for NetFn:[0x%X], Cmd:[0x%X]" " trying Wilcard implementation \n",netfn, cmd); +#endif // Now that we did not find any specific [NetFn,Cmd], tuple, check for // NetFn, WildCard command present. |