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;
Date:   Wed, 21 Jun 2023 10:31:35 -0700
From:   Pawan Gupta <pawan.kumar.gupta@linux.intel.com>
To:     Breno Leitao <leitao@debian.org>
Cc:     Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>,
        Dave Hansen <dave.hansen@linux.intel.com>, x86@kernel.org,
        "H. Peter Anvin" <hpa@zytor.com>,
        Peter Zijlstra <peterz@infradead.org>,
        Josh Poimboeuf <jpoimboe@kernel.org>, leit@fb.com,
        "open list:X86 ARCHITECTURE (32-BIT AND 64-BIT)" 
        <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2] x86/bugs: Break down mitigations configurations
Message-ID: <20230621173135.wiprtgzslhw5z5or@desk>
References: <20230616164851.2559415-1-leitao@debian.org>
 <20230621001327.qdyebewnx7r5aiy3@desk>
 <ZJMdKUr98H8zPSAl@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <ZJMdKUr98H8zPSAl@gmail.com>
Precedence: bulk

On Wed, Jun 21, 2023 at 08:54:17AM -0700, Breno Leitao wrote:
> On Tue, Jun 20, 2023 at 05:13:27PM -0700, Pawan Gupta wrote:
> > On Fri, Jun 16, 2023 at 09:48:50AM -0700, Breno Leitao wrote:
> > > There is no way to disable MDS, TAA, MMIO Stale data mitigation today at
> > > compilation time. These mitigations are enabled even if
> > > CONFIG_SPECULATION_MITIGATIONS is unset.
> > > 
> > > Create a new KCONFIG option for each mitigation under
> > > CONFIG_SPECULATION_MITIGATIONS that allows these
> > > mitigations to be disabled by default at compilation time.
> > 
> > I don't think all mitigations are still controllable at build-time e.g.
> > spectre_v2 eIBRS mitigation will still be deployed irrespective of the
> > config.
> 
> Right. This patchset only cares about MDS, TAA and MMIO. I am more than
> happy to send a new patch to also disable spectre_v2 eIBRS.

What about Retbleed, L1TF, SRBDS etc? I thought the goal is to control
all speculation mitigations?

To be consistent CONFIG_SPECULATION_MITIGATIONS should control all
speculation mitigations.