[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH for-4.1 0/2] Fix check for default backing files
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH for-4.1 0/2] Fix check for default backing files in bdrv_reopen_prepare() |
Date: |
Sat, 13 Apr 2019 02:56:57 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 |
On 11.04.19 16:23, Alberto Garcia wrote:
> Hi,
>
> Patch 1 is the fix for the bug reported by Max here:
>
> https://lists.gnu.org/archive/html/qemu-block/2019-04/msg00293.html
>
> Patch 2 fixes a different (but slightly related) issue that I found
> while preparing the first patch.
I think the real problem is that bs->backing_file is not a cache for
bs->backing->bs->filename.
In fact, every user of bs->backing_file expects it to be something
different. Some expect it to be a cache for bs->backing->bs->filename,
some expect it to be what the image header says (i.e., it if’s a
relative path, it’s relative to the overlay), some expect it to be what
the image header says, but relative paths to be translated so they are
relative to qemu’s CWD.
All of this should be cleaned up and this is what patch 7 in my "block:
Deal with filters" series does:
http://lists.nongnu.org/archive/html/qemu-block/2019-04/msg00308.html
Max
signature.asc
Description: OpenPGP digital signature