Re: [Xen-devel] [PATCH 5/6] X86: MPX IA32_BNDCFGS msr dump

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 28.11.13 at 07:37, "Liu, Jinsong" <jinsong.liu@xxxxxxxxx> wrote:
> From 00d1af4ee854b033a3cffa5abd1031a3c2da3b03 Mon Sep 17 00:00:00 2001
> From: Liu Jinsong <jinsong.liu@xxxxxxxxx>
> Date: Thu, 28 Nov 2013 10:03:24 +0800
> Subject: [PATCH 5/6] X86: MPX IA32_BNDCFGS msr dump
> 
> Signed-off-by: Liu Jinsong <jinsong.liu@xxxxxxxxx>
> ---
>  tools/misc/xen-hvmctx.c |    8 ++++++++
>  1 files changed, 8 insertions(+), 0 deletions(-)
> 
> diff --git a/tools/misc/xen-hvmctx.c b/tools/misc/xen-hvmctx.c
> index 5a69245..b3e4b2b 100644
> --- a/tools/misc/xen-hvmctx.c
> +++ b/tools/misc/xen-hvmctx.c
> @@ -399,6 +399,13 @@ static void dump_tsc_adjust(void)
>      printf("    TSC_ADJUST: tsc_adjust %" PRIx64 "\n", p.tsc_adjust);
>  }
>  
> +static void dump_msr_bndcfgs(void)
> +{
> +    HVM_SAVE_TYPE(MSR_BNDCFGS) p;
> +    READ(p);
> +    printf("    MSR_BNDCFGS: %" PRIx64 "\n", p.msr_bndcfgs);
> +}
> +
>  int main(int argc, char **argv)
>  {
>      int entry, domid;
> @@ -467,6 +474,7 @@ int main(int argc, char **argv)
>          case HVM_SAVE_CODE(VIRIDIAN_VCPU): dump_viridian_vcpu(); break;
>          case HVM_SAVE_CODE(VMCE_VCPU): dump_vmce_vcpu(); break;
>          case HVM_SAVE_CODE(TSC_ADJUST): dump_tsc_adjust(); break;
> +        case HVM_SAVE_CODE(MSR_BNDCFGS): dump_msr_bndcfgs(); break;
>          case HVM_SAVE_CODE(END): break;
>          default:
>              printf(" ** Don't understand type %u: skipping\n",

Any reason why this can't (and really shouldn't) be part of the
previous patch?

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel