On 09/11/2014 03:16 PM, Christoffer Dall wrote:
> On Thu, Sep 11, 2014 at 12:42:15PM -0700, Mario Smarduch wrote:
>> On 09/10/2014 08:12 PM, Christoffer Dall wrote:
>>> On Tue, Aug 26, 2014 at 05:04:03PM -0700, Mario Smarduch wrote:
>>>> Add support to declare architecture specific TLB flush function, for now only
>>>> ARMv7.
>>>>
>>>> Signed-off-by: Mario Smarduch <m.smarduch@xxxxxxxxxxx>
>>>> ---
>>>> virt/kvm/Kconfig | 3 +++
>>>> virt/kvm/kvm_main.c | 4 ++++
>>>> 2 files changed, 7 insertions(+)
>>>>
>>>> diff --git a/virt/kvm/Kconfig b/virt/kvm/Kconfig
>>>> index 13f2d19..f1efaa5 100644
>>>> --- a/virt/kvm/Kconfig
>>>> +++ b/virt/kvm/Kconfig
>>>> @@ -34,3 +34,6 @@ config HAVE_KVM_CPU_RELAX_INTERCEPT
>>>>
>>>> config KVM_VFIO
>>>> bool
>>>> +
>>>> +config HAVE_KVM_ARCH_TLB_FLUSH_ALL
>>>> + bool
>>>> diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
>>>> index 4b6c01b..d0a24f5 100644
>>>> --- a/virt/kvm/kvm_main.c
>>>> +++ b/virt/kvm/kvm_main.c
>>>> @@ -186,12 +186,16 @@ static bool make_all_cpus_request(struct kvm *kvm, unsigned int req)
>>>>
>>>> void kvm_flush_remote_tlbs(struct kvm *kvm)
>>>> {
>>>> +#ifdef CONFIG_HAVE_KVM_ARCH_TLB_FLUSH_ALL
>>>> + kvm_arch_flush_remote_tlbs(kvm);
>>>
>>> missing prototype declaration?
>>
>> A subsequent patch adds HAVE_KVM_ARCH_TLB_FLUSH_ALL support
>> for armv7 that includes prototype definition, so far armv7
>> is only architecture with HAVE_KVM_ARCH_TLB_FLUSH_ALL declared.
>>
> Shouldn't it be declared in include/linux/kvm_host.h?
Yes it should be in generic, will update.
>
> -Christoffer
>
_______________________________________________
kvmarm mailing list
kvmarm@xxxxxxxxxxxxxxxxxxxxx
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
