bnx2x: Revert UNDI flushing mechanism
authorYuval Mintz <Yuval.Mintz@qlogic.com>
Mon, 18 Aug 2014 19:36:23 +0000 (22:36 +0300)
committerDavid S. Miller <davem@davemloft.net>
Fri, 22 Aug 2014 00:54:43 +0000 (17:54 -0700)
commit7c3afd85dc1610bb2fc049644cd1b52c7af96f98
treed741463badad3cd50fe32ccac568a8b2535e6311
parent9a6d33c30790d9eb1143aae4e5d8bbe5fc5d8570
bnx2x: Revert UNDI flushing mechanism

Commit 91ebb929b6f8 ("bnx2x: Add support for Multi-Function UNDI") [which was
later supposedly fixed by de682941eef3 ("bnx2x: Fix UNDI driver unload")]
introduced a bug in which in some [yet-to-be-determined] scenarios the
alternative flushing mechanism which was to guarantee the Rx buffers are
empty before resetting them during device probe will fail.
If this happens, when device will be loaded once more a fatal attention will
occur; Since this most likely happens in boot from SAN scenarios, the machine
will fail to load.

Notice this may occur not only in the 'Multi-Function' scenario but in the
regular scenario as well, i.e., this introduced a regression in the driver's
ability to perform boot from SAN.

The patch reverts the mechanism and applies the old scheme to multi-function
devices as well as to single-function devices.

Signed-off-by: Yuval Mintz <Yuval.Mintz@qlogic.com>
Signed-off-by: Ariel Elior <Ariel.Elior@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c