diff options
| author | Florian Westphal <fw@strlen.de> | 2016-04-24 15:38:14 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-04-26 15:53:05 -0400 |
| commit | f0cdf76c103ffa34ca5ac87dcdef7edffc722cbf (patch) | |
| tree | 893afe6a3b448bfeb0dd36cae35a6ce844230852 /Documentation/networking | |
| parent | a6086a893718db07ef9e7af5624ec27cb376ef0a (diff) | |
net: remove NETDEV_TX_LOCKED support
No more users in the tree, remove NETDEV_TX_LOCKED support.
Adds another hole in softnet_stats struct, but better than keeping
the unused collision counter around.
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'Documentation/networking')
| -rw-r--r-- | Documentation/networking/netdev-features.txt | 10 | ||||
| -rw-r--r-- | Documentation/networking/netdevices.txt | 9 |
2 files changed, 7 insertions, 12 deletions
diff --git a/Documentation/networking/netdev-features.txt b/Documentation/networking/netdev-features.txt index f310edec8a77..7413eb05223b 100644 --- a/Documentation/networking/netdev-features.txt +++ b/Documentation/networking/netdev-features.txt | |||
| @@ -131,13 +131,11 @@ stack. Driver should not change behaviour based on them. | |||
| 131 | 131 | ||
| 132 | * LLTX driver (deprecated for hardware drivers) | 132 | * LLTX driver (deprecated for hardware drivers) |
| 133 | 133 | ||
| 134 | NETIF_F_LLTX should be set in drivers that implement their own locking in | 134 | NETIF_F_LLTX is meant to be used by drivers that don't need locking at all, |
| 135 | transmit path or don't need locking at all (e.g. software tunnels). | 135 | e.g. software tunnels. |
| 136 | In ndo_start_xmit, it is recommended to use a try_lock and return | ||
| 137 | NETDEV_TX_LOCKED when the spin lock fails. The locking should also properly | ||
| 138 | protect against other callbacks (the rules you need to find out). | ||
| 139 | 136 | ||
| 140 | Don't use it for new drivers. | 137 | This is also used in a few legacy drivers that implement their |
| 138 | own locking, don't use it for new (hardware) drivers. | ||
| 141 | 139 | ||
| 142 | * netns-local device | 140 | * netns-local device |
| 143 | 141 | ||
diff --git a/Documentation/networking/netdevices.txt b/Documentation/networking/netdevices.txt index 0b1cf6b2a592..7fec2061a334 100644 --- a/Documentation/networking/netdevices.txt +++ b/Documentation/networking/netdevices.txt | |||
| @@ -69,10 +69,9 @@ ndo_start_xmit: | |||
| 69 | 69 | ||
| 70 | When the driver sets NETIF_F_LLTX in dev->features this will be | 70 | When the driver sets NETIF_F_LLTX in dev->features this will be |
| 71 | called without holding netif_tx_lock. In this case the driver | 71 | called without holding netif_tx_lock. In this case the driver |
| 72 | has to lock by itself when needed. It is recommended to use a try lock | 72 | has to lock by itself when needed. |
| 73 | for this and return NETDEV_TX_LOCKED when the spin lock fails. | 73 | The locking there should also properly protect against |
| 74 | The locking there should also properly protect against | 74 | set_rx_mode. WARNING: use of NETIF_F_LLTX is deprecated. |
| 75 | set_rx_mode. Note that the use of NETIF_F_LLTX is deprecated. | ||
| 76 | Don't use it for new drivers. | 75 | Don't use it for new drivers. |
| 77 | 76 | ||
| 78 | Context: Process with BHs disabled or BH (timer), | 77 | Context: Process with BHs disabled or BH (timer), |
| @@ -83,8 +82,6 @@ ndo_start_xmit: | |||
| 83 | o NETDEV_TX_BUSY Cannot transmit packet, try later | 82 | o NETDEV_TX_BUSY Cannot transmit packet, try later |
| 84 | Usually a bug, means queue start/stop flow control is broken in | 83 | Usually a bug, means queue start/stop flow control is broken in |
| 85 | the driver. Note: the driver must NOT put the skb in its DMA ring. | 84 | the driver. Note: the driver must NOT put the skb in its DMA ring. |
| 86 | o NETDEV_TX_LOCKED Locking failed, please retry quickly. | ||
| 87 | Only valid when NETIF_F_LLTX is set. | ||
| 88 | 85 | ||
| 89 | ndo_tx_timeout: | 86 | ndo_tx_timeout: |
| 90 | Synchronization: netif_tx_lock spinlock; all TX queues frozen. | 87 | Synchronization: netif_tx_lock spinlock; all TX queues frozen. |
