On 02/11/18 15:55, Wei Liu wrote:
> @@ -2078,6 +2092,7 @@ void activate_debugregs(const struct vcpu *curr)
>      }
>  }
>  
> +#ifdef CONFIG_PV
>  /*
>   * Used by hypercalls and the emulator.
>   *  -ENODEV => #UD
> @@ -2193,6 +2208,7 @@ long set_debugreg(struct vcpu *v, unsigned int reg, 
> unsigned long value)
>  
>      return 0;
>  }
> +#endif  /* CONFIG_PV */

set_debugreg() doesn't really belong here.  How about moving it to
pv/misc-hypercalls.c ?

Either way, Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to