From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45387) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xj6sH-0001Hi-QC for qemu-devel@nongnu.org; Tue, 28 Oct 2014 09:31:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Xj6sB-0007do-S2 for qemu-devel@nongnu.org; Tue, 28 Oct 2014 09:31:49 -0400 Received: from mx1.redhat.com ([209.132.183.28]:39401) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xj6sB-0007dZ-Km for qemu-devel@nongnu.org; Tue, 28 Oct 2014 09:31:43 -0400 Date: Tue, 28 Oct 2014 13:31:29 +0000 From: Stefan Hajnoczi Message-ID: <20141028133129.GA22805@stefanha-thinkpad.redhat.com> References: <1414401528-21884-1-git-send-email-pl@kamp.de> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="cWoXeonUoKmBZSoM" Content-Disposition: inline In-Reply-To: <1414401528-21884-1-git-send-email-pl@kamp.de> Subject: Re: [Qemu-devel] [PATCHv6 0/6] introduce max_transfer_length List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Lieven Cc: kwolf@redhat.com, famz@redhat.com, benoit@irqsave.net, armbru@redhat.com, qemu-devel@nongnu.org, ronniesahlberg@gmail.com, mreitz@redhat.com --cWoXeonUoKmBZSoM Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Oct 27, 2014 at 10:18:42AM +0100, Peter Lieven wrote: > This series adds the basics for introducing a maximum transfer length > to the block layer. Its main purpose is currently avoiding that > a multiwrite_merge exceeds the max_xfer_len of an attached iSCSI LUN. > This is a required bug fix. >=20 > Splitting up requests according to the max_transfer_length will follow > in a later series. >=20 > v5->v6: - fix parentheses in Patch 1 [Max] > - rename helper function in Patch 3 [Max] > - cap the limits to highest power of 2 fitting in INT_MAX [Max] > - removed full stops in error messages in Patch 6 [Max] >=20 > v4->v5: - Added check for (a !=3D 0 instead of !!a) [Max] and some commen= ts [Eric] > - Correctly limited max_transfer_length to INT_MAX in Patch 3 [Ma= x] > - Added Patch 5 and Patch 6. >=20 > v3->v4: introduce MIN_NON_ZERO to correctly calculate minimum of 2 limits. > v2->v3: remove Patch 2 completely [Paolo] > v1->v2: do not throw errors but generate trace events in Patch 2 [Paolo] >=20 > Peter Lieven (6): > util: introduce MIN_NON_ZERO > BlockLimits: introduce max_transfer_length > block/iscsi: set max_transfer_length > block: avoid creating oversized writes in multiwrite_merge > block/iscsi: use sector_limits_lun2qemu throughout > iscsi_refresh_limits > block/iscsi: check for oversized requests >=20 > block.c | 9 +++++++++ > block/iscsi.c | 49 ++++++++++++++++++++++++++++++++++-----= ------ > include/block/block_int.h | 3 +++ > include/qemu/osdep.h | 6 ++++++ > 4 files changed, 55 insertions(+), 12 deletions(-) Please use "PATCH v6" in the Subject line instead of "PATCHv6". My patch scanning tools missed the series. Thanks, applied to my block tree: https://github.com/stefanha/qemu/commits/block Stefan --cWoXeonUoKmBZSoM Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQEcBAEBAgAGBQJUT5qxAAoJEJykq7OBq3PIOXAH/3MuZkMpcbJFyWub99YfBCqU NuK/0baPyNHRR0jZ0/AuEbFpFsEWn2u5viLwOaTOvZlHL8jbn39l1fbsALVne+jH Ix1XkfJcKjpg7o3N46Jsj+bd+hrdyhQxfoYDnfHO5rwdG59y8n4AxebB+WAOtlvx nKkeIB+OMDJ2juzkoDR6jeBObkwS7xy6NaFc2GiK0dUQScKzFesHvK+/lQluVPNj 5qOem4X8jvmBzSNGpqEGiSmy4b5+tWXyAixC+wg8y/M37mHSv6utUvDGbapWrJnr R0+hl6Mc4mvC8HMBJ0EuPE0hO1CHwYUOFL009Eo5sxzBJ95xIa95f/8kJYD1uHU= =q7Cu -----END PGP SIGNATURE----- --cWoXeonUoKmBZSoM--