Re: [PATCH v2 1/6] KVM: add kvm_arch_sched_in

From: Radim KrÄmÃÅ
Date: Thu Aug 21 2014 - 07:38:22 EST


2014-08-21 10:29+0200, Paolo Bonzini:
> Il 20/08/2014 22:53, Radim KrÄmÃÅ ha scritto:
> > Introduce preempt notifiers for architecture specific code.
> > Advantage over creating a new notifier in every arch is slightly simpler
> > code and guaranteed call order with respect to kvm_sched_in.
> >
> > Signed-off-by: Radim KrÄmÃÅ <rkrcmar@xxxxxxxxxx>
> > ---
> > arch/arm/kvm/arm.c | 4 ++++
> > arch/mips/kvm/mips.c | 4 ++++
> > arch/powerpc/kvm/powerpc.c | 4 ++++
> > arch/s390/kvm/kvm-s390.c | 4 ++++
> > arch/x86/kvm/x86.c | 4 ++++
>
> What about adding them as static inlines in
> arch/*/include/asm/kvm_host.h (except for arch/x86 of course)?

All empty arch functions are in '.c' files, so it seems better to follow
the same path.
(And have one refactoring patch if GCC does not optimize this.)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/