From: Wei Wang2 <wei.wang2@amd.com>
To: Tim Deegan <Tim.Deegan@citrix.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: [RFC PATCH 0/2] ASID: Flush by ASID
Date: Wed, 12 Jan 2011 13:41:10 +0100 [thread overview]
Message-ID: <201101121341.11200.wei.wang2@amd.com> (raw)
In-Reply-To: <20110112101700.GG5651@whitby.uk.xensource.com>
Hi Tim,
Flush by ASID provides more flexible control of tlb flushing. The most
advantage is to allow hypervisor to flush tagged tlb selectively. Using this
feature, HV is able to flush tlb entries associated with a guest VM directly
instead of allocating a new asid . The whole tlb flush will also be reduced
by reducing asid allocation.
So far, we did not measure drastic performance improvement in testing with
kernbench and X11perf. Actually, we found out that, reducing tlb flushes
accompanying with vmrun does not improve performance very much.
we sent out a patch to optimize hvm_flush_guest_tlbs last week, which reduces
over 90% tlb flushes for vmrun, and we even cannot see signification speedup
with it. Maybe, the latency of vmrun is too big so that the overhead of tlb
flush is negligible?
Thanks,
Wei
On Wednesday 12 January 2011 11:17:00 Tim Deegan wrote:
> At 17:55 +0000 on 11 Jan (1294768552), Wei Wang2 wrote:
> > Future AMD SVM supports a new feature called flush by ASID. The idea is
> > to allow CPU to flush TLBs associated with the ASID assigned to guest VM.
> > So hypervisor doesn't have to reassign a new ASID in order to flush
> > guest's VCPU. Please review it.
>
> What advantage does the new system have? Intuitively it seems like it
> might be a tiny bit fairer and a tiny bit faster (by explicitly flushing
> instead of relying on LRO) but I'm not convinced that it will be visible
> in macro-benchmarks. Have you measured it?
>
> Cheers,
>
> Tim.
>
> > Thanks,
> > Wei
> >
> > Signed-off-by: Wei Huang <wei.huang2@amd.com>
> > Signed-off-by: Wei Wang <wei.wang2@amd.com>
> > --
> > Advanced Micro Devices GmbH
> > Sitz: Dornach, Gemeinde Aschheim,
> > Landkreis München Registergericht München,
> > HRB Nr. 43632
> > WEEE-Reg-Nr: DE 12919551
> > Geschäftsführer:
> > Alberto Bozzo, Andrew Bowd
> >
> >
> > _______________________________________________
> > Xen-devel mailing list
> > Xen-devel@lists.xensource.com
> > http://lists.xensource.com/xen-devel
next prev parent reply other threads:[~2011-01-12 12:41 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-11 17:55 [RFC PATCH 0/2] ASID: Flush by ASID Wei Wang2
2011-01-12 10:17 ` Tim Deegan
2011-01-12 12:41 ` Wei Wang2 [this message]
2011-01-12 12:48 ` Keir Fraser
2011-01-12 13:23 ` Wei Wang2
2011-01-12 13:38 ` Keir Fraser
2011-01-12 17:22 ` Wei Huang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201101121341.11200.wei.wang2@amd.com \
--to=wei.wang2@amd.com \
--cc=Tim.Deegan@citrix.com \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.