On Mon, Dec 17, 2018 at 4:12 PM Stefan Theil stefan.theil@mixed-mode.de wrote:
The cache was only flushed before *transmitting* packets, but not when receiving them, leading to an issue where new packets were handed to the receive handler with old contents in cache. This only happens when a lot of packets are received without sending packages every now and then. Also flushing the receive buffers in the transmit function makes no sense and can be removed.
Signed-off-by: Stefan Theil stefan.theil@mixed-mode.de
Changes for v2: - Use invalidate_dcache_range instead of flush_dcache_range Changes for v3: - Remove unnecessary flushing of all RX buffers in zynq_gem_send Changes for v4: - Invalidate receive buffers after allocating them in zynq_gem_probe Changes for v5: - Clear and flush receive buffers in zynq_gem_probe instead of invalidating the cache
drivers/net/zynq_gem.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-)
Reviewed-by: Bin Meng bmeng.cn@gmail.com