Re: [RFT] kvm with Windows optimization

[Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 813 bytes --]

Avi Kivity wrote:
> Jindrich Makovicka wrote:
>   
>> On Thu, 25 Oct 2007 13:39:33 -0400
>> "Haydn Solomon" <haydn.solomon-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>
>>   
>>     
>>> Thanks, that was quick.
>>>     
>>>       
>> Finally, this (pre)release solved the long standing Java IE Plugin
>> lockup problem for me. Thanks a lot!
>>
>>   
>>     
>
> No, this release is meant to cause problems, not fix them.
>
> (but thanks for the reminder)
>
>   

It's probably due to disabling NX (which is an unwanted side effect of 
this).  Java generates executable code, so it probably needs to modify 
protection attributes.  Maybe we're bad at this.

Can you check the attached patch?  Against normal kvm, not the windows 
accelerator.

-- 
error compiling committee.c: too many arguments to function


[-- Attachment #2: no-nx.patch --]
[-- Type: text/x-patch, Size: 392 bytes --]

diff --git a/drivers/kvm/x86.c b/drivers/kvm/x86.c
index 1fe209d..e08e749 100644
--- a/drivers/kvm/x86.c
+++ b/drivers/kvm/x86.c
@@ -194,7 +194,7 @@ static void cpuid_fix_nx_cap(struct kvm_vcpu *vcpu)
 			break;
 		}
 	}
-	if (entry && (entry->edx & (1 << 20)) && !(efer & EFER_NX)) {
+	if (entry) {
 		entry->edx &= ~(1 << 20);
 		printk(KERN_INFO "kvm: guest NX capability removed\n");
 	}

[-- Attachment #3: Type: text/plain, Size: 314 bytes --]

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/

[-- Attachment #4: Type: text/plain, Size: 186 bytes --]

_______________________________________________
kvm-devel mailing list
kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/kvm-devel