From: "Chen, Tiejun" <tiejun.chen@intel.com>
To: Ian Campbell <ian.campbell@citrix.com>
Cc: Ian.Jackson@eu.citrix.com, wei.liu2@citrix.com,
qemu-devel@nongnu.org, stefano.stabellini@citrix.com,
xen-devel@lists.xen.org
Subject: Re: [v2][PATCH 2/2] libxl: introduce gfx_passthru_kind
Date: Fri, 13 Mar 2015 09:39:38 +0800 [thread overview]
Message-ID: <55023FDA.7030204__43207.3891781785$1426210909$gmane$org@intel.com> (raw)
In-Reply-To: <1426163186.21353.409.camel@citrix.com>
> I don't think you can abort here, since a user can set
> b_info->u.hvm.gfx_passthru_kind to default. You would need to return an
> error.
Then, looks I should do this,
LOG(ERROR, "No supported IGD to passthru,"
" or please force set gfx_passthru=\"igd\".\n");
return NULL;
>
>> @@ -720,6 +720,13 @@ void libxl_mac_copy(libxl_ctx *ctx, libxl_mac *dst,
>> libxl_mac *src);
>> #define LIBXL_HAVE_PSR_MBM 1
>> #endif
>>
>> +/*
>> + * LIBXL_HAVE_GFX_PASSTHRU_KIND
>> + *
>> + * If this is defined, the Graphic Device Passthrough Override is
>> supported.
>
> Almost, please also explicitly name the type field as other similar
> comments do for clarity.
Okay, maybe something is like this,
+/*
+ * LIBXL_HAVE_IGD_GFX_PASSTHRU
+ *
+ * If this is defined, the IGD Graphic Device Passthrough is supported.
+ *
+ * LIBXL_HAVE_IGD_GFX_PASSTHRU indicates that the
+ * libxl_device_pci field in the hvm is present in the pci_info structure
+ * fixup_ids[] which contains all supported IGD devices. So wwe use
+ * "igd-passthru=on" specify on the qemu command-line.
+ */
+#define LIBXL_HAVE_IGD_GFX_PASSTHRU 1
+
>> + */
[snip]
> and this should be in libxl_internal.h not here...
Okay.
I mistakenly understand we always have to expose this in libxl.h...
>
>> But looks libxl__gc{} is defined in the libxl_internal.h file... I guess
[snip]
>> + if (!xlu_cfg_get_long(config, "gfx_passthru", &l, 1)) {
>> + if (l) {
>> + libxl_defbool_set(&b_info->u.hvm.gfx_passthru, true);
>> + } else {
>> + libxl_defbool_set(&b_info->u.hvm.gfx_passthru, false);
>> + }
>
> This is exactly the same as:
> libxl_defbool_set(&b_info->u.hvm.gfx_passthru, l);
>
Sure.
Thanks
Tiejun
next prev parent reply other threads:[~2015-03-13 1:39 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-10 9:42 [Qemu-devel] [v2][PATCH 0/2] libxl: try to support IGD passthrough for qemu upstream Tiejun Chen
2015-03-10 9:42 ` [Qemu-devel] [v2][PATCH 1/2] libxl: introduce libxl__is_igd_vga_passthru Tiejun Chen
2015-03-11 11:26 ` Ian Campbell
2015-03-11 11:26 ` Ian Campbell
2015-03-10 9:42 ` Tiejun Chen
2015-03-10 9:42 ` [v2][PATCH 2/2] libxl: introduce gfx_passthru_kind Tiejun Chen
2015-03-10 9:42 ` [Qemu-devel] " Tiejun Chen
2015-03-11 11:34 ` Ian Campbell
2015-03-12 3:18 ` Chen, Tiejun
2015-03-12 12:26 ` Ian Campbell
2015-03-13 1:39 ` Chen, Tiejun
2015-03-13 10:11 ` Ian Campbell
2015-03-13 10:11 ` [Qemu-devel] " Ian Campbell
2015-03-16 1:07 ` Chen, Tiejun
2015-03-16 1:07 ` [Qemu-devel] " Chen, Tiejun
2015-03-16 12:20 ` Ian Campbell
2015-03-17 7:46 ` Chen, Tiejun
2015-03-17 7:46 ` [Qemu-devel] " Chen, Tiejun
2015-03-17 9:26 ` Ian Campbell
2015-03-18 7:32 ` Chen, Tiejun
2015-03-18 7:32 ` [Qemu-devel] " Chen, Tiejun
2015-03-18 10:25 ` Ian Campbell
2015-03-18 10:25 ` [Qemu-devel] " Ian Campbell
2015-03-19 2:07 ` Chen, Tiejun
2015-03-19 2:07 ` [Qemu-devel] " Chen, Tiejun
2015-03-19 10:44 ` Ian Campbell
2015-03-20 1:04 ` Chen, Tiejun
2015-03-20 1:04 ` [Qemu-devel] " Chen, Tiejun
2015-03-20 9:40 ` Ian Campbell
2015-03-20 9:40 ` [Qemu-devel] " Ian Campbell
2015-03-20 10:08 ` Chen, Tiejun
2015-03-20 10:11 ` Ian Campbell
2015-03-20 10:20 ` Chen, Tiejun
2015-03-20 10:20 ` Chen, Tiejun
2015-03-20 10:11 ` Ian Campbell
2015-03-20 10:08 ` Chen, Tiejun
2015-03-19 10:44 ` Ian Campbell
2015-03-17 9:26 ` Ian Campbell
2015-03-16 12:20 ` Ian Campbell
2015-03-13 1:39 ` Chen, Tiejun [this message]
2015-03-12 12:26 ` Ian Campbell
2015-03-12 3:18 ` Chen, Tiejun
2015-03-11 11:34 ` Ian Campbell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='55023FDA.7030204__43207.3891781785$1426210909$gmane$org@intel.com' \
--to=tiejun.chen@intel.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=ian.campbell@citrix.com \
--cc=qemu-devel@nongnu.org \
--cc=stefano.stabellini@citrix.com \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.