From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:50698) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1REfnp-0006Vl-B5 for qemu-devel@nongnu.org; Fri, 14 Oct 2011 07:19:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1REfno-0001eO-Br for qemu-devel@nongnu.org; Fri, 14 Oct 2011 07:19:49 -0400 Received: from e23smtp02.au.ibm.com ([202.81.31.144]:57767) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1REfnn-0001eF-RD for qemu-devel@nongnu.org; Fri, 14 Oct 2011 07:19:48 -0400 Received: from d23relay05.au.ibm.com (d23relay05.au.ibm.com [202.81.31.247]) by e23smtp02.au.ibm.com (8.14.4/8.13.1) with ESMTP id p9EBCnvr006925 for ; Fri, 14 Oct 2011 22:12:49 +1100 Received: from d23av03.au.ibm.com (d23av03.au.ibm.com [9.190.234.97]) by d23relay05.au.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id p9EBH2OB1781800 for ; Fri, 14 Oct 2011 22:17:02 +1100 Received: from d23av03.au.ibm.com (loopback [127.0.0.1]) by d23av03.au.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id p9EBJSKh013486 for ; Fri, 14 Oct 2011 22:19:28 +1100 Message-ID: <4E981ABD.1070301@linux.vnet.ibm.com> Date: Fri, 14 Oct 2011 16:49:25 +0530 From: Supriya Kannery MIME-Version: 1.0 References: <20111011031046.9587.44474.sendpatchset@skannery.in.ibm.com> <20111011031145.9587.93507.sendpatchset@skannery.in.ibm.com> <4E95A495.3050403@redhat.com> In-Reply-To: <4E95A495.3050403@redhat.com> Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [v7 Patch 4/5]Qemu: Add commandline -drive option 'hostcache' Reply-To: supriyak@linux.vnet.ibm.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf Cc: Stefan Hajnoczi , qemu-devel@nongnu.org, Christoph Hellwig On 10/12/2011 08:00 PM, Kevin Wolf wrote: > Am 11.10.2011 05:11, schrieb Supriya Kannery: >> qemu command option 'hostcache' added to -drive for block devices. >> While starting a VM from qemu commandline, this option can be used >> for setting host cache usage for block data access. Simultaneous >> use of 'hostcache' and 'cache' options not allowed. >> >> Signed-off-by: Supriya Kannery > > I'm not sure if introducing this alone makes sense. I think I would only > do it when we introduce more options that allow replacing all cache=... > options by other parameters. > Can we do transition to alternatives for 'cache=' in a phased manner? Until all other params are ready, we can allow hostcache (as well as other params as and when they are ready) in cmdline with the condition that 'cache=x', if specified, overrides these params. Once we have all other params ready, 'cache=' can be replaced completely. thanks, Supriya