It is currently Fri, 23 Oct 2020 18:32:30 GMT



 
Author Message
 Switch APIC to driver model
Hi!

This switches to driver model, making suspend-to-ram possible. Please
apply,
                                                                Pavel

--- clean/arch/i386/kernel/apic.c       2003-02-28 15:10:01.000000000 +0100
+++ linux/arch/i386/kernel/apic.c       2003-02-28 15:33:45.000000000 +0100
@@ -10,6 +10,8 @@
  *                                     for testing these extensively.
  *     Maciej W. Rozycki       :       Various updates and fixes.
  *     Mikael Pettersson       :       Power Management for UP-APIC.
+ *     Pavel Machek and
+ *     Mikael Pettersson       :       Converted to driver model.
  */

 #include <linux/config.h>
@@ -23,6 +25,10 @@
 #include <linux/interrupt.h>
 #include <linux/mc146818rtc.h>
 #include <linux/kernel_stat.h>
+#include <linux/delay.h>
+#include <linux/slab.h>
+#include <linux/device.h>
+#include <linux/pm.h>

 #include <asm/atomic.h>
 #include <asm/smp.h>
@@ -77,7 +83,7 @@
        return maxlvt;
 }

-void clear_local_APIC(void)
+void clear_lapic(void)
 {
        int maxlvt;
        unsigned long v;
@@ -143,7 +149,7 @@
                /*
                 * Do not trust the local APIC being empty at bootup.
                 */
-               clear_local_APIC();
+               clear_lapic();
                /*
                 * PIC mode, enable APIC mode in the IMCR, i.e.
                 * connect BSP's local APIC to INT and NMI lines.
@@ -169,11 +175,11 @@
        }
 }

-void disable_local_APIC(void)
+void disable_lapic(void)
 {
        unsigned long value;

-       clear_local_APIC();
+       clear_lapic();

        /*
         * Disable APIC (implies clearing of registers
@@ -189,7 +195,7 @@
  * Check these against your board if the CPUs aren't getting
  * started for no apparent reason.
  */
-int __init verify_local_APIC(void)
+int __init verify_lapic(void)
 {
        unsigned int reg0, reg1;

@@ -279,7 +285,7 @@
        /*
         * Do not trust the local APIC being empty at bootup.
         */
-       clear_local_APIC();
+       clear_lapic();

        /*
         * Enable APIC.
@@ -304,7 +310,28 @@
        apic_write_around(APIC_LVT1, value);
 }

-void __init setup_local_APIC (void)
+static struct {
+       /* 'active' is true if the local APIC was enabled by us and
+          not the BIOS; this signifies that we are also responsible
+          for disabling it before entering apm/acpi suspend */
+       int active;
+       /* r/w apic fields */
+       unsigned int apic_id;
+       unsigned int apic_taskpri;
+       unsigned int apic_ldr;
+       unsigned int apic_dfr;
+       unsigned int apic_spiv;
+       unsigned int apic_lvtt;
+       unsigned int apic_lvtpc;
+       unsigned int apic_lvt0;
+       unsigned int apic_lvt1;
+       unsigned int apic_lvterr;
+       unsigned int apic_tmict;
+       unsigned int apic_tdcr;
+       unsigned int apic_thmr;
+} apic_pm_state;
+
+void __init setup_lapic (void)
 {
        unsigned long value, ver, maxlvt;

@@ -445,46 +472,24 @@
                        printk("No ESR for 82489DX.\n");
        }

-       if (nmi_watchdog == NMI_LOCAL_APIC)
-               setup_apic_nmi_watchdog();
+       if (nmi_watchdog == NMI_LOCAL_APIC) {
+               enable_lapic_nmi_watchdog();
+       }
+
+       apic_pm_state.active = 1;
 }

 #ifdef CONFIG_PM
-
-#include <linux/slab.h>
-#include <linux/pm.h>
-
-static struct {
-       /* 'active' is true if the local APIC was enabled by us and
-          not the BIOS; this signifies that we are also responsible
-          for disabling it before entering apm/acpi suspend */
-       int active;
-       /* 'perfctr_pmdev' is here because the current (2.4.1) PM
-          callback system doesn't handle hierarchical dependencies */
-       struct pm_dev *perfctr_pmdev;
-       /* r/w apic fields */
-       unsigned int apic_id;
-       unsigned int apic_taskpri;
-       unsigned int apic_ldr;
-       unsigned int apic_dfr;
-       unsigned int apic_spiv;
-       unsigned int apic_lvtt;
-       unsigned int apic_lvtpc;
-       unsigned int apic_lvt0;
-       unsigned int apic_lvt1;
-       unsigned int apic_lvterr;
-       unsigned int apic_tmict;
-       unsigned int apic_tdcr;
-       unsigned int apic_thmr;
-} apic_pm_state;
-
-static void apic_pm_suspend(void *data)
+static int lapic_suspend(struct device *dev, u32 state, u32 level)
 {
        unsigned int l, h;
        unsigned long flags;

-       if (apic_pm_state.perfctr_pmdev)
-               pm_send(apic_pm_state.perfctr_pmdev, PM_SUSPEND, data);
+       if (level != SUSPEND_POWER_DOWN)
+               return 0;
+       if (!apic_pm_state.active)
+               return 0;
+
        apic_pm_state.apic_id = apic_read(APIC_ID);
        apic_pm_state.apic_taskpri = apic_read(APIC_TASKPRI);
        apic_pm_state.apic_ldr = apic_read(APIC_LDR);
@@ -500,18 +505,26 @@
        apic_pm_state.apic_thmr = apic_read(APIC_LVTTHMR);

        local_irq_save(flags);
-       disable_local_APIC();
+       disable_lapic();
        rdmsr(MSR_IA32_APICBASE, l, h);
        l &= ~MSR_IA32_APICBASE_ENABLE;
        wrmsr(MSR_IA32_APICBASE, l, h);
        local_irq_restore(flags);
+       return 0;
 }

