[ovs-dev] [PATCH v1] ovs-ofctl: Fixed the "snoop" command of ovs-ofctl
blp at ovn.org
Tue Apr 17 20:44:41 UTC 2018
On Tue, Apr 17, 2018 at 01:26:51PM -0700, Ashish Varma wrote:
> In case where "use_names" is set (e.g. in an interactive session) to show
> the port and table names when ovs-ofctl is run with snoop command,
> ovs-ofctl would get stuck in an endless loop inside "table_iterator_next"
> function's while loop checking for "while (ti->send_xid != recv_xid)".
> This would happening because the "vconn" to "<bridge>.snoop" socket would
> not respond to TABLE_FEATURES_REQUEST sent by ovs-ofctl.
> This commit disables showing port or table names in the snoop command.
> Signed-off-by: Ashish Varma <ashishvarma.ovs at gmail.com>
Thanks for figuring this out and fixing it.
Would you mind making the comment more detailed? The current comment
explains what the code does, but it does not explain why it is
important. If you could summarize the rationale from above, then it
would make it clear to the reader why it is important.
> utilities/ovs-ofctl.c | 1 +
> 1 file changed, 1 insertion(+)
> diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c
> index 6708b07..3023787 100644
> --- a/utilities/ovs-ofctl.c
> +++ b/utilities/ovs-ofctl.c
> @@ -2326,6 +2326,7 @@ ofctl_snoop(struct ovs_cmdl_context *ctx)
> struct vconn *vconn;
> + use_names = 0; /* don't show port and table names */
> open_vconn__(ctx->argv, SNOOP, &vconn);
> monitor_vconn(vconn, false, false);
> dev mailing list
> dev at openvswitch.org
More information about the dev