diff mbox series

[ovs-dev] ofproto-dpif-xlate: Added logging when attempting to send out an in_port

Message ID 1533330409-41568-1-git-send-email-zwhitt.vmware@gmail.com
State Superseded
Headers show
Series [ovs-dev] ofproto-dpif-xlate: Added logging when attempting to send out an in_port | expand

Commit Message

Zak Whittington Aug. 3, 2018, 9:06 p.m. UTC 
VMware-BZ: 2158607
---
 ofproto/ofproto-dpif-xlate.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

0-day Robot Aug. 3, 2018, 9:56 p.m. UTC | #1 
Bleep bloop.  Greetings Zak Whittington, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
ERROR: No signatures found.
Lines checked: 37, Warnings: 0, Errors: 1


Please check this out.  If you feel there has been an error, please email aconole@bytheb.org

Thanks,
0-day Robot
diff mbox series

Patch

diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c
index 01f1faf..93fbee5 100644
--- a/ofproto/ofproto-dpif-xlate.c
+++ b/ofproto/ofproto-dpif-xlate.c
@@ -5007,6 +5007,8 @@  xlate_output_action(struct xlate_ctx *ctx, ofp_port_t port,
         if (port != ctx->xin->flow.in_port.ofp_port) {
             compose_output_action(ctx, port, NULL, is_last_action, truncate);
         } else {
+            static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
+            VLOG_INFO_RL(&rl, "skipping output to input port");
             xlate_report(ctx, OFT_WARN, "skipping output to input port");
         }
         break;
@@ -5092,6 +5094,8 @@  xlate_output_trunc_action(struct xlate_ctx *ctx,
                 ctx->xout->slow |= SLOW_ACTION;
             }
         } else {
+            static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
+            VLOG_INFO_RL(&rl, "skipping output to input port");
             xlate_report(ctx, OFT_WARN, "skipping output to input port");
         }
         break;