Re: [kvm-unit-tests PATCH v7 01/35] arch-run: Add functions to help handle migration directives from test

2024-03-25 Thread Thomas Huth

On 19/03/2024 08.58, Nicholas Piggin wrote:

The migration harness will be expanded to deal with more commands
from the test, moving these checks into functions helps keep things
managable.

Signed-off-by: Nicholas Piggin 
---
  scripts/arch-run.bash | 20 +++-
  1 file changed, 15 insertions(+), 5 deletions(-)


Reviewed-by: Thomas Huth 



[kvm-unit-tests PATCH v7 01/35] arch-run: Add functions to help handle migration directives from test

2024-03-19 Thread Nicholas Piggin
The migration harness will be expanded to deal with more commands
from the test, moving these checks into functions helps keep things
managable.

Signed-off-by: Nicholas Piggin 
---
 scripts/arch-run.bash | 20 +++-
 1 file changed, 15 insertions(+), 5 deletions(-)

diff --git a/scripts/arch-run.bash b/scripts/arch-run.bash
index 413f3eda8..e34d784c0 100644
--- a/scripts/arch-run.bash
+++ b/scripts/arch-run.bash
@@ -122,6 +122,16 @@ qmp_events ()
jq -c 'select(has("event"))'
 }
 
+filter_quiet_msgs ()
+{
+   grep -v "Now migrate the VM (quiet)"
+}
+
+seen_migrate_msg ()
+{
+   grep -q -e "Now migrate the VM" < $1
+}
+
 run_migration ()
 {
if ! command -v ncat >/dev/null 2>&1; then
@@ -152,7 +162,7 @@ run_migration ()
-chardev socket,id=mon,path=${src_qmp},server=on,wait=off \
-mon chardev=mon,mode=control > ${src_outfifo} &
live_pid=$!
-   cat ${src_outfifo} | tee ${src_out} | grep -v "Now migrate the VM 
(quiet)" &
+   cat ${src_outfifo} | tee ${src_out} | filter_quiet_msgs &
 
# Start the first destination QEMU machine in advance of the test
# reaching the migration point, since we expect at least one migration.
@@ -162,7 +172,7 @@ run_migration ()
 
while ps -p ${live_pid} > /dev/null ; do
# Wait for test exit or further migration messages.
-   if ! grep -q -i "Now migrate the VM" < ${src_out} ; then
+   if ! seen_migrate_msg ${src_out} ;  then
sleep 0.1
else
do_migration || return $?
@@ -190,11 +200,11 @@ do_migration ()
-mon chardev=mon,mode=control -incoming unix:${dst_incoming} \
< <(cat ${dst_infifo}) > ${dst_outfifo} &
incoming_pid=$!
-   cat ${dst_outfifo} | tee ${dst_out} | grep -v "Now migrate the VM 
(quiet)" &
+   cat ${dst_outfifo} | tee ${dst_out} | filter_quiet_msgs &
 
# The test must prompt the user to migrate, so wait for the
-   # "Now migrate VM" console message.
-   while ! grep -q -i "Now migrate the VM" < ${src_out} ; do
+   # "Now migrate VM" or similar console message.
+   while ! seen_migrate_msg ${src_out} ; do
if ! ps -p ${live_pid} > /dev/null ; then
echo "ERROR: Test exit before migration point." >&2
echo > ${dst_infifo}
-- 
2.42.0