diff options
| author | David S. Miller <davem@davemloft.net> | 2010-03-15 15:23:30 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-04-01 16:02:06 -0700 |
| commit | 8fc51479fb9f3b9b06e65ce266488bae55c2965a (patch) | |
| tree | 821a40de7c30d9ce79c9ecf992cdd13641806e9c /drivers | |
| parent | 45e74e6561f0da79a417be5424d2c1bf26ae79ba (diff) | |
| download | lwn-8fc51479fb9f3b9b06e65ce266488bae55c2965a.tar.gz lwn-8fc51479fb9f3b9b06e65ce266488bae55c2965a.zip | |
e100: Fix ring parameter change handling regression.
[ Upstream commit 211a0d941b1924e667483f822a55e2cc694cd212 ]
When the PCI pool changes were added to fix resume failures:
commit 98468efddb101f8a29af974101c17ba513b07be1
e100: Use pci pool to work around GFP_ATOMIC order 5 memory allocation failu
and
commit 70abc8cb90e679d8519721e2761d8366a18212a6
e100: Fix broken cbs accounting due to missing memset.
This introduced a problem that can happen if the TX ring size
is increased. We need to size the PCI pool using cbs->max
instead of the default cbs->count value.
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/e100.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/e100.c b/drivers/net/e100.c index 839fb2b136d3..a565ea1d6067 100644 --- a/drivers/net/e100.c +++ b/drivers/net/e100.c @@ -2854,7 +2854,7 @@ static int __devinit e100_probe(struct pci_dev *pdev, } nic->cbs_pool = pci_pool_create(netdev->name, nic->pdev, - nic->params.cbs.count * sizeof(struct cb), + nic->params.cbs.max * sizeof(struct cb), sizeof(u32), 0); DPRINTK(PROBE, INFO, "addr 0x%llx, irq %d, MAC addr %pM\n", |