Ping, again? If people are too busy to review the qdev changes, can we at least get the unit test changes included?
On Thu, Aug 21, 2014 at 05:26:15PM -0300, Eduardo Habkost wrote: > Ping? > > 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. > > > > 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 > > > > > > -- > Eduardo > -- Eduardo
