Re: [PATCH v5 08/11] TAAv5 8

On Mon, 14 Oct 2019, speck for Jiri Kosina wrote:

> On Mon, 14 Oct 2019, speck for Borislav Petkov wrote:
> 
> > > 	not everybody (by far) is reading those. If this regressess on a=20
> > > 	handful of platforms, we (together with the OS vendor) could=20
> > > 	probably handle the followup reports, but if it happens all across=20
> > > 	the board, it'd surely be overwhelming, to put it mildly. It'll"
> > > 	basically promptly escalate to a support disaster."

I feared that :)

> > Ok, how about we make it a config option: CONFIG_TSX_DEFAULT_ON
> >
> > and we - distro kernels - enable it. Mainline leaves it off.
> > 
> > Problem solved.
> > 
> > Hmmm?
> 
> If noone would strongly object to that config option, I'll happily send a 
> followup patch on top of Pawan's series that'd be adding it (defaulting to 
> 'n', as that appears to be the common consensus for upstream for totally 
> understandable reasons).
> 
> Otherwise, some distros might just have to flip the default in the sources 
> by non-upstream patch, I am afraid (even though we haven't made it to the 
> point where are are making that decision yet).

The config option is fine.

Thanks,

	tglx