Jay Vosburgh wrote:
I believe I see the race Michael refers to (tlb_choose_channel
may set head, which tlb_init_slave clears), although I was not able to
reproduce it. I have updated his patch for the current netdev-2.6.git
tree and added a version update. His original comment follows:
Our systems have been crashing during testing of PCI HotPlug
support in the various networking components. We've faulted in
the bonding driver due to a bug in bond_alb.c:tlb_clear_slave()
In that routine, the last modification to the TLB hash table is
made without protection of the lock, allowing a race that can lead
tlb_choose_channel() to select an invalid table element.
-J
---
-Jay Vosburgh, IBM Linux Technology Center, [EMAIL PROTECTED]
Signed-off-by: Michael O'Donnell <Michael.ODonnell at stratus dot com>
Signed-off-by: Jay Vosburgh <[EMAIL PROTECTED]>
applied.
Please fix your future submissions to eliminate your signature and
.signature. git-applymbox, quite rightly, elides everything after the
standard "---" delimiter.
Anything that doesn't belong in the kernel changelog must be hand-edited
out by me, before running scripts to merge your change.
Jeff
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
