Messages in this thread | | | Date | Wed, 1 Feb 2017 17:08:52 +0000 | From | Will Deacon <> | Subject | Re: [PATCH v4 3/4] arm64: Use __tlbi() macros in KVM code |
| |
On Wed, Feb 01, 2017 at 05:02:43PM +0000, Punit Agrawal wrote: > Will Deacon <will.deacon@arm.com> writes: > > > On Wed, Jan 25, 2017 at 08:39:43PM +0100, Christoffer Dall wrote: > >> On Wed, Jan 25, 2017 at 10:52:31AM -0500, Christopher Covington wrote: > >> > Refactor the KVM code to use the __tlbi macros, which will allow an errata > >> > workaround that repeats tlbi dsb sequences to only change one location. > >> > This is not intended to change the generated assembly and comparing before > >> > and after vmlinux objdump shows no functional changes. > >> > > >> > Signed-off-by: Christopher Covington <cov@codeaurora.org> > >> > >> Acked-by: Christoffer Dall <christoffer.dall@linaro.org> > > > > Thanks, I'll queue this one via arm64. > > I just noticed this patch but there's been a similar patch from Mark > that I've been carrying as part of the KVM TLB monitoring series[0]. > > [0] http://www.mail-archive.com/kvmarm@lists.cs.columbia.edu/msg09359.html
Well I've already queued the one from Christopher. It's weird that Mark's version appears to miss the local VMID case (but it does have the flush_icache_all I wanted :).
Anyway, I'm not reverting anything, so you'll need to rebase when this lot lands in mainline.
Will
| |