On Thu, 13 Dec 2012, Sander Eikelenboom wrote:
> Signed-off-by: Sander Eikelenboom <li...@eikelenboom.it>
please wrap around the lines > 80 characters.
> hw/xen_pt.c | 4 ++--
> xen-all.c | 4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/hw/xen_pt.c b/hw/xen_pt.c
> index 7a3846e..ca42a14 100644
> --- a/hw/xen_pt.c
> +++ b/hw/xen_pt.c
> @@ -671,7 +671,7 @@ static int xen_pt_initfn(PCIDevice *d)
> s->is_virtfn = s->real_device.is_virtfn;
> if (s->is_virtfn) {
> XEN_PT_LOG(d, "%04x:%02x:%02x.%d is a SR-IOV Virtual Function\n",
> - s->real_device.domain, bus, slot, func);
> + s->real_device.domain, s->real_device.bus,
> s->real_device.dev, s->real_device.func);
> }
>
> /* Initialize virtualized PCI configuration (Extended 256 Bytes) */
> @@ -752,7 +752,7 @@ out:
> memory_listener_register(&s->memory_listener, &address_space_memory);
> memory_listener_register(&s->io_listener, &address_space_io);
> XEN_PT_LOG(d, "Real physical device %02x:%02x.%d registered
> successfuly!\n",
> - bus, slot, func);
> + s->hostaddr.bus, s->hostaddr.slot, s->hostaddr.function);
>
> return 0;
> }
> diff --git a/xen-all.c b/xen-all.c
> index 046cc2a..d225b9d 100644
> --- a/xen-all.c
> +++ b/xen-all.c
> @@ -292,7 +292,7 @@ static int xen_add_to_physmap(XenIOState *state,
> return -1;
>
> go_physmap:
> - DPRINTF("mapping vram to %llx - %llx\n", start_addr, start_addr + size);
> + DPRINTF("mapping vram to %"PRI_xen_pfn" - %"PRI_xen_pfn"\n", start_addr,
> start_addr + size);
given that start_addr and size are hwaddr, these should be HWADDR_PRIx
> pfn = phys_offset >> TARGET_PAGE_BITS;
> start_gpfn = start_addr >> TARGET_PAGE_BITS;
> @@ -365,7 +365,7 @@ static int xen_remove_from_physmap(XenIOState *state,
> phys_offset = physmap->phys_offset;
> size = physmap->size;
>
> - DPRINTF("unmapping vram to %llx - %llx, from %llx\n",
> + DPRINTF("unmapping vram to %"PRI_xen_pfn" - %"PRI_xen_pfn", from
> %"PRI_xen_pfn"\n",
> phys_offset, phys_offset + size, start_addr);
same here
