Message ID | 20230224070145.3572-1-xin3.li@intel.com |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:5915:0:0:0:0:0 with SMTP id v21csp756013wrd; Thu, 23 Feb 2023 23:28:47 -0800 (PST) X-Google-Smtp-Source: AK7set9AjL4ckB/SmRhNbpplQbVMyOaVkhQclVZsV8fwoWw0I8VJ4+9XMwbaFf1pt79ZxfC3FQZV X-Received: by 2002:a17:907:212d:b0:882:2027:c8e2 with SMTP id qo13-20020a170907212d00b008822027c8e2mr19941785ejb.56.1677223727013; Thu, 23 Feb 2023 23:28:47 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1677223726; cv=none; d=google.com; s=arc-20160816; b=TNRkAcLyrMILs3bs2c8x5U6y5NvvnSUvxUBnqAN3pOhe0CnSydMsYOFij0OgpIVryH lG5hnfNxMAdOM6wf7DZehhKP8n4rr74I+doaBfGipyAVbgSH8Dc3xLdfH6MRe82yqC3P JSqKc241cYgjguomYC77c8MKW7sCXqgaPi6NdLmB8Pn3e2E5rj9CVkTB4W7RoHLBAHde lIYoBCOzMSmpGK1GJZiKlRHKz5vwrBmE5vt2kuPI/k+pd3Z210/78NmG6mTFY2yJwRuw 2q5f2A4VZTgJTOsr0ULbRyeIWFT9NRova9J0MFPEYneVk44lmRUVQlnsDF1OUIeReyaz mUSQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=EpLKoRKDnzmLTYycJ2285Vj0U8evj3TOXyMYVKeCjoA=; b=AiuVQZ/+QVYdx8LLJMhOy745vJ0fBN8cjDWZ6dF+Ok+5EpEfsFEN4paLzm9nG9gm3j uWXPPlazGW4URjwOSNxVt6ME6Gn1Q2zjUrv4iD6pd0hskQlejcG2SVVS7brxlTLhenLH Mh/EVV62n4uCIsw5HpBGJB8/W4A0VtqvprW2DnkmPsClYF1nZunSh6LvS89qD6wZ4WDP uuR5PMQ0sw5tSWPRA/ziMCSmVU1rnSo9Wv1N9DEf3NUGGNwA/LJgVAIAS35DTC8diTT7 64QhiCEFxiXfd20q+ZRY313jns96HfjzykxtiM+9OFZ1FwM/cyBs/1bRG/nQafGPVWRx WFwQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=FEWVA7bL; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id fd7-20020a1709072a0700b008d4fbb9f30csi13336716ejc.899.2023.02.23.23.28.22; Thu, 23 Feb 2023 23:28:46 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=FEWVA7bL; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229545AbjBXH15 (ORCPT <rfc822;jeff.pang.chn@gmail.com> + 99 others); Fri, 24 Feb 2023 02:27:57 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40108 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229468AbjBXH1y (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Fri, 24 Feb 2023 02:27:54 -0500 Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0B2CF206BE; Thu, 23 Feb 2023 23:27:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1677223645; x=1708759645; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=nGPqQbVN5Kz2WIfEucUO7pJYlvFGLm+jjwHGwYwktKA=; b=FEWVA7bLFVt5O4D/MkreZ+m61Ltuo8R5WEgTL3WqjWuHuAZR/UpmCgaf oh0pQBIDXBQfUQI/nBxtTdDDgnYgk/TM220TK51hFAgfmShkr+1NiKSar 8B9d9DezXWlvtsJC/ljsIFS5FXnIZm4P+ODBNDQlQQ6VhhP0eb69ku5qu IA6PEcipsG+hrhWF9jhV8/EPQ6NCyQTbZgzcPgmx24EnTwaVhQVsJjRfS daslIIG/gA3RxV5pEPqodQwDuzOTg9Fej015O1jgFnAqy+nlbh3DZGXuT dRXQsMvutW88V3EEcBxAmSozEzTdQ83zGoGY8jVKZOy3jfCxFrQhsNleu Q==; X-IronPort-AV: E=McAfee;i="6500,9779,10630"; a="334835903" X-IronPort-AV: E=Sophos;i="5.97,324,1669104000"; d="scan'208";a="334835903" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Feb 2023 23:27:20 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10630"; a="815639185" X-IronPort-AV: E=Sophos;i="5.97,324,1669104000"; d="scan'208";a="815639185" Received: from unknown (HELO fred..) ([172.25.112.68]) by fmsmga001.fm.intel.com with ESMTP; 23 Feb 2023 23:27:19 -0800 From: Xin Li <xin3.li@intel.com> To: linux-kernel@vger.kernel.org, x86@kernel.org, kvm@vger.kernel.org Cc: tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, dave.hansen@linux.intel.com, hpa@zytor.com, peterz@infradead.org, andrew.cooper3@citrix.com, seanjc@google.com, pbonzini@redhat.com, ravi.v.shankar@intel.com Subject: [RFC PATCH v3 00/32] x86: enable FRED for x86-64 Date: Thu, 23 Feb 2023 23:01:13 -0800 Message-Id: <20230224070145.3572-1-xin3.li@intel.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED, RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE autolearn=ham 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 X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1758696546500503682?= X-GMAIL-MSGID: =?utf-8?q?1758696546500503682?= |
Series |
x86: enable FRED for x86-64
|
|
Message
Li, Xin3
Feb. 24, 2023, 7:01 a.m. UTC
This patch set enables FRED for x86-64, and it's based on the previous LKGS patch set. The Intel flexible return and event delivery (FRED) architecture defines simple new transitions that change privilege level (ring transitions). The FRED architecture was designed with the following goals: 1) Improve overall performance and response time by replacing event delivery through the interrupt descriptor table (IDT event delivery) and event return by the IRET instruction with lower latency transitions. 2) Improve software robustness by ensuring that event delivery establishes the full supervisor context and that event return establishes the full user context. The new transitions defined by the FRED architecture are FRED event delivery and, for returning from events, two FRED return instructions. FRED event delivery can effect a transition from ring 3 to ring 0, but it is used also to deliver events incident to ring 0. One FRED instruction (ERETU) effects a return from ring 0 to ring 3, while the other (ERETS) returns while remaining in ring 0. Search for the latest FRED spec in most search engines with this search pattern: site:intel.com FRED (flexible return and event delivery) specification As of now there is no publicly avaiable CPU supporting FRED, thus the Intel Simics® Simulator is used as software development and testing vehicles. And it can be downloaded from: https://www.intel.com/content/www/us/en/developer/articles/tool/simics-simulator.html To enable FRED, Simics package 8112 QSP-CPU needs to be installed with CPU model configured as: $cpu_comp_class = "x86-experimental-fred" Longer term, we should refactor common code shared by FRED and IDT into common shared files, and contain IDT code using a new config CONFIG_X86_IDT. TODO: call external_interrupt() to reinject IRQ in KVM VMX. Changes since v2: * Improve comments for changes in arch/x86/include/asm/idtentry.h. Changes since v1: * call irqentry_nmi_{enter,exit}() in both IDT and FRED debug fault kernel handler (Peter Zijlstra). * Initialize a FRED exception handler to fred_bad_event() instead of NULL if no FRED handler defined for an exception vector (Peter Zijlstra). * Push calling irqentry_{enter,exit}() and instrumentation_{begin,end}() down into individual FRED exception handlers, instead of in the dispatch framework (Peter Zijlstra). H. Peter Anvin (Intel) (24): x86/traps: let common_interrupt() handle IRQ_MOVE_CLEANUP_VECTOR x86/traps: add a system interrupt table for system interrupt dispatch x86/traps: add external_interrupt() to dispatch external interrupts x86/cpufeature: add the cpu feature bit for FRED x86/opcode: add ERETU, ERETS instructions to x86-opcode-map x86/objtool: teach objtool about ERETU and ERETS x86/cpu: add X86_CR4_FRED macro x86/fred: add Kconfig option for FRED (CONFIG_X86_FRED) x86/fred: if CONFIG_X86_FRED is disabled, disable FRED support x86/cpu: add MSR numbers for FRED configuration x86/fred: header file with FRED definitions x86/fred: make unions for the cs and ss fields in struct pt_regs x86/fred: reserve space for the FRED stack frame x86/fred: add a page fault entry stub for FRED x86/fred: add a debug fault entry stub for FRED x86/fred: add a NMI entry stub for FRED x86/fred: FRED entry/exit and dispatch code x86/fred: FRED initialization code x86/fred: update MSR_IA32_FRED_RSP0 during task switch x86/fred: let ret_from_fork() jmp to fred_exit_user when FRED is enabled x86/fred: disallow the swapgs instruction when FRED is enabled x86/fred: no ESPFIX needed when FRED is enabled x86/fred: allow single-step trap and NMI when starting a new thread x86/fred: allow FRED systems to use interrupt vectors 0x10-0x1f Xin Li (8): x86/traps: add install_system_interrupt_handler() x86/traps: export external_interrupt() for VMX IRQ reinjection x86/fred: header file for event types x86/fred: add a machine check entry stub for FRED x86/fred: fixup fault on ERETU by jumping to fred_entrypoint_user x86/ia32: do not modify the DPL bits for a null selector x86/fred: allow dynamic stack frame size x86/fred: disable FRED by default in its early stage .../admin-guide/kernel-parameters.txt | 4 + arch/x86/Kconfig | 9 + arch/x86/entry/Makefile | 5 +- arch/x86/entry/entry_32.S | 2 +- arch/x86/entry/entry_64.S | 5 + arch/x86/entry/entry_64_fred.S | 59 +++++ arch/x86/entry/entry_fred.c | 234 ++++++++++++++++++ arch/x86/entry/vsyscall/vsyscall_64.c | 2 +- arch/x86/include/asm/cpufeatures.h | 1 + arch/x86/include/asm/disabled-features.h | 8 +- arch/x86/include/asm/entry-common.h | 3 + arch/x86/include/asm/event-type.h | 17 ++ arch/x86/include/asm/extable_fixup_types.h | 4 +- arch/x86/include/asm/fred.h | 131 ++++++++++ arch/x86/include/asm/idtentry.h | 76 +++++- arch/x86/include/asm/irq.h | 5 + arch/x86/include/asm/irq_vectors.h | 15 +- arch/x86/include/asm/msr-index.h | 13 +- arch/x86/include/asm/processor.h | 12 +- arch/x86/include/asm/ptrace.h | 36 ++- arch/x86/include/asm/switch_to.h | 10 +- arch/x86/include/asm/thread_info.h | 35 +-- arch/x86/include/asm/traps.h | 13 + arch/x86/include/asm/vmx.h | 17 +- arch/x86/include/uapi/asm/processor-flags.h | 2 + arch/x86/kernel/Makefile | 1 + arch/x86/kernel/apic/apic.c | 11 +- arch/x86/kernel/apic/vector.c | 8 +- arch/x86/kernel/cpu/acrn.c | 7 +- arch/x86/kernel/cpu/common.c | 88 ++++--- arch/x86/kernel/cpu/mce/core.c | 11 + arch/x86/kernel/cpu/mshyperv.c | 22 +- arch/x86/kernel/espfix_64.c | 8 + arch/x86/kernel/fred.c | 73 ++++++ arch/x86/kernel/head_32.S | 3 +- arch/x86/kernel/idt.c | 6 +- arch/x86/kernel/irq.c | 6 +- arch/x86/kernel/irqinit.c | 7 +- arch/x86/kernel/kvm.c | 4 +- arch/x86/kernel/nmi.c | 28 +++ arch/x86/kernel/process.c | 5 + arch/x86/kernel/process_64.c | 21 +- arch/x86/kernel/signal_32.c | 21 +- arch/x86/kernel/traps.c | 175 +++++++++++-- arch/x86/lib/x86-opcode-map.txt | 2 +- arch/x86/mm/extable.c | 28 +++ arch/x86/mm/fault.c | 20 +- drivers/xen/events/events_base.c | 5 +- kernel/fork.c | 6 + tools/arch/x86/include/asm/cpufeatures.h | 1 + .../arch/x86/include/asm/disabled-features.h | 8 +- tools/arch/x86/include/asm/msr-index.h | 13 +- tools/arch/x86/lib/x86-opcode-map.txt | 2 +- tools/objtool/arch/x86/decode.c | 22 +- 54 files changed, 1156 insertions(+), 174 deletions(-) create mode 100644 arch/x86/entry/entry_64_fred.S create mode 100644 arch/x86/entry/entry_fred.c create mode 100644 arch/x86/include/asm/event-type.h create mode 100644 arch/x86/include/asm/fred.h create mode 100644 arch/x86/kernel/fred.c
Comments
We tested the v3 FRED patch set on the Intel Simics® Simulator and a machine with a 7th Intel(R) Core(TM) CPU. Following are the LTP verion 20220121 test results on X86-64. +--------------------------------------------+-------+-------+-------+-------+ | Config | Pass | Fail | Skip | Hang | +--------------------------------------------+-------+-------+-------+-------+ | the 7th Intel(R) Core(TM) CPU | 1788 | 108 | 453 | 0 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | the 7th Intel(R) Core(TM) CPU | 1788 | 108 | 453 | 0 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/o FRED model | 1797 | 111 | 440 | 1 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/o FRED model | 1797 | 111 | 440 | 1 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1797 | 111 | 440 | 1 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1797 | 111 | 440 | 1 | | 6.2.0-rc7+ w/ FRED patch set FRED disabled | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1797 | 111 | 440 | 1 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ We were unable to identify any regression with the LTP tests. Following are the Kselftest results on X86-64. +--------------------------------------------+-------+-------+-------+-------+ | Config | Pass | Fail | Skip | Hang | +--------------------------------------------+-------+-------+-------+-------+ | the 7th Intel(R) Core(TM) CPU | 3224 | 454 | 591 | 5 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | the 7th Intel(R) Core(TM) CPU | 3224 | 454 | 591 | 5 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/o FRED model | 1853 | 267 | 2143 | 11 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/o FRED model | 1853 | 267 | 2143 | 11 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1853 | 267 | 2143 | 11 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1853 | 267 | 2143 | 11 | | 6.2.0-rc7+ w/ FRED patch set FRED disabled | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1847 | 272 | 2143 | 12 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ No regression was found on the bare metal, while the following are the regressions on the Simics® Simulator. +----------------------+----------------------+----------------------+ | | Intel Simics® | Intel Simics® | | | Simulator w/ FRED | Simulator w/ FRED | | test name | model | model | | | 6.2.0-rc7+ w/ FRED | 6.2.0-rc7+ w/ FRED | | | patch set FRED | patch set | | | disabled | | +----------------------+----------------------+----------------------+ | kvm:hyperv_clock | PASS | FAIL | +----------------------+----------------------+----------------------+ | kvm:hyperv_features | PASS | FAIL | +----------------------+----------------------+----------------------+ | kvm:xen_vmcall_test | PASS | FAIL | +----------------------+----------------------+----------------------+ | kvm:hyperv_ipi | PASS | FAIL | +----------------------+----------------------+----------------------+ | kvm:hyperv_tlb_flush | PASS | FAIL | +----------------------+----------------------+----------------------+ | bpf:test_progs | FAIL | HANG | +----------------------+----------------------+----------------------+ | x86:sysret_rip_64 | PASS | FAIL | +----------------------+----------------------+----------------------+ The test "x86:sysret_rip_64" is NOT a valid test on FRED, and there is a fix from Ammar Faizi after we discussed it in the LKML. Other test failures are under investigation. FRED is a 64-bit only feature, however the 31st patch "x86/fred: allow dynamic stack frame size" applies to X86-32, thus we ran Kselftest and LTP on X86-32 to see if there is any regression, and we din't find any. Following are the Kselftest results on X86-32. +-------------------------------+---------+---------+---------+---------+ | Config | Pass | Fail | Skip | Hang | +-------------------------------+---------+---------+---------+---------+ | the 5th Intel(R) Core(TM) CPU | 1361 | 267 | 301 | 3 | | FRED 6.2.0-rc7+ | | | | | +-------------------------------+---------+---------+---------+---------+ | the 5th Intel(R) Core(TM) CPU | 1361 | 267 | 301 | 3 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +-------------------------------+---------+---------+---------+---------+ Following are the LTP test results on X86-32. +-------------------------------+---------+---------+---------+---------+ | Config | Pass | Fail | Skip | Hang | +-------------------------------+---------+---------+---------+---------+ | the 5th Intel(R) Core(TM) CPU | 1849 | 81 | 419 | 0 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +-------------------------------+---------+---------+---------+---------+ | the 5th Intel(R) Core(TM) CPU | 1849 | 81 | 419 | 0 | | FRED 6.2.0-rc7+ | | | | | +-------------------------------+---------+---------+---------+---------+ Thanks --Shan On Thu, 2023-02-23 at 23:01 -0800, Xin Li wrote: > This patch set enables FRED for x86-64, and it's based on the previous LKGS > patch set. > > The Intel flexible return and event delivery (FRED) architecture defines > simple > new transitions that change privilege level (ring transitions). The FRED > architecture was designed with the following goals: > 1) Improve overall performance and response time by replacing event delivery > through the interrupt descriptor table (IDT event delivery) and event return > by > the IRET instruction with lower latency transitions. > 2) Improve software robustness by ensuring that event delivery establishes the > full supervisor context and that event return establishes the full user > context. > > The new transitions defined by the FRED architecture are FRED event delivery > and, > for returning from events, two FRED return instructions. FRED event delivery > can > effect a transition from ring 3 to ring 0, but it is used also to deliver > events > incident to ring 0. One FRED instruction (ERETU) effects a return from ring 0 > to > ring 3, while the other (ERETS) returns while remaining in ring 0. > > Search for the latest FRED spec in most search engines with this search > pattern: > > site:intel.com FRED (flexible return and event delivery) specification > > As of now there is no publicly avaiable CPU supporting FRED, thus the Intel > Simics® Simulator is used as software development and testing vehicles. And > it can be downloaded from: > > https://www.intel.com/content/www/us/en/developer/articles/tool/simics-simulator.html > > To enable FRED, Simics package 8112 QSP-CPU needs to be installed with CPU > model configured as: > $cpu_comp_class = "x86-experimental-fred" > > Longer term, we should refactor common code shared by FRED and IDT into common > shared files, and contain IDT code using a new config CONFIG_X86_IDT. > > TODO: call external_interrupt() to reinject IRQ in KVM VMX. > > Changes since v2: > * Improve comments for changes in arch/x86/include/asm/idtentry.h. > > Changes since v1: > * call irqentry_nmi_{enter,exit}() in both IDT and FRED debug fault kernel > handler (Peter Zijlstra). > * Initialize a FRED exception handler to fred_bad_event() instead of NULL > if no FRED handler defined for an exception vector (Peter Zijlstra). > * Push calling irqentry_{enter,exit}() and instrumentation_{begin,end}() > down into individual FRED exception handlers, instead of in the dispatch > framework (Peter Zijlstra). > > H. Peter Anvin (Intel) (24): > x86/traps: let common_interrupt() handle IRQ_MOVE_CLEANUP_VECTOR > x86/traps: add a system interrupt table for system interrupt dispatch > x86/traps: add external_interrupt() to dispatch external interrupts > x86/cpufeature: add the cpu feature bit for FRED > x86/opcode: add ERETU, ERETS instructions to x86-opcode-map > x86/objtool: teach objtool about ERETU and ERETS > x86/cpu: add X86_CR4_FRED macro > x86/fred: add Kconfig option for FRED (CONFIG_X86_FRED) > x86/fred: if CONFIG_X86_FRED is disabled, disable FRED support > x86/cpu: add MSR numbers for FRED configuration > x86/fred: header file with FRED definitions > x86/fred: make unions for the cs and ss fields in struct pt_regs > x86/fred: reserve space for the FRED stack frame > x86/fred: add a page fault entry stub for FRED > x86/fred: add a debug fault entry stub for FRED > x86/fred: add a NMI entry stub for FRED > x86/fred: FRED entry/exit and dispatch code > x86/fred: FRED initialization code > x86/fred: update MSR_IA32_FRED_RSP0 during task switch > x86/fred: let ret_from_fork() jmp to fred_exit_user when FRED is > enabled > x86/fred: disallow the swapgs instruction when FRED is enabled > x86/fred: no ESPFIX needed when FRED is enabled > x86/fred: allow single-step trap and NMI when starting a new thread > x86/fred: allow FRED systems to use interrupt vectors 0x10-0x1f > > Xin Li (8): > x86/traps: add install_system_interrupt_handler() > x86/traps: export external_interrupt() for VMX IRQ reinjection > x86/fred: header file for event types > x86/fred: add a machine check entry stub for FRED > x86/fred: fixup fault on ERETU by jumping to fred_entrypoint_user > x86/ia32: do not modify the DPL bits for a null selector > x86/fred: allow dynamic stack frame size > x86/fred: disable FRED by default in its early stage > > .../admin-guide/kernel-parameters.txt | 4 + > arch/x86/Kconfig | 9 + > arch/x86/entry/Makefile | 5 +- > arch/x86/entry/entry_32.S | 2 +- > arch/x86/entry/entry_64.S | 5 + > arch/x86/entry/entry_64_fred.S | 59 +++++ > arch/x86/entry/entry_fred.c | 234 ++++++++++++++++++ > arch/x86/entry/vsyscall/vsyscall_64.c | 2 +- > arch/x86/include/asm/cpufeatures.h | 1 + > arch/x86/include/asm/disabled-features.h | 8 +- > arch/x86/include/asm/entry-common.h | 3 + > arch/x86/include/asm/event-type.h | 17 ++ > arch/x86/include/asm/extable_fixup_types.h | 4 +- > arch/x86/include/asm/fred.h | 131 ++++++++++ > arch/x86/include/asm/idtentry.h | 76 +++++- > arch/x86/include/asm/irq.h | 5 + > arch/x86/include/asm/irq_vectors.h | 15 +- > arch/x86/include/asm/msr-index.h | 13 +- > arch/x86/include/asm/processor.h | 12 +- > arch/x86/include/asm/ptrace.h | 36 ++- > arch/x86/include/asm/switch_to.h | 10 +- > arch/x86/include/asm/thread_info.h | 35 +-- > arch/x86/include/asm/traps.h | 13 + > arch/x86/include/asm/vmx.h | 17 +- > arch/x86/include/uapi/asm/processor-flags.h | 2 + > arch/x86/kernel/Makefile | 1 + > arch/x86/kernel/apic/apic.c | 11 +- > arch/x86/kernel/apic/vector.c | 8 +- > arch/x86/kernel/cpu/acrn.c | 7 +- > arch/x86/kernel/cpu/common.c | 88 ++++--- > arch/x86/kernel/cpu/mce/core.c | 11 + > arch/x86/kernel/cpu/mshyperv.c | 22 +- > arch/x86/kernel/espfix_64.c | 8 + > arch/x86/kernel/fred.c | 73 ++++++ > arch/x86/kernel/head_32.S | 3 +- > arch/x86/kernel/idt.c | 6 +- > arch/x86/kernel/irq.c | 6 +- > arch/x86/kernel/irqinit.c | 7 +- > arch/x86/kernel/kvm.c | 4 +- > arch/x86/kernel/nmi.c | 28 +++ > arch/x86/kernel/process.c | 5 + > arch/x86/kernel/process_64.c | 21 +- > arch/x86/kernel/signal_32.c | 21 +- > arch/x86/kernel/traps.c | 175 +++++++++++-- > arch/x86/lib/x86-opcode-map.txt | 2 +- > arch/x86/mm/extable.c | 28 +++ > arch/x86/mm/fault.c | 20 +- > drivers/xen/events/events_base.c | 5 +- > kernel/fork.c | 6 + > tools/arch/x86/include/asm/cpufeatures.h | 1 + > .../arch/x86/include/asm/disabled-features.h | 8 +- > tools/arch/x86/include/asm/msr-index.h | 13 +- > tools/arch/x86/lib/x86-opcode-map.txt | 2 +- > tools/objtool/arch/x86/decode.c | 22 +- > 54 files changed, 1156 insertions(+), 174 deletions(-) > create mode 100644 arch/x86/entry/entry_64_fred.S > create mode 100644 arch/x86/entry/entry_fred.c > create mode 100644 arch/x86/include/asm/event-type.h > create mode 100644 arch/x86/include/asm/fred.h > create mode 100644 arch/x86/kernel/fred.c >
We tested the v3 FRED patch set on the Intel Simics® Simulator and a machine with a 7th Intel(R) Core(TM) CPU. Following are the LTP verion 20220121 test results on X86-64. +--------------------------------------------+-------+-------+-------+-------+ | Config | Pass | Fail | Skip | Hang | +--------------------------------------------+-------+-------+-------+-------+ | the 7th Intel(R) Core(TM) CPU | 1788 | 108 | 453 | 0 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | the 7th Intel(R) Core(TM) CPU | 1788 | 108 | 453 | 0 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/o FRED model | 1797 | 111 | 440 | 1 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/o FRED model | 1797 | 111 | 440 | 1 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1797 | 111 | 440 | 1 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1797 | 111 | 440 | 1 | | 6.2.0-rc7+ w/ FRED patch set FRED disabled | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1797 | 111 | 440 | 1 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ We were unable to identify any regression with the LTP tests. Following are the Kselftest results on X86-64. +--------------------------------------------+-------+-------+-------+-------+ | Config | Pass | Fail | Skip | Hang | +--------------------------------------------+-------+-------+-------+-------+ | the 7th Intel(R) Core(TM) CPU | 3224 | 454 | 591 | 5 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | the 7th Intel(R) Core(TM) CPU | 3224 | 454 | 591 | 5 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/o FRED model | 1853 | 267 | 2143 | 11 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/o FRED model | 1853 | 267 | 2143 | 11 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1853 | 267 | 2143 | 11 | | 6.2.0-rc7+ | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1853 | 267 | 2143 | 11 | | 6.2.0-rc7+ w/ FRED patch set FRED disabled | | | | | +--------------------------------------------+-------+-------+-------+-------+ | Intel Simics® Simulator w/ FRED model | 1847 | 272 | 2143 | 12 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +--------------------------------------------+-------+-------+-------+-------+ No regression was found on the bare metal, while the following are the regressions on the Simics® Simulator. +----------------------+----------------------+----------------------+ | | Intel Simics® | Intel Simics® | | | Simulator w/ FRED | Simulator w/ FRED | | test name | model | model | | | 6.2.0-rc7+ w/ FRED | 6.2.0-rc7+ w/ FRED | | | patch set FRED | patch set | | | disabled | | +----------------------+----------------------+----------------------+ | kvm:hyperv_clock | PASS | FAIL | +----------------------+----------------------+----------------------+ | kvm:hyperv_features | PASS | FAIL | +----------------------+----------------------+----------------------+ | kvm:xen_vmcall_test | PASS | FAIL | +----------------------+----------------------+----------------------+ | kvm:hyperv_ipi | PASS | FAIL | +----------------------+----------------------+----------------------+ | kvm:hyperv_tlb_flush | PASS | FAIL | +----------------------+----------------------+----------------------+ | bpf:test_progs | FAIL | HANG | +----------------------+----------------------+----------------------+ | x86:sysret_rip_64 | PASS | FAIL | +----------------------+----------------------+----------------------+ The test "x86:sysret_rip_64" is NOT a valid test on FRED, and there is a fix from Ammar Faizi after we discussed it in the LKML. Other test failures are under investigation. FRED is a 64-bit only feature, however the 31st patch "x86/fred: allow dynamic stack frame size" applies to X86-32, thus we ran Kselftest and LTP on X86-32 to see if there is any regression, and we din't find any. Following are the Kselftest results on X86-32. +-------------------------------+---------+---------+---------+---------+ | Config | Pass | Fail | Skip | Hang | +-------------------------------+---------+---------+---------+---------+ | the 5th Intel(R) Core(TM) CPU | 1361 | 267 | 301 | 3 | | FRED 6.2.0-rc7+ | | | | | +-------------------------------+---------+---------+---------+---------+ | the 5th Intel(R) Core(TM) CPU | 1361 | 267 | 301 | 3 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +-------------------------------+---------+---------+---------+---------+ Following are the LTP test results on X86-32. +-------------------------------+---------+---------+---------+---------+ | Config | Pass | Fail | Skip | Hang | +-------------------------------+---------+---------+---------+---------+ | the 5th Intel(R) Core(TM) CPU | 1849 | 81 | 419 | 0 | | 6.2.0-rc7+ w/ FRED patch set | | | | | +-------------------------------+---------+---------+---------+---------+ | the 5th Intel(R) Core(TM) CPU | 1849 | 81 | 419 | 0 | | FRED 6.2.0-rc7+ | | | | | +-------------------------------+---------+---------+---------+---------+ Thanks --Shan On Thu, 2023-02-23 at 23:01 -0800, Xin Li wrote: > This patch set enables FRED for x86-64, and it's based on the previous LKGS > patch set. > > The Intel flexible return and event delivery (FRED) architecture defines > simple > new transitions that change privilege level (ring transitions). The FRED > architecture was designed with the following goals: > 1) Improve overall performance and response time by replacing event delivery > through the interrupt descriptor table (IDT event delivery) and event return > by > the IRET instruction with lower latency transitions. > 2) Improve software robustness by ensuring that event delivery establishes the > full supervisor context and that event return establishes the full user > context. > > The new transitions defined by the FRED architecture are FRED event delivery > and, > for returning from events, two FRED return instructions. FRED event delivery > can > effect a transition from ring 3 to ring 0, but it is used also to deliver > events > incident to ring 0. One FRED instruction (ERETU) effects a return from ring 0 > to > ring 3, while the other (ERETS) returns while remaining in ring 0. > > Search for the latest FRED spec in most search engines with this search > pattern: > > site:intel.com FRED (flexible return and event delivery) specification > > As of now there is no publicly avaiable CPU supporting FRED, thus the Intel > Simics® Simulator is used as software development and testing vehicles. And > it can be downloaded from: > > https://www.intel.com/content/www/us/en/developer/articles/tool/simics-simulator.html > > To enable FRED, Simics package 8112 QSP-CPU needs to be installed with CPU > model configured as: > $cpu_comp_class = "x86-experimental-fred" > > Longer term, we should refactor common code shared by FRED and IDT into common > shared files, and contain IDT code using a new config CONFIG_X86_IDT. > > TODO: call external_interrupt() to reinject IRQ in KVM VMX. > > Changes since v2: > * Improve comments for changes in arch/x86/include/asm/idtentry.h. > > Changes since v1: > * call irqentry_nmi_{enter,exit}() in both IDT and FRED debug fault kernel > handler (Peter Zijlstra). > * Initialize a FRED exception handler to fred_bad_event() instead of NULL > if no FRED handler defined for an exception vector (Peter Zijlstra). > * Push calling irqentry_{enter,exit}() and instrumentation_{begin,end}() > down into individual FRED exception handlers, instead of in the dispatch > framework (Peter Zijlstra). > > H. Peter Anvin (Intel) (24): > x86/traps: let common_interrupt() handle IRQ_MOVE_CLEANUP_VECTOR > x86/traps: add a system interrupt table for system interrupt dispatch > x86/traps: add external_interrupt() to dispatch external interrupts > x86/cpufeature: add the cpu feature bit for FRED > x86/opcode: add ERETU, ERETS instructions to x86-opcode-map > x86/objtool: teach objtool about ERETU and ERETS > x86/cpu: add X86_CR4_FRED macro > x86/fred: add Kconfig option for FRED (CONFIG_X86_FRED) > x86/fred: if CONFIG_X86_FRED is disabled, disable FRED support > x86/cpu: add MSR numbers for FRED configuration > x86/fred: header file with FRED definitions > x86/fred: make unions for the cs and ss fields in struct pt_regs > x86/fred: reserve space for the FRED stack frame > x86/fred: add a page fault entry stub for FRED > x86/fred: add a debug fault entry stub for FRED > x86/fred: add a NMI entry stub for FRED > x86/fred: FRED entry/exit and dispatch code > x86/fred: FRED initialization code > x86/fred: update MSR_IA32_FRED_RSP0 during task switch > x86/fred: let ret_from_fork() jmp to fred_exit_user when FRED is > enabled > x86/fred: disallow the swapgs instruction when FRED is enabled > x86/fred: no ESPFIX needed when FRED is enabled > x86/fred: allow single-step trap and NMI when starting a new thread > x86/fred: allow FRED systems to use interrupt vectors 0x10-0x1f > > Xin Li (8): > x86/traps: add install_system_interrupt_handler() > x86/traps: export external_interrupt() for VMX IRQ reinjection > x86/fred: header file for event types > x86/fred: add a machine check entry stub for FRED > x86/fred: fixup fault on ERETU by jumping to fred_entrypoint_user > x86/ia32: do not modify the DPL bits for a null selector > x86/fred: allow dynamic stack frame size > x86/fred: disable FRED by default in its early stage > > .../admin-guide/kernel-parameters.txt | 4 + > arch/x86/Kconfig | 9 + > arch/x86/entry/Makefile | 5 +- > arch/x86/entry/entry_32.S | 2 +- > arch/x86/entry/entry_64.S | 5 + > arch/x86/entry/entry_64_fred.S | 59 +++++ > arch/x86/entry/entry_fred.c | 234 ++++++++++++++++++ > arch/x86/entry/vsyscall/vsyscall_64.c | 2 +- > arch/x86/include/asm/cpufeatures.h | 1 + > arch/x86/include/asm/disabled-features.h | 8 +- > arch/x86/include/asm/entry-common.h | 3 + > arch/x86/include/asm/event-type.h | 17 ++ > arch/x86/include/asm/extable_fixup_types.h | 4 +- > arch/x86/include/asm/fred.h | 131 ++++++++++ > arch/x86/include/asm/idtentry.h | 76 +++++- > arch/x86/include/asm/irq.h | 5 + > arch/x86/include/asm/irq_vectors.h | 15 +- > arch/x86/include/asm/msr-index.h | 13 +- > arch/x86/include/asm/processor.h | 12 +- > arch/x86/include/asm/ptrace.h | 36 ++- > arch/x86/include/asm/switch_to.h | 10 +- > arch/x86/include/asm/thread_info.h | 35 +-- > arch/x86/include/asm/traps.h | 13 + > arch/x86/include/asm/vmx.h | 17 +- > arch/x86/include/uapi/asm/processor-flags.h | 2 + > arch/x86/kernel/Makefile | 1 + > arch/x86/kernel/apic/apic.c | 11 +- > arch/x86/kernel/apic/vector.c | 8 +- > arch/x86/kernel/cpu/acrn.c | 7 +- > arch/x86/kernel/cpu/common.c | 88 ++++--- > arch/x86/kernel/cpu/mce/core.c | 11 + > arch/x86/kernel/cpu/mshyperv.c | 22 +- > arch/x86/kernel/espfix_64.c | 8 + > arch/x86/kernel/fred.c | 73 ++++++ > arch/x86/kernel/head_32.S | 3 +- > arch/x86/kernel/idt.c | 6 +- > arch/x86/kernel/irq.c | 6 +- > arch/x86/kernel/irqinit.c | 7 +- > arch/x86/kernel/kvm.c | 4 +- > arch/x86/kernel/nmi.c | 28 +++ > arch/x86/kernel/process.c | 5 + > arch/x86/kernel/process_64.c | 21 +- > arch/x86/kernel/signal_32.c | 21 +- > arch/x86/kernel/traps.c | 175 +++++++++++-- > arch/x86/lib/x86-opcode-map.txt | 2 +- > arch/x86/mm/extable.c | 28 +++ > arch/x86/mm/fault.c | 20 +- > drivers/xen/events/events_base.c | 5 +- > kernel/fork.c | 6 + > tools/arch/x86/include/asm/cpufeatures.h | 1 + > .../arch/x86/include/asm/disabled-features.h | 8 +- > tools/arch/x86/include/asm/msr-index.h | 13 +- > tools/arch/x86/lib/x86-opcode-map.txt | 2 +- > tools/objtool/arch/x86/decode.c | 22 +- > 54 files changed, 1156 insertions(+), 174 deletions(-) > create mode 100644 arch/x86/entry/entry_64_fred.S > create mode 100644 arch/x86/entry/entry_fred.c > create mode 100644 arch/x86/include/asm/event-type.h > create mode 100644 arch/x86/include/asm/fred.h > create mode 100644 arch/x86/kernel/fred.c >
I plan to remove RFC in the next round, any objection? BTW, the KVM selftest failures Shan reported will be fixed with FRED KVM patch set, which is planned to be sent to LKML for review in March. Thanks! Xin > -----Original Message----- > From: Kang, Shan <shan.kang@intel.com> > Sent: Friday, February 24, 2023 1:27 AM > To: Li, Xin3 <xin3.li@intel.com>; kvm@vger.kernel.org; linux- > kernel@vger.kernel.org; x86@kernel.org > Cc: Christopherson,, Sean <seanjc@google.com>; bp@alien8.de; > dave.hansen@linux.intel.com; peterz@infradead.org; hpa@zytor.com; > mingo@redhat.com; tglx@linutronix.de; andrew.cooper3@citrix.com; > pbonzini@redhat.com; Shankar, Ravi V <ravi.v.shankar@intel.com> > Subject: Re: [RFC PATCH v3 00/32] x86: enable FRED for x86-64 > > We tested the v3 FRED patch set on the Intel Simics® Simulator and a machine with > a 7th Intel(R) Core(TM) CPU. > > Following are the LTP verion 20220121 test results on X86-64. > +--------------------------------------------+-------+-------+-------+-------+ > | Config | Pass | Fail | Skip | Hang | > +--------------------------------------------+-------+-------+-------+-------+ > | the 7th Intel(R) Core(TM) CPU | 1788 | 108 | 453 | 0 | > | 6.2.0-rc7+ | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | the 7th Intel(R) Core(TM) CPU | 1788 | 108 | 453 | 0 | > | 6.2.0-rc7+ w/ FRED patch set | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | Intel Simics® Simulator w/o FRED model | 1797 | 111 | 440 | 1 | > | 6.2.0-rc7+ | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | Intel Simics® Simulator w/o FRED model | 1797 | 111 | 440 | 1 | > | 6.2.0-rc7+ w/ FRED patch set | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | Intel Simics® Simulator w/ FRED model | 1797 | 111 | 440 | 1 | > | 6.2.0-rc7+ | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | Intel Simics® Simulator w/ FRED model | 1797 | 111 | 440 | 1 | > | 6.2.0-rc7+ w/ FRED patch set FRED disabled | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | Intel Simics® Simulator w/ FRED model | 1797 | 111 | 440 | 1 | > | 6.2.0-rc7+ w/ FRED patch set | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > We were unable to identify any regression with the LTP tests. > > Following are the Kselftest results on X86-64. > +--------------------------------------------+-------+-------+-------+-------+ > | Config | Pass | Fail | Skip | Hang | > +--------------------------------------------+-------+-------+-------+-------+ > | the 7th Intel(R) Core(TM) CPU | 3224 | 454 | 591 | 5 | > | 6.2.0-rc7+ | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | the 7th Intel(R) Core(TM) CPU | 3224 | 454 | 591 | 5 | > | 6.2.0-rc7+ w/ FRED patch set | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | Intel Simics® Simulator w/o FRED model | 1853 | 267 | 2143 | 11 | > | 6.2.0-rc7+ | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | Intel Simics® Simulator w/o FRED model | 1853 | 267 | 2143 | 11 | > | 6.2.0-rc7+ w/ FRED patch set | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | Intel Simics® Simulator w/ FRED model | 1853 | 267 | 2143 | 11 | > | 6.2.0-rc7+ | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | Intel Simics® Simulator w/ FRED model | 1853 | 267 | 2143 | 11 | > | 6.2.0-rc7+ w/ FRED patch set FRED disabled | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > | Intel Simics® Simulator w/ FRED model | 1847 | 272 | 2143 | 12 | > | 6.2.0-rc7+ w/ FRED patch set | | | | | > +--------------------------------------------+-------+-------+-------+-------+ > No regression was found on the bare metal, while the following are the regressions > on the Simics® Simulator. > +----------------------+----------------------+----------------------+ > | | Intel Simics® | Intel Simics® | > | | Simulator w/ FRED | Simulator w/ FRED | > | test name | model | model | > | | 6.2.0-rc7+ w/ FRED | 6.2.0-rc7+ w/ FRED | > | | patch set FRED | patch set | > | | disabled | | > +----------------------+----------------------+----------------------+ > | kvm:hyperv_clock | PASS | FAIL | > +----------------------+----------------------+----------------------+ > | kvm:hyperv_features | PASS | FAIL | > +----------------------+----------------------+----------------------+ > | kvm:xen_vmcall_test | PASS | FAIL | > +----------------------+----------------------+----------------------+ > | kvm:hyperv_ipi | PASS | FAIL | > +----------------------+----------------------+----------------------+ > | kvm:hyperv_tlb_flush | PASS | FAIL | > +----------------------+----------------------+----------------------+ > | bpf:test_progs | FAIL | HANG | > +----------------------+----------------------+----------------------+ > | x86:sysret_rip_64 | PASS | FAIL | > +----------------------+----------------------+----------------------+ > The test "x86:sysret_rip_64" is NOT a valid test on FRED, and there is a fix from > Ammar Faizi after we discussed it in the LKML. Other test failures are under > investigation. > > FRED is a 64-bit only feature, however the 31st patch "x86/fred: allow dynamic > stack frame size" applies to X86-32, thus we ran Kselftest and LTP on X86-32 to see > if there is any regression, and we din't find any. > > Following are the Kselftest results on X86-32. > +-------------------------------+---------+---------+---------+---------+ > | Config | Pass | Fail | Skip | Hang | > +-------------------------------+---------+---------+---------+---------+ > | the 5th Intel(R) Core(TM) CPU | 1361 | 267 | 301 | 3 | > | FRED 6.2.0-rc7+ | | | | | > +-------------------------------+---------+---------+---------+---------+ > | the 5th Intel(R) Core(TM) CPU | 1361 | 267 | 301 | 3 | > | 6.2.0-rc7+ w/ FRED patch set | | | | | > +-------------------------------+---------+---------+---------+---------+ > > Following are the LTP test results on X86-32. > +-------------------------------+---------+---------+---------+---------+ > | Config | Pass | Fail | Skip | Hang | > +-------------------------------+---------+---------+---------+---------+ > | the 5th Intel(R) Core(TM) CPU | 1849 | 81 | 419 | 0 | > | 6.2.0-rc7+ w/ FRED patch set | | | | | > +-------------------------------+---------+---------+---------+---------+ > | the 5th Intel(R) Core(TM) CPU | 1849 | 81 | 419 | 0 | > | FRED 6.2.0-rc7+ | | | | | > +-------------------------------+---------+---------+---------+---------+ > > Thanks > --Shan > > On Thu, 2023-02-23 at 23:01 -0800, Xin Li wrote: > > This patch set enables FRED for x86-64, and it's based on the previous > > LKGS patch set. > > > > The Intel flexible return and event delivery (FRED) architecture > > defines simple new transitions that change privilege level (ring > > transitions). The FRED architecture was designed with the following > > goals: > > 1) Improve overall performance and response time by replacing event > > delivery through the interrupt descriptor table (IDT event delivery) > > and event return by the IRET instruction with lower latency > > transitions. > > 2) Improve software robustness by ensuring that event delivery > > establishes the full supervisor context and that event return > > establishes the full user context. > > > > The new transitions defined by the FRED architecture are FRED event > > delivery and, for returning from events, two FRED return instructions. > > FRED event delivery can effect a transition from ring 3 to ring 0, but > > it is used also to deliver events incident to ring 0. One FRED > > instruction (ERETU) effects a return from ring 0 to ring 3, while the > > other (ERETS) returns while remaining in ring 0. > > > > Search for the latest FRED spec in most search engines with this > > search > > pattern: > > > > site:intel.com FRED (flexible return and event delivery) > > specification > > > > As of now there is no publicly avaiable CPU supporting FRED, thus the > > Intel Simics® Simulator is used as software development and testing > > vehicles. And it can be downloaded from: > > > > https://www.intel.com/content/www/us/en/developer/articles/tool/simics > > -simulator.html > > > > To enable FRED, Simics package 8112 QSP-CPU needs to be installed with > > CPU model configured as: > > $cpu_comp_class = "x86-experimental-fred" > > > > Longer term, we should refactor common code shared by FRED and IDT > > into common shared files, and contain IDT code using a new config > CONFIG_X86_IDT. > > > > TODO: call external_interrupt() to reinject IRQ in KVM VMX. > > > > Changes since v2: > > * Improve comments for changes in arch/x86/include/asm/idtentry.h. > > > > Changes since v1: > > * call irqentry_nmi_{enter,exit}() in both IDT and FRED debug fault kernel > > handler (Peter Zijlstra). > > * Initialize a FRED exception handler to fred_bad_event() instead of NULL > > if no FRED handler defined for an exception vector (Peter Zijlstra). > > * Push calling irqentry_{enter,exit}() and instrumentation_{begin,end}() > > down into individual FRED exception handlers, instead of in the dispatch > > framework (Peter Zijlstra). > > > > H. Peter Anvin (Intel) (24): > > x86/traps: let common_interrupt() handle IRQ_MOVE_CLEANUP_VECTOR > > x86/traps: add a system interrupt table for system interrupt dispatch > > x86/traps: add external_interrupt() to dispatch external interrupts > > x86/cpufeature: add the cpu feature bit for FRED > > x86/opcode: add ERETU, ERETS instructions to x86-opcode-map > > x86/objtool: teach objtool about ERETU and ERETS > > x86/cpu: add X86_CR4_FRED macro > > x86/fred: add Kconfig option for FRED (CONFIG_X86_FRED) > > x86/fred: if CONFIG_X86_FRED is disabled, disable FRED support > > x86/cpu: add MSR numbers for FRED configuration > > x86/fred: header file with FRED definitions > > x86/fred: make unions for the cs and ss fields in struct pt_regs > > x86/fred: reserve space for the FRED stack frame > > x86/fred: add a page fault entry stub for FRED > > x86/fred: add a debug fault entry stub for FRED > > x86/fred: add a NMI entry stub for FRED > > x86/fred: FRED entry/exit and dispatch code > > x86/fred: FRED initialization code > > x86/fred: update MSR_IA32_FRED_RSP0 during task switch > > x86/fred: let ret_from_fork() jmp to fred_exit_user when FRED is > > enabled > > x86/fred: disallow the swapgs instruction when FRED is enabled > > x86/fred: no ESPFIX needed when FRED is enabled > > x86/fred: allow single-step trap and NMI when starting a new thread > > x86/fred: allow FRED systems to use interrupt vectors 0x10-0x1f > > > > Xin Li (8): > > x86/traps: add install_system_interrupt_handler() > > x86/traps: export external_interrupt() for VMX IRQ reinjection > > x86/fred: header file for event types > > x86/fred: add a machine check entry stub for FRED > > x86/fred: fixup fault on ERETU by jumping to fred_entrypoint_user > > x86/ia32: do not modify the DPL bits for a null selector > > x86/fred: allow dynamic stack frame size > > x86/fred: disable FRED by default in its early stage > > > > .../admin-guide/kernel-parameters.txt | 4 + > > arch/x86/Kconfig | 9 + > > arch/x86/entry/Makefile | 5 +- > > arch/x86/entry/entry_32.S | 2 +- > > arch/x86/entry/entry_64.S | 5 + > > arch/x86/entry/entry_64_fred.S | 59 +++++ > > arch/x86/entry/entry_fred.c | 234 ++++++++++++++++++ > > arch/x86/entry/vsyscall/vsyscall_64.c | 2 +- > > arch/x86/include/asm/cpufeatures.h | 1 + > > arch/x86/include/asm/disabled-features.h | 8 +- > > arch/x86/include/asm/entry-common.h | 3 + > > arch/x86/include/asm/event-type.h | 17 ++ > > arch/x86/include/asm/extable_fixup_types.h | 4 +- > > arch/x86/include/asm/fred.h | 131 ++++++++++ > > arch/x86/include/asm/idtentry.h | 76 +++++- > > arch/x86/include/asm/irq.h | 5 + > > arch/x86/include/asm/irq_vectors.h | 15 +- > > arch/x86/include/asm/msr-index.h | 13 +- > > arch/x86/include/asm/processor.h | 12 +- > > arch/x86/include/asm/ptrace.h | 36 ++- > > arch/x86/include/asm/switch_to.h | 10 +- > > arch/x86/include/asm/thread_info.h | 35 +-- > > arch/x86/include/asm/traps.h | 13 + > > arch/x86/include/asm/vmx.h | 17 +- > > arch/x86/include/uapi/asm/processor-flags.h | 2 + > > arch/x86/kernel/Makefile | 1 + > > arch/x86/kernel/apic/apic.c | 11 +- > > arch/x86/kernel/apic/vector.c | 8 +- > > arch/x86/kernel/cpu/acrn.c | 7 +- > > arch/x86/kernel/cpu/common.c | 88 ++++--- > > arch/x86/kernel/cpu/mce/core.c | 11 + > > arch/x86/kernel/cpu/mshyperv.c | 22 +- > > arch/x86/kernel/espfix_64.c | 8 + > > arch/x86/kernel/fred.c | 73 ++++++ > > arch/x86/kernel/head_32.S | 3 +- > > arch/x86/kernel/idt.c | 6 +- > > arch/x86/kernel/irq.c | 6 +- > > arch/x86/kernel/irqinit.c | 7 +- > > arch/x86/kernel/kvm.c | 4 +- > > arch/x86/kernel/nmi.c | 28 +++ > > arch/x86/kernel/process.c | 5 + > > arch/x86/kernel/process_64.c | 21 +- > > arch/x86/kernel/signal_32.c | 21 +- > > arch/x86/kernel/traps.c | 175 +++++++++++-- > > arch/x86/lib/x86-opcode-map.txt | 2 +- > > arch/x86/mm/extable.c | 28 +++ > > arch/x86/mm/fault.c | 20 +- > > drivers/xen/events/events_base.c | 5 +- > > kernel/fork.c | 6 + > > tools/arch/x86/include/asm/cpufeatures.h | 1 + > > .../arch/x86/include/asm/disabled-features.h | 8 +- > > tools/arch/x86/include/asm/msr-index.h | 13 +- > > tools/arch/x86/lib/x86-opcode-map.txt | 2 +- > > tools/objtool/arch/x86/decode.c | 22 +- > > 54 files changed, 1156 insertions(+), 174 deletions(-) create mode > > 100644 arch/x86/entry/entry_64_fred.S create mode 100644 > > arch/x86/entry/entry_fred.c create mode 100644 > > arch/x86/include/asm/event-type.h create mode 100644 > > arch/x86/include/asm/fred.h create mode 100644 arch/x86/kernel/fred.c > >