Cover Letter Detail
Show a cover letter.
GET /api/covers/813502/?format=api
{ "id": 813502, "url": "http://patchwork.ozlabs.org/api/covers/813502/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/cover/20170913160333.23622-1-eblake@redhat.com/", "project": { "id": 14, "url": "http://patchwork.ozlabs.org/api/projects/14/?format=api", "name": "QEMU Development", "link_name": "qemu-devel", "list_id": "qemu-devel.nongnu.org", "list_email": "qemu-devel@nongnu.org", "web_url": "", "scm_url": "", "webscm_url": "", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20170913160333.23622-1-eblake@redhat.com>", "list_archive_url": null, "date": "2017-09-13T16:03:10", "name": "[v4,00/23] make bdrv_get_block_status byte-based", "submitter": { "id": 6591, "url": "http://patchwork.ozlabs.org/api/people/6591/?format=api", "name": "Eric Blake", "email": "eblake@redhat.com" }, "mbox": "http://patchwork.ozlabs.org/project/qemu-devel/cover/20170913160333.23622-1-eblake@redhat.com/mbox/", "series": [ { "id": 2944, "url": "http://patchwork.ozlabs.org/api/series/2944/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/list/?series=2944", "date": "2017-09-13T16:03:10", "name": "make bdrv_get_block_status byte-based", "version": 4, "mbox": "http://patchwork.ozlabs.org/series/2944/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/813502/comments/", "headers": { "Return-Path": "<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>", "X-Original-To": "incoming@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming@bilbo.ozlabs.org", "Authentication-Results": [ "ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)", "ext-mx02.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com", "ext-mx02.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=eblake@redhat.com" ], "Received": [ "from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xsmhj2CR6z9sBW\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 14 Sep 2017 02:05:07 +1000 (AEST)", "from localhost ([::1]:43314 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1dsA9w-0001Ii-4v\n\tfor incoming@patchwork.ozlabs.org; Wed, 13 Sep 2017 12:05:04 -0400", "from eggs.gnu.org ([2001:4830:134:3::10]:45453)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <eblake@redhat.com>) id 1dsA8o-00013O-Ro\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 12:03:59 -0400", "from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <eblake@redhat.com>) id 1dsA8l-0005iP-Lw\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 12:03:54 -0400", "from mx1.redhat.com ([209.132.183.28]:37468)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from <eblake@redhat.com>)\n\tid 1dsA8b-0005al-L7; Wed, 13 Sep 2017 12:03:41 -0400", "from smtp.corp.redhat.com\n\t(int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mx1.redhat.com (Postfix) with ESMTPS id 61A7AEEACA;\n\tWed, 13 Sep 2017 16:03:40 +0000 (UTC)", "from red.redhat.com (ovpn-120-201.rdu2.redhat.com [10.10.120.201])\n\tby smtp.corp.redhat.com (Postfix) with ESMTP id E08E764433;\n\tWed, 13 Sep 2017 16:03:35 +0000 (UTC)" ], "DMARC-Filter": "OpenDMARC Filter v1.3.2 mx1.redhat.com 61A7AEEACA", "From": "Eric Blake <eblake@redhat.com>", "To": "qemu-devel@nongnu.org", "Date": "Wed, 13 Sep 2017 11:03:10 -0500", "Message-Id": "<20170913160333.23622-1-eblake@redhat.com>", "X-Scanned-By": "MIMEDefang 2.79 on 10.5.11.11", "X-Greylist": "Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.26]);\n\tWed, 13 Sep 2017 16:03:40 +0000 (UTC)", "X-detected-operating-system": "by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic]\n\t[fuzzy]", "X-Received-From": "209.132.183.28", "Subject": "[Qemu-devel] [PATCH v4 00/23] make bdrv_get_block_status byte-based", "X-BeenThere": "qemu-devel@nongnu.org", "X-Mailman-Version": "2.1.21", "Precedence": "list", "List-Id": "<qemu-devel.nongnu.org>", "List-Unsubscribe": "<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<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>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>", "Cc": "kwolf@redhat.com, jsnow@redhat.com, famz@redhat.com,\n\tqemu-block@nongnu.org", "Errors-To": "qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org", "Sender": "\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>" }, "content": "There are patches floating around to add NBD_CMD_BLOCK_STATUS,\nbut NBD wants to report status on byte granularity (even if the\nreporting will probably be naturally aligned to sectors or even\nmuch higher levels). I've therefore started the task of\nconverting our block status code to report at a byte granularity\nrather than sectors.\n\nNow that 2.11 is open, I'm rebasing/reposting the remaining patches.\n\nThe overall conversion currently looks like:\npart 1: bdrv_is_allocated (merged, commit 51b0a488)\npart 2: dirty-bitmap (v7 is posted [1], mostly reviewed)\npart 3: bdrv_get_block_status (this series, v3 at [2])\npart 4: .bdrv_co_block_status (v2 is posted [3], but needs a rebase)\n\nAvailable as a tag at:\ngit fetch git://repo.or.cz/qemu/ericb.git nbd-byte-status-v4\n\nBased-on: <20170912203119.24166-1-eblake@redhat.com>\n([PATCH v7 00/20] make dirty-bitmap byte-based)\n\n[1] https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg03160.html\n[2] https://lists.gnu.org/archive/html/qemu-devel/2017-07/msg03853.html\n[3] https://lists.gnu.org/archive/html/qemu-devel/2017-07/msg04370.html\n\nSince v3:\n- Minor rebasing\n\n001/23:[----] [-C] 'block: Allow NULL file for bdrv_get_block_status()'\n002/23:[----] [--] 'block: Add flag to avoid wasted work in bdrv_is_allocated()'\n003/23:[----] [-C] 'block: Make bdrv_round_to_clusters() signature more useful'\n004/23:[----] [--] 'qcow2: Switch is_zero_sectors() to byte-based'\n005/23:[----] [--] 'block: Switch bdrv_make_zero() to byte-based'\n006/23:[----] [--] 'qemu-img: Switch get_block_status() to byte-based'\n007/23:[----] [--] 'block: Convert bdrv_get_block_status() to bytes'\n008/23:[----] [--] 'block: Switch bdrv_co_get_block_status() to byte-based'\n009/23:[----] [--] 'block: Switch BdrvCoGetBlockStatusData to byte-based'\n010/23:[----] [--] 'block: Switch bdrv_common_block_status_above() to byte-based'\n011/23:[----] [--] 'block: Switch bdrv_co_get_block_status_above() to byte-based'\n012/23:[0002] [FC] 'block: Convert bdrv_get_block_status_above() to bytes'\n013/23:[----] [--] 'qemu-img: Simplify logic in img_compare()'\n014/23:[----] [--] 'qemu-img: Speed up compare on pre-allocated larger file'\n015/23:[----] [--] 'qemu-img: Add find_nonzero()'\n016/23:[----] [--] 'qemu-img: Drop redundant error message in compare'\n017/23:[----] [--] 'qemu-img: Change check_empty_sectors() to byte-based'\n018/23:[----] [--] 'qemu-img: Change compare_sectors() to be byte-based'\n019/23:[----] [--] 'qemu-img: Change img_rebase() to be byte-based'\n020/23:[----] [--] 'qemu-img: Change img_compare() to be byte-based'\n021/23:[----] [--] 'block: Align block status requests'\n022/23:[----] [--] 'block: Relax bdrv_aligned_preadv() assertion'\n023/23:[----] [--] 'qemu-io: Relax 'alloc' now that block-status doesn't assert'\nEric Blake (23):\n block: Allow NULL file for bdrv_get_block_status()\n block: Add flag to avoid wasted work in bdrv_is_allocated()\n block: Make bdrv_round_to_clusters() signature more useful\n qcow2: Switch is_zero_sectors() to byte-based\n block: Switch bdrv_make_zero() to byte-based\n qemu-img: Switch get_block_status() to byte-based\n block: Convert bdrv_get_block_status() to bytes\n block: Switch bdrv_co_get_block_status() to byte-based\n block: Switch BdrvCoGetBlockStatusData to byte-based\n block: Switch bdrv_common_block_status_above() to byte-based\n block: Switch bdrv_co_get_block_status_above() to byte-based\n block: Convert bdrv_get_block_status_above() to bytes\n qemu-img: Simplify logic in img_compare()\n qemu-img: Speed up compare on pre-allocated larger file\n qemu-img: Add find_nonzero()\n qemu-img: Drop redundant error message in compare\n qemu-img: Change check_empty_sectors() to byte-based\n qemu-img: Change compare_sectors() to be byte-based\n qemu-img: Change img_rebase() to be byte-based\n qemu-img: Change img_compare() to be byte-based\n block: Align block status requests\n block: Relax bdrv_aligned_preadv() assertion\n qemu-io: Relax 'alloc' now that block-status doesn't assert\n\n include/block/block.h | 26 ++--\n include/block/block_int.h | 11 +-\n block/io.c | 287 ++++++++++++++++++++---------------\n block/blkdebug.c | 13 +-\n block/mirror.c | 24 +--\n block/qcow2-cluster.c | 2 +-\n block/qcow2.c | 53 +++----\n qemu-img.c | 365 ++++++++++++++++++++-------------------------\n qemu-io-cmds.c | 13 --\n block/trace-events | 2 +-\n tests/qemu-iotests/074.out | 2 -\n tests/qemu-iotests/177 | 12 +-\n tests/qemu-iotests/177.out | 19 ++-\n 13 files changed, 420 insertions(+), 409 deletions(-)" }