On 1/21/21 9:56 AM, Wei Liu wrote:
> On Wed, Jan 20, 2021 at 05:49:09PM -0500, Boris Ostrovsky wrote:
>> This option allows guest administrator specify what should happen when
>> guest accesses an MSR which is not explicitly emulated by the hypervisor.
>>
>> Signed-off-by: Boris Ostrovsky <boris.ostrov...@oracle.com>
>> ---
>> docs/man/xl.cfg.5.pod.in | 20 +++++++++++++++++++-
>> tools/libs/light/libxl_types.idl | 7 +++++++
>> tools/xl/xl_parse.c | 7 +++++++
>> 3 files changed, 33 insertions(+), 1 deletion(-)
>
> It is mainly missing a #define LIBXL_HAVE_XXX in libxl.h.
>
> Other than that, this patch looks good to me. If you end up resending
> this series, please fix that issue.
>
> If other patches are all reviewed, you can provide some text to be
> merged into this patch.
>
> Wei.
>
Ah yes, I forgot about this.
diff --git a/tools/include/libxl.h b/tools/include/libxl.h
index 3433c950f9aa..f249740daf3f 100644
--- a/tools/include/libxl.h
+++ b/tools/include/libxl.h
@@ -1310,6 +1310,13 @@ void libxl_mac_copy(libxl_ctx *ctx, libxl_mac *dst,
const libxl_mac *src);
*/
#define LIBXL_HAVE_CREATEINFO_XEND_SUSPEND_EVTCHN_COMPAT
+/*
+ * LIBXL_HAVE_IGNORE_MSRS indicates that the libxl_ignore_msrs field is
+ * present in libxl_domain_build_info. This field describes hypervisor
+ * behavior on guest accesses to unimplemented MSRs.
+ */
+#define LIBXL_HAVE_IGNORE_MSRS 1
+
typedef char **libxl_string_list;
void libxl_string_list_dispose(libxl_string_list *sl);
int libxl_string_list_length(const libxl_string_list *sl);