On Fri, Sep 23, 2016 at 02:32:57PM +0800, Yuanhan Liu wrote: > On Thu, Sep 22, 2016 at 04:54:57PM +0100, Arek Kusztal wrote: > > This patch adds AES Galois Counter Mode test case for cryptodev > > QAT tests. Test is performed with different buffer sizes, burst > > size of 32 and 128b key. Test vectors are placed in > > app/test/test_cryptodev_perf_vectors.h file. > > > > This patch depends on the following patches/patchsets: > > > > "app/test: fix linkage scope of gmac plaintext array" > > (http://dpdk.org/dev/patchwork/patch/16023/) > > > > Signed-off-by: Arek Kusztal <arkadiuszx.kusztal at intel.com> > > FYI, > > My robot caught a build error when this patch applied. > > --yliu > > --- > test_cryptodev.o:(.data+0x1be0): multiple definition of `gmac_plaintext' > test_cryptodev_perf.o:(.data+0x4a0): first defined here > collect2: error: ld returned 1 exit status
One more: it breaks 32 bit build. --- /yeti/vm/ubuntu-initrd-16.04-i386-build/dpdk/app/test/test_cryptodev_perf.c: In function 'perf_AES_GCM': /yeti/vm/ubuntu-initrd-16.04-i386-build/dpdk/app/test/test_cryptodev_perf.c:3187:10: error: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type 'uint64_t {aka long long unsigned int}' [-Werror=format=] printf("\n%u\t\t%6.2f\t%16.2f\t%8lu\t%10lu", ^ /yeti/vm/ubuntu-initrd-16.04-i386-build/dpdk/app/test/test_cryptodev_perf.c:3187:10: error: format '%lu' expects argument of type 'long unsigned int', but argument 6 has type 'uint64_t {aka long long unsigned int}' [-Werror=format=] cc1: all warnings being treated as errors