[MODERATED] Re: [PATCH v7 10/10] TAAv7 10
Borislav Petkov <[email protected]> Wed, 23 Oct 2019 09:24:48 +0200
| Newsgroups | org.kernel.lore.historical-speck |
|---|---|
| Message-ID | <[email protected]> |
On Tue, Oct 22, 2019 at 01:39:20PM -0500, speck for Josh Poimboeuf wrote:
> Regardless, this patch adds complexity and still seems very fragile. It
> will add maintenance issues and probably introduce bugs. And, most
> importantly there's *still* not a real world justification for it.
> Please drop it from the series.
Yeah, and here's nasty use case if we did the runtime toggle thing: you
boot with TSX on, start guests, they see TSX and start using it and then
you disable it in the host.
I believe I read somewhere Andrew saying it won't #UD but it would abort
all transactions, which is still not nice and leaves the burden on the
sysadmin explaining why guests experience perf penatly all of a sudden.
Not good.
--
Regards/Gruss,
Boris.
SUSE Software Solutions Germany GmbH, GF: Felix Imendörffer, HRB 36809, AG Nürnberg
--