Re: [EXT] [PATCH resend v3 0/6] crypto/uadk: introduce uadk crypto driver

[Zhangfei Gao <zhangfei.gao@linaro.org>]

Hi, Akhil

On 2022/10/11 下午5:22, Akhil Goyal wrote:
>>>> gakhil@cavium-SR1-T640:/home/gakhil/up/uadk$ make
>>>> make  all-recursive
>>>> make[1]: Entering directory '/home/gakhil/up/uadk'
>>>> Making all in .
>>>> make[2]: Entering directory '/home/gakhil/up/uadk'
>>>> depbase=`echo wd.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
>>>> /bin/bash ./libtool  --tag=CC   --mode=compile gcc -DHAVE_CONFIG_H -I.    -
>>> Wall -Werror -fno-strict-aliasing -I./include  -
>>> DUADK_VERSION_NUMBER="\"UADK version: 2.3.37\"" -
>>> DUADK_RELEASED_TIME="\"Released Jun 11, 2022\""  -g -O2 -MT wd.lo -MD -
>>> MP -MF $depbase.Tpo -c -o wd.lo wd.c &&\
>>>> mv -f $depbase.Tpo $depbase.Plo
>>>> libtool: compile:  gcc -DHAVE_CONFIG_H -I. -Wall -Werror -fno-strict-
>> aliasing -
>>> I./include "-DUADK_VERSION_NUMBER=\"UADK version: 2.3.37\"" "-
>>> DUADK_RELEASED_TIME=\"Released Jun 11, 2022\"" -g -O2 -MT wd.lo -MD -
>> MP
>>> -MF .deps/wd.Tpo -c wd.c  -fPIC -DPIC -o .libs/wd.o
>>>> depbase=`echo wd_mempool.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
>>>> /bin/bash ./libtool  --tag=CC   --mode=compile gcc -DHAVE_CONFIG_H -I.    -
>>> Wall -Werror -fno-strict-aliasing -I./include  -
>>> DUADK_VERSION_NUMBER="\"UADK version: 2.3.37\"" -
>>> DUADK_RELEASED_TIME="\"Released Jun 11, 2022\""  -g -O2 -MT
>>> wd_mempool.lo -MD -MP -MF $depbase.Tpo -c -o wd_mempool.lo
>>> wd_mempool.c &&\
>>>> mv -f $depbase.Tpo $depbase.Plo
>>>> libtool: compile:  gcc -DHAVE_CONFIG_H -I. -Wall -Werror -fno-strict-
>> aliasing -
>>> I./include "-DUADK_VERSION_NUMBER=\"UADK version: 2.3.37\"" "-
>>> DUADK_RELEASED_TIME=\"Released Jun 11, 2022\"" -g -O2 -MT
>>> wd_mempool.lo -MD -MP -MF .deps/wd_mempool.Tpo -c wd_mempool.c  -
>> fPIC
>>> -DPIC -o .libs/wd_mempool.o
>>>> depbase=`echo v1/wd.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
>>>> /bin/bash ./libtool  --tag=CC   --mode=compile gcc -DHAVE_CONFIG_H -I.    -
>>> Wall -Werror -fno-strict-aliasing -I./include  -
>>> DUADK_VERSION_NUMBER="\"UADK version: 2.3.37\"" -
>>> DUADK_RELEASED_TIME="\"Released Jun 11, 2022\""  -g -O2 -MT v1/wd.lo -
>> MD
>>> -MP -MF $depbase.Tpo -c -o v1/wd.lo v1/wd.c &&\
>>>> mv -f $depbase.Tpo $depbase.Plo
>>>> libtool: compile:  gcc -DHAVE_CONFIG_H -I. -Wall -Werror -fno-strict-
>> aliasing -
>>> I./include "-DUADK_VERSION_NUMBER=\"UADK version: 2.3.37\"" "-
>>> DUADK_RELEASED_TIME=\"Released Jun 11, 2022\"" -g -O2 -MT v1/wd.lo -
>> MD -
>>> MP -MF v1/.deps/wd.Tpo -c v1/wd.c  -fPIC -DPIC -o v1/.libs/wd.o
>>>> In file included from v1/wd.c:30:
>>>> ./v1/wd_util.h:354:2: error: #error "no platform mb, define one before
>>> compiling"
>>>>     354 | #error "no platform mb, define one before compiling"
>>>>         |  ^~~~~
>>>> make[2]: *** [Makefile:788: v1/wd.lo] Error 1
>>>> make[2]: Leaving directory '/home/gakhil/up/uadk'
>>>> make[1]: *** [Makefile:859: all-recursive] Error 1
>>>> make[1]: Leaving directory '/home/gakhil/up/uadk'
>>>> make: *** [Makefile:565: all] Error 2
>>> By the way, what platform are you using.
>>> I can not reproduce this error.
>>>
>> I am compiling on Intel XEON server with ubuntu 18.04
> Any update??
The uadk lib needs cross-compile on x86.
Toolchain: 
https://releases.linaro.org/components/toolchain/binaries/latest-7/aarch64-linux-gnu/
Details are documented in https://github.com/Linaro/uadk/blob/master/INSTALL

Considering the confusion caused by simple build info in README,
We are planning to remove the simple build step in README, only keeps 
INSTALL.
https://github.com/Linaro/uadk/pull/510

Since there are some dependent libraries (libz & openssl) causes 
cross-compile complicated.
We are planning to omit build tests if not found these libs
https://github.com/Linaro/uadk/pull/508

Only libnuma is must required for lib.
Fortunately the numa cross compile is simple

https://github.com/Linaro/uadk/blob/master/INSTALL
     $git clone https://github.com/numactl/numactl.git
     $cd numactl
     $./autogen.sh
     $autoconf -i
     $CC=aarch64-linux-gnu-gcc ./configure --host=i386-linux-gnu 
--target=aarch64-linux-gnu --prefix=${LIBROOT}/usr/local
     $make
     $make install

Do you think we still need paste uadk build steps (and cross-compile steps)
on dpdk doc/guides/cryptodevs/uadk.rst

Sorry for the confusion.

Thanks