Re: [Qemu-devel] block/vpc: Fix vhd extra sectors issue

[Jeff Cody]

On Mon, Nov 16, 2015 at 10:17:11AM +0100, Kevin Wolf wrote:
> Am 13.11.2015 um 16:32 hat Lucian Petrut geschrieben:
> > 
> > At the moment, qemu-img extends new image virtual sizes based
> > on the CHS algorithm provided by the VHD specs in order to
> > ensure that the disk geometry (and payload as seen by some
> > guests which use the CHS value) can fit in the requested disk.
> > 
> > This patch drops this behavior, as it breaks compatibility with
> > Azure, which requires the MB alignment to be preserved.
> > 
> > Signed-off-by: Lucian Petrut <address@hidden>
> > ---
> > Proposed fix for https://bugs.launchpad.net/qemu/+bug/1490611
> 
> This may fix one scenario, but it's sure to break others which are
> currently working. The problem has been discussed more than once and
> it's essentially a problem with MS using their own file format
> inconsistently.
> 
> I think we once came to the conclusion that looking at the creator
> string might be a working heuristics. Apparently this was never
> implemented - I don't remember whether that was because we noticed a
> problem with it, or just because noone got to it.
> 
> Jeff and Peter, I seem to remember that you were involved the last time
> we discussed this, so does one of you remember why we didn't implement
> this heuristics in the end?
> 

I believe we can look at the creator field.  The VHD files created by
Virtual PC and Hyper-V differed, but we could theoretically
differentiate between by the creator field.

I'm not sure if there was anything that actually blocked the
implementation.  It may have just been because we weren't sure if this
was the case across all versions of VPC and Hyper-V, although I
suspect it is (as an aside - is VirtualPC essentially dead, or is it
maintained / supported by MS?).

-Jeff