[ovs-dev] [PATCH] datapath: Support for kernel 3.15

Pritesh Kothari (pritkoth) pritkoth at cisco.com
Wed Jul 2 23:24:54 UTC 2014


Thanks for the review.

>> +  OVS_GREP_IFELSE([$KSRC/include/linux/if.h], [IFF_LIVE_ADDR_CHANGE])
>> +
>>   OVS_GREP_IFELSE([$KSRC/include/linux/if_vlan.h], [vlan_set_encap_proto])
>> 
> Why are you changing existing compat code in this patch? Even if you
> have a reason for such change please send separate patch with the
> explanation.

Sure.

> 
> @@ -322,11 +329,17 @@ AC_DEFUN([OVS_CHECK_LINUX_COMPAT], [
>>   OVS_GREP_IFELSE([$KSRC/include/linux/skbuff.h], [skb_get_hash])
>>   OVS_GREP_IFELSE([$KSRC/include/linux/skbuff.h], [skb_clear_hash])
>>   OVS_GREP_IFELSE([$KSRC/include/linux/skbuff.h], [l4_rxhash])
>> +  OVS_GREP_IFELSE([$KSRC/include/linux/skbuff.h], [encapsulation])
>> 
> I think better way to search for encap is searching for GRE offload
> feature, I done same check in patch that I posted "datapath: Enable
> tunnel GSO features.”

will do.

>> -#endif /* 3.12 */
>> +struct sk_buff *iptunnel_handle_offloads(struct sk_buff *skb,
>> +                                        bool csum_help,
>> +                                        int gso_type_mask)
>> +{
>> +       int err;
>> +
> This function is not called from any other function in OVS. why do you
> want to add it to ovs compat layer?

actually gre_handle_offloads calls it, will see if i can figure out a way to not to include this.

Regards,
Pritesh.


More information about the dev mailing list