Re: [PATCH v2 0/2] crypto: qat - Fix for invalid dma mapping and numa

On Wed, Oct 15, 2014 at 07:25:45AM -0400, Prarit Bhargava wrote:

On 10/15/2014 06:35 AM, Nikolay Aleksandrov wrote:

quoted

On 14/10/14 03:24, Tadeusz Struk wrote:

quoted

Hi,
These two patches fix invalid (zero length) dma mapping and
enforce numa configuration for maximum performance.

Change log:
v2 - Removed numa node calculation based bus number and use predefined
functions instead.

Signed-off-by: Tadeusz Struk <redacted>
---

Tadeusz Struk (2):
       crypto: qat - Prevent dma mapping zero length assoc data
       crypto: qat - Enforce valid numa configuration


  drivers/crypto/qat/qat_common/adf_accel_devices.h |    3 +-
  drivers/crypto/qat/qat_common/adf_transport.c     |   12 +++++---
  drivers/crypto/qat/qat_common/qat_algs.c          |    7 +++--
  drivers/crypto/qat/qat_common/qat_crypto.c        |    8 +++--
  drivers/crypto/qat/qat_dh895xcc/adf_admin.c       |    2 +
  drivers/crypto/qat/qat_dh895xcc/adf_drv.c         |   32 ++++++++-------------
  drivers/crypto/qat/qat_dh895xcc/adf_isr.c         |    2 +
  7 files changed, 32 insertions(+), 34 deletions(-)

I just gave a quick run of these patches and they seem to fix the NUMA issue and
the 0 length warnings.

Tested-by: Nikolay Aleksandrov <redacted>

Thanks Nik :)

Reviewed-by: Prarit Bhargava <redacted>

Patch applied to crypto and I will push this to stable.

Thanks everyone!
-- 
Email: Herbert Xu [off-list ref]
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt