Commit ee6cd5f8 authored by Corey Minyard's avatar Corey Minyard Committed by Linus Torvalds

ipmi: allow shared interrupts

The IPMI driver used enable_irq and disable_irq when it got into situations
where it couldn't allocate memory; it did this to avoid having the interrupt
just lock the machine when it couldn't get memory to perform the transaction
to disable the interrupt.

This patch modifies the driver to not use disable_irq and enable_irq.  It
instead sends the messages to the BMC to perform this operation.  It also
makes sure interrupts are cleanly disabled when the interface is shut down and
cleans up some shutdown things that are no longer necessary.
Signed-off-by: default avatarCorey Minyard <cminyard@mvista.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent dba9b4f6
...@@ -82,6 +82,12 @@ ...@@ -82,6 +82,12 @@
#define SI_SHORT_TIMEOUT_USEC 250 /* .25ms when the SM request a #define SI_SHORT_TIMEOUT_USEC 250 /* .25ms when the SM request a
short timeout */ short timeout */
/* Bit for BMC global enables. */
#define IPMI_BMC_RCV_MSG_INTR 0x01
#define IPMI_BMC_EVT_MSG_INTR 0x02
#define IPMI_BMC_EVT_MSG_BUFF 0x04
#define IPMI_BMC_SYS_LOG 0x08
enum si_intf_state { enum si_intf_state {
SI_NORMAL, SI_NORMAL,
SI_GETTING_FLAGS, SI_GETTING_FLAGS,
...@@ -90,7 +96,9 @@ enum si_intf_state { ...@@ -90,7 +96,9 @@ enum si_intf_state {
SI_CLEARING_FLAGS_THEN_SET_IRQ, SI_CLEARING_FLAGS_THEN_SET_IRQ,
SI_GETTING_MESSAGES, SI_GETTING_MESSAGES,
SI_ENABLE_INTERRUPTS1, SI_ENABLE_INTERRUPTS1,
SI_ENABLE_INTERRUPTS2 SI_ENABLE_INTERRUPTS2,
SI_DISABLE_INTERRUPTS1,
SI_DISABLE_INTERRUPTS2
/* FIXME - add watchdog stuff. */ /* FIXME - add watchdog stuff. */
}; };
...@@ -339,6 +347,17 @@ static void start_enable_irq(struct smi_info *smi_info) ...@@ -339,6 +347,17 @@ static void start_enable_irq(struct smi_info *smi_info)
smi_info->si_state = SI_ENABLE_INTERRUPTS1; smi_info->si_state = SI_ENABLE_INTERRUPTS1;
} }
static void start_disable_irq(struct smi_info *smi_info)
{
unsigned char msg[2];
msg[0] = (IPMI_NETFN_APP_REQUEST << 2);
msg[1] = IPMI_GET_BMC_GLOBAL_ENABLES_CMD;
smi_info->handlers->start_transaction(smi_info->si_sm, msg, 2);
smi_info->si_state = SI_DISABLE_INTERRUPTS1;
}
static void start_clear_flags(struct smi_info *smi_info) static void start_clear_flags(struct smi_info *smi_info)
{ {
unsigned char msg[3]; unsigned char msg[3];
...@@ -359,7 +378,7 @@ static void start_clear_flags(struct smi_info *smi_info) ...@@ -359,7 +378,7 @@ static void start_clear_flags(struct smi_info *smi_info)
static inline void disable_si_irq(struct smi_info *smi_info) static inline void disable_si_irq(struct smi_info *smi_info)
{ {
if ((smi_info->irq) && (!smi_info->interrupt_disabled)) { if ((smi_info->irq) && (!smi_info->interrupt_disabled)) {
disable_irq_nosync(smi_info->irq); start_disable_irq(smi_info);
smi_info->interrupt_disabled = 1; smi_info->interrupt_disabled = 1;
} }
} }
...@@ -367,7 +386,7 @@ static inline void disable_si_irq(struct smi_info *smi_info) ...@@ -367,7 +386,7 @@ static inline void disable_si_irq(struct smi_info *smi_info)
static inline void enable_si_irq(struct smi_info *smi_info) static inline void enable_si_irq(struct smi_info *smi_info)
{ {
if ((smi_info->irq) && (smi_info->interrupt_disabled)) { if ((smi_info->irq) && (smi_info->interrupt_disabled)) {
enable_irq(smi_info->irq); start_enable_irq(smi_info);
smi_info->interrupt_disabled = 0; smi_info->interrupt_disabled = 0;
} }
} }
...@@ -589,7 +608,9 @@ static void handle_transaction_done(struct smi_info *smi_info) ...@@ -589,7 +608,9 @@ static void handle_transaction_done(struct smi_info *smi_info)
} else { } else {
msg[0] = (IPMI_NETFN_APP_REQUEST << 2); msg[0] = (IPMI_NETFN_APP_REQUEST << 2);
msg[1] = IPMI_SET_BMC_GLOBAL_ENABLES_CMD; msg[1] = IPMI_SET_BMC_GLOBAL_ENABLES_CMD;
msg[2] = msg[3] | 1; /* enable msg queue int */ msg[2] = (msg[3] |
IPMI_BMC_RCV_MSG_INTR |
IPMI_BMC_EVT_MSG_INTR);
smi_info->handlers->start_transaction( smi_info->handlers->start_transaction(
smi_info->si_sm, msg, 3); smi_info->si_sm, msg, 3);
smi_info->si_state = SI_ENABLE_INTERRUPTS2; smi_info->si_state = SI_ENABLE_INTERRUPTS2;
...@@ -611,6 +632,45 @@ static void handle_transaction_done(struct smi_info *smi_info) ...@@ -611,6 +632,45 @@ static void handle_transaction_done(struct smi_info *smi_info)
smi_info->si_state = SI_NORMAL; smi_info->si_state = SI_NORMAL;
break; break;
} }
case SI_DISABLE_INTERRUPTS1:
{
unsigned char msg[4];
/* We got the flags from the SMI, now handle them. */
smi_info->handlers->get_result(smi_info->si_sm, msg, 4);
if (msg[2] != 0) {
printk(KERN_WARNING
"ipmi_si: Could not disable interrupts"
", failed get.\n");
smi_info->si_state = SI_NORMAL;
} else {
msg[0] = (IPMI_NETFN_APP_REQUEST << 2);
msg[1] = IPMI_SET_BMC_GLOBAL_ENABLES_CMD;
msg[2] = (msg[3] &
~(IPMI_BMC_RCV_MSG_INTR |
IPMI_BMC_EVT_MSG_INTR));
smi_info->handlers->start_transaction(
smi_info->si_sm, msg, 3);
smi_info->si_state = SI_DISABLE_INTERRUPTS2;
}
break;
}
case SI_DISABLE_INTERRUPTS2:
{
unsigned char msg[4];
/* We got the flags from the SMI, now handle them. */
smi_info->handlers->get_result(smi_info->si_sm, msg, 4);
if (msg[2] != 0) {
printk(KERN_WARNING
"ipmi_si: Could not disable interrupts"
", failed set.\n");
}
smi_info->si_state = SI_NORMAL;
break;
}
} }
} }
...@@ -864,9 +924,6 @@ static void smi_timeout(unsigned long data) ...@@ -864,9 +924,6 @@ static void smi_timeout(unsigned long data)
struct timeval t; struct timeval t;
#endif #endif
if (atomic_read(&smi_info->stop_operation))
return;
spin_lock_irqsave(&(smi_info->si_lock), flags); spin_lock_irqsave(&(smi_info->si_lock), flags);
#ifdef DEBUG_TIMING #ifdef DEBUG_TIMING
do_gettimeofday(&t); do_gettimeofday(&t);
...@@ -922,15 +979,11 @@ static irqreturn_t si_irq_handler(int irq, void *data) ...@@ -922,15 +979,11 @@ static irqreturn_t si_irq_handler(int irq, void *data)
smi_info->interrupts++; smi_info->interrupts++;
spin_unlock(&smi_info->count_lock); spin_unlock(&smi_info->count_lock);
if (atomic_read(&smi_info->stop_operation))
goto out;
#ifdef DEBUG_TIMING #ifdef DEBUG_TIMING
do_gettimeofday(&t); do_gettimeofday(&t);
printk("**Interrupt: %d.%9.9d\n", t.tv_sec, t.tv_usec); printk("**Interrupt: %d.%9.9d\n", t.tv_sec, t.tv_usec);
#endif #endif
smi_event_handler(smi_info, 0); smi_event_handler(smi_info, 0);
out:
spin_unlock_irqrestore(&(smi_info->si_lock), flags); spin_unlock_irqrestore(&(smi_info->si_lock), flags);
return IRQ_HANDLED; return IRQ_HANDLED;
} }
...@@ -1118,7 +1171,7 @@ static int std_irq_setup(struct smi_info *info) ...@@ -1118,7 +1171,7 @@ static int std_irq_setup(struct smi_info *info)
if (info->si_type == SI_BT) { if (info->si_type == SI_BT) {
rv = request_irq(info->irq, rv = request_irq(info->irq,
si_bt_irq_handler, si_bt_irq_handler,
IRQF_DISABLED, IRQF_SHARED | IRQF_DISABLED,
DEVICE_NAME, DEVICE_NAME,
info); info);
if (!rv) if (!rv)
...@@ -1128,7 +1181,7 @@ static int std_irq_setup(struct smi_info *info) ...@@ -1128,7 +1181,7 @@ static int std_irq_setup(struct smi_info *info)
} else } else
rv = request_irq(info->irq, rv = request_irq(info->irq,
si_irq_handler, si_irq_handler,
IRQF_DISABLED, IRQF_SHARED | IRQF_DISABLED,
DEVICE_NAME, DEVICE_NAME,
info); info);
if (rv) { if (rv) {
...@@ -1708,15 +1761,11 @@ static u32 ipmi_acpi_gpe(void *context) ...@@ -1708,15 +1761,11 @@ static u32 ipmi_acpi_gpe(void *context)
smi_info->interrupts++; smi_info->interrupts++;
spin_unlock(&smi_info->count_lock); spin_unlock(&smi_info->count_lock);
if (atomic_read(&smi_info->stop_operation))
goto out;
#ifdef DEBUG_TIMING #ifdef DEBUG_TIMING
do_gettimeofday(&t); do_gettimeofday(&t);
printk("**ACPI_GPE: %d.%9.9d\n", t.tv_sec, t.tv_usec); printk("**ACPI_GPE: %d.%9.9d\n", t.tv_sec, t.tv_usec);
#endif #endif
smi_event_handler(smi_info, 0); smi_event_handler(smi_info, 0);
out:
spin_unlock_irqrestore(&(smi_info->si_lock), flags); spin_unlock_irqrestore(&(smi_info->si_lock), flags);
return ACPI_INTERRUPT_HANDLED; return ACPI_INTERRUPT_HANDLED;
...@@ -2942,28 +2991,33 @@ static void cleanup_one_si(struct smi_info *to_clean) ...@@ -2942,28 +2991,33 @@ static void cleanup_one_si(struct smi_info *to_clean)
list_del(&to_clean->link); list_del(&to_clean->link);
/* Tell the timer and interrupt handlers that we are shutting /* Tell the driver that we are shutting down. */
down. */
spin_lock_irqsave(&(to_clean->si_lock), flags);
spin_lock(&(to_clean->msg_lock));
atomic_inc(&to_clean->stop_operation); atomic_inc(&to_clean->stop_operation);
if (to_clean->irq_cleanup) /* Make sure the timer and thread are stopped and will not run
to_clean->irq_cleanup(to_clean); again. */
spin_unlock(&(to_clean->msg_lock));
spin_unlock_irqrestore(&(to_clean->si_lock), flags);
/* Wait until we know that we are out of any interrupt
handlers might have been running before we freed the
interrupt. */
synchronize_sched();
wait_for_timer_and_thread(to_clean); wait_for_timer_and_thread(to_clean);
/* Interrupts and timeouts are stopped, now make sure the /* Timeouts are stopped, now make sure the interrupts are off
interface is in a clean state. */ for the device. A little tricky with locks to make sure
there are no races. */
spin_lock_irqsave(&to_clean->si_lock, flags);
while (to_clean->curr_msg || (to_clean->si_state != SI_NORMAL)) {
spin_unlock_irqrestore(&to_clean->si_lock, flags);
poll(to_clean);
schedule_timeout_uninterruptible(1);
spin_lock_irqsave(&to_clean->si_lock, flags);
}
disable_si_irq(to_clean);
spin_unlock_irqrestore(&to_clean->si_lock, flags);
while (to_clean->curr_msg || (to_clean->si_state != SI_NORMAL)) {
poll(to_clean);
schedule_timeout_uninterruptible(1);
}
/* Clean up interrupts and make sure that everything is done. */
if (to_clean->irq_cleanup)
to_clean->irq_cleanup(to_clean);
while (to_clean->curr_msg || (to_clean->si_state != SI_NORMAL)) { while (to_clean->curr_msg || (to_clean->si_state != SI_NORMAL)) {
poll(to_clean); poll(to_clean);
schedule_timeout_uninterruptible(1); schedule_timeout_uninterruptible(1);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment