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?
-Christoffer
_______________________________________________
kvmarm mailing list
kvmarm@xxxxxxxxxxxxxxxxxxxxx
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
