[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] x86: Add MMIO Stale Data arch_caps to hardware domain
- To: Jason Andryuk <jandryuk@xxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Mon, 18 Jul 2022 16:18:09 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=5jl2yXiaCydCHyslEEvnLSLsXVm32pHu4jDvcXmC7u4=; b=ezZrLJ0trKVeyAfnPgqHUYmdtpbrQTfAEsdHNykNT2TgyvMmdCy3HMDRd3XzCCxYg0L+OFrIOWATMl4eETZrlJMvOrbRFFp/wANh8GHSdGShBJRtBghffCJPvGKl17l/R2SJUOzBDnBQeinQa1+pDDraLvINoxFoTtqz4h1fKGL8YDFShWmhnmuMFpGB2xCTa02axmmal39ox97eF7D0ez2NHi2wB8OhfWXvKEqy+Uit6WCzbDHbkpri/rW64U20on9Bix1kA4kvQWf5KzO0MaHc6GHuOtP11dl2wiruWLnVEqCd/M94zWddYrZdlPSHilH3zot2iy+wn7oUQfwxVg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hLcKxY4DT3Raw0QB1TqdPmH40fYRY3XIX97VLDFEq/8OF96++CuB2H+zpWJPH1idUteIUoTd8J+YM7qpIkElZAzkcrqUl7eRAVsKRuKJ4YvjnKrUoFnBv+fDsgrxhFJtY4vZ1ABxL0ACmB3HYmhlWDMzZxXr69zBDFUraonZhxD2M+anc9j/ktxzFVQpoMHGN3pYIZEz4riU1DzwdTvGU2cCX7262LY62ADX8I0f7kzDOHS2/mGTM7dRU2J+XJH48zA4livdSclmx4bh42K2K2cznrOUCbLgFqc4rgqa9R2iTlD0AIgeo+nffU18XUWIimqajGo6RdasdeURAKp7fg==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
- Delivery-date: Mon, 18 Jul 2022 14:18:21 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 18.07.2022 16:05, Jason Andryuk wrote:
> Let the hardware domain know about the hardware it is running on. This
> allows a linux Dom0 to know that it has the appropriate microcode via
> FB_CLEAR. /sys/devices/system/cpu/vulnerabilities/mmio_stale_data
> changes from:
> "Vulnerable: Clear CPU buffers attempted, no microcode; SMT Host state
> unknown"
> to:
> "Mitigation: Clear CPU buffers; SMT Host state unknown"
>
> Signed-off-by: Jason Andryuk <jandryuk@xxxxxxxxx>
> ---
> Should calculate_host_policy()'s arch_caps mask also be updated? They
> are not identical today, but I'm don't know this code to understand why
> they differ.
I think this wants updating too, yes. I'm afraid I have to leave it to
Andrew to provide the reasons for the differences between the two.
I would further suggest to also consider adding RRSBA and BHI_NO, even
if then the title would want adjusting. And finally I'd like to ask to
add a proper Fixes: tag (or more), as it looks like the updating here
was simply forgotten when the bits were introduced. Ideally we'd have
a way for the compiler to remind us of updates being needed (or at
least be considered) here.
Jan
|