diff options
| author |   Stephen Hemminger <shemminger@osdl.org> | 2006-05-15 16:32:39 -0700 |
|---|---|---|
| committer |   Jeff Garzik <jeff@garzik.org> | 2006-05-20 00:17:07 -0400 |
| commit | a06631cbdc09fe33892f08238be498eaa84892ee (patch) | |
| tree | 52d7d6d7677815f8be197bde05bb4f2df1a76bf4 /drivers/net/skge.c | |
| parent | [PATCH] skge: bad checksums on big-endian platforms (diff) | |
| download | linux-dev-a06631cbdc09fe33892f08238be498eaa84892ee.tar.xz linux-dev-a06631cbdc09fe33892f08238be498eaa84892ee.zip | |
[PATCH] skge: don't allow transmit ring to be too small
The driver will get stuck (permanent transmit timeout), if the transmit
ring size is set too small. It needs to have enough ring elements to
hold one maximum size transmit.
Signed-off-by: Stephen Hemminger <shemminger@osdl.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/skge.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/skge.c b/drivers/net/skge.c index 0dd42470136d..5ca5a1b546a1 100644 --- a/drivers/net/skge.c +++ b/drivers/net/skge.c @@ -401,7 +401,7 @@ static int skge_set_ring_param(struct net_device *dev, int err; if (p->rx_pending == 0 || p->rx_pending > MAX_RX_RING_SIZE || - p->tx_pending == 0 || p->tx_pending > MAX_TX_RING_SIZE) + p->tx_pending < MAX_SKB_FRAGS+1 || p->tx_pending > MAX_TX_RING_SIZE) return -EINVAL; skge->rx_ring.count = p->rx_pending; |