From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Jackson <Ian.Jackson@eu.citrix.com>
Subject: Re: [v7][PATCH 10/16] tools: introduce some new
	parameters to set rdm policy
Date: Thu, 9 Jul 2015 19:02:49 +0100
Message-ID: <21918.46921.870193.155258@mariner.uk.xensource.com>
References: <1436420047-25356-1-git-send-email-tiejun.chen@intel.com>
	<1436420047-25356-11-git-send-email-tiejun.chen@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <1436420047-25356-11-git-send-email-tiejun.chen@intel.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Tiejun Chen <tiejun.chen@intel.com>
Cc: Stefano Stabellini <stefano.stabellini@eu.citrix.com>, Wei Liu <wei.liu2@citrix.com>, Ian Campbell <ian.campbell@citrix.com>, xen-devel@lists.xen.org
List-Id: xen-devel@lists.xenproject.org

Tiejun Chen writes ("[v7][PATCH 10/16] tools: introduce some new parameters to set rdm policy"):
> This patch introduces user configurable parameters to specify RDM
> resource and according policies,
...

>  int libxl__device_pci_setdefault(libxl__gc *gc, libxl_device_pci *pci)
>  {
> +    /* We'd like to force reserve rdm specific to a device by default.*/
> +    if ( pci->rdm_policy == LIBXL_RDM_RESERVE_POLICY_INVALID)
           ^

I have just spotted that spurious whitespace.  However I won't block
this for that.

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

(actually).

I would appreciate it if you could ensure that this is fixed in any
repost.  You may retain my ack if you do that.  Committers should feel
free to fix it on commit.

Thanks,
Ian.