Re: [Qemu-devel] [PULL] Memory API conversions, batch 5

[Jan Kiszka]

On 2011-08-25 20:27, Edgar E. Iglesias wrote:
> On Thu, Aug 25, 2011 at 08:04:34AM -0500, Anthony Liguori wrote:
>> On 08/25/2011 03:01 AM, Avi Kivity wrote:
>>> Please pull from
>>>
>>> git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git memory/batch
>>>
>>> to receive the fifth memory API conversions batch. There is really
>>> nothing notable, except perhaps the first use of the transaction API.
>>>
>>> While the patches have been on the list for just a short time, they have
>>> received quite extensive review, for which I am thankful.
>>
>> Pulled.  Thanks.
> 
> Guys, please start testing before applying these patches. It's not hard,
> there are images on the wiki with scripts that area easy to run.
> 
> Cheers
> 
> $ gdb --args 
> ~/src/c/qemu/build-qemu/microblaze-softmmu/qemu-system-microblaze -kernel 
> linux.bin -serial stdio
> GNU gdb (Ubuntu/Linaro 7.3-0ubuntu1) 7.3-2011.08~bzr
> Copyright (C) 2011 Free Software Foundation, Inc.
> License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
> This is free software: you are free to change and redistribute it.
> There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
> and "show warranty" for details.
> This GDB was configured as "x86_64-linux-gnu".
> For bug reporting instructions, please see:
> <http://www.gnu.org/software/gdb/bugs/>...
> Reading symbols from 
> /home/edgar/src/c/qemu/build-qemu/microblaze-softmmu/qemu-system-microblaze...done.
> (gdb) run
> Starting program: 
> /home/edgar/src/c/qemu/build-qemu/microblaze-softmmu/qemu-system-microblaze 
> -kernel linux.bin -serial stdio
> [Thread debugging using libthread_db enabled]
> [New Thread 0x7fffeef3e700 (LWP 25891)]
> 
> Program received signal SIGSEGV, Segmentation fault.
> 0x000000000041be54 in memory_region_init_rom_device (mr=0xfb6900, 
>     ops=<optimized out>, dev=<optimized out>, 
>     name=0x564c5d "petalogix_s3adsp1800.flash", size=16777216)
>     at /home/edgar/src/c/qemu/qemu/memory.c:973
> 973         mr->ram_addr |= cpu_register_io_memory(memory_region_read_thunk,
> (gdb) 

That looks pretty much like what I stumbled over today as well. Should
be fine with [1], need to test again.

Jan

[1] http://thread.gmane.org/gmane.comp.emulators.qemu/114619

signature.asc
Description: OpenPGP digital signature