IB/ipath: Shadow the gpio_mask register
authorArthur Jones <arthur.jones@qlogic.com>
Thu, 10 May 2007 19:10:49 +0000 (12:10 -0700)
committerRoland Dreier <rolandd@cisco.com>
Mon, 14 May 2007 20:22:42 +0000 (13:22 -0700)
commit8f140b407f3be04e7202be9aa0cfef3006d14c9f
tree34f308cacd261f03cc3e490a187df369f0bac7f3
parent26c6bc7b812b4157ba929035e467c0f4dd165916
IB/ipath: Shadow the gpio_mask register

Once upon a time, GPIO interrupts were rare.  But then a chip bug in
the waldo series forced the use of a GPIO interrupt to signal packet
reception.  This greatly increased the frequency of GPIO interrupts
which have the gpio_mask bits set on the waldo chips.  Other bits in
the gpio_status register are used for I2C clock and data lines, these
bits are usually on.  An "unlikely" annotation leftover from the old
days was improperly applied to these bits, and an unnecessary chip
mmio read was being accessed in the interrupt fast path on waldo.

Remove the stagnant unlikely annotation in the interrupt handler and
keep a shadow copy of the gpio_mask register to avoid the slow mmio
read when testing for interruptable GPIO bits.

Signed-off-by: Arthur Jones <arthur.jones@qlogic.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
drivers/infiniband/hw/ipath/ipath_iba6120.c
drivers/infiniband/hw/ipath/ipath_intr.c
drivers/infiniband/hw/ipath/ipath_kernel.h
drivers/infiniband/hw/ipath/ipath_verbs.c