qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: 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 10:31:40 +0100	[thread overview]
Message-ID: <CAFEAcA9b51aQdkr2Zc0Oyn89f_4cLQghdqCcS1ovRbrDRG-fvg@mail.gmail.com> (raw)
In-Reply-To: <1435747190-18017-1-git-send-email-quintela@redhat.com>

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.

thanks
-- PMM

  parent reply	other threads:[~2015-07-02  9:32 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 ` Peter Maydell [this message]
2015-07-02  9:51   ` [Qemu-devel] [PULL 00/26] Migration pull request Wen Congyang
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=CAFEAcA9b51aQdkr2Zc0Oyn89f_4cLQghdqCcS1ovRbrDRG-fvg@mail.gmail.com \
    --to=peter.maydell@linaro.org \
    --cc=amit.shah@redhat.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).