[MODERATED] Re: [PATCH v7 00/10] TAAv7 0

On 10/21/19 1:22 PM, speck for Pawan Gupta wrote:
> From: Pawan Gupta <[email protected]>
> Subject: [PATCH v7 00/10] TAAv7
> 
> Changes since v6:
> - Add Michal's patch to allow tsx=on|off|auto via CONFIG
> - Rebase to v5.4-rc4
> - Changelog, comments and documentation update.
> 
> Changes since v5:
> - Remove unsafe X86_FEATURE_RTM toggles.

I'm wondering if maybe these patches shouldn't touch the cpu
capabilities at all.  After all, even with TSX toggled off, TSX is still
present -- XBEGIN doesn't give #UD.  By making this change, we avoid
needing to even consider what happens when a cpu capability bit changes
after boot.

I think it would be nice to expose whether TSX is enabled somewhere in
sysfs, but I'm not convinced that "features" is the place for it.