Re: [Xen-devel] [PATCH] Fix compile errors when enabling Xen debug logging.

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Thu, 13 Dec 2012, Sander Eikelenboom wrote:
> Signed-off-by: Sander Eikelenboom <linux@xxxxxxxxxxxxxx>

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@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel