[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] VT-d: fix and extend RMRR reservation check

  • To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
  • Date: Mon, 9 Mar 2020 06:27:12 +0000
  • Accept-language: en-US
  • Delivery-date: Mon, 09 Mar 2020 06:28:37 +0000
  • Dlp-product: dlpe-windows
  • Dlp-reaction: no-action
  • Dlp-version:
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHV7V8eTQ4+CKubykyXA8xZd0Ltgag/3K0Q
  • Thread-topic: [PATCH] VT-d: fix and extend RMRR reservation check

> From: Jan Beulich <jbeulich@xxxxxxxx>
> Sent: Thursday, February 27, 2020 7:15 PM
> First of all in commit d6573bc6e6b7 ("VT-d: check all of an RMRR for
> being E820-reserved") along with changing the function used, the enum-
> like value passed should have been changed too (to E820_*). Do so now.
> (Luckily the actual values of RAM_TYPE_RESERVED and E820_RESERVED
> match, so the breakage introduced was "only" latent.)
> Furthermore one of my systems surfaces RMRR in an ACPI NVS E820 range.
> The purpose of the check is just to make sure there won't be "ordinary"
> mappings of these ranges, and domains (including Dom0) won't want to
> use the region to e.g. put PCI device BARs there. The two ACPI related
> E820 types are good enough for this purpose, so allow them as well.
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Reviewed-by: Kevin Tian <kevin.tian@xxxxxxxxx>
Xen-devel mailing list



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