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

Re: [Xen-devel] svm vmexit action sequence

To: Jan Beulich <jbeulich@xxxxxxxxxx>, Mats Petersson <Mats.Petersson@xxxxxxx>
Subject: Re: [Xen-devel] svm vmexit action sequence
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Fri, 11 May 2007 08:37:00 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Fri, 11 May 2007 00:33:52 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <46443730.76E4.0078.0@xxxxxxxxxx>
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
Thread-index: AceTnyj7Z5ESk/+SEduD4gAWy6hiGQ==
Thread-topic: [Xen-devel] svm vmexit action sequence
User-agent: Microsoft-Entourage/11.3.3.061214
See the unstable staging tree for some changes already in this area. As you
can see there is still quite some scope for improvement - I bet we can just
get rid of this push/popf pair for example. And the horrendous
VMLOAD/VMSAVE/VMLOAD/VMSAVE sequence can probably be shoved off at least to
only context switches.

 -- Keir

On 11/5/07 08:28, "Jan Beulich" <jbeulich@xxxxxxxxxx> wrote:

> One more question: both variants have HVM_SAVE_ALL_NOSEGREGS do a
> forced reset of eflags/rflags - what is this needed for? #VMEXIT supposedly
> restores them. Jan
> 
>>>> "Petersson, Mats" <Mats.Petersson@xxxxxxx> 10.05.07 18:12 >>>
> 
> 
>> -----Original Message-----
>> From: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
>> [mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of
>> Jan Beulich
>> Sent: 10 May 2007 17:02
>> To: xen-devel@xxxxxxxxxxxxxxxxxxx
>> Subject: [Xen-devel] svm vmexit action sequence
>> 
>> Is there any particular reason why on 32-bits the order is VMLOAD then
>> HVM_SAVE_ALL_NOSEGREGS, while on 64-bits its is the other way around?
>> Trying to put in the saving of EAX, I could save a
>> GET_CURRENT() on 32-bits
>> if I could order things the same way as on 64-bits.
> 
> I don't see any reason why these shouldn't be the same (or at least as
> similar as possible).
>> 
>> Also, both versions seem to have a redundant GET_CURRENT() right after
>> the clgi/sti sequence - again, is there a particular reason for this?
> 
> No reason as far as I can tell. Assuming rbx (in 64-bit case) isn't
> clobbered by called functions, that is. I can't remember for 64-bit if
> rbx is "safe" or not. [It certainly is safe in 32-bit].
> 
> Thanks for spotting these things.
> 
> --
> Mats
>> 
>> Thanks, Jan
>> 
>> 
>> 
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@xxxxxxxxxxxxxxxxxxx
>> http://lists.xensource.com/xen-devel
>> 
>> 
>> 
> 
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



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