25 files changed, 187 insertions, 207 deletions

diff --git a/drivers/char/agp/intel-agp.h b/drivers/char/agp/intel-agp.h
index 5da67f165afa..7ea18a5fe71c 100644
--- a/drivers/char/agp/intel-agp.h
+++ b/drivers/char/agp/intel-agp.h
@@ -234,6 +234,7 @@
 #define PCI_DEVICE_ID_INTEL_IVYBRIDGE_M_GT2_IG          0x0166
 #define PCI_DEVICE_ID_INTEL_IVYBRIDGE_S_HB              0x0158  /* Server */
 #define PCI_DEVICE_ID_INTEL_IVYBRIDGE_S_GT1_IG          0x015A
+#define PCI_DEVICE_ID_INTEL_IVYBRIDGE_S_GT2_IG          0x016A
 
 int intel_gmch_probe(struct pci_dev *pdev,
                                struct agp_bridge_data *bridge);
diff --git a/drivers/char/agp/intel-gtt.c b/drivers/char/agp/intel-gtt.c
index 5cf47ac2d401..7f025fb620de 100644
--- a/drivers/char/agp/intel-gtt.c
+++ b/drivers/char/agp/intel-gtt.c
@@ -1190,7 +1190,6 @@ static inline int needs_idle_maps(void)
 {
 #ifdef CONFIG_INTEL_IOMMU
         const unsigned short gpu_devid = intel_private.pcidev->device;
-        extern int intel_iommu_gfx_mapped;
 
         /* Query intel_iommu to see if we need the workaround. Presumably that
          * was loaded first.
@@ -1459,6 +1458,8 @@ static const struct intel_gtt_driver_description {
             "Ivybridge", &sandybridge_gtt_driver },
         { PCI_DEVICE_ID_INTEL_IVYBRIDGE_S_GT1_IG,
             "Ivybridge", &sandybridge_gtt_driver },
+        { PCI_DEVICE_ID_INTEL_IVYBRIDGE_S_GT2_IG,
+            "Ivybridge", &sandybridge_gtt_driver },
         { 0, NULL, NULL }
 };
 
diff --git a/drivers/char/apm-emulation.c b/drivers/char/apm-emulation.c
index f4837a893dfa..46118f845948 100644
--- a/drivers/char/apm-emulation.c
+++ b/drivers/char/apm-emulation.c
@@ -31,7 +31,6 @@
 #include <linux/kthread.h>
 #include <linux/delay.h>
 
-#include <asm/system.h>
 
 /*
  * The apm_bios device is one of the misc char devices.
@@ -302,7 +301,7 @@ apm_ioctl(struct file *filp, u_int cmd, u_long arg)
                          * anything critical, chill a bit on each iteration.
                          */
                         while (wait_event_freezable(apm_suspend_waitqueue,
-                                        as->suspend_state == SUSPEND_DONE))
+                                        as->suspend_state != SUSPEND_ACKED))
                                 msleep(10);
                         break;
                 case SUSPEND_ACKTO:
diff --git a/drivers/char/ds1302.c b/drivers/char/ds1302.c
index ed8303f9890c..7d34b203718a 100644
--- a/drivers/char/ds1302.c
+++ b/drivers/char/ds1302.c
@@ -24,7 +24,6 @@
 #include <linux/uaccess.h>
 #include <linux/io.h>
 
-#include <asm/system.h>
 #include <asm/rtc.h>
 #if defined(CONFIG_M32R)
 #include <asm/m32r.h>
diff --git a/drivers/char/efirtc.c b/drivers/char/efirtc.c
index 53c524e7b829..a082d00b0f11 100644
--- a/drivers/char/efirtc.c
+++ b/drivers/char/efirtc.c
@@ -37,7 +37,6 @@
 #include <linux/efi.h>
 #include <linux/uaccess.h>
 
-#include <asm/system.h>
 
 #define EFI_RTC_VERSION         "0.4"
 
diff --git a/drivers/char/genrtc.c b/drivers/char/genrtc.c
index f773a9dd14f3..21cb980f1157 100644
--- a/drivers/char/genrtc.c
+++ b/drivers/char/genrtc.c
@@ -56,7 +56,6 @@
 #include <linux/workqueue.h>
 
 #include <asm/uaccess.h>
-#include <asm/system.h>
 #include <asm/rtc.h>
 
 /*
diff --git a/drivers/char/hpet.c b/drivers/char/hpet.c
index 0833896cf6f2..3845ab44c330 100644
--- a/drivers/char/hpet.c
+++ b/drivers/char/hpet.c
@@ -36,7 +36,6 @@
 #include <linux/io.h>
 
 #include <asm/current.h>
-#include <asm/system.h>
 #include <asm/irq.h>
 #include <asm/div64.h>
 
diff --git a/drivers/char/hw_random/omap-rng.c b/drivers/char/hw_random/omap-rng.c
index b757fac3cd1f..a07a5caa599c 100644
--- a/drivers/char/hw_random/omap-rng.c
+++ b/drivers/char/hw_random/omap-rng.c
@@ -26,6 +26,8 @@
 
 #include <asm/io.h>
 
+#include <plat/cpu.h>
+
 #define RNG_OUT_REG             0x00            /* Output register */
 #define RNG_STAT_REG            0x04            /* Status register
                                                         [0] = STAT_BUSY */
