[lkml]   [2007]   [Oct]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[patch 17/34] appldata_base: Misc cpuinit annotations and bugfix
    From: Satyam Sharma <>

    appldata_offline_cpu() is only called from __cpuinit-marked hotplug
    notifier callback and from the __exit-marked module_exit function,
    therefore candidate for __cpuexit.

    BTW the __exit module_exit function appldata_exit() of this driver fails to
    unregister_hotcpu_notifier() the notifier_block that was registered by
    appldata_init() during module startup. This will lead to oops if hotplug
    notification comes after module has been unloaded. Let's fix this by
    unregistering the notifier appropriately (before appldata_offline_cpu()'ing
    the CPUs).

    Signed-off-by: Satyam Sharma <>
    Cc: Heiko Carstens <>
    Signed-off-by: Andrew Morton <>
    Signed-off-by: Martin Schwidefsky <>

    arch/s390/appldata/appldata_base.c | 11 ++++++-----
    1 file changed, 6 insertions(+), 5 deletions(-)

    Index: quilt-2.6/arch/s390/appldata/appldata_base.c
    --- quilt-2.6.orig/arch/s390/appldata/appldata_base.c
    +++ quilt-2.6/arch/s390/appldata/appldata_base.c
    @@ -547,8 +547,7 @@ static void __cpuinit appldata_online_cp

    -static void
    -appldata_offline_cpu(int cpu)
    +static void __cpuexit appldata_offline_cpu(int cpu)
    del_virt_timer(&per_cpu(appldata_timer, cpu));
    if (atomic_dec_and_test(&appldata_expire_count)) {
    @@ -560,9 +559,9 @@ appldata_offline_cpu(int cpu)

    -static int __cpuinit
    -appldata_cpu_notify(struct notifier_block *self,
    - unsigned long action, void *hcpu)
    +static int __cpuinit appldata_cpu_notify(struct notifier_block *self,
    + unsigned long action,
    + void *hcpu)
    switch (action) {
    case CPU_ONLINE:
    @@ -646,6 +645,8 @@ static void __exit appldata_exit(void)

    + unregister_hotcpu_notifier(&appldata_nb);

    blue skies,

    "Reality continues to ruin my life." - Calvin.

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2007-10-04 13:37    [W:0.046 / U:13.960 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site