From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50978) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YI5hA-0000x5-2O for qemu-devel@nongnu.org; Sun, 01 Feb 2015 20:20:57 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YI5h6-0005Lq-O6 for qemu-devel@nongnu.org; Sun, 01 Feb 2015 20:20:56 -0500 Received: from mga14.intel.com ([192.55.52.115]:4485) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YI5h6-0005Le-Ib for qemu-devel@nongnu.org; Sun, 01 Feb 2015 20:20:52 -0500 From: Tiejun Chen Date: Mon, 2 Feb 2015 09:17:23 +0800 Message-Id: <1422839843-25622-1-git-send-email-tiejun.chen@intel.com> Subject: [Qemu-devel] [v2][PATCH] libxl: add one machine property to support IGD GFX passthrough List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: wei.liu2@citrix.com, Ian.Jackson@eu.citrix.com, ian.campbell@citrix.com, stefano.stabellini@citrix.com Cc: xen-devel@lists.xen.org, kraxel@redhat.com, qemu-devel@nongnu.org When we're working to support IGD GFX passthrough with qemu upstream, instead of "-gfx_passthru" we'd like to make that a machine option, "-machine xxx,-igd-passthru=on". This need to bring a change on tool side. Signed-off-by: Tiejun Chen --- v2: * Based on some discussions with Wei we'd like to keep both old option, -gfx_passthru, and new machine property option, "-machine xxx,-igd-passthru=on" at the same time but deprecate the old one. Then finally we remove the old one at that point that to give downstream (in this case, Xen) time to cope with the change. tools/libxl/libxl_dm.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/tools/libxl/libxl_dm.c b/tools/libxl/libxl_dm.c index c2b0487..8405f0b 100644 --- a/tools/libxl/libxl_dm.c +++ b/tools/libxl/libxl_dm.c @@ -701,6 +701,11 @@ static char ** libxl__build_device_model_args_new(libxl__gc *gc, flexarray_append(dm_args, "-net"); flexarray_append(dm_args, "none"); } + /* + * Although we already introduce 'igd-passthru', but we'd like + * to remove this until we give downstream time to cope with + * the change. + */ if (libxl_defbool_val(b_info->u.hvm.gfx_passthru)) { flexarray_append(dm_args, "-gfx_passthru"); } @@ -748,6 +753,11 @@ static char ** libxl__build_device_model_args_new(libxl__gc *gc, machinearg, max_ram_below_4g); } } + + if (libxl_defbool_val(b_info->u.hvm.gfx_passthru)) { + machinearg = libxl__sprintf(gc, "%s,igd-passthru=on", machinearg); + } + flexarray_append(dm_args, machinearg); for (i = 0; b_info->extra_hvm && b_info->extra_hvm[i] != NULL; i++) flexarray_append(dm_args, b_info->extra_hvm[i]); -- 1.9.1