Re: [PATCH v3 01/10] drivers base/arch_topology: free cpumask cpus_to_visit

From: Juri Lelli
Date: Tue Aug 01 2017 - 06:58:34 EST


Hi Greg,

On 28/07/17 12:51, Viresh Kumar wrote:
> On 27-07-17, 20:33, Dietmar Eggemann wrote:
> > Free cpumask cpus_to_visit in case registering
> > init_cpu_capacity_notifier has failed or the parsing of the cpu
> > capacity-dmips-mhz property is done. The cpumask cpus_to_visit is
> > only used inside the notifier call init_cpu_capacity_callback.
> >
> > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> > Cc: Juri Lelli <juri.lelli@xxxxxxx>
> > Reported-by: Vincent Guittot <vincent.guittot@xxxxxxxxxx>
> > Signed-off-by: Dietmar Eggemann <dietmar.eggemann@xxxxxxx>
> > Acked-by: Vincent Guittot <vincent.guittot@xxxxxxxxxx>
> > Tested-by: Juri Lelli <juri.lelli@xxxxxxx>
> > Reviewed-by: Juri Lelli <juri.lelli@xxxxxxx>
> > ---
> > drivers/base/arch_topology.c | 12 ++++++++++--
> > 1 file changed, 10 insertions(+), 2 deletions(-)
>
> Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

This is actually a fix/optimization which is orthogonal to the rest of
the set. Would it make sense to take only this for now and keep
reviewing the other patches?

Thanks,

- Juri