[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170425045713.GU26900@vireshk-i7>
Date: Tue, 25 Apr 2017 10:27:13 +0530
From: Viresh Kumar <viresh.kumar@...aro.org>
To: Lukasz Luba <lukasz.luba@....com>
Cc: Javi Merino <javi.merino@...nel.org>,
Zhang Rui <rui.zhang@...el.com>,
Eduardo Valentin <edubezval@...il.com>,
Amit Daniel Kachhap <amit.kachhap@...il.com>,
linaro-kernel@...ts.linaro.org, Rafael Wysocki <rjw@...ysocki.net>,
linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org,
Vincent Guittot <vincent.guittot@...aro.org>,
Punit Agrawal <punit.agrawal@....com>
Subject: Re: [PATCH V3 11/17] thermal: cpu_cooling: get rid of 'allowed_cpus'
On 24-04-17, 17:53, Lukasz Luba wrote:
> The policy pointer forwarded from cpufreq_update_policy()
> is a local variable 'new_policy' so cannot be compared with pinned
> policy pointer in the cooling device.
> You should do the cpumask test like before:
> if (!cpumask_test_cpu(policy->cpu,
> cpufreq_cdev->policy->related_cpus))
Right. I have fixed it a bit differently now.
> But there is something still in the patch set...
> I will try to check it tomorrow.
I reviewed all the patches very carefully again, trying to find out the culprit
(I don't have the right hardware to test it like you have).
Found out that max_level isn't used properly at few places, fixed and pushed my
branch now. See if it works fine now.
HEAD: 6a883ddf73cd
--
viresh
Powered by blists - more mailing lists