On 26.03.2025 06:50, Penny Zheng wrote: > --- a/xen/common/sysctl.c > +++ b/xen/common/sysctl.c > @@ -58,6 +58,7 @@ long do_sysctl(XEN_GUEST_HANDLE_PARAM(xen_sysctl_t) > u_sysctl) > > switch ( op->cmd ) > { > +#ifdef CONFIG_SYSCTL > case XEN_SYSCTL_readconsole: > ret = xsm_readconsole(XSM_HOOK, op->u.readconsole.clear); > if ( ret ) > @@ -66,6 +67,7 @@ long do_sysctl(XEN_GUEST_HANDLE_PARAM(xen_sysctl_t) > u_sysctl) > ret = read_console_ring(&op->u.readconsole); > break; > > +#endif /* CONFIG_SYSCTL */ > case XEN_SYSCTL_tbuf_op:
Nit: Can the #endif please live ahead of the blank line? Jan