From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] qat: change optimization flag for Intel QuickAssist Technology Date: Fri, 13 May 2016 10:17:43 +0200 Message-ID: <1802357.As1ss59lSu@xps13> References: <1462872255-13100-1-git-send-email-arkadiuszx.kusztal@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, fiona.trahe@intel.com, john.griffin@intel.com, deepak.k.jain@intel.com To: Arek Kusztal Return-path: Received: from mail-wm0-f51.google.com (mail-wm0-f51.google.com [74.125.82.51]) by dpdk.org (Postfix) with ESMTP id 2406E5AA5 for ; Fri, 13 May 2016 10:17:46 +0200 (CEST) Received: by mail-wm0-f51.google.com with SMTP id n129so12468419wmn.1 for ; Fri, 13 May 2016 01:17:46 -0700 (PDT) In-Reply-To: <1462872255-13100-1-git-send-email-arkadiuszx.kusztal@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2016-05-10 10:24, Arek Kusztal: > From: Arkadiusz Kusztal > > Changed to -O3 optimization flag in Intel QuickAssist Technology Makefile There is another change below. Should it be a separate patch? > --- a/drivers/crypto/qat/qat_adf/qat_algs_build_desc.c > +++ b/drivers/crypto/qat/qat_adf/qat_algs_build_desc.c > @@ -616,10 +616,12 @@ int qat_alg_aead_session_create_content_desc_auth(struct qat_session *cdesc, > * Write (the length of AAD) into bytes 16-19 of state2 > * in big-endian format. This field is 8 bytes > */ > - *(uint32_t *)&(hash->sha.state1[ > + uint32_t *aad_len = (uint32_t *)&hash->sha.state1[ > ICP_QAT_HW_GALOIS_128_STATE1_SZ + > - ICP_QAT_HW_GALOIS_H_SZ]) = > - rte_bswap32(add_auth_data_length); > + ICP_QAT_HW_GALOIS_H_SZ]; > + > + *aad_len = rte_bswap32(add_auth_data_length); > + > proto = ICP_QAT_FW_LA_GCM_PROTO; > } else if (cdesc->qat_hash_alg == ICP_QAT_HW_AUTH_ALGO_SNOW_3G_UIA2) { > proto = ICP_QAT_FW_LA_SNOW_3G_PROTO; >