Thread (2 messages) 2 messages, 2 authors, 2012-07-30

RE: [PATCH v4 0/7] Raid: enable talitos xor offload for improving performance

From: Liu Qiang-B32616 <hidden>
Date: 2012-07-30 02:06:10
Also in: linux-crypto

Hi,

Vinod, Dan, ping?

-----Original Message-----
From: linux-crypto-owner@vger.kernel.org [mailto:linux-crypto-
owner@vger.kernel.org] On Behalf Of qiang.liu@freescale.com
Sent: Friday, July 27, 2012 5:16 PM
To: linux-crypto@vger.kernel.org; vinod.koul@intel.com;
dan.j.williams@intel.com; herbert@gondor.hengli.com.au; linuxppc-
dev@lists.ozlabs.org
Cc: Li Yang-R58472; Phillips Kim-R1AAHA
Subject: [PATCH v4 0/7] Raid: enable talitos xor offload for improving
performance
=20
Hi,
=20
The following 7 patches enabling fsl-dma and talitos offload raid
operations for improving raid performance and balancing CPU load.
=20
Write performance will be improved by 25-30% tested by iozone.
Write performance is improved about 2% after using spin_lock_bh replace
spin_lock_irqsave.
CPU load will be reduced by 8%.
=20
Changes in V4:
	- fix an error in talitos when dest addr is same with src addr,
dest
	should be freed only one time if src is same with dest addr;
	- correct coding style in fsl-dma according to Ira's comments;
	- fix a race condition in fsl-dma fsl_tx_status(), remove the
interface
	which is used to free descriptors in queue ld_completed, this
interface
	has been included in fsldma_cleanup_descriptor(), in v3, there is
one
	place missed spin_lock protect;
	- split the original patch 3/4 up to 2 patches 3/7 and 4/7
according to
	Li Yang's comments.
	- fix a warning of unitialized cookie;
	- add memory copy self test in fsl-dma;
	- add more detail description about use spin_lock_bh() to instead
of
	spin_lock_irqsave() according to Timur's comments;
=20
Changes in v3:
	- change release process of fsl-dma descriptor for resolve the
	potential race condition
	- add test result when use spin_lock_bh replace spin_lock_irqsave
	- modify the benchmark results according to the latest patch
=20
Changes in v2:
	- rebase onto cryptodev tree
	- split the patch 3/4 up to 3 independent patches
	- remove the patch 4/4, the fix is not for cryptodev tree
=20
=20
Qiang Liu (4):
      Talitos: Support for async_tx XOR offload
      fsl-dma: remove attribute DMA_INTERRUPT of dmaengine
      fsl-dma: change release process of dma descriptor for supporting
async_tx
      fsl-dma: use spin_lock_bh to instead of spin_lock_irqsave
=20
Qiang Liu (7):
      Talitos: Support for async_tx XOR offload
      fsl-dma: remove attribute DMA_INTERRUPT of dmaengine
      fsl-dma: change release process of dma descriptor for supporting
async_tx
      fsl-dma: move the function ahead of its invoke function
      fsl-dma: use spin_lock_bh to instead of spin_lock_irqsave
      fsl-dma: fix a warning of unitialized cookie
      fsl-dma: add memcpy self test interface
=20
 drivers/crypto/Kconfig   |    9 +
 drivers/crypto/talitos.c |  413 ++++++++++++++++++++++++++++++++++
 drivers/crypto/talitos.h |   53 +++++
 drivers/dma/fsldma.c     |  550 +++++++++++++++++++++++++++++-----------
------
 drivers/dma/fsldma.h     |    1 +
 5 files changed, 822 insertions(+), 204 deletions(-)
=20
--
To unsubscribe from this list: send the line "unsubscribe linux-crypto"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help