[PATCH] system/qtest: Support comments in input commands

Peter Maydell posted 1 patch 3 days, 12 hours ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260226175700.1319767-1-peter.maydell@linaro.org
Maintainers: Fabiano Rosas <farosas@suse.de>, Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
system/qtest.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
[PATCH] system/qtest: Support comments in input commands
Posted by Peter Maydell 3 days, 12 hours ago
Allow the qtest input to include comment lines, which start with '#'.
This allows writing an input file for qtest which includes commentary,
like this:

  # set up TCR in bank 0
  write 0x1001000e 2 0
  # TCR TXEN
  write 0x10010000 2 1

which can make hand-writing or annotating reproduce cases a bit
more convenient.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 system/qtest.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/system/qtest.c b/system/qtest.c
index e42b83ce67..cf90cd53ad 100644
--- a/system/qtest.c
+++ b/system/qtest.c
@@ -71,6 +71,7 @@ static void *qtest_server_send_opaque;
  *
  * Extra ASCII space characters in command inputs are permitted and ignored.
  * Lines containing only spaces are permitted and ignored.
+ * Lines that start with a '#' character (comments) are permitted and ignored.
  *
  * Valid requests
  * ^^^^^^^^^^^^^^
@@ -370,8 +371,8 @@ static void qtest_process_command(CharFrontend *chr, gchar **words)
         fprintf(qtest_log_fp, "\n");
     }
 
-    if (!command) {
-        /* Input line was blank: ignore it */
+    if (!command || command[0] == '#') {
+        /* Input line was blank or a comment: ignore it */
         return;
     }
 
-- 
2.43.0
Re: [PATCH] system/qtest: Support comments in input commands
Posted by Philippe Mathieu-Daudé 3 days, 9 hours ago
On 26/2/26 18:57, Peter Maydell wrote:
> Allow the qtest input to include comment lines, which start with '#'.
> This allows writing an input file for qtest which includes commentary,
> like this:
> 
>    # set up TCR in bank 0
>    write 0x1001000e 2 0
>    # TCR TXEN
>    write 0x10010000 2 1
> 
> which can make hand-writing or annotating reproduce cases a bit
> more convenient.

Great idea!

> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>   system/qtest.c | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>