Skip to content

Commit

Permalink
xen: fix two errors when debug is enabled
Browse files Browse the repository at this point in the history
This patch fixes:
1. build error in xen_pt.c when XEN_PT_LOGGING_ENABLED is defined
2. debug output format string error when DEBUG_XEN is defined

In the second case I also have the output info in consistent with the
output in mapping function - that is, print start_addr instead of
phys_offset.

Signed-off-by: Sander Eikelenboom <linux@eikelenboom.it>
Signed-off-by: Wei Liu <wei.liu2@citrix.com>
Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
  • Loading branch information
Wei Liu authored and Stefano Stabellini committed Dec 1, 2013
1 parent 7dc65c0 commit d18e173
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion hw/xen/xen_pt.c
Expand Up @@ -570,7 +570,8 @@ static void xen_pt_region_update(XenPCIPassthroughState *s,
if (args.rc) {
XEN_PT_WARN(d, "Region: %d (addr: %#"FMT_PCIBUS
", len: %#"FMT_PCIBUS") is overlapped.\n",
bar, sec->offset_within_address_space, sec->size);
bar, sec->offset_within_address_space,
int128_get64(sec->size));
}

if (d->io_regions[bar].type & PCI_BASE_ADDRESS_SPACE_IO) {
Expand Down
4 changes: 2 additions & 2 deletions xen-all.c
Expand Up @@ -369,8 +369,8 @@ static int xen_remove_from_physmap(XenIOState *state,
phys_offset = physmap->phys_offset;
size = physmap->size;

DPRINTF("unmapping vram to %"HWADDR_PRIx" - %"HWADDR_PRIx", from ",
"%"HWADDR_PRIx"\n", phys_offset, phys_offset + size, start_addr);
DPRINTF("unmapping vram to %"HWADDR_PRIx" - %"HWADDR_PRIx", at "
"%"HWADDR_PRIx"\n", start_addr, start_addr + size, phys_offset);

size >>= TARGET_PAGE_BITS;
start_addr >>= TARGET_PAGE_BITS;
Expand Down

0 comments on commit d18e173

Please sign in to comment.