From: Len Bao Date: Sat, 23 May 2026 13:28:01 +0000 (+0000) Subject: xen/mcelog: mark g_physinfo, ncpus and xen_mce_chrdev_device as __ro_after_init X-Git-Url: http://git.ipfire.org/gitweb/?a=commitdiff_plain;h=19c5d6401d3d081f9194a7371ae3c3e5d7759efa;p=thirdparty%2Fkernel%2Flinux.git xen/mcelog: mark g_physinfo, ncpus and xen_mce_chrdev_device as __ro_after_init The 'g_physinfo' and 'ncpus' variables are initialized only during the init phase in the 'bind_virq_for_mce' function and never changed. So, mark them as __ro_after_init. The 'xen_mce_chrdev_device' variable is initialized only in the declaration and never changed. So, this variable could be 'const', but using the 'misc_register' and 'misc_deregister' functions discards the 'const' qualifier. Therefore, as an alternative, mark it as __ro_after_init. Signed-off-by: Len Bao Reviewed-by: Juergen Gross Signed-off-by: Juergen Gross Message-ID: <20260523132802.25391-1-len.bao@gmx.us> --- diff --git a/drivers/xen/mcelog.c b/drivers/xen/mcelog.c index 53a8720f5caeb..32ab419bb5034 100644 --- a/drivers/xen/mcelog.c +++ b/drivers/xen/mcelog.c @@ -54,8 +54,8 @@ #include static struct mc_info g_mi; -static struct mcinfo_logical_cpu *g_physinfo; -static uint32_t ncpus; +static struct mcinfo_logical_cpu *g_physinfo __ro_after_init; +static uint32_t ncpus __ro_after_init; static DEFINE_MUTEX(mcelog_lock); @@ -182,7 +182,7 @@ static const struct file_operations xen_mce_chrdev_ops = { .unlocked_ioctl = xen_mce_chrdev_ioctl, }; -static struct miscdevice xen_mce_chrdev_device = { +static struct miscdevice xen_mce_chrdev_device __ro_after_init = { MISC_MCELOG_MINOR, "mcelog", &xen_mce_chrdev_ops,