From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.linutronix.de (193.142.43.55:993) by crypto-ml.lab.linutronix.de with IMAP4-SSL for ; 15 Oct 2019 23:13:25 -0000 Received: from mx1.redhat.com ([209.132.183.28]) by Galois.linutronix.de with esmtps (TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80) (envelope-from ) id 1iKW0J-0001vX-B7 for speck@linutronix.de; Wed, 16 Oct 2019 01:13:23 +0200 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 9A04F3CBC3 for ; Tue, 15 Oct 2019 23:13:15 +0000 (UTC) Received: from treble (ovpn-120-223.rdu2.redhat.com [10.10.120.223]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 119BB60852 for ; Tue, 15 Oct 2019 23:13:09 +0000 (UTC) Date: Tue, 15 Oct 2019 18:12:52 -0500 From: Josh Poimboeuf Subject: [MODERATED] Re: [PATCH v5 08/11] TAAv5 8 Message-ID: <20191015231252.kggxh6ffrciz2dfy@treble> References: <20191015103454.GW317@dhcp22.suse.cz> <20191015130627.7jkhqy2zrtm35ool@treble> <20191015152649.yim4krwuttrh6xgi@treble> <20191015200024.hxs4brxi7gbvmcdy@treble> <20191015205631.GF30412@guptapadev.amr> MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit To: speck@linutronix.de List-ID: On Tue, Oct 15, 2019 at 11:14:03PM +0200, speck for Jiri Kosina wrote: > On Tue, 15 Oct 2019, speck for Pawan Gupta wrote: > > > tsx_ctrl_is_supported() checks for the TSX_CTRL MSR support and then > > only attempts at disabling TSX. MDS_NO=0 CPUs will not get the ucode > > update for TSX control, therefore tsx=auto wont cause regression on > > older CPUs. So just to clarify, CPUs with TAA_BUG and MDS_NO=0 will *not* have ARCH_CAP_TSX_CTRL_MSR? I didn't see that important detail mentioned anywhere in the patches. At the very least, the documentation for tsx=auto and tsx=off need to be clarified, as they don't seem to mention any MDS_NO=0 contingencies. > OK, that piece of information finally made it to make sense again :) > > So I believe distros still want the option (Michal's patch) to default to > 'auto', so that actual heavy users of TSX will get the right thing once > they update their CPUs to !TAA_BUG ones, but it's less urgent that I > originally thought. So if I understand correctly, you're postulating that distros want: a) TAA_BUG && MDS_NO=0 => TSX on b) TAA_BUG && MDS_NO=1 => TSX off c) !TAA_BUG => TSX on How are you reaching that conclusion? It seems horribly confusing for TSX users, but again maybe I'm missing something. It seems to me that "heavy users of TSX" would want tsx=on, no matter what. And so we would need to leave that as the default in order to not break those users. -- Josh