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=-11.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable 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 0524DC4727F for ; Mon, 21 Sep 2020 22:31:13 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 7E33223A5B for ; Mon, 21 Sep 2020 22:31:12 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7E33223A5B 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 CB03790000B; Mon, 21 Sep 2020 18:31:11 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id C60B5900008; Mon, 21 Sep 2020 18:31:11 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id B9E4090000B; Mon, 21 Sep 2020 18:31:11 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0047.hostedemail.com [216.40.44.47]) by kanga.kvack.org (Postfix) with ESMTP id A0E7C900008 for ; Mon, 21 Sep 2020 18:31:11 -0400 (EDT) Received: from smtpin08.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id 5F392181AEF07 for ; Mon, 21 Sep 2020 22:31:11 +0000 (UTC) X-FDA: 77288515542.08.trail59_420900527148 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin08.hostedemail.com (Postfix) with ESMTP id 3B4DD1819E766 for ; Mon, 21 Sep 2020 22:31:11 +0000 (UTC) X-HE-Tag: trail59_420900527148 X-Filterd-Recvd-Size: 5788 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by imf09.hostedemail.com (Postfix) with ESMTP for ; Mon, 21 Sep 2020 22:31:09 +0000 (UTC) IronPort-SDR: rb4yygTTz4lR8NVh6nRlQdDvb3w54FcYQycQVYigoUI/Yey/zJHRfSfJ+YzpBtUFZrHUC56uBH /7J8uYf4iy6w== X-IronPort-AV: E=McAfee;i="6000,8403,9751"; a="160554933" X-IronPort-AV: E=Sophos;i="5.77,288,1596524400"; d="scan'208";a="160554933" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 21 Sep 2020 15:31:08 -0700 IronPort-SDR: 5R1SQoNcWLlKqNjxwtPEh8x6viFzeqqQMSnvUtxSvUDhsjtdZBpyPsKbBfiWbuG2Wbb8jGUb+n /N8qJ38F82EA== X-IronPort-AV: E=Sophos;i="5.77,288,1596524400"; d="scan'208";a="290176507" Received: from yyu32-mobl1.amr.corp.intel.com (HELO [10.212.102.78]) ([10.212.102.78]) by fmsmga007-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 21 Sep 2020 15:31:07 -0700 Subject: Re: [PATCH v12 1/8] x86/cet/ibt: Add Kconfig option for user-mode Indirect Branch Tracking To: Pavel Machek Cc: Randy Dunlap , 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 , Borislav Petkov , Cyrill Gorcunov , Dave Hansen , Eugene Syromiatnikov , Florian Weimer , "H.J. Lu" , Jann Horn , Jonathan Corbet , Kees Cook , Mike Kravetz , Nadav Amit , Oleg Nesterov , Peter Zijlstra , "Ravi V. Shankar" , Vedvyas Shanbhogue , Dave Martin , Weijiang Yang References: <20200918192312.25978-1-yu-cheng.yu@intel.com> <20200918192312.25978-2-yu-cheng.yu@intel.com> <20200918205933.GB4304@duo.ucw.cz> <019b5e45-b116-7f3d-f1f2-3680afbd676c@intel.com> <20200918214020.GF4304@duo.ucw.cz> From: "Yu, Yu-cheng" Message-ID: Date: Mon, 21 Sep 2020 15:30:46 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 MIME-Version: 1.0 In-Reply-To: <20200918214020.GF4304@duo.ucw.cz> Content-Type: text/plain; charset=windows-1252; 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 9/18/2020 2:40 PM, Pavel Machek wrote: > On Fri 2020-09-18 14:25:12, Yu, Yu-cheng wrote: >> On 9/18/2020 1:59 PM, Pavel Machek wrote: >>> On Fri 2020-09-18 13:24:13, Randy Dunlap wrote: >>>> Hi, >>>> >>>> If you do another version of this: >>>> >>>> On 9/18/20 12:23 PM, Yu-cheng Yu wrote: >>>>> Introduce Kconfig option X86_INTEL_BRANCH_TRACKING_USER. >>>>> >>>>> Indirect Branch Tracking (IBT) provides protection against CALL-/JMP- >>>>> oriented programming attacks. It is active when the kernel has this >>>>> feature enabled, and the processor and the application support it. >>>>> When this feature is enabled, legacy non-IBT applications continue to >>>>> work, but without IBT protection. >>>>> >>>>> Signed-off-by: Yu-cheng Yu >>>>> --- >>>>> v10: >>>>> - Change build-time CET check to config depends on. >>>>> >>>>> arch/x86/Kconfig | 16 ++++++++++++++++ >>>>> 1 file changed, 16 insertions(+) >>>>> >>>>> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig >>>>> index 6b6dad011763..b047e0a8d1c2 100644 >>>>> --- a/arch/x86/Kconfig >>>>> +++ b/arch/x86/Kconfig >>>>> @@ -1963,6 +1963,22 @@ config X86_INTEL_SHADOW_STACK_USER >>>>> If unsure, say y. >>>>> +config X86_INTEL_BRANCH_TRACKING_USER >>>>> + prompt "Intel Indirect Branch Tracking for user-mode" >>>>> + def_bool n >>>>> + depends on CPU_SUP_INTEL && X86_64 >>>>> + depends on $(cc-option,-fcf-protection) >>>>> + select X86_INTEL_CET >>>>> + help >>>>> + Indirect Branch Tracking (IBT) provides protection against >>>>> + CALL-/JMP-oriented programming attacks. It is active when >>>>> + the kernel has this feature enabled, and the processor and >>>>> + the application support it. When this feature is enabled, >>>>> + legacy non-IBT applications continue to work, but without >>>>> + IBT protection. >>>>> + >>>>> + If unsure, say y >>>> >>>> If unsure, say y. >>> >>> Actually, it would be "If unsure, say Y.", to be consistent with the >>> rest of the Kconfig. >>> >>> But I wonder if Yes by default is good idea. Only very new CPUs will >>> support this, right? Are they even available at the market? Should the >>> help text say "if your CPU is Whatever Lake or newer, ...." :-) ? >> >> I will revise the wording if there is another version. But a CET-capable >> kernel can run on legacy systems. We have been testing that combination. > > Yes, but enabling CET is unneccessary overhead on older systems. And > Kconfig is great place to explain that. Maybe I'll add: If the kernel is to be used only on older systems that do not support IBT, and the size of the binary is important, you can save 900 KB by disabling this feature. Otherwise, if unsure, say Y. How is that? Thanks, Yu-cheng