linux-kernel - Re: [PATCH 3/3] crypto: Added Chelsio Menu to the Kconfig file

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 12 Jul 2016 08:14:19 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Yeshaswi M R Gowda <yeshaswi@...lsio.com>
Cc:	kbuild-all@...org, hariprasad@...lsio.com, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org, herbert@...dor.apana.org.au,
	davem@...emloft.net, linux-crypto@...r.kernel.org,
	jlulla@...lsio.com, atul.gupta@...lsio.com, harsh@...lsio.com,
	Yeshaswi M R Gowda <yeshaswi@...lsio.com>
Subject: Re: [PATCH 3/3] crypto: Added Chelsio Menu to the Kconfig file

Hi,

[auto build test WARNING on net-next/master]
[also build test WARNING on v4.7-rc7 next-20160711]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Yeshaswi-M-R-Gowda/crypto-chcr-Add-Chelsio-Crypto-Driver/20160712-023513
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/crypto/chelsio/chcr_core.c: In function 'cpl_fw6_pld_handler':
>> drivers/crypto/chelsio/chcr_core.c:134:8: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
     req = (struct crypto_async_request *)cookie;
           ^
--
   In file included from include/linux/swab.h:4:0,
                    from include/uapi/linux/byteorder/little_endian.h:12,
                    from include/linux/byteorder/little_endian.h:4,
                    from arch/x86/include/uapi/asm/byteorder.h:4,
                    from include/asm-generic/bitops/le.h:5,
                    from arch/x86/include/asm/bitops.h:504,
                    from include/linux/bitops.h:36,
                    from include/linux/kernel.h:10,
                    from drivers/crypto/chelsio/chcr_algo.c:42:
   drivers/crypto/chelsio/chcr_algo.c: In function 'create_wreq':
>> drivers/crypto/chelsio/chcr_algo.c:454:29: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
     wreq->cookie = cpu_to_be64((u64)req);
                                ^
   include/uapi/linux/swab.h:126:54: note: in definition of macro '__swab64'
    #define __swab64(x) (__u64)__builtin_bswap64((__u64)(x))
                                                         ^
>> include/linux/byteorder/generic.h:91:21: note: in expansion of macro '__cpu_to_be64'
    #define cpu_to_be64 __cpu_to_be64
                        ^~~~~~~~~~~~~
>> drivers/crypto/chelsio/chcr_algo.c:454:17: note: in expansion of macro 'cpu_to_be64'
     wreq->cookie = cpu_to_be64((u64)req);
                    ^~~~~~~~~~~
   drivers/crypto/chelsio/chcr_algo.c: In function 'chcr_register_alg':
>> drivers/crypto/chelsio/chcr_algo.c:1471:48: warning: operation on 'driver_algs[i].alg.hash.halg.base.cra_init' may be undefined [-Wsequence-point]
        driver_algs[i].alg.hash.halg.base.cra_init =
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
        driver_algs[i].alg.hash.halg.base.cra_init =
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
         chcr_hmac_cra_init;
         ~~~~~~~~~~~~~~~~~~                         

vim +134 drivers/crypto/chelsio/chcr_core.c

5c923415 Yeshaswi M R Gowda 2016-07-11  118  	u_ctx->dev = NULL;
5c923415 Yeshaswi M R Gowda 2016-07-11  119  	atomic_dec(&dev_count);
5c923415 Yeshaswi M R Gowda 2016-07-11  120  	return 0;
5c923415 Yeshaswi M R Gowda 2016-07-11  121  }
5c923415 Yeshaswi M R Gowda 2016-07-11  122  
5c923415 Yeshaswi M R Gowda 2016-07-11  123  static int cpl_fw6_pld_handler(struct chcr_dev *dev,
5c923415 Yeshaswi M R Gowda 2016-07-11  124  			       unsigned char *input)
5c923415 Yeshaswi M R Gowda 2016-07-11  125  {
5c923415 Yeshaswi M R Gowda 2016-07-11  126  	struct crypto_async_request *req;
5c923415 Yeshaswi M R Gowda 2016-07-11  127  	struct cpl_fw6_pld *fw6_pld;
5c923415 Yeshaswi M R Gowda 2016-07-11  128  	u64 cookie;
5c923415 Yeshaswi M R Gowda 2016-07-11  129  	u32 ack_err_status = 0;
5c923415 Yeshaswi M R Gowda 2016-07-11  130  	int error_status = 0;
5c923415 Yeshaswi M R Gowda 2016-07-11  131  
5c923415 Yeshaswi M R Gowda 2016-07-11  132  	fw6_pld = (struct cpl_fw6_pld *)input;
5c923415 Yeshaswi M R Gowda 2016-07-11  133  	cookie = be64_to_cpu(fw6_pld->data[1]);
5c923415 Yeshaswi M R Gowda 2016-07-11 @134  	req = (struct crypto_async_request *)cookie;
5c923415 Yeshaswi M R Gowda 2016-07-11  135  
5c923415 Yeshaswi M R Gowda 2016-07-11  136  	ack_err_status =
5c923415 Yeshaswi M R Gowda 2016-07-11  137  		ntohl(*(__be32 *)((unsigned char *)&fw6_pld->data[0] + 4));
5c923415 Yeshaswi M R Gowda 2016-07-11  138  	if (ack_err_status) {
5c923415 Yeshaswi M R Gowda 2016-07-11  139  		if (CHK_MAC_ERR_BIT(ack_err_status) ||
5c923415 Yeshaswi M R Gowda 2016-07-11  140  		    CHK_PAD_ERR_BIT(ack_err_status))
5c923415 Yeshaswi M R Gowda 2016-07-11  141  			error_status = -EINVAL;
5c923415 Yeshaswi M R Gowda 2016-07-11  142  	}

:::::: The code at line 134 was first introduced by commit
:::::: 5c9234157776103907606c9f4c93a311467e246f chcr: Support for Chelsio's Crypto Hardware

:::::: TO: Yeshaswi M R Gowda <yeshaswi@...lsio.com>
:::::: CC: 0day robot <fengguang.wu@...el.com>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/octet-stream" (55097 bytes)