From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55482) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W042C-0001pI-Nm for qemu-devel@nongnu.org; Mon, 06 Jan 2014 01:51:41 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W0427-0002aT-S5 for qemu-devel@nongnu.org; Mon, 06 Jan 2014 01:51:36 -0500 Received: from mx.ipv6.kamp.de ([2a02:248:0:51::16]:57210 helo=mx01.kamp.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W0427-0002aO-HD for qemu-devel@nongnu.org; Mon, 06 Jan 2014 01:51:31 -0500 Message-ID: <52CA52D2.3000405@kamp.de> Date: Mon, 06 Jan 2014 07:53:06 +0100 From: Peter Lieven MIME-Version: 1.0 References: <1388062130-21126-1-git-send-email-pl@kamp.de> <52C69EDF.7020500@kamp.de> <20140106011843.GA8817@stefanha-thinkpad.redhat.com> In-Reply-To: <20140106011843.GA8817@stefanha-thinkpad.redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCHv5] block: add native support for NFS List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Kevin Wolf , Fam Zheng , Stefan Hajnoczi , qemu-devel , Orit Wasserman , ronnie sahlberg , Paolo Bonzini On 06.01.2014 02:18, Stefan Hajnoczi wrote: > On Fri, Jan 03, 2014 at 12:28:31PM +0100, Peter Lieven wrote: >> On 03.01.2014 11:37, Stefan Hajnoczi wrote: >>> Looks good. In order to merge this new block driver qemu-iotests >>> support for nfs is required. That way the block driver can be exercised >>> and checked for regressions (I guess you performed manual testing >>> during development). >>> >>> Please see tests/qemu-iotests/common for examples of >>> NBD/SSH/Sheepdog/etc support. >>> >>> The qemu-iotests test suite with raw, qcow2, and vmdk formats should >>> work on top of NFS. Assuming you have an NFS server already running >>> on localhost, something like the following should succeed: >>> >>> cd tests/qemu-iotests >>> ln -s ../../x86_64-softmmu/qemu-system-x86_64 qemu >>> ln -s ../../qemu-img . >>> ln -s ../../qemu-io . >>> ./check -nfs # raw format by default >>> ./check -nfs -qcow2 >>> ./check -nfs -vmdk >>> >>> Maybe -nfs should take the base NFS URI as an argument to allow more >>> flexible test configurations. It's up to you. >>> >>> More info on qemu-iotests: http://qemu-project.org/Documentation/QemuIoTests >> it seems that several tests are broken since they use commands like >> rm -f or mv and have protocol generic. shall I fix this? > The _cleanup_test_img() function in common.rc has cases for > nbd/rbd/sheepdog. They don't invoke regular rm(1). Maybe you need to > add a case for NFS there too. > > If you find something broken and can fix it that's great. If you cannot > fix it please report it so the nbd/rbd/sheepdog/ssh/etc maintainers can > investigate. I was not thinking of the cleanup. Its mainly almost every test that does not deal with the RAW format that its not working. Please see my v2 of the patch for the NFS tests. In general, Ronnie and I thought of a LDPRELOAD library which enables every posix binary to talk to an NFS share. In this case even qcow2.py would work out of the box. Peter