Sorry, sent the previous email too quickly.

 

> -----Original Message-----

> From: Marat Khalili

> Sent: Thursday 31 July 2025 11:52

> To: '苏赛' <susai.ss@bytedance.com>; jasvinder.singh@intel.com

> Cc: dev@dpdk.org

> Subject: RE: [PATCH] net/cksum: compute raw cksum for several segments

 

> > +static inline uint16_t

> > +__rte_raw_cksum_reduce_u64(uint64_t sum)

> > +{

> > +        uint32_t tmp;

> > +

> > +        tmp = __rte_raw_cksum_reduce((uint32_t)sum);

> > +        tmp += __rte_raw_cksum_reduce((uint32_t)(sum >> 32));

>

> What if this addition overflows?

 

Realized it cannot actually overflow, my bad (maybe still needs a comment). Now this function looks good to me as well.

 

> > +        return __rte_raw_cksum_reduce(tmp);

> > +}