diff --git a/drivers/char/ipmi/ipmi_devintf.c b/drivers/char/ipmi/ipmi_devintf.c
index 2aa3977aae5e..9eb360ff8cab 100644
--- a/drivers/char/ipmi/ipmi_devintf.c
+++ b/drivers/char/ipmi/ipmi_devintf.c
@@ -34,7 +34,6 @@
 #include <linux/module.h>
 #include <linux/moduleparam.h>
 #include <linux/errno.h>
-#include <asm/system.h>
 #include <linux/poll.h>
 #include <linux/sched.h>
 #include <linux/spinlock.h>
diff --git a/drivers/char/ipmi/ipmi_kcs_sm.c b/drivers/char/ipmi/ipmi_kcs_sm.c
index cf82fedae099..e53fc24c6af3 100644
--- a/drivers/char/ipmi/ipmi_kcs_sm.c
+++ b/drivers/char/ipmi/ipmi_kcs_sm.c
@@ -118,8 +118,8 @@ enum kcs_states {
 #define MAX_KCS_WRITE_SIZE IPMI_MAX_MSG_LENGTH
 
 /* Timeouts in microseconds. */
-#define IBF_RETRY_TIMEOUT 1000000
-#define OBF_RETRY_TIMEOUT 1000000
+#define IBF_RETRY_TIMEOUT 5000000
+#define OBF_RETRY_TIMEOUT 5000000
 #define MAX_ERROR_RETRIES 10
 #define ERROR0_OBF_WAIT_JIFFIES (2*HZ)
 
diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/char/ipmi/ipmi_msghandler.c
index 58c0e6387cf7..2c29942b1326 100644
--- a/drivers/char/ipmi/ipmi_msghandler.c
+++ b/drivers/char/ipmi/ipmi_msghandler.c
@@ -33,7 +33,6 @@
 
 #include <linux/module.h>
 #include <linux/errno.h>
-#include <asm/system.h>
 #include <linux/poll.h>
 #include <linux/sched.h>
 #include <linux/seq_file.h>
@@ -46,6 +45,7 @@
 #include <linux/init.h>
 #include <linux/proc_fs.h>
 #include <linux/rcupdate.h>
+#include <linux/interrupt.h>
 
 #define PFX "IPMI message handler: "
 
@@ -53,6 +53,8 @@
 
 static struct ipmi_recv_msg *ipmi_alloc_recv_msg(void);
 static int ipmi_init_msghandler(void);
+static void smi_recv_tasklet(unsigned long);
+static void handle_new_recv_msgs(ipmi_smi_t intf);
 
 static int initialized;
 
@@ -355,12 +357,15 @@ struct ipmi_smi {
         int curr_seq;
 
         /*
-         * Messages that were delayed for some reason (out of memory,
-         * for instance), will go in here to be processed later in a
-         * periodic timer interrupt.
+         * Messages queued for delivery.  If delivery fails (out of memory
+         * for instance), They will stay in here to be processed later in a
+         * periodic timer interrupt.  The tasklet is for handling received
+         * messages directly from the handler.
          */
         spinlock_t       waiting_msgs_lock;
         struct list_head waiting_msgs;
+        atomic_t         watchdog_pretimeouts_to_deliver;
+        struct tasklet_struct recv_tasklet;
 
         /*
          * The list of command receivers that are registered for commands
@@ -493,6 +498,8 @@ static void clean_up_interface_data(ipmi_smi_t intf)
         struct cmd_rcvr  *rcvr, *rcvr2;
         struct list_head list;
 
+        tasklet_kill(&intf->recv_tasklet);
+
         free_smi_msg_list(&intf->waiting_msgs);
         free_recv_msg_list(&intf->waiting_events);
 
@@ -2786,12 +2793,17 @@ channel_handler(ipmi_smi_t intf, struct ipmi_recv_msg *msg)
         return;
 }
 
-void ipmi_poll_interface(ipmi_user_t user)
+static void ipmi_poll(ipmi_smi_t intf)
 {
-        ipmi_smi_t intf = user->intf;
-
         if (intf->handlers->poll)
                 intf->handlers->poll(intf->send_info);
+        /* In case something came in */
+        handle_new_recv_msgs(intf);
+}
+
+void ipmi_poll_interface(ipmi_user_t user)
+{
+        ipmi_poll(user->intf);
 }
 EXPORT_SYMBOL(ipmi_poll_interface);
 
@@ -2860,6 +2872,10 @@ int ipmi_register_smi(struct ipmi_smi_handlers *handlers,
 #endif
         spin_lock_init(&intf->waiting_msgs_lock);
         INIT_LIST_HEAD(&intf->waiting_msgs);
+        tasklet_init(&intf->recv_tasklet,
+                     smi_recv_tasklet,
+                     (unsigned long) intf);
+        atomic_set(&intf->watchdog_pretimeouts_to_deliver, 0);
         spin_lock_init(&intf->events_lock);
         INIT_LIST_HEAD(&intf->waiting_events);
         intf->waiting_events_count = 0;
@@ -3622,11 +3638,11 @@ static int handle_bmc_rsp(ipmi_smi_t          intf,
 }
 
 /*
- * Handle a new message.  Return 1 if the message should be requeued,
+ * Handle a received message.  Return 1 if the message should be requeued,
  * 0 if the message should be freed, or -1 if the message should not
  * be freed or requeued.
  */
-static int handle_new_recv_msg(ipmi_smi_t          intf,
+static int handle_one_recv_msg(ipmi_smi_t          intf,
                                struct ipmi_smi_msg *msg)
 {
         int requeue;
@@ -3784,12 +3800,72 @@ static int handle_new_recv_msg(ipmi_smi_t          intf,
         return requeue;
 }
 
+/*
+ * If there are messages in the queue or pretimeouts, handle them.
+ */
+static void handle_new_recv_msgs(ipmi_smi_t intf)
+{
+        struct ipmi_smi_msg  *smi_msg;
+        unsigned long        flags = 0;
+        int                  rv;
+        int                  run_to_completion = intf->run_to_completion;
+
+        /* See if any waiting messages need to be processed. */
+        if (!run_to_completion)
+                spin_lock_irqsave(&intf->waiting_msgs_lock, flags);
+        while (!list_empty(&intf->waiting_msgs)) {
+                smi_msg = list_entry(intf->waiting_msgs.next,
+                                     struct ipmi_smi_msg, link);
+                list_del(&smi_msg->link);
+                if (!run_to_completion)
+                        spin_unlock_irqrestore(&intf->waiting_msgs_lock, flags);
+                rv = handle_one_recv_msg(intf, smi_msg);
+                if (!run_to_completion)
+                        spin_lock_irqsave(&intf->waiting_msgs_lock, flags);
+                if (rv == 0) {
+                        /* Message handled */
+                        ipmi_free_smi_msg(smi_msg);
+                } else if (rv < 0) {
+                        /* Fatal error on the message, del but don't free. */
+                } else {
+                        /*
+                         * To preserve message order, quit if we
+                         * can't handle a message.
+                         */
+                        list_add(&smi_msg->link, &intf->waiting_msgs);
+                        break;
+                }
+        }
+        if (!run_to_completion)
+                spin_unlock_irqrestore(&intf->waiting_msgs_lock, flags);
+
+        /*
+         * If the pretimout count is non-zero, decrement one from it and
+         * deliver pretimeouts to all the users.
+         */
+        if (atomic_add_unless(&intf->watchdog_pretimeouts_to_deliver, -1, 0)) {
+                ipmi_user_t user;
+
+                rcu_read_lock();
+                list_for_each_entry_rcu(user, &intf->users, link) {
+                        if (user->handler->ipmi_watchdog_pretimeout)
+                                user->handler->ipmi_watchdog_pretimeout(
+                                        user->handler_data);
+                }
+                rcu_read_unlock();
+        }
+}
+
+static void smi_recv_tasklet(unsigned long val)
+{
+        handle_new_recv_msgs((ipmi_smi_t) val);
+}
+
 /* Handle a new message from the lower layer. */
 void ipmi_smi_msg_received(ipmi_smi_t          intf,
                            struct ipmi_smi_msg *msg)
 {
         unsigned long flags = 0; /* keep us warning-free. */
-        int           rv;
         int           run_to_completion;
 
 
@@ -3843,31 +3919,11 @@ void ipmi_smi_msg_received(ipmi_smi_t          intf,
         run_to_completion = intf->run_to_completion;
         if (!run_to_completion)
                 spin_lock_irqsave(&intf->waiting_msgs_lock, flags);
-        if (!list_empty(&intf->waiting_msgs)) {
-                list_add_tail(&msg->link, &intf->waiting_msgs);
-                if (!run_to_completion)
-                        spin_unlock_irqrestore(&intf->waiting_msgs_lock, flags);
-                goto out;
-        }
+        list_add_tail(&msg->link, &intf->waiting_msgs);
         if (!run_to_completion)
                 spin_unlock_irqrestore(&intf->waiting_msgs_lock, flags);
 
-        rv = handle_new_recv_msg(intf, msg);
-        if (rv > 0) {
-                /*
-                 * Could not handle the message now, just add it to a
-                 * list to handle later.
-                 */
-                run_to_completion = intf->run_to_completion;
-                if (!run_to_completion)
-                        spin_lock_irqsave(&intf->waiting_msgs_lock, flags);
-                list_add_tail(&msg->link, &intf->waiting_msgs);
-                if (!run_to_completion)
-                        spin_unlock_irqrestore(&intf->waiting_msgs_lock, flags);
-        } else if (rv == 0) {
-                ipmi_free_smi_msg(msg);
-        }
-
+        tasklet_schedule(&intf->recv_tasklet);
  out:
         return;
 }
@@ -3875,16 +3931,8 @@ EXPORT_SYMBOL(ipmi_smi_msg_received);
 
 void ipmi_smi_watchdog_pretimeout(ipmi_smi_t intf)
 {
-        ipmi_user_t user;
-
-        rcu_read_lock();
-        list_for_each_entry_rcu(user, &intf->users, link) {
-                if (!user->handler->ipmi_watchdog_pretimeout)
-                        continue;
-
-                user->handler->ipmi_watchdog_pretimeout(user->handler_data);
-        }
-        rcu_read_unlock();
+        atomic_set(&intf->watchdog_pretimeouts_to_deliver, 1);
+        tasklet_schedule(&intf->recv_tasklet);
 }
 EXPORT_SYMBOL(ipmi_smi_watchdog_pretimeout);
 
@@ -3998,28 +4046,12 @@ static void ipmi_timeout_handler(long timeout_period)
         ipmi_smi_t           intf;
         struct list_head     timeouts;
         struct ipmi_recv_msg *msg, *msg2;
-        struct ipmi_smi_msg  *smi_msg, *smi_msg2;
         unsigned long        flags;
         int                  i;
 
         rcu_read_lock();
         list_for_each_entry_rcu(intf, &ipmi_interfaces, link) {
-                /* See if any waiting messages need to be processed. */
-                spin_lock_irqsave(&intf->waiting_msgs_lock, flags);
-                list_for_each_entry_safe(smi_msg, smi_msg2,
-                                         &intf->waiting_msgs, link) {
-                        if (!handle_new_recv_msg(intf, smi_msg)) {
-                                list_del(&smi_msg->link);
-                                ipmi_free_smi_msg(smi_msg);
-                        } else {
-                                /*
-                                 * To preserve message order, quit if we
-                                 * can't handle a message.
-                                 */
-                                break;
-                        }
-                }
-                spin_unlock_irqrestore(&intf->waiting_msgs_lock, flags);
+                tasklet_schedule(&intf->recv_tasklet);
 
                 /*
                  * Go through the seq table and find any messages that
@@ -4173,12 +4205,48 @@ EXPORT_SYMBOL(ipmi_free_recv_msg);
 
 #ifdef CONFIG_IPMI_PANIC_EVENT
 
+static atomic_t panic_done_count = ATOMIC_INIT(0);
+
 static void dummy_smi_done_handler(struct ipmi_smi_msg *msg)
 {
+        atomic_dec(&panic_done_count);
 }
 
 static void dummy_recv_done_handler(struct ipmi_recv_msg *msg)
 {
+        atomic_dec(&panic_done_count);
+}
+
+/*
+ * Inside a panic, send a message and wait for a response.
+ */
+static void ipmi_panic_request_and_wait(ipmi_smi_t           intf,
+                                        struct ipmi_addr     *addr,
+                                        struct kernel_ipmi_msg *msg)
+{
+        struct ipmi_smi_msg  smi_msg;
+        struct ipmi_recv_msg recv_msg;
+        int rv;
+
+        smi_msg.done = dummy_smi_done_handler;
+        recv_msg.done = dummy_recv_done_handler;
+        atomic_add(2, &panic_done_count);
+        rv = i_ipmi_request(NULL,
+                            intf,
+                            addr,
+                            0,
+                            msg,
+                            intf,
+                            &smi_msg,
+                            &recv_msg,
+                            0,
+                            intf->channels[0].address,
+                            intf->channels[0].lun,
+                            0, 1); /* Don't retry, and don't wait. */
+        if (rv)
+                atomic_sub(2, &panic_done_count);
+        while (atomic_read(&panic_done_count) != 0)
+                ipmi_poll(intf);
 }
 
 #ifdef CONFIG_IPMI_PANIC_STRING
@@ -4217,8 +4285,6 @@ static void send_panic_events(char *str)
         unsigned char                     data[16];
         struct ipmi_system_interface_addr *si;
         struct ipmi_addr                  addr;
-        struct ipmi_smi_msg               smi_msg;
-        struct ipmi_recv_msg              recv_msg;
 
         si = (struct ipmi_system_interface_addr *) &addr;
         si->addr_type = IPMI_SYSTEM_INTERFACE_ADDR_TYPE;
@@ -4246,9 +4312,6 @@ static void send_panic_events(char *str)
                 data[7] = str[2];
         }
 
-        smi_msg.done = dummy_smi_done_handler;
-        recv_msg.done = dummy_recv_done_handler;
-
         /* For every registered interface, send the event. */
         list_for_each_entry_rcu(intf, &ipmi_interfaces, link) {
                 if (!intf->handlers)
@@ -4258,18 +4321,7 @@ static void send_panic_events(char *str)
                 intf->run_to_completion = 1;
                 /* Send the event announcing the panic. */
                 intf->handlers->set_run_to_completion(intf->send_info, 1);
-                i_ipmi_request(NULL,
-                               intf,
-                               &addr,
-                               0,
-                               &msg,
-                               intf,
-                               &smi_msg,
-                               &recv_msg,
-                               0,
-                               intf->channels[0].address,
-                               intf->channels[0].lun,
-                               0, 1); /* Don't retry, and don't wait. */
+                ipmi_panic_request_and_wait(intf, &addr, &msg);
         }
 
 #ifdef CONFIG_IPMI_PANIC_STRING
@@ -4317,18 +4369,7 @@ static void send_panic_events(char *str)
                 msg.data = NULL;
                 msg.data_len = 0;
                 intf->null_user_handler = device_id_fetcher;
-                i_ipmi_request(NULL,
-                               intf,
-                               &addr,
-                               0,
-                               &msg,
-                               intf,
-                               &smi_msg,
-                               &recv_msg,
-                               0,
-                               intf->channels[0].address,
-                               intf->channels[0].lun,
-                               0, 1); /* Don't retry, and don't wait. */
+                ipmi_panic_request_and_wait(intf, &addr, &msg);
 
                 if (intf->local_event_generator) {
                         /* Request the event receiver from the local MC. */
@@ -4337,18 +4378,7 @@ static void send_panic_events(char *str)
                         msg.data = NULL;
                         msg.data_len = 0;
                         intf->null_user_handler = event_receiver_fetcher;
-                        i_ipmi_request(NULL,
-                                       intf,
-                                       &addr,
-                                       0,
-                                       &msg,
-                                       intf,
-                                       &smi_msg,
-                                       &recv_msg,
-                                       0,
-                                       intf->channels[0].address,
-                                       intf->channels[0].lun,
-                                       0, 1); /* no retry, and no wait. */
+                        ipmi_panic_request_and_wait(intf, &addr, &msg);
                 }
                 intf->null_user_handler = NULL;
 
@@ -4405,18 +4435,7 @@ static void send_panic_events(char *str)
                         strncpy(data+5, p, 11);
                         p += size;
 
-                        i_ipmi_request(NULL,
-                                       intf,
-                                       &addr,
-                                       0,
-                                       &msg,
-                                       intf,
-                                       &smi_msg,
-                                       &recv_msg,
-                                       0,
-                                       intf->channels[0].address,
-                                       intf->channels[0].lun,
-                                       0, 1); /* no retry, and no wait. */
+                        ipmi_panic_request_and_wait(intf, &addr, &msg);
                 }
         }
 #endif /* CONFIG_IPMI_PANIC_STRING */
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c
index 50fcf9c04569..1e638fff40ea 100644
--- a/drivers/char/ipmi/ipmi_si_intf.c
+++ b/drivers/char/ipmi/ipmi_si_intf.c
@@ -41,7 +41,6 @@
 
 #include <linux/module.h>
 #include <linux/moduleparam.h>
-#include <asm/system.h>
 #include <linux/sched.h>
 #include <linux/seq_file.h>
 #include <linux/timer.h>
@@ -171,7 +170,6 @@ struct smi_info {
         struct si_sm_handlers  *handlers;
         enum si_type           si_type;
         spinlock_t             si_lock;
-        spinlock_t             msg_lock;
         struct list_head       xmit_msgs;
         struct list_head       hp_xmit_msgs;
         struct ipmi_smi_msg    *curr_msg;
@@ -320,16 +318,8 @@ static int register_xaction_notifier(struct notifier_block *nb)
 static void deliver_recv_msg(struct smi_info *smi_info,
                              struct ipmi_smi_msg *msg)
 {
-        /* Deliver the message to the upper layer with the lock
-           released. */
-
-        if (smi_info->run_to_completion) {
-                ipmi_smi_msg_received(smi_info->intf, msg);
-        } else {
-                spin_unlock(&(smi_info->si_lock));
-                ipmi_smi_msg_received(smi_info->intf, msg);
-                spin_lock(&(smi_info->si_lock));
-        }
+        /* Deliver the message to the upper layer. */
+        ipmi_smi_msg_received(smi_info->intf, msg);
 }
 
 static void return_hosed_msg(struct smi_info *smi_info, int cCode)
@@ -358,13 +348,6 @@ static enum si_sm_result start_next_msg(struct smi_info *smi_info)
         struct timeval t;
 #endif
 
-        /*
-         * No need to save flags, we aleady have interrupts off and we
-         * already hold the SMI lock.
-         */
-        if (!smi_info->run_to_completion)
-                spin_lock(&(smi_info->msg_lock));
-
         /* Pick the high priority queue first. */
         if (!list_empty(&(smi_info->hp_xmit_msgs))) {
                 entry = smi_info->hp_xmit_msgs.next;
@@ -402,9 +385,6 @@ static enum si_sm_result start_next_msg(struct smi_info *smi_info)
                 rv = SI_SM_CALL_WITHOUT_DELAY;
         }
  out:
-        if (!smi_info->run_to_completion)
-                spin_unlock(&(smi_info->msg_lock));
-
         return rv;
 }
 
@@ -481,9 +461,7 @@ static void handle_flags(struct smi_info *smi_info)
 
                 start_clear_flags(smi_info);
                 smi_info->msg_flags &= ~WDT_PRE_TIMEOUT_INT;
-                spin_unlock(&(smi_info->si_lock));
                 ipmi_smi_watchdog_pretimeout(smi_info->intf);
-                spin_lock(&(smi_info->si_lock));
         } else if (smi_info->msg_flags & RECEIVE_MSG_AVAIL) {
                 /* Messages available. */
                 smi_info->curr_msg = ipmi_alloc_smi_msg();
@@ -889,19 +867,6 @@ static void sender(void                *send_info,
         printk("**Enqueue: %d.%9.9d\n", t.tv_sec, t.tv_usec);
 #endif
 
-        /*
-         * last_timeout_jiffies is updated here to avoid
-         * smi_timeout() handler passing very large time_diff
-         * value to smi_event_handler() that causes
-         * the send command to abort.
-         */
-        smi_info->last_timeout_jiffies = jiffies;
-
-        mod_timer(&smi_info->si_timer, jiffies + SI_TIMEOUT_JIFFIES);
-
-        if (smi_info->thread)
-                wake_up_process(smi_info->thread);
-
         if (smi_info->run_to_completion) {
                 /*
                  * If we are running to completion, then throw it in
@@ -924,16 +889,29 @@ static void sender(void                *send_info,
                 return;
         }
 
-        spin_lock_irqsave(&smi_info->msg_lock, flags);
+        spin_lock_irqsave(&smi_info->si_lock, flags);
         if (priority > 0)
                 list_add_tail(&msg->link, &smi_info->hp_xmit_msgs);
         else
                 list_add_tail(&msg->link, &smi_info->xmit_msgs);
-        spin_unlock_irqrestore(&smi_info->msg_lock, flags);
 
-        spin_lock_irqsave(&smi_info->si_lock, flags);
-        if (smi_info->si_state == SI_NORMAL && smi_info->curr_msg == NULL)
+        if (smi_info->si_state == SI_NORMAL && smi_info->curr_msg == NULL) {
+                /*
+                 * last_timeout_jiffies is updated here to avoid
+                 * smi_timeout() handler passing very large time_diff
+                 * value to smi_event_handler() that causes
+                 * the send command to abort.
+                 */
+                smi_info->last_timeout_jiffies = jiffies;
+
+                mod_timer(&smi_info->si_timer, jiffies + SI_TIMEOUT_JIFFIES);
+
+                if (smi_info->thread)
+                        wake_up_process(smi_info->thread);
+
                 start_next_msg(smi_info);
+                smi_event_handler(smi_info, 0);
+        }
         spin_unlock_irqrestore(&smi_info->si_lock, flags);
 }
 
@@ -1034,16 +1012,19 @@ static int ipmi_thread(void *data)
 static void poll(void *send_info)
 {
         struct smi_info *smi_info = send_info;
-        unsigned long flags;
+        unsigned long flags = 0;
+        int run_to_completion = smi_info->run_to_completion;
 
         /*
          * Make sure there is some delay in the poll loop so we can
          * drive time forward and timeout things.
          */
         udelay(10);
-        spin_lock_irqsave(&smi_info->si_lock, flags);
+        if (!run_to_completion)
+                spin_lock_irqsave(&smi_info->si_lock, flags);
         smi_event_handler(smi_info, 10);
-        spin_unlock_irqrestore(&smi_info->si_lock, flags);
+        if (!run_to_completion)
+                spin_unlock_irqrestore(&smi_info->si_lock, flags);
 }
 
 static void request_events(void *send_info)
@@ -1680,10 +1661,8 @@ static struct smi_info *smi_info_alloc(void)
 {
         struct smi_info *info = kzalloc(sizeof(*info), GFP_KERNEL);
 
-        if (info) {
+        if (info)
                 spin_lock_init(&info->si_lock);
-                spin_lock_init(&info->msg_lock);
-        }
         return info;
 }
 
diff --git a/drivers/char/ipmi/ipmi_watchdog.c b/drivers/char/ipmi/ipmi_watchdog.c
index 34767a6d7f42..7ed356e52035 100644
--- a/drivers/char/ipmi/ipmi_watchdog.c
+++ b/drivers/char/ipmi/ipmi_watchdog.c
@@ -153,7 +153,7 @@
 #endif
 
 static DEFINE_MUTEX(ipmi_watchdog_mutex);
-static int nowayout = WATCHDOG_NOWAYOUT;
+static bool nowayout = WATCHDOG_NOWAYOUT;
 
 static ipmi_user_t watchdog_user;
 static int watchdog_ifnum;
@@ -320,7 +320,7 @@ module_param(start_now, int, 0444);
 MODULE_PARM_DESC(start_now, "Set to 1 to start the watchdog as"
                  "soon as the driver is loaded.");
 
-module_param(nowayout, int, 0644);
+module_param(nowayout, bool, 0644);
 MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started "
                  "(default=CONFIG_WATCHDOG_NOWAYOUT)");
 
@@ -520,6 +520,7 @@ static void panic_halt_ipmi_heartbeat(void)
         msg.cmd = IPMI_WDOG_RESET_TIMER;
         msg.data = NULL;
         msg.data_len = 0;
+        atomic_add(2, &panic_done_count);
         rv = ipmi_request_supply_msgs(watchdog_user,
                                       (struct ipmi_addr *) &addr,
                                       0,
@@ -528,8 +529,8 @@ static void panic_halt_ipmi_heartbeat(void)
                                       &panic_halt_heartbeat_smi_msg,
                                       &panic_halt_heartbeat_recv_msg,
                                       1);
-        if (!rv)
-                atomic_add(2, &panic_done_count);
+        if (rv)
+                atomic_sub(2, &panic_done_count);
 }
 
 static struct ipmi_smi_msg panic_halt_smi_msg = {
@@ -553,16 +554,18 @@ static void panic_halt_ipmi_set_timeout(void)
         /* Wait for the messages to be free. */
         while (atomic_read(&panic_done_count) != 0)
                 ipmi_poll_interface(watchdog_user);
+        atomic_add(2, &panic_done_count);
         rv = i_ipmi_set_timeout(&panic_halt_smi_msg,
                                 &panic_halt_recv_msg,
                                 &send_heartbeat_now);
-        if (!rv) {
-                atomic_add(2, &panic_done_count);
-                if (send_heartbeat_now)
-                        panic_halt_ipmi_heartbeat();
-        } else
+        if (rv) {
+                atomic_sub(2, &panic_done_count);
                 printk(KERN_WARNING PFX
                        "Unable to extend the watchdog timeout.");
+        } else {
+                if (send_heartbeat_now)
+                        panic_halt_ipmi_heartbeat();
+        }
         while (atomic_read(&panic_done_count) != 0)
                 ipmi_poll_interface(watchdog_user);
 }
@@ -1164,7 +1167,7 @@ static int wdog_reboot_handler(struct notifier_block *this,
                 if (code == SYS_POWER_OFF || code == SYS_HALT) {
                         /* Disable the WDT if we are shutting down. */
                         ipmi_watchdog_state = WDOG_TIMEOUT_NONE;
-                        panic_halt_ipmi_set_timeout();
+                        ipmi_set_timeout(IPMI_SET_TIMEOUT_NO_HB);
                 } else if (ipmi_watchdog_state != WDOG_TIMEOUT_NONE) {
                         /* Set a long timer to let the reboot happens, but
                            reboot if it hangs, but only if the watchdog
@@ -1172,7 +1175,7 @@ static int wdog_reboot_handler(struct notifier_block *this,
                         timeout = 120;
                         pretimeout = 0;
                         ipmi_watchdog_state = WDOG_TIMEOUT_RESET;
-                        panic_halt_ipmi_set_timeout();
+                        ipmi_set_timeout(IPMI_SET_TIMEOUT_NO_HB);
                 }
         }
         return NOTIFY_OK;
diff --git a/drivers/char/lp.c b/drivers/char/lp.c
index f43485607063..a741e418b456 100644
--- a/drivers/char/lp.c
+++ b/drivers/char/lp.c
@@ -135,7 +135,6 @@
 
 #include <asm/irq.h>
 #include <asm/uaccess.h>
-#include <asm/system.h>
 
 /* if you have more than 8 printers, remember to increase LP_NO */
 #define LP_NO 8
@@ -706,16 +705,13 @@ static long lp_compat_ioctl(struct file *file, unsigned int cmd,
 {
         unsigned int minor;
         struct timeval par_timeout;
-        struct compat_timeval __user *tc;
         int ret;
 
         minor = iminor(file->f_path.dentry->d_inode);
         mutex_lock(&lp_mutex);
         switch (cmd) {
         case LPSETTIMEOUT:
-                tc = compat_ptr(arg);
-                if (get_user(par_timeout.tv_sec, &tc->tv_sec) ||
-                    get_user(par_timeout.tv_usec, &tc->tv_usec)) {
+                if (compat_get_timeval(&par_timeout, compat_ptr(arg))) {
                         ret = -EFAULT;
                         break;
                 }
diff --git a/drivers/char/mbcs.c b/drivers/char/mbcs.c
index 1aeaaba680d2..47ff7e470d87 100644
--- a/drivers/char/mbcs.c
+++ b/drivers/char/mbcs.c
@@ -28,7 +28,6 @@
 #include <linux/slab.h>
 #include <asm/io.h>
 #include <asm/uaccess.h>
-#include <asm/system.h>
 #include <asm/pgtable.h>
 #include <asm/sn/addrs.h>
 #include <asm/sn/intr.h>
diff --git a/drivers/char/mspec.c b/drivers/char/mspec.c
index 5c0d96a820fa..8b78750f1efe 100644
--- a/drivers/char/mspec.c
+++ b/drivers/char/mspec.c
@@ -44,7 +44,6 @@
 #include <linux/slab.h>
 #include <linux/numa.h>
 #include <asm/page.h>
-#include <asm/system.h>
 #include <asm/pgtable.h>
 #include <linux/atomic.h>
 #include <asm/tlbflush.h>
diff --git a/drivers/char/mwave/3780i.c b/drivers/char/mwave/3780i.c
index 492dbfb2efd6..881c9e595939 100644
--- a/drivers/char/mwave/3780i.c
+++ b/drivers/char/mwave/3780i.c
@@ -56,7 +56,6 @@
 
 #include <asm/io.h>
 #include <asm/uaccess.h>
-#include <asm/system.h>
 #include <asm/irq.h>
 #include "smapi.h"
 #include "mwavedd.h"
diff --git a/drivers/char/nvram.c b/drivers/char/nvram.c
index eaade8a1ecd7..9df78e2cc45d 100644
--- a/drivers/char/nvram.c
+++ b/drivers/char/nvram.c
@@ -111,7 +111,6 @@
 #include <linux/uaccess.h>
 #include <linux/mutex.h>
 
-#include <asm/system.h>
 
 static DEFINE_MUTEX(nvram_mutex);
 static DEFINE_SPINLOCK(nvram_state_lock);
diff --git a/drivers/char/nwflash.c b/drivers/char/nwflash.c
index bf586ae1ee83..d45c3345b4af 100644
--- a/drivers/char/nwflash.c
+++ b/drivers/char/nwflash.c
@@ -32,7 +32,6 @@
 #include <asm/io.h>
 #include <asm/leds.h>
 #include <asm/mach-types.h>
-#include <asm/system.h>
 #include <asm/uaccess.h>
 
 /*****************************************************************************/
diff --git a/drivers/char/pcmcia/synclink_cs.c b/drivers/char/pcmcia/synclink_cs.c
index f6453df4921c..0a484b4a1b02 100644
--- a/drivers/char/pcmcia/synclink_cs.c
+++ b/drivers/char/pcmcia/synclink_cs.c
@@ -60,7 +60,6 @@
 #include <linux/ioctl.h>
 #include <linux/synclink.h>
 
-#include <asm/system.h>
 #include <asm/io.h>
 #include <asm/irq.h>
 #include <asm/dma.h>
diff --git a/drivers/char/rtc.c b/drivers/char/rtc.c
index 872e09a02d23..af9437488b6c 100644
--- a/drivers/char/rtc.c
+++ b/drivers/char/rtc.c
@@ -83,7 +83,6 @@
 #include <linux/ratelimit.h>
 
 #include <asm/current.h>
-#include <asm/system.h>
 
 #ifdef CONFIG_X86
 #include <asm/hpet.h>
diff --git a/drivers/char/sonypi.c b/drivers/char/sonypi.c
index 1ee8ce7d2762..45713f0e7d61 100644
--- a/drivers/char/sonypi.c
+++ b/drivers/char/sonypi.c
@@ -54,7 +54,6 @@
 
 #include <asm/uaccess.h>
 #include <asm/io.h>
-#include <asm/system.h>
 
 #include <linux/sonypi.h>
 
diff --git a/drivers/char/tile-srom.c b/drivers/char/tile-srom.c
index 4dc019408fac..3b22a606f79d 100644
--- a/drivers/char/tile-srom.c
+++ b/drivers/char/tile-srom.c
@@ -194,17 +194,17 @@ static ssize_t srom_read(struct file *filp, char __user *buf,
 
                 hv_retval = _srom_read(srom->hv_devhdl, kernbuf,
                                        *f_pos, bytes_this_pass);
-                if (hv_retval > 0) {
-                        if (copy_to_user(buf, kernbuf, hv_retval) != 0) {
-                                retval = -EFAULT;
-                                break;
-                        }
-                } else if (hv_retval <= 0) {
+                if (hv_retval <= 0) {
                         if (retval == 0)
                                 retval = hv_retval;
                         break;
                 }
 
+                if (copy_to_user(buf, kernbuf, hv_retval) != 0) {
+                        retval = -EFAULT;
+                        break;
+                }
+
                 retval += hv_retval;
                 *f_pos += hv_retval;
                 buf += hv_retval;
diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c
index b58b56187065..ddf86b6500b7 100644
--- a/drivers/char/virtio_console.c
+++ b/drivers/char/virtio_console.c
@@ -1038,12 +1038,6 @@ static struct attribute_group port_attribute_group = {
         .attrs = port_sysfs_entries,
 };
 
-static int debugfs_open(struct inode *inode, struct file *filp)
-{
-        filp->private_data = inode->i_private;
-        return 0;
-}
-
 static ssize_t debugfs_read(struct file *filp, char __user *ubuf,
                             size_t count, loff_t *offp)
 {
@@ -1087,7 +1081,7 @@ static ssize_t debugfs_read(struct file *filp, char __user *ubuf,
 
 static const struct file_operations port_debugfs_ops = {
         .owner = THIS_MODULE,
-        .open  = debugfs_open,
+        .open  = simple_open,
         .read  = debugfs_read,
 };
 
diff --git a/drivers/char/xilinx_hwicap/xilinx_hwicap.c b/drivers/char/xilinx_hwicap/xilinx_hwicap.c
index e90e1c74fd4c..31ba11ca75e1 100644
--- a/drivers/char/xilinx_hwicap/xilinx_hwicap.c
+++ b/drivers/char/xilinx_hwicap/xilinx_hwicap.c
@@ -89,7 +89,6 @@
 
 #include <asm/io.h>
 #include <asm/uaccess.h>
-#include <asm/system.h>
 
 #ifdef CONFIG_OF
 /* For open firmware. */