Re: [PATCH V2 00/11] CPUFreq: Fixes & Cleanups for 3.12

From: Rafael J. Wysocki
Date: Tue Aug 06 2013 - 20:11:01 EST


On Tuesday, August 06, 2013 10:53:02 PM Viresh Kumar wrote:
> Hi Rafael,
>
> This is V2 of the patch series which was posted here earlier:
>
> http://www.gossamer-threads.com/lists/linux/kernel/1759514
>
> This patchset tries to fix & cleanup many existing cpufreq core issues. First
> four patches tries to cleanup basic problems in cpufreq core. Its first patch
> was earlier sent separately but now is part of this series.
>
> Fifth patch was also sent earlier as reply to your patches and was reviewed by
> Srivatsa. Sixth patch was picked from Lukasz's patchset on introducing software
> "boost" feature in core. It will be used by this patchset.
>
> And 7-10 are are the most significant part of this set. They try to make many
> things simple and robust.
>
> Last patch in the series is new and wasn't part of V1.
>
> This is rebased of your bleeding-edge branch + two patches from you:
> 18a6b03 cpufreq: Avoid double kobject_put() for the same kobject in error code path
> d0cde63 cpufreq: Do not hold driver module references for additional policy CPUs
> abe513f Merge branch 'acpi-sleep-next' into linux-next
>
> They are also pushed in my cpufreq-next branch.
>
> They are tested fairly well on ARM Vexpress TC2 board (big LITTLE).
>
> Lukasz Majewski (1):
> cpufreq: Store cpufreq policies in a list
>
> Viresh Kumar (10):
> cpufreq: Cleanup header files included in core
> cpufreq: Re-arrange declarations in cpufreq.h
> cpufreq: Give consistent names for struct cpufreq_policy *
> cpufreq: Use sizeof(*ptr) form for finding size of a struct
> cpufreq: Pass policy to cpufreq_add_policy_cpu()
> cpufreq: Use cpufreq_policy_list for iterating over policies
> cpufreq: Fix broken usage of governor->owner's refcount
> cpufreq: Don't use cpufreq_driver->owner's refcount to protect
> critical sections
> cpufreq: Remove struct cpufreq_driver's owner field
> cpufreq: improve error checking on return values of
> __cpufreq_governor()

Applied to bleeding-edge except for patch [05/11] that I've applied already
earlier.

> Documentation/cpu-freq/cpu-drivers.txt | 2 -
> drivers/cpufreq/acpi-cpufreq.c | 5 +-
> drivers/cpufreq/at32ap-cpufreq.c | 1 -
> drivers/cpufreq/blackfin-cpufreq.c | 1 -
> drivers/cpufreq/cpufreq-nforce2.c | 1 -
> drivers/cpufreq/cpufreq.c | 426 ++++++++++++++++-----------------
> drivers/cpufreq/cpufreq_conservative.c | 14 +-
> drivers/cpufreq/cpufreq_governor.c | 6 -
> drivers/cpufreq/cpufreq_governor.h | 7 +-
> drivers/cpufreq/cpufreq_ondemand.c | 24 +-
> drivers/cpufreq/cpufreq_performance.c | 3 +-
> drivers/cpufreq/cpufreq_powersave.c | 3 +-
> drivers/cpufreq/cpufreq_stats.c | 23 +-
> drivers/cpufreq/cris-artpec3-cpufreq.c | 1 -
> drivers/cpufreq/cris-etraxfs-cpufreq.c | 1 -
> drivers/cpufreq/e_powersaver.c | 5 +-
> drivers/cpufreq/elanfreq.c | 1 -
> drivers/cpufreq/exynos-cpufreq.c | 2 +-
> drivers/cpufreq/freq_table.c | 4 +-
> drivers/cpufreq/gx-suspmod.c | 3 +-
> drivers/cpufreq/ia64-acpi-cpufreq.c | 5 +-
> drivers/cpufreq/intel_pstate.c | 1 -
> drivers/cpufreq/kirkwood-cpufreq.c | 1 -
> drivers/cpufreq/longhaul.c | 1 -
> drivers/cpufreq/longrun.c | 1 -
> drivers/cpufreq/loongson2_cpufreq.c | 1 -
> drivers/cpufreq/maple-cpufreq.c | 1 -
> drivers/cpufreq/p4-clockmod.c | 1 -
> drivers/cpufreq/pasemi-cpufreq.c | 1 -
> drivers/cpufreq/pcc-cpufreq.c | 1 -
> drivers/cpufreq/pmac32-cpufreq.c | 1 -
> drivers/cpufreq/pmac64-cpufreq.c | 6 +-
> drivers/cpufreq/powernow-k6.c | 1 -
> drivers/cpufreq/powernow-k7.c | 14 +-
> drivers/cpufreq/powernow-k8.c | 7 +-
> drivers/cpufreq/ppc-corenet-cpufreq.c | 1 -
> drivers/cpufreq/ppc_cbe_cpufreq.c | 1 -
> drivers/cpufreq/s3c2416-cpufreq.c | 1 -
> drivers/cpufreq/s3c24xx-cpufreq.c | 6 +-
> drivers/cpufreq/s3c64xx-cpufreq.c | 1 -
> drivers/cpufreq/sc520_freq.c | 1 -
> drivers/cpufreq/sh-cpufreq.c | 1 -
> drivers/cpufreq/sparc-us2e-cpufreq.c | 6 +-
> drivers/cpufreq/sparc-us3-cpufreq.c | 6 +-
> drivers/cpufreq/speedstep-centrino.c | 1 -
> drivers/cpufreq/speedstep-ich.c | 1 -
> drivers/cpufreq/speedstep-smi.c | 1 -
> include/linux/cpufreq.h | 386 ++++++++++++++---------------
> 48 files changed, 442 insertions(+), 547 deletions(-)

I like these statistics. :-)

Thanks,
Rafael

--
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/