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

Re: [Xen-devel] [PATCH VTPM v8 0/8] Vtpm patch set

Sigh... It works fine for me. 

What commands did you use to build? Are they the same as you showed before?

-----Original Message-----
From: Ian Campbell [mailto:Ian.Campbell@xxxxxxxxxx] 
Sent: Thursday, January 10, 2013 10:30 AM
To: Fioravante, Matthew E.
Cc: xen-devel@xxxxxxxxxxxxx
Subject: Re: [PATCH VTPM v8 0/8] Vtpm patch set

On Thu, 2013-01-03 at 15:37 +0000, Matthew Fioravante wrote:
> The following are the final set of patches for vtpm in xen.
> They are derived from the previous v7 and Ian's xen vtpm respository 
> with the various minor bug fixes we have discussed and fixed along the 
> way. The bug fixes are baked into their respective patches.


Unfortunately I'm afraid this fails to build for me when cmake is installed. I 

ld -nostdlib 
  -m elf_x86_64 -T arch/x86/minios-x86_64.lds \
/local/scratch/ianc/devel/committer.git/stubdom/mini-os-x86_64-vtpm/mini-os.o  \
ld: warning: section `.bss' type changed to PROGBITS
In function `mkdirs':
tpm_emulator_extern.c:(.text+0xbd95): undefined reference to `__errno_location'
tpm_emulator_extern.c:(.text+0xbdad): undefined reference to `__errno_location'
In function `_tpm_extern_init':
tpm_emulator_extern.c:(.text+0xbe7f): undefined reference to `open'
tpm_emulator_extern.c:(.text+0xbe9b): undefined reference to `__errno_location'
In function `_tpm_write_to_storage':
tpm_emulator_extern.c:(.text+0xc1e9): undefined reference to `open'
In function `_tpm_read_from_storage':
tpm_emulator_extern.c:(.text+0xc27b): undefined reference to `open'
tpm_emulator_extern.c:(.text+0xc2a2): undefined reference to `lseek'
tpm_emulator_extern.c:(.text+0xc2ba): undefined reference to `lseek'
make[2]: *** 
Error 1
make[2]: Leaving directory 

lseek and open are defined in extras/mini-os/lib/sys.c but:
$ nm stubdom/mini-os-x86_64-vtpm/lib/sys.o | egrep T.\(open\|lseek\)
0000000000002456 T lseek64
0000000000002a4e T open64

Perhaps some sort of -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE mismatch between 
the libc and the app libraries?

Not sure about the __errno_location one other than:

$ grep -B2 __errno_location stubdom/caml/main-caml.c
/* Ugly binary compatibility with Linux */ FILE *_stderr asm("stderr"); int 

Everything else seems to build ok. so without cmake installed it works fine.

> Matthew Fioravante (8):
>   add vtpm-stubdom code
>   add stubdom/vtpmmgr code
>   vtpm/vtpmmgr and required libs to stubdom/Makefile
>   Add vtpm documentation
>   README: Add cmake dependency
>   stubdom: Add autoconf
>   Add a top level configure script
>   Add conditional build of subsystems to configure.ac


Xen-devel mailing list



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