> From: David Miller [mailto:da...@davemloft.net] > > Looking at the bnx2x driver, it's management of link state is very convoluted. > > The link parameters and the "link_up" state is maintained separately from > the values that are snapshot when the carrier is enabled. ... > If the carrier is up, you should be able to provide the link speed and duplex > values immediately not some indeterminate amount of time later.
This was steady state, the machine was stuck like this for hours/days. > I don't think we should reward drivers for behaving this way. I'll send v3 without this patch.