On 04.02.20 13:16, David Hildenbrand wrote: > >> arch/s390/include/asm/kvm_host.h | 2 ++ >> arch/s390/kvm/intercept.c | 10 ++++++++++ >> 2 files changed, 12 insertions(+) >> >> diff --git a/arch/s390/include/asm/kvm_host.h b/arch/s390/include/asm/kvm_host.h >> index 841690d05080..d63ed05272ec 100644 >> --- a/arch/s390/include/asm/kvm_host.h >> +++ b/arch/s390/include/asm/kvm_host.h >> @@ -215,6 +215,8 @@ struct kvm_s390_sie_block { >> #define ICPT_PARTEXEC 0x38 >> #define ICPT_IOINST 0x40 >> #define ICPT_KSS 0x5c >> +#define ICPT_PV_MCHKR 0x60 >> +#define ICPT_PV_INT_EN 0x64 >> __u8 icptcode; /* 0x0050 */ >> __u8 icptstatus; /* 0x0051 */ >> __u16 ihcpu; /* 0x0052 */ >> diff --git a/arch/s390/kvm/intercept.c b/arch/s390/kvm/intercept.c >> index a389fa85cca2..eaa2a21c3170 100644 >> --- a/arch/s390/kvm/intercept.c >> +++ b/arch/s390/kvm/intercept.c >> @@ -480,6 +480,16 @@ int kvm_handle_sie_intercept(struct kvm_vcpu *vcpu) >> case ICPT_KSS: >> rc = kvm_s390_skey_check_enable(vcpu); >> break; >> + case ICPT_PV_MCHKR: >> + /* fallthrough */ > > No need for this annotation if there is no additional code in the case. ack