[CPUFREQ] Mark policy_rwsem as going static in cpufreq.c wont be exported
authorvenkatesh.pallipadi@intel.com <venkatesh.pallipadi@intel.com>
Fri, 3 Jul 2009 00:08:31 +0000 (17:08 -0700)
committerDave Jones <davej@redhat.com>
Tue, 7 Jul 2009 01:38:28 +0000 (21:38 -0400)
lock_policy_rwsem_* and unlock_policy_rwsem_* routines in cpufreq.c are
currently exported to drivers. Improper use of those locks can
result in deadlocks and it is better to keep the locks localized.
Two previous in-kernel users of these interfaces (ondemand and conservative),
do not use this interfaces any more. Schedule them for removal.

Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
Signed-off-by: Dave Jones <davej@redhat.com>
Documentation/feature-removal-schedule.txt

index f8cd450..09e031c 100644 (file)
@@ -458,3 +458,13 @@ Why:       Remove the old legacy 32bit machine check code. This has been
        but the old version has been kept around for easier testing. Note this
        doesn't impact the old P5 and WinChip machine check handlers.
 Who:   Andi Kleen <andi@firstfloor.org>
+
+----------------------------
+
+What:  lock_policy_rwsem_* and unlock_policy_rwsem_* will not be
+       exported interface anymore.
+When:  2.6.33
+Why:   cpu_policy_rwsem has a new cleaner definition making it local to
+       cpufreq core and contained inside cpufreq.c. Other dependent
+       drivers should not use it in order to safely avoid lockdep issues.
+Who:   Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>