From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49882) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d2dEs-00028s-TE for qemu-devel@nongnu.org; Mon, 24 Apr 2017 08:37:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d2dEs-0005r4-52 for qemu-devel@nongnu.org; Mon, 24 Apr 2017 08:37:10 -0400 Date: Mon, 24 Apr 2017 20:36:59 +0800 From: Fam Zheng Message-ID: <20170424123659.GI14416@lemon.lan> References: <1492050868-16200-1-git-send-email-lidongchen@tencent.com> <20170414060015.GA12191@lemon> <20170417034925.GC6717@lemon.lan> <20170424120906.GG14416@lemon.lan> <20170424121933.GH14416@lemon.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH v6] migration/block: use blk_pwrite_zeroes for each zero cluster List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: 858585 jemmy Cc: qemu block , Juan Quintela , qemu-devel , Dave Gilbert , Stefan Hajnoczi , Lidong Chen On Mon, 04/24 20:26, 858585 jemmy wrote: > > 2) qcow2 with cluster_size = 512 is probably too uncommon to be optimized for. > if culster_size is very small, should disable metadata check default? > No, I don't think it's worth the inconsistent behavior. People who want performance shouldn't use 512 bytes anyway. Fam