[meta-intel] [PATCH 2/4 dizzy] meta-crystalforest: Intel QAT v1.5 recipe under recipes-extended

[Saul Wold]

Hi,

When I build this with dizzy I get a build failure that:

> | ../../common/cpa_sample_utils.c cpa_dc_stateless_multi_op_checksum_sample.c ../../../performance/qae/linux/user_space/qae_mem_utils.c ../stateless_sample/cpa_dc_sample_user.c /srv/ssd/builds/world/tmp/work/crystalforest-poky-linux/qat/2.2.0-30-r0/QAT1.5/build/libicp_qa_al_s.so -L/usr/Lib -lpthread -lcrypto -o dc_stateless_multi_op_sample
> | /usr/bin/ld: cannot find -lcrypto
> | collect2: error: ld returned 1 exit status
> | Makefile:111: recipe for target 'default' failed

This sysroot does contain a /lib/libcrypto.so, and you can see the 
-L/usr/lib, but no -L/lib, Did you do a compile check?


Also, I am confused with this version-ing of 1.5 and 1.6, but you set PV 
in those recipes to 2.2.0-30, which could cause problems with upgrade path.

Have you tested building qat 1.6 version?  I think it will have the same 
crypto problem.

I thing a better thing would be to name them qat15_2.2.0-30 and 
qat16_2.2.0-30, that way the package names are distinct. You can still 
control which QAT is installed in an image using a virtual/qat set up 
similar to the kernel.

Sau!

On 06/19/2015 12:15 AM, venkatasubramanian pattu wrote:
> This is an Initial version of Intel QuickAssist Technology(QAT)
> recipe support. This recipe is targeting on Intel QAT v1.5.
>
> Intel's QuickAssist Technology is designed to optimize the use
> and deployment of algorithm accelerators in networking and security applications.
>
> The package version of QAT1.5 is 1.7.0-30.
>
> "QAT1.5" is for use with Intel Communications Chipset 8900 to 8920(DH89xxCC)Series.
> Therefore, this recipe supports QAT on Cave Creek platforms.
> This recipe is created under meta-intel/meta-crystalforest.
>
> Signed-off-by: venkatasubramanian pattu <venkatasubramanianx.pattu at intel.com>
> ---
>   .../qat/qat/qat-1.5-enable-dynamic.patch           | 35 ++++++++++++++++++++++
>   meta-crystalforest/recipes-extended/qat/qat_1.5.bb | 16 ++++++++++
>   2 files changed, 51 insertions(+)
>   create mode 100644 meta-crystalforest/recipes-extended/qat/qat/qat-1.5-enable-dynamic.patch
>   create mode 100644 meta-crystalforest/recipes-extended/qat/qat_1.5.bb
>
> diff --git a/meta-crystalforest/recipes-extended/qat/qat/qat-1.5-enable-dynamic.patch b/meta-crystalforest/recipes-extended/qat/qat/qat-1.5-enable-dynamic.patch
> new file mode 100644
> index 0000000..6f5eb9f
> --- /dev/null
> +++ b/meta-crystalforest/recipes-extended/qat/qat/qat-1.5-enable-dynamic.patch
> @@ -0,0 +1,35 @@
> +From f428c0198d61087ba031985c571574a099096e1c Mon Sep 17 00:00:00 2001
> +From: venkatasubramanian pattu <venkatasubramanianx.pattu at intel.com>
> +Date: Wed, 17 Jun 2015 14:34:05 +0800
> +Subject: [PATCH] app: Enable the Dynamic Compression feature
> +
> +This Patch is used to enable the Dynamic Compression feature
> +
> +This will fix for qat applications failure.
> +ERROR : dcStatelessSample(): cpaDcQueryCapabilities
> +dcStatelessSample(): cpaDcGetSessionSize
> +[error] dcCheckSessionData() - : Invalid API Param - Invalid huffType value,
> +dynamic sessions not supported
> +
> +Upstream Status: Inappropriate
> +
> +Signed-off-by: venkatasubramanian pattu <venkatasubramanianx.pattu at intel.com>
> +---
> + .../lookaside/access_layer/src/common/compression/include/dc_session.h  | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/quickassist/lookaside/access_layer/src/common/compression/include/dc_session.h b/quickassist/lookaside/access_layer/src/common/compression/include/dc_session.h
> +index da69433..2b755dc 100644
> +--- a/quickassist/lookaside/access_layer/src/common/compression/include/dc_session.h
> ++++ b/quickassist/lookaside/access_layer/src/common/compression/include/dc_session.h
> +@@ -80,7 +80,7 @@
> + #include "sal_qat_cmn_msg.h"
> +
> + /* Defeature Dynamic Compression */
> +-#define ICP_DC_DYN_NOT_SUPPORTED         (1)
> ++/*#define ICP_DC_DYN_NOT_SUPPORTED         (1) */
> +
> + /* Maximum size of the state registers 64 bytes */
> + #define DC_QAT_STATE_REGISTERS_MAX_SIZE (64)
> +--
> +1.9.1
> diff --git a/meta-crystalforest/recipes-extended/qat/qat_1.5.bb b/meta-crystalforest/recipes-extended/qat/qat_1.5.bb
> new file mode 100644
> index 0000000..8b428d9
> --- /dev/null
> +++ b/meta-crystalforest/recipes-extended/qat/qat_1.5.bb
> @@ -0,0 +1,16 @@
> +include qat.inc
> +
> +PV = "2.2.0-30"
> +
> +export QAT_VERSION = "1.7.0-30"
> +export ICP_DH89XX = "QAT1.5"
> +export ICP_DRIVER_TYPE = "${ICP_DH89XX}"
> +export ICP_FIRMWARE_DIR="firmware"
> +
> +SRC_URI="https://01.org/sites/default/files/page/qatmux.l.${PV}.tar_.gz;name=qat \
> +	file://00-qat_qa.rules \
> +	file://qat-1.5-enable-dynamic.patch \
> +	"
> +
> +SRC_URI[qat.md5sum] = "9567adeba4a4af50f4ac3ab70a3091ed"
> +SRC_URI[qat.sha256sum] = "00ac245dc6226d6bcba2326d7b6fa045f91a616c67a003f299f404af84d52032"
>