On Tue, Nov 05, 2013 at 02:15:48PM -0800, Gurucharan Shetty wrote:
> On Wed, Oct 30, 2013 at 2:13 PM, Ben Pfaff <b...@nicira.com> wrote:
> > This new function will have an additional caller in an upcoming commit.
> >
> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> 
> This fails to compile for me. The arguments for ofproto_trace() has 
> mis-matches.
> Otherwise looks good.

Oops, sorry about that.

I folded in this incremental:

diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index 617b546..2551370 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -5418,7 +5418,7 @@ ofproto_unixctl_trace(struct unixctl_conn *conn, int 
argc, const char *argv[],
         struct ds result;
 
         ds_init(&result);
-        ofproto_trace(ofproto, &flow, packet, NULL, 0, &result);
+        ofproto_trace(ofproto, &flow, packet, &result);
         unixctl_command_reply(conn, ds_cstr(&result));
         ds_destroy(&result);
         ofpbuf_delete(packet);
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to