-static void apic_pm_resume(void *data)
+static int lapic_resume(struct device *dev, u32 level)
 {
        unsigned int l, h;
        unsigned long flags;

+       if (level != RESUME_POWER_ON)
+               return 0;
+       if (!apic_pm_state.active)
+               return 0;
+
+       set_fixmap_nocache(FIX_APIC_BASE, APIC_DEFAULT_PHYS_BASE);      /* FIXME: this is needed for S3 resume, but why? */
+
        local_irq_save(flags);
        rdmsr(MSR_IA32_APICBASE, l, h);
        l &= ~MSR_IA32_APICBASE_BASE;
@@ -536,74 +549,35 @@
        apic_write(APIC_ESR, 0);
        apic_read(APIC_ESR);
        local_irq_restore(flags);
-       if (apic_pm_state.perfctr_pmdev)
-               pm_send(apic_pm_state.perfctr_pmdev, PM_RESUME, data);
-}
-
-static int apic_pm_callback(struct pm_dev *dev, pm_request_t rqst, void *data)
-{
-       switch (rqst) {
-       case PM_SUSPEND:
-               apic_pm_suspend(data);
-               break;
-       case PM_RESUME:
-               apic_pm_resume(data);
-               break;
-       }
        return 0;
 }

-/* perfctr driver should call this instead of pm_register() */
-struct pm_dev *apic_pm_register(pm_dev_t type,
-                               unsigned long id,
-                               pm_callback callback)
-{
-       struct pm_dev *dev;
-
-       if (!apic_pm_state.active)
-               return pm_register(type, id, callback);
-       if (apic_pm_state.perfctr_pmdev)
-               return NULL;    /* we're busy */
-       dev = kmalloc(sizeof(struct pm_dev), GFP_KERNEL);
-       if (dev) {
-               memset(dev, 0, sizeof(*dev));
-               dev->type = type;
-               dev->id = id;
-               dev->callback = callback;
-               apic_pm_state.perfctr_pmdev = dev;
-       }
-       return dev;
-}
+static struct device_driver lapic_driver = {
+       .name           = "apic",
+       .bus            = &system_bus_type,
+       .resume         = lapic_resume,
+       .suspend        = lapic_suspend,
+};
+
+/* not static, needed by child devices */
+struct sys_device device_lapic = {
+       .name           = "apic",
+       .id             = 0,
+       .dev            = {
+               .name   = "lapic",
+               .driver = &lapic_driver,
+       },
+};

-/* perfctr driver should call this instead of pm_unregister() */
-void apic_pm_unregister(struct pm_dev *dev)
-{
-       if (!apic_pm_state.active) {
-               pm_unregister(dev);
-       } else if (dev == apic_pm_state.perfctr_pmdev) {
-               apic_pm_state.perfctr_pmdev = NULL;
-               kfree(dev);
-       }
-}
-
-static void __init apic_pm_init1(void)
-{
-       /* can't pm_register() at this early stage in the boot process
-          (causes an immediate reboot), so just set the flag */
-       apic_pm_state.active = 1;
-}
-
-static void __init apic_pm_init2(void)
+static int __init init_apic_devicefs(void)
 {
+       driver_register(&lapic_driver);
        if (apic_pm_state.active)
-               pm_register(PM_SYS_DEV, 0, apic_pm_callback);
+               return sys_device_register(&device_lapic);
+       return 0;
 }

