lkml.org 
[lkml]   [2005]   [Apr]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Subject[patch 064/198] i386: Use loaddebug macro consistently
From
Date

From: Roland McGrath <roland@redhat.com>

This moves the macro loaddebug from asm-i386/suspend.h to
asm-i386/processor.h, which is the place that makes sense for it to be
defined, removes the extra copy of the same macro in
arch/i386/kernel/process.c, and makes arch/i386/kernel/signal.c use the
macro in place of its expansion.

This is a purely cosmetic cleanup for the normal i386 kernel. However, it
is handy for Xen to be able to just redefine the loaddebug macro once
instead of also changing the signal.c code.

Signed-off-by: Roland McGrath <roland@redhat.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
---

25-akpm/arch/i386/kernel/process.c | 7 -------
25-akpm/arch/i386/kernel/signal.c | 2 +-
25-akpm/include/asm-i386/processor.h | 8 ++++++++
25-akpm/include/asm-i386/suspend.h | 5 -----
4 files changed, 9 insertions(+), 13 deletions(-)

diff -puN arch/i386/kernel/process.c~i386-use-loaddebug-macro-consistently arch/i386/kernel/process.c
--- 25/arch/i386/kernel/process.c~i386-use-loaddebug-macro-consistently 2005-04-12 03:21:18.732289160 -0700
+++ 25-akpm/arch/i386/kernel/process.c 2005-04-12 03:21:18.740287944 -0700
@@ -558,13 +558,6 @@ handle_io_bitmap(struct thread_struct *n
*/
tss->io_bitmap_base = INVALID_IO_BITMAP_OFFSET_LAZY;
}
-/*
- * This special macro can be used to load a debugging register
- */
-#define loaddebug(thread,register) \
- __asm__("movl %0,%%db" #register \
- : /* no output */ \
- :"r" (thread->debugreg[register]))

/*
* switch_to(x,yn) should switch tasks from x to y.
diff -puN arch/i386/kernel/signal.c~i386-use-loaddebug-macro-consistently arch/i386/kernel/signal.c
--- 25/arch/i386/kernel/signal.c~i386-use-loaddebug-macro-consistently 2005-04-12 03:21:18.734288856 -0700
+++ 25-akpm/arch/i386/kernel/signal.c 2005-04-12 03:21:18.741287792 -0700
@@ -618,7 +618,7 @@ int fastcall do_signal(struct pt_regs *r
* inside the kernel.
*/
if (unlikely(current->thread.debugreg[7])) {
- __asm__("movl %0,%%db7" : : "r" (current->thread.debugreg[7]));
+ loaddebug(&current->thread, 7);
}

/* Whee! Actually deliver the signal. */
diff -puN include/asm-i386/processor.h~i386-use-loaddebug-macro-consistently include/asm-i386/processor.h
--- 25/include/asm-i386/processor.h~i386-use-loaddebug-macro-consistently 2005-04-12 03:21:18.735288704 -0700
+++ 25-akpm/include/asm-i386/processor.h 2005-04-12 03:21:18.741287792 -0700
@@ -499,6 +499,14 @@ static inline void load_esp0(struct tss_
regs->esp = new_esp; \
} while (0)

+/*
+ * This special macro can be used to load a debugging register
+ */
+#define loaddebug(thread,register) \
+ __asm__("movl %0,%%db" #register \
+ : /* no output */ \
+ :"r" ((thread)->debugreg[register]))
+
/* Forward declaration, a strange C thing */
struct task_struct;
struct mm_struct;
diff -puN include/asm-i386/suspend.h~i386-use-loaddebug-macro-consistently include/asm-i386/suspend.h
--- 25/include/asm-i386/suspend.h~i386-use-loaddebug-macro-consistently 2005-04-12 03:21:18.736288552 -0700
+++ 25-akpm/include/asm-i386/suspend.h 2005-04-12 03:21:18.742287640 -0700
@@ -36,11 +36,6 @@ struct saved_context {
unsigned long return_address;
} __attribute__((packed));

-#define loaddebug(thread,register) \
- __asm__("movl %0,%%db" #register \
- : /* no output */ \
- :"r" ((thread)->debugreg[register]))
-
#ifdef CONFIG_ACPI_SLEEP
extern unsigned long saved_eip;
extern unsigned long saved_esp;
_
-
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: 2005-04-13 03:52    [W:0.033 / U:0.068 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site