From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:59310) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RanXP-0005Qk-Tv for qemu-devel@nongnu.org; Wed, 14 Dec 2011 07:02:21 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RanXK-0001pm-Hv for qemu-devel@nongnu.org; Wed, 14 Dec 2011 07:02:19 -0500 Received: from mx1.redhat.com ([209.132.183.28]:60553) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RanXK-0001pV-9A for qemu-devel@nongnu.org; Wed, 14 Dec 2011 07:02:14 -0500 Message-ID: <4EE890FF.1050004@redhat.com> Date: Wed, 14 Dec 2011 13:05:19 +0100 From: Kevin Wolf MIME-Version: 1.0 References: <1323779840-4235-1-git-send-email-pbonzini@redhat.com> <4EE884BD.4090808@redhat.com> <4EE88CC3.7090903@redhat.com> In-Reply-To: <4EE88CC3.7090903@redhat.com> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 00/17] Support mismatched host and guest logical block sizes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: Christoph Hellwig , qemu-devel@nongnu.org, Stefan Hajnoczi Am 14.12.2011 12:47, schrieb Paolo Bonzini: > On 12/14/2011 12:13 PM, Kevin Wolf wrote: >> As we discussed before, the really interesting point here is defaults, >> and whatever you choose to do is wrong in some respect. >> >> So it looks like you chose to make the virtual device default to the >> host block size. > > ... wait wait, I default to 512. :) > > Here is the rationale. 512-over-4k may be slow, but is safe (but it is > not slow if you align partitions properly). 4k-over-512 is unsafe. So, > defaulting to 512 seemed the right thing after all. Which means bounce buffers by default on 4k hosts. Is this going to become our next cache=writethrough? At some point 4k disks will be in wide use, but we'll still be stuck with a slow default of 512. No matter what we decide here, I think it might really be a good idea to save the block size in the image and use that as the default if nothing else is specified on the command line. Kevin