On Thu 14 May 19:32 PDT 2020, Doug Anderson wrote: > Hi, > > On Mon, May 4, 2020 at 10:50 AM Douglas Anderson <diand...@chromium.org> > wrote: > > > > cpu_pm_notify() is basically a wrapper of notifier_call_chain(). > > notifier_call_chain() doesn't initialize *nr_calls to 0 before it > > starts incrementing it--presumably it's up to the callers to do this. > > > > Unfortunately the callers of cpu_pm_notify() don't init *nr_calls. > > This potentially means you could get too many or two few calls to > > CPU_PM_ENTER_FAILED or CPU_CLUSTER_PM_ENTER_FAILED depending on the > > luck of the stack. > > > > Let's fix this. > > > > Fixes: ab10023e0088 ("cpu_pm: Add cpu power management notifiers") > > Signed-off-by: Douglas Anderson <diand...@chromium.org> > > Cc: Rafael J. Wysocki <rafael.j.wyso...@intel.com> > > Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org> > > Reviewed-by: Stephen Boyd <swb...@chromium.org> > > --- > > This seems to be an ownerless file. I'm hoping this patch can just go > > through the Qualcomm tree. It would be nice if we could get an Ack > > from Rafael or Greg KH though. > > Greg / Rafael: any thoughts on this patch? Care to give it an Ack so > it could go through the Qualcomm tree? > > Andrew: I think you are the adopter or orphan patches, usually. If > nobody else will take this patch, will you consider taking it into > your tree? It should be a super straightforward bugfix. I'm happy to > re-post it CCing you if need be. >
Thanks for the patch and the reviews. I've picked all 5 patches through my tree, with the added stable@ Cc on this one. Regards, Bjorn