From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60415) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zj9IT-00053e-45 for qemu-devel@nongnu.org; Mon, 05 Oct 2015 13:11:34 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zj9IP-0001lL-3c for qemu-devel@nongnu.org; Mon, 05 Oct 2015 13:11:33 -0400 Received: from mx1.redhat.com ([209.132.183.28]:33703) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zj9IO-0001lA-S9 for qemu-devel@nongnu.org; Mon, 05 Oct 2015 13:11:29 -0400 From: Markus Armbruster References: <1443806441-23519-1-git-send-email-armbru@redhat.com> <87612l95e1.fsf@blackfin.pond.sub.org> Date: Mon, 05 Oct 2015 19:11:25 +0200 In-Reply-To: (Peter Maydell's message of "Mon, 5 Oct 2015 12:55:51 +0100") Message-ID: <874mi55jgi.fsf@blackfin.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: QEMU Developers Peter Maydell writes: > On 5 October 2015 at 07:49, Markus Armbruster wrote: >> Peter Maydell writes: >> >>> On 2 October 2015 at 18:20, Markus Armbruster wrote: >>>> QMP command device-list-properties regressed in 2.1: it can crash or >>>> leave dangling pointers behind. >>>> >>>> -device FOO,help regressed in 2.2: it no longer works for >>>> non-pluggable devices. I tried to fix that some time ago[*], but my >>>> fix failed review. This is my second, more comprehensive try. >>>> >>>> PATCH 1-3 fix one class of bugs involved in the regressions, PATCH 4-5 >>>> are libqtest preliminaries, PATCH 6 adds tests to demonstrate the >>>> remaining bugs, PATCH 7-9 fix them to a degree (see PATCH 8 for >>>> limitations), and PATCH 10 cleans up. >>> >>> This ordering breaks bisection of 'make check', as I found out when >>> I tried to figure out which of the patches in this pull was causing >>> an OSX test failure. Please can you reorder them so that 'make check' >>> works at all points in the series? >> >> My ordering may be bad (and I'll recheck it, of course), or it may >> temporarily expose a hidden bug. I better figure out what's going on >> here. All commits pass make check for me. >>>> The following changes since commit ff770b07f34d28b79013a83989bd6c85f8f16b2f: >>>> >>>> Merge remote-tracking branch >>>> 'remotes/cody/tags/block-pull-request' into staging (2015-10-02 >>>> 11:01:18 +0100) >>>> >>>> are available in the git repository at: >>>> >>>> git://repo.or.cz/qemu/armbru.git tags/pull-monitor-2015-10-02 >>>> >>>> for you to fetch changes up to e927162a6fa2fa6144de9d1d11cc9448a2143671: >>>> >>>> Revert "qdev: Use qdev_get_device_class() for -device >>>> ,help" (2015-10-02 16:45:53 +0200) >>>> >>>> ---------------------------------------------------------------- >>>> Fix device introspection regressions >>>> >>>> ---------------------------------------------------------------- >>> >>> 'make check' failure on OSX: >>> >>> /aarch64/device/introspect/list: OK >>> /aarch64/device/introspect/none: OK >>> /aarch64/device/introspect/abstract: OK >>> /aarch64/device/introspect/concrete: ** >>> ERROR:/Users/pm215/src/qemu-for-merges/qom/object.c:333:void >>> object_initialize_with_type(void *, size_t, TypeImpl *): assertion >>> failed: (type != NULL) >>> Broken pipe >>> FAIL >>> >>> I have no idea why this only failed on OSX... >> >> Can you re-run this with valgrind spliced in? > > Valgrind is not particularly helpful: it reports a couple of > irrelevancies and an unimplemented syscall, then just > reports the backtrace for the abort: [...] Sigh. I'll stare at the original backtrace some more.