On Wed, Apr 16, 2014 at 2:14 PM, Jesse Gross <[email protected]> wrote:
> On Tue, Apr 15, 2014 at 6:21 PM, Andy Zhou <[email protected]> wrote:
>> diff --git a/datapath/flow_netlink.c b/datapath/flow_netlink.c
>> index 0a1effa..883d9bf 100644
>> --- a/datapath/flow_netlink.c
>> +++ b/datapath/flow_netlink.c
>> @@ -129,7 +129,8 @@ static bool match_validate(const struct sw_flow_match
>> *match,
>> /* Always allowed mask fields. */
>> mask_allowed |= ((1ULL << OVS_KEY_ATTR_TUNNEL)
>> | (1ULL << OVS_KEY_ATTR_IN_PORT)
>> - | (1ULL << OVS_KEY_ATTR_ETHERTYPE));
>> + | (1ULL << OVS_KEY_ATTR_ETHERTYPE)
>> + | (1ULL << OVS_KEY_ATTR_RECIRC_ID));
>
> These are for attributes where a mask can be specified without a key
> (mostly legacy). I'm not sure that recirculation falls into that - it
> should also have a key whenever there is a mask.
You are right. I will fix it in V3.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
