Re: [PATCH v4 3/4] arm64: Use __tlbi() macros in KVM code

From: Punit Agrawal
Date: Wed Feb 01 2017 - 12:14:42 EST


Will Deacon <will.deacon@xxxxxxx> writes:

> On Wed, Feb 01, 2017 at 05:02:43PM +0000, Punit Agrawal wrote:
>> Will Deacon <will.deacon@xxxxxxx> 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@xxxxxxxxxxxxxx>
>> >>
>> >> Acked-by: Christoffer Dall <christoffer.dall@xxxxxxxxxx>
>> >
>> > 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@xxxxxxxxxxxxxxxxxxxxx/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.

Sure. I'll drop the patch the next time around.

The patch is unrelated to that series and should've gone in when
we introduced the __tlbi() macro but got left out somehow.

>
> Will
> _______________________________________________
> kvmarm mailing list
> kvmarm@xxxxxxxxxxxxxxxxxxxxx
> https://lists.cs.columbia.edu/mailman/listinfo/kvmarm