diff mbox

[ovs-dev] netdev-tc-offloads: Add prefix to identify source of log msg

Message ID 1502886472-61183-1-git-send-email-roid@mellanox.com
State Accepted
Headers show

Commit Message

Roi Dayan Aug. 16, 2017, 12:27 p.m. UTC 
There is an identical log msg from multiple api calls.
Add a prefix to identify the source function of the log msg.

Signed-off-by: Roi Dayan <roid@mellanox.com>
Reviewed-by: Paul Blakey <paulb@mellanox.com>
---
 lib/netdev-tc-offloads.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Comments

Joe Stringer Aug. 16, 2017, 10:18 p.m. UTC | #1 
On 16 August 2017 at 05:27, Roi Dayan <roid@mellanox.com> wrote:
> There is an identical log msg from multiple api calls.
> Add a prefix to identify the source function of the log msg.
>
> Signed-off-by: Roi Dayan <roid@mellanox.com>
> Reviewed-by: Paul Blakey <paulb@mellanox.com>
> ---

Applied, thanks!
diff mbox

Patch

diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c
index e51e638..be6e109 100644
--- a/lib/netdev-tc-offloads.c
+++ b/lib/netdev-tc-offloads.c
@@ -232,7 +232,7 @@  netdev_tc_flow_flush(struct netdev *netdev)
     int ifindex = netdev_get_ifindex(netdev);
 
     if (ifindex < 0) {
-        VLOG_ERR_RL(&error_rl, "failed to get ifindex for %s: %s",
+        VLOG_ERR_RL(&error_rl, "flow_flush: failed to get ifindex for %s: %s",
                     netdev_get_name(netdev), ovs_strerror(-ifindex));
         return -ifindex;
     }
@@ -249,7 +249,7 @@  netdev_tc_flow_dump_create(struct netdev *netdev,
 
     ifindex = netdev_get_ifindex(netdev);
     if (ifindex < 0) {
-        VLOG_ERR_RL(&error_rl, "failed to get ifindex for %s: %s",
+        VLOG_ERR_RL(&error_rl, "dump_create: failed to get ifindex for %s: %s",
                     netdev_get_name(netdev), ovs_strerror(-ifindex));
         return -ifindex;
     }
@@ -682,7 +682,7 @@  netdev_tc_flow_put(struct netdev *netdev, struct match *match,
 
     ifindex = netdev_get_ifindex(netdev);
     if (ifindex < 0) {
-        VLOG_ERR_RL(&error_rl, "failed to get ifindex for %s: %s",
+        VLOG_ERR_RL(&error_rl, "flow_put: failed to get ifindex for %s: %s",
                     netdev_get_name(netdev), ovs_strerror(-ifindex));
         return -ifindex;
     }
@@ -883,7 +883,7 @@  netdev_tc_flow_get(struct netdev *netdev OVS_UNUSED,
 
     ifindex = netdev_get_ifindex(dev);
     if (ifindex < 0) {
-        VLOG_ERR_RL(&error_rl, "failed to get ifindex for %s: %s",
+        VLOG_ERR_RL(&error_rl, "flow_get: failed to get ifindex for %s: %s",
                     netdev_get_name(dev), ovs_strerror(-ifindex));
         netdev_close(dev);
         return -ifindex;
@@ -926,7 +926,7 @@  netdev_tc_flow_del(struct netdev *netdev OVS_UNUSED,
 
     ifindex = netdev_get_ifindex(dev);
     if (ifindex < 0) {
-        VLOG_ERR_RL(&error_rl, "failed to get ifindex for %s: %s",
+        VLOG_ERR_RL(&error_rl, "flow_del: failed to get ifindex for %s: %s",
                     netdev_get_name(dev), ovs_strerror(-ifindex));
         netdev_close(dev);
         return -ifindex;
@@ -951,7 +951,7 @@  netdev_tc_init_flow_api(struct netdev *netdev)
 
     ifindex = netdev_get_ifindex(netdev);
     if (ifindex < 0) {
-        VLOG_ERR_RL(&error_rl, "failed to get ifindex for %s: %s",
+        VLOG_ERR_RL(&error_rl, "init: failed to get ifindex for %s: %s",
                     netdev_get_name(netdev), ovs_strerror(-ifindex));
         return -ifindex;
     }