forked from Minki/linux
powerpc/64s/exception: Only test KVM in SRR interrupts when PR KVM is supported
Apart from SRESET, MCE, and syscall (hcall variant), the SRR type interrupts are not escalated to hypervisor mode, so are delivered to the OS. When running PR KVM, the OS is the hypervisor, and the guest runs with MSR[PR]=1 (ie. usermode), so these interrupts must test if a guest was running when interrupted. These tests are required at the real-mode entry points because the PR KVM host runs with LPCR[AIL]=0. In HV KVM and nested HV KVM, the guest always receives these interrupts, so there is no need for the host to make this test. So remove the tests if PR KVM is not configured. Signed-off-by: Nicholas Piggin <npiggin@gmail.com> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> Link: https://lore.kernel.org/r/20200225173541.1549955-21-npiggin@gmail.com
This commit is contained in:
parent
94325357e8
commit
2284ffea8f
@ -214,9 +214,36 @@ do_define_int n
|
||||
#ifdef CONFIG_KVM_BOOK3S_64_HANDLER
|
||||
#ifdef CONFIG_KVM_BOOK3S_HV_POSSIBLE
|
||||
/*
|
||||
* If hv is possible, interrupts come into to the hv version
|
||||
* of the kvmppc_interrupt code, which then jumps to the PR handler,
|
||||
* kvmppc_interrupt_pr, if the guest is a PR guest.
|
||||
* All interrupts which set HSRR registers, as well as SRESET and MCE and
|
||||
* syscall when invoked with "sc 1" switch to MSR[HV]=1 (HVMODE) to be taken,
|
||||
* so they all generally need to test whether they were taken in guest context.
|
||||
*
|
||||
* Note: SRESET and MCE may also be sent to the guest by the hypervisor, and be
|
||||
* taken with MSR[HV]=0.
|
||||
*
|
||||
* Interrupts which set SRR registers (with the above exceptions) do not
|
||||
* elevate to MSR[HV]=1 mode, though most can be taken when running with
|
||||
* MSR[HV]=1 (e.g., bare metal kernel and userspace). So these interrupts do
|
||||
* not need to test whether a guest is running because they get delivered to
|
||||
* the guest directly, including nested HV KVM guests.
|
||||
*
|
||||
* The exception is PR KVM, where the guest runs with MSR[PR]=1 and the host
|
||||
* runs with MSR[HV]=0, so the host takes all interrupts on behalf of the
|
||||
* guest. PR KVM runs with LPCR[AIL]=0 which causes interrupts to always be
|
||||
* delivered to the real-mode entry point, therefore such interrupts only test
|
||||
* KVM in their real mode handlers, and only when PR KVM is possible.
|
||||
*
|
||||
* Interrupts that are taken in MSR[HV]=0 and escalate to MSR[HV]=1 are always
|
||||
* delivered in real-mode when the MMU is in hash mode because the MMU
|
||||
* registers are not set appropriately to translate host addresses. In nested
|
||||
* radix mode these can be delivered in virt-mode as the host translations are
|
||||
* used implicitly (see: effective LPID, effective PID).
|
||||
*/
|
||||
|
||||
/*
|
||||
* If an interrupt is taken while a guest is running, it is immediately routed
|
||||
* to KVM to handle. If both HV and PR KVM arepossible, KVM interrupts go first
|
||||
* to kvmppc_interrupt_hv, which handles the PR guest case.
|
||||
*/
|
||||
#define kvmppc_interrupt kvmppc_interrupt_hv
|
||||
#else
|
||||
@ -1274,8 +1301,10 @@ INT_DEFINE_BEGIN(data_access)
|
||||
IVEC=0x300
|
||||
IDAR=1
|
||||
IDSISR=1
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_SKIP=1
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(data_access)
|
||||
|
||||
EXC_REAL_BEGIN(data_access, 0x300, 0x80)
|
||||
@ -1322,8 +1351,10 @@ INT_DEFINE_BEGIN(data_access_slb)
|
||||
IAREA=PACA_EXSLB
|
||||
IRECONCILE=0
|
||||
IDAR=1
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_SKIP=1
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(data_access_slb)
|
||||
|
||||
EXC_REAL_BEGIN(data_access_slb, 0x380, 0x80)
|
||||
@ -1373,7 +1404,9 @@ INT_DEFINE_BEGIN(instruction_access)
|
||||
IISIDE=1
|
||||
IDAR=1
|
||||
IDSISR=1
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(instruction_access)
|
||||
|
||||
EXC_REAL_BEGIN(instruction_access, 0x400, 0x80)
|
||||
@ -1412,7 +1445,9 @@ INT_DEFINE_BEGIN(instruction_access_slb)
|
||||
IRECONCILE=0
|
||||
IISIDE=1
|
||||
IDAR=1
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(instruction_access_slb)
|
||||
|
||||
EXC_REAL_BEGIN(instruction_access_slb, 0x480, 0x80)
|
||||
@ -1504,7 +1539,9 @@ INT_DEFINE_BEGIN(alignment)
|
||||
IVEC=0x600
|
||||
IDAR=1
|
||||
IDSISR=1
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(alignment)
|
||||
|
||||
EXC_REAL_BEGIN(alignment, 0x600, 0x100)
|
||||
@ -1534,7 +1571,9 @@ EXC_COMMON_BEGIN(alignment_common)
|
||||
*/
|
||||
INT_DEFINE_BEGIN(program_check)
|
||||
IVEC=0x700
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(program_check)
|
||||
|
||||
EXC_REAL_BEGIN(program_check, 0x700, 0x100)
|
||||
@ -1597,7 +1636,9 @@ EXC_COMMON_BEGIN(program_check_common)
|
||||
INT_DEFINE_BEGIN(fp_unavailable)
|
||||
IVEC=0x800
|
||||
IRECONCILE=0
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(fp_unavailable)
|
||||
|
||||
EXC_REAL_BEGIN(fp_unavailable, 0x800, 0x100)
|
||||
@ -1659,7 +1700,9 @@ END_FTR_SECTION_IFSET(CPU_FTR_TM)
|
||||
INT_DEFINE_BEGIN(decrementer)
|
||||
IVEC=0x900
|
||||
IMASK=IRQS_DISABLED
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(decrementer)
|
||||
|
||||
EXC_REAL_BEGIN(decrementer, 0x900, 0x80)
|
||||
@ -1744,7 +1787,9 @@ EXC_COMMON_BEGIN(hdecrementer_common)
|
||||
INT_DEFINE_BEGIN(doorbell_super)
|
||||
IVEC=0xa00
|
||||
IMASK=IRQS_DISABLED
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(doorbell_super)
|
||||
|
||||
EXC_REAL_BEGIN(doorbell_super, 0xa00, 0x100)
|
||||
@ -1922,7 +1967,9 @@ END_FTR_SECTION_IFSET(CPU_FTR_HAS_PPR)
|
||||
*/
|
||||
INT_DEFINE_BEGIN(single_step)
|
||||
IVEC=0xd00
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(single_step)
|
||||
|
||||
EXC_REAL_BEGIN(single_step, 0xd00, 0x100)
|
||||
@ -2213,7 +2260,9 @@ EXC_VIRT_NONE(0x4ee0, 0x20)
|
||||
INT_DEFINE_BEGIN(performance_monitor)
|
||||
IVEC=0xf00
|
||||
IMASK=IRQS_PMI_DISABLED
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(performance_monitor)
|
||||
|
||||
EXC_REAL_BEGIN(performance_monitor, 0xf00, 0x20)
|
||||
@ -2242,7 +2291,9 @@ EXC_COMMON_BEGIN(performance_monitor_common)
|
||||
INT_DEFINE_BEGIN(altivec_unavailable)
|
||||
IVEC=0xf20
|
||||
IRECONCILE=0
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(altivec_unavailable)
|
||||
|
||||
EXC_REAL_BEGIN(altivec_unavailable, 0xf20, 0x20)
|
||||
@ -2296,7 +2347,9 @@ END_FTR_SECTION_IFSET(CPU_FTR_ALTIVEC)
|
||||
INT_DEFINE_BEGIN(vsx_unavailable)
|
||||
IVEC=0xf40
|
||||
IRECONCILE=0
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(vsx_unavailable)
|
||||
|
||||
EXC_REAL_BEGIN(vsx_unavailable, 0xf40, 0x20)
|
||||
@ -2349,7 +2402,9 @@ END_FTR_SECTION_IFSET(CPU_FTR_VSX)
|
||||
*/
|
||||
INT_DEFINE_BEGIN(facility_unavailable)
|
||||
IVEC=0xf60
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(facility_unavailable)
|
||||
|
||||
EXC_REAL_BEGIN(facility_unavailable, 0xf60, 0x20)
|
||||
@ -2439,8 +2494,10 @@ EXC_VIRT_NONE(0x5200, 0x100)
|
||||
|
||||
INT_DEFINE_BEGIN(instruction_breakpoint)
|
||||
IVEC=0x1300
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_SKIP=1
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(instruction_breakpoint)
|
||||
|
||||
EXC_REAL_BEGIN(instruction_breakpoint, 0x1300, 0x100)
|
||||
@ -2611,7 +2668,9 @@ EXC_VIRT_NONE(0x5600, 0x100)
|
||||
|
||||
INT_DEFINE_BEGIN(altivec_assist)
|
||||
IVEC=0x1700
|
||||
#ifdef CONFIG_KVM_BOOK3S_PR_POSSIBLE
|
||||
IKVM_REAL=1
|
||||
#endif
|
||||
INT_DEFINE_END(altivec_assist)
|
||||
|
||||
EXC_REAL_BEGIN(altivec_assist, 0x1700, 0x100)
|
||||
|
Loading…
Reference in New Issue
Block a user