From: Tom Lendacky <thomas.lenda...@amd.com>

The current code checks only for sme_active() when determining whether
to perform the encryption attribute change.  Include sev_active() in this
check so that memory attribute changes can occur under SME and SEV.

Signed-off-by: Tom Lendacky <thomas.lenda...@amd.com>
Signed-off-by: Brijesh Singh <brijesh.si...@amd.com>
Reviewed-by: Borislav Petkov <b...@suse.de>
Tested-by: Borislav Petkov <b...@suse.de>
Cc: Thomas Gleixner <t...@linutronix.de>
Cc: Ingo Molnar <mi...@redhat.com>
Cc: "H. Peter Anvin" <h...@zytor.com>
Cc: Borislav Petkov <b...@suse.de>
Cc: Andy Lutomirski <l...@kernel.org>
Cc: John Ogness <john.ogn...@linutronix.de>
Cc: Matt Fleming <m...@codeblueprint.co.uk>
Cc: Laura Abbott <labb...@redhat.com>
Cc: Dan Williams <dan.j.willi...@intel.com>
Cc: "Kirill A. Shutemov" <kirill.shute...@linux.intel.com>
Cc: linux-kernel@vger.kernel.org
Cc: x...@kernel.org
---
 arch/x86/mm/pageattr.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/mm/pageattr.c b/arch/x86/mm/pageattr.c
index dfb7d657cf43..3fe68483463c 100644
--- a/arch/x86/mm/pageattr.c
+++ b/arch/x86/mm/pageattr.c
@@ -1781,8 +1781,8 @@ static int __set_memory_enc_dec(unsigned long addr, int 
numpages, bool enc)
        unsigned long start;
        int ret;
 
-       /* Nothing to do if the SME is not active */
-       if (!sme_active())
+       /* Nothing to do if memory encryption is not active */
+       if (!mem_encrypt_active())
                return 0;
 
        /* Should not be working on unaligned addresses */
-- 
2.9.5

Reply via email to