On 25-07-15, 00:17, Rafael J. Wysocki wrote: > To avoid that warning, use the observation that cpufreq doesn't > need to care about CPUs that have never been online.
I have concerns over the very philosophy behind the patch and so wanted to discuss more on that. It will be really confusing to have a scenario where: - we have a four related CPUs: 0-3. - 0-1 are online and have /sys/devices/system/cpu/cpuX/cpufreq directory - 2 is offline but was once online and so still has a directory - 3 never came online after the cpufreq driver is registered (we need to think about cpufreq driver being a module here, its possible CPU was online earlier) and so it doesn't have a directory. How will the user distinguish between cpu 3 and 4, both being offline and user may not know one of them was never online. And the related CPUs of 0-2 will include CPU 3 as well.. I think, we just moved into the wrong direction. We have a valid policy for CPU4, with all valid data. Why not show it up in sysfs? So, what we discussed over IRC earlier was, cpufreq shouldn't care about CPUs, which are offline and that don't have a policy allocated for them. So if all the CPUs of a policy never came online after the driver is registered, we shouldn't care about them. I think, for know your earlier version of the patch was just fine, with the improvements I suggested. And we should go ahead with solution like what I gave, the diff of that was quite big for an rc fix and so I said your patch looks better. -- viresh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/