Greg, Paul, Any comment on this? -- Steve On Thu, 19 Aug 2021 20:42:04 -0400 Steven Rostedt <rostedt@xxxxxxxxxxx> wrote: > On Thu, 19 Aug 2021 17:09:33 -0400 > Steven Rostedt <rostedt@xxxxxxxxxxx> wrote: > > > Mathieu, seems that the "slow down 10x" patch was able to be backported > > to 5.10, where as this patch was not. Reason being is that > > start_poll_synchronize_rcu() was added in 5.13. > > I can get this to work if I backport the following RCU patches: > > 29d2bb94a8a126ce80ffbb433b648b32fdea524e > srcu: Provide internal interface to start a Tree SRCU grace period > > 5358c9fa54b09b5d3d7811b033aa0838c1bbaaf2 > srcu: Provide polling interfaces for Tree SRCU grace periods > > 1a893c711a600ab57526619b56e6f6b7be00956e > srcu: Provide internal interface to start a Tiny SRCU grace period > > 8b5bd67cf6422b63ee100d76d8de8960ca2df7f0 > srcu: Provide polling interfaces for Tiny SRCU grace periods > > The first three can be cherry-picked without issue. The last one has a > small conflict, of: > > include/linux/srcutiny.h.rej: > --- include/linux/srcutiny.h > +++ include/linux/srcutiny.h > @@ -16,6 +16,7 @@ > struct srcu_struct { > short srcu_lock_nesting[2]; /* srcu_read_lock() nesting depth. */ > unsigned short srcu_idx; /* Current reader array element in bit 0x2. */ > + unsigned short srcu_idx_max; /* Furthest future srcu_idx request. */ > u8 srcu_gp_running; /* GP workqueue running? */ > u8 srcu_gp_waiting; /* GP waiting for readers? */ > struct swait_queue_head srcu_wq; > > > Which I just added that line, and everything worked. > > Paul, do you have any issues with these four patches getting backported? > > Greg, Are you OK with them too? > > Once those are backported, this patch can be backported as well, and > everything should work. This patch really needs to stay with: > > 231264d6927f6740af36855a622d0e240be9d94c > tracepoint: Fix static call function vs data state mismatch > > Otherwise I would say to revert it if this one can't be backported with > it. > > -- Steve