From patchwork Tue Jun 27 12:34:10 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peter Lieven X-Patchwork-Id: 9811571 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 852E360329 for ; Tue, 27 Jun 2017 12:38:23 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 76D8828668 for ; Tue, 27 Jun 2017 12:38:23 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 6821D2867F; Tue, 27 Jun 2017 12:38:23 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id C6D1228668 for ; Tue, 27 Jun 2017 12:38:22 +0000 (UTC) Received: from localhost ([::1]:52644 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dPpl7-0007Ur-QL for patchwork-qemu-devel@patchwork.kernel.org; Tue, 27 Jun 2017 08:38:21 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42465) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dPphQ-00057f-Em for qemu-devel@nongnu.org; Tue, 27 Jun 2017 08:34:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dPphO-0005Hg-RZ for qemu-devel@nongnu.org; Tue, 27 Jun 2017 08:34:32 -0400 Received: from mx-v6.kamp.de ([2a02:248:0:51::16]:53826 helo=mx01.kamp.de) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dPphO-0005Gc-HS for qemu-devel@nongnu.org; Tue, 27 Jun 2017 08:34:30 -0400 Received: (qmail 22880 invoked by uid 89); 27 Jun 2017 12:34:29 -0000 Received: from [195.62.97.28] by client-16-kamp (envelope-from , uid 89) with qmail-scanner-2010/03/19-MF (clamdscan: 0.99.2/23511. avast: 1.2.2/17010300. spamassassin: 3.4.1. Clear:RC:1(195.62.97.28):. Processed in 0.227462 secs); 27 Jun 2017 12:34:29 -0000 Received: from smtp.kamp.de (HELO submission.kamp.de) ([195.62.97.28]) by mx01.kamp.de with ESMTPS (DHE-RSA-AES256-GCM-SHA384 encrypted); 27 Jun 2017 12:34:25 -0000 X-GL_Whitelist: yes Received: (qmail 29621 invoked from network); 27 Jun 2017 12:34:20 -0000 Received: from lieven-pc.kamp-intra.net (HELO lieven-pc) (relay@kamp.de@::ffff:172.21.12.60) by submission.kamp.de with ESMTPS (DHE-RSA-AES256-GCM-SHA384 encrypted) ESMTPA; 27 Jun 2017 12:34:20 -0000 Received: by lieven-pc (Postfix, from userid 1000) id 838A826C38; Tue, 27 Jun 2017 14:34:20 +0200 (CEST) From: Peter Lieven To: qemu-block@nongnu.org Date: Tue, 27 Jun 2017 14:34:10 +0200 Message-Id: <1498566850-7934-5-git-send-email-pl@kamp.de> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1498566850-7934-1-git-send-email-pl@kamp.de> References: <1498566850-7934-1-git-send-email-pl@kamp.de> X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 2a02:248:0:51::16 Subject: [Qemu-devel] [PATCH 4/4] block/qcow2: add zlib-fast compression algorithm X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, Peter Lieven , qemu-devel@nongnu.org, mreitz@redhat.com, den@openvz.org, lersek@redhat.com Errors-To: qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org Sender: "Qemu-devel" X-Virus-Scanned: ClamAV using ClamSMTP this adds support for optimized zlib settings which almost tripples the compression speed while maintaining about the same compressed size. Signed-off-by: Peter Lieven --- block/qcow2-cluster.c | 3 ++- block/qcow2.c | 11 +++++++++-- block/qcow2.h | 1 + qemu-img.texi | 1 + 4 files changed, 13 insertions(+), 3 deletions(-) diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c index ecb059b..d8e2378 100644 --- a/block/qcow2-cluster.c +++ b/block/qcow2-cluster.c @@ -1532,6 +1532,7 @@ static int decompress_buffer(uint8_t *out_buf, int out_buf_size, switch (compression_algorithm_id) { case QCOW2_COMPRESSION_ZLIB: + case QCOW2_COMPRESSION_ZLIB_FAST: memset(strm, 0, sizeof(*strm)); strm->next_in = (uint8_t *)buf; @@ -1539,7 +1540,7 @@ static int decompress_buffer(uint8_t *out_buf, int out_buf_size, strm->next_out = out_buf; strm->avail_out = out_buf_size; - ret = inflateInit2(strm, -12); + ret = inflateInit2(strm, -15); if (ret != Z_OK) { return -1; } diff --git a/block/qcow2.c b/block/qcow2.c index bd65582..f07d8f0 100644 --- a/block/qcow2.c +++ b/block/qcow2.c @@ -87,6 +87,8 @@ static uint32_t is_compression_algorithm_supported(char *algorithm) /* no algorithm means the old default of zlib compression * with 12 window bits */ return QCOW2_COMPRESSION_ZLIB; + } else if (!strcmp(algorithm, "zlib-fast")) { + return QCOW2_COMPRESSION_ZLIB_FAST; #ifdef CONFIG_LZO } else if (!strcmp(algorithm, "lzo")) { return QCOW2_COMPRESSION_LZO; @@ -2722,6 +2724,7 @@ qcow2_co_pwritev_compressed(BlockDriverState *bs, uint64_t offset, QEMUIOVector hd_qiov; struct iovec iov; z_stream strm; + int z_level = Z_DEFAULT_COMPRESSION, z_windowBits = -12; int ret, out_len = 0; uint8_t *buf, *out_buf = NULL, *local_buf = NULL, *work_buf = NULL; uint64_t cluster_offset; @@ -2749,13 +2752,17 @@ qcow2_co_pwritev_compressed(BlockDriverState *bs, uint64_t offset, } switch (s->compression_algorithm_id) { + case QCOW2_COMPRESSION_ZLIB_FAST: + z_level = Z_BEST_SPEED; + z_windowBits = -15; + /* fall-through */ case QCOW2_COMPRESSION_ZLIB: out_buf = g_malloc(s->cluster_size); /* best compression, small window, no zlib header */ memset(&strm, 0, sizeof(strm)); - ret = deflateInit2(&strm, Z_DEFAULT_COMPRESSION, - Z_DEFLATED, -12, + ret = deflateInit2(&strm, z_level, + Z_DEFLATED, z_windowBits, 9, Z_DEFAULT_STRATEGY); if (ret != 0) { ret = -EINVAL; diff --git a/block/qcow2.h b/block/qcow2.h index 716012c..a89f986 100644 --- a/block/qcow2.h +++ b/block/qcow2.h @@ -173,6 +173,7 @@ typedef struct Qcow2UnknownHeaderExtension { enum { QCOW2_COMPRESSION_ZLIB = 0xC0318301, QCOW2_COMPRESSION_LZO = 0xC0318302, + QCOW2_COMPRESSION_ZLIB_FAST = 0xC0318303, }; enum { diff --git a/qemu-img.texi b/qemu-img.texi index 043c1ba..83a5db2 100644 --- a/qemu-img.texi +++ b/qemu-img.texi @@ -627,6 +627,7 @@ The following options are available if support for the respective libraries has been enabled at compile time: zlib Uses standard zlib compression (default) + zlib-fast Uses zlib compression with optimized compression parameters lzo Uses LZO1X compression The compression algorithm can only be defined at image create time and cannot