Re: [Xen-devel] Re: Xen 4.0.1 failed to compile on Ubuntu Natty 32 bits

[Martinx - ジェームズ <thiagocmartinsc@xxxxxxxxx>]

Ian,

ÂI chenge the files but, the error persist, check it out:

root@natty02:/usr/src/xen-4.0.1# head tools/blktap/drivers/blk_linux.c 
#include <inttypes.h>
#include <sys/ioctl.h>

#include <sys/mount.h>
#include "tapdisk.h"
#include "blk.h"

int blk_getimagesize(int fd, uint64_t *size)
{
ÂÂÂ int rc;

root@natty02:/usr/src/xen-4.0.1# make clean ; make tools

....
make[3]: Leaving directory `/usr/src/xen-4.0.1/tools/memshr'
make[2]: Leaving directory `/usr/src/xen-4.0.1/tools'
make[2]: Entering directory `/usr/src/xen-4.0.1/tools'
make -C blktap install

make[3]: Entering directory `/usr/src/xen-4.0.1/tools/blktap'
make[4]: Entering directory `/usr/src/xen-4.0.1/tools/blktap'
make -C lib install
make[5]: Entering directory `/usr/src/xen-4.0.1/tools/blktap/lib'

../../../tools/cross-install -d -m0755 -p /usr/src/xen-4.0.1/dist/install/usr/lib64
../../../tools/cross-install -d -m0755 -p /usr/src/xen-4.0.1/dist/install/usr/include
../../../tools/cross-install -m0644 -p libblktap.a libblktap.so.3.0.0 /usr/src/xen-4.0.1/dist/install/usr/lib64

ln -sf libblktap.so.3.0.0 /usr/src/xen-4.0.1/dist/install/usr/lib64/libblktap.so.3.0
ln -sf libblktap.so.3.0 /usr/src/xen-4.0.1/dist/install/usr/lib64/libblktap.so
../../../tools/cross-install -m0644 -p blktaplib.h /usr/src/xen-4.0.1/dist/install/usr/include

make[5]: Leaving directory `/usr/src/xen-4.0.1/tools/blktap/lib'
make[4]: Leaving directory `/usr/src/xen-4.0.1/tools/blktap'
make[4]: Entering directory `/usr/src/xen-4.0.1/tools/blktap'
make -C drivers install

make[5]: Entering directory `/usr/src/xen-4.0.1/tools/blktap/drivers'
Makefile:23: === libgcrypt not installed: falling back to libcrypto ===
gcc -O2 -fomit-frame-pointer -m64 -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes -Wno-unused-value -Wdeclaration-after-statement -D__XEN_TOOLS__ -MMD -MF .blktapctrl.d -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Werror -Wno-unused -I../lib -I../../../tools/libxc -I../../../tools/include -I../../../tools/xenstore -I../../../tools/include -I ../../libaio/src -I ../../memshr -D_GNU_SOURCE -DMEMSHR -o blktapctrl blktapctrl.o blktapctrl_linux.o -L../../../tools/libxc -lxenctrl -L../../../tools/xenstore -lxenstore ../../memshr/libmemshr.a -L../lib -lblktap -lrt -lm -lpthread

../../memshr/libmemshr.a(interface.o): In function `memshr_vbd_initialize':
/usr/src/xen-4.0.1/tools/memshr/interface.c:116: undefined reference to `xc_interface_open'
../../memshr/libmemshr.a(interface.o): In function `memshr_vbd_issue_ro_request':

/usr/src/xen-4.0.1/tools/memshr/interface.c:165: undefined reference to `xc_memshr_nominate_gref'
/usr/src/xen-4.0.1/tools/memshr/interface.c:179: undefined reference to `xc_memshr_share'
collect2: ld returned 1 exit status

make[5]: *** [blktapctrl] Error 1
make[5]: Leaving directory `/usr/src/xen-4.0.1/tools/blktap/drivers'
make[4]: *** [subdir-install-drivers] Error 2
make[4]: Leaving directory `/usr/src/xen-4.0.1/tools/blktap'

make[3]: *** [subdirs-install] Error 2
make[3]: Leaving directory `/usr/src/xen-4.0.1/tools/blktap'
make[2]: *** [subdir-install-blktap] Error 2
make[2]: Leaving directory `/usr/src/xen-4.0.1/tools'
make[1]: *** [subdirs-install] Error 2

make[1]: Leaving directory `/usr/src/xen-4.0.1/tools'
make: *** [install-tools] Error 2

Thank you!
Thiago

2010/12/16 Ian Campbell <Ian.Campbell@xxxxxxxxxx>

On Wed, 2010-12-15 at 22:15 +0000, Martinx - ããããã wrote:
>
> Â Â Â Â gcc Â-O1 -fno-omit-frame-pointer -fno-optimize-sibling-calls
> Â Â Â Â -m32 -march=i686 -g -fno-strict-aliasing -std=gnu99 -Wall
> Â Â Â Â -Wstrict-prototypes -Wno-unused-value
> Â Â Â Â -Wdeclaration-after-statement Â-D__XEN_TOOLS__ -MMD
> Â Â Â Â -MF .blk_linux.o.d -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64
> Â Â Â Â -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE
> Â Â Â Â -mno-tls-direct-seg-refs -Werror -Wno-unused -I../lib
> Â Â Â Â -I../../../tools/libxc -I../../../tools/include
> Â Â Â Â -I../../../tools/xenstore -I../../../tools/include
> Â Â Â Â -I ../../libaio/src -I ../../memshr -D_GNU_SOURCE -DMEMSHR -c
> Â Â Â Â -o blk_linux.o blk_linux.c
> Â Â Â Â cc1: warnings being treated as errors
> Â Â Â Â In file included from tapdisk.h:62:0,
> Â Â Â Â Â Â Â Â Â Â Â Â Âfrom blk_linux.c:4:
> Â Â Â Â ../lib/blktaplib.h:199:0: error: "WRITE" redefined
> Â Â Â Â /usr/include/linux/fs.h:160:0: note: this is the location of
> Â Â Â Â the previous definition

The headers on natty appear to include a definition of WRITE which
conflicts with the use of that name in blktap.

This should avoid the issue.

Ian.

8<---------------------------------------

# HG changeset patch
# User Ian Campbell <ian.campbell@xxxxxxxxxx>
# Date 1292492034 0
# Node ID 0236d8adb081b93e509ec52654201ce88ad6570b
# Parent Âd70d7c3311c495305ac1bde256122b09c58463e5
blktap[12]: include <sys/mount.h> instead of <linux/fs.h>

The former is a userspace sanitised header which contains the
definitions we need. In some distros linux/fs.h defines WRITE which
conflicts with blktaps own use of that name.

Also there is no reason to use <linux/errno.h> over the more normal
<errno.h>.

Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

diff -r d70d7c3311c4 -r 0236d8adb081 tools/blktap/drivers/blk_linux.c
--- a/tools/blktap/drivers/blk_linux.c ÂTue Dec 14 17:10:49 2010 +0000
+++ b/tools/blktap/drivers/blk_linux.c ÂThu Dec 16 09:33:54 2010 +0000
@@ -1,6 +1,6 @@
Â#include <inttypes.h>
Â#include <sys/ioctl.h>
-#include <linux/fs.h>
+#include <sys/mount.h>
Â#include "tapdisk.h"
Â#include "blk.h"

diff -r d70d7c3311c4 -r 0236d8adb081 tools/blktap2/drivers/blk_linux.c
--- a/tools/blktap2/drivers/blk_linux.c Tue Dec 14 17:10:49 2010 +0000
+++ b/tools/blktap2/drivers/blk_linux.c Thu Dec 16 09:33:54 2010 +0000
@@ -1,7 +1,7 @@
Â#include <inttypes.h>
+#include <errno.h>
Â#include <sys/ioctl.h>
-#include <linux/fs.h>
-#include <linux/errno.h>
+#include <sys/mount.h>
Â#include "tapdisk.h"
Â#include "blk.h"

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