Re: [Qemu-devel] Staging update (0.12 pending freeze)

[Aurelien Jarno]

On Sun, Dec 06, 2009 at 03:44:59PM +0000, Blue Swirl wrote:
> On Sun, Dec 6, 2009 at 11:59 AM, Aurelien Jarno <address@hidden> wrote:
> > On Sat, Dec 05, 2009 at 08:07:13PM +0000, Blue Swirl wrote:
> >> On Sat, Dec 5, 2009 at 8:05 PM, Aurelien Jarno <address@hidden> wrote:
> >> > On Thu, Dec 03, 2009 at 10:03:18PM +0200, Blue Swirl wrote:
> >> >> On Thu, Dec 3, 2009 at 9:26 PM, Aurelien Jarno <address@hidden> wrote:
> >> >> > On Wed, Dec 02, 2009 at 10:46:11AM -0600, Anthony Liguori wrote:
> >> >> >> I've got all of the patches I'm considering for 0.12 currently in
> >> >> >> staging.  I'm going to work through and test/commit these in a few
> >> >> >> chunks over the next few days before freezing the tree.
> >> >> >>
> >> >> >
> >> >> > What are the plans on the OpenBIOS side? The version currently 
> >> >> > included
> >> >> > in QEMU is old compared to the SVN. Is it plan to sync a release of
> >> >> > OpenBIOS with QEMU?
> >> >>
> >> >> At least the images should be updated.
> >> >>
> >> >
> >> > Now that version 0.12.0-rca has been tagged, we should probably do that
> >> > asap. Should I do it?
> >>
> >> Please do.
> >>
> >
> > I have seen you have been faster than me, thanks.
> >
> > Anyway I am not able to fully build the powerpc images here,
> > openbios-unix fails to build with:
> >
> > | libmodules.a(elf-loader.o): In function `elf_loader_init_program':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/elf-loader.c:77: 
> > undefined reference to `flush_icache_range'
> > | libmodules.a(xcoff-loader.o): In function `xcoff_loader_init_program':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/xcoff-loader.c:110: 
> > undefined reference to `flush_icache_range'
> > | libmodules.a(ofmem_common.o): In function `ofmem_translate':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:680: 
> > undefined reference to `ofmem_arch_get_private'
> > | libmodules.a(ofmem_common.o): In function `ofmem_free':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:129: 
> > undefined reference to `ofmem_arch_get_private'
> > | libmodules.a(ofmem_common.o): In function `split_trans':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:505: 
> > undefined reference to `ofmem_arch_get_private'
> > | libmodules.a(ofmem_common.o): In function `ofmem_claim_virt_':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:422: 
> > undefined reference to `ofmem_arch_get_private'
> > | libmodules.a(ofmem_common.o): In function `ofmem_update_translations':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:256: 
> > undefined reference to `ofmem_arch_get_private'
> > | 
> > libmodules.a(ofmem_common.o):/home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:35:
> >  more undefined references to `ofmem_arch_get_private' follow
> > | libmodules.a(ofmem_common.o): In function `unmap_page_range':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:609: 
> > undefined reference to `ofmem_arch_unmap_pages'
> > | libmodules.a(ofmem_common.o): In function `ofmem_map_page_range':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:524: 
> > undefined reference to `ofmem_arch_get_private'
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:551: 
> > undefined reference to `ofmem_arch_unmap_pages'
> > | libmodules.a(ofmem_common.o): In function `ofmem_map':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:647: 
> > undefined reference to `ofmem_arch_default_translation_mode'
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:654: 
> > undefined reference to `ofmem_arch_early_map_pages'
> > | libmodules.a(ofmem_common.o): In function `ofmem_claim':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:456: 
> > undefined reference to `ofmem_arch_get_private'
> > | libmodules.a(ofmem_common.o): In function `get_ram_size':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:35: 
> > undefined reference to `ofmem_arch_get_private'
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:35: 
> > undefined reference to `ofmem_arch_get_private'
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:35: 
> > undefined reference to `ofmem_arch_get_private'
> > | libmodules.a(ofmem_common.o): In function `ofmem_claim_virt':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:448: 
> > undefined reference to `ofmem_arch_get_virt_top'
> > | libmodules.a(ofmem_common.o): In function `ofmem_malloc':
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:83: 
> > undefined reference to `ofmem_arch_get_private'
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:92: 
> > undefined reference to `ofmem_arch_get_malloc_base'
> > | /home/aurel32/openbios-devel/obj-ppc/../modules/ofmem_common.c:108: 
> > undefined reference to `ofmem_arch_get_heap_top'
> > | collect2: ld returned 1 exit status
> > | make[1]: *** [openbios-unix] Error 1
> > | make[1]: Leaving directory `/home/aurel32/openbios-devel/obj-ppc'
> >
> > This is something that needs to be fixed before an OpenBIOS release,
> > though I don't know if its planned to release OpenBIOS in sync with QEMU.
> 
> Does this patch fix the problem?

Not it's worse. With the CONFIG_OFMEM, openbios-qemu.elf doesn't build
anymore. With only the flush_icache_range, I see no change.

-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
address@hidden                 http://www.aurel32.net