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