From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49902) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XPZUR-0005LK-Hk for qemu-devel@nongnu.org; Thu, 04 Sep 2014 12:02:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XPZUL-00022D-C4 for qemu-devel@nongnu.org; Thu, 04 Sep 2014 12:02:27 -0400 Received: from mx1.redhat.com ([209.132.183.28]:18796) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XPZUL-000228-3w for qemu-devel@nongnu.org; Thu, 04 Sep 2014 12:02:21 -0400 Date: Thu, 4 Sep 2014 19:05:32 +0300 From: "Michael S. Tsirkin" Message-ID: <20140904160532.GA21505@redhat.com> References: <1407524611-10843-1-git-send-email-ehabkost@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1407524611-10843-1-git-send-email-ehabkost@redhat.com> Subject: Re: [Qemu-devel] [PATCH v3 0/6] Improve global validation test cases, move validation to a single function List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eduardo Habkost Cc: Peter Maydell , qemu-devel@nongnu.org, Don Slutz , Markus Armbruster , Paolo Bonzini , Igor Mammedov , Andreas =?iso-8859-1?Q?F=E4rber?= On Fri, Aug 08, 2014 at 04:03:25PM -0300, Eduardo Habkost wrote: > New version of what was previously a single patch, submitted as: > Subject: [PATCH v2] qdev: Move global validation to a single function > > Now the test cases run in a subprocess, and we are ensuring the warning messages > are actually printed. Applied, thanks! > Eduardo Habkost (6): > test-qdev-global-props: Trivial comment fix > test-qdev-global-props: Run tests on subprocess > test-qdev-global-props: Initialize not_used=true for all props > test-qdev-global-props: Test handling of hotpluggable and non-device > types > qdev: Rename qdev_prop_check_global() to qdev_prop_check_globals() > qdev: Move global validation to a single function > > hw/core/qdev-properties-system.c | 18 +---- > hw/core/qdev-properties.c | 30 ++++++-- > include/hw/qdev-core.h | 10 +-- > include/hw/qdev-properties.h | 2 +- > tests/test-qdev-global-props.c | 159 ++++++++++++++++++++++++++++++++++++--- > vl.c | 2 +- > 6 files changed, 180 insertions(+), 41 deletions(-) > > -- > 1.9.3