From: Michael Chan <mc...@broadcom.com> Date: Thu, 16 Apr 2015 09:39:13 -0700
> On Thu, 2015-04-16 at 09:24 -0300, casca...@linux.vnet.ibm.com wrote: >> Yes, this looks like the driver is not syncing the DMA buffers. Unmap is >> supposed to synchronize as well. >> > > For small rx packets (< 256 bytes), we sync the DMA buffer before we > copy the data to another SKB. For larger packets, we unmap the DMA > buffer. Do we see the corruption in both cases? I wonder about that prefetch which is done before the DMA sync. Also we should think about whether that DMA sync applies to the proper region. The 'len' is relative to "data+TG3_RX_OFFSET" yet if I read the code correctly we are effectively sync'ing from 'data' to 'data+len'. There is some bug hiding in here I think... -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
