IB/qib: Correct ordering of reregister vs. port active events
authorTodd Rimmer <todd.rimmer@qlogic.com>
Wed, 2 May 2012 18:35:18 +0000 (14:35 -0400)
committerRoland Dreier <roland@purestorage.com>
Mon, 14 May 2012 19:38:28 +0000 (12:38 -0700)
commit4ccf28a26c981bcc6f938a7ea293a623d2d66d7b
tree63eb2386d413a526ca8458d6f4508c0562a51cc9
parentbb77a077232e78476d7bc39c080f9e6685cbfd3c
IB/qib: Correct ordering of reregister vs. port active events

When a port first goes active with SMA Set(PortInfo) and reregister
bit set, the driver sends up the reregister event followed by a port
active event.

The problem is that in response to reregister event most apps try to
issue a SA query of some sort, but that fails because port is not
active.

The qib driver needs to a trivial change to correct this behavior.

This issue has been there for a while; however the recent serdes work
has probably made the delay between the reregister event and the
active event larger and hence opened the race far enough so that its
being seen more often.

The patch also changes the clientrereg local to a u8 and saves off the
rereg bit into it.  The code following the nested subn_get_portinfo()
now restores that bit per o14-12.2.1 with a logical OR from that copy.

Reviewed-by: Ram Vepa <ram.vepa@qlogic.com>
Signed-off-by: Mike Marciniszyn <mike.marciniszyn@qlogic.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
drivers/infiniband/hw/qib/qib_mad.c