spapr_pci: s/INT64_MAX/UINT64_MAX/
authorMichael S. Tsirkin <mst@redhat.com>
Wed, 6 Nov 2013 18:25:21 +0000 (20:25 +0200)
committerMichael S. Tsirkin <mst@redhat.com>
Tue, 10 Dec 2013 10:29:56 +0000 (12:29 +0200)
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 <mst@redhat.com>
Acked-by: Alexander Graf <agraf@suse.de>
hw/ppc/spapr_pci.c

index edb4cb04132cb80ed3e7eff126a3ac6115815c8b..2beedd45e958111c7bae8cfd6ae175db9b53fafa 100644 (file)
@@ -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),