From: Andrew Cooper <andrew.cooper3@citrix.com>
To: dave.hansen@intel.com
Cc: Amit.Shah@amd.com, Babu.Moger@amd.com, David.Kaplan@amd.com,
Sandipan.Das@amd.com, Thomas.Lendacky@amd.com,
boris.ostrovsky@oracle.com, bp@alien8.de, corbet@lwn.net,
daniel.sneddon@linux.intel.com, dave.hansen@linux.intel.com,
hpa@zytor.com, jpoimboe@kernel.org, kai.huang@intel.com,
kvm@vger.kernel.org, linux-doc@vger.kernel.org,
linux-kernel@vger.kernel.org, mingo@redhat.com,
pawan.kumar.gupta@linux.intel.com, pbonzini@redhat.com,
peterz@infradead.org, seanjc@google.com, tglx@linutronix.de,
x86@kernel.org
Subject: Re: [PATCH 1/2] x86: cpu/bugs: add support for AMD ERAPS feature
Date: Tue, 5 Nov 2024 00:04:28 +0000 [thread overview]
Message-ID: <e5463e70-a0a6-4cce-9c2c-4a09c2e622ef@citrix.com> (raw)
In-Reply-To: <62063466-69bc-4eca-9f22-492c70b02250@intel.com>
> So, I'll flip this back around. Today, X86_FEATURE_RSB_CTXSW zaps the
> RSB whenever RSP is updated to a new task stack. Please convince me
> that ERAPS provides superior coverage or is unnecessary in all the
> possible combinations switching between:
>
> different thread, same mm
> user=>kernel, same mm
> kernel=>user, same mm
> different mm (we already covered this)
>
> Because several of those switches can happen without a CR3 write or INVPCID.
user=>kernel=>user, same mm explicitly does not want to flush the RAS,
because if the system call is shallow enough, some of the userspace RAS
is still intact on when you get back into user mode.
The case which I expect will go wrong is user=>kernel=>different kthread
because this stays on the same mm.
That does need to flush the RAS and won't hit any TLB maintenance
instructions that I'm aware of.
~Andrew
next prev parent reply other threads:[~2024-11-05 0:04 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-31 15:39 [PATCH 0/2] Add support for the " Amit Shah
2024-10-31 15:39 ` [PATCH 1/2] x86: cpu/bugs: add support for AMD " Amit Shah
2024-10-31 23:03 ` Pawan Gupta
2024-11-04 8:57 ` Shah, Amit
2024-11-04 14:52 ` Andrew Cooper
2024-11-04 15:00 ` Shah, Amit
2024-10-31 23:11 ` Dave Hansen
2024-11-04 8:58 ` Shah, Amit
2024-11-04 16:11 ` Dave Hansen
2024-11-04 16:13 ` Shah, Amit
2024-11-04 16:26 ` Dave Hansen
2024-11-04 17:22 ` Shah, Amit
2024-11-04 17:45 ` Dave Hansen
2024-11-05 0:04 ` Andrew Cooper [this message]
2024-11-05 10:39 ` Shah, Amit
2024-11-05 14:54 ` Kaplan, David
2024-11-05 16:19 ` Dave Hansen
2024-11-05 16:25 ` Shah, Amit
2024-11-04 5:40 ` [tip: x86/cpu] x86/bugs: Add " tip-bot2 for Amit Shah
2024-10-31 15:39 ` [PATCH 2/2] x86: kvm: svm: add support for ERAPS and FLUSH_RAP_ON_VMRUN Amit Shah
2024-10-31 23:13 ` Pawan Gupta
2024-11-01 4:14 ` kernel test robot
2024-11-01 9:26 ` Amit Shah
2024-11-04 5:18 ` Borislav Petkov
2024-11-04 11:16 ` Shah, Amit
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e5463e70-a0a6-4cce-9c2c-4a09c2e622ef@citrix.com \
--to=andrew.cooper3@citrix.com \
--cc=Amit.Shah@amd.com \
--cc=Babu.Moger@amd.com \
--cc=David.Kaplan@amd.com \
--cc=Sandipan.Das@amd.com \
--cc=Thomas.Lendacky@amd.com \
--cc=boris.ostrovsky@oracle.com \
--cc=bp@alien8.de \
--cc=corbet@lwn.net \
--cc=daniel.sneddon@linux.intel.com \
--cc=dave.hansen@intel.com \
--cc=dave.hansen@linux.intel.com \
--cc=hpa@zytor.com \
--cc=jpoimboe@kernel.org \
--cc=kai.huang@intel.com \
--cc=kvm@vger.kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=pawan.kumar.gupta@linux.intel.com \
--cc=pbonzini@redhat.com \
--cc=peterz@infradead.org \
--cc=seanjc@google.com \
--cc=tglx@linutronix.de \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®