On Thu, Sep 8, 2016 at 8:35 AM, Eric Dumazet <[email protected]> wrote: > From: Eric Dumazet <[email protected]> > > As reported by Cong Wang, I was lazy when I did initial RCU conversion > of tc_mirred, as I thought I could avoid allocation/freeing of a > parameter block.
Quote from Eric Dumazet: https://www.mail-archive.com/[email protected]/msg115482.html <Quote> Well, I added a READ_ONCE() to read tcf_action once. Adding rcu here would mean adding a pointer and extra cache line, to deref the values. IMHO the race here has no effect . You either read the old or new value. </Quote> Me with facepalm... ;-)
