From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-1.mimecast.com ([205.139.110.61]:33718 "EHLO us-smtp-delivery-1.mimecast.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726593AbgAMNKT (ORCPT ); Mon, 13 Jan 2020 08:10:19 -0500 Subject: Re: [kvm-unit-tests PATCH v7 4/4] s390x: SCLP unit test References: <20200110184050.191506-1-imbrenda@linux.ibm.com> <20200110184050.191506-5-imbrenda@linux.ibm.com> <8d7fb5c4-9e2c-e28a-16c0-658afcc8178d@redhat.com> <20200113133325.417bf657@p-imbrenda> <1b86b00a-261e-3d8c-fa52-c30e67463ad5@redhat.com> <20200113135832.1c6d3bb8@p-imbrenda> From: David Hildenbrand Message-ID: <22b5ce6a-18af-edec-efc6-e03450faddf8@redhat.com> Date: Mon, 13 Jan 2020 14:10:10 +0100 MIME-Version: 1.0 In-Reply-To: <20200113135832.1c6d3bb8@p-imbrenda> Content-Type: text/plain; charset=windows-1252 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-s390-owner@vger.kernel.org List-ID: To: Claudio Imbrenda Cc: kvm@vger.kernel.org, linux-s390@vger.kernel.org, thuth@redhat.com, borntraeger@de.ibm.com, frankja@linux.ibm.com On 13.01.20 13:58, Claudio Imbrenda wrote: > On Mon, 13 Jan 2020 13:48:17 +0100 > David Hildenbrand wrote: > > [...] > >>>> >>>> I wonder if something like the following would be possible: >>>> >>>> expect_pgm_int(); >>>> ... >>>> asm volatiole(); >>>> ... >>>> sclp_wait_busy(); >>>> check_pgm_int_code(PGM_INT_CODE_SPECIFICATION); >>> >>> we do not expect a specification exception, if that happens it's >>> a bug and the test should rightfully fail. >> >> Which one do we expect? (you're not checking for a specific one, >> should you?) > > nothing, the call should succeed :) :) I'm confused by the fact that you "expect_pgm_int()" but actually don't expect one ... Please enlighten me why this isn't + sclp_mark_busy(); + h->length = 8; + sclp_setup_int(); + + asm volatile( + " .insn rre,0xb2204200,%1,%2\n" /* servc %1,%2 */ + " ipm %0\n" + " srl %0,28" + : "=&d" (cc) : "d" (valid_code), "a" (__pa(pagebuf)) + : "cc", "memory"); + if (!cc) + sclp_wait_busy(); + report(cc == 0, "Instruction format ignored bits"); I feel like I am missing something important. -- Thanks, David / dhildenb