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=-6.5 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, 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 92957C433DF for ; Mon, 8 Jun 2020 08:35:33 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 5B5EF206A4 for ; Mon, 8 Jun 2020 08:35:33 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="dwdFLGzn" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5B5EF206A4 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([::1]:55916 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jiDFk-0006Wo-DG for qemu-devel@archiver.kernel.org; Mon, 08 Jun 2020 04:35:32 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:42706) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jiDEi-0005Wn-Ef for qemu-devel@nongnu.org; Mon, 08 Jun 2020 04:34:28 -0400 Received: from us-smtp-1.mimecast.com ([205.139.110.61]:58139 helo=us-smtp-delivery-1.mimecast.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1jiDEh-0008G2-2a for qemu-devel@nongnu.org; Mon, 08 Jun 2020 04:34:28 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1591605265; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=J1SSBrHC0SLdApii9ByG5DxNkOzsiUpxx5+7Vxh/4Tg=; b=dwdFLGznw8eGgtFFfwtaxo1s5ldA4ERR5AFbDoGq4ooBJ3lZPpIV7vzeFtoCUMmhXM/6VO tm1ttJby0jLhh1ouLtf7Gk1Dwox/QPOpi5TrNGmpjCKldn+QNsHROvH8lgOJUPmWh6HQ/1 u5CD/JKx2K3Hp+RF+hrjbdNsDz+jKAc= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-324-NpLTMZ1XO_y_ndFBxWvESQ-1; Mon, 08 Jun 2020 04:34:21 -0400 X-MC-Unique: NpLTMZ1XO_y_ndFBxWvESQ-1 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 293ED100CCC1; Mon, 8 Jun 2020 08:34:20 +0000 (UTC) Received: from localhost (unknown [10.40.208.64]) by smtp.corp.redhat.com (Postfix) with ESMTP id 0A60B610F2; Mon, 8 Jun 2020 08:34:08 +0000 (UTC) Date: Mon, 8 Jun 2020 10:34:03 +0200 From: Igor Mammedov To: Auger Eric Subject: Re: [RFC 4/6] tests: tpm-emu: Remove assert on TPM2_ST_NO_SESSIONS Message-ID: <20200608103403.00dcfacc@redhat.com> In-Reply-To: References: <20200601102113.1207-1-eric.auger@redhat.com> <20200601102113.1207-5-eric.auger@redhat.com> <50a54958-e9e0-c95f-3893-f7f790186e0e@linux.ibm.com> <80ce5833-90ee-cbc5-9822-cca1fabc33e6@redhat.com> <228f7928-9ffa-498d-0158-aa08e7c3ca55@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Received-SPF: pass client-ip=205.139.110.61; envelope-from=imammedo@redhat.com; helo=us-smtp-delivery-1.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/06/08 01:20:45 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001 autolearn=_AUTOLEARN X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: peter.maydell@linaro.org, drjones@redhat.com, mst@redhat.com, philmd@redhat.com, qemu-devel@nongnu.org, shannon.zhaosl@gmail.com, qemu-arm@nongnu.org, marcandre.lureau@redhat.com, Stefan Berger , lersek@redhat.com, ardb@kernel.org, eric.auger.pro@gmail.com Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On Fri, 5 Jun 2020 17:47:08 +0200 Auger Eric wrote: > Hi Stefan, >=20 > On 6/5/20 5:25 PM, Stefan Berger wrote: > > On 6/5/20 5:35 AM, Auger Eric wrote: =20 > >> Hi Stefan, > >> > >> On 6/2/20 6:17 PM, Stefan Berger wrote: =20 > >>> On 6/2/20 12:13 PM, Auger Eric wrote: =20 > >>>> Hi Stefan, > >>>> > >>>> On 6/2/20 3:39 PM, Stefan Berger wrote: =20 > >>>>> On 6/1/20 6:21 AM, Eric Auger wrote: =20 > >>>>>> While writing tests for checking the content of TPM2 and DSDT > >>>>>> along with TPM-TIS instantiation I attempted to reuse the > >>>>>> framework used for TPM-TIS tests. However While dumping the > >>>>>> ACPI tables I get an assert on TPM2_ST_NO_SESSIONS. My assumption > >>>>>> is maybe the other tests did not execute long enough to encounter > >>>>>> this. So I tentatively propose to remove the assert as it > >>>>>> does not seem to break other tests and enable the new ones. > >>>>>> > >>>>>> Signed-off-by: Eric Auger > >>>>>> --- > >>>>>> =C2=A0=C2=A0=C2=A0 tests/qtest/tpm-emu.c | 1 - > >>>>>> =C2=A0=C2=A0=C2=A0 1 file changed, 1 deletion(-) > >>>>>> > >>>>>> diff --git a/tests/qtest/tpm-emu.c b/tests/qtest/tpm-emu.c > >>>>>> index c43ac4aef8..298d0eec74 100644 > >>>>>> --- a/tests/qtest/tpm-emu.c > >>>>>> +++ b/tests/qtest/tpm-emu.c > >>>>>> @@ -49,7 +49,6 @@ static void *tpm_emu_tpm_thread(void *data) > >>>>>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= s->tpm_msg->tag =3D be16_to_cpu(s->tpm_msg->tag); > >>>>>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= s->tpm_msg->len =3D be32_to_cpu(s->tpm_msg->len); > >>>>>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= g_assert_cmpint(s->tpm_msg->len, >=3D, minhlen); > >>>>>> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 g_assert_cmpint(s->tpm= _msg->tag, =3D=3D, TPM2_ST_NO_SESSIONS); =20 > >>>>> You should not have to remove this. The tests are skipped if swtpm > >>>>> does > >>>>> not support TPM 2 via --tpm2 option. This would be a very old swtpm > >>>>> version, though. So, all tests are run with --tpm2 option and any > >>>>> response received from the TPM would be a TPM 2 response that shoul= d > >>>>> have TPM2_ST_NO_SESSIONS as the tag. I'd be curious what other > >>>>> value you > >>>>> are seeing there. =20 > >>>> If I revert this patch I am getting TPM2_ST_SESSIONS on my end. =20 > >>> Is firmware/BIOS active? There's no TPM2_ST_SESSIONS coming out of QE= MU. =20 > >> So it looks SeaBIOS is in use (bios-256k.bin loaded). > >> > >> I can see MMIO accesses to the TPM and the following commands are > >> observable: > >> tpm_emu_tpm_thread code=3D0x181 tag=3D0x8001 len=3D0xa > >> tpm_emu_tpm_thread code=3D0x144 tag=3D0x8001 len=3D0xc > >> tpm_emu_tpm_thread code=3D0x121 tag=3D0x8002 len=3D0x20 > >> This last one causes the assert (TPM2_CC_HierarchyControl) > >> > >> I checked in Seabios and effectively tpm20_hierarchycontrol() tags the > >> TPM2_CC_HierarchyControl command with TPM2_ST_SESSIONS > >> > >> Due to our emulation, maybe tpm_set_failure() gets called, inducing > >> tpm20_hierarchycontrol() call. > >> > >> That being said, what do you recommend? Remove the assert, improve the > >> emulation, other? =20 > >=20 > > So this is an ACPI test. What role does the firmware play for success o= f > > the test? If the test relies on the firmware showing some sort of > > expected result, then I would recommend only running this test with an > > attached swtpm, like we run some other tests. If we don't need the > > firmware to succeed then I would just get rid of the assert. Probably n= o > > other test we have implemented so far was running the firmware... =20 > FWIU The goal of this test is to compare the acpi tables generated by > qemu against reference ones. I dont think we expect from the FW any > specific result but I would prefer Igor or Michael to confirm. Firmware is needed to fetch tables from QEMU and place them in guest RAM, it will also patch cross table pointers accordingly. So bios-tables-test checks both QEMU and firmware at the same time, and reference tables are in form guest OS will see them. >=20 > In that case, removing the assert() allows to compare the specific DSDT > and TPM2 tables and that's our expectation here I think. >=20 > Thanks >=20 > Eric > >=20 > >=20 > > =C2=A0=C2=A0 Stefan > >=20 > > =20 > >> > >> Thank you in advance > >> > >> Best Regards > >> > >> Eric > >> =20 > >>> =C2=A0=C2=A0=C2=A0 Stefan > >>> > >>> > >>> =20 > >=20 > > =20