From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nikolay Aleksandrov Subject: Re: [PATCH v2 0/2] crypto: qat - Fix for invalid dma mapping and numa Date: Wed, 15 Oct 2014 12:35:43 +0200 Message-ID: <543E4DFF.9000101@redhat.com> References: <20141014012251.742.5500.stgit@tstruk-mobl1> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: prarit@redhat.com, bruce.w.allan@intel.com, qat-linux@intel.com, naleksan@redhat.com, linux-crypto@vger.kernel.org, davem@davemloft.net To: Tadeusz Struk , herbert@gondor.apana.org.au Return-path: Received: from mx1.redhat.com ([209.132.183.28]:10485 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751257AbaJOKfy (ORCPT ); Wed, 15 Oct 2014 06:35:54 -0400 In-Reply-To: <20141014012251.742.5500.stgit@tstruk-mobl1> Sender: linux-crypto-owner@vger.kernel.org List-ID: 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 (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