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

On Mon, Oct 07, 2019 at 09:57:07PM -0500, speck for Josh Poimboeuf wrote:
> 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?

I am not sure about this.

> 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.

Ok.

Thanks,
Pawan