On Aug 23, 2013, at 10:04 AM, Ben Pfaff <b...@nicira.com> wrote:
> @@ -1667,7 +1674,18 @@ static void
> xlate_table_action(struct xlate_ctx *ctx,
> ofp_port_t in_port, uint8_t table_id, bool may_packet_in)
> {
> - if (ctx->recurse < MAX_RESUBMIT_RECURSION) {
> + static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 1);
> +
> + if (ctx->recurse >= MAX_RESUBMIT_RECURSION) {
> + VLOG_ERR_RL(&rl, "resubmit actions recursed over %d times",
> + MAX_RESUBMIT_RECURSION);
> + } else if (ctx->resubmits >= MAX_RESUBMITS) {
> + VLOG_ERR_RL(&rl, "over %d resubmit actions", MAX_RESUBMITS);
> + } else if (ctx->xout->odp_actions.size >= 65536) {
In other parts of the code, the max action size is UINT16_MAX. It might be
nice to be consistent.
Acked-by: Justin Pettit <jpet...@nicira.com>
--Justin
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
