On Wed, Mar 18, 2020 at 09:41:35PM +0200, Jarkko Sakkinen wrote: > On Wed, Mar 18, 2020 at 09:40:06PM +0200, Jarkko Sakkinen wrote: > > On Wed, Mar 18, 2020 at 09:03:16AM -0700, Sean Christopherson wrote: > > > How on earth is someone doing to dredge up the above information without a > > > comment? Anyone looking at this code without a priori knowledge of the > > > development history will assume the missing synchronize_srcu() is a bug. > > > > By reading the source code of the driver obviously. > > Secondly, there is no development history. It is in epoch state. One bigger artifact that I noticed is that a loop construct would be better than looping with goto. do-while would fit just fine. /Jarkko