[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH 4/4] SVM: streamline entry.S code
>>> On 04.09.13 at 16:39, Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx> wrote: > On 08/23/2013 10:04 AM, Jan Beulich wrote: >> @@ -92,25 +97,26 @@ UNLIKELY_END(svm_trace) >> >> VMRUN >> >> + GET_CURRENT(%rax) >> push %rdi >> push %rsi >> push %rdx >> push %rcx >> + mov VCPU_svm_vmcb(%rax),%rcx >> push %rax >> push %r8 >> push %r9 >> push %r10 >> push %r11 >> push %rbx >> + mov %rax,%rbx > > Can you continue with using %rax as 'current' pointer below and not save it > to %rbx? %rax appears to be a temp register so perhaps you can use, say, > %rdi, for that purpose. I'm sorry, but I don't understand what you're asking for. We actually _want_ "current" to be in a register thats callee-saved, such that we don't need to reload it after function calls. One goal of the patch in fact is to eliminate such redundant reloads. But as I'm not sure what you want, this explanation may be entirely off. And as a side note: At this point I'm not really expecting requests for further changes, unless you spot a mistake in the patch. It was around for review for long enough, and as said it had two reviews already. And further optimization should go in a separate, incremental patch. Jan _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |