Hey, On Wed, Mar 02, 2011 at 10:27:38AM +0100, Thomas Renninger wrote: > > -cpufreq_freq_attr_ro_perm(cpuinfo_cur_freq, 0400); > > +cpufreq_freq_attr_ro(cpuinfo_cur_freq); > > cpufreq_freq_attr_ro(cpuinfo_min_freq); > Having this read only for root goes back to the beginning > of git history of this file in 2005... > It looks like this is done on purpose, but I do not see why > this should not be readable for others. > > Maybe Dominik or Dave have an idea. This talks to hardware, and might (in some implementations) talk to the BIOS, causing it to do nasty things, such as executing things in SMM. That shouldn't be available to the world. However, as ACPI still does not provide a ->get() method, making this attribute world readable _might_ not cause too much harm, but it isn't worth the hassle IMO. Especially as everyone may read out scaling_cur_freq, and scaling_cur_freq differs from cpuinfo_cur_freq _only_ if something is broken. Best, Dominik -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
