All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Nathan Huckleberry <nhuck@google.com>
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org
Subject: Re: [PATCH v5 7/8] crypto: arm64/polyval: Add PMULL accelerated implementation of POLYVAL
Date: Sat, 30 Apr 2022 09:38:14 +0800	[thread overview]
Message-ID: <202204300918.SdgoNFfx-lkp@intel.com> (raw)
In-Reply-To: <20220427003759.1115361-8-nhuck@google.com>

Hi Nathan,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on herbert-cryptodev-2.6/master]
[also build test ERROR on herbert-crypto-2.6/master linus/master v5.18-rc4 next-20220429]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/intel-lab-lkp/linux/commits/Nathan-Huckleberry/crypto-HCTR2-support/20220427-084044
base:   https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git master
config: arm64-allmodconfig (https://download.01.org/0day-ci/archive/20220430/202204300918.SdgoNFfx-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project c59473aacce38cd7dd77eebceaf3c98c5707ab3b)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm64 cross compiling tool for clang build
        # apt-get install binutils-aarch64-linux-gnu
        # https://github.com/intel-lab-lkp/linux/commit/3d3f8a04f4735ce4fc1a9b59cd6fa091e11a35c3
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Nathan-Huckleberry/crypto-HCTR2-support/20220427-084044
        git checkout 3d3f8a04f4735ce4fc1a9b59cd6fa091e11a35c3
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash arch/arm64/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

>> arch/arm64/crypto/polyval-ce-glue.c:188:1: error: redefinition of '__inittest'
   module_init(polyval_ce_mod_init);
   ^
   include/linux/module.h:131:42: note: expanded from macro 'module_init'
           static inline initcall_t __maybe_unused __inittest(void)                \
                                                   ^
   arch/arm64/crypto/polyval-ce-glue.c:186:1: note: previous definition is here
   module_cpu_feature_match(PMULL, polyval_ce_mod_init)
   ^
   include/linux/cpufeature.h:54:10: note: expanded from macro 'module_cpu_feature_match'
   }                                                               \
                                                                   ^
   include/linux/module.h:131:42: note: expanded from macro '\
   module_init'
           static inline initcall_t __maybe_unused __inittest(void)                \
                                                   ^
>> arch/arm64/crypto/polyval-ce-glue.c:188:1: error: redefinition of 'init_module'
   module_init(polyval_ce_mod_init);
   ^
   include/linux/module.h:133:6: note: expanded from macro 'module_init'
           int init_module(void) __copy(initfn)                    \
               ^
   arch/arm64/crypto/polyval-ce-glue.c:186:1: note: previous definition is here
   module_cpu_feature_match(PMULL, polyval_ce_mod_init)
   ^
   include/linux/cpufeature.h:54:10: note: expanded from macro 'module_cpu_feature_match'
   }                                                               \
                                                                   ^
   include/linux/module.h:133:6: note: expanded from macro '\
   module_init'
           int init_module(void) __copy(initfn)                    \
               ^
   2 errors generated.


vim +/__inittest +188 arch/arm64/crypto/polyval-ce-glue.c

   187	
 > 188	module_init(polyval_ce_mod_init);
   189	module_exit(polyval_ce_mod_exit);
   190	

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

  reply	other threads:[~2022-04-30  1:38 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-27  0:37 [PATCH v5 0/8] crypto: HCTR2 support Nathan Huckleberry
2022-04-27  0:37 ` Nathan Huckleberry
2022-04-27  0:37 ` [PATCH v5 1/8] crypto: xctr - Add XCTR support Nathan Huckleberry
2022-04-27  0:37   ` Nathan Huckleberry
2022-04-27  0:37 ` [PATCH v5 2/8] crypto: polyval - Add POLYVAL support Nathan Huckleberry
2022-04-27  0:37   ` Nathan Huckleberry
2022-04-27  0:37 ` [PATCH v5 3/8] crypto: hctr2 - Add HCTR2 support Nathan Huckleberry
2022-04-27  0:37   ` Nathan Huckleberry
2022-05-01 18:33   ` Eric Biggers
2022-05-01 18:33     ` Eric Biggers
2022-05-02 18:25   ` Eric Biggers
2022-05-02 18:25     ` Eric Biggers
2022-04-27  0:37 ` [PATCH v5 4/8] crypto: x86/aesni-xctr: Add accelerated implementation of XCTR Nathan Huckleberry
2022-04-27  0:37   ` Nathan Huckleberry
2022-04-27  4:26   ` kernel test robot
2022-05-01 21:31   ` Eric Biggers
2022-05-01 21:31     ` Eric Biggers
2022-04-27  0:37 ` [PATCH v5 5/8] crypto: arm64/aes-xctr: " Nathan Huckleberry
2022-04-27  0:37   ` Nathan Huckleberry
2022-05-01 22:08   ` Eric Biggers
2022-05-01 22:08     ` Eric Biggers
2022-04-27  0:37 ` [PATCH v5 6/8] crypto: x86/polyval: Add PCLMULQDQ accelerated implementation of POLYVAL Nathan Huckleberry
2022-04-27  0:37   ` Nathan Huckleberry
2022-05-01 20:43   ` Eric Biggers
2022-05-01 20:43     ` Eric Biggers
2022-05-02 18:08     ` Eric Biggers
2022-05-02 18:08       ` Eric Biggers
2022-04-27  0:37 ` [PATCH v5 7/8] crypto: arm64/polyval: Add PMULL " Nathan Huckleberry
2022-04-27  0:37   ` Nathan Huckleberry
2022-04-30  1:38   ` kernel test robot [this message]
2022-05-01 20:21   ` Eric Biggers
2022-05-01 20:21     ` Eric Biggers
2022-05-02 18:11     ` Eric Biggers
2022-05-02 18:11       ` Eric Biggers
2022-04-27  0:37 ` [PATCH v5 8/8] fscrypt: Add HCTR2 support for filename encryption Nathan Huckleberry
2022-04-27  0:37   ` Nathan Huckleberry
2022-05-01 18:37   ` Eric Biggers
2022-05-01 18:37     ` Eric Biggers

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=202204300918.SdgoNFfx-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    --cc=llvm@lists.linux.dev \
    --cc=nhuck@google.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.