From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [RESEND 07/19] crypto: ccree: convert tasklets to use new tasklet_setup() API
Date: Tue, 08 Dec 2020 02:23:26 +0800 [thread overview]
Message-ID: <202012080253.oXhEd5RV-lkp@intel.com> (raw)
In-Reply-To: <20201207085931.661267-8-allen.lkml@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 5498 bytes --]
Hi Allen,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on crypto/master]
[also build test ERROR on rockchip/for-next sparc-next/master v5.10-rc7 next-20201207]
[cannot apply to cryptodev/master]
[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/Allen-Pais/crypto-convert-tasklets-to-use-new-tasklet_setup-API/20201207-170424
base: https://git.kernel.org/pub/scm/linux/kernel/git/herbert/crypto-2.6.git master
config: powerpc-randconfig-m031-20201207 (attached as .config)
compiler: powerpc64le-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/ecb2ecb81c98d4b5b97ac311cc967a806adf0681
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Allen-Pais/crypto-convert-tasklets-to-use-new-tasklet_setup-API/20201207-170424
git checkout ecb2ecb81c98d4b5b97ac311cc967a806adf0681
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc
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 >>):
>> drivers/crypto/ccree/cc_fips.c:112:13: error: conflicting types for 'fips_dsr'
112 | static void fips_dsr(struct tasklet_struct *t)
| ^~~~~~~~
drivers/crypto/ccree/cc_fips.c:11:13: note: previous declaration of 'fips_dsr' was here
11 | static void fips_dsr(unsigned long devarg);
| ^~~~~~~~
In file included from <command-line>:
drivers/crypto/ccree/cc_fips.c: In function 'fips_dsr':
>> include/linux/kernel.h:1002:51: error: 'struct cc_drvdata' has no member named 'tasklet'
1002 | BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
| ^~
include/linux/compiler_types.h:299:9: note: in definition of macro '__compiletime_assert'
299 | if (!(condition)) \
| ^~~~~~~~~
include/linux/compiler_types.h:319:2: note: in expansion of macro '_compiletime_assert'
319 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ^~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert'
39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
| ^~~~~~~~~~~~~~~~~~
include/linux/kernel.h:1002:2: note: in expansion of macro 'BUILD_BUG_ON_MSG'
1002 | BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
| ^~~~~~~~~~~~~~~~
include/linux/kernel.h:1002:20: note: in expansion of macro '__same_type'
1002 | BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
| ^~~~~~~~~~~
include/linux/interrupt.h:637:2: note: in expansion of macro 'container_of'
637 | container_of(callback_tasklet, typeof(*var), tasklet_fieldname)
| ^~~~~~~~~~~~
drivers/crypto/ccree/cc_fips.c:114:31: note: in expansion of macro 'from_tasklet'
114 | struct cc_drvdata *drvdata = from_tasklet(drvdata, t, tasklet);
| ^~~~~~~~~~~~
>> include/linux/compiler_types.h:135:35: error: 'struct cc_drvdata' has no member named 'tasklet'
135 | #define __compiler_offsetof(a, b) __builtin_offsetof(a, b)
| ^~~~~~~~~~~~~~~~~~
include/linux/stddef.h:17:32: note: in expansion of macro '__compiler_offsetof'
17 | #define offsetof(TYPE, MEMBER) __compiler_offsetof(TYPE, MEMBER)
| ^~~~~~~~~~~~~~~~~~~
include/linux/kernel.h:1005:21: note: in expansion of macro 'offsetof'
1005 | ((type *)(__mptr - offsetof(type, member))); })
| ^~~~~~~~
include/linux/interrupt.h:637:2: note: in expansion of macro 'container_of'
637 | container_of(callback_tasklet, typeof(*var), tasklet_fieldname)
| ^~~~~~~~~~~~
drivers/crypto/ccree/cc_fips.c:114:31: note: in expansion of macro 'from_tasklet'
114 | struct cc_drvdata *drvdata = from_tasklet(drvdata, t, tasklet);
| ^~~~~~~~~~~~
vim +/fips_dsr +112 drivers/crypto/ccree/cc_fips.c
110
111 /* Deferred service handler, run as interrupt-fired tasklet */
> 112 static void fips_dsr(struct tasklet_struct *t)
113 {
114 struct cc_drvdata *drvdata = from_tasklet(drvdata, t, tasklet);
115 u32 irq, val;
116
117 irq = (drvdata->irq & (CC_GPR0_IRQ_MASK));
118
119 if (irq) {
120 cc_tee_handle_fips_error(drvdata);
121 }
122
123 /* after verifying that there is nothing to do,
124 * unmask AXI completion interrupt.
125 */
126 val = (CC_REG(HOST_IMR) & ~irq);
127 cc_iowrite(drvdata, CC_REG(HOST_IMR), val);
128 }
129
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 30422 bytes --]
next prev parent reply other threads:[~2020-12-07 18:23 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-07 8:59 [RESEND 00/19] crypto: convert tasklets to use new tasklet_setup API() Allen Pais
2020-12-07 8:59 ` [RESEND 01/19] crypto: amcc: convert tasklets to use new tasklet_setup() API Allen Pais
2020-12-07 8:59 ` [RESEND 02/19] crypto: atmel: " Allen Pais
2020-12-07 8:59 ` [RESEND 03/19] crypto: axis: " Allen Pais
2020-12-07 8:59 ` [RESEND 04/19] crypto: caam: " Allen Pais
2020-12-14 17:21 ` Horia Geantă
2020-12-07 8:59 ` [RESEND 05/19] crypto: cavium: " Allen Pais
2020-12-07 8:59 ` [RESEND 06/19] crypto: ccp: " Allen Pais
2020-12-07 8:59 ` [RESEND 07/19] crypto: ccree: " Allen Pais
2020-12-07 18:23 ` kernel test robot [this message]
2020-12-09 6:46 ` Gilad Ben-Yossef
2020-12-09 11:35 ` Allen
2020-12-07 8:59 ` [RESEND 08/19] crypto: hifn_795x: " Allen Pais
2020-12-07 8:59 ` [RESEND 09/19] crypto: img-hash: " Allen Pais
2020-12-07 8:59 ` [RESEND 10/19] crypto: ixp4xx: " Allen Pais
2020-12-07 8:59 ` [RESEND 11/19] crypto: mediatek: " Allen Pais
2020-12-07 8:59 ` [RESEND 12/19] crypto: omap: " Allen Pais
2020-12-07 8:59 ` [RESEND 13/19] crypto: picoxcell: " Allen Pais
2020-12-07 8:59 ` [RESEND 14/19] crypto: qat: " Allen Pais
2020-12-07 8:59 ` [RESEND 15/19] crypto: qce: " Allen Pais
2020-12-07 8:59 ` [RESEND 16/19] crypto: rockchip: " Allen Pais
2020-12-07 8:59 ` [RESEND 17/19] crypto: s5p: " Allen Pais
2020-12-07 9:50 ` Krzysztof Kozlowski
2020-12-07 8:59 ` [RESEND 18/19] crypto: talitos: " Allen Pais
2020-12-07 8:59 ` [RESEND 19/19] crypto: octeontx: " Allen Pais
2020-12-07 9:13 ` [RESEND 00/19] crypto: convert tasklets to use new tasklet_setup API() Krzysztof Kozlowski
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=202012080253.oXhEd5RV-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@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 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.