net: thunderbolt: Clear finished Tx frame bus address in tbnet_tx_callback()
authorMika Westerberg <mika.westerberg@linux.intel.com>
Thu, 9 Nov 2017 10:46:28 +0000 (13:46 +0300)
committerDavid S. Miller <davem@davemloft.net>
Sat, 11 Nov 2017 10:20:45 +0000 (19:20 +0900)
When Thunderbolt network interface is disabled or when the cable is
unplugged the driver releases all allocated buffers by calling
tbnet_free_buffers() for each ring. This function then calls
dma_unmap_page() for each buffer it finds where bus address is non-zero.
Now, we only clear this bus address when the Tx buffer is sent to the
hardware so it is possible that the function finds an entry that has
already been unmapped.

Enabling DMA-API debugging catches this as well:

  thunderbolt 0000:06:00.0: DMA-API: device driver tries to free DMA
    memory it has not allocated [device address=0x0000000068321000] [size=4096 bytes]

Fix this by clearing the bus address of a Tx frame right after we have
unmapped the buffer.

Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/thunderbolt.c

index 435854688a7a9ae0d66790de8c1f1cc43d4b691a..228d4aa6d9ae3e08d688296cc4d8e81e6dcde81e 100644 (file)
@@ -536,6 +536,7 @@ static void tbnet_tx_callback(struct tb_ring *ring, struct ring_frame *frame,
 
        dma_unmap_page(dma_dev, tf->frame.buffer_phy, tbnet_frame_size(tf),
                       DMA_TO_DEVICE);
+       tf->frame.buffer_phy = 0;
 
        /* Return buffer to the ring */
        net->tx_ring.prod++;