[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



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.