[0/5] enhance debugging with qtest framework

Message

Zhijian Li (Fujitsu)" via Jan. 26, 2021, 11:36 p.m. UTC

This series is a follow-up of the following previous series:
https://lists.gnu.org/archive/html/qemu-devel/2020-10/msg02251.html
The 9p patches of the previous series have already been merged.

This series consists of 2 parts:

1. libqos patch 1 removes a limitation of the qtest/libqos subsystem:
   support for more than one device using the same (official) QEMU device
   name.

   Like discussed in the previous series, if nobody finds this patch useful
   then just ignore it. I needed it in the previou series before but
   eventually decided for a different approach and personally don't need it
   in near future.

2. Patches 2 to 5 enhance debugging issues with the qtest framework. I would
   appreciate if they got merged, because I still find them useful while
   working on new test cases.

Changes of these patches from derived series:

  * Squashed previous patches 1 & 2 -> [patch 1].

  * Dropped ANSI color escape sequences [patch 3].

  * Squashed previous patches 4 & 5 -> [patch 3].

  * Extended commit log to provide more details about purpose [patch 4].

Christian Schoenebeck (5):
  libqos/qgraph: add qos_node_create_driver_named()
  libqos/qgraph_internal: add qos_printf() and qos_printf_literal()
  tests/qtest/qos-test: dump qos graph if verbose
  tests/qtest/qos-test: dump environment variables if verbose
  tests/qtest/qos-test: dump QEMU command if verbose

 tests/qtest/libqos/qgraph.c          | 99 +++++++++++++++++++++++++++-
 tests/qtest/libqos/qgraph.h          | 36 ++++++++++
 tests/qtest/libqos/qgraph_internal.h | 12 ++++
 tests/qtest/qos-test.c               | 15 ++++-
 4 files changed, 158 insertions(+), 4 deletions(-)

Comments

Paolo Bonzini Feb. 9, 2021, 1:57 p.m. UTC | #1

On 27/01/21 00:36, Christian Schoenebeck wrote:
> This series is a follow-up of the following previous series:
> https://lists.gnu.org/archive/html/qemu-devel/2020-10/msg02251.html
> The 9p patches of the previous series have already been merged.
> 
> This series consists of 2 parts:
> 
> 1. libqos patch 1 removes a limitation of the qtest/libqos subsystem:
>     support for more than one device using the same (official) QEMU device
>     name.
> 
>     Like discussed in the previous series, if nobody finds this patch useful
>     then just ignore it. I needed it in the previou series before but
>     eventually decided for a different approach and personally don't need it
>     in near future.
> 
> 2. Patches 2 to 5 enhance debugging issues with the qtest framework. I would
>     appreciate if they got merged, because I still find them useful while
>     working on new test cases.
> 
> Changes of these patches from derived series:
> 
>    * Squashed previous patches 1 & 2 -> [patch 1].
> 
>    * Dropped ANSI color escape sequences [patch 3].
> 
>    * Squashed previous patches 4 & 5 -> [patch 3].
> 
>    * Extended commit log to provide more details about purpose [patch 4].
> 
> Christian Schoenebeck (5):
>    libqos/qgraph: add qos_node_create_driver_named()
>    libqos/qgraph_internal: add qos_printf() and qos_printf_literal()
>    tests/qtest/qos-test: dump qos graph if verbose
>    tests/qtest/qos-test: dump environment variables if verbose
>    tests/qtest/qos-test: dump QEMU command if verbose
> 
>   tests/qtest/libqos/qgraph.c          | 99 +++++++++++++++++++++++++++-
>   tests/qtest/libqos/qgraph.h          | 36 ++++++++++
>   tests/qtest/libqos/qgraph_internal.h | 12 ++++
>   tests/qtest/qos-test.c               | 15 ++++-
>   4 files changed, 158 insertions(+), 4 deletions(-)
> 

Queued, thanks.

Paolo