On 11/14/2014 12:49 PM, Alexandru Badicioiu wrote:
Hi,
for build errors - which compiler/toolchain do you use?

Alex, I see the same with:

COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/4.6/lto-wrapper
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu/Linaro 4.6.3-1ubuntu5' --with-bugurl=file:///usr/share/doc/gcc-4.6/README.Bugs --enable-languages=c,c++,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-4.6 --enable-shared --enable-linker-build-id --with-system-zlib --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.6 --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --enable-gnu-unique-object --enable-plugin --enable-objc-gc --disable-werror --with-arch-32=i686 --with-tune=generic --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 4.6.3 (Ubuntu/Linaro 4.6.3-1ubuntu5)

gcc -DHAVE_CONFIG_H -I. -pthread -DODP_DEBUG_PRINT=1 -DODP_DEBUG=1 -W -Wall -Werror -Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wold-style-definition -Wpointer-arith -Wcast-align -Wnested-externs -Wcast-qual -Wformat-nonliteral -Wformat-security -Wundef -Wwrite-strings -I. -I../../../platform/linux-generic/include/api -I../../../platform/linux-generic/include/api -I../../../helper/include -I/include -g -O2 -MT odp_crypto-odp_crypto_test_async_inp.o -MD -MP -MF .deps/odp_crypto-odp_crypto_test_async_inp.Tpo -c -o odp_crypto-odp_crypto_test_async_inp.o `test -f 'odp_crypto_test_async_inp.c' || echo './'`odp_crypto_test_async_inp.c odp_crypto_test_async_inp.c:363:2: error: initialization discards 'const' qualifier from pointer target type [-Werror] odp_crypto_test_async_inp.c:364:2: error: initialization discards 'const' qualifier from pointer target type [-Werror] odp_crypto_test_async_inp.c:365:2: error: initialization discards 'const' qualifier from pointer target type [-Werror] odp_crypto_test_async_inp.c:366:2: error: initialization discards 'const' qualifier from pointer target type [-Werror] odp_crypto_test_async_inp.c:367:2: error: initialization discards 'const' qualifier from pointer target type [-Werror] odp_crypto_test_async_inp.c:368:2: error: initialization discards 'const' qualifier from pointer target type [-Werror]
cc1: all warnings being treated as errors
make: *** [odp_crypto-odp_crypto_test_async_inp.o] Error 1

Maxim.


I did not modify in any way compilation flags and I have no problem with building with a PPC toolchain and also with the native gcc on my development machine? I use CUnit-2.1-3.

WARNING: braces {} are not necessary for any arm of this statement
#304: FILE: test/cunit/crypto/odp_crypto_test_async_inp.c:114:
+ if (compl_new == ODP_BUFFER_INVALID) {
[...]
+ } else {
[...]
If we remove the braces the test will not compile, it seems the CUnit macros are the problem.
I'll shorten the lines over 80 characters.

Alex


On 14 November 2014 10:23, Anders Roxell <[email protected] <mailto:[email protected]>> wrote:

    On 2014-11-13 16:14, [email protected]
    <mailto:[email protected]> wrote:
    > From: Alexandru Badicioiu <[email protected]
    <mailto:[email protected]>>
    >
    > This patch adds a suite for sync and async inplace mode of
    crypto APIs.
    > Correctness of crypto operation output is verified with known test
    > vectors. Various options and functionalities like use session IV
    > or operation IV for ciphering are exercised for both modes.
    > For async mode there are options to use input packet buffer or a
    > separate buffer as the completion event and to set and retrieve the
    > context associated with an operation from the completion event.
    >
    > Signed-off-by: Alexandru Badicioiu
    <[email protected]
    <mailto:[email protected]>>

    I haven't had time to look into this patch will send feedback later
    today.

    one thing is it doesn't build:

    Making all in crypto
    make[3]: Entering directory
    '/home/anders/src/check-odp/odp/test/cunit/crypto'
    CC odp_crypto-odp_crypto_test_async_inp.o
    odp_crypto_test_async_inp.c:364:2: error: initialization discards
    ‘const’ qualifier from pointer target type [-Werror]
    {ASYNC_INP_ENC_ALG_3DES_CBC, test_async_enc_alg_3des_cbc },
    ^
    odp_crypto_test_async_inp.c:365:2: error: initialization discards
    ‘const’ qualifier from pointer target type [-Werror]
    {ASYNC_INP_DEC_ALG_3DES_CBC, test_async_dec_alg_3des_cbc },
    ^
    odp_crypto_test_async_inp.c:366:2: error: initialization discards
    ‘const’ qualifier from pointer target type [-Werror]
    {ASYNC_INP_ENC_ALG_3DES_CBC_OVR_IV,
    test_async_enc_alg_3des_cbc_ovr_iv },
    ^
    odp_crypto_test_async_inp.c:367:2: error: initialization discards
    ‘const’ qualifier from pointer target type [-Werror]
    {ASYNC_INP_DEC_ALG_3DES_CBC_OVR_IV,
    test_async_dec_alg_3des_cbc_ovr_iv },
    ^
    odp_crypto_test_async_inp.c:368:2: error: initialization discards
    ‘const’ qualifier from pointer target type [-Werror]
    {ASYNC_INP_ALG_HMAC_MD5, test_async_alg_hmac_md5 },
    ^
    odp_crypto_test_async_inp.c:369:2: error: initialization discards
    ‘const’ qualifier from pointer target type [-Werror]
    {ASYNC_INP_ENC_ALG_3DES_CBC_COMPL_NEW,
    test_async_enc_alg_3des_cbc_compl_new },
    ^
    cc1: all warnings being treated as errors
    Makefile:456: recipe for target
    'odp_crypto-odp_crypto_test_async_inp.o' failed
    make[3]: *** [odp_crypto-odp_crypto_test_async_inp.o] Error 1
    make[3]: Leaving directory
    '/home/anders/src/check-odp/odp/test/cunit/crypto'
    Makefile:732: recipe for target 'all-recursive' failed
    make[2]: *** [all-recursive] Error 1
    Makefile:369: recipe for target 'all-recursive' failed
    make[1]: *** [all-recursive] Error 1
    Makefile:454: recipe for target 'all-recursive' failed
    make: *** [all-recursive] Error 1
    Making check in crypto

    Check patch compliains on this:
    WARNING: braces {} are not necessary for any arm of this statement
    #304: FILE: test/cunit/crypto/odp_crypto_test_async_inp.c:114:
    + if (compl_new == ODP_BUFFER_INVALID) {
    [...]
    + } else {
    [...]

    WARNING: line over 80 characters
    #556: FILE: test/cunit/crypto/odp_crypto_test_async_inp.c:366:
    + {ASYNC_INP_ENC_ALG_3DES_CBC_OVR_IV,
    test_async_enc_alg_3des_cbc_ovr_iv },

    WARNING: line over 80 characters
    #557: FILE: test/cunit/crypto/odp_crypto_test_async_inp.c:367:
    + {ASYNC_INP_DEC_ALG_3DES_CBC_OVR_IV,
    test_async_dec_alg_3des_cbc_ovr_iv },

    WARNING: line over 80 characters
    #559: FILE: test/cunit/crypto/odp_crypto_test_async_inp.c:369:
    + {ASYNC_INP_ENC_ALG_3DES_CBC_COMPL_NEW,
    test_async_enc_alg_3des_cbc_compl_new },


    Cheers,
    Anders




_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp


_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to