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=-10.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,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 10546C63697 for ; Sat, 28 Nov 2020 16:24:07 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 997C6246C5 for ; Sat, 28 Nov 2020 16:24:06 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 997C6246C5 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 332596B0068; Sat, 28 Nov 2020 11:24:06 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 2E4EA8D0001; Sat, 28 Nov 2020 11:24:06 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 1D1296B0075; Sat, 28 Nov 2020 11:24:06 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0072.hostedemail.com [216.40.44.72]) by kanga.kvack.org (Postfix) with ESMTP id 030B56B0068 for ; Sat, 28 Nov 2020 11:24:05 -0500 (EST) Received: from smtpin25.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id C3C8C8249980 for ; Sat, 28 Nov 2020 16:24:05 +0000 (UTC) X-FDA: 77534348850.25.back84_370d53327392 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin25.hostedemail.com (Postfix) with ESMTP id 9D4831804E3C1 for ; Sat, 28 Nov 2020 16:24:05 +0000 (UTC) X-HE-Tag: back84_370d53327392 X-Filterd-Recvd-Size: 5191 Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by imf29.hostedemail.com (Postfix) with ESMTP for ; Sat, 28 Nov 2020 16:24:03 +0000 (UTC) IronPort-SDR: wYl+x0MRV+914ZKZZpjh+ADNkIm8yT8xRF8GDcEV6lIaIuKNxkABi2XdJvN5Zo8jpJjW3UbH+O tUFAuR+GfnNg== X-IronPort-AV: E=McAfee;i="6000,8403,9819"; a="236613792" X-IronPort-AV: E=Sophos;i="5.78,377,1599548400"; d="scan'208";a="236613792" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Nov 2020 08:24:00 -0800 IronPort-SDR: FDIYYBr2hlKPLMbEhzG7QTIT1dc8+Q0vPjFxJnrDDSbotYpzRlODAOgHg1F1ZOFB7gou8+8goX dGWqsJsDMvIw== X-IronPort-AV: E=Sophos;i="5.78,377,1599548400"; d="scan'208";a="480006706" Received: from jckaplan-mobl1.amr.corp.intel.com (HELO [10.212.23.254]) ([10.212.23.254]) by orsmga004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Nov 2020 08:23:59 -0800 Subject: Re: [PATCH v15 05/26] x86/cet/shstk: Add Kconfig option for user-mode Shadow Stack To: Borislav Petkov Cc: x86@kernel.org, "H. Peter Anvin" , Thomas Gleixner , Ingo Molnar , linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, linux-mm@kvack.org, linux-arch@vger.kernel.org, linux-api@vger.kernel.org, Arnd Bergmann , Andy Lutomirski , Balbir Singh , Cyrill Gorcunov , Dave Hansen , Eugene Syromiatnikov , Florian Weimer , "H.J. Lu" , Jann Horn , Jonathan Corbet , Kees Cook , Mike Kravetz , Nadav Amit , Oleg Nesterov , Pavel Machek , Peter Zijlstra , Randy Dunlap , "Ravi V. Shankar" , Vedvyas Shanbhogue , Dave Martin , Weijiang Yang , Pengfei Xu References: <20201110162211.9207-1-yu-cheng.yu@intel.com> <20201110162211.9207-6-yu-cheng.yu@intel.com> <20201127171012.GD13163@zn.tnic> From: "Yu, Yu-cheng" Message-ID: <98e1b159-bf32-5c67-455b-f798023770ef@intel.com> Date: Sat, 28 Nov 2020 08:23:59 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.0) Gecko/20100101 Thunderbird/68.12.1 MIME-Version: 1.0 In-Reply-To: <20201127171012.GD13163@zn.tnic> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit 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: On 11/27/2020 9:10 AM, Borislav Petkov wrote: > On Tue, Nov 10, 2020 at 08:21:50AM -0800, Yu-cheng Yu wrote: >> +config X86_CET >> + def_bool n >> + >> +config ARCH_HAS_SHADOW_STACK >> + def_bool n >> + >> +config X86_SHADOW_STACK_USER > > Is X86_SHADOW_STACK_KERNEL coming too? > > Regardless, you can add it when it comes and you can use only X86_CET > for now and drop this one and simplify this pile of Kconfig symbols. We have X86_BRANCH_TRACKING_USER too. My thought was, X86_CET means any of kernel/user shadow stack/ibt. > >> + prompt "Intel Shadow Stacks for user-mode" >> + def_bool n >> + depends on CPU_SUP_INTEL && X86_64 >> + depends on AS_HAS_SHADOW_STACK >> + select ARCH_USES_HIGH_VMA_FLAGS >> + select X86_CET >> + select ARCH_HAS_SHADOW_STACK >> + help >> + Shadow Stacks provides protection against program stack >> + corruption. It's a hardware feature. This only matters >> + if you have the right hardware. It's a security hardening >> + feature and apps must be enabled to use it. You get no >> + protection "for free" on old userspace. The hardware can >> + support user and kernel, but this option is for user space >> + only. >> + Support for this feature is only known to be present on >> + processors released in 2020 or later. CET features are also >> + known to increase kernel text size by 3.7 KB. > > This help text needs some rewriting. You can find an inspiration about > more adequate style in that same Kconfig file. > I will work on it. >> + >> + If unsure, say N. >> + >> config EFI >> bool "EFI runtime service support" >> depends on ACPI >> diff --git a/scripts/as-x86_64-has-shadow-stack.sh b/scripts/as-x86_64-has-shadow-stack.sh >> new file mode 100755 >> index 000000000000..fac1d363a1b8 >> --- /dev/null >> +++ b/scripts/as-x86_64-has-shadow-stack.sh >> @@ -0,0 +1,4 @@ >> +#!/bin/sh >> +# SPDX-License-Identifier: GPL-2.0 >> + >> +echo "wrussq %rax, (%rbx)" | $* -x assembler -c - > > 2> /dev/null > > otherwise you get > > {standard input}: Assembler messages: > {standard input}:1: Error: no such instruction: `wrussq %rax,(%rbx) > > on non-enlightened toolchains during build. > Yes, I will fix this in the next revision. Yu-cheng > Thx. >