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

RE: [Xen-ia64-devel] [PATCH] Remove the last compile warnings


  • To: "Alex Williamson" <alex.williamson@xxxxxx>
  • From: "Zhang, Xiantao" <xiantao.zhang@xxxxxxxxx>
  • Date: Tue, 14 Mar 2006 13:30:37 +0800
  • Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
  • Delivery-date: Tue, 14 Mar 2006 05:31:56 +0000
  • List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
  • Thread-index: AcZHHzxBCIN5bH82RJepX2JzB8id2AACCF0g
  • Thread-topic: [Xen-ia64-devel] [PATCH] Remove the last compile warnings

Hi Alex,
        Sorry, I remembered that I have added it to repository, but didn't 
diffed to the patch file. So I attached it. I think it is necessary to add this 
header file to solve some warnings and for future extension.
Thanks
-Xiantao

> -----Original Message-----
> From: Alex Williamson [mailto:alex.williamson@xxxxxx]
> Sent: 2006年3月14日 12:25
> To: Zhang, Xiantao
> Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [Xen-ia64-devel] [PATCH] Remove the last compile warnings
> 
> On Sun, 2006-03-12 at 15:29 +0800, Zhang, Xiantao wrote:
> > Hi Dan/Alex
> >     This patch intends to remove the last warnings(C code) when
> > compile. Currently due to lack of header files corresponding to the c
> > file, some function declarations have to be placed in the file which
> > refers to them. It seems that we should add some header files to solve
> > it in future.
> 
> Hi Xiantao,
> 
>    Looks like a good start, but it doesn't build for me:
> 
> gcc -nostdinc -fno-builtin -fno-common -fno-strict-aliasing -O2
> -iwithprefix include -Wall -fomit-frame-pointer -I/opt/xen/include
> -D__KERNEL__ -I/opt/xen/include/asm-ia64
> -I/opt/xen/include/asm-ia64/linux -I/opt/xen/include/asm-ia64/linux
> -I/opt/xen/include/asm-ia64/linux-xen
> -I/opt/xen/include/asm-ia64/linux-null -I/opt/xen/arch/ia64/linux
> -I/opt/xen/arch/ia64/linux-xen -DIA64 -DXEN -DLINUX_2_6
> -DV_IOSAPIC_READY -ffixed-r13 -mfixed-range=f12-f15,f32-f127 -g -g
> -D__XEN__ -DNDEBUG -c vmx/vmx_init.c -o vmx_init.o
> vmx/vmx_init.c:54:25: asm/vlsapic.h: No such file or directory
> vmx/vmx_init.c: In function `vmx_setup_platform':
> vmx/vmx_init.c:440: warning: implicit declaration of function
> `vmx_virq_line_init'
> make[1]: *** [vmx_init.o] Error 1
> 
> Was vlsapic.h supposed to be added with this patch?  It builds w/o the
> include added to vmx_init.c.  Thanks,
> 
>       Alex
> 
> --
> Alex Williamson                             HP Linux & Open Source Lab

Attachment: vlsapic.h
Description: vlsapic.h

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

 


Rackspace

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