On Sun, 15 Jan 2023 16:34:58 -0500 Joel Fernandes <joel@xxxxxxxxxxxxxxxxx> wrote: > > > +EXPORT_SYMBOL_GPL(rcu_async_hurry); > > > > > > > Where do you plan on calling these externally, as they are being > > marked exported? > > > > If you allow random drivers to enable this, I can see something > > enabling it and hitting an error path that causes it to never disable > > it. > > You mean, just like rcu_expedite_gp() ? > > > I wouldn't have EXPORT_SYMBOL_GPL() unless you really know that it is > > needed externally. > > At the moment it is not called externally but in the future, it could > be from rcutorture. If you see rcu_expedite_gp(), that is exported > too. I was just modeling it around that API. The reason for the export should have been mentioned in the change log if the patch is not obvious to why it is being exported. Thanks, -- Steve
