Re: [PATCH v4 2/2] misc: Add Nitro Secure Module driver

kernel test robot
2023-10-13T15:43:56Z

Hi Alexander,

kernel test robot noticed the following build errors:

[auto build test ERROR on char-misc/char-misc-testing]
[also build test ERROR on char-misc/char-misc-next char-misc/char-misc-linus soc/for-next linus/master v6.6-rc5 next-20231012]
[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#_base_tree_information]

url: https://github.com/intel-lab-lkp/linux/commits/Alexander-Graf/Import-CBOR-library/20231010-052529
base: char-misc/char-misc-testing
patch link: https://lore.kernel.org/r/20231009212053.2007-3-graf%40amazon.com
patch subject: [PATCH v4 2/2] misc: Add Nitro Secure Module driver
config: powerpc-allmodconfig (https://download.01.org/0day-ci/archive/20231013/202310132329.UIMS3J8a-lkp@intel.com/config)
compiler: powerpc64-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231013/202310132329.UIMS3J8a-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202310132329.UIMS3J8a-lkp@intel.com/

All errors (new ones prefixed by >>):

In file included from include/asm-generic/ioctl.h:5,
from arch/powerpc/include/uapi/asm/ioctl.h:12,
from include/uapi/linux/ioctl.h:5,
from include/uapi/linux/random.h:12,
from include/linux/random.h:10,
from include/linux/nodemask.h:97,
from include/linux/list_lru.h:12,
from include/linux/fs.h:13,
from drivers/misc/nsm.c:23:
>> include/uapi/asm-generic/ioctl.h:70:9: error: initializer element is not constant
70 | (((dir) << _IOC_DIRSHIFT) | \
| ^
include/uapi/asm-generic/ioctl.h:88:33: note: in expansion of macro '_IOC'
88 | #define _IOWR(type,nr,size) _IOC(_IOC_READ|_IOC_WRITE,(type),(nr),(_IOC_TYPECHECK(size)))
| ^~~~
include/uapi/linux/nsm.h:172:33: note: in expansion of macro '_IOWR'
172 | #define NSM_IOCTL_ATTESTATION _IOWR(NSM_MAGIC, 0x6, union nsm_attestation)
| ^~~~~
drivers/misc/nsm.c:1209:11: note: in expansion of macro 'NSM_IOCTL_ATTESTATION'
1209 | { NSM_IOCTL_ATTESTATION, _fill_req_attestation, _parse_resp_attestation },
| ^~~~~~~~~~~~~~~~~~~~~
include/uapi/asm-generic/ioctl.h:70:9: note: (near initialization for 'nsm_ioctls[6].cmd')
70 | (((dir) << _IOC_DIRSHIFT) | \
| ^
include/uapi/asm-generic/ioctl.h:88:33: note: in expansion of macro '_IOC'
88 | #define _IOWR(type,nr,size) _IOC(_IOC_READ|_IOC_WRITE,(type),(nr),(_IOC_TYPECHECK(size)))
| ^~~~
include/uapi/linux/nsm.h:172:33: note: in expansion of macro '_IOWR'
172 | #define NSM_IOCTL_ATTESTATION _IOWR(NSM_MAGIC, 0x6, union nsm_attestation)
| ^~~~~
drivers/misc/nsm.c:1209:11: note: in expansion of macro 'NSM_IOCTL_ATTESTATION'
1209 | { NSM_IOCTL_ATTESTATION, _fill_req_attestation, _parse_resp_attestation },
| ^~~~~~~~~~~~~~~~~~~~~
>> include/uapi/asm-generic/ioctl.h:70:9: error: initializer element is not constant
70 | (((dir) << _IOC_DIRSHIFT) | \
| ^
include/uapi/asm-generic/ioctl.h:88:33: note: in expansion of macro '_IOC'
88 | #define _IOWR(type,nr,size) _IOC(_IOC_READ|_IOC_WRITE,(type),(nr),(_IOC_TYPECHECK(size)))
| ^~~~
include/uapi/linux/nsm.h:186:33: note: in expansion of macro '_IOWR'
186 | #define NSM_IOCTL_RANDOM _IOWR(NSM_MAGIC, 0x7, union nsm_get_random)
| ^~~~~
drivers/misc/nsm.c:1210:11: note: in expansion of macro 'NSM_IOCTL_RANDOM'
1210 | { NSM_IOCTL_RANDOM, _fill_req_get_random, _parse_resp_get_random },
| ^~~~~~~~~~~~~~~~
include/uapi/asm-generic/ioctl.h:70:9: note: (near initialization for 'nsm_ioctls[7].cmd')
70 | (((dir) << _IOC_DIRSHIFT) | \
| ^
include/uapi/asm-generic/ioctl.h:88:33: note: in expansion of macro '_IOC'
88 | #define _IOWR(type,nr,size) _IOC(_IOC_READ|_IOC_WRITE,(type),(nr),(_IOC_TYPECHECK(size)))
| ^~~~
include/uapi/linux/nsm.h:186:33: note: in expansion of macro '_IOWR'
186 | #define NSM_IOCTL_RANDOM _IOWR(NSM_MAGIC, 0x7, union nsm_get_random)
| ^~~~~
drivers/misc/nsm.c:1210:11: note: in expansion of macro 'NSM_IOCTL_RANDOM'
1210 | { NSM_IOCTL_RANDOM, _fill_req_get_random, _parse_resp_get_random },
| ^~~~~~~~~~~~~~~~

vim +70 include/uapi/asm-generic/ioctl.h

8a1ab3155c2ac7 David Howells 2012-10-04 68
8a1ab3155c2ac7 David Howells 2012-10-04 69 #define _IOC(dir,type,nr,size) \
8a1ab3155c2ac7 David Howells 2012-10-04 @70 (((dir) << _IOC_DIRSHIFT) | \
8a1ab3155c2ac7 David Howells 2012-10-04 71 ((type) << _IOC_TYPESHIFT) | \
8a1ab3155c2ac7 David Howells 2012-10-04 72 ((nr) << _IOC_NRSHIFT) | \
8a1ab3155c2ac7 David Howells 2012-10-04 73 ((size) << _IOC_SIZESHIFT))
8a1ab3155c2ac7 David Howells 2012-10-04 74

--
0-DAY CI Kernel Test Service