qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] [PATCH v4 0/3] Add common QEMU control functionality to qem


From: Jeff Cody
Subject: [Qemu-devel] [PATCH v4 0/3] Add common QEMU control functionality to qemu-iotests
Date: Tue, 29 Apr 2014 18:39:54 -0400

Changes from v3:

Patch 1:   - Make bash version check work with >= 5 as well (Thanks Kevin)
           - Suppress error message if killing non-existent PID
           - Close QEMU fifo in/out file descriptors on cleanup (Thanks Kevin)
           - Remove extra newline at the end of the file (Thanks Kevin)

Patch 2:    - Update commit message to document the commandline change 
                of accel=qtest due to Patch 1 (Thanks Kevin)

Patch 3:    - Whitespace cleanup (Thanks Kevin)
            - Cleanly flush and exit QEMU on vm2, and run
               qemu-img check -r all (Thanks Kevin)   
            - YATNC - Yet Another Test Number Conflict;
                move test from 090 to 091 (Thanks Kevin)

Patches 4,5:  Already applied to Kevin's block tree


Prior cover letter details:


Changes from v2:

Updated Reviewed-by for Fam and Benoit (Benoit's from the v1 patch, I forgot to
add those to v2)

    Patch 1: * updated commit message (Thanks Fam)
             * Addded '-machine accel=qtest' to qemu launch args (Thanks Fam)
    Patch 3: * Moved from test 089 -> test 090 to avoid collision with
               Fam's series (Thanks Fam)
    
Changes from v1:

    Patch 1: * Fixed commit message, clarified comments (Thanks BenoƮt)
             * Changed 'shift' line to be POSIX-friendly, instead of
               relying on bashism (Thanks Eric)
             * Added ability to repeat qmp or hmp commands an arbitrary
               number of times
    Patch 3: New patch, for live migration

Original Description:

This adds some common functionality to control QEMU for qemu-iotests.

Additionally, test 085 is updated to use this new functionality.

Some minor fixups along the way, to clear up spaced pathname issues, 
for common.rc, test 019, and test 086.


Jeff Cody (3):
  block: qemu-iotests - add common.qemu, for bash-controlled qemu tests
  block: qemu-iotests - update 085 to use common.qemu
  block: qemu-iotests - test for live migration

 tests/qemu-iotests/085         |  73 +++------------
 tests/qemu-iotests/091         | 102 +++++++++++++++++++++
 tests/qemu-iotests/091.out     |  25 ++++++
 tests/qemu-iotests/common.qemu | 197 +++++++++++++++++++++++++++++++++++++++++
 tests/qemu-iotests/group       |   1 +
 5 files changed, 337 insertions(+), 61 deletions(-)
 create mode 100755 tests/qemu-iotests/091
 create mode 100644 tests/qemu-iotests/091.out
 create mode 100644 tests/qemu-iotests/common.qemu

-- 
1.8.3.1




reply via email to

[Prev in Thread] Current Thread [Next in Thread]