From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <ian.campbell@citrix.com>
Subject: Re: [v2][PATCH 2/2] libxl: introduce gfx_passthru_kind
Date: Fri, 20 Mar 2015 10:11:45 +0000
Message-ID: <1426846305.21742.116.camel__33312.5590267175$1426846400$gmane$org@citrix.com>
References: <1425980538-5508-1-git-send-email-tiejun.chen@intel.com>
	<1425980538-5508-3-git-send-email-tiejun.chen@intel.com>
	<1426073641.21353.192.camel@citrix.com> <55010596.2030106@intel.com>
	<1426163186.21353.409.camel@citrix.com> <55023FDA.7030204@intel.com>
	<1426241479.32572.94.camel@citrix.com> <55062CD2.8050305@intel.com>
	<1426508400.18247.33.camel@citrix.com> <5507DBD6.8030401@intel.com>
	<1426584396.18247.136.camel@citrix.com> <55092A24.8010409@intel.com>
	<1426674310.18247.318.camel@citrix.com> <550A2F6A.4060508@intel.com>
	<1426761860.610.22.camel@citrix.com> <550B7207.6020503@intel.com>
	<1426844430.21742.101.camel@citrix.com> <550BF188.20401@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: <550BF188.20401@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: "Chen, Tiejun" <tiejun.chen@intel.com>
Cc: Ian.Jackson@eu.citrix.com, wei.liu2@citrix.com, qemu-devel@nongnu.org, stefano.stabellini@citrix.com, xen-devel@lists.xen.org
List-Id: xen-devel@lists.xenproject.org

On Fri, 2015-03-20 at 18:08 +0800, Chen, Tiejun wrote:
> +        if (!xlu_cfg_get_string(config, "gfx_passthru_kind", &buf, 0)) {
> +            if (libxl_gfx_passthru_kind_from_string(buf,
> + 
> &b_info->u.hvm.gfx_passthru_kind)) {
> +                fprintf(stderr,
> +                        "ERROR: invalid value \"%s\" for 
> \"gfx_passthru_kind\"\n",
> +                        buf);
> +                exit (1);
> +            }
> +        }

This unnecessary bit seems to have crept back in.

Don't forget to update the manpages if you haven't already.