DPDK ports use masks while reporting rx checksum flags. OVS should use these mask along with reported checksum flag while validating the good checksum.
Added two new functions to validate bad checksum reported by DPDK NIC port. These two functions will be used in the following patch for enabling rx checksum offload in conntrack module. Signed-off-by: Sugesh Chandran <sugesh.chand...@intel.com> Co-authored-by: Darrell Ball <db...@vmware.com> Signed-off-by: Darrell Ball <db...@vmware.com> Acked-by: Antonio Fishetti <antonio.fische...@intel.com> Acked-by: Darrell Ball <db...@vmware.com> --- lib/dp-packet.h | 28 ++++++++++++++++++++++++++-- 1 file changed, 26 insertions(+), 2 deletions(-) diff --git a/lib/dp-packet.h b/lib/dp-packet.h index 38282bd..7d0838d 100644 --- a/lib/dp-packet.h +++ b/lib/dp-packet.h @@ -612,7 +612,19 @@ static inline bool dp_packet_ip_checksum_valid(struct dp_packet *p) { #ifdef DPDK_NETDEV - return p->mbuf.ol_flags & PKT_RX_IP_CKSUM_GOOD; + return (p->mbuf.ol_flags & PKT_RX_IP_CKSUM_MASK) == + PKT_RX_IP_CKSUM_GOOD; +#else + return 0 && p; +#endif +} + +static inline bool +dp_packet_ip_checksum_bad(struct dp_packet *p) +{ +#ifdef DPDK_NETDEV + return (p->mbuf.ol_flags & PKT_RX_IP_CKSUM_MASK) == + PKT_RX_IP_CKSUM_BAD; #else return 0 && p; #endif @@ -622,7 +634,19 @@ static inline bool dp_packet_l4_checksum_valid(struct dp_packet *p) { #ifdef DPDK_NETDEV - return p->mbuf.ol_flags & PKT_RX_L4_CKSUM_GOOD; + return (p->mbuf.ol_flags & PKT_RX_L4_CKSUM_MASK) == + PKT_RX_L4_CKSUM_GOOD; +#else + return 0 && p; +#endif +} + +static inline bool +dp_packet_l4_checksum_bad(struct dp_packet *p) +{ +#ifdef DPDK_NETDEV + return (p->mbuf.ol_flags & PKT_RX_L4_CKSUM_MASK) == + PKT_RX_L4_CKSUM_BAD; #else return 0 && p; #endif -- 2.7.4 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev