Re: [PATCH] net: optimize cmpxchg in ip_idents_reserve

From: Arvind Sankar
Date: Fri Jan 17 2020 - 13:03:31 EST


On Fri, Jan 17, 2020 at 08:35:07AM -0800, Eric Dumazet wrote:
>
>
> On 1/17/20 4:32 AM, Peter Zijlstra wrote:
>
> >
> > That's crazy, just accept that UBSAN is taking bonghits and ignore it.
> > Use atomic_add_return() unconditionally.
> >
>
> Yes, we might simply add a comment so that people do not bug us if
> their compiler is too old.
>
> /* If UBSAN reports an error there, please make sure your compiler
> * supports -fno-strict-overflow before reporting it.
> */
> return atomic_add_return(segs + delta, p_id) - segs;
>

Do we need that comment any more? The flag was apparently introduced in
gcc-4.2 and we only support 4.6+ anyway?