All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wen Congyang <wency@cn.fujitsu.com>
To: Peter Maydell <peter.maydell@linaro.org>,
	Juan Quintela <quintela@redhat.com>
Cc: Amit Shah <amit.shah@redhat.com>,
	QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PULL 00/26] Migration pull request
Date: Thu, 2 Jul 2015 17:51:33 +0800	[thread overview]
Message-ID: <559509A5.8020309@cn.fujitsu.com> (raw)
In-Reply-To: <CAFEAcA9b51aQdkr2Zc0Oyn89f_4cLQghdqCcS1ovRbrDRG-fvg@mail.gmail.com>

On 07/02/2015 05:31 PM, Peter Maydell wrote:
> On 1 July 2015 at 11:39, Juan Quintela <quintela@redhat.com> wrote:
>> Hi
>>
>> This series includes:
>> - rdma fixes by Dave
>> - rdma memory fix by gonglei
>> - vmdescription for old machine types (dave)
>> - fix footers for power (dave)
>> - migration bitmap extensions (Li)
>>   just fixed the compilation issues for linux-users
>> - migration events (me)
>> - optional secttions (me)
>> - global  configuration (me)
>>
>>
>> Please, Apply.
>>
>>
>> The following changes since commit d2966f804d70a244f5dde395fc5d22a50ed3e74e:
>>
>>   Merge remote-tracking branch 'remotes/vivier/tags/pull-m68k-20150629' into staging (2015-06-29 17:03:20 +0100)
>>
>> are available in the git repository at:
>>
>>   git://github.com/juanquintela/qemu.git tags/migration/20150701
>>
>> for you to fetch changes up to a4fe58b0ea0d78f92461607f4f90be3384fa30e5:
>>
>>   migration: Add migration events on target side (2015-07-01 12:35:05 +0200)
>>
>> ----------------------------------------------------------------
>> migration/next for 20150701
> 
> On OSX at least every QEMU executable aborts immediately
> with "qemu: qemu_mutex_lock: Invalid argument". Here's a backtrace:
> 
> #0  0x00007fff92c29286 in __pthread_kill ()
> #1  0x00007fff8a4f342f in pthread_kill ()
> #2  0x00007fff9240eb53 in abort ()
> #3  0x00000001002b0915 in error_exit (err=<value temporarily
> unavailable, due to optimizations>, msg=<value temporarily
> unavailable, due to optimizations>) at
> /Users/pm215/src/qemu/util/qemu-thread-posix.c:48
> #4  0x00000001002b095d in qemu_mutex_lock (mutex=<value temporarily
> unavailable, due to optimizations>) at
> /Users/pm215/src/qemu/util/qemu-thread-posix.c:75
> #5  0x0000000100050f20 in migration_bitmap_extend (old=0, new=32768)
> at /Users/pm215/src/qemu/migration/ram.c:1068
> #6  0x0000000100002a8d in ram_block_add [inlined] () at
> /Users/pm215/src/qemu/exec.c:1407
> #7  0x0000000100002a8d in qemu_ram_alloc_internal (size=<value
> temporarily unavailable, due to optimizations>, max_size=<value
> temporarily unavailable, due to optimizations>, resized=<value
> temporarily unavailable, due to optimizations>, host=<value
> temporarily unavailable, due to optimizations>, resizeable=false,
> mr=0x100e97320, errp=<value temporarily unavailable, due to
> optimizations>) at /Users/pm215/src/qemu/exec.c:1537
> #8  0x0000000100002e71 in qemu_ram_alloc (size=1823, mr=0x100e97320,
> errp=<value temporarily unavailable, due to optimizations>) at
> /Users/pm215/src/qemu/exec.c:1554
> #9  0x000000010004353f in memory_region_init_ram (mr=0x100e97320,
> owner=<value temporarily unavailable, due to optimizations>,
> name=<value temporarily unavailable, due to optimizations>,
> size=134217728, errp=0x0) at /Users/pm215/src/qemu/memory.c:1216
> #10 0x000000010003f10a in allocate_system_memory_nonnuma [inlined] ()
> at /Users/pm215/src/qemu/numa.c:339
> #11 0x000000010003f10a in memory_region_allocate_system_memory
> (mr=0x100e97320, owner=0x6, name=0x0, ram_size=140734799798744) at
> /Users/pm215/src/qemu/numa.c:352
> #12 0x00000001000781cb in pc_memory_init (machine=0x102198630,
> system_memory=0x102199690, below_4g_mem_size=134217728,
> above_4g_mem_size=0, rom_memory=0x100e96de0,
> ram_memory=0x7fff5fbfe4b8, guest_info=<value temporarily unavailable,
> due to optimizations>) at /Users/pm215/src/qemu/hw/i386/pc.c:1254
> #13 0x000000010007aae6 in pc_init1 (machine=0x102198630) at
> /Users/pm215/src/qemu/hw/i386/pc_piix.c:182
> #14 0x00000001000e3cf9 in realtime_init [inlined] () at
> /Users/pm215/src/qemu/vl.c:4503
> #15 0x00000001000e3cf9 in qemu_main (argc=<value temporarily
> unavailable, due to optimizations>, argv=<value temporarily
> unavailable, due to optimizations>, envp=0x0) at
> /Users/pm215/src/qemu/vl.c:4505
> #16 0x00000001002117be in -[QemuCocoaAppController
> startEmulationWithArgc:argv:] (self=<value temporarily unavailable,
> due to optimizations>, _cmd=<value temporarily unavailable, due to
> optimizations>, argc=1823, argv=0x102199690) at
> /Users/pm215/src/qemu/ui/cocoa.m:941
> 
> A breakpoint on ram_save_setup() is never hit, so it looks
> like the problem is the mutex is being used before it is
> initialized.

