I/OAT: list usage cleanup
authorEric Sesterhenn <[email protected]>
Thu, 26 Feb 2009 10:05:30 +0000 (11:05 +0100)
committerDan Williams <[email protected]>
Wed, 4 Mar 2009 23:04:40 +0000 (16:04 -0700)
Trivial cleanup, list_del(); list_add_tail() is equivalent
to list_move_tail(). Semantic patch for coccinelle can be
found at www.cccmz.de/~snakebyte/list_move_tail.spatch

Signed-off-by: Eric Sesterhenn <[email protected]>
Signed-off-by: Maciej Sosnowski <[email protected]>
Signed-off-by: Shannon Nelson <[email protected]>
Acked-by: Jeff Kirsher <[email protected]>
Signed-off-by: Dan Williams <[email protected]>
drivers/dma/ioat_dma.c

index fc9b845ee8937094b15db37097bf1f3883a45ed7..ae8c0ce3b86af97140ec1a3647d0968d9fdd04d1 100644 (file)
@@ -1171,9 +1171,8 @@ static void ioat_dma_memcpy_cleanup(struct ioat_dma_chan *ioat_chan)
                                 * up if the client is done with the descriptor
                                 */
                                if (async_tx_test_ack(&desc->async_tx)) {
-                                       list_del(&desc->node);
-                                       list_add_tail(&desc->node,
-                                                     &ioat_chan->free_desc);
+                                       list_move_tail(&desc->node,
+                                                      &ioat_chan->free_desc);
                                } else
                                        desc->async_tx.cookie = 0;
                        } else {