From patchwork Mon Jan 27 18:28:07 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peter Lieven X-Patchwork-Id: 314456 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 2F7C62C00A5 for ; Tue, 28 Jan 2014 05:28:54 +1100 (EST) Received: from localhost ([::1]:32975 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W7qvT-0000Kl-8A for incoming@patchwork.ozlabs.org; Mon, 27 Jan 2014 13:28:51 -0500 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52851) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W7qv2-0000Jg-UL for qemu-devel@nongnu.org; Mon, 27 Jan 2014 13:28:33 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W7quu-0006A5-F3 for qemu-devel@nongnu.org; Mon, 27 Jan 2014 13:28:24 -0500 Received: from mx.ipv6.kamp.de ([2a02:248:0:51::16]:47834 helo=mx01.kamp.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W7quu-00069Z-3o for qemu-devel@nongnu.org; Mon, 27 Jan 2014 13:28:16 -0500 Received: (qmail 1564 invoked by uid 89); 27 Jan 2014 18:28:12 -0000 Received: from [195.62.97.28] by client-16-kamp (envelope-from , uid 89) with qmail-scanner-2010/03/19-MF (clamdscan: 0.98.1/18404. hbedv: 8.2.12.180/7.11.127.90. spamassassin: 3.3.1. Clear:RC:1(195.62.97.28):SA:0(-2.0/5.0):. Processed in 3.172551 secs); 27 Jan 2014 18:28:12 -0000 Received: from smtp.kamp.de (HELO submission.kamp.de) ([195.62.97.28]) by mx01.kamp.de with SMTP; 27 Jan 2014 18:28:07 -0000 X-GL_Whitelist: yes Received: (qmail 21572 invoked from network); 27 Jan 2014 18:28:05 -0000 Received: from ac37.vpn.kamp-intra.net (HELO ?172.20.250.37?) (pl@kamp.de@172.20.250.37) by submission.kamp.de with ESMTPS (DHE-RSA-CAMELLIA256-SHA encrypted) ESMTPA; 27 Jan 2014 18:28:05 -0000 Message-ID: <52E6A537.9050206@kamp.de> Date: Mon, 27 Jan 2014 19:28:07 +0100 From: Peter Lieven User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 MIME-Version: 1.0 To: Stefan Hajnoczi References: <1389608518-4813-1-git-send-email-pl@kamp.de> <20140127161145.GC23974@stefanha-thinkpad.redhat.com> In-Reply-To: <20140127161145.GC23974@stefanha-thinkpad.redhat.com> X-detected-operating-system: by eggs.gnu.org: Error: Malformed IPv6 address (bad octet value). X-Received-From: 2a02:248:0:51::16 Cc: Kevin Wolf , Fam Zheng , Jeff Cody , qemu-devel@nongnu.org, Max Reitz , owasserm@redhat.com, Federico Simoncelli , ronniesahlberg@gmail.com, Wenchao Xia Subject: Re: [Qemu-devel] [PATCHv6 0/6] block: add native support for NFS X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Am 27.01.2014 17:11, schrieb Stefan Hajnoczi: > On Mon, Jan 13, 2014 at 11:21:52AM +0100, Peter Lieven wrote: >> This adds v6 of the NFS protocol driver + qemu-iotest adjustments. >> >> v5->v6: >> - use internal qemu function to parse the NFS url [Kevin] >> - zero pad short reads [Kevin, Paolo] >> - added qemu-iotests patches for basic nfs protocol support >> >> v4->v5: >> - disussed with Ronnie and decided to move URL + Paramter parsing to LibNFS. >> This allows for URL parameter processing directly in LibNFS without altering >> the qemu NFS block driver. This bumps the version requirement for LibNFS >> to 1.9.0 though. >> - added a pointer to the LibNFS readme where additional information about >> ROOT privilidge requirements can be found as this raised a few concerns. >> - removed a trailing dot in an error statement [Fam]. >> >> v3->v4: >> - finally added full implementation of bdrv_get_allocated_file_size [Stefan] >> - removed trailing \n from error statements [Stefan] >> >> v2->v3: >> - rebased the stefanha/block >> - use pkg_config to check for libnfs (ignoring cflags which are broken in 1.8.0) [Stefan] >> - fixed NFSClient declaration [Stefan] >> - renamed Task variables to task [Stefan] >> - renamed NFSTask to NFSRPC [Ronnie] >> - do not update bs->total_sectors in nfs_co_writev [Stefan] >> - return -ENOMEM on all async call failures [Stefan,Ronnie] >> - fully implement ftruncate >> - use util/uri.c for URL parsing [Stefan] >> - reworked nfs_file_open_common to nfs_client_open which works on NFSClient [Stefan] >> - added a comment ot the connect message that libnfs support NFSv3 only at the moment. >> - DID NOT add full implementation of bdrv_get_allocated_file_size because >> we are not in a coroutine context and I cannot do an async call here. >> I could do a sync call if there would be a guarantee that no requests >> are in flight. [Stefan] >> >> v1->v2: >> - fixed block/Makefile.objs [Ronnie] >> - do not always register a read handler [Ronnie] >> - add support for reading beyond EOF [Fam] >> - fixed struct and paramter naming [Fam] >> - fixed overlong lines and whitespace errors [Fam] >> - return return status from libnfs whereever possible [Fam] >> - added comment why we set allocated_file_size to -ENOTSUP after write [Fam] >> - avoid segfault when parsing filname [Fam] >> - remove unused close_bh from NFSClient [Fam] >> - avoid dividing and mutliplying total_size by BDRV_SECTOR_SIZE in nfs_file_create [Fam] >> >> Peter Lieven (6): >> block: add native support for NFS >> qemu-iotests: change _supported_proto to file for various tests >> qemu-iotests: enable support for NFS protocol >> qemu-iotests: enable test 016 and 025 to work with NFS protocol >> qemu-iotests: fix expected output of test 067 >> qemu-iotests: blacklist test 020 for NFS protocol >> >> MAINTAINERS | 5 + >> block/Makefile.objs | 1 + >> block/nfs.c | 444 ++++++++++++++++++++++++++++++++++++++++++ >> configure | 26 +++ >> qapi-schema.json | 1 + >> tests/qemu-iotests/013 | 2 +- >> tests/qemu-iotests/014 | 2 +- >> tests/qemu-iotests/016 | 2 +- >> tests/qemu-iotests/018 | 2 +- >> tests/qemu-iotests/019 | 2 +- >> tests/qemu-iotests/020 | 7 +- >> tests/qemu-iotests/023 | 2 +- >> tests/qemu-iotests/024 | 2 +- >> tests/qemu-iotests/025 | 2 +- >> tests/qemu-iotests/026 | 2 +- >> tests/qemu-iotests/028 | 2 +- >> tests/qemu-iotests/031 | 2 +- >> tests/qemu-iotests/034 | 2 +- >> tests/qemu-iotests/036 | 2 +- >> tests/qemu-iotests/037 | 2 +- >> tests/qemu-iotests/038 | 2 +- >> tests/qemu-iotests/039 | 2 +- >> tests/qemu-iotests/043 | 2 +- >> tests/qemu-iotests/046 | 2 +- >> tests/qemu-iotests/052 | 2 +- >> tests/qemu-iotests/054 | 2 +- >> tests/qemu-iotests/059 | 2 +- >> tests/qemu-iotests/060 | 2 +- >> tests/qemu-iotests/061 | 2 +- >> tests/qemu-iotests/063 | 2 +- >> tests/qemu-iotests/067.out | 8 +- >> tests/qemu-iotests/069 | 2 +- >> tests/qemu-iotests/common | 22 ++- >> tests/qemu-iotests/common.rc | 3 + >> 34 files changed, 534 insertions(+), 33 deletions(-) >> create mode 100644 block/nfs.c > Any update on the qemu-iotests changes discussed in this thread? The > actual NFS patch looks fine. I would like to leave the rework of patch 5 to the maintainers. Maybe you can merge the rest of the series except for this change in patch 2. Peter > > Stefan diff --git a/tests/qemu-iotests/020 b/tests/qemu-iotests/020 index b3c86d8..a42f32f 100755 --- a/tests/qemu-iotests/020 +++ b/tests/qemu-iotests/020 @@ -43,7 +43,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15 # Any format supporting backing files _supported_fmt qcow qcow2 vmdk qed -_supported_proto generic +_supported_proto file _supported_os Linux I blacklist patch 20 for NFS manually in Patch 6 as Fam noticed.