Yes. I don't know why our test doesn't trigger this problem.
Will fix it soon.

Thanks
Wen Congyang

> 
> thanks
> -- PMM
> 
> .
> 

  reply	other threads:[~2015-07-02  9:47 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-01 10:39 [Qemu-devel] [PULL 00/26] Migration pull request Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 01/26] migration: protect migration_bitmap Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 02/26] migration: extend migration_bitmap Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 03/26] rdma: fix memory leak Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 04/26] Only try and read a VMDescription if it should be there Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 05/26] rdma typos Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 06/26] Store block name in local blocks structure Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 07/26] Translate offsets to destination address space Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 08/26] Rework ram_control_load_hook to hook during block load Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 09/26] Allow rdma_delete_block to work without the hash Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 10/26] Rework ram block hash Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 11/26] Sort destination RAMBlocks to be the same as the source Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 12/26] Sanity check RDMA remote data Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 13/26] Fail more cleanly in mismatched RAM cases Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 14/26] Fix older machine type compatibility on power with section footers Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 15/26] runstate: Add runstate store Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 16/26] runstate: migration allows more transitions now Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 17/26] migration: create new section to store global state Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 18/26] global_state: Make section optional Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 19/26] vmstate: Create optional sections Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 20/26] migration: Add configuration section Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 21/26] migration: Use cmpxchg correctly Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 22/26] migration: ensure we start in NONE state Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 23/26] migration: Use always helper to set state Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 24/26] migration: No need to call trace_migrate_set_state() Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 25/26] migration: create migration event Juan Quintela
2015-07-01 10:39 ` [Qemu-devel] [PULL 26/26] migration: Add migration events on target side Juan Quintela
2015-07-02  9:31 ` [Qemu-devel] [PULL 00/26] Migration pull request Peter Maydell
2015-07-02  9:51   ` Wen Congyang [this message]
2015-07-02 12:17   ` Juan Quintela
2015-07-02 12:30     ` Peter Maydell
2015-07-02 12:55       ` [Qemu-devel] Mutex error checking (was: [PULL 00/26] Migration pull request) Markus Armbruster
2015-07-02 12:58         ` Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=559509A5.8020309@cn.fujitsu.com \
    --to=wency@cn.fujitsu.com \
    --cc=amit.shah@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.