[ovs-dev] [PATCH v2] ofp-monitor: Added support for OpenFlow 1.4+ Flow Monitor
ashishvarma.ovs at gmail.com
Wed Feb 6 00:26:56 UTC 2019
Thanks for the review!
In the parse_flow_monitor_request(), usable_protocols is an out parameter.
(and is set in parse_flow_monitor_request() only if a match field is
allowed_versions on the other hand is used only in the ofctl_monitor
function. That is the reason for the check in ofctl_monitor.
Would add the examples in tests/ofp-print.at
I will add the monitoring support for OpenFlow 1.1, 1.2 and 1.3 as a
On Mon, Feb 4, 2019 at 3:48 PM Ben Pfaff <blp at ovn.org> wrote:
> On Thu, Jan 31, 2019 at 03:49:39PM -0800, Ashish Varma wrote:
> > OVS supports Nicira version of Flow Monitor feature which allows an
> > controller to keep track of any changes in the flow table. (The changes
> > done by the controller itself or by any other controller connected to
> > This patch adds support for the OpenFlow 1.4+ OFPMP_FLOW_MONITOR
> > message.
> > Also added support in "ovs-ofctl monitor" to send OpenFlow 1.4+ messages
> > OVS.
> > Added unit test cases to test the OpenFlow version of Flow Monitor
> > Signed-off-by: Ashish Varma <ashishvarma.ovs at gmail.com>
> Thanks for the revision! It will be nice to finally have this feature!
> Since parse_flow_monitor_request() already has a 'usable_protocols'
> parameter, I would suggest using that to limit the support for
> out_group, rather than putting a special case into ofctl_monitor().
> I would add an example of the new message to tests/ofp-print.at.
> I think we talked about supporting flow monitors in OF1.1, 1.2, and
> 1.3. I seem to recall that you plan to add that afterward, in a
> separate patch. Is that correct? I hope that, if so, you can start
> work on it soon after this patch is accepted.
> I would add an item to NEWS.
More information about the dev