[PATCH] mtrr: Add lock annotations for prepare_set and post_set
authorJosh Triplett <josht@us.ibm.com>
Tue, 26 Sep 2006 06:32:36 +0000 (23:32 -0700)
committerLinus Torvalds <torvalds@g5.osdl.org>
Tue, 26 Sep 2006 15:48:57 +0000 (08:48 -0700)
The functions prepare_set and post_set in kernel/cpu/mtrr/generic.c wrap
the spinlock set_atomicity_lock: prepare_set returns with the lock held,
and post_set releases the lock without acquiring it.  Add lock annotations
to these two functions so that sparse can check callers for lock pairing,
and so that sparse will not complain about these functions since they
intentionally use locks in this manner.

Signed-off-by: Josh Triplett <josh@freedesktop.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
arch/i386/kernel/cpu/mtrr/generic.c

index 169ac8e0db689e069cc5b02b044458c1142468b7..0b61eed8bbd819cb3f4d71a59feec3376acec5c6 100644 (file)
@@ -243,7 +243,7 @@ static DEFINE_SPINLOCK(set_atomicity_lock);
  * has been called.
  */
 
-static void prepare_set(void)
+static void prepare_set(void) __acquires(set_atomicity_lock)
 {
        unsigned long cr0;
 
@@ -274,7 +274,7 @@ static void prepare_set(void)
        mtrr_wrmsr(MTRRdefType_MSR, deftype_lo & 0xf300UL, deftype_hi);
 }
 
-static void post_set(void)
+static void post_set(void) __releases(set_atomicity_lock)
 {
        /*  Flush TLBs (no need to flush caches - they are disabled)  */
        __flush_tlb();