[SPDK] Re: Unit test failure for review submission

Hi Wenhua,

Thanks for the report.  Can you file a github issue for this?  There's no review link required - it's easily reproducible by just running "./configure --enable-werror; make CC=clang CXX=clang++".  It's possible this has been fixed with more recent DPDK, or there is some flag we can pass to the DPDK build to disable this warning.

Regards,

-Jim




On 5/1/20, 5:45 AM, "Luse, Paul E" <paul.e.luse(a)intel.com> wrote:

    Hi Wenhua,

    Can you provide a link to the actual review?

    Thanks
    Paul

    -----Original Message-----
    From: Wenhua Liu [mailto:liuw(a)vmware.com] 
    Sent: Thursday, April 30, 2020 8:05 PM
    To: Storage Performance Development Kit <spdk(a)lists.01.org>
    Subject: [SPDK] Unit test failure for review submission

    Hi,

    I tried to submit a change for review, some unit test was run and failed, but I did not touch any of these files. I’m on top of the master branch.




    warning: redirecting to https://review.spdk.io/gerrit/spdk/spdk/



    Running make with clang ...

                                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

    /home/vmware/spdk/spdk/dpdk/build/include/rte_memcpy.h:359:23: error: cast from 'uint8_t *' (aka 'unsigned char *') to '__m256i *' increases required alignment from 1 to 32 [-Werror,-Wcast-align]

                    _mm256_storeu_si256((__m256i *)((uint8_t *)dst + 1 * 32), ymm1);

                                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

    /home/vmware/spdk/spdk/dpdk/build/include/rte_memcpy.h:360:23: error: cast from 'uint8_t *' (aka 'unsigned char *') to '__m256i *' increases required alignment from 1 to 32 [-Werror,-Wcast-align]

                    _mm256_storeu_si256((__m256i *)((uint8_t *)dst + 2 * 32), ymm2);

                                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

    /home/vmware/spdk/spdk/dpdk/build/include/rte_memcpy.h:361:23: error: cast from 'uint8_t *' (aka 'unsigned char *') to '__m256i *' increases required alignment from 1 to 32 [-Werror,-Wcast-align]

                    _mm256_storeu_si256((__m256i *)((uint8_t *)dst + 3 * 32), ymm3);

                                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

    12 errors generated.

    make[8]: *** [/home/vmware/spdk/spdk/dpdk/mk/internal/rte.compile-pre.mk:114: malloc_heap.o] Error 1

    make[7]: *** [/home/vmware/spdk/spdk/dpdk/mk/rte.subdir.mk:37: eal] Error 2

    make[6]: *** [/home/vmware/spdk/spdk/dpdk/mk/rte.subdir.mk:37: linux] Error 2

    make[5]: *** [/home/vmware/spdk/spdk/dpdk/mk/rte.subdir.mk:37: librte_eal] Error 2

    make[4]: *** [/home/vmware/spdk/spdk/dpdk/mk/rte.sdkbuild.mk:48: lib] Error 2

    make[3]: *** [/home/vmware/spdk/spdk/dpdk/mk/rte.sdkroot.mk:99: all] Error 2

    make[2]: *** [Makefile:12: all] Error 2

    make[1]: *** [Makefile:150: all] Error 2

    make: *** [/home/vmware/spdk/spdk/mk/spdk.subdirs.mk:44: dpdkbuild] Error 2



    ERROR make CC=clang CXX=clang++ returned errors!

    ERROR Fix the problem and use 'git commit' to update your changes.

    ERROR See /home/vmware/spdk/spdk/make.log for more information.



    error: failed to push some refs to 'https://review.spdk.io/spdk/spdk'



    Checking make.log, I did not see any test failed.



    What could be wrong?





    Thanks,

    -Wenhua Liu
    _______________________________________________
    SPDK mailing list -- spdk(a)lists.01.org
    To unsubscribe send an email to spdk-leave(a)lists.01.org
    _______________________________________________
    SPDK mailing list -- spdk(a)lists.01.org
    To unsubscribe send an email to spdk-leave(a)lists.01.org