From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37567) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X0Eso-0001Li-Ew for qemu-devel@nongnu.org; Thu, 26 Jun 2014 14:59:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1X0Esi-0007rv-BC for qemu-devel@nongnu.org; Thu, 26 Jun 2014 14:58:54 -0400 Received: from mx1.redhat.com ([209.132.183.28]:37168) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X0Esi-0007rO-44 for qemu-devel@nongnu.org; Thu, 26 Jun 2014 14:58:48 -0400 Message-ID: <53AC6D60.7020306@redhat.com> Date: Thu, 26 Jun 2014 20:58:40 +0200 From: Max Reitz MIME-Version: 1.0 References: <1403043250-28472-1-git-send-email-mreitz@redhat.com> <20140623083544.GA10862@stefanha-thinkpad.redhat.com> In-Reply-To: <20140623083544.GA10862@stefanha-thinkpad.redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/2] block: Fix unset "filename" for certain drivers List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Kevin Wolf , qemu-devel@nongnu.org, Stefan Hajnoczi On 23.06.2014 10:35, Stefan Hajnoczi wrote: > On Wed, Jun 18, 2014 at 12:14:08AM +0200, Max Reitz wrote: >> For some protocol block drivers, the "filename" attribute in their BDSs >> is unset due to bdrv_file_open() removing it from the options QDict >> before bdrv_open_common() is able to copy it into the BDS. Fix this by >> not removing it until until bdrv_open_common() has indeed copied it. >> >> >> Max Reitz (2): >> block: Do not prematurely remove "filename" >> iotests: Add test for set "filename" for NBD >> >> block.c | 12 ++++++-- >> tests/qemu-iotests/097 | 72 ++++++++++++++++++++++++++++++++++++++++++++++ >> tests/qemu-iotests/097.out | 13 +++++++++ >> tests/qemu-iotests/group | 1 + >> 4 files changed, 95 insertions(+), 3 deletions(-) >> create mode 100755 tests/qemu-iotests/097 >> create mode 100644 tests/qemu-iotests/097.out > Please check qemu-iotests 051. Dropping from block queue for now. Thanks for catching this. The problem was me always removing the "filename" entry from the QDict if the driver could not handle it; however, to maintain the current behavior, it should only be removed if it has been parsed before. I'll send a v2. Max