commit 03f84ba1c62099d9669c108b0b90d505847db475
author Patrick Venture <venture@google.com> Wed Sep 20 09:15:33 2017 -0700
committer Patrick Venture <venture@google.com> Wed Sep 20 09:17:12 2017 -0700
tree f1ce238b6c0f857a8288ca0bd74bbfe212cdafff
parent 580af36bd4668723dccedee792350edc5673f417

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>

ipmid.cpp

diff --git a/ipmid.cpp b/ipmid.cpp
index f6ab2fd..f06adc5 100644
--- a/ipmid.cpp
+++ b/ipmid.cpp
@@ -194,8 +194,11 @@
     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.