From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44238) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YSoI5-0007CO-1u for qemu-devel@nongnu.org; Tue, 03 Mar 2015 09:59:22 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YSoHz-0008Of-6V for qemu-devel@nongnu.org; Tue, 03 Mar 2015 09:59:21 -0500 Received: from mx1.redhat.com ([209.132.183.28]:60646) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YSoHy-0008Ob-Vy for qemu-devel@nongnu.org; Tue, 03 Mar 2015 09:59:15 -0500 Message-ID: <54F5CC39.7070603@redhat.com> Date: Tue, 03 Mar 2015 09:59:05 -0500 From: Max Reitz MIME-Version: 1.0 References: <1425379316-19639-1-git-send-email-pl@kamp.de> <1425379316-19639-3-git-send-email-pl@kamp.de> In-Reply-To: <1425379316-19639-3-git-send-email-pl@kamp.de> Content-Type: text/plain; charset=iso-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 2/5] vpc: Ignore geometry for large images List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Lieven , qemu-devel@nongnu.org Cc: kwolf@redhat.com, carnold@suse.com, jcody@redhat.com, famz@redhat.com, stefanha@redhat.com On 2015-03-03 at 05:41, Peter Lieven wrote: > From: Kevin Wolf > > The CHS calculation as done per the VHD spec imposes a maximum image > size of ~127 GB. Real VHD images exist that are larger than that. > > Apparently there are two separate non-standard ways to achieve this: > You could use more heads than the spec does - this is the option that > qemu-img create chooses. > > However, other images exist where the geometry is set to the maximum > (65535/16/255), but the actual image size is larger. Until now, such > images are truncated at 127 GB when opening them with qemu. > > This patch changes the vpc driver to ignore geometry in this case and > only trust the size field in the header. > > Signed-off-by: Kevin Wolf > [PL: Fixed maximum geometry in the commit msg] > Signed-off-by: Peter Lieven > --- > block/vpc.c | 10 ++++------ > 1 file changed, 4 insertions(+), 6 deletions(-) Reviewed-by: Max Reitz