Skip to contentSkip to conversation

remove the second arg of do_timer_interrupt()

RFC PATCH New Adrian Bunk 2 participants 2 messages Aug 19, 2005 – Aug 20, 2005

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

RFC PATCH 1/1 remove the second arg of do_timer_interrupt()

The second arg of do_timer_interrupt() is not used in the functions, and all callers pass NULL.

Is there any reason not to remove it?

---

 arch/i386/kernel/time.c |    5 ++---
 arch/sh/kernel/time.c   |    4 ++--
 arch/sh64/kernel/time.c |    4 ++--
 3 files changed, 6 insertions(+), 7 deletions(-)
arch/i386/kernel/time.c renamed from arch/i386/kernel/time.c.old +2 -3
@@ -252,8 +252,7 @@
  * timer_interrupt() needs to keep up the real-time clock,
  * as well as call the "do_timer()" routine every clocktick
  */
-static inline void do_timer_interrupt(int irq, void *dev_id,
-					struct pt_regs *regs)
+static inline void do_timer_interrupt(int irq, struct pt_regs *regs)
 {
 #ifdef CONFIG_X86_IO_APIC
 	if (timer_ack) {
@@ -307,7 +306,7 @@
 
 	cur_timer->mark_offset();
  
-	do_timer_interrupt(irq, NULL, regs);
+	do_timer_interrupt(irq, regs);
 
 	write_sequnlock(&xtime_lock);
 	return IRQ_HANDLED;
arch/sh/kernel/time.c renamed from arch/sh/kernel/time.c.old +2 -2
@@ -234,7 +234,7 @@
  * timer_interrupt() needs to keep up the real-time clock,
  * as well as call the "do_timer()" routine every clocktick
  */
-static inline void do_timer_interrupt(int irq, void *dev_id, struct pt_regs *regs)
+static inline void do_timer_interrupt(int irq, struct pt_regs *regs)
 {
 	do_timer(regs);
 #ifndef CONFIG_SMP
@@ -285,7 +285,7 @@
 	 * locally disabled. -arca
 	 */
 	write_seqlock(&xtime_lock);
-	do_timer_interrupt(irq, NULL, regs);
+	do_timer_interrupt(irq, regs);
 	write_sequnlock(&xtime_lock);
 
 	return IRQ_HANDLED;
arch/sh64/kernel/time.c renamed from arch/sh64/kernel/time.c.old +2 -2
@@ -303,7 +303,7 @@
  * timer_interrupt() needs to keep up the real-time clock,
  * as well as call the "do_timer()" routine every clocktick
  */
-static inline void do_timer_interrupt(int irq, void *dev_id, struct pt_regs *regs)
+static inline void do_timer_interrupt(int irq, struct pt_regs *regs)
 {
 	unsigned long long current_ctc;
 	asm ("getcon cr62, %0" : "=r" (current_ctc));
@@ -361,7 +361,7 @@
 	 * locally disabled. -arca
 	 */
 	write_lock(&xtime_lock);
-	do_timer_interrupt(irq, NULL, regs);
+	do_timer_interrupt(irq, regs);
 	write_unlock(&xtime_lock);
 
 	return IRQ_HANDLED;
#1 lore raw

PATCH 1/1 SLAB : removes local_irq_save()/local_irq_restore() pair in ksize()

This patch removes unnecessary critical section in ksize() function, as cli/sti are rather expensive on modern CPUS.

mm/slab.c +0 -2
diff -Nru linux-2.6.13-rc6-ed/mm/slab.c linux-2.6.13-rc6/mm/slab.c
@@ -3080,10 +3080,8 @@
 	unsigned int size = 0;
 
 	if (likely(objp != NULL)) {
-		local_irq_save(flags);
 		c = GET_PAGE_CACHE(virt_to_page(objp));
 		size = kmem_cache_size(c);
-		local_irq_restore(flags);
 	}
 
 	return size;