| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 1 | #include <linux/module.h> | 
| Al Viro | f6a5703 | 2006-10-18 01:47:25 -0400 | [diff] [blame] | 2 | #include <linux/sched.h> | 
| Pekka Paalanen | 2f1dafe | 2008-05-12 21:21:01 +0200 | [diff] [blame] | 3 | #include <linux/mutex.h> | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 4 | #include <linux/list.h> | 
| Andi Kleen | 19d36cc | 2007-07-22 11:12:31 +0200 | [diff] [blame] | 5 | #include <linux/kprobes.h> | 
|  | 6 | #include <linux/mm.h> | 
|  | 7 | #include <linux/vmalloc.h> | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 8 | #include <asm/alternative.h> | 
|  | 9 | #include <asm/sections.h> | 
| Andi Kleen | 19d36cc | 2007-07-22 11:12:31 +0200 | [diff] [blame] | 10 | #include <asm/pgtable.h> | 
| Andi Kleen | 8f4e956 | 2007-07-22 11:12:32 +0200 | [diff] [blame] | 11 | #include <asm/mce.h> | 
|  | 12 | #include <asm/nmi.h> | 
| Dave Jones | b097976 | 2007-10-14 22:57:45 +0200 | [diff] [blame] | 13 | #include <asm/vsyscall.h> | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 14 | #include <asm/cacheflush.h> | 
|  | 15 | #include <asm/io.h> | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 16 |  | 
| Andi Kleen | ab144f5 | 2007-08-10 22:31:03 +0200 | [diff] [blame] | 17 | #define MAX_PATCH_LEN (255-1) | 
|  | 18 |  | 
| Jan Beulich | 0948816 | 2007-07-21 17:10:25 +0200 | [diff] [blame] | 19 | #ifdef CONFIG_HOTPLUG_CPU | 
|  | 20 | static int smp_alt_once; | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 21 |  | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 22 | static int __init bootonly(char *str) | 
|  | 23 | { | 
|  | 24 | smp_alt_once = 1; | 
|  | 25 | return 1; | 
|  | 26 | } | 
| Jeremy Fitzhardinge | b7fb4af | 2007-05-02 19:27:13 +0200 | [diff] [blame] | 27 | __setup("smp-alt-boot", bootonly); | 
| Jan Beulich | 0948816 | 2007-07-21 17:10:25 +0200 | [diff] [blame] | 28 | #else | 
|  | 29 | #define smp_alt_once 1 | 
|  | 30 | #endif | 
|  | 31 |  | 
|  | 32 | static int debug_alternative; | 
| Jeremy Fitzhardinge | b7fb4af | 2007-05-02 19:27:13 +0200 | [diff] [blame] | 33 |  | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 34 | static int __init debug_alt(char *str) | 
|  | 35 | { | 
|  | 36 | debug_alternative = 1; | 
|  | 37 | return 1; | 
|  | 38 | } | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 39 | __setup("debug-alternative", debug_alt); | 
|  | 40 |  | 
| Jan Beulich | 0948816 | 2007-07-21 17:10:25 +0200 | [diff] [blame] | 41 | static int noreplace_smp; | 
|  | 42 |  | 
| Jeremy Fitzhardinge | b7fb4af | 2007-05-02 19:27:13 +0200 | [diff] [blame] | 43 | static int __init setup_noreplace_smp(char *str) | 
|  | 44 | { | 
|  | 45 | noreplace_smp = 1; | 
|  | 46 | return 1; | 
|  | 47 | } | 
|  | 48 | __setup("noreplace-smp", setup_noreplace_smp); | 
|  | 49 |  | 
| Jeremy Fitzhardinge | 959b4fd | 2007-05-02 19:27:16 +0200 | [diff] [blame] | 50 | #ifdef CONFIG_PARAVIRT | 
|  | 51 | static int noreplace_paravirt = 0; | 
|  | 52 |  | 
|  | 53 | static int __init setup_noreplace_paravirt(char *str) | 
|  | 54 | { | 
|  | 55 | noreplace_paravirt = 1; | 
|  | 56 | return 1; | 
|  | 57 | } | 
|  | 58 | __setup("noreplace-paravirt", setup_noreplace_paravirt); | 
|  | 59 | #endif | 
| Jeremy Fitzhardinge | b7fb4af | 2007-05-02 19:27:13 +0200 | [diff] [blame] | 60 |  | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 61 | #define DPRINTK(fmt, args...) if (debug_alternative) \ | 
|  | 62 | printk(KERN_DEBUG fmt, args) | 
|  | 63 |  | 
|  | 64 | #ifdef GENERIC_NOP1 | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 65 | /* Use inline assembly to define this because the nops are defined | 
|  | 66 | as inline assembly strings in the include files and we cannot | 
|  | 67 | get them easily into strings. */ | 
| Jan Beulich | 121d7bf | 2007-10-17 18:04:37 +0200 | [diff] [blame] | 68 | asm("\t.section .rodata, \"a\"\nintelnops: " | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 69 | GENERIC_NOP1 GENERIC_NOP2 GENERIC_NOP3 GENERIC_NOP4 GENERIC_NOP5 GENERIC_NOP6 | 
| Steven Rostedt | f4be31e | 2008-04-09 19:04:07 -0400 | [diff] [blame] | 70 | GENERIC_NOP7 GENERIC_NOP8 | 
|  | 71 | "\t.previous"); | 
| Jan Beulich | 121d7bf | 2007-10-17 18:04:37 +0200 | [diff] [blame] | 72 | extern const unsigned char intelnops[]; | 
|  | 73 | static const unsigned char *const intel_nops[ASM_NOP_MAX+1] = { | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 74 | NULL, | 
|  | 75 | intelnops, | 
|  | 76 | intelnops + 1, | 
|  | 77 | intelnops + 1 + 2, | 
|  | 78 | intelnops + 1 + 2 + 3, | 
|  | 79 | intelnops + 1 + 2 + 3 + 4, | 
|  | 80 | intelnops + 1 + 2 + 3 + 4 + 5, | 
|  | 81 | intelnops + 1 + 2 + 3 + 4 + 5 + 6, | 
|  | 82 | intelnops + 1 + 2 + 3 + 4 + 5 + 6 + 7, | 
|  | 83 | }; | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 84 | #endif | 
|  | 85 |  | 
|  | 86 | #ifdef K8_NOP1 | 
| Jan Beulich | 121d7bf | 2007-10-17 18:04:37 +0200 | [diff] [blame] | 87 | asm("\t.section .rodata, \"a\"\nk8nops: " | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 88 | K8_NOP1 K8_NOP2 K8_NOP3 K8_NOP4 K8_NOP5 K8_NOP6 | 
| Steven Rostedt | f4be31e | 2008-04-09 19:04:07 -0400 | [diff] [blame] | 89 | K8_NOP7 K8_NOP8 | 
|  | 90 | "\t.previous"); | 
| Jan Beulich | 121d7bf | 2007-10-17 18:04:37 +0200 | [diff] [blame] | 91 | extern const unsigned char k8nops[]; | 
|  | 92 | static const unsigned char *const k8_nops[ASM_NOP_MAX+1] = { | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 93 | NULL, | 
|  | 94 | k8nops, | 
|  | 95 | k8nops + 1, | 
|  | 96 | k8nops + 1 + 2, | 
|  | 97 | k8nops + 1 + 2 + 3, | 
|  | 98 | k8nops + 1 + 2 + 3 + 4, | 
|  | 99 | k8nops + 1 + 2 + 3 + 4 + 5, | 
|  | 100 | k8nops + 1 + 2 + 3 + 4 + 5 + 6, | 
|  | 101 | k8nops + 1 + 2 + 3 + 4 + 5 + 6 + 7, | 
|  | 102 | }; | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 103 | #endif | 
|  | 104 |  | 
|  | 105 | #ifdef K7_NOP1 | 
| Jan Beulich | 121d7bf | 2007-10-17 18:04:37 +0200 | [diff] [blame] | 106 | asm("\t.section .rodata, \"a\"\nk7nops: " | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 107 | K7_NOP1 K7_NOP2 K7_NOP3 K7_NOP4 K7_NOP5 K7_NOP6 | 
| Steven Rostedt | f4be31e | 2008-04-09 19:04:07 -0400 | [diff] [blame] | 108 | K7_NOP7 K7_NOP8 | 
|  | 109 | "\t.previous"); | 
| Jan Beulich | 121d7bf | 2007-10-17 18:04:37 +0200 | [diff] [blame] | 110 | extern const unsigned char k7nops[]; | 
|  | 111 | static const unsigned char *const k7_nops[ASM_NOP_MAX+1] = { | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 112 | NULL, | 
|  | 113 | k7nops, | 
|  | 114 | k7nops + 1, | 
|  | 115 | k7nops + 1 + 2, | 
|  | 116 | k7nops + 1 + 2 + 3, | 
|  | 117 | k7nops + 1 + 2 + 3 + 4, | 
|  | 118 | k7nops + 1 + 2 + 3 + 4 + 5, | 
|  | 119 | k7nops + 1 + 2 + 3 + 4 + 5 + 6, | 
|  | 120 | k7nops + 1 + 2 + 3 + 4 + 5 + 6 + 7, | 
|  | 121 | }; | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 122 | #endif | 
|  | 123 |  | 
| Jan Beulich | 32c464f | 2007-10-17 18:04:41 +0200 | [diff] [blame] | 124 | #ifdef P6_NOP1 | 
|  | 125 | asm("\t.section .rodata, \"a\"\np6nops: " | 
|  | 126 | P6_NOP1 P6_NOP2 P6_NOP3 P6_NOP4 P6_NOP5 P6_NOP6 | 
| Steven Rostedt | f4be31e | 2008-04-09 19:04:07 -0400 | [diff] [blame] | 127 | P6_NOP7 P6_NOP8 | 
|  | 128 | "\t.previous"); | 
| Jan Beulich | 32c464f | 2007-10-17 18:04:41 +0200 | [diff] [blame] | 129 | extern const unsigned char p6nops[]; | 
|  | 130 | static const unsigned char *const p6_nops[ASM_NOP_MAX+1] = { | 
|  | 131 | NULL, | 
|  | 132 | p6nops, | 
|  | 133 | p6nops + 1, | 
|  | 134 | p6nops + 1 + 2, | 
|  | 135 | p6nops + 1 + 2 + 3, | 
|  | 136 | p6nops + 1 + 2 + 3 + 4, | 
|  | 137 | p6nops + 1 + 2 + 3 + 4 + 5, | 
|  | 138 | p6nops + 1 + 2 + 3 + 4 + 5 + 6, | 
|  | 139 | p6nops + 1 + 2 + 3 + 4 + 5 + 6 + 7, | 
|  | 140 | }; | 
|  | 141 | #endif | 
|  | 142 |  | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 143 | #ifdef CONFIG_X86_64 | 
|  | 144 |  | 
|  | 145 | extern char __vsyscall_0; | 
| Steven Rostedt | dfa60ab | 2008-05-12 21:20:43 +0200 | [diff] [blame] | 146 | const unsigned char *const *find_nop_table(void) | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 147 | { | 
| H. Peter Anvin | f31d731 | 2008-08-18 17:50:33 -0700 | [diff] [blame] | 148 | if (boot_cpu_data.x86_vendor == X86_VENDOR_INTEL && | 
|  | 149 | boot_cpu_has(X86_FEATURE_NOPL)) | 
|  | 150 | return p6_nops; | 
|  | 151 | else | 
|  | 152 | return k8_nops; | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 153 | } | 
|  | 154 |  | 
|  | 155 | #else /* CONFIG_X86_64 */ | 
|  | 156 |  | 
| Steven Rostedt | dfa60ab | 2008-05-12 21:20:43 +0200 | [diff] [blame] | 157 | const unsigned char *const *find_nop_table(void) | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 158 | { | 
| H. Peter Anvin | f31d731 | 2008-08-18 17:50:33 -0700 | [diff] [blame] | 159 | if (boot_cpu_has(X86_FEATURE_K8)) | 
|  | 160 | return k8_nops; | 
|  | 161 | else if (boot_cpu_has(X86_FEATURE_K7)) | 
|  | 162 | return k7_nops; | 
|  | 163 | else if (boot_cpu_has(X86_FEATURE_NOPL)) | 
|  | 164 | return p6_nops; | 
|  | 165 | else | 
|  | 166 | return intel_nops; | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 167 | } | 
|  | 168 |  | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 169 | #endif /* CONFIG_X86_64 */ | 
|  | 170 |  | 
| Andi Kleen | ab144f5 | 2007-08-10 22:31:03 +0200 | [diff] [blame] | 171 | /* Use this to add nops to a buffer, then text_poke the whole buffer. */ | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 172 | void add_nops(void *insns, unsigned int len) | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 173 | { | 
| Jan Beulich | 121d7bf | 2007-10-17 18:04:37 +0200 | [diff] [blame] | 174 | const unsigned char *const *noptable = find_nop_table(); | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 175 |  | 
|  | 176 | while (len > 0) { | 
|  | 177 | unsigned int noplen = len; | 
|  | 178 | if (noplen > ASM_NOP_MAX) | 
|  | 179 | noplen = ASM_NOP_MAX; | 
| Andi Kleen | ab144f5 | 2007-08-10 22:31:03 +0200 | [diff] [blame] | 180 | memcpy(insns, noptable[noplen], noplen); | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 181 | insns += noplen; | 
|  | 182 | len -= noplen; | 
|  | 183 | } | 
|  | 184 | } | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 185 | EXPORT_SYMBOL_GPL(add_nops); | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 186 |  | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 187 | extern struct alt_instr __alt_instructions[], __alt_instructions_end[]; | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 188 | extern u8 *__smp_locks[], *__smp_locks_end[]; | 
|  | 189 |  | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 190 | /* Replace instructions with better alternatives for this CPU type. | 
|  | 191 | This runs before SMP is initialized to avoid SMP problems with | 
|  | 192 | self modifying code. This implies that assymetric systems where | 
|  | 193 | APs have less capabilities than the boot processor are not handled. | 
|  | 194 | Tough. Make sure you disable such features by hand. */ | 
|  | 195 |  | 
|  | 196 | void apply_alternatives(struct alt_instr *start, struct alt_instr *end) | 
|  | 197 | { | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 198 | struct alt_instr *a; | 
| Andi Kleen | ab144f5 | 2007-08-10 22:31:03 +0200 | [diff] [blame] | 199 | char insnbuf[MAX_PATCH_LEN]; | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 200 |  | 
| Harvey Harrison | 77bf90e | 2008-03-03 11:37:23 -0800 | [diff] [blame] | 201 | DPRINTK("%s: alt table %p -> %p\n", __func__, start, end); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 202 | for (a = start; a < end; a++) { | 
| Andi Kleen | ab144f5 | 2007-08-10 22:31:03 +0200 | [diff] [blame] | 203 | u8 *instr = a->instr; | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 204 | BUG_ON(a->replacementlen > a->instrlen); | 
| Andi Kleen | ab144f5 | 2007-08-10 22:31:03 +0200 | [diff] [blame] | 205 | BUG_ON(a->instrlen > sizeof(insnbuf)); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 206 | if (!boot_cpu_has(a->cpuid)) | 
|  | 207 | continue; | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 208 | #ifdef CONFIG_X86_64 | 
|  | 209 | /* vsyscall code is not mapped yet. resolve it manually. */ | 
|  | 210 | if (instr >= (u8 *)VSYSCALL_START && instr < (u8*)VSYSCALL_END) { | 
|  | 211 | instr = __va(instr - (u8*)VSYSCALL_START + (u8*)__pa_symbol(&__vsyscall_0)); | 
|  | 212 | DPRINTK("%s: vsyscall fixup: %p => %p\n", | 
| Harvey Harrison | 77bf90e | 2008-03-03 11:37:23 -0800 | [diff] [blame] | 213 | __func__, a->instr, instr); | 
| Gerd Hoffmann | d167a51 | 2006-06-26 13:56:16 +0200 | [diff] [blame] | 214 | } | 
|  | 215 | #endif | 
| Andi Kleen | ab144f5 | 2007-08-10 22:31:03 +0200 | [diff] [blame] | 216 | memcpy(insnbuf, a->replacement, a->replacementlen); | 
|  | 217 | add_nops(insnbuf + a->replacementlen, | 
|  | 218 | a->instrlen - a->replacementlen); | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 219 | text_poke_early(instr, insnbuf, a->instrlen); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 220 | } | 
|  | 221 | } | 
|  | 222 |  | 
| Gerd Hoffmann | 8ec4d41 | 2006-07-01 04:36:18 -0700 | [diff] [blame] | 223 | #ifdef CONFIG_SMP | 
|  | 224 |  | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 225 | static void alternatives_smp_lock(u8 **start, u8 **end, u8 *text, u8 *text_end) | 
|  | 226 | { | 
|  | 227 | u8 **ptr; | 
|  | 228 |  | 
|  | 229 | for (ptr = start; ptr < end; ptr++) { | 
|  | 230 | if (*ptr < text) | 
|  | 231 | continue; | 
|  | 232 | if (*ptr > text_end) | 
|  | 233 | continue; | 
| Mathieu Desnoyers | f88f07e | 2008-08-14 16:58:15 -0400 | [diff] [blame] | 234 | /* turn DS segment override prefix into lock prefix */ | 
|  | 235 | text_poke(*ptr, ((unsigned char []){0xf0}), 1); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 236 | }; | 
|  | 237 | } | 
|  | 238 |  | 
|  | 239 | static void alternatives_smp_unlock(u8 **start, u8 **end, u8 *text, u8 *text_end) | 
|  | 240 | { | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 241 | u8 **ptr; | 
|  | 242 |  | 
| Jeremy Fitzhardinge | b7fb4af | 2007-05-02 19:27:13 +0200 | [diff] [blame] | 243 | if (noreplace_smp) | 
|  | 244 | return; | 
|  | 245 |  | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 246 | for (ptr = start; ptr < end; ptr++) { | 
|  | 247 | if (*ptr < text) | 
|  | 248 | continue; | 
|  | 249 | if (*ptr > text_end) | 
|  | 250 | continue; | 
| Mathieu Desnoyers | f88f07e | 2008-08-14 16:58:15 -0400 | [diff] [blame] | 251 | /* turn lock prefix into DS segment override prefix */ | 
|  | 252 | text_poke(*ptr, ((unsigned char []){0x3E}), 1); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 253 | }; | 
|  | 254 | } | 
|  | 255 |  | 
|  | 256 | struct smp_alt_module { | 
|  | 257 | /* what is this ??? */ | 
|  | 258 | struct module	*mod; | 
|  | 259 | char		*name; | 
|  | 260 |  | 
|  | 261 | /* ptrs to lock prefixes */ | 
|  | 262 | u8		**locks; | 
|  | 263 | u8		**locks_end; | 
|  | 264 |  | 
|  | 265 | /* .text segment, needed to avoid patching init code ;) */ | 
|  | 266 | u8		*text; | 
|  | 267 | u8		*text_end; | 
|  | 268 |  | 
|  | 269 | struct list_head next; | 
|  | 270 | }; | 
|  | 271 | static LIST_HEAD(smp_alt_modules); | 
| Pekka Paalanen | 2f1dafe | 2008-05-12 21:21:01 +0200 | [diff] [blame] | 272 | static DEFINE_MUTEX(smp_alt); | 
| Andi Kleen | ca74a6f | 2008-01-30 13:33:17 +0100 | [diff] [blame] | 273 | static int smp_mode = 1;	/* protected by smp_alt */ | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 274 |  | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 275 | void alternatives_smp_module_add(struct module *mod, char *name, | 
|  | 276 | void *locks, void *locks_end, | 
|  | 277 | void *text,  void *text_end) | 
|  | 278 | { | 
|  | 279 | struct smp_alt_module *smp; | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 280 |  | 
| Jeremy Fitzhardinge | b7fb4af | 2007-05-02 19:27:13 +0200 | [diff] [blame] | 281 | if (noreplace_smp) | 
|  | 282 | return; | 
|  | 283 |  | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 284 | if (smp_alt_once) { | 
|  | 285 | if (boot_cpu_has(X86_FEATURE_UP)) | 
|  | 286 | alternatives_smp_unlock(locks, locks_end, | 
|  | 287 | text, text_end); | 
|  | 288 | return; | 
|  | 289 | } | 
|  | 290 |  | 
|  | 291 | smp = kzalloc(sizeof(*smp), GFP_KERNEL); | 
|  | 292 | if (NULL == smp) | 
|  | 293 | return; /* we'll run the (safe but slow) SMP code then ... */ | 
|  | 294 |  | 
|  | 295 | smp->mod	= mod; | 
|  | 296 | smp->name	= name; | 
|  | 297 | smp->locks	= locks; | 
|  | 298 | smp->locks_end	= locks_end; | 
|  | 299 | smp->text	= text; | 
|  | 300 | smp->text_end	= text_end; | 
|  | 301 | DPRINTK("%s: locks %p -> %p, text %p -> %p, name %s\n", | 
| Harvey Harrison | 77bf90e | 2008-03-03 11:37:23 -0800 | [diff] [blame] | 302 | __func__, smp->locks, smp->locks_end, | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 303 | smp->text, smp->text_end, smp->name); | 
|  | 304 |  | 
| Pekka Paalanen | 2f1dafe | 2008-05-12 21:21:01 +0200 | [diff] [blame] | 305 | mutex_lock(&smp_alt); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 306 | list_add_tail(&smp->next, &smp_alt_modules); | 
|  | 307 | if (boot_cpu_has(X86_FEATURE_UP)) | 
|  | 308 | alternatives_smp_unlock(smp->locks, smp->locks_end, | 
|  | 309 | smp->text, smp->text_end); | 
| Pekka Paalanen | 2f1dafe | 2008-05-12 21:21:01 +0200 | [diff] [blame] | 310 | mutex_unlock(&smp_alt); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 311 | } | 
|  | 312 |  | 
|  | 313 | void alternatives_smp_module_del(struct module *mod) | 
|  | 314 | { | 
|  | 315 | struct smp_alt_module *item; | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 316 |  | 
| Jeremy Fitzhardinge | b7fb4af | 2007-05-02 19:27:13 +0200 | [diff] [blame] | 317 | if (smp_alt_once || noreplace_smp) | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 318 | return; | 
|  | 319 |  | 
| Pekka Paalanen | 2f1dafe | 2008-05-12 21:21:01 +0200 | [diff] [blame] | 320 | mutex_lock(&smp_alt); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 321 | list_for_each_entry(item, &smp_alt_modules, next) { | 
|  | 322 | if (mod != item->mod) | 
|  | 323 | continue; | 
|  | 324 | list_del(&item->next); | 
| Pekka Paalanen | 2f1dafe | 2008-05-12 21:21:01 +0200 | [diff] [blame] | 325 | mutex_unlock(&smp_alt); | 
| Harvey Harrison | 77bf90e | 2008-03-03 11:37:23 -0800 | [diff] [blame] | 326 | DPRINTK("%s: %s\n", __func__, item->name); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 327 | kfree(item); | 
|  | 328 | return; | 
|  | 329 | } | 
| Pekka Paalanen | 2f1dafe | 2008-05-12 21:21:01 +0200 | [diff] [blame] | 330 | mutex_unlock(&smp_alt); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 331 | } | 
|  | 332 |  | 
|  | 333 | void alternatives_smp_switch(int smp) | 
|  | 334 | { | 
|  | 335 | struct smp_alt_module *mod; | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 336 |  | 
| Ingo Molnar | 3047e99 | 2006-07-03 00:24:57 -0700 | [diff] [blame] | 337 | #ifdef CONFIG_LOCKDEP | 
|  | 338 | /* | 
| Ingo Molnar | 17abecf | 2008-01-30 13:33:24 +0100 | [diff] [blame] | 339 | * Older binutils section handling bug prevented | 
|  | 340 | * alternatives-replacement from working reliably. | 
|  | 341 | * | 
|  | 342 | * If this still occurs then you should see a hang | 
|  | 343 | * or crash shortly after this line: | 
| Ingo Molnar | 3047e99 | 2006-07-03 00:24:57 -0700 | [diff] [blame] | 344 | */ | 
| Ingo Molnar | 17abecf | 2008-01-30 13:33:24 +0100 | [diff] [blame] | 345 | printk("lockdep: fixing up alternatives.\n"); | 
| Ingo Molnar | 3047e99 | 2006-07-03 00:24:57 -0700 | [diff] [blame] | 346 | #endif | 
|  | 347 |  | 
| Jeremy Fitzhardinge | b7fb4af | 2007-05-02 19:27:13 +0200 | [diff] [blame] | 348 | if (noreplace_smp || smp_alt_once) | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 349 | return; | 
|  | 350 | BUG_ON(!smp && (num_online_cpus() > 1)); | 
|  | 351 |  | 
| Pekka Paalanen | 2f1dafe | 2008-05-12 21:21:01 +0200 | [diff] [blame] | 352 | mutex_lock(&smp_alt); | 
| Andi Kleen | ca74a6f | 2008-01-30 13:33:17 +0100 | [diff] [blame] | 353 |  | 
|  | 354 | /* | 
|  | 355 | * Avoid unnecessary switches because it forces JIT based VMs to | 
|  | 356 | * throw away all cached translations, which can be quite costly. | 
|  | 357 | */ | 
|  | 358 | if (smp == smp_mode) { | 
|  | 359 | /* nothing */ | 
|  | 360 | } else if (smp) { | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 361 | printk(KERN_INFO "SMP alternatives: switching to SMP code\n"); | 
| Jeremy Fitzhardinge | 53756d3 | 2008-01-30 13:30:55 +0100 | [diff] [blame] | 362 | clear_cpu_cap(&boot_cpu_data, X86_FEATURE_UP); | 
|  | 363 | clear_cpu_cap(&cpu_data(0), X86_FEATURE_UP); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 364 | list_for_each_entry(mod, &smp_alt_modules, next) | 
|  | 365 | alternatives_smp_lock(mod->locks, mod->locks_end, | 
|  | 366 | mod->text, mod->text_end); | 
|  | 367 | } else { | 
|  | 368 | printk(KERN_INFO "SMP alternatives: switching to UP code\n"); | 
| Jeremy Fitzhardinge | 53756d3 | 2008-01-30 13:30:55 +0100 | [diff] [blame] | 369 | set_cpu_cap(&boot_cpu_data, X86_FEATURE_UP); | 
|  | 370 | set_cpu_cap(&cpu_data(0), X86_FEATURE_UP); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 371 | list_for_each_entry(mod, &smp_alt_modules, next) | 
|  | 372 | alternatives_smp_unlock(mod->locks, mod->locks_end, | 
|  | 373 | mod->text, mod->text_end); | 
|  | 374 | } | 
| Andi Kleen | ca74a6f | 2008-01-30 13:33:17 +0100 | [diff] [blame] | 375 | smp_mode = smp; | 
| Pekka Paalanen | 2f1dafe | 2008-05-12 21:21:01 +0200 | [diff] [blame] | 376 | mutex_unlock(&smp_alt); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 377 | } | 
|  | 378 |  | 
| Gerd Hoffmann | 8ec4d41 | 2006-07-01 04:36:18 -0700 | [diff] [blame] | 379 | #endif | 
|  | 380 |  | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 381 | #ifdef CONFIG_PARAVIRT | 
| Jeremy Fitzhardinge | 98de032 | 2007-05-02 19:27:14 +0200 | [diff] [blame] | 382 | void apply_paravirt(struct paravirt_patch_site *start, | 
|  | 383 | struct paravirt_patch_site *end) | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 384 | { | 
| Jeremy Fitzhardinge | 98de032 | 2007-05-02 19:27:14 +0200 | [diff] [blame] | 385 | struct paravirt_patch_site *p; | 
| Andi Kleen | ab144f5 | 2007-08-10 22:31:03 +0200 | [diff] [blame] | 386 | char insnbuf[MAX_PATCH_LEN]; | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 387 |  | 
| Jeremy Fitzhardinge | 959b4fd | 2007-05-02 19:27:16 +0200 | [diff] [blame] | 388 | if (noreplace_paravirt) | 
|  | 389 | return; | 
|  | 390 |  | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 391 | for (p = start; p < end; p++) { | 
|  | 392 | unsigned int used; | 
|  | 393 |  | 
| Andi Kleen | ab144f5 | 2007-08-10 22:31:03 +0200 | [diff] [blame] | 394 | BUG_ON(p->len > MAX_PATCH_LEN); | 
| Chris Wright | d34fda4 | 2007-08-18 14:31:41 -0700 | [diff] [blame] | 395 | /* prep the buffer with the original instructions */ | 
|  | 396 | memcpy(insnbuf, p->instr, p->len); | 
| Jeremy Fitzhardinge | 93b1eab | 2007-10-16 11:51:29 -0700 | [diff] [blame] | 397 | used = pv_init_ops.patch(p->instrtype, p->clobbers, insnbuf, | 
|  | 398 | (unsigned long)p->instr, p->len); | 
| Jeremy Fitzhardinge | 7f63c41 | 2007-05-02 19:27:13 +0200 | [diff] [blame] | 399 |  | 
| Jeremy Fitzhardinge | 63f7027 | 2007-05-02 19:27:14 +0200 | [diff] [blame] | 400 | BUG_ON(used > p->len); | 
|  | 401 |  | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 402 | /* Pad the rest with nops */ | 
| Andi Kleen | ab144f5 | 2007-08-10 22:31:03 +0200 | [diff] [blame] | 403 | add_nops(insnbuf + used, p->len - used); | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 404 | text_poke_early(p->instr, insnbuf, p->len); | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 405 | } | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 406 | } | 
| Jeremy Fitzhardinge | 98de032 | 2007-05-02 19:27:14 +0200 | [diff] [blame] | 407 | extern struct paravirt_patch_site __start_parainstructions[], | 
| Rusty Russell | 139ec7c | 2006-12-07 02:14:08 +0100 | [diff] [blame] | 408 | __stop_parainstructions[]; | 
|  | 409 | #endif	/* CONFIG_PARAVIRT */ | 
|  | 410 |  | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 411 | void __init alternative_instructions(void) | 
|  | 412 | { | 
| Andi Kleen | 8f4e956 | 2007-07-22 11:12:32 +0200 | [diff] [blame] | 413 | /* The patching is not fully atomic, so try to avoid local interruptions | 
|  | 414 | that might execute the to be patched code. | 
|  | 415 | Other CPUs are not running. */ | 
|  | 416 | stop_nmi(); | 
| Adrian Bunk | d2d0251 | 2007-08-10 22:31:06 +0200 | [diff] [blame] | 417 | #ifdef CONFIG_X86_MCE | 
| Andi Kleen | 8f4e956 | 2007-07-22 11:12:32 +0200 | [diff] [blame] | 418 | stop_mce(); | 
|  | 419 | #endif | 
|  | 420 |  | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 421 | apply_alternatives(__alt_instructions, __alt_instructions_end); | 
|  | 422 |  | 
|  | 423 | /* switch to patch-once-at-boottime-only mode and free the | 
|  | 424 | * tables in case we know the number of CPUs will never ever | 
|  | 425 | * change */ | 
|  | 426 | #ifdef CONFIG_HOTPLUG_CPU | 
|  | 427 | if (num_possible_cpus() < 2) | 
|  | 428 | smp_alt_once = 1; | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 429 | #endif | 
|  | 430 |  | 
| Gerd Hoffmann | 8ec4d41 | 2006-07-01 04:36:18 -0700 | [diff] [blame] | 431 | #ifdef CONFIG_SMP | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 432 | if (smp_alt_once) { | 
|  | 433 | if (1 == num_possible_cpus()) { | 
|  | 434 | printk(KERN_INFO "SMP alternatives: switching to UP code\n"); | 
| Jeremy Fitzhardinge | 53756d3 | 2008-01-30 13:30:55 +0100 | [diff] [blame] | 435 | set_cpu_cap(&boot_cpu_data, X86_FEATURE_UP); | 
|  | 436 | set_cpu_cap(&cpu_data(0), X86_FEATURE_UP); | 
|  | 437 |  | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 438 | alternatives_smp_unlock(__smp_locks, __smp_locks_end, | 
|  | 439 | _text, _etext); | 
|  | 440 | } | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 441 | } else { | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 442 | alternatives_smp_module_add(NULL, "core kernel", | 
|  | 443 | __smp_locks, __smp_locks_end, | 
|  | 444 | _text, _etext); | 
| Andi Kleen | ca74a6f | 2008-01-30 13:33:17 +0100 | [diff] [blame] | 445 |  | 
|  | 446 | /* Only switch to UP mode if we don't immediately boot others */ | 
| Thomas Gleixner | 649c665 | 2008-10-05 16:52:24 +0200 | [diff] [blame] | 447 | if (num_present_cpus() == 1 || setup_max_cpus <= 1) | 
| Andi Kleen | ca74a6f | 2008-01-30 13:33:17 +0100 | [diff] [blame] | 448 | alternatives_smp_switch(0); | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 449 | } | 
| Gerd Hoffmann | 8ec4d41 | 2006-07-01 04:36:18 -0700 | [diff] [blame] | 450 | #endif | 
| Jeremy Fitzhardinge | 441d40d | 2007-05-02 19:27:16 +0200 | [diff] [blame] | 451 | apply_paravirt(__parainstructions, __parainstructions_end); | 
| Andi Kleen | 8f4e956 | 2007-07-22 11:12:32 +0200 | [diff] [blame] | 452 |  | 
| Fengguang Wu | f68fd5f | 2007-10-17 18:04:34 +0200 | [diff] [blame] | 453 | if (smp_alt_once) | 
|  | 454 | free_init_pages("SMP alternatives", | 
|  | 455 | (unsigned long)__smp_locks, | 
|  | 456 | (unsigned long)__smp_locks_end); | 
|  | 457 |  | 
| Andi Kleen | 8f4e956 | 2007-07-22 11:12:32 +0200 | [diff] [blame] | 458 | restart_nmi(); | 
| Adrian Bunk | d2d0251 | 2007-08-10 22:31:06 +0200 | [diff] [blame] | 459 | #ifdef CONFIG_X86_MCE | 
| Andi Kleen | 8f4e956 | 2007-07-22 11:12:32 +0200 | [diff] [blame] | 460 | restart_mce(); | 
|  | 461 | #endif | 
| Gerd Hoffmann | 9a0b581 | 2006-03-23 02:59:32 -0800 | [diff] [blame] | 462 | } | 
| Andi Kleen | 19d36cc | 2007-07-22 11:12:31 +0200 | [diff] [blame] | 463 |  | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 464 | /** | 
|  | 465 | * text_poke_early - Update instructions on a live kernel at boot time | 
|  | 466 | * @addr: address to modify | 
|  | 467 | * @opcode: source of the copy | 
|  | 468 | * @len: length to copy | 
|  | 469 | * | 
| Andi Kleen | 19d36cc | 2007-07-22 11:12:31 +0200 | [diff] [blame] | 470 | * When you use this code to patch more than one byte of an instruction | 
|  | 471 | * you need to make sure that other CPUs cannot execute this code in parallel. | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 472 | * Also no thread must be currently preempted in the middle of these | 
|  | 473 | * instructions. And on the local CPU you need to be protected again NMI or MCE | 
|  | 474 | * handlers seeing an inconsistent instruction while you patch. | 
| Andi Kleen | 19d36cc | 2007-07-22 11:12:31 +0200 | [diff] [blame] | 475 | */ | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 476 | void *text_poke_early(void *addr, const void *opcode, size_t len) | 
| Andi Kleen | 19d36cc | 2007-07-22 11:12:31 +0200 | [diff] [blame] | 477 | { | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 478 | unsigned long flags; | 
|  | 479 | local_irq_save(flags); | 
| Andi Kleen | 19d36cc | 2007-07-22 11:12:31 +0200 | [diff] [blame] | 480 | memcpy(addr, opcode, len); | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 481 | local_irq_restore(flags); | 
| Andi Kleen | 19d36cc | 2007-07-22 11:12:31 +0200 | [diff] [blame] | 482 | sync_core(); | 
| Andi Kleen | a534b67 | 2007-09-06 16:59:52 +0200 | [diff] [blame] | 483 | /* Could also do a CLFLUSH here to speed up CPU recovery; but | 
|  | 484 | that causes hangs on some VIA CPUs. */ | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 485 | return addr; | 
|  | 486 | } | 
|  | 487 |  | 
|  | 488 | /** | 
|  | 489 | * text_poke - Update instructions on a live kernel | 
|  | 490 | * @addr: address to modify | 
|  | 491 | * @opcode: source of the copy | 
|  | 492 | * @len: length to copy | 
|  | 493 | * | 
|  | 494 | * Only atomic text poke/set should be allowed when not doing early patching. | 
|  | 495 | * It means the size must be writable atomically and the address must be aligned | 
|  | 496 | * in a way that permits an atomic write. It also makes sure we fit on a single | 
|  | 497 | * page. | 
|  | 498 | */ | 
|  | 499 | void *__kprobes text_poke(void *addr, const void *opcode, size_t len) | 
|  | 500 | { | 
|  | 501 | unsigned long flags; | 
|  | 502 | char *vaddr; | 
|  | 503 | int nr_pages = 2; | 
| Mathieu Desnoyers | b7b66ba | 2008-04-24 11:03:33 -0400 | [diff] [blame] | 504 | struct page *pages[2]; | 
|  | 505 | int i; | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 506 |  | 
| Mathieu Desnoyers | b7b66ba | 2008-04-24 11:03:33 -0400 | [diff] [blame] | 507 | if (!core_kernel_text((unsigned long)addr)) { | 
|  | 508 | pages[0] = vmalloc_to_page(addr); | 
|  | 509 | pages[1] = vmalloc_to_page(addr + PAGE_SIZE); | 
| Mathieu Desnoyers | 15a601e | 2008-03-12 11:54:16 -0400 | [diff] [blame] | 510 | } else { | 
| Mathieu Desnoyers | b7b66ba | 2008-04-24 11:03:33 -0400 | [diff] [blame] | 511 | pages[0] = virt_to_page(addr); | 
| Ingo Molnar | 00c6b2d | 2008-04-25 17:07:03 +0200 | [diff] [blame] | 512 | WARN_ON(!PageReserved(pages[0])); | 
| Mathieu Desnoyers | b7b66ba | 2008-04-24 11:03:33 -0400 | [diff] [blame] | 513 | pages[1] = virt_to_page(addr + PAGE_SIZE); | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 514 | } | 
| Mathieu Desnoyers | b7b66ba | 2008-04-24 11:03:33 -0400 | [diff] [blame] | 515 | BUG_ON(!pages[0]); | 
|  | 516 | if (!pages[1]) | 
|  | 517 | nr_pages = 1; | 
|  | 518 | vaddr = vmap(pages, nr_pages, VM_MAP, PAGE_KERNEL); | 
|  | 519 | BUG_ON(!vaddr); | 
|  | 520 | local_irq_save(flags); | 
|  | 521 | memcpy(&vaddr[(unsigned long)addr & ~PAGE_MASK], opcode, len); | 
|  | 522 | local_irq_restore(flags); | 
|  | 523 | vunmap(vaddr); | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 524 | sync_core(); | 
|  | 525 | /* Could also do a CLFLUSH here to speed up CPU recovery; but | 
|  | 526 | that causes hangs on some VIA CPUs. */ | 
| Mathieu Desnoyers | b7b66ba | 2008-04-24 11:03:33 -0400 | [diff] [blame] | 527 | for (i = 0; i < len; i++) | 
|  | 528 | BUG_ON(((char *)addr)[i] != ((char *)opcode)[i]); | 
| Mathieu Desnoyers | e587cad | 2008-03-06 08:48:49 -0500 | [diff] [blame] | 529 | return addr; | 
| Andi Kleen | 19d36cc | 2007-07-22 11:12:31 +0200 | [diff] [blame] | 530 | } |