On Tue, 2012-03-13 at 14:47 +0100, Alexander Graf wrote:
> > +++ b/arch/powerpc/include/asm/kvm_host.h
> > @@ -183,10 +183,14 @@ struct kvm_arch {
> > unsigned long lpcr;
> > unsigned long rmor;
> > struct kvmppc_rma_info *rma;
> > - struct list_head spapr_tce_tables;
> > unsigned short last_vcpu[NR_CPUS];
> > struct kvmppc_vcore *vcores[KVM_MAX_VCORES];
> > + struct list_head spapr_tce_tables;
> > #endif /* CONFIG_KVM_BOOK3S_64_HV */
> > +
> > +#ifdef CONFIG_KVM_BOOK3S_64_PR
> > + struct list_head spapr_tce_tables;
> > +#endif /* CONFIG_KVM_BOOK3S_64_PR */
>
> Please move the definition to an #ifdef on CONFIG_KVM_BOOK3S_64. That way we get rid of the duplication here.
I did that initially but that doesn't work when it's a module, as
CONFIG_KVM_BOOK3S_64 is "m" in Kconfig and thus not defined as such
but CONFIG_KVM_BOOK3S_64_MODULE is)
Maybe we should change the way our Kconfig is organized but I din't
feel like doing so yesterday :-)
> Could you please enable rename support in git format-patch? This way it's
> really hard to see what you changed between the 2 files - if anything.
I thought I had, I'll dbl check. There was no code change, just moves,
the original file only had the one small function in it and I moved
over the rest from book3s_hv.c
> > -#ifdef CONFIG_KVM_BOOK3S_64_HV
> > +#if defined(CONFIG_KVM_BOOK3S_64_PR) || defined(CONFIG_KVM_BOOK3S_64_HV)
>
> CONFIG_KVM_BOOK3S_64
Breaks modules.
> Otherwise a nice patch - thanks a lot for tackling this :). Also, please always CC kvm@vger in
> addition to kvm-ppc@vger, so Avi can't complain that he didn't see the patch earlier ;).
Heh ok.
Cheers,
Ben.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
