diff options
author | Jeff Garzik <jgarzik@redhat.com> | 2007-10-17 23:26:43 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2007-10-17 23:26:43 -0700 |
commit | bfaae0f04c68bafc12ec50c6922d71a90deea3e2 (patch) | |
tree | 91153680d9ad5ed9aa3d29207fbbeecc82bbb738 /README | |
parent | 45542479fb261342d5244869cf3ca4636b7ffd43 (diff) | |
download | op-kernel-dev-bfaae0f04c68bafc12ec50c6922d71a90deea3e2.zip op-kernel-dev-bfaae0f04c68bafc12ec50c6922d71a90deea3e2.tar.gz |
[NET]: fix carrier-on bug?
While looking at a net driver with the following construct,
if (!netif_carrier_ok(dev))
netif_carrier_on(dev);
it stuck me that the netif_carrier_ok() check was redundant, since
netif_carrier_on() checks bit __LINK_STATE_NOCARRIER anyway. This is
the same reason why netif_queue_stopped() need not be called prior to
netif_wake_queue().
This is true, but there is however an unwanted side effect from assuming
that netif_carrier_on() can be called multiple times: it touches the
watchdog, regardless of pre-existing carrier state.
The fix: move watchdog-up inside the bit-cleared code path.
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'README')
0 files changed, 0 insertions, 0 deletions