Re: [Xen-devel] [PATCH v2 1/4] x86/PVHv2: Add memory map pointer to hvm_start_info struct

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 14.03.18 at 00:31, <maran.wilson@xxxxxxxxxx> wrote:
> + * For x86 implementations at least, the values used in the type field will
> + * match the Address Range Types as defined in section 15 (System Address
> + * Map Interfaces) of the ACPI Specification (http://uefi.org/specifications)
> + * where:
> + *     AddressRangeMemory = 1 (E820_RAM)
> + *     AddressRangeReserved = 2 (E820_RESERVED)
> + *     AddressRangeACPI = 3 (E820_ACPI)
> + *     AddressRangeNVS = 4 (E820_NVS)
> + *     AddressRangeUnusable = 5 (E820_UNUSABLE)
> + *     AddressRangeDisabled = 6 (E820_DISABLED)
> + *     AddressRangePersistentMemory = 7 (E820_PMEM)

Would you mind waiting for a discussion to settle before sending
out new patch versions? As indicated in an earlier reply to v1, I
consider this still insufficient. And no, I'm not asking for you to
add redundant and potentially conflicting definitions of E820_*,
but instead you want to use Xen specific ones (prefixed e.g.
by XEN_HVM_MEMMAP_TYPE_).

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel