diff options
author | Francois Romieu <romieu@fr.zoreil.com> | 2008-08-28 22:55:33 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-09-08 03:20:16 -0700 |
commit | a3ec4ed4b27926b224a5acc2325d70a50f75c48d (patch) | |
tree | 183ca014ae218f07b83108950e5bf599912fa136 | |
parent | fde5378952ed3550b5e0eb2310be6687baeed143 (diff) | |
download | lwn-a3ec4ed4b27926b224a5acc2325d70a50f75c48d.tar.gz lwn-a3ec4ed4b27926b224a5acc2325d70a50f75c48d.zip |
r8169: balance pci_map / pci_unmap pair
commit a866bbf6aacf95f849810079442a20be118ce905 upstream
The leak hurts with swiotlb and jumbo frames.
Fix http://bugzilla.kernel.org/show_bug.cgi?id=9468.
Heavily hinted by Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>.
Signed-off-by: Francois Romieu <romieu@fr.zoreil.com>
Tested-by: Alistair John Strachan <alistair@devzero.co.uk>
Tested-by: Timothy J Fontaine <tjfontaine@atxconsulting.com>
Cc: Edward Hsu <edward_hsu@realtek.com.tw>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/net/r8169.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/r8169.c b/drivers/net/r8169.c index 42d7c0a38477..0e4eb15d3ce7 100644 --- a/drivers/net/r8169.c +++ b/drivers/net/r8169.c @@ -2822,7 +2822,7 @@ static int rtl8169_rx_interrupt(struct net_device *dev, pkt_size, PCI_DMA_FROMDEVICE); rtl8169_mark_to_asic(desc, tp->rx_buf_sz); } else { - pci_unmap_single(pdev, addr, pkt_size, + pci_unmap_single(pdev, addr, tp->rx_buf_sz, PCI_DMA_FROMDEVICE); tp->Rx_skbuff[entry] = NULL; } |