Attachment 120787 Details for Bug 228344

[patch] polished patch, reduces binary size by one for loop for all calls to set_fixed_range

bug228344-fix4-tested.diff (text/plain), 4.87 KB, created by Bernhard Kaindl on 2007-02-23 15:28:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Bernhard Kaindl

Created: 2007-02-23 15:28:16 UTC

Size: 4.87 KB

patch

obsolete

>--- linux-2.6.20/arch/i386/kernel/cpu/mtrr/generic.c
>+++ linux-228344/arch/i386/kernel/cpu/mtrr/generic.c
>@@ -20,12 +20,28 @@
> 	mtrr_type def_type;
> };
> 
>+struct fixed_range_block {
>+	int base_msr;
>+	int ranges;
>+};
>+
>+static struct fixed_range_block fixed_range_blocks[] = {
>+	{ MTRRfix64K_00000_MSR, 1},
>+	{ MTRRfix16K_80000_MSR, 2},
>+	{ MTRRfix4K_C0000_MSR,  8},
>+	{ 0,  0}
>+};
>+
> static unsigned long smp_changes_mask;
> static struct mtrr_state mtrr_state = {};
> 
> #undef MODULE_PARAM_PREFIX
> #define MODULE_PARAM_PREFIX "mtrr."
> 
>+#define MSR_K8_SYSCFG                   0xC0010010
>+#define MSR_K8_SYSCFG_ENABLE_IORRs      0x000C0000
>+#define MTRR_K8_RDMEM_WRMEM_MASK        0x18181818
>+
> static __initdata int mtrr_show;
> module_param_named(show, mtrr_show, bool, 0);
> 
>@@ -37,7 +53,10 @@
> 	rdmsr(MTRRphysMask_MSR(index), vr->mask_lo, vr->mask_hi);
> }
> 
>-static void __init
>+/*
>+ * Retrieves the current fixed-range MTRRs from the CPU
>+ */
>+static void 
> get_fixed_ranges(mtrr_type * frs)
> {
> 	unsigned int *p = (unsigned int *) frs;
>@@ -51,6 +70,44 @@
> 		rdmsr(MTRRfix4K_C0000_MSR + i, p[6 + i * 2], p[7 + i * 2]);
> }
> 
>+/*
>+ * Called from save_processor_context() to update our copy of the MTRRs
>+ */
>+void mtrr_save_fixed_ranges(void)
>+{
>+	get_fixed_ranges(mtrr_state.fixed_ranges);
>+}
>+
>+/*
>+ * Enables the extended fixed-range MTRR bits on K8 CPUs
>+ */
>+static inline void k8_enable_fixed_iorrs(void)
>+{
>+	unsigned lo, hi;
>+
>+	rdmsr(MSR_K8_SYSCFG, lo, hi);
>+	mtrr_wrmsr(MSR_K8_SYSCFG, lo | MSR_K8_SYSCFG_ENABLE_IORRs, hi);
>+}
>+
>+/*
>+ * Updates an MTRR. Updates *changed if changes were made (or tried)
>+ */
>+static void set_fixed_range(int msr, int * changed, unsigned int * msrwords)
>+{
>+	unsigned lo, hi;
>+
>+	rdmsr(msr, lo, hi);
>+
>+	if (lo != msrwords[0] || hi != msrwords[1]) {
>+		if (boot_cpu_data.x86_vendor == X86_VENDOR_AMD &&
>+		    boot_cpu_data.x86 == 15 &&
>+		    ((msrwords[0] | msrwords[1]) & MTRR_K8_RDMEM_WRMEM_MASK))
>+			k8_enable_fixed_iorrs();
>+		mtrr_wrmsr(msr, msrwords[0], msrwords[1]);
>+		*changed = TRUE;
>+	}
>+}
>+
> static void __init print_fixed(unsigned base, unsigned step, const mtrr_type*types)
> {
> 	unsigned i;
>@@ -196,36 +253,17 @@
> 	*type = base_lo & 0xff;
> }
> 
>-static int set_fixed_ranges(mtrr_type * frs)
>+static int set_fixed_ranges(mtrr_type * fixed_range_setup)
> {
>-	unsigned int *p = (unsigned int *) frs;
>-	int changed = FALSE;
>-	int i;
>-	unsigned int lo, hi;
>-
>-	rdmsr(MTRRfix64K_00000_MSR, lo, hi);
>-	if (p[0] != lo || p[1] != hi) {
>-		mtrr_wrmsr(MTRRfix64K_00000_MSR, p[0], p[1]);
>-		changed = TRUE;
>-	}
>-
>-	for (i = 0; i < 2; i++) {
>-		rdmsr(MTRRfix16K_80000_MSR + i, lo, hi);
>-		if (p[2 + i * 2] != lo || p[3 + i * 2] != hi) {
>-			mtrr_wrmsr(MTRRfix16K_80000_MSR + i, p[2 + i * 2],
>-			      p[3 + i * 2]);
>-			changed = TRUE;
>-		}
>-	}
>+	unsigned long long *saved = (unsigned long long *) fixed_range_setup;
>+	int changed = FALSE;
>+	int block=-1, range;
>+
>+	while (fixed_range_blocks[++block].ranges)
>+	    for (range=0; range < fixed_range_blocks[block].ranges; range++)
>+		set_fixed_range(fixed_range_blocks[block].base_msr + range,
>+		    &changed, (unsigned int *) saved++);
> 
>-	for (i = 0; i < 8; i++) {
>-		rdmsr(MTRRfix4K_C0000_MSR + i, lo, hi);
>-		if (p[6 + i * 2] != lo || p[7 + i * 2] != hi) {
>-			mtrr_wrmsr(MTRRfix4K_C0000_MSR + i, p[6 + i * 2],
>-			      p[7 + i * 2]);
>-			changed = TRUE;
>-		}
>-	}
> 	return changed;
> }
> 
>--- linux-2.6.20/arch/i386/power/cpu.c
>+++ linux-228344/arch/i386/power/cpu.c
>@@ -21,6 +21,7 @@
> 
> void __save_processor_state(struct saved_context *ctxt)
> {
>+	mtrr_save_fixed_ranges();
> 	kernel_fpu_begin();
> 
> 	/*
>--- linux-2.6.20/arch/x86_64/kernel/suspend.c
>+++ linux-228344/arch/x86_64/kernel/suspend.c
>@@ -46,6 +46,7 @@
> 	rdmsrl(MSR_FS_BASE, ctxt->fs_base);
> 	rdmsrl(MSR_GS_BASE, ctxt->gs_base);
> 	rdmsrl(MSR_KERNEL_GS_BASE, ctxt->gs_kernel_base);
>+	mtrr_save_fixed_ranges();
> 
> 	/*
> 	 * control registers 
>--- linux-2.6.20/include/asm-i386/mtrr.h
>+++ linux-228344/include/asm-i386/mtrr.h
>@@ -78,7 +78,8 @@
> extern void mtrr_centaur_report_mcr(int mcr, u32 lo, u32 hi);
> extern void mtrr_ap_init(void);
> extern void mtrr_bp_init(void);
>-#  else
>+extern void mtrr_save_fixed_ranges(void);
>+#  else  /*  CONFIG_MTRR set/not set */
> static __inline__ int mtrr_add (unsigned long base, unsigned long size,
> 				unsigned int type, char increment)
> {
>@@ -104,8 +105,9 @@
> 
> #define mtrr_ap_init() do {} while (0)
> #define mtrr_bp_init() do {} while (0)
>-#  endif
>+#define mtrr_save_fixed_ranges() do {} while (0)
>+#  endif  /*  !CONFIG_MTRR  */
> 
>-#endif
>+#endif  /*  __KERNEL__  */
> 
> #endif  /*  _LINUX_MTRR_H  */
>--- linux-2.6.20/include/asm-x86_64/proto.h
>+++ linux-228344/include/asm-x86_64/proto.h
>@@ -19,7 +19,9 @@
> #ifdef CONFIG_MTRR
> extern void mtrr_ap_init(void);
> extern void mtrr_bp_init(void);
>+extern void mtrr_save_fixed_ranges(void);
> #else
>+#define mtrr_save_fixed_ranges() do {} while (0)
> #define mtrr_ap_init() do {} while (0)
> #define mtrr_bp_init() do {} while (0)
> #endif

Actions: View | Diff

Attachments on bug 228344: 109606 | 112500 | 112752 | 114778 | 120361 | 120608 | 120787 | 121120 | 121351 | 121632