[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [Xen-devel] [PATCH] warning on printf format


  • To: "aq" <aquynh@xxxxxxxxx>, "List: Xen Developers" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: "Ian Pratt" <m+Ian.Pratt@xxxxxxxxxxxx>
  • Date: Sun, 14 Aug 2005 18:39:03 +0100
  • Delivery-date: Sun, 14 Aug 2005 17:37:24 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcWg9qUVNLh6IfO+SqebhddGzQKjtQAAD9yQ
  • Thread-topic: [Xen-devel] [PATCH] warning on printf format

 
> This patch fixes a warning message on printf format.

It's already fixed in the latest tree. You need to use PRIpte as the
format string when printing PTEs in code that is shared between x86_32
and PAE or x86-64.

Ian
 
> Signed-off-by: Nguyen Anh Quynh <aquynh@xxxxxxxxx>
> 
> 
> diff -r 3fe7b0b7f6c5 xen/arch/x86/mm.c
> --- a/xen/arch/x86/mm.c Sun Aug 14 09:17:54 2005
> +++ b/xen/arch/x86/mm.c Mon Aug 15 02:33:21 2005
> @@ -3054,7 +3054,7 @@
>           */
>          BUG();
>      }
> -    PTWR_PRINTK("[%c] disconnected_l1va at %p is %lx\n",
> +    PTWR_PRINTK("[%c] disconnected_l1va at %p is %x\n",
>                  PTWR_PRINT_WHICH, ptep, pte.l1);
>      l1e_remove_flags(pte, _PAGE_RW);
>  
> @@ -3072,7 +3072,7 @@
>      /* Ensure that there are no stale writable mappings in 
> any TLB. */
>      /* NB. INVLPG is a serialising instruction: flushes 
> pending updates. */
>      flush_tlb_one_mask(d->cpumask, l1va);
> -    PTWR_PRINTK("[%c] disconnected_l1va at %p now %lx\n",
> +    PTWR_PRINTK("[%c] disconnected_l1va at %p now %x\n",
>                  PTWR_PRINT_WHICH, ptep, pte.l1);
>  
>      /*
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.