On 05/09/2016 01:00 PM, Christian Borntraeger wrote: > On 05/09/2016 12:32 PM, Heiko Carstens wrote: >> On Mon, May 09, 2016 at 11:20:15AM +0200, Christian Borntraeger wrote: >>> From: David Hildenbrand <dahi@xxxxxxxxxxxxxxxxxx> >>> >>> If we don't have SIGP SENSE RUNNING STATUS enabled for the guest, let's >>> not enable interpretation so we can correctly report an invalid order. >>> >>> Reviewed-by: Christian Borntraeger <borntraeger@xxxxxxxxxx> >>> Signed-off-by: David Hildenbrand <dahi@xxxxxxxxxxxxxxxxxx> >>> Signed-off-by: Christian Borntraeger <borntraeger@xxxxxxxxxx> >>> --- >>> arch/s390/include/asm/sigp.h | 1 + >>> arch/s390/kvm/kvm-s390.c | 4 +++- >>> arch/s390/kvm/sigp.c | 6 ++++++ >>> 3 files changed, 10 insertions(+), 1 deletion(-) >>> >>> diff --git a/arch/s390/include/asm/sigp.h b/arch/s390/include/asm/sigp.h >>> index ec60cf7..35c91da 100644 >>> --- a/arch/s390/include/asm/sigp.h >>> +++ b/arch/s390/include/asm/sigp.h >>> @@ -33,6 +33,7 @@ >>> #define SIGP_STATUS_INVALID_PARAMETER 0x00000100UL >>> #define SIGP_STATUS_INCORRECT_STATE 0x00000200UL >>> #define SIGP_STATUS_NOT_RUNNING 0x00000400UL >>> +#define SIGP_STATUS_INVALID_ORDER 0x00000002UL >> >> This new define should have been added at the top the list to keep the list >> sorted. Maybe Paolo or Radim can change this when applying this patch. > > Its a signed pull request and I must redo it if requested. > > > Conny noticed a typo in the subject of patch3. So I will redo the pull request as we have now 2 things to change. Christian -- 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