From: Michael S. Tsirkin Date: Wed, 6 Nov 2013 18:25:21 +0000 (+0200) Subject: spapr_pci: s/INT64_MAX/UINT64_MAX/ X-Git-Tag: v2.0.0-rc0~181^2~20 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=92b8e39c7f582e15f9e9423bc9fd3f186536b073;p=thirdparty%2Fqemu.git spapr_pci: s/INT64_MAX/UINT64_MAX/ It doesn't make sense for a region to be INT64_MAX in size: memory core uses UINT64_MAX as a special value meaning "all 64 bit" this is what was meant here. While this should never affect the spapr system which at the moment always has < 63 bit size, this makes us hit all kind of corner case bugs with sub-pages, so users are probably better off if we just use UINT64_MAX instead. Signed-off-by: Michael S. Tsirkin Acked-by: Alexander Graf --- diff --git a/hw/ppc/spapr_pci.c b/hw/ppc/spapr_pci.c index edb4cb04132..2beedd45e95 100644 --- a/hw/ppc/spapr_pci.c +++ b/hw/ppc/spapr_pci.c @@ -555,7 +555,7 @@ static int spapr_phb_init(SysBusDevice *s) /* Initialize memory regions */ sprintf(namebuf, "%s.mmio", sphb->dtbusname); - memory_region_init(&sphb->memspace, OBJECT(sphb), namebuf, INT64_MAX); + memory_region_init(&sphb->memspace, OBJECT(sphb), namebuf, UINT64_MAX); sprintf(namebuf, "%s.mmio-alias", sphb->dtbusname); memory_region_init_alias(&sphb->memwindow, OBJECT(sphb),