On Fri, May 8, 2015 at 1:52 PM, Ben Pfaff <[email protected]> wrote:
> The correct group ID to avoid requiring any particular output group when
> removing a flow is OFPG_ANY.  OFPG_ALL just caused the OFPFC_DELETE_STRICT
> commands to be ignored because no OVN flows output to OFPG_ALL.
>
> Signed-off-by: Ben Pfaff <[email protected]>
This fixes the problem where-in before this patch flows weren't
getting deleted when logical ports were deleted.
Acked-by: Gurucharan Shetty <[email protected]>


> ---
>  ovn/controller/ofctrl.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ovn/controller/ofctrl.c b/ovn/controller/ofctrl.c
> index 3b6c169..6e1ac6b 100644
> --- a/ovn/controller/ofctrl.c
> +++ b/ovn/controller/ofctrl.c
> @@ -366,7 +366,7 @@ queue_flow_mod(struct ofputil_flow_mod *fm)
>  {
>      fm->buffer_id = UINT32_MAX;
>      fm->out_port = OFPP_ANY;
> -    fm->out_group = OFPG_ALL;
> +    fm->out_group = OFPG_ANY;
>      queue_msg(ofputil_encode_flow_mod(fm, OFPUTIL_P_OF13_OXM));
>  }
>
> --
> 1.7.10.4
>
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to