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

[Prarit Bhargava <prarit@redhat.com>]

On 10/15/2014 06:35 AM, Nikolay Aleksandrov wrote:
> On 14/10/14 03:24, Tadeusz Struk wrote:
>> 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 <tadeusz.struk@intel.com>
>> ---
>>
>> 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 <nikolay@redhat.com>

Thanks Nik :)

Reviewed-by: Prarit Bhargava <prarit@redhat.com>

P.