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 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).