linux-kernel - [patch 0/1] remove existing enclosed mtrr region

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <fbd277f20810090117g7b916d2fia201b9184d41bd7d@mail.gmail.com>
Date:	Thu, 9 Oct 2008 16:17:15 +0800
From:	"bibo mao" <bibo.mao@...il.com>
To:	linux-kernel@...r.kernel.org
Cc:	akpm@...ux-foundation.org
Subject: [patch 0/1] remove existing enclosed mtrr region

Hi,
 This patch is remove enclosed mtrr region when adding new region.

 Signed-off-by: bibo mao <bibo.mao@...il.com>

 thanks
 bibo,mao


 kernel/cpu/mtrr/main.c          |   35 +++++++++++++++++++++++++++++++----
 3 files changed, 31 insertions(+), 4 deletions(-)
---------------------------------------------------------------------------------------------------------------------

diff --git a/arch/x86/kernel/cpu/mtrr/main.c b/arch/x86/kernel/cpu/mtrr/main.c
index 6a1e278..6e212f3 100644
--- a/arch/x86/kernel/cpu/mtrr/main.c
+++ b/arch/x86/kernel/cpu/mtrr/main.c
@@ -303,7 +303,7 @@ static void set_mtrr(unsigned int reg, unsigned long base,
 int mtrr_add_page(unsigned long base, unsigned long size,
 		  unsigned int type, bool increment)
 {
-	int i, replace, error;
+	int i, j, replace, error;
 	mtrr_type ltype;
 	unsigned long lbase, lsize;

@@ -351,7 +351,7 @@ int mtrr_add_page(unsigned long base, unsigned long size,
 			if (base <= lbase && base + size - 1 >= lbase + lsize - 1) {
 				/*  New region encloses an existing region  */
 				if (type == ltype) {
-					replace = replace == -1 ? i : -2;
+					replace = i;
 					continue;
 				}
 				else if (types_compatible(type, ltype))
@@ -391,6 +391,20 @@ int mtrr_add_page(unsigned long base, unsigned long size,
 				set_mtrr(replace, 0, 0, 0);
 				mtrr_usage_table[replace] = 0;
 			}
+
+			/* remove existing enclosed region */
+			for (j = 0; j < num_var_ranges; ++j) {
+				if (i == j)
+					continue;
+				mtrr_if->get(j, &lbase, &lsize, &ltype);
+				if (lsize && type == ltype && base <= lbase
+					 && base + size - 1 >= lbase + lsize - 1) {
+					/*  New region encloses an existing region  */
+					mtrr_usage_table[i] += mtrr_usage_table[j];
+					set_mtrr(j, 0, 0, 0);
+					mtrr_usage_table[j]=0;
+				}	
+			}
 		}
 	} else
 		printk(KERN_INFO "mtrr: no more MTRRs available\n");
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/