s390/vtimer: rework virtual timer interface
[sfrench/cifs-2.6.git] / arch / s390 / appldata / appldata_base.c
index fadefce099624a4fd48876243b9ce9e01082118e..bae0f402bf2ab7eb3c8c501c23bbd7e8f81e364a 100644 (file)
@@ -27,7 +27,7 @@
 #include <linux/suspend.h>
 #include <linux/platform_device.h>
 #include <asm/appldata.h>
-#include <asm/timer.h>
+#include <asm/vtimer.h>
 #include <asm/uaccess.h>
 #include <asm/io.h>
 #include <asm/smp.h>
@@ -82,8 +82,7 @@ static struct ctl_table appldata_dir_table[] = {
 /*
  * Timer
  */
-static DEFINE_PER_CPU(struct vtimer_list, appldata_timer);
-static atomic_t appldata_expire_count = ATOMIC_INIT(0);
+static struct vtimer_list appldata_timer;
 
 static DEFINE_SPINLOCK(appldata_timer_lock);
 static int appldata_interval = APPLDATA_CPU_INTERVAL;
@@ -113,10 +112,7 @@ static LIST_HEAD(appldata_ops_list);
  */
 static void appldata_timer_function(unsigned long data)
 {
-       if (atomic_dec_and_test(&appldata_expire_count)) {
-               atomic_set(&appldata_expire_count, num_online_cpus());
-               queue_work(appldata_wq, (struct work_struct *) data);
-       }
+       queue_work(appldata_wq, (struct work_struct *) data);
 }
 
 /*
@@ -129,7 +125,6 @@ static void appldata_work_fn(struct work_struct *work)
        struct list_head *lh;
        struct appldata_ops *ops;
 
-       get_online_cpus();
        mutex_lock(&appldata_ops_mutex);
        list_for_each(lh, &appldata_ops_list) {
                ops = list_entry(lh, struct appldata_ops, list);
@@ -138,7 +133,6 @@ static void appldata_work_fn(struct work_struct *work)
                }
        }
        mutex_unlock(&appldata_ops_mutex);
-       put_online_cpus();
 }
 
 /*
@@ -166,20 +160,6 @@ int appldata_diag(char record_nr, u16 function, unsigned long buffer,
 
 /****************************** /proc stuff **********************************/
 
-/*
- * appldata_mod_vtimer_wrap()
- *
- * wrapper function for mod_virt_timer(), because smp_call_function_single()
- * accepts only one parameter.
- */
-static void __appldata_mod_vtimer_wrap(void *p) {
-       struct {
-               struct vtimer_list *timer;
-               u64    expires;
-       } *args = p;
-       mod_virt_timer_periodic(args->timer, args->expires);
-}
-
 #define APPLDATA_ADD_TIMER     0
 #define APPLDATA_DEL_TIMER     1
 #define APPLDATA_MOD_TIMER     2
@@ -190,49 +170,28 @@ static void __appldata_mod_vtimer_wrap(void *p) {
  * Add, delete or modify virtual timers on all online cpus.
  * The caller needs to get the appldata_timer_lock spinlock.
  */
-static void
-__appldata_vtimer_setup(int cmd)
+static void __appldata_vtimer_setup(int cmd)
 {
-       u64 per_cpu_interval;
-       int i;
+       u64 timer_interval = (u64) appldata_interval * 1000 * TOD_MICRO;
 
        switch (cmd) {
        case APPLDATA_ADD_TIMER:
                if (appldata_timer_active)
                        break;
-               per_cpu_interval = (u64) (appldata_interval*1000 /
-                                         num_online_cpus()) * TOD_MICRO;
-               for_each_online_cpu(i) {
-                       per_cpu(appldata_timer, i).expires = per_cpu_interval;
-                       smp_call_function_single(i, add_virt_timer_periodic,
-                                                &per_cpu(appldata_timer, i),
-                                                1);
-               }
+               appldata_timer.expires = timer_interval;
+               add_virt_timer_periodic(&appldata_timer);
                appldata_timer_active = 1;
                break;
        case APPLDATA_DEL_TIMER:
-               for_each_online_cpu(i)
-                       del_virt_timer(&per_cpu(appldata_timer, i));
+               del_virt_timer(&appldata_timer);
                if (!appldata_timer_active)
                        break;
                appldata_timer_active = 0;
-               atomic_set(&appldata_expire_count, num_online_cpus());
                break;
        case APPLDATA_MOD_TIMER:
-               per_cpu_interval = (u64) (appldata_interval*1000 /
-                                         num_online_cpus()) * TOD_MICRO;
                if (!appldata_timer_active)
                        break;
-               for_each_online_cpu(i) {
-                       struct {
-                               struct vtimer_list *timer;
-                               u64    expires;
-                       } args;
-                       args.timer = &per_cpu(appldata_timer, i);
-                       args.expires = per_cpu_interval;
-                       smp_call_function_single(i, __appldata_mod_vtimer_wrap,
-                                                &args, 1);
-               }
+               mod_virt_timer_periodic(&appldata_timer, timer_interval);
        }
 }
 
@@ -263,14 +222,12 @@ appldata_timer_handler(ctl_table *ctl, int write,
        len = *lenp;
        if (copy_from_user(buf, buffer, len > sizeof(buf) ? sizeof(buf) : len))
                return -EFAULT;
-       get_online_cpus();
        spin_lock(&appldata_timer_lock);
        if (buf[0] == '1')
                __appldata_vtimer_setup(APPLDATA_ADD_TIMER);
        else if (buf[0] == '0')
                __appldata_vtimer_setup(APPLDATA_DEL_TIMER);
        spin_unlock(&appldata_timer_lock);
-       put_online_cpus();
 out:
        *lenp = len;
        *ppos += len;
@@ -303,20 +260,17 @@ appldata_interval_handler(ctl_table *ctl, int write,
                goto out;
        }
        len = *lenp;
-       if (copy_from_user(buf, buffer, len > sizeof(buf) ? sizeof(buf) : len)) {
+       if (copy_from_user(buf, buffer, len > sizeof(buf) ? sizeof(buf) : len))
                return -EFAULT;
-       }
        interval = 0;
        sscanf(buf, "%i", &interval);
        if (interval <= 0)
                return -EINVAL;
 
-       get_online_cpus();
        spin_lock(&appldata_timer_lock);
        appldata_interval = interval;
        __appldata_vtimer_setup(APPLDATA_MOD_TIMER);
        spin_unlock(&appldata_timer_lock);
-       put_online_cpus();
 out:
        *lenp = len;
        *ppos += len;
@@ -483,14 +437,12 @@ static int appldata_freeze(struct device *dev)
        int rc;
        struct list_head *lh;
 
-       get_online_cpus();
        spin_lock(&appldata_timer_lock);
        if (appldata_timer_active) {
                __appldata_vtimer_setup(APPLDATA_DEL_TIMER);
                appldata_timer_suspended = 1;
        }
        spin_unlock(&appldata_timer_lock);
-       put_online_cpus();
 
        mutex_lock(&appldata_ops_mutex);
        list_for_each(lh, &appldata_ops_list) {
@@ -514,14 +466,12 @@ static int appldata_restore(struct device *dev)
        int rc;
        struct list_head *lh;
 
-       get_online_cpus();
        spin_lock(&appldata_timer_lock);
        if (appldata_timer_suspended) {
                __appldata_vtimer_setup(APPLDATA_ADD_TIMER);
                appldata_timer_suspended = 0;
        }
        spin_unlock(&appldata_timer_lock);
-       put_online_cpus();
 
        mutex_lock(&appldata_ops_mutex);
        list_for_each(lh, &appldata_ops_list) {
@@ -565,53 +515,6 @@ static struct platform_driver appldata_pdrv = {
 
 /******************************* init / exit *********************************/
 
-static void __cpuinit appldata_online_cpu(int cpu)
-{
-       init_virt_timer(&per_cpu(appldata_timer, cpu));
-       per_cpu(appldata_timer, cpu).function = appldata_timer_function;
-       per_cpu(appldata_timer, cpu).data = (unsigned long)
-               &appldata_work;
-       atomic_inc(&appldata_expire_count);
-       spin_lock(&appldata_timer_lock);
-       __appldata_vtimer_setup(APPLDATA_MOD_TIMER);
-       spin_unlock(&appldata_timer_lock);
-}
-
-static void __cpuinit appldata_offline_cpu(int cpu)
-{
-       del_virt_timer(&per_cpu(appldata_timer, cpu));
-       if (atomic_dec_and_test(&appldata_expire_count)) {
-               atomic_set(&appldata_expire_count, num_online_cpus());
-               queue_work(appldata_wq, &appldata_work);
-       }
-       spin_lock(&appldata_timer_lock);
-       __appldata_vtimer_setup(APPLDATA_MOD_TIMER);
-       spin_unlock(&appldata_timer_lock);
-}
-
-static int __cpuinit appldata_cpu_notify(struct notifier_block *self,
-                                        unsigned long action,
-                                        void *hcpu)
-{
-       switch (action) {
-       case CPU_ONLINE:
-       case CPU_ONLINE_FROZEN:
-               appldata_online_cpu((long) hcpu);
-               break;
-       case CPU_DEAD:
-       case CPU_DEAD_FROZEN:
-               appldata_offline_cpu((long) hcpu);
-               break;
-       default:
-               break;
-       }
-       return NOTIFY_OK;
-}
-
-static struct notifier_block __cpuinitdata appldata_nb = {
-       .notifier_call = appldata_cpu_notify,
-};
-
 /*
  * appldata_init()
  *
@@ -619,7 +522,10 @@ static struct notifier_block __cpuinitdata appldata_nb = {
  */
 static int __init appldata_init(void)
 {
-       int i, rc;
+       int rc;
+
+       appldata_timer.function = appldata_timer_function;
+       appldata_timer.data = (unsigned long) &appldata_work;
 
        rc = platform_driver_register(&appldata_pdrv);
        if (rc)
@@ -637,14 +543,6 @@ static int __init appldata_init(void)
                goto out_device;
        }
 
-       get_online_cpus();
-       for_each_online_cpu(i)
-               appldata_online_cpu(i);
-       put_online_cpus();
-
-       /* Register cpu hotplug notifier */
-       register_hotcpu_notifier(&appldata_nb);
-
        appldata_sysctl_header = register_sysctl_table(appldata_dir_table);
        return 0;