]> git.baikalelectronics.ru Git - kernel.git/commit
ioat2: fix performance regression
authorDan Williams <dan.j.williams@intel.com>
Wed, 13 Oct 2010 22:43:10 +0000 (15:43 -0700)
committerDan Williams <dan.j.williams@intel.com>
Wed, 13 Oct 2010 22:43:10 +0000 (15:43 -0700)
commit6c8de1beb5d5ccd4ee5a6a4ac7a625825540d620
tree271750360d57da1d982686d8435b8eb3bf3086e4
parent9c7897fd128eb940072c9c568023e950aed319c5
ioat2: fix performance regression

Commit 79bd524 "DMAENGINE: generic channel status v2" changed the interface for
how dma channel progress is retrieved.  It inadvertently exported an internal
helper function ioat_tx_status() instead of ioat_dma_tx_status().  The latter
polls the hardware to get the latest completion state, while the helper just
evaluates the current state without touching hardware.  The effect is that we
end up waiting for completion timeouts or descriptor allocation errors before
the completion state is updated.

iperf (before fix):
[SUM]  0.0-41.3 sec   364 MBytes  73.9 Mbits/sec

iperf (after fix):
[SUM]  0.0- 4.5 sec   499 MBytes   940 Mbits/sec

This is a regression starting with 2.6.35.

Cc: <stable@kernel.org>
Cc: Dave Jiang <dave.jiang@intel.com>
Cc: Jesse Brandeburg <jesse.brandeburg@intel.com>
Cc: Linus Walleij <linus.walleij@stericsson.com>
Cc: Maciej Sosnowski <maciej.sosnowski@intel.com>
Reported-by: Richard Scobie <richard@sauce.co.nz>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
drivers/dma/ioat/dma_v2.c