Re: Problem adding new source files

[Uri Lublin <uril@qumranet.com>]

Hacking, Stuart wrote:
>> From: Uri Lublin [mailto:uril@qumranet.com] 
>> Hacking, Stuart wrote:
>>> Hello,
>> Hi,
>>>  
>>> We have been doing some experimentation with modifications to the 
>>> migration code in the Qemu and came up against a problem.  
>> We included 
>>> some code in a different file and are receiving the following error 
>>> from
>>> make:
>>>  
>>> ---
>>> Migration.o: In function `migrate_prepare_page':
>>> /root/tmp/KVM/qemu/migration.c:367: undefined reference to 
>>> `get_cached_page'
>>> /root/tmp/KVM/qemu/migration.c:367: undefined reference to 
>>> `get_cached_page'
>>> /root/tmp/KVM/qemu/migration.c:367: undefined reference to 
>>> `get_cached_page'
>>> collect2: ld returned 1 exit status
>>> Make[2]: *** [qemu-system_x86_64] Error 1
>>> Make[1]: *** [subdir-x86_64-softmmu] Error2
>>> Make: *** [qemu] Error 2
>>> ---
>> Did you define new functions in a different .c file(s) ? Did 
>> you provide prototypes for all new functions ? Did you add 
>> all new files to the Makefile ?
>>
> 
> As far as I know all the code is organised properly and the function
> prototypes provided in header files.  As for adding to the Makefile -
> that's where we are struggling.  We have tried the following
> 'experiments':  adding our new source files to the 'OBJS' variable
> (OBJS+=s1.o s2.o); creating a migration.o directive which depends on
> s1.o and s2.o (this actually produces a slightly different error:
> 

Try just adding your new .o files (e.g: OBJS+=s1.o s2.o) to 
<kvmdir>/qemu/Makefile.target