On Wed, 2008-11-19 at 14:12 +0100, Johannes Berg wrote: > On Wed, 2008-11-19 at 14:09 +0100, Johannes Berg wrote: > > On Wed, 2008-11-19 at 15:02 +0200, Tomas Winkler wrote: > > > > > > Here's a thought: iwl_rx_queue_update_write_ptr doesn't synchronise the > > > > pointer rxq->bd memory to the device. So if you have a platform that's > > > > not cache coherent for IO devices (mine is) that might be a problem. > > > > > > Need to issue wmb() there or something like this? > > > > well rb is also dma memory, no? So you'd have to use dma_sync_ or > > something, just like with skb data, no? > > or maybe not? I can't look into this today, sorry. Ah, no, you don't have to because this is allocated with pci_alloc_consistent. I'm getting confused, sorry. johannes
Attachment:
signature.asc
Description: This is a digitally signed message part