From: Peter Xu <peterx@redhat.com>
To: Fam Zheng <famz@redhat.com>
Cc: qemu-devel@nongnu.org, "Alex Bennée" <alex.bennee@linaro.org>,
pbonzini@redhat.com, "Philippe Mathieu-Daudé" <f4bug@amsat.org>
Subject: Re: [Qemu-devel] [PATCH 0/3] Add a CentOS test image to run docker tests
Date: Wed, 4 Apr 2018 13:03:47 +0800 [thread overview]
Message-ID: <20180404050347.GI26441@xz-mi> (raw)
In-Reply-To: <20180404025201.11921-1-famz@redhat.com>
On Wed, Apr 04, 2018 at 10:51:58AM +0800, Fam Zheng wrote:
> Docker testing on patchew has long suffered from 'make check' hangings. The
> cleanness of VM testing is the cure. Now let's add a CentOS 7 image to run the
> tests. It's purely ad-hoc, but hopefully still easy to understand and use for
> everyone.
>
> The first patch makes passing source code from host to the container in VM
> working, and is a nice clean up.
>
> The second patch makes caches work, to speed up repetitive runs like on
> patchew.
>
> The last patch adds the new image that does the job. Two out of three docker
> tests running on patchew.org are added to the image. I'll wait for Peter to fix
> the 'docker-test-quick@centos6' hanging (oob test) before adding it too.
This hang does not happen on centos7 (tested myself manually). I
consider this can be related to different default versions of glib2:
centos6: glib2-2.28.8-9.el6.x86_64
centos7: glib2-2.50.3-3.el7.x86_64
(both fetched from docker.io)
qmp-test hangs at oob-test, while QEMU failed to respond to SIGTERM
when trying to take a lock:
#0 0x00007f18eed02334 in __lll_lock_wait () from /lib64/libpthread.so.0
#1 0x00007f18eecfd5d8 in _L_lock_854 () from /lib64/libpthread.so.0
#2 0x00007f18eecfd4a7 in pthread_mutex_lock () from /lib64/libpthread.so.0
#3 0x00007f18eb476c9d in ?? () from /lib64/libglib-2.0.so.0
#4 0x0000000000aa6672 in aio_context_unref (ctx=0x225b600) at /root/qemu/util/async.c:497
#5 0x000000000065851c in iothread_instance_finalize (obj=0x225b380) at /root/qemu/iothread.c:129
#6 0x0000000000962d79 in object_deinit (obj=0x225b380, type=0x2199960) at /root/qemu/qom/object.c:462
#7 0x0000000000962e0d in object_finalize (data=0x225b380) at /root/qemu/qom/object.c:476
#8 0x0000000000964146 in object_unref (obj=0x225b380) at /root/qemu/qom/object.c:924
#9 0x0000000000965880 in object_finalize_child_property (obj=0x2257640, name=0x225aca0 "mon_iothread", opaque=0x225b380) at /root/qemu/qom/object.c:1436
#10 0x0000000000962c33 in object_property_del_child (obj=0x2257640, child=0x225b380, errp=0x0) at /root/qemu/qom/object.c:436
#11 0x0000000000962d26 in object_unparent (obj=0x225b380) at /root/qemu/qom/object.c:455
#12 0x0000000000658f00 in iothread_destroy (iothread=0x225b380) at /root/qemu/iothread.c:365
#13 0x00000000004c67a8 in monitor_cleanup () at /root/qemu/monitor.c:4663
#14 0x0000000000669e27 in main (argc=16, argv=0x7ffe33ce3a08, envp=0x7ffe33ce3a90) at /root/qemu/vl.c:4749
Currently I have no quick idea on why the lock was held by anyone
else (I suppose that's the GMainContext's lock), and I'm not sure
whether it could be a glib bug.
--
Peter Xu
next prev parent reply other threads:[~2018-04-04 5:04 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-04 2:51 [Qemu-devel] [PATCH 0/3] Add a CentOS test image to run docker tests Fam Zheng
2018-04-04 2:51 ` [Qemu-devel] [PATCH 1/3] archive-source.sh: Drop submodule code Fam Zheng
2018-04-04 8:23 ` Daniel P. Berrangé
2018-04-04 9:07 ` Fam Zheng
2018-04-04 9:24 ` Daniel P. Berrangé
2018-04-04 9:37 ` Fam Zheng
2018-04-04 2:52 ` [Qemu-devel] [PATCH 2/3] tests: Add an option for snapshot (default: off) Fam Zheng
2018-04-04 2:52 ` [Qemu-devel] [PATCH 3/3] tests: Add centos VM testing Fam Zheng
2018-04-04 3:01 ` [Qemu-devel] [PATCH 0/3] Add a CentOS test image to run docker tests no-reply
2018-04-04 3:04 ` no-reply
2018-04-04 5:03 ` Peter Xu [this message]
2018-04-04 6:56 ` Peter Xu
2018-04-04 8:19 ` Daniel P. Berrangé
2018-04-04 8:24 ` Fam Zheng
2018-04-06 16:47 ` Paolo Bonzini
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=20180404050347.GI26441@xz-mi \
--to=peterx@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=f4bug@amsat.org \
--cc=famz@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/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.