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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id DB334C4332F for ; Tue, 7 Nov 2023 20:28:53 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 724DE8D0054; Tue, 7 Nov 2023 15:28:53 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 6D4758D0001; Tue, 7 Nov 2023 15:28:53 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5C3158D0054; Tue, 7 Nov 2023 15:28:53 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 4DFC38D0001 for ; Tue, 7 Nov 2023 15:28:53 -0500 (EST) Received: from smtpin29.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id 2A0E2B5CF3 for ; Tue, 7 Nov 2023 20:28:53 +0000 (UTC) X-FDA: 81432296946.29.D19A611 Received: from mail.alien8.de (mail.alien8.de [65.109.113.108]) by imf09.hostedemail.com (Postfix) with ESMTP id 0C0EE140012 for ; Tue, 7 Nov 2023 20:28:49 +0000 (UTC) Authentication-Results: imf09.hostedemail.com; dkim=pass header.d=alien8.de header.s=alien8 header.b=UF8gBcsq; dmarc=pass (policy=none) header.from=alien8.de; spf=pass (imf09.hostedemail.com: domain of bp@alien8.de designates 65.109.113.108 as permitted sender) smtp.mailfrom=bp@alien8.de ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1699388930; h=from:from:sender: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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=D1AOnodvSAeHKdK51GuEmjvSdMbLjI6XL/dvvveTUuc=; b=SCHTTgnmYeDd5JAWNPgaMkdSs1fOb1IlYHIgCeKOnklOFazibf45zbFwYIgdcnDfCruWUO rAHfV94reM142QXd28bpwGDP7tbMULsL2pv6cQh8eBTDyPFfy4gJlRGU1IDnl38Zn9ISd2 6m2FoNt3mVVJsJEa90dex3OjJAnc2VM= ARC-Authentication-Results: i=1; imf09.hostedemail.com; dkim=pass header.d=alien8.de header.s=alien8 header.b=UF8gBcsq; dmarc=pass (policy=none) header.from=alien8.de; spf=pass (imf09.hostedemail.com: domain of bp@alien8.de designates 65.109.113.108 as permitted sender) smtp.mailfrom=bp@alien8.de ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1699388930; a=rsa-sha256; cv=none; b=POnSu0ISX1rzDvLtMGf9UHzZ6kcVQ2hzcl6wlcND59KJPHtx+B6y7qYRdRnqLPeugaV6iV EeRV/ArnoCTzMVGINjPVApoUIzyADOHTzbP3Rx0tSMnMwZ7UkVDsUrZDZPflGA6VQ2a/uk xFgO4uOqipSRxOnJOdKeekYqWeXqC4U= Received: from localhost (localhost.localdomain [127.0.0.1]) by mail.alien8.de (SuperMail on ZX Spectrum 128k) with ESMTP id 0A09740E0192; Tue, 7 Nov 2023 20:28:46 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at mail.alien8.de Received: from mail.alien8.de ([127.0.0.1]) by localhost (mail.alien8.de [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id UaF3s4xZRiN9; Tue, 7 Nov 2023 20:28:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=alien8.de; s=alien8; t=1699388923; bh=D1AOnodvSAeHKdK51GuEmjvSdMbLjI6XL/dvvveTUuc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=UF8gBcsqmQg7yr0n+/mI3nYpyr0kPLOu1iGxkvrKz8xCx8puKxZfvs2hkj4DPvwST Y2sbrNyYowiT2thhcpZMMgO9QPGF65dWCoZ2yqfjI1dFz1R0OHiOSCv4eJRombMmUm YGb6L3QVAW4OzJDbP0H4jrYmfzts8R34+u3hSh/dBomrsZL2h+ViOGdCNDIxzkAV/r 8TTwo+TGmxAdtisAd4v6QOsJSYp/OCS/8cCCHIcCxc40q2e6pkl8JBQFMRqMvYLZyi xDBkBIuR1t/6RX+QNJh0pDFjmT4rnkpex/HeXO/qly0M2xa7yc8i/3SLM4p+gazB2A /9jTJ85EsIJSP0COuBp4hhO6ilsfYihP9gfgyZNFYQ50Lj7YY3PMqSUiy0HSxG1uZf lc1O+T9WXKMK1N3WIfmUh/18NYkZSiW2ho1EZT6nA7GQps01r6EgW7UWuIVmlnqy3U 7305R90fOPj6+JfVUJu/yOGPzfSHf6YhYA7onfM86bjI5G7M38HZMTaccXkHvYhZTQ OOISofDfzua+NUwykAoQ11MNYMGuWqZw5yHh6gG7OKXEyLlJ9rbXL0I9ADVTKW/bqj EtmuUjN4550UzGt8d7oLIt8RAhwdn1kh2aaKgs2LW8yLp2OuQvABTD7pGeHTazZtt6 ATn+Zb67pytmJ8V4ka6ju3O4= Received: from zn.tnic (pd95304da.dip0.t-ipconnect.de [217.83.4.218]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature ECDSA (P-256) server-digest SHA256) (No client certificate requested) by mail.alien8.de (SuperMail on ZX Spectrum 128k) with ESMTPSA id 8645940E0176; Tue, 7 Nov 2023 20:28:02 +0000 (UTC) Date: Tue, 7 Nov 2023 21:27:57 +0100 From: Borislav Petkov To: "Kalra, Ashish" Cc: Michael Roth , kvm@vger.kernel.org, linux-coco@lists.linux.dev, linux-mm@kvack.org, linux-crypto@vger.kernel.org, x86@kernel.org, linux-kernel@vger.kernel.org, tglx@linutronix.de, mingo@redhat.com, jroedel@suse.de, thomas.lendacky@amd.com, hpa@zytor.com, ardb@kernel.org, pbonzini@redhat.com, seanjc@google.com, vkuznets@redhat.com, jmattson@google.com, luto@kernel.org, dave.hansen@linux.intel.com, slp@redhat.com, pgonda@google.com, peterz@infradead.org, srinivas.pandruvada@linux.intel.com, rientjes@google.com, dovmurik@linux.ibm.com, tobin@ibm.com, vbabka@suse.cz, kirill@shutemov.name, ak@linux.intel.com, tony.luck@intel.com, marcorr@google.com, sathyanarayanan.kuppuswamy@linux.intel.com, alpergun@google.com, jarkko@kernel.org, nikunj.dadhania@amd.com, pankaj.gupta@amd.com, liam.merwick@oracle.com, zhi.a.wang@intel.com, Brijesh Singh Subject: Re: [PATCH v10 06/50] x86/sev: Add the host SEV-SNP initialization support Message-ID: <20231107202757.GEZUqdzYyzVBHTBhZX@fat_crate.local> References: <20231016132819.1002933-1-michael.roth@amd.com> <20231016132819.1002933-7-michael.roth@amd.com> <20231107163142.GAZUpmbt/i3himIf+E@fat_crate.local> <4a2016d6-dc1f-ff68-9827-0b72b7c8eac2@amd.com> <20231107191931.GCZUqNwxP8JcSbjZ0/@fat_crate.local> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20231107191931.GCZUqNwxP8JcSbjZ0/@fat_crate.local> X-Rspamd-Queue-Id: 0C0EE140012 X-Rspam-User: X-Rspamd-Server: rspam02 X-Stat-Signature: 96c1kpmbsm5iqntkep5ad3shqyu5htfj X-HE-Tag: 1699388929-110871 X-HE-Meta: U2FsdGVkX1+jT8ogL/ig2MSBn9MJbhd5zzQPd+EvJpJlfqbUe6hi1/+91CtJlBUsGaO6wvsJ373VGfgDfRYRPZ1fHcWcyQYAm8+6ZI60sx8lUIoO0dpdcT136xGW3MEU5G4Y/Lqzv1n5dk1Vo2utGmFtx9RgT545A/bna5V22+fIq7qBVCvk4SGD/Hc2+4lzKVZy8OLAiZLe9g6IpobONTbKilIrL4VQYT0AnQDBtxbgtFft92+8C/5YO78dl1mqfQ5GYJlXSDag9ShyNhcpSx2zM/2GgTnNijOYiqrG/Ag7C61G4Q2OqMR/bhCuhhWA4jaV+/fx9UdzrruTzTfOvN11bhuJsVZGPQ/F2c9tcNV70zJpi8fhLnvTnpz9RD3C4CbT9eFahtROXp3Y7/LcBG/Iho7RRXGg1TtXY2UMDT4IR4xKexTOCAZZXi32q1oT0xBGxwolwfhjxmVFgj4L5hFrjkFmPzSwFPbmh0xASH70h9gRprLgWnToOPrwGBij6cTxVKgmm3YAf80mQZnOFdl8gluyI48FLtqpN7mT+K3LHgvTKev2IJe2IEOE863MmtsPl7e65hQ1c9SnMCr/HEoIsYWvQQvOA5Wgko0vVXMpKd7TW1wosaMdgQmKphthCayqpp6meTWTdjpUhLhBF5Ow1XEeeN089s/9rZRPxYDfkK382YEj83YA2Qc3k8VKZhg5WvqV0qiCMNmo76xhEsmn3U1h+O8ZpdUUcBtjLFUyoegbkvec8zPbGutyda2SMSC2fmrkvri1Nu52QxkyPfFYgZjoSDRy8yt4WGSfdw6JoO/EJpj4Fl128jL2A40Txw6VlWFBugGcOKeqcyc0wEtgt902GkN/vfHqHbZGYzjfBjeyNg9gjCKBjlqr8hoGgx3Stfg6dc+luhPdaEmxM7Pog9CXPnUGMNzU2jUqy4Lx5j1hJ65uoWNo12WCSbfJxVQZhyyEyixp09u3UvC K81GKjzX bw0zMjlWOdVAXh+dnTfoAj1IMONVqEzJsS04de2JkREAyAEtPwlS3Yi3yoSpmV5k0NPNInPj8ALmHJt/rfEW7WxcUEnseJc7Arsccz/J03JQVom8MLCWSdd0tCuggxb//zlmGu90uKJbTbSb1WwhM62Vi6BFynFZnYWUIraQHajAuN71EpPwLwb8Y16gNZLRMWa8+5/ieP5hd30M= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Tue, Nov 07, 2023 at 08:19:31PM +0100, Borislav Petkov wrote: > Arch code does not call drivers - arch code sets up the arch and > provides facilities which the drivers use. IOW (just an example diff): diff --git a/drivers/iommu/amd/init.c b/drivers/iommu/amd/init.c index 1c9924de607a..00cdbc844961 100644 --- a/drivers/iommu/amd/init.c +++ b/drivers/iommu/amd/init.c @@ -3290,6 +3290,7 @@ static int __init state_next(void) break; case IOMMU_ENABLED: register_syscore_ops(&amd_iommu_syscore_ops); + amd_iommu_snp_enable(); ret = amd_iommu_init_pci(); init_state = ret ? IOMMU_INIT_ERROR : IOMMU_PCI_INIT; break; @@ -3814,16 +3815,6 @@ int amd_iommu_snp_enable(void) return -EINVAL; } - /* - * Prevent enabling SNP after IOMMU_ENABLED state because this process - * affect how IOMMU driver sets up data structures and configures - * IOMMU hardware. - */ - if (init_state > IOMMU_ENABLED) { - pr_err("SNP: Too late to enable SNP for IOMMU.\n"); - return -EINVAL; - } - amd_iommu_snp_en = check_feature_on_all_iommus(FEATURE_SNP); if (!amd_iommu_snp_en) return -EINVAL; and now you only need to line up snp_rmptable_init() after IOMMU init instead of having it be a fs_initcall which happens right after pci_subsys_init() so that PCI is there but at the right time when iommu init state is at IOMMU_ENABLED but no later because then it is too late. And there you need to test amd_iommu_snp_en which is already exported anyway. Ok? -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette