[ovs-dev] [PATCH 5/6] Add some missing Open Flow 1.1 definitions

Simon Horman horms at verge.net.au
Sat Mar 31 00:44:01 UTC 2012


On Fri, Mar 30, 2012 at 09:19:12AM -0700, Ben Pfaff wrote:
> On Fri, Mar 30, 2012 at 11:30:00AM +0900, Simon Horman wrote:
> > Signed-off-by: Simon Horman <horms at verge.net.au>
> 
> Thanks a lot!   I applied this.  It added the following warning:
> 
> ../lib/ofp-print.c:1021: error: enumeration value ‘OFPRR_GROUP_DELETE’ not handled in switch
> 
> so I folded in this fix:
> 
> diff --git a/lib/ofp-print.c b/lib/ofp-print.c
> index 79e23a9..23dadd9 100644
> --- a/lib/ofp-print.c
> +++ b/lib/ofp-print.c
> @@ -1025,6 +1025,8 @@ ofp_flow_removed_reason_to_string(enum ofp_flow_removed_reason reason)
>          return "hard";
>      case OFPRR_DELETE:
>          return "delete";
> +    case OFPRR_GROUP_DELETE:
> +        return "group_delete";
>      default:
>          sprintf(s, "%d", (int) reason);
>          return s;

Thanks, sorry for not fixing that.



More information about the dev mailing list