mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-06 10:57:46 +00:00
powerpc/kvm/book3s_hv: Rework the secondary inhibit code
As part of the support for split core on POWER8, we want to be able to block splitting of the core while KVM VMs are active. The logic to do that would be exactly the same as the code we currently have for inhibiting onlining of secondaries. Instead of adding an identical mechanism to block split core, rework the secondary inhibit code to be a "HV KVM is active" check. We can then use that in both the cpu hotplug code and the upcoming split core code. Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> Signed-off-by: Michael Neuling <mikey@neuling.org> Acked-by: Alexander Graf <agraf@suse.de> Acked-by: Paul Mackerras <paulus@samba.org> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
This commit is contained in:
parent
64bb80d87f
commit
441c19c8a2
5 changed files with 45 additions and 43 deletions
|
@ -337,6 +337,10 @@ static inline void kvmppc_fast_vcpu_kick(struct kvm_vcpu *vcpu)
|
||||||
vcpu->kvm->arch.kvm_ops->fast_vcpu_kick(vcpu);
|
vcpu->kvm->arch.kvm_ops->fast_vcpu_kick(vcpu);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
extern void kvm_hv_vm_activated(void);
|
||||||
|
extern void kvm_hv_vm_deactivated(void);
|
||||||
|
extern bool kvm_hv_mode_active(void);
|
||||||
|
|
||||||
#else
|
#else
|
||||||
static inline void __init kvm_cma_reserve(void)
|
static inline void __init kvm_cma_reserve(void)
|
||||||
{}
|
{}
|
||||||
|
@ -356,6 +360,9 @@ static inline void kvmppc_fast_vcpu_kick(struct kvm_vcpu *vcpu)
|
||||||
{
|
{
|
||||||
kvm_vcpu_kick(vcpu);
|
kvm_vcpu_kick(vcpu);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static inline bool kvm_hv_mode_active(void) { return false; }
|
||||||
|
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#ifdef CONFIG_KVM_XICS
|
#ifdef CONFIG_KVM_XICS
|
||||||
|
|
|
@ -68,14 +68,6 @@ void generic_mach_cpu_die(void);
|
||||||
void generic_set_cpu_dead(unsigned int cpu);
|
void generic_set_cpu_dead(unsigned int cpu);
|
||||||
void generic_set_cpu_up(unsigned int cpu);
|
void generic_set_cpu_up(unsigned int cpu);
|
||||||
int generic_check_cpu_restart(unsigned int cpu);
|
int generic_check_cpu_restart(unsigned int cpu);
|
||||||
|
|
||||||
extern void inhibit_secondary_onlining(void);
|
|
||||||
extern void uninhibit_secondary_onlining(void);
|
|
||||||
|
|
||||||
#else /* HOTPLUG_CPU */
|
|
||||||
static inline void inhibit_secondary_onlining(void) {}
|
|
||||||
static inline void uninhibit_secondary_onlining(void) {}
|
|
||||||
|
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#ifdef CONFIG_PPC64
|
#ifdef CONFIG_PPC64
|
||||||
|
|
|
@ -36,6 +36,7 @@
|
||||||
#include <linux/atomic.h>
|
#include <linux/atomic.h>
|
||||||
#include <asm/irq.h>
|
#include <asm/irq.h>
|
||||||
#include <asm/hw_irq.h>
|
#include <asm/hw_irq.h>
|
||||||
|
#include <asm/kvm_ppc.h>
|
||||||
#include <asm/page.h>
|
#include <asm/page.h>
|
||||||
#include <asm/pgtable.h>
|
#include <asm/pgtable.h>
|
||||||
#include <asm/prom.h>
|
#include <asm/prom.h>
|
||||||
|
@ -458,38 +459,9 @@ int generic_check_cpu_restart(unsigned int cpu)
|
||||||
return per_cpu(cpu_state, cpu) == CPU_UP_PREPARE;
|
return per_cpu(cpu_state, cpu) == CPU_UP_PREPARE;
|
||||||
}
|
}
|
||||||
|
|
||||||
static atomic_t secondary_inhibit_count;
|
static bool secondaries_inhibited(void)
|
||||||
|
|
||||||
/*
|
|
||||||
* Don't allow secondary CPU threads to come online
|
|
||||||
*/
|
|
||||||
void inhibit_secondary_onlining(void)
|
|
||||||
{
|
{
|
||||||
/*
|
return kvm_hv_mode_active();
|
||||||
* This makes secondary_inhibit_count stable during cpu
|
|
||||||
* online/offline operations.
|
|
||||||
*/
|
|
||||||
get_online_cpus();
|
|
||||||
|
|
||||||
atomic_inc(&secondary_inhibit_count);
|
|
||||||
put_online_cpus();
|
|
||||||
}
|
|
||||||
EXPORT_SYMBOL_GPL(inhibit_secondary_onlining);
|
|
||||||
|
|
||||||
/*
|
|
||||||
* Allow secondary CPU threads to come online again
|
|
||||||
*/
|
|
||||||
void uninhibit_secondary_onlining(void)
|
|
||||||
{
|
|
||||||
get_online_cpus();
|
|
||||||
atomic_dec(&secondary_inhibit_count);
|
|
||||||
put_online_cpus();
|
|
||||||
}
|
|
||||||
EXPORT_SYMBOL_GPL(uninhibit_secondary_onlining);
|
|
||||||
|
|
||||||
static int secondaries_inhibited(void)
|
|
||||||
{
|
|
||||||
return atomic_read(&secondary_inhibit_count);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
#else /* HOTPLUG_CPU */
|
#else /* HOTPLUG_CPU */
|
||||||
|
|
|
@ -2317,10 +2317,10 @@ static int kvmppc_core_init_vm_hv(struct kvm *kvm)
|
||||||
spin_lock_init(&kvm->arch.slot_phys_lock);
|
spin_lock_init(&kvm->arch.slot_phys_lock);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Don't allow secondary CPU threads to come online
|
* Track that we now have a HV mode VM active. This blocks secondary
|
||||||
* while any KVM VMs exist.
|
* CPU threads from coming online.
|
||||||
*/
|
*/
|
||||||
inhibit_secondary_onlining();
|
kvm_hv_vm_activated();
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
@ -2336,7 +2336,7 @@ static void kvmppc_free_vcores(struct kvm *kvm)
|
||||||
|
|
||||||
static void kvmppc_core_destroy_vm_hv(struct kvm *kvm)
|
static void kvmppc_core_destroy_vm_hv(struct kvm *kvm)
|
||||||
{
|
{
|
||||||
uninhibit_secondary_onlining();
|
kvm_hv_vm_deactivated();
|
||||||
|
|
||||||
kvmppc_free_vcores(kvm);
|
kvmppc_free_vcores(kvm);
|
||||||
if (kvm->arch.rma) {
|
if (kvm->arch.rma) {
|
||||||
|
|
|
@ -6,6 +6,7 @@
|
||||||
* published by the Free Software Foundation.
|
* published by the Free Software Foundation.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
|
#include <linux/cpu.h>
|
||||||
#include <linux/kvm_host.h>
|
#include <linux/kvm_host.h>
|
||||||
#include <linux/preempt.h>
|
#include <linux/preempt.h>
|
||||||
#include <linux/export.h>
|
#include <linux/export.h>
|
||||||
|
@ -181,3 +182,33 @@ void __init kvm_cma_reserve(void)
|
||||||
kvm_cma_declare_contiguous(selected_size, align_size);
|
kvm_cma_declare_contiguous(selected_size, align_size);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
|
* When running HV mode KVM we need to block certain operations while KVM VMs
|
||||||
|
* exist in the system. We use a counter of VMs to track this.
|
||||||
|
*
|
||||||
|
* One of the operations we need to block is onlining of secondaries, so we
|
||||||
|
* protect hv_vm_count with get/put_online_cpus().
|
||||||
|
*/
|
||||||
|
static atomic_t hv_vm_count;
|
||||||
|
|
||||||
|
void kvm_hv_vm_activated(void)
|
||||||
|
{
|
||||||
|
get_online_cpus();
|
||||||
|
atomic_inc(&hv_vm_count);
|
||||||
|
put_online_cpus();
|
||||||
|
}
|
||||||
|
EXPORT_SYMBOL_GPL(kvm_hv_vm_activated);
|
||||||
|
|
||||||
|
void kvm_hv_vm_deactivated(void)
|
||||||
|
{
|
||||||
|
get_online_cpus();
|
||||||
|
atomic_dec(&hv_vm_count);
|
||||||
|
put_online_cpus();
|
||||||
|
}
|
||||||
|
EXPORT_SYMBOL_GPL(kvm_hv_vm_deactivated);
|
||||||
|
|
||||||
|
bool kvm_hv_mode_active(void)
|
||||||
|
{
|
||||||
|
return atomic_read(&hv_vm_count) != 0;
|
||||||
|
}
|
||||||
|
|
Loading…
Reference in a new issue