From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60133) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YuN1a-0001VT-8Y for qemu-devel@nongnu.org; Mon, 18 May 2015 11:32:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YuN1V-0008Dc-3O for qemu-devel@nongnu.org; Mon, 18 May 2015 11:32:14 -0400 Message-ID: <555A05F2.4030103@redhat.com> Date: Mon, 18 May 2015 17:32:02 +0200 From: Max Reitz MIME-Version: 1.0 References: <1431105726-3682-1-git-send-email-kwolf@redhat.com> <1431105726-3682-34-git-send-email-kwolf@redhat.com> <55564620.2050309@redhat.com> <20150518143902.GF5999@noname.str.redhat.com> In-Reply-To: <20150518143902.GF5999@noname.str.redhat.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 33/34] qemu-iotests: Test cache mode option inheritance List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf Cc: armbru@redhat.com, qemu-block@nongnu.org, qemu-devel@nongnu.org On 18.05.2015 16:39, Kevin Wolf wrote: > Am 15.05.2015 um 21:16 hat Max Reitz geschrieben: >> On 08.05.2015 19:22, Kevin Wolf wrote: >>> This is doing a more complete test on setting cache modes both while >>> opening an image (i.e. in a -drive command line) and in reopen >>> situations. It checks that reopen can specify options for child nodes >>> and that cache modes are correctly inherited from parent nodes where >>> they are not specified. >>> >>> Signed-off-by: Kevin Wolf >>> --- >>> tests/qemu-iotests/132 | 336 ++++++++++++++++++++ >>> tests/qemu-iotests/132.out | 767 +++++++++++++++++++++++++++++++++++++++++++++ >>> tests/qemu-iotests/group | 1 + >>> 3 files changed, 1104 insertions(+) >>> create mode 100755 tests/qemu-iotests/132 >>> create mode 100644 tests/qemu-iotests/132.out >> >> So starting from now I once again need to specify -c writethrough >> for running tests in tmpfs... Welp. >> >> ("Welp" because there is no actual strict requirement to have >> O_DIRECT, because we don't actually use it but only check the >> configuration; so using null-co would be fine, too, but we can only >> use that with raw as the format driver, and raw doesn't support >> backing files...) > Perhaps we should add a switch to tell qemu-iotests to skip test cases > that would require cache.direct=on? Or create a group 'direct' so you > can use '-x direct'? Well, it would be 'non-direct', and in this case it's not better than -c writethrough. ;-) I was lucky I got away without specifying the cache mode until now, and now I'll just have to live with it. If we (I) would want to make it easier, we'd have to probe (in ./check) whether $TEST_DIR supports O_DIRECT or not. Max