From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by mx.groups.io with SMTP id smtpd.web09.23442.1633427722027546202 for ; Tue, 05 Oct 2021 02:55:22 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=e74iZhWT; spf=pass (domain: kernel.org, ip: 198.145.29.99, mailfrom: ardb@kernel.org) Received: by mail.kernel.org (Postfix) with ESMTPSA id 77F6C61526 for ; Tue, 5 Oct 2021 09:55:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1633427721; bh=rzJL5dRBLiQOsYJrRJ4JlEGJpaXVAsOLegE/MKzIbxw=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=e74iZhWTqRj5R1nQFFTAe4SA5nrV5YAExSDmpAGAnQbTA2Zv7dDgtsiSzUcscPwnR GWSdWH7P2O8Be6m4D+9l7+AIXgWMnB7I8pl6pC2/OUZEeIvUX6MBJa9OIljryV1QgC ASHRxnc4+e8lYJFyUNpA8ddfDsKNmpnwYFhfy/dKb5BBiBXreloJnBXBMawJHAKOCU ErxNFq4hggHgUcxKqvRkRoe2Di2sDZ2Q7agqjP8qc46f9yfdj0WQF8pZ8Qk22LFtt9 DDauLvM9sjNXWOkGK44/WcQzcJ718nMydAZePof75+qYTzMzSBdX7nXh//RitkefFo YBcOnARD+AdWg== Received: by mail-ot1-f50.google.com with SMTP id g62-20020a9d2dc4000000b0054752cfbc59so25181903otb.1 for ; Tue, 05 Oct 2021 02:55:21 -0700 (PDT) X-Gm-Message-State: AOAM533Lv27nigqfLmq7Tl3g2BZxrXnjdJE7Sc8SsY8oP1RFP+FG5x7S eQSbxEdBofNE9Y73/UqM6tMSSST9hDsckjnn7MY= X-Google-Smtp-Source: ABdhPJwwnhzOciRIwWJbel1RHluqazwXUXCjI81fXzroCVVJYmGRe7Qoiryi0QoNioUQiOvY1ERLRYBjC80CU17zWIk= X-Received: by 2002:a9d:7b48:: with SMTP id f8mr13789584oto.112.1633427720880; Tue, 05 Oct 2021 02:55:20 -0700 (PDT) MIME-Version: 1.0 References: <20210924114221.3132368-1-stefanb@linux.ibm.com> In-Reply-To: From: "Ard Biesheuvel" Date: Tue, 5 Oct 2021 11:55:10 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-devel] [PATCH v4 0/3] ArmVirtPkg: Disable the TPM 2 platform hierarchy To: edk2-devel-groups-io , Stefan Berger Cc: "Yao, Jiewen" , =?UTF-8?B?TWFyYy1BbmRyw6kgTHVyZWF1?= , Gerd Hoffmann , Ard Biesheuvel , Leif Lindholm , Sami Mujawar Content-Type: text/plain; charset="UTF-8" Merged as #2041 What is the status of the OVMF series with the same purpose? On Mon, 4 Oct 2021 at 21:56, Stefan Berger wrote: > > Yao, > > I think this series has the needed R-b's and should be commit-able. > > Cheers! > > Stefan > > > On 9/24/21 7:42 AM, Stefan Berger wrote: > > This series of patches disables the TPM 2 platform hierarchy. > > We just added the same functionality to the OvmfPkg. However, on x86, we > > could use the notification mechanism around gEfiDxeSmmReadyToLockProtocolGuid > > to indirectly invoke ConfigureTpmPlatformHierarchy(). Since ARM does not > > have an SMM mode this series now use direct invocation of this function > > at the same place in PlatformBootManagerBeforeConsole() as it is done > > on x86. > > > > Regards, > > Stefan > > > > v4: > > - Added Sami's R-b tag to 1/3 > > > > v3: > > - Addressed Ard's comments on 1/3 > > > > v2: > > - Move Null implementation to SecurityPkg > > - Added suggested texts to commit messages and added Sami's R-b tags > > > > Stefan Berger (3): > > ArmVirtPkg/TPM: Add a NULL implementation of TpmPlatformHierarchyLib > > ArmVirtPkg: Reference new TPM classes in the build system for > > compilation > > ArmVirtPkg: Disable the TPM2 platform hierarchy > > > > ArmVirtPkg/ArmVirtCloudHv.dsc | 1 + > > ArmVirtPkg/ArmVirtQemu.dsc | 2 ++ > > ArmVirtPkg/ArmVirtQemuKernel.dsc | 1 + > > ArmVirtPkg/ArmVirtXen.dsc | 1 + > > .../PlatformBootManagerLib/PlatformBm.c | 6 ++++ > > .../PlatformBootManagerLib.inf | 2 ++ > > .../PeiDxeTpmPlatformHierarchyLib.c | 22 +++++++++++++ > > .../PeiDxeTpmPlatformHierarchyLib.inf | 31 +++++++++++++++++++ > > SecurityPkg/SecurityPkg.dsc | 1 + > > 9 files changed, 67 insertions(+) > > create mode 100644 SecurityPkg/Library/PeiDxeTpmPlatformHierarchyLibNull/PeiDxeTpmPlatformHierarchyLib.c > > create mode 100644 SecurityPkg/Library/PeiDxeTpmPlatformHierarchyLibNull/PeiDxeTpmPlatformHierarchyLib.inf > > > > > > >