From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MRGhn-0005TB-Hm for qemu-devel@nongnu.org; Wed, 15 Jul 2009 22:28:19 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MRGhi-0005Rs-Th for qemu-devel@nongnu.org; Wed, 15 Jul 2009 22:28:18 -0400 Received: from [199.232.76.173] (port=37226 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MRGhi-0005Rp-Ny for qemu-devel@nongnu.org; Wed, 15 Jul 2009 22:28:14 -0400 Received: from e39.co.us.ibm.com ([32.97.110.160]:59719) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1MRGhi-0004ZK-6g for qemu-devel@nongnu.org; Wed, 15 Jul 2009 22:28:14 -0400 Received: from d03relay04.boulder.ibm.com (d03relay04.boulder.ibm.com [9.17.195.106]) by e39.co.us.ibm.com (8.13.1/8.13.1) with ESMTP id n6G2NaB2023382 for ; Wed, 15 Jul 2009 20:23:36 -0600 Received: from d03av02.boulder.ibm.com (d03av02.boulder.ibm.com [9.17.195.168]) by d03relay04.boulder.ibm.com (8.13.8/8.13.8/NCO v9.2) with ESMTP id n6G2S7eN188814 for ; Wed, 15 Jul 2009 20:28:07 -0600 Received: from d03av02.boulder.ibm.com (loopback [127.0.0.1]) by d03av02.boulder.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id n6G2S78T002154 for ; Wed, 15 Jul 2009 20:28:07 -0600 From: Ram Pai In-Reply-To: <20090715210459.GJ3056@shareable.org> References: <1246511321.6429.31.camel@localhost> <4A4C754D.10109@redhat.com> <4A4CAD86.9020607@us.ibm.com> <4A4CB39F.5070506@redhat.com> <1247041831.6297.12.camel@localhost> <1247644283.14246.3.camel@localhost> <4A5DA1B7.5000204@siemens.com> <1247677395.14246.38.camel@localhost> <20090715182025.GC3056@shareable.org> <1247683475.14246.90.camel@localhost> <20090715210459.GJ3056@shareable.org> Content-Type: text/plain Date: Wed, 15 Jul 2009 19:28:05 -0700 Message-Id: <1247711285.14246.116.camel@localhost> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: qcow2 relative paths (was: [PATCH] rev5: support colon in filenames) Reply-To: linuxram@us.ibm.com List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jamie Lokier Cc: Kevin Wolf , Jan Kiszka , Anthony Liguori , qemu-devel@nongnu.org, kvm-devel On Wed, 2009-07-15 at 22:04 +0100, Jamie Lokier wrote: > Ram Pai wrote: > > I have successfully verified qcow2 files. But then I may not be trying > > out the exact thing that you are talking about. Can you give me a test > > case that I can verify. > > Commands tried with qemu-0.10.0-1ubuntu1: > > $ mkdir unlikely_subdir > $ cd unlikely_subdir > $ qemu-img create -f qcow2 backing.img 10 > Formatting 'backing.img', fmt=qcow2, size=10 kB > $ qemu-img create -f qcow2 -b ../unlikely_subdir/backing.img main.img 10 > Formatting 'main.img', fmt=qcow2, backing_file=../unlikely_subdir/backing.img, size=10 kB > $ cd .. > $ qemu-img info unlikely_subdir/main.img > image: unlikely_subdir/main.img > file format: qcow2 > virtual size: 10K (10240 bytes) > disk size: 16K > cluster_size: 4096 > highest_alloc: 16384 > backing file: ../unlikely_subdir/backing.img (actual path: unlikely_subdir/../unlikely_subdir/backing.img) > > See especially the "actual path" line. > > $ mv unlikely_subdir other_subdir > $ ls -l other_subdir > total 32 > -rw-r--r-- 1 jamie jamie 16384 2009-07-15 21:59 backing.img > -rw-r--r-- 1 jamie jamie 16384 2009-07-15 21:59 main.img > $ qemu-img info other_subdir/main.img > qemu-img: Could not open 'other_subdir/main.img' Turns out that I did introduce a bug by deleting the call to path_combine() before calling bdrv_open() on the backing filename. However the call to realpath() is still not needed. It feels like a kludge introduced to stop path_combine() from munging backing_filename. I will send out yet another revision with the fix. I just dont' know what else I will be breaking without having a regression test harness. :( > > What an unhelpful error message... There isn't even a way to find out > the backing file path which the tool is looking for. Ok. i have introduced a message towards the effect, in the next revision of the patch. Hope that will make things a little easier. I have to go through the all the other mails to understand what has been proposed, and what I need to incorporate. Looks like a tall order. For now i will send out revision 6 in the next few hours. RP > > > And one other thing. Let me know if there a test-suite that I can try > > for regressions. > > Sorry, I don't know anything about any QEMU test suites. > > -- Jamie RP