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=-8.1 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,USER_AGENT_SANE_1 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 05091C433DF for ; Mon, 8 Jun 2020 09:12:29 +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 C0BAF206C3 for ; Mon, 8 Jun 2020 09:12:28 +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="RVNYb4wn" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C0BAF206C3 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]:33804 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jiDpT-0006vx-VZ for qemu-devel@archiver.kernel.org; Mon, 08 Jun 2020 05:12:28 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:46594) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jiDoW-0005uS-Vg for qemu-devel@nongnu.org; Mon, 08 Jun 2020 05:11:28 -0400 Received: from us-smtp-delivery-1.mimecast.com ([205.139.110.120]:29502 helo=us-smtp-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 1jiDoV-0005ze-4E for qemu-devel@nongnu.org; Mon, 08 Jun 2020 05:11:28 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1591607485; 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=pxeQKS5SoNo4nuLxe3e44s1xCvMRgSxI6cSBVEGz5lw=; b=RVNYb4wn+CIKlMICPfbp7oV7OiLxbc/nGHOG33QK1OmygxOH/83fBYUND7osQB9YuxdwiM lEKT6dTzyNj4rCHL2rrtU8plL4OG242zBYBMAXY+0fjbCRwGRBDZCDAFFTRLAgdvxl2h9d VAlMQ3c/HBtmUEHHhN8wHBxax8X6uys= 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-2-RXK7qIc4P16RbHiYvy6-lQ-1; Mon, 08 Jun 2020 05:11:21 -0400 X-MC-Unique: RXK7qIc4P16RbHiYvy6-lQ-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id C0914BFC6; Mon, 8 Jun 2020 09:11:19 +0000 (UTC) Received: from [10.36.112.85] (ovpn-112-85.ams2.redhat.com [10.36.112.85]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 14F9A891E0; Mon, 8 Jun 2020 09:11:10 +0000 (UTC) Subject: Re: [RFC 4/6] tests: tpm-emu: Remove assert on TPM2_ST_NO_SESSIONS To: Igor Mammedov 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> <20200608103403.00dcfacc@redhat.com> From: Auger Eric Message-ID: Date: Mon, 8 Jun 2020 11:11:09 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 MIME-Version: 1.0 In-Reply-To: <20200608103403.00dcfacc@redhat.com> Content-Language: en-US X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Received-SPF: pass client-ip=205.139.110.120; envelope-from=eric.auger@redhat.com; helo=us-smtp-1.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/06/08 01:05:50 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, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, 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, lersek@redhat.com, qemu-devel@nongnu.org, shannon.zhaosl@gmail.com, qemu-arm@nongnu.org, marcandre.lureau@redhat.com, eric.auger.pro@gmail.com, philmd@redhat.com, ardb@kernel.org, Stefan Berger Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" Hi Igor, On 6/8/20 10:34 AM, Igor Mammedov wrote: > On Fri, 5 Jun 2020 17:47:08 +0200 > Auger Eric wrote: > >> Hi Stefan, >> >> On 6/5/20 5:25 PM, Stefan Berger wrote: >>> On 6/5/20 5:35 AM, Auger Eric wrote: >>>> Hi Stefan, >>>> >>>> On 6/2/20 6:17 PM, Stefan Berger wrote: >>>>> On 6/2/20 12:13 PM, Auger Eric wrote: >>>>>> Hi Stefan, >>>>>> >>>>>> On 6/2/20 3:39 PM, Stefan Berger wrote: >>>>>>> On 6/1/20 6:21 AM, Eric Auger wrote: >>>>>>>> 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 >>>>>>>> --- >>>>>>>>     tests/qtest/tpm-emu.c | 1 - >>>>>>>>     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) >>>>>>>>             s->tpm_msg->tag = be16_to_cpu(s->tpm_msg->tag); >>>>>>>>             s->tpm_msg->len = be32_to_cpu(s->tpm_msg->len); >>>>>>>>             g_assert_cmpint(s->tpm_msg->len, >=, minhlen); >>>>>>>> -        g_assert_cmpint(s->tpm_msg->tag, ==, TPM2_ST_NO_SESSIONS); >>>>>>> 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 should >>>>>>> have TPM2_ST_NO_SESSIONS as the tag. I'd be curious what other >>>>>>> value you >>>>>>> are seeing there. >>>>>> If I revert this patch I am getting TPM2_ST_SESSIONS on my end. >>>>> Is firmware/BIOS active? There's no TPM2_ST_SESSIONS coming out of QEMU. >>>> 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=0x181 tag=0x8001 len=0xa >>>> tpm_emu_tpm_thread code=0x144 tag=0x8001 len=0xc >>>> tpm_emu_tpm_thread code=0x121 tag=0x8002 len=0x20 >>>> 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? >>> >>> So this is an ACPI test. What role does the firmware play for success of >>> 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 no >>> other test we have implemented so far was running the firmware... >> 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. Thank you for your input. I guess Stefan's concern was: is that an issue if the FW accesses to the TPM silently fail at some (assumed late) point. I guess the job we expect from the FW, ie. copy the ACPI tables and patch tables, is done prior to that failure, hence the test success. So this functional failure should not be an issue for bios-table-test, right? Thanks Eric > >> >> In that case, removing the assert() allows to compare the specific DSDT >> and TPM2 tables and that's our expectation here I think. >> >> Thanks >> >> Eric >>> >>> >>>    Stefan >>> >>> >>>> >>>> Thank you in advance >>>> >>>> Best Regards >>>> >>>> Eric >>>> >>>>>     Stefan >>>>> >>>>> >>>>> >>> >>> > >