From: Peter Lieven <pl@kamp.de>
To: qemu-devel@nongnu.org
Cc: kwolf@redhat.com, pbonzini@redhat.com, Peter Lieven <pl@kamp.de>,
ronniesahlberg@gmail.com, stefanha@redhat.com
Subject: [Qemu-devel] [PATCHv2 00/11] iscsi/qemu-img/block-migration enhancements
Date: Thu, 27 Jun 2013 15:11:24 +0200 [thread overview]
Message-ID: <1372338695-411-1-git-send-email-pl@kamp.de> (raw)
this series adds logical block provisioning functions to the iscsi layer.
it adds the ability to write sparse images to iscsi targets which
support UNMAP and LBPRZ via qemu-img. additional block-migration is
enhanced to support an encoding for zero blocks on the network
and efficiently writing those zero blocks at the destination if
the driver supports it.
many thanks to Paolo, Kevin and Ronnie for their useful comments.
changes in v2:
- added patch to read unmap limits from block limits VPD page
- use a generic callback and new framework for coroutines that
can also be used for further porting of the remaining aio
routines. iscsi_co_{write_zeroes,is_allocated} use this new
framework. commands are now also retried on check conditions.
- iscsi_co_write_zeroes checks now for BDRV_O_UNMAP and
if the request is not too big for a single UNMAP call.
- added a patch to let iscsi_create unmap the whole device
if thin provisioning is supported and the unmapped blocks
read as zero. in this case return .has_zero_init = 1.
- added a patch to let aio_discard silently fail if UNMAP
is not supported or the request is too big.
- factored out almost all conversions between qemu and lun
sector size.
- changed the alignment checks from assert() to failing
the request.
- dropped the qemu-img patch that called out to bdrv_write_zeroes.
Peter
Peter Lieven (11):
iscsi: add logical block provisioning information to iscsilun
iscsi: read unmap info from block limits vpd page
iscsi: add bdrv_co_is_allocated
iscsi: add bdrv_co_write_zeroes
block: add bdrv_write_zeroes()
block/raw: add bdrv_co_write_zeroes
iscsi: let bdrv_create conditionally zero out the device
block-migration: efficiently encode zero blocks
iscsi: factor out sector conversions
iscsi: ignore aio_discard if unsupported
iscsi: assert that sectors are aligned to LUN blocksize
block-migration.c | 29 +++-
block.c | 27 ++-
block/iscsi.c | 365 ++++++++++++++++++++++++++++++++++++++++-
block/raw.c | 8 +
include/block/block.h | 2 +
include/migration/qemu-file.h | 1 +
savevm.c | 2 +-
7 files changed, 414 insertions(+), 20 deletions(-)
--
1.7.9.5
next reply other threads:[~2013-06-27 13:12 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-27 13:11 Peter Lieven [this message]
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 01/11] iscsi: add logical block provisioning information to iscsilun Peter Lieven
2013-07-01 13:35 ` Stefan Hajnoczi
2013-07-01 16:08 ` Peter Lieven
2013-07-10 9:19 ` Kevin Wolf
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 02/11] iscsi: read unmap info from block limits vpd page Peter Lieven
2013-07-03 3:43 ` ronnie sahlberg
2013-07-03 21:23 ` Peter Lieven
2013-07-04 12:37 ` Paolo Bonzini
2013-07-04 21:07 ` Peter Lieven
2013-07-05 6:28 ` Paolo Bonzini
2013-07-05 7:11 ` ronnie sahlberg
2013-07-06 22:15 ` Peter Lieven
2013-07-06 23:23 ` ronnie sahlberg
2013-07-10 9:23 ` Kevin Wolf
2013-07-10 9:25 ` Kevin Wolf
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 03/11] iscsi: add bdrv_co_is_allocated Peter Lieven
2013-07-01 13:46 ` Stefan Hajnoczi
2013-07-01 16:00 ` Peter Lieven
2013-07-10 9:41 ` Kevin Wolf
2013-07-10 13:49 ` Peter Lieven
2013-07-10 14:43 ` Kevin Wolf
2013-07-10 14:49 ` Peter Lieven
2013-07-10 14:54 ` Kevin Wolf
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 04/11] iscsi: add bdrv_co_write_zeroes Peter Lieven
2013-07-10 9:54 ` Kevin Wolf
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 05/11] block: add bdrv_write_zeroes() Peter Lieven
2013-07-10 9:56 ` Kevin Wolf
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 06/11] block/raw: add bdrv_co_write_zeroes Peter Lieven
2013-07-10 9:57 ` Kevin Wolf
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 07/11] iscsi: let bdrv_create conditionally zero out the device Peter Lieven
2013-07-01 13:58 ` Stefan Hajnoczi
2013-07-01 20:20 ` Paolo Bonzini
2013-07-01 21:36 ` Peter Lieven
2013-07-02 9:22 ` Paolo Bonzini
2013-07-02 10:36 ` Peter Lieven
2013-07-02 10:49 ` Paolo Bonzini
2013-07-02 10:56 ` Peter Lieven
2013-07-02 11:04 ` Paolo Bonzini
2013-07-02 11:18 ` Peter Lieven
2013-07-10 10:14 ` Kevin Wolf
2013-07-10 13:52 ` Peter Lieven
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 08/11] block-migration: efficiently encode zero blocks Peter Lieven
2013-07-01 14:13 ` Stefan Hajnoczi
2013-07-01 15:55 ` Peter Lieven
2013-07-02 7:40 ` Stefan Hajnoczi
2013-07-02 10:51 ` Paolo Bonzini
2013-07-01 16:09 ` Peter Lieven
2013-07-02 7:36 ` Stefan Hajnoczi
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 09/11] iscsi: factor out sector conversions Peter Lieven
2013-07-10 11:29 ` Kevin Wolf
2013-07-10 14:07 ` Peter Lieven
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 10/11] iscsi: ignore aio_discard if unsupported Peter Lieven
2013-07-10 11:33 ` Kevin Wolf
2013-07-10 14:04 ` Peter Lieven
2013-07-10 14:28 ` Kevin Wolf
2013-07-10 14:49 ` Peter Lieven
2013-07-10 14:58 ` Kevin Wolf
2013-07-10 20:31 ` Peter Lieven
2013-06-27 13:11 ` [Qemu-devel] [PATCHv2 11/11] iscsi: assert that sectors are aligned to LUN blocksize Peter Lieven
2013-07-01 14:35 ` Stefan Hajnoczi
2013-07-01 15:59 ` Peter Lieven
2013-07-02 7:44 ` Stefan Hajnoczi
2013-07-02 8:28 ` Peter Lieven
2013-07-02 10:44 ` Paolo Bonzini
2013-07-02 10:49 ` Peter Lieven
2013-07-02 10:53 ` Paolo Bonzini
2013-07-10 11:38 ` Kevin Wolf
2013-07-10 14:02 ` Peter Lieven
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1372338695-411-1-git-send-email-pl@kamp.de \
--to=pl@kamp.de \
--cc=kwolf@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=ronniesahlberg@gmail.com \
--cc=stefanha@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).