Mailing List Archive

[PATCH] Fix compile errors when enabling Xen debug logging.
Signed-off-by: Sander Eikelenboom <linux@eikelenboom.it>
---
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);

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);

size >>= TARGET_PAGE_BITS;
--
1.7.2.5


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
Re: [PATCH] Fix compile errors when enabling Xen debug logging. [ In reply to ]
On Thu, 13 Dec 2012, Sander Eikelenboom wrote:
> Signed-off-by: Sander Eikelenboom <linux@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

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel