From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olaf Hering Subject: Re: [PATCH v6] libxl: add option for discard support to xl disk configuration Date: Mon, 19 May 2014 15:01:48 +0200 Message-ID: <20140519130148.GA11296@aepfle.de> References: <1400493019-18114-1-git-send-email-olaf@aepfle.de> <21369.61499.949523.549356@mariner.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <21369.61499.949523.549356@mariner.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Jackson Cc: stefano.stabellini@eu.citrix.com, ian.campbell@citrix.com, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Mon, May 19, Ian Jackson wrote: > Olaf Hering writes ("[PATCH v6] libxl: add option for discard support to xl disk configuration"): > > Handle new boolean option discard/no-discard for disk configuration. It > > is supposed to disable discard support if file based backing storage was > > intentionally created non-sparse to avoid fragmentation of the file. > > > > The option intended for the backend driver. A new boolean property > > "discard-enable" is written to the backend node. An upcoming patch for > > qemu will make use of this property. The kernel blkback driver may be > > updated as well to disable discard for phy based backing storage. > > Acked-by: Ian Jackson > Committed-by: Ian Jackson > > (I made a tiny edit to the doc file as I committed, removing a > spurious comma.) The patch changes .l and .idl files. Did I place the comment incorrectly, did you overlook the comment or should my patch contain also the updates to generated files? Perhaps the required steps for the commiter should be written down somewhere, so that patch submissions for templates fits in your workflow. Olaf