> -----Original Message----- > From: Stephen Hemminger <[email protected]> > Sent: Wednesday, July 8, 2020 1:13 AM > To: Phil Yang <[email protected]> > Cc: [email protected]; [email protected]; [email protected]; > Honnappa Nagarahalli <[email protected]>; > [email protected]; Ruifeng Wang <[email protected]>; nd > <[email protected]> > Subject: Re: [dpdk-dev] [PATCH v2] mbuf: use C11 atomics for refcnt > operations > > On Tue, 7 Jul 2020 18:10:33 +0800 > Phil Yang <[email protected]> wrote: > > > + return (uint16_t)(__atomic_add_fetch((int16_t *)&shinfo- > >refcnt_atomic, > > + > > Why do you need so many casts here? > The type of refcnt_atomic is now uint16 after your patch.
In the existing code, the input parameter type for this API is signed integer. For example: drivers/net/netvsc/hn_rxtx.c:531 lib/librte_mbuf/rte_mbuf.h:1194 However, the output type of rte_mbuf_ext_refcnt related APIs is not uniform. We use these typecast to consistent with the current API definition.
