diff options
| author | Krzysztof Hałasa <khc@pm.waw.pl> | 2009-08-24 04:02:13 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-08-24 04:02:13 +0200 |
| commit | 6ff9c2e7fa8ca63a575792534b63c5092099c286 (patch) | |
| tree | 60ff80e67bea9798539700f2aa876ed2458dc0be /drivers/net/e100.c | |
| parent | pkt_sched: Convert CBQ to tasklet_hrtimer. (diff) | |
| download | linux-6ff9c2e7fa8ca63a575792534b63c5092099c286.tar.xz linux-6ff9c2e7fa8ca63a575792534b63c5092099c286.zip | |
E100: fix interaction with swiotlb on X86.
E100 places it's RX packet descriptors inside skb->data and uses them
with bidirectional streaming DMA mapping. Data in descriptors is
accessed simultaneously by the chip (writing status and size when
a packet is received) and CPU (reading to check if the packet was
received). This isn't a valid usage of PCI DMA API, which requires use
of the coherent (consistent) memory for such purpose. Unfortunately e100
chips working in "simplified" RX mode have to store received data
directly after the descriptor. Fixing the driver to conform to the API
would require using unsupported "flexible" RX mode or receiving data
into a coherent memory and using CPU to copy it to network buffers.
This patch, while not yet making the driver conform to the PCI DMA API,
allows it to work correctly on X86 with swiotlb (while not breaking
other architectures).
Signed-off-by: Krzysztof Hałasa <khc@pm.waw.pl>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/e100.c')
| -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 41b648a67fec..3a6735dc9f6a 100644 --- a/drivers/net/e100.c +++ b/drivers/net/e100.c @@ -1899,7 +1899,7 @@ static int e100_rx_indicate(struct nic *nic, struct rx *rx, nic->ru_running = RU_SUSPENDED; pci_dma_sync_single_for_device(nic->pdev, rx->dma_addr, sizeof(struct rfd), - PCI_DMA_BIDIRECTIONAL); + PCI_DMA_FROMDEVICE); return -ENODATA; } |