-#else  /* CONFIG_PM */
-
-static inline void apic_pm_init1(void) { }
-static inline void apic_pm_init2(void) { }
-
+device_initcall(init_apic_devicefs);
 #endif /* CONFIG_PM */

 /*
@@ -669,9 +643,6 @@
                nmi_watchdog = NMI_LOCAL_APIC;

        printk("Found and enabled local APIC!\n");
-
-       apic_pm_init1();
-
        return 0;

 no_apic:
@@ -682,7 +653,6 @@
 void __init init_apic_mappings(void)
 {
        unsigned long apic_phys;
-
        /*
         * If no local APIC can be found then set up a fake all
         * zeroes page to simulate the local APIC and another
@@ -1149,15 +1119,13 @@
                return -1;
        }

-       verify_local_APIC();
+       verify_lapic();

        connect_bsp_APIC();

        phys_cpu_present_map = 1 << boot_cpu_physical_apicid;

-       apic_pm_init2();
-
-       setup_local_APIC();
+       setup_lapic();

        if (nmi_watchdog == NMI_LOCAL_APIC)
                check_nmi_watchdog();
--- clean/arch/i386/kernel/apm.c        2003-02-28 15:10:01.000000000 +0100
+++ linux/arch/i386/kernel/apm.c        2003-02-28 15:33:45.000000000 +0100
@@ -1237,6 +1237,10 @@
                }
                printk(KERN_CRIT "apm: suspend was vetoed, but suspending anyway.\n");
        }
+
+      
+       device_suspend(3, SUSPEND_POWER_DOWN);
+
        /* serialize with the timer interrupt */
        write_seqlock_irq(&xtime_lock);

@@ -1257,6 +1261,7 @@
        if (err != APM_SUCCESS)
                apm_error("suspend", err);
        err = (err == APM_SUCCESS) ? 0 : -EIO;
+       device_resume(RESUME_POWER_ON);
        pm_send_all(PM_RESUME, (void *)0);
        queue_event(APM_NORMAL_RESUME, NULL);
  out:
@@ -1370,6 +1375,7 @@
                                write_seqlock_irq(&xtime_lock);
                                set_time();
                                write_sequnlock_irq(&xtime_lock);
+                               device_resume(RESUME_POWER_ON);
                                pm_send_all(PM_RESUME, (void *)0);
                                queue_event(event, NULL);
                        }
--- clean/arch/i386/kernel/nmi.c        2003-02-28 15:10:04.000000000 +0100
+++ linux/arch/i386/kernel/nmi.c        2003-02-28 15:33:45.000000000 +0100
@@ -9,6 +9,8 @@
  *  Mikael Pettersson  : AMD K7 support for local APIC NMI watchdog.
  *  Mikael Pettersson  : Power Management for local APIC NMI watchdog.
  *  Mikael Pettersson  : Pentium 4 support for local APIC NMI watchdog.
+ *  Pavel Machek and
+ *  Mikael Pettersson  : PM converted to driver model. disable/enable API.
  */

 #include <linux/config.h>
@@ -20,6 +22,8 @@
 #include <linux/interrupt.h>
 #include <linux/mc146818rtc.h>
 #include <linux/kernel_stat.h>
+#include <linux/device.h>
+#include <linux/module.h>

 #include <asm/smp.h>
 #include <asm/mtrr.h>
@@ -29,6 +33,7 @@
 static unsigned int nmi_hz = HZ;
 unsigned int nmi_perfctr_msr;  /* the MSR to reset in NMI handler */
 extern void show_registers(struct pt_regs *regs);
+static int nmi_active;

 #define K7_EVNTSEL_ENABLE      (1 << 22)
 #define K7_EVNTSEL_INT         (1 << 20)
@@ -117,7 +122,7 @@
         */
        if ((nmi == NMI_LOCAL_APIC) &&
                        (boot_cpu_data.x86_vendor == X86_VENDOR_INTEL) &&
-                       (boot_cpu_data.x86 == 6 || boot_cpu_data.x86 == 15))
+                       (boot_cpu_data.x86 == 6 || boot_cpu_data.x86 == 15))
                nmi_watchdog = nmi;
        if ((nmi == NMI_LOCAL_APIC) &&
                        (boot_cpu_data.x86_vendor == X86_VENDOR_AMD) &&
@@ -134,14 +139,11 @@

 __setup("nmi_watchdog=", setup_nmi_watchdog);

-#ifdef CONFIG_PM
-
-#include <linux/pm.h>
-
-struct pm_dev *nmi_pmdev;

-static void ...

read more »



 Sat, 20 Aug 2005 00:00:13 GMT   
 Switch APIC to driver model

This version works a lot better than the previous one(s). My P4,
which suspends/resumes via apm just fine with UP_APIC, survived
two suspend/resume cycles with this patch: one synchronous
(apm --suspend), and one asynchronous (short press on power button).
Not having IDE oops in a BUG_ON() is a definite improvement.

I had to add an #include <linux/device.h> to apm.c, a patch hunk
failed in oprofile, and there are some cosmetic things I don't like.
I'll merge this with my previous version tomorrow.

/Mikael
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



 Sat, 20 Aug 2005 02:40:09 GMT   
 
   [ 2 post ] 

Similar Threads

1. Switch APIC to driver model (and make S3 sleep with APIC on)

2. Switch APIC (+nmi, +oprofile) to driver model

3. Switch APIC (+nmi, +oprofile) to driver model

4. [2.5.74] x86_64 apic/nmi driver model conversion cleanups

5. Convert APIC to driver model: now it works with SMP

6. Convert APIC to driver model

7. Driver model for APIC

8. APIC Model in Linux

9. APIC Cluster Model


 
Powered by phpBB © 2000, 2002, 2005, 2007 phpBB Group.
Designed by ST Software