Re: [Update][PATCH] cpufreq: Do not hold driver module references foradditional policy CPUs

From: Viresh Kumar
Date: Fri Aug 02 2013 - 02:59:32 EST


On 2 August 2013 12:19, Srivatsa S. Bhat
<srivatsa.bhat@xxxxxxxxxxxxxxxxxx> wrote:
> But lsmod shows 0 for the cpufreq driver right? (Note, your related_cpus
> should have only 1 CPU each, for you to see 0. Else, you'll see a non-zero
> value due to the very bug/inconsistency that Rafael is fixing in this
> patch).

I have hacked the driver this way:

@@ -2114,10 +2114,16 @@ int cpufreq_register_driver(struct
cpufreq_driver *driver_data)
cpufreq_driver = driver_data;
write_unlock_irqrestore(&cpufreq_driver_lock, flags);

+ printk(KERN_INFO "%s: Module refcount: %lu\n", __func__,
+ module_refcount(cpufreq_driver->owner));
+
ret = subsys_interface_register(&cpufreq_interface);
if (ret)
goto err_null_driver;


And this gave me 1..
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/