[{"id":1767598,"web_url":"http://patchwork.ozlabs.org/comment/1767598/","msgid":"<83c3006b-d62e-441b-57f1-dea64b141c11@redhat.com>","list_archive_url":null,"date":"2017-09-13T06:47:32","subject":"Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests","submitter":{"id":66152,"url":"http://patchwork.ozlabs.org/api/people/66152/","name":"Thomas Huth","email":"thuth@redhat.com"},"content":"On 12.09.2017 16:44, Paolo Bonzini wrote:\n> The purpose of this series is to separate the \"check\" sources from\n> the tests. After these patches, common.config is reduced to simple\n> shell initialization, and common.rc is only included by the tests.\n> \n> Along the way, a lot of dead code is removed too.\n> \n> In v2, the following patches:\n> \n> qemu-iotests: do not do useless search for QEMU_*_PROG\n> qemu-iotests: do not search for binaries in the current directory\n> qemu-iotests: include common.env and common.config early\n> \n> have been replaced by \"qemu-iotests: cleanup and fix search for programs\",\n> which also preserves the behavior of searching for programs as symlinks\n> in the current directory.\n> \n> Paolo\n> \n> Paolo Bonzini (10):\n> qemu-iotests: remove dead code\n> qemu-iotests: get rid of AWK_PROG\n> qemu-iotests: move \"check\" code out of common.rc\n> qemu-iotests: cleanup and fix search for programs\n> qemu-iotests: limit non-_PROG-suffixed variables to common.rc\n> qemu-iotests: do not include common.rc in \"check\"\n> qemu-iotests: disintegrate more parts of common.config\n> qemu-iotests: fix uninitialized variable\n> qemu-iotests: get rid of $iam\n> qemu-iotests: merge \"check\" and \"common\"\n> \n> tests/qemu-iotests/039.out | 10 +-\n> tests/qemu-iotests/061.out | 4 +-\n> tests/qemu-iotests/137.out | 2 +-\n> tests/qemu-iotests/check | 575 ++++++++++++++++++++++++++++++++++-----\n> tests/qemu-iotests/common | 459 -------------------------------\n> tests/qemu-iotests/common.config | 206 +-------------\n> tests/qemu-iotests/common.qemu | 1 +\n> tests/qemu-iotests/common.rc | 205 +++++++-------\n\nMeta comment: Could we maybe also rename \"tests/qemu-iotests\" to\n\"tests/iotests\" ? The \"qemu\" prefix sounds always very superfluous to me\nhere...\n\n Thomas","headers":{"Return-Path":"","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=)","ext-mx03.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com","ext-mx03.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=thuth@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 3xsXL06fVjz9s7M\n\tfor ;\n\tWed, 13 Sep 2017 16:48:06 +1000 (AEST)","from localhost ([::1]:40425 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t)\n\tid 1ds1St-00032f-HM\n\tfor incoming@patchwork.ozlabs.org; Wed, 13 Sep 2017 02:48:03 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:33247)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from ) id 1ds1SZ-00031h-1L\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 02:47:44 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from ) id 1ds1SY-0005Bw-6U\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 02:47:43 -0400","from mx1.redhat.com ([209.132.183.28]:57920)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from )\n\tid 1ds1SS-00057B-Sn; Wed, 13 Sep 2017 02:47:37 -0400","from smtp.corp.redhat.com\n\t(int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13])\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 B1E337EBD5;\n\tWed, 13 Sep 2017 06:47:35 +0000 (UTC)","from [10.36.116.123] (ovpn-116-123.ams2.redhat.com [10.36.116.123])\n\tby smtp.corp.redhat.com (Postfix) with ESMTPS id A4DB8781E6;\n\tWed, 13 Sep 2017 06:47:34 +0000 (UTC)"],"DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com B1E337EBD5","To":"Paolo Bonzini , qemu-devel@nongnu.org","References":"<20170912144459.11359-1-pbonzini@redhat.com>","From":"Thomas Huth ","Message-ID":"<83c3006b-d62e-441b-57f1-dea64b141c11@redhat.com>","Date":"Wed, 13 Sep 2017 08:47:32 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170912144459.11359-1-pbonzini@redhat.com>","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-US","Content-Transfer-Encoding":"7bit","X-Scanned-By":"MIMEDefang 2.79 on 10.5.11.13","X-Greylist":"Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.27]);\n\tWed, 13 Sep 2017 06:47:35 +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":"Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"kwolf@redhat.com, qemu-block@nongnu.org","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t"}},{"id":1767656,"web_url":"http://patchwork.ozlabs.org/comment/1767656/","msgid":"<20170913075914.GA3929@lemon>","list_archive_url":null,"date":"2017-09-13T07:59:14","subject":"Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests","submitter":{"id":24872,"url":"http://patchwork.ozlabs.org/api/people/24872/","name":"Fam Zheng","email":"famz@redhat.com"},"content":"On Wed, 09/13 08:47, Thomas Huth wrote:\n> Meta comment: Could we maybe also rename \"tests/qemu-iotests\" to\n> \"tests/iotests\" ? The \"qemu\" prefix sounds always very superfluous to me\n> here...\n\nSounds good, and saves typing for when this path is manually entered. But maybe\nkeep tests/qemu-iotests as a symlink to keep old scripts happy?\n\nFam","headers":{"Return-Path":"","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=)","ext-mx05.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com","ext-mx05.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=famz@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 3xsYwr2cTpz9sPm\n\tfor ;\n\tWed, 13 Sep 2017 17:59:56 +1000 (AEST)","from localhost ([::1]:40734 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t)\n\tid 1ds2aQ-00043I-H8\n\tfor incoming@patchwork.ozlabs.org; Wed, 13 Sep 2017 03:59:54 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:48878)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from ) id 1ds2Zw-00041b-Sp\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 03:59:25 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from ) id 1ds2Zw-0000kO-41\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 03:59:24 -0400","from mx1.redhat.com ([209.132.183.28]:36546)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from )\n\tid 1ds2Zp-0000bU-W3; Wed, 13 Sep 2017 03:59:18 -0400","from smtp.corp.redhat.com\n\t(int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14])\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 D6AEE329E;\n\tWed, 13 Sep 2017 07:59:16 +0000 (UTC)","from localhost (ovpn-12-56.pek2.redhat.com [10.72.12.56])\n\tby smtp.corp.redhat.com (Postfix) with ESMTP id 1409D5D976;\n\tWed, 13 Sep 2017 07:59:15 +0000 (UTC)"],"DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com D6AEE329E","Date":"Wed, 13 Sep 2017 15:59:14 +0800","From":"Fam Zheng ","To":"Thomas Huth ","Message-ID":"<20170913075914.GA3929@lemon>","References":"<20170912144459.11359-1-pbonzini@redhat.com>\n\t<83c3006b-d62e-441b-57f1-dea64b141c11@redhat.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<83c3006b-d62e-441b-57f1-dea64b141c11@redhat.com>","User-Agent":"Mutt/1.8.3 (2017-05-23)","X-Scanned-By":"MIMEDefang 2.79 on 10.5.11.14","X-Greylist":"Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.29]);\n\tWed, 13 Sep 2017 07:59:17 +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":"Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"kwolf@redhat.com, Paolo Bonzini ,\n\tqemu-devel@nongnu.org, qemu-block@nongnu.org","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t"}},{"id":1768219,"web_url":"http://patchwork.ozlabs.org/comment/1768219/","msgid":"<3c085442-744b-e08c-279c-fc0c74559551@redhat.com>","list_archive_url":null,"date":"2017-09-13T22:06:08","subject":"Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests","submitter":{"id":2701,"url":"http://patchwork.ozlabs.org/api/people/2701/","name":"Paolo Bonzini","email":"pbonzini@redhat.com"},"content":"On 13/09/2017 09:59, Fam Zheng wrote:\n> On Wed, 09/13 08:47, Thomas Huth wrote:\n>> Meta comment: Could we maybe also rename \"tests/qemu-iotests\" to\n>> \"tests/iotests\" ? The \"qemu\" prefix sounds always very superfluous to me\n>> here...\n> \n> Sounds good, and saves typing for when this path is manually entered. But maybe\n> keep tests/qemu-iotests as a symlink to keep old scripts happy?\n\nThe simplest way to keep old scripts happy would be too keep the prefix\n:) even though it does sound unnecessary.\n\nPaolo","headers":{"Return-Path":"","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=)","ext-mx07.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com","ext-mx07.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=pbonzini@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 3xswk60M4xz9ryQ\n\tfor ;\n\tThu, 14 Sep 2017 08:06:51 +1000 (AEST)","from localhost ([::1]:44834 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t)\n\tid 1dsFnz-0006yL-SW\n\tfor incoming@patchwork.ozlabs.org; Wed, 13 Sep 2017 18:06:47 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:56314)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from ) id 1dsFna-0006vr-DF\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 18:06:23 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from ) id 1dsFnZ-00079C-DT\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 18:06:22 -0400","from mx1.redhat.com ([209.132.183.28]:49310)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from )\n\tid 1dsFnU-000773-6O; Wed, 13 Sep 2017 18:06:16 -0400","from smtp.corp.redhat.com\n\t(int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14])\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 11E18C047B97;\n\tWed, 13 Sep 2017 22:06:15 +0000 (UTC)","from [10.36.116.117] (ovpn-116-117.ams2.redhat.com [10.36.116.117])\n\tby smtp.corp.redhat.com (Postfix) with ESMTPS id 9FE1C5D724;\n\tWed, 13 Sep 2017 22:06:11 +0000 (UTC)"],"DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com 11E18C047B97","To":"Fam Zheng , Thomas Huth ","References":"<20170912144459.11359-1-pbonzini@redhat.com>\n\t<83c3006b-d62e-441b-57f1-dea64b141c11@redhat.com>\n\t<20170913075914.GA3929@lemon>","From":"Paolo Bonzini ","Message-ID":"<3c085442-744b-e08c-279c-fc0c74559551@redhat.com>","Date":"Thu, 14 Sep 2017 00:06:08 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<20170913075914.GA3929@lemon>","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-US","Content-Transfer-Encoding":"7bit","X-Scanned-By":"MIMEDefang 2.79 on 10.5.11.14","X-Greylist":"Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.31]);\n\tWed, 13 Sep 2017 22:06:15 +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":"Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"kwolf@redhat.com, qemu-devel@nongnu.org, qemu-block@nongnu.org","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t"}},{"id":1768223,"web_url":"http://patchwork.ozlabs.org/comment/1768223/","msgid":"","list_archive_url":null,"date":"2017-09-13T22:14:12","subject":"Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests","submitter":{"id":6591,"url":"http://patchwork.ozlabs.org/api/people/6591/","name":"Eric Blake","email":"eblake@redhat.com"},"content":"On 09/13/2017 05:06 PM, Paolo Bonzini wrote:\n> On 13/09/2017 09:59, Fam Zheng wrote:\n>> On Wed, 09/13 08:47, Thomas Huth wrote:\n>>> Meta comment: Could we maybe also rename \"tests/qemu-iotests\" to\n>>> \"tests/iotests\" ? The \"qemu\" prefix sounds always very superfluous to me\n>>> here...\n>>\n>> Sounds good, and saves typing for when this path is manually entered. But maybe\n>> keep tests/qemu-iotests as a symlink to keep old scripts happy?\n> \n> The simplest way to keep old scripts happy would be too keep the prefix\n> :) even though it does sound unnecessary.\n\nCan't someone manually add the symlink to keep their own scripts working\nboth before and after the change, even if we don't commit the symlink\ninto git?","headers":{"Return-Path":"","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=)","ext-mx08.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com","ext-mx08.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 3xswvR029qz9sNw\n\tfor ;\n\tThu, 14 Sep 2017 08:14:59 +1000 (AEST)","from localhost ([::1]:44857 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t)\n\tid 1dsFvt-0001t3-3r\n\tfor incoming@patchwork.ozlabs.org; Wed, 13 Sep 2017 18:14:57 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:59784)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from ) id 1dsFvN-0001oB-Ed\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 18:14:26 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from ) id 1dsFvM-0007EB-CX\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 18:14:25 -0400","from mx1.redhat.com ([209.132.183.28]:58690)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from )\n\tid 1dsFvH-00077y-PG; Wed, 13 Sep 2017 18:14:19 -0400","from smtp.corp.redhat.com\n\t(int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12])\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 87D3FC0587C5;\n\tWed, 13 Sep 2017 22:14:16 +0000 (UTC)","from [10.10.120.201] (ovpn-120-201.rdu2.redhat.com [10.10.120.201])\n\tby smtp.corp.redhat.com (Postfix) with ESMTP id 74C0F60BE4;\n\tWed, 13 Sep 2017 22:14:13 +0000 (UTC)"],"DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com 87D3FC0587C5","To":"Paolo Bonzini , Fam Zheng ,\n\tThomas Huth ","References":"<20170912144459.11359-1-pbonzini@redhat.com>\n\t<83c3006b-d62e-441b-57f1-dea64b141c11@redhat.com>\n\t<20170913075914.GA3929@lemon>\n\t<3c085442-744b-e08c-279c-fc0c74559551@redhat.com>","From":"Eric Blake ","Openpgp":"url=http://people.redhat.com/eblake/eblake.gpg","Organization":"Red Hat, Inc.","Message-ID":"","Date":"Wed, 13 Sep 2017 17:14:12 -0500","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<3c085442-744b-e08c-279c-fc0c74559551@redhat.com>","Content-Type":"multipart/signed; micalg=pgp-sha256;\n\tprotocol=\"application/pgp-signature\";\n\tboundary=\"mETuNXGCrVeqUerWoUHX9h80qkgJKx3iT\"","X-Scanned-By":"MIMEDefang 2.79 on 10.5.11.12","X-Greylist":"Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.32]);\n\tWed, 13 Sep 2017 22:14:16 +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","X-Content-Filtered-By":"Mailman/MimeDel 2.1.21","Subject":"Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"kwolf@redhat.com, qemu-devel@nongnu.org, qemu-block@nongnu.org","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t"}},{"id":1778291,"web_url":"http://patchwork.ozlabs.org/comment/1778291/","msgid":"<8315593b-4ae8-b034-01d6-d6f66925b6bb@redhat.com>","list_archive_url":null,"date":"2017-10-02T12:32:26","subject":"Re: [Qemu-devel] [Qemu-block] [PATCH v2 00/10] cleanup qemu-iotests","submitter":{"id":2701,"url":"http://patchwork.ozlabs.org/api/people/2701/","name":"Paolo Bonzini","email":"pbonzini@redhat.com"},"content":"On 12/09/2017 16:44, Paolo Bonzini wrote:\n> The purpose of this series is to separate the \"check\" sources from\n> the tests. After these patches, common.config is reduced to simple\n> shell initialization, and common.rc is only included by the tests.\n> \n> Along the way, a lot of dead code is removed too.\n> \n> In v2, the following patches:\n> \n> qemu-iotests: do not do useless search for QEMU_*_PROG\n> qemu-iotests: do not search for binaries in the current directory\n> qemu-iotests: include common.env and common.config early\n> \n> have been replaced by \"qemu-iotests: cleanup and fix search for programs\",\n> which also preserves the behavior of searching for programs as symlinks\n> in the current directory.\n> \n> Paolo\n\nKevin,\n\nping?\n\nPaolo\n\n> Paolo Bonzini (10):\n> qemu-iotests: remove dead code\n> qemu-iotests: get rid of AWK_PROG\n> qemu-iotests: move \"check\" code out of common.rc\n> qemu-iotests: cleanup and fix search for programs\n> qemu-iotests: limit non-_PROG-suffixed variables to common.rc\n> qemu-iotests: do not include common.rc in \"check\"\n> qemu-iotests: disintegrate more parts of common.config\n> qemu-iotests: fix uninitialized variable\n> qemu-iotests: get rid of $iam\n> qemu-iotests: merge \"check\" and \"common\"\n> \n> tests/qemu-iotests/039.out | 10 +-\n> tests/qemu-iotests/061.out | 4 +-\n> tests/qemu-iotests/137.out | 2 +-\n> tests/qemu-iotests/check | 575 ++++++++++++++++++++++++++++++++++-----\n> tests/qemu-iotests/common | 459 -------------------------------\n> tests/qemu-iotests/common.config | 206 +-------------\n> tests/qemu-iotests/common.qemu | 1 +\n> tests/qemu-iotests/common.rc | 205 +++++++-------\n> 8 files changed, 621 insertions(+), 841 deletions(-)\n> delete mode 100644 tests/qemu-iotests/common\n>","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=gnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@gnu.org;\n\treceiver=)","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 3y5M5X0qHRz9t48\n\tfor ;\n\tMon, 2 Oct 2017 23:33:17 +1100 (AEDT)","from localhost ([::1]:52082 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t)\n\tid 1dyzuJ-0007ac-7z\n\tfor incoming@patchwork.ozlabs.org; Mon, 02 Oct 2017 08:33:11 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:39598)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from ) id 1dyztm-0007Y3-OS\n\tfor qemu-devel@nongnu.org; Mon, 02 Oct 2017 08:32:42 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from ) id 1dyztg-0001rI-Tc\n\tfor qemu-devel@nongnu.org; Mon, 02 Oct 2017 08:32:38 -0400","from mail-wm0-f54.google.com ([74.125.82.54]:45742)\n\tby eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.71) (envelope-from ) id 1dyztg-0001qM-Ms\n\tfor qemu-devel@nongnu.org; Mon, 02 Oct 2017 08:32:32 -0400","by mail-wm0-f54.google.com with SMTP id q124so10292073wmb.0\n\tfor ; Mon, 02 Oct 2017 05:32:31 -0700 (PDT)","from [192.168.10.165]\n\t(dynamic-adsl-78-12-246-117.clienti.tiscali.it.\n\t[78.12.246.117]) by smtp.gmail.com with ESMTPSA id\n\to138sm8723957wmg.36.2017.10.02.05.32.28\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tMon, 02 Oct 2017 05:32:28 -0700 (PDT)"],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:subject:from:to:cc:references:message-id:date\n\t:user-agent:mime-version:in-reply-to:content-language\n\t:content-transfer-encoding;\n\tbh=65tzfv8mOTjKbGLpTTmPQa7caufrco7Ttovkhsqp62Y=;\n\tb=XvQa758eBt3gse252tE0vpF2D6Hx3YWhRoJETHAyz53uCS6Dgc79hGAzrc0xOTDM4I\n\ti/Yw57+nkss1oIh9rOYgOmQNdIBBBcvBbYedw/z/ZkM8pNKB4RTwtt65pJe5PqR9l4VI\n\tegRmG3kCH5QEGxig0+ZLJOhbRFjUvNdPbRavwK5SJd10g/vpxZXrAJ/k+4MX19Do9M6V\n\tB93ebqL5bIMzYbNTt4j823q7DGUhFZtKv+2S4VLhL5xghMCaRqJmtRYyXb2kUO/KYyvY\n\tM7zIkojMDQtAsf0c0jerk0a+8ChN+83ayrSlL7Hc2eQHZET+JE3iO3c2vSuSiY6QPRXC\n\tMuRg==","X-Gm-Message-State":"AMCzsaXs7d9G9qKOc+Khx2hUSIiizL/l5VU4qfcOQQKC82qqBU8WLqNt\n\tVrQk6qBpXb/zj6kxgzO1lM8svw==","X-Google-Smtp-Source":"AOwi7QAm6t3LXF7dsTQvUSEhQePSIAm2DRrMohdyhfSmxIHEDxwE7QMT97RhEEJruN+d5tQwOh4VEg==","X-Received":"by 10.28.193.207 with SMTP id r198mr6222231wmf.60.1506947549885; \n\tMon, 02 Oct 2017 05:32:29 -0700 (PDT)","From":"Paolo Bonzini ","To":"qemu-devel@nongnu.org","References":"<20170912144459.11359-1-pbonzini@redhat.com>","Message-ID":"<8315593b-4ae8-b034-01d6-d6f66925b6bb@redhat.com>","Date":"Mon, 2 Oct 2017 14:32:26 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170912144459.11359-1-pbonzini@redhat.com>","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-US","Content-Transfer-Encoding":"7bit","X-detected-operating-system":"by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic]\n\t[fuzzy]","X-Received-From":"74.125.82.54","Subject":"Re: [Qemu-devel] [Qemu-block] [PATCH v2 00/10] cleanup qemu-iotests","X-BeenThere":"qemu-devel@gnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"kwolf@redhat.com, qemu-block@nongnu.org","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@gnu.org","Sender":"\"Qemu-devel\"\n\t"}},{"id":1779976,"web_url":"http://patchwork.ozlabs.org/comment/1779976/","msgid":"<20171004162620.GD9801@localhost.localdomain>","list_archive_url":null,"date":"2017-10-04T16:26:20","subject":"Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests","submitter":{"id":2714,"url":"http://patchwork.ozlabs.org/api/people/2714/","name":"Kevin Wolf","email":"kwolf@redhat.com"},"content":"Am 12.09.2017 um 16:44 hat Paolo Bonzini geschrieben:\n> The purpose of this series is to separate the \"check\" sources from\n> the tests. After these patches, common.config is reduced to simple\n> shell initialization, and common.rc is only included by the tests.\n> \n> Along the way, a lot of dead code is removed too.\n> \n> In v2, the following patches:\n> \n> qemu-iotests: do not do useless search for QEMU_*_PROG\n> qemu-iotests: do not search for binaries in the current directory\n> qemu-iotests: include common.env and common.config early\n> \n> have been replaced by \"qemu-iotests: cleanup and fix search for programs\",\n> which also preserves the behavior of searching for programs as symlinks\n> in the current directory.\n\nThanks, applied to the block branch.\n\nKevin","headers":{"Return-Path":"","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=)","ext-mx10.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com","ext-mx10.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=kwolf@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 3y6hCq66Y7z9sxR\n\tfor ;\n\tThu, 5 Oct 2017 03:28:23 +1100 (AEDT)","from localhost ([::1]:36064 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t)\n\tid 1dzmWz-0003HW-Vw\n\tfor incoming@patchwork.ozlabs.org; Wed, 04 Oct 2017 12:28:22 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:56854)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from ) id 1dzmVB-000247-9N\n\tfor qemu-devel@nongnu.org; Wed, 04 Oct 2017 12:26:30 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from ) id 1dzmVA-0004DQ-HW\n\tfor qemu-devel@nongnu.org; Wed, 04 Oct 2017 12:26:29 -0400","from mx1.redhat.com ([209.132.183.28]:58520)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from )\n\tid 1dzmV6-00044Y-3t; Wed, 04 Oct 2017 12:26:24 -0400","from smtp.corp.redhat.com\n\t(int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16])\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 358635D5EB;\n\tWed, 4 Oct 2017 16:26:23 +0000 (UTC)","from localhost.localdomain (ovpn-117-103.ams2.redhat.com\n\t[10.36.117.103])\n\tby smtp.corp.redhat.com (Postfix) with ESMTPS id 42F195C1A1;\n\tWed, 4 Oct 2017 16:26:22 +0000 (UTC)"],"DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com 358635D5EB","Date":"Wed, 4 Oct 2017 18:26:20 +0200","From":"Kevin Wolf ","To":"Paolo Bonzini ","Message-ID":"<20171004162620.GD9801@localhost.localdomain>","References":"<20170912144459.11359-1-pbonzini@redhat.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<20170912144459.11359-1-pbonzini@redhat.com>","User-Agent":"Mutt/1.9.0 (2017-09-02)","X-Scanned-By":"MIMEDefang 2.79 on 10.5.11.16","X-Greylist":"Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.39]);\n\tWed, 04 Oct 2017 16:26:23 +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":"Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"qemu-devel@nongnu.org, qemu-block@nongnu.org","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t"}}]