From e4603fe139e2161464d7e75faa3a650e31f057fc Mon Sep 17 00:00:00 2001 From: Kevin Wolf Date: Tue, 7 Apr 2015 15:03:16 +0200 Subject: qcow2: Fix header update with overridden backing file In recent qemu versions, it is possible to override the backing file name and format that is stored in the image file with values given at runtime. In such cases, the temporary override could end up in the image header if the qcow2 header was updated, while obviously correct behaviour would be to leave the on-disk backing file path/format unchanged. Fix this and add a test case for it. Reported-by: Michael Tokarev Signed-off-by: Kevin Wolf Reviewed-by: Eric Blake Message-id: 1428411796-2852-1-git-send-email-kwolf@redhat.com Signed-off-by: Stefan Hajnoczi --- tests/qemu-iotests/130 | 95 ++++++++++++++++++++++++++++++++++++++++++++++ tests/qemu-iotests/130.out | 43 +++++++++++++++++++++ tests/qemu-iotests/group | 1 + 3 files changed, 139 insertions(+) create mode 100755 tests/qemu-iotests/130 create mode 100644 tests/qemu-iotests/130.out (limited to 'tests') diff --git a/tests/qemu-iotests/130 b/tests/qemu-iotests/130 new file mode 100755 index 0000000000..bc26247e3f --- /dev/null +++ b/tests/qemu-iotests/130 @@ -0,0 +1,95 @@ +#!/bin/bash +# +# Test that temporary backing file overrides (on the command line or in +# blockdev-add) don't replace the original path stored in the image during +# header updates. +# +# Copyright (C) 2015 Red Hat, Inc. +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 2 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . +# + +# creator +owner=kwolf@redhat.com + +seq="$(basename $0)" +echo "QA output created by $seq" + +here="$PWD" +tmp=/tmp/$$ +status=1 # failure is the default! + +_cleanup() +{ + _cleanup_test_img +} +trap "_cleanup; exit \$status" 0 1 2 3 15 + +# get standard environment, filters and checks +. ./common.rc +. ./common.filter +. ./common.qemu + +_supported_fmt qcow2 +_supported_proto generic +_supported_os Linux + +qemu_comm_method="monitor" + + +TEST_IMG="$TEST_IMG.orig" _make_test_img 64M +TEST_IMG="$TEST_IMG.base" _make_test_img 64M +_make_test_img 64M +_img_info | _filter_img_info + +echo +echo "=== HMP commit ===" +echo +# bdrv_make_empty() involves a header update for qcow2 + +# Test that a backing file isn't written +_launch_qemu -drive file="$TEST_IMG",backing.file.filename="$TEST_IMG.base" +_send_qemu_cmd $QEMU_HANDLE "commit ide0-hd0" "(qemu)" +_send_qemu_cmd $QEMU_HANDLE '' '(qemu)' +_cleanup_qemu +_img_info | _filter_img_info + +# Make sure that if there was a backing file that was just overridden on the +# command line, that backing file is retained, with the right format +_make_test_img -F raw -b "$TEST_IMG.orig" 64M +_launch_qemu -drive file="$TEST_IMG",backing.file.filename="$TEST_IMG.base",backing.driver=$IMGFMT +_send_qemu_cmd $QEMU_HANDLE "commit ide0-hd0" "(qemu)" +_send_qemu_cmd $QEMU_HANDLE '' '(qemu)' +_cleanup_qemu +_img_info | _filter_img_info + +echo +echo "=== Marking image dirty (lazy refcounts) ===" +echo + +# Test that a backing file isn't written +_make_test_img 64M +$QEMU_IO -c "open -o backing.file.filename=$TEST_IMG.base,lazy-refcounts=on $TEST_IMG" -c "write 0 4k" | _filter_qemu_io +_img_info | _filter_img_info + +# Make sure that if there was a backing file that was just overridden on the +# command line, that backing file is retained, with the right format +_make_test_img -F raw -b "$TEST_IMG.orig" 64M +$QEMU_IO -c "open -o backing.file.filename=$TEST_IMG.base,backing.driver=$IMGFMT,lazy-refcounts=on $TEST_IMG" -c "write 0 4k" | _filter_qemu_io +_img_info | _filter_img_info + +# success, all done +echo '*** done' +rm -f $seq.full +status=0 diff --git a/tests/qemu-iotests/130.out b/tests/qemu-iotests/130.out new file mode 100644 index 0000000000..ea68b5d283 --- /dev/null +++ b/tests/qemu-iotests/130.out @@ -0,0 +1,43 @@ +QA output created by 130 +Formatting 'TEST_DIR/t.IMGFMT.orig', fmt=IMGFMT size=67108864 +Formatting 'TEST_DIR/t.IMGFMT.base', fmt=IMGFMT size=67108864 +Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=67108864 +image: TEST_DIR/t.IMGFMT +file format: IMGFMT +virtual size: 64M (67108864 bytes) + +=== HMP commit === + +QEMU X.Y.Z monitor - type 'help' for more information +(qemu) ccocomcommcommicommitcommit commit icommit idcommit idecommit ide0commit ide0-commit ide0-hcommit ide0-hdcommit ide0-hd0 +(qemu) +image: TEST_DIR/t.IMGFMT +file format: IMGFMT +virtual size: 64M (67108864 bytes) +Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=67108864 backing_file='TEST_DIR/t.IMGFMT.orig' backing_fmt='raw' +QEMU X.Y.Z monitor - type 'help' for more information +(qemu) ccocomcommcommicommitcommit commit icommit idcommit idecommit ide0commit ide0-commit ide0-hcommit ide0-hdcommit ide0-hd0 +(qemu) +image: TEST_DIR/t.IMGFMT +file format: IMGFMT +virtual size: 64M (67108864 bytes) +backing file: TEST_DIR/t.IMGFMT.orig +backing file format: raw + +=== Marking image dirty (lazy refcounts) === + +Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=67108864 +wrote 4096/4096 bytes at offset 0 +4 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +image: TEST_DIR/t.IMGFMT +file format: IMGFMT +virtual size: 64M (67108864 bytes) +Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=67108864 backing_file='TEST_DIR/t.IMGFMT.orig' backing_fmt='raw' +wrote 4096/4096 bytes at offset 0 +4 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +image: TEST_DIR/t.IMGFMT +file format: IMGFMT +virtual size: 64M (67108864 bytes) +backing file: TEST_DIR/t.IMGFMT.orig +backing file format: raw +*** done diff --git a/tests/qemu-iotests/group b/tests/qemu-iotests/group index 62621278e2..bcf25786ab 100644 --- a/tests/qemu-iotests/group +++ b/tests/qemu-iotests/group @@ -124,3 +124,4 @@ 121 rw auto 123 rw auto quick 128 rw auto quick +130 rw auto quick -- cgit v1.2.3