[MODERATED] Re: [PATCH v5 09/11] TAAv5 9

On Fri, Oct 04, 2019 at 11:34:31PM -0700, speck for Pawan Gupta wrote:
> Transactional Synchronization Extensions (TSX) is an extension to the
> x86 instruction set architecture (ISA) that adds Hardware Transactional
> Memory (HTM) support. Changing TSX state currently requires a reboot.
> This may not be desirable when rebooting imposes a huge penalty.

This is still missing a real world justification for the added
complexity.  Don't production users typically know at boot time whether
they plan to use TSX?

It looks like this patch should be combined with patch 10, since
otherwise this patch causes a regression in the sysfs mitigation
reporting, right?

And again... this patch needs to be last.  Even after the documentation.
If it triggers documentation changes then those changes should be
separated out and included in this patch.  Otherwise, this patch may
block the merging of the documentation patch.

-- 
Josh