]> git.baikalelectronics.ru Git - kernel.git/commit
powerpc: Eeh: Kill another abuse of irq_desc
authorThomas Gleixner <tglx@linutronix.de>
Sun, 23 Feb 2014 21:40:09 +0000 (21:40 +0000)
committerThomas Gleixner <tglx@linutronix.de>
Tue, 4 Mar 2014 16:37:52 +0000 (17:37 +0100)
commit52b7273219411a5811abad4ac75bdffcde23c23e
tree51fec4ea3be90869ff05f7540aed030e3fdcd5c5
parente736fd1577fd2b272df15bf7492a95dade36dea4
powerpc: Eeh: Kill another abuse of irq_desc

commit cff1b9533 (powerpc/eeh: Fix unbalanced enable for IRQ) is
another brilliant example of trainwreck engineering.

The patch "fixes" the issue of an unbalanced call to irq_enable()
which causes a prominent warning by checking the disabled state of the
interrupt line and call conditionally into the core code.

This is wrong in two aspects:

1) The warning is there to tell users, that they need to fix their
   asymetric enable/disable patterns by finding the root cause and
   solving it there.

   It's definitely not meant to work around it by conditionally
   calling into the core code depending on the random state of the irq
   line.

   Asymetric irq_disable/enable calls are a clear sign of wrong usage
   of the interfaces which have to be cured at the root and not by
   somehow hacking around it.

2) The abuse of core internal data structure instead of using the
   proper interfaces for retrieving the information for the 'hack
   around'

   irq_desc is core internal and it's clear enough stated.

Replace at least the irq_desc abuse with the proper functions and add
a big fat comment why this is absurd and completely wrong.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Gavin Shan <shangw@linux.vnet.ibm.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: ppc <linuxppc-dev@lists.ozlabs.org>
Link: http://lkml.kernel.org/r/20140223212736.562906212@linutronix.de
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
arch/powerpc/kernel/eeh_driver.c