12 Nov
2016
12 Nov
'16
5:24 a.m.
On 12 November 2016 at 03:46, Rafael J. Wysocki rafael@kernel.org wrote:
+static void sugov_work(struct kthread_work *work) {
struct sugov_policy *sg_policy = container_of(work, struct sugov_policy, work);
struct sugov_policy *sg_policy =container_of(work, struct sugov_policy, work);Why this change?
Mistake ..
static struct sugov_policy *sugov_policy_alloc(struct cpufreq_policy *policy) { struct sugov_policy *sg_policy;
struct task_struct *thread;struct sched_param param = { .sched_priority = 50 };I'd define a symbol for the 50. It's just one extra line of code ...
Sure.
As I asked in the cover letter, will you be fine if I send the same patch for ondemand/conservative governors ?