[PATCH openEuler-1.0-LTS] cpuidle: fix a build error when compiling haltpoll into module
hulk inclusion category: bugfix bugzilla: https://gitee.com/openeuler/kernel/issues/I3ZURN CVE: NA
--------
Kernel build would fail in case of CONFIG_HALTPOLL_CPUIDLE=m, caused by haltpoll_switch_governor() not marked as an exported symbol. Fix this by complementing the EXPORT_SYMBOL statement.
Fixes: 97c227885e0a ("cpuidle: fix container_of err in cpuidle_device and cpuidle_driver") Signed-off-by: GONG, Ruiqi gongruiqi1@huawei.com Cc: Jiajun Chen chenjiajun8@huawei.com --- drivers/cpuidle/driver.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/drivers/cpuidle/driver.c b/drivers/cpuidle/driver.c index 484d0e9655fc..c97d08c90655 100644 --- a/drivers/cpuidle/driver.c +++ b/drivers/cpuidle/driver.c @@ -260,6 +260,8 @@ void haltpoll_switch_governor(struct cpuidle_driver *drv) } }
+EXPORT_SYMBOL_GPL(haltpoll_switch_governor); + /** * cpuidle_register_driver - registers a driver * @drv: a pointer to a valid struct cpuidle_driver
Looks good to me, Reviewed-by: Xie XiuQi xiexiuqi@huawei.com
On 2021/7/9 10:22, GONG, Ruiqi wrote:
hulk inclusion category: bugfix bugzilla: https://gitee.com/openeuler/kernel/issues/I3ZURN CVE: NA
Kernel build would fail in case of CONFIG_HALTPOLL_CPUIDLE=m, caused by haltpoll_switch_governor() not marked as an exported symbol. Fix this by complementing the EXPORT_SYMBOL statement.
Fixes: 97c227885e0a ("cpuidle: fix container_of err in cpuidle_device and cpuidle_driver") Signed-off-by: GONG, Ruiqi gongruiqi1@huawei.com Cc: Jiajun Chen chenjiajun8@huawei.com
drivers/cpuidle/driver.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/drivers/cpuidle/driver.c b/drivers/cpuidle/driver.c index 484d0e9655fc..c97d08c90655 100644 --- a/drivers/cpuidle/driver.c +++ b/drivers/cpuidle/driver.c @@ -260,6 +260,8 @@ void haltpoll_switch_governor(struct cpuidle_driver *drv) } }
+EXPORT_SYMBOL_GPL(haltpoll_switch_governor);
/**
- cpuidle_register_driver - registers a driver
- @drv: a pointer to a valid struct cpuidle_driver
Hi Ruiqi,
On 2021/7/9 10:22, GONG, Ruiqi wrote:
hulk inclusion category: bugfix bugzilla: https://gitee.com/openeuler/kernel/issues/I3ZURN CVE: NA
Kernel build would fail in case of CONFIG_HALTPOLL_CPUIDLE=m, caused by haltpoll_switch_governor() not marked as an exported symbol. Fix this by complementing the EXPORT_SYMBOL statement.
Fixes: 97c227885e0a ("cpuidle: fix container_of err in cpuidle_device and cpuidle_driver") Signed-off-by: GONG, Ruiqi gongruiqi1@huawei.com Cc: Jiajun Chen chenjiajun8@huawei.com
drivers/cpuidle/driver.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/drivers/cpuidle/driver.c b/drivers/cpuidle/driver.c index 484d0e9655fc..c97d08c90655 100644 --- a/drivers/cpuidle/driver.c +++ b/drivers/cpuidle/driver.c @@ -260,6 +260,8 @@ void haltpoll_switch_governor(struct cpuidle_driver *drv) } }
nit: extra blank line.
+EXPORT_SYMBOL_GPL(haltpoll_switch_governor);
/**
- cpuidle_register_driver - registers a driver
- @drv: a pointer to a valid struct cpuidle_driver
With that fixed: Reviewed-by: Keqian Zhu zhukeqian1@huawei.com
-
GONG, Ruiqi -
Keqian Zhu -
Xie XiuQi