> From: Lance Richardson [mailto:lance.richard...@broadcom.com] > Sent: Thursday, 16 September 2021 18.11 > > Minor corrections and improvements to documentation > for checksum APIs. > > Fixes: 6006818cfb26 ("net: new checksum functions") > Fixes: 45a08ef55e44 ("net: introduce functions to verify L4 checksums") > Cc: sta...@dpdk.org > Signed-off-by: Lance Richardson <lance.richard...@broadcom.com> > --- > lib/net/rte_ip.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/lib/net/rte_ip.h b/lib/net/rte_ip.h > index 05948b69b7..fd08ea31b2 100644 > --- a/lib/net/rte_ip.h > +++ b/lib/net/rte_ip.h > @@ -488,7 +488,7 @@ rte_ipv6_phdr_cksum(const struct rte_ipv6_hdr > *ipv6_hdr, uint64_t ol_flags) > } > > /** > - * @internal Calculate the non-complemented IPv4 L4 checksum > + * @internal Calculate the non-complemented IPv6 L4 checksum > */ > static inline uint16_t > __rte_ipv6_udptcp_cksum(const struct rte_ipv6_hdr *ipv6_hdr, const > void *l4_hdr) > @@ -509,15 +509,15 @@ __rte_ipv6_udptcp_cksum(const struct rte_ipv6_hdr > *ipv6_hdr, const void *l4_hdr) > /** > * Process the IPv6 UDP or TCP checksum. > * > - * The IPv4 header should not contains options. The layer 4 checksum > - * must be set to 0 in the packet by the caller. > + * The IPv6 header must not be followed by extension headers. The > layer 4 > + * checksum must be set to 0 in the L4 header by the caller. > * > * @param ipv6_hdr > * The pointer to the contiguous IPv6 header. > * @param l4_hdr > * The pointer to the beginning of the L4 header. > * @return > - * The complemented checksum to set in the IP packet. > + * The complemented checksum to set in the L4 header. > */ > static inline uint16_t > rte_ipv6_udptcp_cksum(const struct rte_ipv6_hdr *ipv6_hdr, const void > *l4_hdr) > -- > 2.25.1
Reviewed-by: Morten Brørup <m...@smartsharesystems.com>
