qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCHv8 0/5] block: add native support for NFS


From: Max Reitz
Subject: Re: [Qemu-devel] [PATCHv8 0/5] block: add native support for NFS
Date: Fri, 31 Jan 2014 21:46:08 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0

On 31.01.2014 21:41, Stefan Hajnoczi wrote:
On Fri, Jan 31, 2014 at 6:32 PM, Peter Lieven <address@hidden> wrote:
Am 31.01.2014 um 17:07 schrieb Stefan Hajnoczi <address@hidden>:

On Fri, Jan 31, 2014 at 12:38:48PM +0100, Peter Lieven wrote:
This adds v6 of the NFS protocol driver + qemu-iotest adjustments.

v7->v8:
- bump libnfs requirement to 1.9.2
- renamed NFSRPC->status to NFSRPC->ret [Benoit]
- avoid double invocation of nfs_client_close in nfs_file_create [Benoit]
- fixed error propagation in nfs_file_open [Benoit]
I was able to create a qcow2 image and boot a VM successfully over NFS.
This test is obviously new. It is not suitable for protocols other than file as 
it seems (as a lot
of other tests). The file not found error looks like it and not a problem with 
the driver itself.
I think it needs to be adjusted to work with file only for now.
I think you are right.  Max is the author of the tests, he can confirm
that the -o file.file.filename= syntax he used and the blkverify
driver only support the file protocol for now.

Yes, you are correct. Sorry for not checking whether this test is really generic, neither regarding the format tested nor the protocol.

Max

Peter: Please send another revision that restricts protocol to 'file'
for these tests.  It can be part of the patch that adjusts the other
test cases.

Stefan




reply via email to

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