From: kernel test robot <lkp@intel.com>
To: Meng Yu <yumeng18@huawei.com>,
herbert@gondor.apana.org.au, davem@davemloft.net,
marcel@holtmann.org, johan.hedberg@gmail.com,
luiz.dentz@gmail.com, tudor.ambarus@microchip.com
Cc: kbuild-all@lists.01.org, linux-crypto@vger.kernel.org,
xuzaibo@huawei.com, wangzhou1@hisilicon.com, yumeng18@huawei.com
Subject: Re: [PATCH v8 5/9] crypto: move curve_id of ECDH to algorithm name
Date: Tue, 9 Feb 2021 11:48:30 +0800 [thread overview]
Message-ID: <202102091138.LGKujJ8m-lkp@intel.com> (raw)
In-Reply-To: <1612777137-51067-6-git-send-email-yumeng18@huawei.com>
[-- Attachment #1: Type: text/plain, Size: 5019 bytes --]
Hi Meng,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on cryptodev/master]
[also build test ERROR on crypto/master v5.11-rc6 next-20210125]
[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/0day-ci/linux/commits/Meng-Yu/add-ECDH-and-CURVE25519-algorithms-support-for-Kunpeng-930/20210209-073324
base: https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git master
config: sparc-randconfig-p002-20210209 (attached as .config)
compiler: sparc-linux-gcc (GCC) 9.3.0
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
# https://github.com/0day-ci/linux/commit/eeb0603ffdb2894db839c813cc66c4c08920d61b
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Meng-Yu/add-ECDH-and-CURVE25519-algorithms-support-for-Kunpeng-930/20210209-073324
git checkout eeb0603ffdb2894db839c813cc66c4c08920d61b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=sparc
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 >>):
In file included from crypto/testmgr.c:73:
>> crypto/testmgr.h:2304:2: error: expected '}' before ';' token
2304 | };
| ^
crypto/testmgr.h:2264:59: note: to match this '{'
2264 | static const struct kpp_testvec ecdh_p192_tv_template[] = {
| ^
crypto/testmgr.h:2264:33: warning: 'ecdh_p192_tv_template' defined but not used [-Wunused-const-variable=]
2264 | static const struct kpp_testvec ecdh_p192_tv_template[] = {
| ^~~~~~~~~~~~~~~~~~~~~
crypto/testmgr.h:1039:33: warning: 'curve25519_tv_template' defined but not used [-Wunused-const-variable=]
1039 | static const struct kpp_testvec curve25519_tv_template[] = {
| ^~~~~~~~~~~~~~~~~~~~~~
crypto/testmgr.h:818:33: warning: 'dh_tv_template' defined but not used [-Wunused-const-variable=]
818 | static const struct kpp_testvec dh_tv_template[] = {
| ^~~~~~~~~~~~~~
crypto/testmgr.h:726:38: warning: 'pkcs1pad_rsa_tv_template' defined but not used [-Wunused-const-variable=]
726 | static const struct akcipher_testvec pkcs1pad_rsa_tv_template[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~
crypto/testmgr.h:572:38: warning: 'ecrdsa_tv_template' defined but not used [-Wunused-const-variable=]
572 | static const struct akcipher_testvec ecrdsa_tv_template[] = {
| ^~~~~~~~~~~~~~~~~~
crypto/testmgr.h:182:38: warning: 'rsa_tv_template' defined but not used [-Wunused-const-variable=]
182 | static const struct akcipher_testvec rsa_tv_template[] = {
| ^~~~~~~~~~~~~~~
crypto/testmgr.h:177:19: warning: 'zeroed_string' defined but not used [-Wunused-const-variable=]
177 | static const char zeroed_string[48];
| ^~~~~~~~~~~~~
vim +2304 crypto/testmgr.h
2263
2264 static const struct kpp_testvec ecdh_p192_tv_template[] = {
2265 {
2266 #ifndef CONFIG_CRYPTO_FIPS
2267 .secret =
2268 #ifdef __LITTLE_ENDIAN
2269 "\x02\x00" /* type */
2270 "\x1e\x00" /* len */
2271 "\x18\x00" /* key_size */
2272 #else
2273 "\x00\x02" /* type */
2274 "\x00\x1e" /* len */
2275 "\x00\x18" /* key_size */
2276 #endif
2277 "\xb5\x05\xb1\x71\x1e\xbf\x8c\xda"
2278 "\x4e\x19\x1e\x62\x1f\x23\x23\x31"
2279 "\x36\x1e\xd3\x84\x2f\xcc\x21\x72",
2280 .b_public =
2281 "\xc3\xba\x67\x4b\x71\xec\xd0\x76"
2282 "\x7a\x99\x75\x64\x36\x13\x9a\x94"
2283 "\x5d\x8b\xdc\x60\x90\x91\xfd\x3f"
2284 "\xb0\x1f\x8a\x0a\x68\xc6\x88\x6e"
2285 "\x83\x87\xdd\x67\x09\xf8\x8d\x96"
2286 "\x07\xd6\xbd\x1c\xe6\x8d\x9d\x67",
2287 .expected_a_public =
2288 "\x1a\x04\xdb\xa5\xe1\xdd\x4e\x79"
2289 "\xa3\xe6\xef\x0e\x5c\x80\x49\x85"
2290 "\xfa\x78\xb4\xef\x49\xbd\x4c\x7c"
2291 "\x22\x90\x21\x02\xf9\x1b\x81\x5d"
2292 "\x0c\x8a\xa8\x98\xd6\x27\x69\x88"
2293 "\x5e\xbc\x94\xd8\x15\x9e\x21\xce",
2294 .expected_ss =
2295 "\xf4\x57\xcc\x4f\x1f\x4e\x31\xcc"
2296 "\xe3\x40\x60\xc8\x06\x93\xc6\x2e"
2297 "\x99\x80\x81\x28\xaf\xc5\x51\x74",
2298 .secret_size = 32,
2299 .b_public_size = 48,
2300 .expected_a_public_size = 48,
2301 .expected_ss_size = 24
2302 }
2303 #endif
> 2304 };
2305
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 24971 bytes --]
next prev parent reply other threads:[~2021-02-09 3:54 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-08 9:38 [PATCH v8 0/9] add ECDH and CURVE25519 algorithms support for Kunpeng 930 Meng Yu
2021-02-08 9:38 ` [PATCH v8 1/9] crypto: hisilicon/hpre - add version adapt to new algorithms Meng Yu
2021-02-08 9:38 ` [PATCH v8 2/9] crypto: hisilicon/hpre - add algorithm type Meng Yu
2021-02-08 9:38 ` [PATCH v8 3/9] crypto: atmel-ecc - move curve_id of ECDH from the key to algorithm name Meng Yu
2021-02-10 4:56 ` Herbert Xu
2021-02-18 2:14 ` yumeng
2021-02-08 9:38 ` [PATCH v8 4/9] net/bluetooth: modify ECDH name in 'crypto_alloc_kpp' Meng Yu
2021-02-08 9:38 ` [PATCH v8 5/9] crypto: move curve_id of ECDH to algorithm name Meng Yu
2021-02-09 3:48 ` kernel test robot [this message]
2021-02-09 5:33 ` kernel test robot
2021-02-08 9:38 ` [PATCH v8 6/9] crypto: add new ecc curve and expose them Meng Yu
2021-02-08 9:38 ` [PATCH v8 7/9] crypto: hisilicon/hpre - add 'ECDH' algorithm Meng Yu
2021-02-10 4:57 ` Herbert Xu
2021-02-18 2:24 ` yumeng
2021-02-18 20:01 ` Herbert Xu
2021-02-19 1:25 ` yumeng
2021-02-19 4:01 ` Herbert Xu
2021-02-08 9:38 ` [PATCH v8 8/9] crypto: add curve25519 params and expose them Meng Yu
2021-02-08 9:38 ` [PATCH v8 9/9] crypto: hisilicon/hpre - add 'CURVE25519' algorithm Meng Yu
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=202102091138.LGKujJ8m-lkp@intel.com \
--to=lkp@intel.com \
--cc=davem@davemloft.net \
--cc=herbert@gondor.apana.org.au \
--cc=johan.hedberg@gmail.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-crypto@vger.kernel.org \
--cc=luiz.dentz@gmail.com \
--cc=marcel@holtmann.org \
--cc=tudor.ambarus@microchip.com \
--cc=wangzhou1@hisilicon.com \
--cc=xuzaibo@huawei.com \
--cc=yumeng18@huawei.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).