On 9/6/2022 5:48 AM, Frederic Weisbecker wrote: >> } >> - rcu_nocb_unlock_irqrestore(rdp, flags); >> + >> + // The flush succeeded and we moved CBs into the ->cblist. >> + // However, the bypass timer might still be running. Wakeup the > That part of the comment mentioning the bypass timer looks strange... > > >> + // GP thread by calling a helper with was_all_done set so that >> + // wake up happens (needed if main CB list was empty before). > How about: > > // The flush succeeded and we moved CBs into the regular list. > // Don't wait for the wake up timer as it may be too far ahead. > // Wake up now GP thread instead if the cblist was empty > > Thanks. > > Other than that the patch looks good, thanks! > I updated it accordingly and will send it for next revision, thank you! - Joel
