> -----Original Message----- > From: Cong Wang [mailto:[email protected]] > Sent: Tuesday, October 17, 2017 11:53 PM > To: Chris Mi <[email protected]> > Cc: Linux Kernel Network Developers <[email protected]>; Jamal Hadi > Salim <[email protected]>; Lucas Bates <[email protected]>; Jiri Pirko > <[email protected]>; David Miller <[email protected]> > Subject: Re: [patch net v2 1/4] net/sched: Change tc_action refcnt and > bindcnt to atomic > > On Mon, Oct 16, 2017 at 6:14 PM, Chris Mi <[email protected]> wrote: > > I don't think this bug were introduced by above two commits only. > > Actually, this bug were introduced by several commits, at least the > following: > > 1. refcnt and bindcnt are not atomic > > Nope, it is perfectly okay with non-atomic as long as no parallel, and without > RCU callback they are perfectly serialized by RTNL. Agree. > > > > 2. passing actions using list instead of arrays (I think initially we > > are using arrays) > > We are discussing patch 1/4, this is patch 2/4, so irrelevant. Agree. > > > > 3. using RCU callbacks > > This introduces problem 1. I think this patch set only fixes one problem, that's the race and the panic. What do you mean by problem 1. > > > > So instead of blaming the latest commit, it is better to say it is a > > pre-git error. > > You are wrong. OK, you are right. But could I know what's your suggestion for this patch set? 1. reject it? 2. change the "Fixes" as you suggested? 3. something else?
Thanks, Chris
