From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:53086)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <den@openvz.org>) id 1YqmG0-0007C2-6f
	for qemu-devel@nongnu.org; Fri, 08 May 2015 13:40:19 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <den@openvz.org>) id 1YqmFu-0006vd-FH
	for qemu-devel@nongnu.org; Fri, 08 May 2015 13:40:16 -0400
Received: from mx2.parallels.com ([199.115.105.18]:50561)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <den@openvz.org>) id 1YqmFu-0006jE-A7
	for qemu-devel@nongnu.org; Fri, 08 May 2015 13:40:10 -0400
Message-ID: <554CF4EC.8010002@openvz.org>
Date: Fri, 8 May 2015 20:39:56 +0300
From: "Denis V. Lunev" <den@openvz.org>
MIME-Version: 1.0
References: <1430207220-24458-1-git-send-email-den@openvz.org>
In-Reply-To: <1430207220-24458-1-git-send-email-den@openvz.org>
Content-Type: text/plain; charset="windows-1252"; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH v4 0/27] write/create for Parallels images
 with reasonable performance
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
Cc: Kevin Wolf <kwolf@redhat.com>, qemu-devel@nongnu.org, Stefan Hajnoczi <stefanha@redhat.com>, Roman Kagan <rkagan@parallels.com>

On 28/04/15 10:46, Denis V. Lunev wrote:
> This patchset provides an ability to create of/write to Parallels
> images and some testing of the new code. Readings and writings are
> optimized out and I expect the same or slightly better performance
> as qcow2.
>
> Changes from v4:
> - parallels format driver marked as bdrv_has_zero_init_1
> - added missed unlocks to parallels_co_readv/writev on error path, locking
>    is shortened and simplified
> - changed test number for created images
> - added check for bdrv_has_zero_init() and availability of bdrv_truncate()
>    in parallels_open() and proper error handling in alloc_cluster
> - some patch comments are improved
>
> Changes from v3:
> - fixed checkpatch warnings even in just moved code. I am tired of them
> - fixed contingency check in patch 18
>
> Changes from v2:
> - read performance is almost doubled (up to 360 Mb/sec), write performance
>    is improved by 15-20%
> - bat caching approach changed completely. bat_bitmap now contains the data
>    in on-disk format, which allows to use this data for metadata cache
> - incorrect close detection code is added (inuse field in the header)
> - very basic check consistency code added
>
> Changes from v1:
> - patches 13-19 added, which boosts performance from 800 KiB/sec to
>    near native performance
>
> Signed-off-by: Denis V. Lunev <den@openvz.org>
> CC: Roman Kagan <rkagan@parallels.com>
> CC: Kevin Wolf <kwolf@redhat.com>
> CC: Stefan Hajnoczi <stefanha@redhat.com>
>
ping