From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59780) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XhFI2-0004jD-GS for qemu-devel@nongnu.org; Thu, 23 Oct 2014 06:06:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XhFHv-0003F9-Fw for qemu-devel@nongnu.org; Thu, 23 Oct 2014 06:06:42 -0400 Received: from mx1.redhat.com ([209.132.183.28]:58470) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XhFHv-0003F2-8r for qemu-devel@nongnu.org; Thu, 23 Oct 2014 06:06:35 -0400 Received: from int-mx13.intmail.prod.int.phx2.redhat.com (int-mx13.intmail.prod.int.phx2.redhat.com [10.5.11.26]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s9NA6Ybm003013 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Thu, 23 Oct 2014 06:06:34 -0400 Date: Thu, 23 Oct 2014 12:06:32 +0200 From: Kevin Wolf Message-ID: <20141023100632.GI3522@noname.redhat.com> References: <1413982283-10186-1-git-send-email-mreitz@redhat.com> <1413982283-10186-7-git-send-email-mreitz@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1413982283-10186-7-git-send-email-mreitz@redhat.com> Subject: Re: [Qemu-devel] [PATCH v13 06/14] iotests: Omit length/offset test in 040 and 041 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Max Reitz Cc: qemu-devel@nongnu.org, Stefan Hajnoczi Am 22.10.2014 um 14:51 hat Max Reitz geschrieben: > As of a follow-up patch to this one, the length of a mirror block job > will no longer directly depend on the size of the block device; > therefore, drop these checks from this test. Instead, just check whether > the final offset equals the block job length. > > As 041 uses the wait_until_completed function from iotests.py, the same > applies there as well which in turn affects tests 030, 055 and 056. On > the other hand, a block job's length does not have to be related to the > length of the image file in the first place, so that check was > questionable anyway. > > Signed-off-by: Max Reitz > Reviewed-by: Eric Blake Possibly a result of rebasing over a long time, but 041 still has some checks of len left after this patch. Kevin