WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[PATCH] Re: [Xen-devel] redundant set_cr0 in vmxassist setup.c

To: "Li, Xin B" <xin.b.li@xxxxxxxxx>
Subject: [PATCH] Re: [Xen-devel] redundant set_cr0 in vmxassist setup.c
From: Leendert van Doorn <leendert@xxxxxxxxxxxxxx>
Date: Fri, 10 Feb 2006 22:23:26 -0500
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Sat, 11 Feb 2006 03:37:28 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: Your message of "Fri, 10 Feb 2006 17:03:16 +0800." <59D45D057E9702469E5775CBB56411F1019AB20B@pdsmsx406>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
# >Leendert, when I'm reading the vmxassist code, I think we need this
# >patch, can you take a look for me?

Actually all that code can go. It used to be necessary to force Xen to
switch to the right page table before switching to the vmxassist code.
Asit&Jun removed that dependency a long time ago by using the 1-1 table
that the vmx builder constructed (and that Xen now constructs). This code
was left over.

        Leendert

vmxassist/setup.c cleanup. It is no longer necessary to switch to an
alternate page table before transfering into the real mode emulator.

Signed-Off-By: "Li, Xin B" <xin.b.li@xxxxxxxxx>
Signed-Off-By: Leendert van Doorn <leendert@xxxxxxxxxxxxxx>

diff -r 1d36cca98fc3 tools/firmware/vmxassist/setup.c
--- a/tools/firmware/vmxassist/setup.c  Fri Feb 10 11:35:19 2006
+++ b/tools/firmware/vmxassist/setup.c  Fri Feb 10 23:12:49 2006
@@ -353,8 +353,6 @@
 void
 start_bios(void)
 {
-       unsigned long cr0;
-
        if (booting_cpu == 0)
                printf("Start BIOS ...\n");
        else
@@ -362,11 +360,7 @@
                       booting_cpu, booting_vector << 12);
 
        initialize_real_mode = 1;
-       cr0 = get_cr0();
-#ifndef TEST
-       set_cr0(cr0 | CR0_PE);
-#endif
-       set_cr0(cr0 & ~CR0_PE);
+       set_cr0(get_cr0() & ~CR0_PE);
        panic("vmxassist returned"); /* "cannot happen" */
 }
 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>