Storage Performance Development Kit (SPDK)
 help / color / mirror / Atom feed
From: Harris, James R <james.r.harris at intel.com>
To: spdk@lists.01.org
Subject: [SPDK] Re: Unit test failure for review submission
Date: Fri, 01 May 2020 14:52:14 +0000	[thread overview]
Message-ID: <64AFF97A-B591-418E-B892-8AD63A481D90@intel.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 3761 bytes --]

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


             reply	other threads:[~2020-05-01 14:52 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-01 14:52 Harris, James R [this message]
  -- strict thread matches above, loose matches on Subject: below --
2020-05-01 17:33 [SPDK] Re: Unit test failure for review submission Wenhua Liu
2020-05-01 16:58 Wenhua Liu
2020-05-01 15:06 Harris, James R
2020-05-01 12:45 Luse, Paul E

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=64AFF97A-B591-418E-B892-8AD63A481D90@intel.com \
    --to=spdk@lists.01.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox