[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCHv3 5/7] spapr: Adjust placement of PCI host bridge
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-ppc] [PATCHv3 5/7] spapr: Adjust placement of PCI host bridge to allow > 1TiB RAM |
Date: |
Thu, 13 Oct 2016 09:35:32 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 |
On 13/10/2016 01:57, David Gibson wrote:
> Currently the default PCI host bridge for the 'pseries' machine type is
> constructed with its IO windows in the 1TiB..(1TiB + 64GiB) range in
> guest memory space. This means that if > 1TiB of guest RAM is specified,
> the RAM will collide with the PCI IO windows, causing serious problems.
>
> Problems won't be obvious until guest RAM goes a bit beyond 1TiB, because
> there's a little unused space at the bottom of the area reserved for PCI,
> but essentially this means that > 1TiB of RAM has never worked with the
> pseries machine type.
>
> This patch fixes this by altering the placement of PHBs on large-RAM VMs.
> Instead of always placing the first PHB at 1TiB, it is placed at the next
> 1 TiB boundary after the maximum RAM address.
>
> Technically, this changes behaviour in a migration-breaking way for
> existing machines with > 1TiB maximum memory, but since having > 1 TiB
> memory was broken anyway, this seems like a reasonable trade-off.
>
> Signed-off-by: David Gibson <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
> ---
> hw/ppc/spapr.c | 16 ++++++++++++++--
> 1 file changed, 14 insertions(+), 2 deletions(-)
>
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index cb9da96..e6b110d 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -2375,15 +2375,27 @@ static void spapr_phb_placement(sPAPRMachineState
> *spapr, uint32_t index,
> unsigned n_dma, uint32_t *liobns, Error
> **errp)
> {
> const uint64_t base_buid = 0x800000020000000ULL;
> - const hwaddr phb0_base = 0x10000000000ULL; /* 1 TiB */
> const hwaddr phb_spacing = 0x1000000000ULL; /* 64 GiB */
> const hwaddr mmio_offset = 0xa0000000; /* 2 GiB + 512 MiB */
> const hwaddr pio_offset = 0x80000000; /* 2 GiB */
> const uint32_t max_index = 255;
> + const hwaddr phb0_alignment = 0x10000000000ULL; /* 1 TiB */
>
> - hwaddr phb_base;
> + uint64_t ram_top = MACHINE(spapr)->ram_size;
> + hwaddr phb0_base, phb_base;
> int i;
>
> + /* Do we have hotpluggable memory? */
> + if (MACHINE(spapr)->maxram_size > ram_top) {
> + /* Can't just use maxram_size, because there may be an
> + * alignment gap between normal and hotpluggable memory
> + * regions */
> + ram_top = spapr->hotplug_memory.base +
> + memory_region_size(&spapr->hotplug_memory.mr);
> + }
> +
> + phb0_base = QEMU_ALIGN_UP(ram_top, phb0_alignment);
> +
> if (index > max_index) {
> error_setg(errp, "\"index\" for PAPR PHB is too large (max %u)",
> max_index);
>
- [Qemu-ppc] [PATCHv3 0/7] Improve PCI IO window orgnaization for pseries, David Gibson, 2016/10/12
- [Qemu-ppc] [PATCHv3 2/7] libqos: Correct error in PCI hole sizing for spapr, David Gibson, 2016/10/12
- [Qemu-ppc] [PATCHv3 5/7] spapr: Adjust placement of PCI host bridge to allow > 1TiB RAM, David Gibson, 2016/10/12
- Re: [Qemu-ppc] [PATCHv3 5/7] spapr: Adjust placement of PCI host bridge to allow > 1TiB RAM,
Laurent Vivier <=
- [Qemu-ppc] [PATCHv3 3/7] libqos: Limit spapr-pci to 32-bit MMIO for now, David Gibson, 2016/10/12
- [Qemu-ppc] [PATCHv3 4/7] spapr_pci: Delegate placement of PCI host bridges to machine type, David Gibson, 2016/10/12
- [Qemu-ppc] [PATCHv3 6/7] spapr_pci: Add a 64-bit MMIO window, David Gibson, 2016/10/12
- [Qemu-ppc] [PATCHv3 1/7] libqos: Isolate knowledge of spapr memory map to qpci_init_spapr(), David Gibson, 2016/10/12
- [Qemu-ppc] [PATCHv3 7/7] spapr: Improved placement of PCI host bridges in guest memory map, David Gibson, 2016/10/12