lkml.org 
[lkml]   [2006]   [Jun]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH -mm 1/6] cpu_relax(): interrupt.h

Add cpu_relax() to tasklet_unlock_wait() loop.


I kept barrier() since it is said to still be required, in various older
postings.
Took care of kernel style formatting.

Tested on 2.6.17-mm1.


This is the first patch within a set, which aims to improve P4 HT performance
a bit:
these SMT CPUs want a rep nop (pause) (as done by cpu_relax()) when
busy-polling, in order to free pipeline resources for the SMT CPU sibling.

During I/O polling cpu_relax() will help for non-P4 CPUs, too (reduce
power consumption there).

Signed-off-by: Andreas Mohr <andi@lisas.de>


diff -urN linux-2.6.17-mm1.orig/include/linux/interrupt.h linux-2.6.17-mm1.my/include/linux/interrupt.h
--- linux-2.6.17-mm1.orig/include/linux/interrupt.h 2006-06-21 14:28:19.000000000 +0200
+++ linux-2.6.17-mm1.my/include/linux/interrupt.h 2006-06-21 21:34:12.000000000 +0200
@@ -227,7 +227,10 @@

static inline void tasklet_unlock_wait(struct tasklet_struct *t)
{
- while (test_bit(TASKLET_STATE_RUN, &(t)->state)) { barrier(); }
+ while (test_bit(TASKLET_STATE_RUN, &(t)->state)) {
+ barrier();
+ cpu_relax();
+ }
}
#else
#define tasklet_trylock(t) 1
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2006-06-21 23:02    [W:0.034 / U:0.228 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site