From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DA4A3C433DF for ; Wed, 19 Aug 2020 11:51:17 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id B4ACF20FC3 for ; Wed, 19 Aug 2020 11:51:17 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=ibm.com header.i=@ibm.com header.b="mAqzFmN9" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728400AbgHSLvG (ORCPT ); Wed, 19 Aug 2020 07:51:06 -0400 Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:3802 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728324AbgHSLtq (ORCPT ); Wed, 19 Aug 2020 07:49:46 -0400 Received: from pps.filterd (m0098404.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.42/8.16.0.42) with SMTP id 07JBYbQO147303; Wed, 19 Aug 2020 07:49:45 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=from : to : cc : subject : in-reply-to : references : date : message-id : mime-version : content-type : content-transfer-encoding; s=pp1; bh=9cNPlplRw+LP5S/qDQ4bMW92g7lSbHvH8XNz1+luTM4=; b=mAqzFmN9EfkabblW3ZAUry/paW4vaSsq/eQxcdoH71RXHNXsEo3SX8S/8doj66eAsDzu cAMfzlaPlvxrrX2S5wHJrpzLX5Lmzsl///AfzVy9XUTRwkTc1zkSSZ8t+JvNuvfjfLoq OSE0NgILIAPdxLfcWhJ97CwYMky9DwofmGn2BLnQms8uuRwAX54kDvuUfpMUF57A+Trt 6FOfZCBjrjrL0d/FK0cdk9KEIk1laRl97oQYvrKeuC+wBAUSNlGROKd4o+Yk2iZdHyID g2v2DPPeWwgkCefS1r77pNdZ+zIPvvyUmD08hb4HgzK7SeiayM1NFnXY0PpuVAeLSHaX TA== Received: from pps.reinject (localhost [127.0.0.1]) by mx0a-001b2d01.pphosted.com with ESMTP id 330ccb1gnc-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 19 Aug 2020 07:49:45 -0400 Received: from m0098404.ppops.net (m0098404.ppops.net [127.0.0.1]) by pps.reinject (8.16.0.36/8.16.0.36) with SMTP id 07JBZ6cq149739; Wed, 19 Aug 2020 07:49:45 -0400 Received: from ppma06ams.nl.ibm.com (66.31.33a9.ip4.static.sl-reverse.com [169.51.49.102]) by mx0a-001b2d01.pphosted.com with ESMTP id 330ccb1gmh-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 19 Aug 2020 07:49:45 -0400 Received: from pps.filterd (ppma06ams.nl.ibm.com [127.0.0.1]) by ppma06ams.nl.ibm.com (8.16.0.42/8.16.0.42) with SMTP id 07JBg8oM024610; Wed, 19 Aug 2020 11:49:42 GMT Received: from b06cxnps4074.portsmouth.uk.ibm.com (d06relay11.portsmouth.uk.ibm.com [9.149.109.196]) by ppma06ams.nl.ibm.com with ESMTP id 330tbvrg75-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 19 Aug 2020 11:49:42 +0000 Received: from d06av22.portsmouth.uk.ibm.com (d06av22.portsmouth.uk.ibm.com [9.149.105.58]) by b06cxnps4074.portsmouth.uk.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 07JBndI120906248 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 19 Aug 2020 11:49:39 GMT Received: from d06av22.portsmouth.uk.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 83C614C058; Wed, 19 Aug 2020 11:49:39 +0000 (GMT) Received: from d06av22.portsmouth.uk.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 0B4414C044; Wed, 19 Aug 2020 11:49:39 +0000 (GMT) Received: from marcibm (unknown [9.145.51.222]) by d06av22.portsmouth.uk.ibm.com (Postfix) with ESMTPS; Wed, 19 Aug 2020 11:49:38 +0000 (GMT) From: Marc Hartmayer To: David Hildenbrand , Marc Hartmayer , kvm@vger.kernel.org Cc: Thomas Huth , Janosch Frank , Cornelia Huck , Andrew Jones , Paolo Bonzini , Christian Borntraeger , linux-s390@vger.kernel.org Subject: Re: [kvm-unit-tests PATCH 4/4] s390x: add Protected VM support In-Reply-To: <6bd69c28-12df-f300-9cb2-9da1d80672f1@redhat.com> References: <20200818130424.20522-1-mhartmay@linux.ibm.com> <20200818130424.20522-5-mhartmay@linux.ibm.com> <6bd69c28-12df-f300-9cb2-9da1d80672f1@redhat.com> Date: Wed, 19 Aug 2020 13:49:36 +0200 Message-ID: <87o8n6svn3.fsf@linux.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-TM-AS-GCONF: 00 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.235,18.0.687 definitions=2020-08-19_04:2020-08-19,2020-08-19 signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 mlxscore=0 mlxlogscore=999 suspectscore=1 impostorscore=0 lowpriorityscore=0 spamscore=0 malwarescore=0 clxscore=1015 phishscore=0 priorityscore=1501 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2006250000 definitions=main-2008190101 Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org On Wed, Aug 19, 2020 at 12:42 PM +0200, David Hildenbrand wrote: > On 18.08.20 15:04, Marc Hartmayer wrote: >> Add support for Protected Virtual Machine (PVM) tests. For starting a >> PVM guest we must be able to generate a PVM image by using the >> `genprotimg` tool from the s390-tools collection. This requires the >> ability to pass a machine-specific host-key document, so the option >> `--host-key-document` is added to the configure script. >>=20 >> Signed-off-by: Marc Hartmayer >> --- >> configure | 9 +++++++++ >> s390x/Makefile | 17 +++++++++++++++-- >> s390x/selftest.parmfile | 1 + >> s390x/unittests.cfg | 1 + >> scripts/s390x/func.bash | 35 +++++++++++++++++++++++++++++++++++ >> 5 files changed, 61 insertions(+), 2 deletions(-) >> create mode 100644 s390x/selftest.parmfile >> create mode 100644 scripts/s390x/func.bash >>=20 >> diff --git a/configure b/configure >> index f9d030fd2f03..0e64af58b3c1 100755 >> --- a/configure >> +++ b/configure >> @@ -18,6 +18,7 @@ u32_long=3D >> vmm=3D"qemu" >> errata_force=3D0 >> erratatxt=3D"$srcdir/errata.txt" >> +host_key_document=3D >>=20=20 >> usage() { >> cat <<-EOF >> @@ -40,6 +41,9 @@ usage() { >> no environ is provided by the user (enabled= by default) >> --erratatxt=3DFILE specify a file to use instead of errata.t= xt. Use >> '--erratatxt=3D' to ensure no file is used. >> + --host-key-document=3DHOST_KEY_DOCUMENT >> + Specify the machine-specific host-key docum= ent for creating >> + a PVM image with 'genprotimg' (s390x only) >> EOF >> exit 1 >> } >> @@ -92,6 +96,9 @@ while [[ "$1" =3D -* ]]; do >> erratatxt=3D >> [ "$arg" ] && erratatxt=3D$(eval realpath "$arg") >> ;; >> + --host-key-document) >> + host_key_document=3D"$arg" >> + ;; >> --help) >> usage >> ;; >> @@ -205,6 +212,8 @@ PRETTY_PRINT_STACKS=3D$pretty_print_stacks >> ENVIRON_DEFAULT=3D$environ_default >> ERRATATXT=3D$erratatxt >> U32_LONG_FMT=3D$u32_long >> +GENPROTIMG=3D${GENPROTIMG-genprotimg} >> +HOST_KEY_DOCUMENT=3D$host_key_document >> EOF >>=20=20 >> cat < lib/config.h >> diff --git a/s390x/Makefile b/s390x/Makefile >> index 0f54bf43bfb7..cd4e270952ec 100644 >> --- a/s390x/Makefile >> +++ b/s390x/Makefile >> @@ -18,12 +18,19 @@ tests +=3D $(TEST_DIR)/skrf.elf >> tests +=3D $(TEST_DIR)/smp.elf >> tests +=3D $(TEST_DIR)/sclp.elf >> tests +=3D $(TEST_DIR)/css.elf >> -tests_binary =3D $(patsubst %.elf,%.bin,$(tests)) >>=20=20 >> -all: directories test_cases test_cases_binary >> +tests_binary =3D $(patsubst %.elf,%.bin,$(tests)) >> +ifneq ($(HOST_KEY_DOCUMENT),) >> +tests_pv_binary =3D $(patsubst %.bin,%.pv.bin,$(tests_binary)) >> +else >> +tests_pv_binary =3D >> +endif >> + >> +all: directories test_cases test_cases_binary test_cases_pv >>=20=20 >> test_cases: $(tests) >> test_cases_binary: $(tests_binary) >> +test_cases_pv: $(tests_pv_binary) >>=20=20 >> CFLAGS +=3D -std=3Dgnu99 >> CFLAGS +=3D -ffreestanding >> @@ -72,6 +79,12 @@ FLATLIBS =3D $(libcflat) >> %.bin: %.elf >> $(OBJCOPY) -O binary $< $@ >>=20=20 >> +%selftest.pv.bin: %selftest.bin $(HOST_KEY_DOCUMENT) $(patsubst %.pv.bi= n,%.parmfile,$@) >> + $(GENPROTIMG) --host-key-document $(HOST_KEY_DOCUMENT) --parmfile $(pa= tsubst %.pv.bin,%.parmfile,$@) --no-verify --image $< -o $@ >> + >> +%.pv.bin: %.bin $(HOST_KEY_DOCUMENT) >> + $(GENPROTIMG) --host-key-document $(HOST_KEY_DOCUMENT) --no-verify --i= mage $< -o $@ >> + >> arch_clean: asm_offsets_clean >> $(RM) $(TEST_DIR)/*.{o,elf,bin} $(TEST_DIR)/.*.d lib/s390x/.*.d >>=20=20 >> diff --git a/s390x/selftest.parmfile b/s390x/selftest.parmfile >> new file mode 100644 >> index 000000000000..5613931aa5c6 >> --- /dev/null >> +++ b/s390x/selftest.parmfile >> @@ -0,0 +1 @@ >> +test 123 >> \ No newline at end of file >> diff --git a/s390x/unittests.cfg b/s390x/unittests.cfg >> index 0f156afbe741..12f6fb613995 100644 >> --- a/s390x/unittests.cfg >> +++ b/s390x/unittests.cfg >> @@ -21,6 +21,7 @@ >> [selftest-setup] >> file =3D selftest.elf >> groups =3D selftest >> +# please keep the kernel cmdline in sync with $(TEST_DIR)/selftest.parm= file >> extra_params =3D -append 'test 123' > > This is ugly, can't we somehow create the parmfiles dynamically? An alternative would be to parse the unittests.cfg (and the extra_params) in the Makefile. Not sure it's worth the effort. > >>=20=20 >> [intercept] >> diff --git a/scripts/s390x/func.bash b/scripts/s390x/func.bash >> new file mode 100644 >> index 000000000000..b2d59d0d6f25 >> --- /dev/null >> +++ b/scripts/s390x/func.bash >> @@ -0,0 +1,35 @@ >> +# The file scripts/common.bash has to be the only file sourcing this >> +# arch helper file >> +source config.mak >> + >> +ARCH_CMD=3Darch_cmd_s390x >> + >> +function arch_cmd_s390x() >> +{ >> + local cmd=3D$1 >> + local testname=3D$2 >> + local groups=3D$3 >> + local smp=3D$4 >> + local kernel=3D$5 >> + local opts=3D$6 >> + local arch=3D$7 >> + local check=3D$8 >> + local accel=3D$9 >> + local timeout=3D${10} >> + >> + # run the normal test case >> + "$cmd" "${testname}" "$groups" "$smp" "$kernel" "$opts" "$arch" "$chec= k" "$accel" "$timeout" >> + > > Why ${testname} instead of $testname ? Probably to be in common with "${testname}_PV" :) I can change it here. > >> + # run PV test case >> + kernel=3D${kernel%.elf}.pv.bin >> + if [ ! -f "${kernel}" ]; then >> + if [ -z "${HOST_KEY_DOCUMENT}" ]; then >> + print_result 'SKIP' $testname '(no host-key document specified)' >> + return 2 >> + fi >> + >> + print_result 'SKIP' $testname '(PVM image was not created)' >> + return 2 >> + fi >> + "$cmd" "${testname}_PV" "$groups pv" "$smp" "$kernel" "$opts" "$arch" = "$check" "$accel" "$timeout" > > dito $testname_PV would refer to a non-existent variable. Thanks for the feedback! > >> +} >>=20 > > > --=20 > Thanks, > > David / dhildenb > --=20 Kind regards / Beste Gr=C3=BC=C3=9Fe Marc Hartmayer IBM Deutschland Research & Development GmbH Vorsitzender des Aufsichtsrats: Gregor Pillen=20 Gesch=C3=A4ftsf=C3=BChrung: Dirk Wittkopp Sitz der Gesellschaft: B=C3=B6blingen Registergericht: Amtsgericht Stuttgart, HRB 243294