From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53514) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aNu7v-0002f8-HA for qemu-devel@nongnu.org; Mon, 25 Jan 2016 22:17:08 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aNu7u-0005zP-Fr for qemu-devel@nongnu.org; Mon, 25 Jan 2016 22:17:07 -0500 Date: Tue, 26 Jan 2016 11:16:55 +0800 From: Fam Zheng Message-ID: <20160126031655.GA23224@ad.usersys.redhat.com> References: <1453688783-11825-1-git-send-email-famz@redhat.com> <20160125111601.GC5154@noname.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160125111601.GC5154@noname.redhat.com> Subject: Re: [Qemu-devel] [PATCH] vmdk: Fix converting to streamOptimized List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf Cc: qemu-devel@nongnu.org, qemu-block@nongnu.org On Mon, 01/25 12:16, Kevin Wolf wrote: > Am 25.01.2016 um 03:26 hat Fam Zheng geschrieben: > > Commit d62d9dc4b8 lifted streamOptimized images's version to 3, but we > > now refuse to open version 3 images read-write. We need to make > > streamOptimized an exception to allow converting to it. This fixes the > > accidentally broken iotests case 059 for the same reason. > > > > Signed-off-by: Fam Zheng > > How different are version 3 images for other subformats? Are we > arbitrarily restrictring their use or is it really that they don't work > with our driver? And if they don't work with our driver, are we sure > that streamOptimized images can't use the features we don't support? > > Or is the version defined per subformat and doesn't necessarily exist > for other types? Version 3 images are undocumented except in the VMware KB article mentioned in the comment around this line (http://kb.vmware.com/kb/2064959). A few years ago, when users complained that QEMU doesn't support version 3 images, we presumed from the article that reading is okay, as the new feature is "persistent changed block tracking" (although it didn't say it is the only feature enabled by version 3), and went ahead enabling it. This time, it seems newer VMware products only accept version 3 if the subformat is streamOptimized. Again, without any documentation/specification update. Then our users complains again, so we add another exception to mitigate. As this subformat doesn't allow overwrite, the only use case is qemu-img converting to it. So this is pretty safe - it's always operating a new image - and the approach is tested by multiple users (both upstream and downstream). Fam