On 13 April 2017 at 12:07, Andy Zhou <az...@ovn.org> wrote:
> On Thu, Apr 13, 2017 at 11:10 AM, Joe Stringer <j...@ovn.org> wrote:
>> On 6 April 2017 at 17:18, Andy Zhou <az...@ovn.org> wrote:
>>> This config flag was not consistently checked. Fix it.
>>>
>>> Signed-off-by: Andy Zhou <az...@ovn.org>
>>
>> nf_ct_labels_find() already changes its implementation based on
>> CONFIG_NF_CONNTRACK_LABELS so the OVS code doesn't have to become more
>> complicated by handling this case in the function. AFAICT this has no
>> visible change in behaviour. Did you find a problem compiling this on
>> a particular kernel or config?
>
> ON a 3.10 kernel, It complained about undefined NF_CT_LABELS_MAX_SIZE
> when CONFIG_NF_CONNTRACK_LABLES is not defined.

OK, maybe a simpler method is just to define that symbol in the right
header if it's not available.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to