Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Received-SPF: Pass (protection.outlook.com: domain of amd.com designates
 165.204.84.17 as permitted sender) receiver=protection.outlook.com;
 client-ip=165.204.84.17; helo=SATLEXMB04.amd.com;
Date:   Fri, 22 Apr 2022 10:40:56 -0500
From:   Michael Roth <michael.roth@amd.com>
To:     Tom Lendacky <thomas.lendacky@amd.com>
CC:     <linux-kernel@vger.kernel.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        "Ingo Molnar" <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>,
        Dave Hansen <dave.hansen@linux.intel.com>, <x86@kernel.org>,
        "H . Peter Anvin" <hpa@zytor.com>,
        Kees Cook <keescook@chromium.org>,
        "Kirill A . Shutemov" <kirill.shutemov@linux.intel.com>,
        Tony Luck <tony.luck@intel.com>,
        "Nick Desaulniers" <ndesaulniers@google.com>,
        Kuppuswamy Sathyanarayanan 
        <sathyanarayanan.kuppuswamy@linux.intel.com>,
        Venu Busireddy <venu.busireddy@oracle.com>,
        Joerg Roedel <jroedel@suse.de>,
        Tianyu Lan <Tianyu.Lan@microsoft.com>,
        Brijesh Singh <brijesh.singh@amd.com>
Subject: Re: [PATCH v3 2/2] x86/sev: Get the AP jump table address from
 secrets page
Message-ID: <20220422154056.lhzmii34he6blvwv@amd.com>
References: <20220422135624.114172-1-michael.roth@amd.com>
 <20220422135624.114172-3-michael.roth@amd.com>
 <c3baaf5d-db99-f432-125f-9884b57b40ef@amd.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <c3baaf5d-db99-f432-125f-9884b57b40ef@amd.com>
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Apr 2022 15:41:18.0222
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: f8dacfca-4a51-4fb8-1428-08da24768ae2
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com]
X-MS-Exchange-CrossTenant-AuthSource: DM6NAM11FT038.eop-nam11.prod.protection.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR12MB1421
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE autolearn=no
	autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
	lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Apr 22, 2022 at 10:15:08AM -0500, Tom Lendacky wrote:
> On 4/22/22 08:56, Michael Roth wrote:
> > From: Brijesh Singh <brijesh.singh@amd.com>
> > 
> > The GHCB specification section 2.7 states that when SEV-SNP is enabled,
> > a hypervisor must provide the AP jump table physical address through
> 
> I missed this on the first version. It's not the hypervisor, but the guest
> BIOS that directly provides the AP jump table physical address, in our case
> OVMF sets the address in the SNP secrets page. This allows communication
> between UEFI/BIOS and OS without hypervisor involvement.

How about this wording for the commit message?

  The GHCB specification section 2.7 states that when SEV-SNP is enabled,
  a guest should not rely on the hypervisor to provide the address of the
  AP jump table. Instead, if a guest BIOS wants provide an AP jump table,
  it should record the address in the SNP secrets page so the guest
  operating system can obtain it directly from there.

  Fix this on the guest kernel side by having SNP guests use the AP jump
  table address published in the secrets page rather than issuing a GHCB
  request to get it.

> 
> > the SNP secrets pages.
> > 
> > Fixes: 0afb6b660a6b ("x86/sev: Use SEV-SNP AP creation to start secondary CPUs")
> > Signed-off-by: Brijesh Singh <brijesh.singh@amd.com>
> > [ mroth: improve error handling when ioremap()/memremap() return NULL ]
> > [ mroth: don't mix function calls with declarations ]
> > [ mroth: add missing __init ]
> > Signed-off-by: Michael Roth <michael.roth@amd.com>
> 
> With the commit message change:
> 
> Reviewed-by: Tom Lendacky <thomas.lendacky@amd.com>