Thx Ben for clearing this up,

Please see my comments below:


> +.IP \fBtcp_src=\fIport\fR
> +.IQ \fBtcp_dst=\fIport\fR
> +.IQ \fBudp_src=\fIport\fR
> +.IQ \fBudp_dst=\fIport\fR
> +.IQ \fBsctp_src=\fIport\fR
> +.IQ \fBsctp_dst=\fIport\fR
> +Matches a TCP, UDP, or SCTP source or destination port \fIport\fR,
> +which is specified as a decimal number between 0 and 65535, inclusive.
> +.IP
> +When \fBdl_type\fR and \fBnw_proto\fR are wildcarded or set to values
> +that do not indicate an appropriate protocol, the values of these
> +settings are ignored (see \fBFlow Syntax\fR above).
>


I checked on current master, if the prerequisite test failed, the flow is
just discarded (not installed).

Otherwise, looks good,

Acked-by: Alex Wang <al...@nicira.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to