>>> On 19.10.18 at 17:20, <roger....@citrix.com> wrote: > No functional change intended. > > Signed-off-by: Roger Pau Monné <roger....@citrix.com>
Reviewed-by: Jan Beulich <jbeul...@suse.com> with one remark: > --- a/xen/include/public/arch-x86/xen.h > +++ b/xen/include/public/arch-x86/xen.h > @@ -346,6 +346,12 @@ struct xen_arch_domainconfig { > #define XEN_CPUID XEN_EMULATE_PREFIX "cpuid" > #endif > > +/* > + * Debug console IO port, also called "port E9 hack". Each character written > + * to this IO port will be printed on the hypervisor console. > + */ > +#define XEN_HVM_DEBUGCONS_IOPORT 0xe9 Perhaps amend the comment with "..., subject to log level restrictions"? Easily done while committing. FTR I'm a little uneasy about adding this to the public interface, but otoh external components already use it as if it was part of it. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel