Re: [PATCH 1/2] ceph: Use generic debugging facility
by kernel test robot
Hi Joe,
I love your patch! Yet something to improve:
[auto build test ERROR on ceph-client/for-linus]
[also build test ERROR on v5.9-rc1 next-20200814]
[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/Joe-Perches/ceph-Neaten-debuggin...
base: https://github.com/ceph/ceph-client.git for-linus
config: x86_64-randconfig-a012-20200817 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project ab9fc8bae805c785066779e76e7846aabad5609e)
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 x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
>> net/ceph/ceph_common.c:227:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("parse_fsid '%s'\n", str);
^
net/ceph/ceph_common.c:227:2: note: did you mean 'dput'?
include/linux/dcache.h:374:13: note: 'dput' declared here
extern void dput(struct dentry *);
^
net/ceph/ceph_common.c:342:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("destroy_options %p\n", opt);
^
net/ceph/ceph_common.c:430:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s fs_parse '%s' token %d\n", __func__, param->key, token);
^
net/ceph/ceph_common.c:726:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("destroy_client %p\n", client);
^
net/ceph/ceph_common.c:779:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("mount waiting for mon_map\n");
^
net/ceph/ceph_common.c:802:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("open_session start\n");
^
net/ceph/ceph_common.c:864:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("exit_ceph_lib\n");
^
7 errors generated.
--
net/ceph/messenger.c:204:5: warning: format specifies type 'unsigned short' but the argument has type 'int' [-Wformat]
ntohs(in4->sin_port));
^~~~~~~~~~~~~~~~~~~~
include/linux/byteorder/generic.h:142:18: note: expanded from macro 'ntohs'
#define ntohs(x) ___ntohs(x)
^~~~~~~~~~~
include/linux/byteorder/generic.h:137:21: note: expanded from macro '___ntohs'
#define ___ntohs(x) __be16_to_cpu(x)
^~~~~~~~~~~~~~~~
include/uapi/linux/byteorder/little_endian.h:42:26: note: expanded from macro '__be16_to_cpu'
#define __be16_to_cpu(x) __swab16((__force __u16)(__be16)(x))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/uapi/linux/swab.h:105:2: note: expanded from macro '__swab16'
(__builtin_constant_p((__u16)(x)) ? \
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
net/ceph/messenger.c:210:5: warning: format specifies type 'unsigned short' but the argument has type 'int' [-Wformat]
ntohs(in6->sin6_port));
^~~~~~~~~~~~~~~~~~~~~
include/linux/byteorder/generic.h:142:18: note: expanded from macro 'ntohs'
#define ntohs(x) ___ntohs(x)
^~~~~~~~~~~
include/linux/byteorder/generic.h:137:21: note: expanded from macro '___ntohs'
#define ___ntohs(x) __be16_to_cpu(x)
^~~~~~~~~~~~~~~~
include/uapi/linux/byteorder/little_endian.h:42:26: note: expanded from macro '__be16_to_cpu'
#define __be16_to_cpu(x) __swab16((__force __u16)(__be16)(x))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/uapi/linux/swab.h:105:2: note: expanded from macro '__swab16'
(__builtin_constant_p((__u16)(x)) ? \
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> net/ceph/messenger.c:309:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s con %p sock %d -> %d\n", __func__, con, old_state,
^
net/ceph/messenger.c:309:2: note: did you mean 'dput'?
include/linux/dcache.h:374:13: note: 'dput' declared here
extern void dput(struct dentry *);
^
net/ceph/messenger.c:320:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s con %p sock %d -> %d\n", __func__, con, old_state,
^
net/ceph/messenger.c:331:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s con %p sock %d -> %d\n", __func__, con, old_state,
^
net/ceph/messenger.c:344:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s con %p sock %d -> %d\n", __func__, con, old_state,
^
net/ceph/messenger.c:358:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s con %p sock %d -> %d\n", __func__, con, old_state,
^
net/ceph/messenger.c:375:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s on %p state = %lu, queueing work\n", __func__,
^
net/ceph/messenger.c:395:4: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s %p queueing write work\n", __func__, con);
^
net/ceph/messenger.c:400:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s %p nothing to write\n", __func__, con);
^
net/ceph/messenger.c:409:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s %p state = %lu sk_state = %u\n", __func__,
^
net/ceph/messenger.c:477:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("connect %s\n", ceph_pr_addr(&con->peer_addr));
^
net/ceph/messenger.c:597:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("con_close_socket on %p sock %p\n", con, con->sock);
^
net/ceph/messenger.c:639:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("reset_connection %p\n", con);
^
net/ceph/messenger.c:668:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("con_close %p peer %s\n", con, ceph_pr_addr(&con->peer_addr));
^
net/ceph/messenger.c:692:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("con_open %p %s\n", con, ceph_pr_addr(addr));
^
net/ceph/messenger.c:722:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("con_init %p\n", con);
^
net/ceph/messenger.c:1237:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("prepare_write_message_footer %p\n", con);
^
net/ceph/messenger.c:1293:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("prepare_write_message %p seq %lld type %d len %d+%d+%zd\n",
^
net/ceph/messenger.c:1346:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("prepare_write_ack %p %llu -> %llu\n", con,
^
net/ceph/messenger.c:1367:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("prepare_write_seq %p %llu -> %llu\n", con,
^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
2 warnings and 20 errors generated.
--
>> net/ceph/msgpool.c:20:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("msgpool_alloc %s failed\n", pool->name);
^
net/ceph/msgpool.c:20:3: note: did you mean 'dput'?
include/linux/dcache.h:374:13: note: 'dput' declared here
extern void dput(struct dentry *);
^
net/ceph/msgpool.c:22:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("msgpool_alloc %s %p\n", pool->name, msg);
^
net/ceph/msgpool.c:33:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("msgpool_release %s %p\n", pool->name, msg);
^
net/ceph/msgpool.c:42:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("msgpool %s init\n", name);
^
net/ceph/msgpool.c:55:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("msgpool %s destroy\n", pool->name);
^
net/ceph/msgpool.c:77:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("msgpool_get %s %p\n", pool->name, msg);
^
net/ceph/msgpool.c:83:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("msgpool_put %s %p\n", pool->name, msg);
^
7 errors generated.
--
>> net/ceph/buffer.c:29:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("buffer_new %p\n", b);
^
net/ceph/buffer.c:29:2: note: did you mean 'dput'?
include/linux/dcache.h:374:13: note: 'dput' declared here
extern void dput(struct dentry *);
^
net/ceph/buffer.c:38:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("buffer_release %p\n", b);
^
net/ceph/buffer.c:50:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("decode_buffer len %d\n", (int)len);
^
3 errors generated.
--
>> net/ceph/mon_client.c:53:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("monmap_decode %p %p len %d (%d)\n", p, end, len, (int)(end-p));
^
net/ceph/mon_client.c:53:2: note: did you mean 'dput'?
include/linux/dcache.h:374:13: note: 'dput' declared here
extern void dput(struct dentry *);
^
net/ceph/mon_client.c:123:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("__close_session closing mon%d\n", monc->cur_mon);
^
net/ceph/mon_client.c:165:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s mon%d -> mon%d out of %d mons\n", __func__, old_mon,
^
net/ceph/mon_client.c:188:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s opening mon%d\n", __func__, monc->cur_mon);
^
net/ceph/mon_client.c:228:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s hunt_mult now %d\n", __func__, monc->hunt_mult);
^
net/ceph/mon_client.c:243:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("__schedule_delayed after %lu\n", delay);
^
net/ceph/mon_client.c:267:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s sent %lu\n", __func__, monc->sub_renew_sent);
^
net/ceph/mon_client.c:327:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s sent %lu duration %d renew after %lu\n", __func__,
^
net/ceph/mon_client.c:331:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s sent %lu renew after %lu, ignoring\n", __func__,
^
net/ceph/mon_client.c:353:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s %s epoch %u continuous %d\n", __func__, ceph_sub_str[sub],
^
net/ceph/mon_client.c:389:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s %s epoch %u\n", __func__, ceph_sub_str[sub], epoch);
^
net/ceph/mon_client.c:475:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("handle_monmap\n");
^
net/ceph/mon_client.c:513:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s greq %p request %p reply %p\n", __func__, req, req->request,
^
net/ceph/mon_client.c:550:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s greq %p\n", __func__, req);
^
net/ceph/mon_client.c:570:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s greq %p tid %llu\n", __func__, req, req->tid);
^
net/ceph/mon_client.c:579:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s greq %p tid %llu\n", __func__, req, req->tid);
^
net/ceph/mon_client.c:606:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s greq %p tid %llu\n", __func__, req, req->tid);
^
net/ceph/mon_client.c:623:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s greq %p tid %llu\n", __func__, req, req->tid);
^
net/ceph/mon_client.c:645:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("get_generic_reply %lld dne\n", tid);
^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
--
>> net/ceph/cls_lock_client.c:71:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s lock_name %s type %d cookie %s tag %s desc %s flags 0x%x\n",
^
net/ceph/cls_lock_client.c:71:2: note: did you mean 'dput'?
include/linux/dcache.h:374:13: note: 'dput' declared here
extern void dput(struct dentry *);
^
net/ceph/cls_lock_client.c:120:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s lock_name %s cookie %s\n", __func__, lock_name, cookie);
^
net/ceph/cls_lock_client.c:172:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s lock_name %s cookie %s locker %s%llu\n", __func__, lock_name,
^
net/ceph/cls_lock_client.c:223:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s lock_name %s type %d old_cookie %s tag %s new_cookie %s\n",
^
net/ceph/cls_lock_client.c:276:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s %s%llu cookie %s addr %s\n", __func__,
^
net/ceph/cls_lock_client.c:364:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s lock_name %s\n", __func__, lock_name);
^
6 errors generated.
--
>> net/ceph/osd_client.c:115:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout(" skipping last %llu, final file extent %llu~%llu\n",
^
net/ceph/osd_client.c:115:3: note: did you mean 'dput'?
include/linux/dcache.h:374:13: note: 'dput' declared here
extern void dput(struct dentry *);
^
net/ceph/osd_client.c:119:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("calc_layout objnum=%llx %llu~%llu\n", *objnum, *objoff, *objlen);
^
net/ceph/osd_client.c:485:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s %p (r_request %p r_reply %p)\n", __func__, req,
^
net/ceph/osd_client.c:510:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s %p (was %d)\n", __func__, req,
^
net/ceph/osd_client.c:519:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s %p (was %d)\n", __func__, req,
^
net/ceph/osd_client.c:558:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s req %p\n", __func__, req);
^
net/ceph/osd_client.c:604:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s req %p\n", __func__, req);
^
net/ceph/osd_client.c:1249:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("get_osd %p %d -> %d\n", osd, refcount_read(&osd->o_ref)-1,
^
net/ceph/osd_client.c:1253:3: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("get_osd %p FAIL\n", osd);
^
net/ceph/osd_client.c:1260:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("put_osd %p %d -> %d\n", osd, refcount_read(&osd->o_ref),
^
net/ceph/osd_client.c:1274:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s osd %p osd%d\n", __func__, osd, osd->o_osd);
^
net/ceph/osd_client.c:1295:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s osd %p osd%d\n", __func__, osd, osd->o_osd);
^
net/ceph/osd_client.c:1313:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s osd %p osd%d\n", __func__, osd, osd->o_osd);
^
net/ceph/osd_client.c:1352:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s osd %p osd%d\n", __func__, osd, osd->o_osd);
^
net/ceph/osd_client.c:1408:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s osdc %p osd%d -> osd %p\n", __func__, osdc, o, osd);
^
net/ceph/osd_client.c:1421:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s osd %p osd%d req %p tid %llu\n", __func__, osd, osd->o_osd,
^
net/ceph/osd_client.c:1438:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s osd %p osd%d req %p tid %llu\n", __func__, osd, osd->o_osd,
^
net/ceph/osd_client.c:1505:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s picked osd%d, primary osd%d\n", __func__,
^
net/ceph/osd_client.c:1535:2: error: implicit declaration of function 'dout' [-Werror,-Wimplicit-function-declaration]
dout("%s picked osd%d with locality %d, primary osd%d\n", __func__,
^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
..
# https://github.com/0day-ci/linux/commit/ab0413062c34a692c0480a3237a04534f...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Joe-Perches/ceph-Neaten-debugging/20200817-060623
git checkout ab0413062c34a692c0480a3237a04534f823e02d
vim +/dout +227 net/ceph/ceph_common.c
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 219
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 220 static int parse_fsid(const char *str, struct ceph_fsid *fsid)
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 221 {
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 222 int i = 0;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 223 char tmp[3];
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 224 int err = -EINVAL;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 225 int d;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 226
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 @227 dout("parse_fsid '%s'\n", str);
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 228 tmp[2] = 0;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 229 while (*str && i < 16) {
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 230 if (ispunct(*str)) {
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 231 str++;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 232 continue;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 233 }
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 234 if (!isxdigit(str[0]) || !isxdigit(str[1]))
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 235 break;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 236 tmp[0] = str[0];
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 237 tmp[1] = str[1];
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 238 if (sscanf(tmp, "%x", &d) < 1)
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 239 break;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 240 fsid->fsid[i] = d & 0xff;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 241 i++;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 242 str += 2;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 243 }
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 244
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 245 if (i == 16)
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 246 err = 0;
4c069a5821ddc5 Chengguang Xu 2018-01-30 247 dout("parse_fsid ret %d got fsid %pU\n", err, fsid);
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 248 return err;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 249 }
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 250
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
Re: [PATCH 1/2] ceph: Use generic debugging facility
by kernel test robot
Hi Joe,
I love your patch! Yet something to improve:
[auto build test ERROR on ceph-client/for-linus]
[also build test ERROR on v5.9-rc1 next-20200814]
[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/Joe-Perches/ceph-Neaten-debuggin...
base: https://github.com/ceph/ceph-client.git for-linus
config: openrisc-randconfig-r026-20200817 (attached as .config)
compiler: or1k-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
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=openrisc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
net/ceph/ceph_common.c: In function 'parse_fsid':
>> net/ceph/ceph_common.c:227:2: error: implicit declaration of function 'dout'; did you mean 'dput'? [-Werror=implicit-function-declaration]
227 | dout("parse_fsid '%s'\n", str);
| ^~~~
| dput
In file included from net/ceph/ceph_common.c:24:
At top level:
include/linux/ceph/ceph_features.h:21:24: warning: 'DEPRECATED_CEPH_FEATUREMASK_RESERVED_BROKEN' defined but not used [-Wunused-const-variable=]
21 | static const uint64_t DEPRECATED_CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:172:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE_DEPRECATED'
172 | DEFINE_CEPH_FEATURE_DEPRECATED(63, 1, RESERVED_BROKEN, LUMINOUS) // client-facing
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:20:24: warning: 'DEPRECATED_CEPH_FEATURE_RESERVED_BROKEN' defined but not used [-Wunused-const-variable=]
20 | static const uint64_t DEPRECATED_CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:172:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE_DEPRECATED'
172 | DEFINE_CEPH_FEATURE_DEPRECATED(63, 1, RESERVED_BROKEN, LUMINOUS) // client-facing
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_RESERVED' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:171:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
171 | DEFINE_CEPH_FEATURE(62, 1, RESERVED) // do not use; used as a sentinal
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_RESERVED' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:171:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
171 | DEFINE_CEPH_FEATURE(62, 1, RESERVED) // do not use; used as a sentinal
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_CEPHX_V2' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:169:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
169 | DEFINE_CEPH_FEATURE(61, 1, CEPHX_V2) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_OSD_RECOVERY_DELETES' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:168:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
168 | DEFINE_CEPH_FEATURE(60, 1, OSD_RECOVERY_DELETES) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_OSD_RECOVERY_DELETES' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:168:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
168 | DEFINE_CEPH_FEATURE(60, 1, OSD_RECOVERY_DELETES) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_MSG_ADDR2' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:167:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
167 | DEFINE_CEPH_FEATURE(59, 1, MSG_ADDR2) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_FS_CHANGE_ATTR' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:166:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
166 | DEFINE_CEPH_FEATURE(59, 1, FS_CHANGE_ATTR) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_FS_CHANGE_ATTR' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:166:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
166 | DEFINE_CEPH_FEATURE(59, 1, FS_CHANGE_ATTR) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_FS_BTIME' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:165:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
165 | DEFINE_CEPH_FEATURE(59, 1, FS_BTIME)
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_FS_BTIME' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:165:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
165 | DEFINE_CEPH_FEATURE(59, 1, FS_BTIME)
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_FS_FILE_LAYOUT_V2' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:164:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
164 | DEFINE_CEPH_FEATURE(58, 1, FS_FILE_LAYOUT_V2) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_FS_FILE_LAYOUT_V2' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:164:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
164 | DEFINE_CEPH_FEATURE(58, 1, FS_FILE_LAYOUT_V2) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_NEW_OSDOPREPLY_ENCODING' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:163:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
163 | DEFINE_CEPH_FEATURE(58, 1, NEW_OSDOPREPLY_ENCODING) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_CRUSH_TUNABLES5' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:162:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
162 | DEFINE_CEPH_FEATURE(58, 1, CRUSH_TUNABLES5)
--
net/ceph/messenger.c: In function 'con_sock_state_init':
>> net/ceph/messenger.c:309:2: error: implicit declaration of function 'dout'; did you mean 'dput'? [-Werror=implicit-function-declaration]
309 | dout("%s con %p sock %d -> %d\n", __func__, con, old_state,
| ^~~~
| dput
In file included from net/ceph/messenger.c:21:
At top level:
include/linux/ceph/ceph_features.h:21:24: warning: 'DEPRECATED_CEPH_FEATUREMASK_RESERVED_BROKEN' defined but not used [-Wunused-const-variable=]
21 | static const uint64_t DEPRECATED_CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:172:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE_DEPRECATED'
172 | DEFINE_CEPH_FEATURE_DEPRECATED(63, 1, RESERVED_BROKEN, LUMINOUS) // client-facing
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:20:24: warning: 'DEPRECATED_CEPH_FEATURE_RESERVED_BROKEN' defined but not used [-Wunused-const-variable=]
20 | static const uint64_t DEPRECATED_CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:172:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE_DEPRECATED'
172 | DEFINE_CEPH_FEATURE_DEPRECATED(63, 1, RESERVED_BROKEN, LUMINOUS) // client-facing
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_RESERVED' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:171:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
171 | DEFINE_CEPH_FEATURE(62, 1, RESERVED) // do not use; used as a sentinal
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_RESERVED' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:171:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
171 | DEFINE_CEPH_FEATURE(62, 1, RESERVED) // do not use; used as a sentinal
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_CEPHX_V2' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:169:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
169 | DEFINE_CEPH_FEATURE(61, 1, CEPHX_V2) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_CEPHX_V2' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:169:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
169 | DEFINE_CEPH_FEATURE(61, 1, CEPHX_V2) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_OSD_RECOVERY_DELETES' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:168:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
168 | DEFINE_CEPH_FEATURE(60, 1, OSD_RECOVERY_DELETES) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_OSD_RECOVERY_DELETES' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:168:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
168 | DEFINE_CEPH_FEATURE(60, 1, OSD_RECOVERY_DELETES) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_MSG_ADDR2' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:167:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
167 | DEFINE_CEPH_FEATURE(59, 1, MSG_ADDR2) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_MSG_ADDR2' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:167:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
167 | DEFINE_CEPH_FEATURE(59, 1, MSG_ADDR2) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_FS_CHANGE_ATTR' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:166:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
166 | DEFINE_CEPH_FEATURE(59, 1, FS_CHANGE_ATTR) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_FS_CHANGE_ATTR' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:166:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
166 | DEFINE_CEPH_FEATURE(59, 1, FS_CHANGE_ATTR) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_FS_BTIME' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:165:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
165 | DEFINE_CEPH_FEATURE(59, 1, FS_BTIME)
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_FS_BTIME' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:165:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
165 | DEFINE_CEPH_FEATURE(59, 1, FS_BTIME)
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_FS_FILE_LAYOUT_V2' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:164:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
164 | DEFINE_CEPH_FEATURE(58, 1, FS_FILE_LAYOUT_V2) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_FS_FILE_LAYOUT_V2' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:164:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
164 | DEFINE_CEPH_FEATURE(58, 1, FS_FILE_LAYOUT_V2) // overlap
--
net/ceph/msgpool.c: In function 'msgpool_alloc':
>> net/ceph/msgpool.c:20:3: error: implicit declaration of function 'dout'; did you mean 'dput'? [-Werror=implicit-function-declaration]
20 | dout("msgpool_alloc %s failed\n", pool->name);
| ^~~~
| dput
cc1: some warnings being treated as errors
--
net/ceph/buffer.c: In function 'ceph_buffer_new':
>> net/ceph/buffer.c:29:2: error: implicit declaration of function 'dout'; did you mean 'dput'? [-Werror=implicit-function-declaration]
29 | dout("buffer_new %p\n", b);
| ^~~~
| dput
cc1: some warnings being treated as errors
--
net/ceph/mon_client.c: In function 'ceph_monmap_decode':
>> net/ceph/mon_client.c:53:2: error: implicit declaration of function 'dout'; did you mean 'dput'? [-Werror=implicit-function-declaration]
53 | dout("monmap_decode %p %p len %d (%d)\n", p, end, len, (int)(end-p));
| ^~~~
| dput
In file included from net/ceph/mon_client.c:10:
At top level:
include/linux/ceph/ceph_features.h:21:24: warning: 'DEPRECATED_CEPH_FEATUREMASK_RESERVED_BROKEN' defined but not used [-Wunused-const-variable=]
21 | static const uint64_t DEPRECATED_CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:172:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE_DEPRECATED'
172 | DEFINE_CEPH_FEATURE_DEPRECATED(63, 1, RESERVED_BROKEN, LUMINOUS) // client-facing
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:20:24: warning: 'DEPRECATED_CEPH_FEATURE_RESERVED_BROKEN' defined but not used [-Wunused-const-variable=]
20 | static const uint64_t DEPRECATED_CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:172:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE_DEPRECATED'
172 | DEFINE_CEPH_FEATURE_DEPRECATED(63, 1, RESERVED_BROKEN, LUMINOUS) // client-facing
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_RESERVED' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:171:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
171 | DEFINE_CEPH_FEATURE(62, 1, RESERVED) // do not use; used as a sentinal
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_RESERVED' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:171:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
171 | DEFINE_CEPH_FEATURE(62, 1, RESERVED) // do not use; used as a sentinal
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_CEPHX_V2' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:169:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
169 | DEFINE_CEPH_FEATURE(61, 1, CEPHX_V2) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_CEPHX_V2' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:169:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
169 | DEFINE_CEPH_FEATURE(61, 1, CEPHX_V2) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_OSD_RECOVERY_DELETES' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:168:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
168 | DEFINE_CEPH_FEATURE(60, 1, OSD_RECOVERY_DELETES) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_OSD_RECOVERY_DELETES' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:168:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
168 | DEFINE_CEPH_FEATURE(60, 1, OSD_RECOVERY_DELETES) // *do not share this bit*
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_MSG_ADDR2' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:167:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
167 | DEFINE_CEPH_FEATURE(59, 1, MSG_ADDR2) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_MSG_ADDR2' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:167:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
167 | DEFINE_CEPH_FEATURE(59, 1, MSG_ADDR2) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_FS_CHANGE_ATTR' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:166:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
166 | DEFINE_CEPH_FEATURE(59, 1, FS_CHANGE_ATTR) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_FS_CHANGE_ATTR' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:166:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
166 | DEFINE_CEPH_FEATURE(59, 1, FS_CHANGE_ATTR) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_FS_BTIME' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:165:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
165 | DEFINE_CEPH_FEATURE(59, 1, FS_BTIME)
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_FS_BTIME' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:165:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
165 | DEFINE_CEPH_FEATURE(59, 1, FS_BTIME)
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:15:24: warning: 'CEPH_FEATUREMASK_FS_FILE_LAYOUT_V2' defined but not used [-Wunused-const-variable=]
15 | static const uint64_t CEPH_FEATUREMASK_##name = \
| ^~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:164:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
164 | DEFINE_CEPH_FEATURE(58, 1, FS_FILE_LAYOUT_V2) // overlap
| ^~~~~~~~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:14:24: warning: 'CEPH_FEATURE_FS_FILE_LAYOUT_V2' defined but not used [-Wunused-const-variable=]
14 | static const uint64_t CEPH_FEATURE_##name = (1ULL<<bit); \
| ^~~~~~~~~~~~~
include/linux/ceph/ceph_features.h:164:1: note: in expansion of macro 'DEFINE_CEPH_FEATURE'
164 | DEFINE_CEPH_FEATURE(58, 1, FS_FILE_LAYOUT_V2) // overlap
..
# https://github.com/0day-ci/linux/commit/ab0413062c34a692c0480a3237a04534f...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Joe-Perches/ceph-Neaten-debugging/20200817-060623
git checkout ab0413062c34a692c0480a3237a04534f823e02d
vim +227 net/ceph/ceph_common.c
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 219
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 220 static int parse_fsid(const char *str, struct ceph_fsid *fsid)
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 221 {
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 222 int i = 0;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 223 char tmp[3];
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 224 int err = -EINVAL;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 225 int d;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 226
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 @227 dout("parse_fsid '%s'\n", str);
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 228 tmp[2] = 0;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 229 while (*str && i < 16) {
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 230 if (ispunct(*str)) {
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 231 str++;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 232 continue;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 233 }
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 234 if (!isxdigit(str[0]) || !isxdigit(str[1]))
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 235 break;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 236 tmp[0] = str[0];
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 237 tmp[1] = str[1];
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 238 if (sscanf(tmp, "%x", &d) < 1)
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 239 break;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 240 fsid->fsid[i] = d & 0xff;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 241 i++;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 242 str += 2;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 243 }
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 244
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 245 if (i == 16)
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 246 err = 0;
4c069a5821ddc5 Chengguang Xu 2018-01-30 247 dout("parse_fsid ret %d got fsid %pU\n", err, fsid);
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 248 return err;
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 249 }
3d14c5d2b6e15c Yehuda Sadeh 2010-04-06 250
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
Re: [PATCH] seqlock: <linux/seqlock.h>: fix multiple kernel-doc warnings
by kernel test robot
Hi Randy,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[also build test WARNING on v5.9-rc1 next-20200814]
[cannot apply to tip/locking/core linux/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/Randy-Dunlap/seqlock-linux-seqlo...
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 9123e3a74ec7b934a4a099e98af6a61c2f80bbf5
config: s390-randconfig-r025-20200817 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project ab9fc8bae805c785066779e76e7846aabad5609e)
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 s390 cross compiling tool for clang build
# apt-get install binutils-s390x-linux-gnu
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=s390
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from arch/s390/kernel/asm-offsets.c:11:
In file included from include/linux/kvm_host.h:12:
In file included from include/linux/sched.h:20:
In file included from include/linux/hrtimer.h:20:
include/linux/seqlock.h:156:8: error: 'inline' can only appear on functions
static __always_inline void \
^
include/linux/compiler_attributes.h:70:41: note: expanded from macro '__always_inline'
#define __always_inline inline __attribute__((__always_inline__))
^
include/linux/compiler_types.h:144:16: note: expanded from macro 'inline'
#define inline inline __gnu_inline __inline_maybe_unused notrace
^
In file included from arch/s390/kernel/asm-offsets.c:11:
In file included from include/linux/kvm_host.h:12:
In file included from include/linux/sched.h:20:
In file included from include/linux/hrtimer.h:20:
include/linux/seqlock.h:156:8: warning: '__gnu_inline__' attribute only applies to functions [-Wignored-attributes]
include/linux/compiler_attributes.h:70:41: note: expanded from macro '__always_inline'
#define __always_inline inline __attribute__((__always_inline__))
^
include/linux/compiler_types.h:144:23: note: expanded from macro 'inline'
#define inline inline __gnu_inline __inline_maybe_unused notrace
^
include/linux/compiler_attributes.h:167:56: note: expanded from macro '__gnu_inline'
#define __gnu_inline __attribute__((__gnu_inline__))
^
In file included from arch/s390/kernel/asm-offsets.c:11:
In file included from include/linux/kvm_host.h:12:
In file included from include/linux/sched.h:20:
In file included from include/linux/hrtimer.h:20:
>> include/linux/seqlock.h:156:8: warning: '__no_instrument_function__' attribute only applies to functions [-Wignored-attributes]
include/linux/compiler_attributes.h:70:41: note: expanded from macro '__always_inline'
#define __always_inline inline __attribute__((__always_inline__))
^
include/linux/compiler_types.h:144:58: note: expanded from macro 'inline'
#define inline inline __gnu_inline __inline_maybe_unused notrace
^
include/linux/compiler_types.h:124:34: note: expanded from macro 'notrace'
#define notrace __attribute__((__no_instrument_function__))
^
In file included from arch/s390/kernel/asm-offsets.c:11:
In file included from include/linux/kvm_host.h:12:
In file included from include/linux/sched.h:20:
In file included from include/linux/hrtimer.h:20:
include/linux/seqlock.h:156:8: warning: '__always_inline__' attribute only applies to functions [-Wignored-attributes]
include/linux/compiler_attributes.h:70:63: note: expanded from macro '__always_inline'
#define __always_inline inline __attribute__((__always_inline__))
^
In file included from arch/s390/kernel/asm-offsets.c:11:
In file included from include/linux/kvm_host.h:12:
In file included from include/linux/sched.h:20:
In file included from include/linux/hrtimer.h:20:
include/linux/seqlock.h:156:34: error: variable has incomplete type 'void'
static __always_inline void \
^
include/linux/seqlock.h:157:10: error: expected ';' after top level declarator
seqcount_##lockname##_init(seqcount_##lockname##_t *s, locktype *lock) \
^
;
include/linux/seqlock.h:213:1: error: unknown type name 'seqcount_raw_spinlock_t'
SEQCOUNT_LOCKTYPE(raw_spinlock_t, raw_spinlock, false, s->lock)
^
include/linux/seqlock.h:177:29: note: expanded from macro 'SEQCOUNT_LOCKTYPE'
__seqcount_##lockname##_ptr(seqcount_##lockname##_t *s) \
^
<scratch space>:28:1: note: expanded from here
seqcount_raw_spinlock_t
^
In file included from arch/s390/kernel/asm-offsets.c:11:
In file included from include/linux/kvm_host.h:12:
In file included from include/linux/sched.h:20:
In file included from include/linux/hrtimer.h:20:
include/linux/seqlock.h:213:1: error: unknown type name 'seqcount_raw_spinlock_t'
include/linux/seqlock.h:183:37: note: expanded from macro 'SEQCOUNT_LOCKTYPE'
__seqcount_##lockname##_preemptible(seqcount_##lockname##_t *s) \
^
<scratch space>:32:1: note: expanded from here
seqcount_raw_spinlock_t
^
In file included from arch/s390/kernel/asm-offsets.c:11:
In file included from include/linux/kvm_host.h:12:
In file included from include/linux/sched.h:20:
In file included from include/linux/hrtimer.h:20:
include/linux/seqlock.h:213:1: error: unknown type name 'seqcount_raw_spinlock_t'
include/linux/seqlock.h:189:32: note: expanded from macro 'SEQCOUNT_LOCKTYPE'
__seqcount_##lockname##_assert(seqcount_##lockname##_t *s) \
^
<scratch space>:36:1: note: expanded from here
seqcount_raw_spinlock_t
^
In file included from arch/s390/kernel/asm-offsets.c:11:
In file included from include/linux/kvm_host.h:12:
In file included from include/linux/sched.h:20:
In file included from include/linux/hrtimer.h:20:
include/linux/seqlock.h:751:17: error: implicit declaration of function '__seqcount_ptr' [-Werror,-Wimplicit-function-declaration]
unsigned ret = read_seqcount_begin(&sl->seqcount);
^
include/linux/seqlock.h:306:24: note: expanded from macro 'read_seqcount_begin'
read_seqcount_t_begin(__seqcount_ptr(s))
^
include/linux/seqlock.h:248:28: note: expanded from macro '__seqcount_ptr'
#define __seqcount_ptr(s) __seqprop(s, ptr)
^
include/linux/seqlock.h:241:15: note: expanded from macro '__seqprop'
seqcount_t: __seqcount_##prop((void *)(s)), \
^
<scratch space>:129:1: note: expanded from here
__seqcount_ptr
^
include/linux/seqlock.h:751:17: note: did you mean '__seqcount_init'?
include/linux/seqlock.h:306:24: note: expanded from macro 'read_seqcount_begin'
read_seqcount_t_begin(__seqcount_ptr(s))
^
include/linux/seqlock.h:248:28: note: expanded from macro '__seqcount_ptr'
#define __seqcount_ptr(s) __seqprop(s, ptr)
^
include/linux/seqlock.h:241:15: note: expanded from macro '__seqprop'
seqcount_t: __seqcount_##prop((void *)(s)), \
^
<scratch space>:129:1: note: expanded from here
__seqcount_ptr
^
include/linux/seqlock.h:71:20: note: '__seqcount_init' declared here
static inline void __seqcount_init(seqcount_t *s, const char *name,
^
include/linux/seqlock.h:751:17: error: unknown type name 'seqcount_raw_spinlock_t'; did you mean 'seqcount_spinlock_t'?
unsigned ret = read_seqcount_begin(&sl->seqcount);
^
include/linux/seqlock.h:306:24: note: expanded from macro 'read_seqcount_begin'
read_seqcount_t_begin(__seqcount_ptr(s))
^
# https://github.com/0day-ci/linux/commit/360da27b2632939e5188f48d211b8a39c...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Randy-Dunlap/seqlock-linux-seqlock-h-fix-multiple-kernel-doc-warnings/20200817-062836
git checkout 360da27b2632939e5188f48d211b8a39cfdb8cde
vim +/__no_instrument_function__ +156 include/linux/seqlock.h
139
140 /**
141 * typedef seqcount_LOCKNAME_t - sequence counter with LOCKTYPE associated
142 * @seqcount: The real sequence counter
143 * @lock: Pointer to the associated spinlock
144 *
145 * A plain sequence counter with external writer synchronization by a
146 * spinlock. The spinlock is associated to the sequence count in the
147 * static initializer or init function. This enables lockdep to validate
148 * that the write side critical section is properly serialized.
149 */
150
151 /*
152 * seqcount_LOCKNAME_init() - runtime initializer for seqcount_LOCKNAME_t
153 * @s: Pointer to the seqcount_LOCKNAME_t instance
154 * @lock: Pointer to the associated LOCKTYPE
155 */
> 156 static __always_inline void \
157 seqcount_##lockname##_init(seqcount_##lockname##_t *s, locktype *lock) \
158 { \
159 seqcount_init(&s->seqcount); \
160 __SEQ_LOCK(s->lock = lock); \
161 } \
162
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
Re: [PATCH] seqlock: <linux/seqlock.h>: fix multiple kernel-doc warnings
by kernel test robot
Hi Randy,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v5.9-rc1 next-20200814]
[cannot apply to tip/locking/core linux/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/Randy-Dunlap/seqlock-linux-seqlo...
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 9123e3a74ec7b934a4a099e98af6a61c2f80bbf5
config: x86_64-randconfig-s022-20200817 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.2-180-g49f7e13a-dirty
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
In file included from include/linux/mmzone.h:16,
from include/linux/gfp.h:6,
from include/linux/slab.h:15,
from include/linux/crypto.h:20,
from arch/x86/kernel/asm-offsets.c:9:
>> include/linux/seqlock.h:157:10: error: stray '##' in program
157 | seqcount_##lockname##_init(seqcount_##lockname##_t *s, locktype *lock) \
| ^~
>> include/linux/seqlock.h:157:12: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'lockname'
157 | seqcount_##lockname##_init(seqcount_##lockname##_t *s, locktype *lock) \
| ^~~~~~~~
include/linux/seqlock.h:157:20: error: stray '##' in program
157 | seqcount_##lockname##_init(seqcount_##lockname##_t *s, locktype *lock) \
| ^~
>> include/linux/seqlock.h:157:12: error: unknown type name 'lockname'
157 | seqcount_##lockname##_init(seqcount_##lockname##_t *s, locktype *lock) \
| ^~~~~~~~
include/linux/seqlock.h:157:37: error: stray '##' in program
157 | seqcount_##lockname##_init(seqcount_##lockname##_t *s, locktype *lock) \
| ^~
include/linux/seqlock.h:157:47: error: stray '##' in program
157 | seqcount_##lockname##_init(seqcount_##lockname##_t *s, locktype *lock) \
| ^~
make[2]: *** [scripts/Makefile.build:117: arch/x86/kernel/asm-offsets.s] Error 1
make[2]: Target '__build' not remade because of errors.
make[1]: *** [Makefile:1203: prepare0] Error 2
make[1]: Target 'prepare' not remade because of errors.
make: *** [Makefile:185: __sub-make] Error 2
make: Target 'prepare' not remade because of errors.
# https://github.com/0day-ci/linux/commit/360da27b2632939e5188f48d211b8a39c...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Randy-Dunlap/seqlock-linux-seqlock-h-fix-multiple-kernel-doc-warnings/20200817-062836
git checkout 360da27b2632939e5188f48d211b8a39cfdb8cde
vim +157 include/linux/seqlock.h
139
140 /**
141 * typedef seqcount_LOCKNAME_t - sequence counter with LOCKTYPE associated
142 * @seqcount: The real sequence counter
143 * @lock: Pointer to the associated spinlock
144 *
145 * A plain sequence counter with external writer synchronization by a
146 * spinlock. The spinlock is associated to the sequence count in the
147 * static initializer or init function. This enables lockdep to validate
148 * that the write side critical section is properly serialized.
149 */
150
151 /*
152 * seqcount_LOCKNAME_init() - runtime initializer for seqcount_LOCKNAME_t
153 * @s: Pointer to the seqcount_LOCKNAME_t instance
154 * @lock: Pointer to the associated LOCKTYPE
155 */
156 static __always_inline void \
> 157 seqcount_##lockname##_init(seqcount_##lockname##_t *s, locktype *lock) \
158 { \
159 seqcount_init(&s->seqcount); \
160 __SEQ_LOCK(s->lock = lock); \
161 } \
162
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
drivers/gpu/drm/i915/gvt/gtt.c:263:19: error: unused function 'get_pt_type'
by kernel test robot
Hi Masahiro,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 9123e3a74ec7b934a4a099e98af6a61c2f80bbf5
commit: 6863f5643dd717376c2fdc85a47a00f9d738a834 kbuild: allow Clang to find unused static inline functions for W=1 build
date: 11 months ago
config: x86_64-randconfig-r013-20200817 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project ab9fc8bae805c785066779e76e7846aabad5609e)
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 x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
git checkout 6863f5643dd717376c2fdc85a47a00f9d738a834
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
>> drivers/gpu/drm/i915/gvt/gtt.c:263:19: error: unused function 'get_pt_type' [-Werror,-Wunused-function]
static inline int get_pt_type(int type)
^
>> drivers/gpu/drm/i915/gvt/gtt.c:587:20: error: unused function 'ppgtt_set_guest_root_entry' [-Werror,-Wunused-function]
static inline void ppgtt_set_guest_root_entry(struct intel_vgpu_mm *mm,
^
2 errors generated.
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 6863f5643dd717376c2fdc85a47a00f9d738a834
vim +/get_pt_type +263 drivers/gpu/drm/i915/gvt/gtt.c
2707e44466881d Zhi Wang 2016-03-28 262
054f4eba2a2985 Zhi Wang 2017-10-10 @263 static inline int get_pt_type(int type)
054f4eba2a2985 Zhi Wang 2017-10-10 264 {
054f4eba2a2985 Zhi Wang 2017-10-10 265 return gtt_type_table[type].pt_type;
054f4eba2a2985 Zhi Wang 2017-10-10 266 }
054f4eba2a2985 Zhi Wang 2017-10-10 267
:::::: The code at line 263 was first introduced by commit
:::::: 054f4eba2a2985b1db43353b7b5ce90e96cf9bb9 drm/i915/gvt: Introduce page table type of current level in GTT type enumerations
:::::: TO: Zhi Wang <zhi.a.wang(a)intel.com>
:::::: CC: Zhenyu Wang <zhenyuw(a)linux.intel.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
[afaerber:rtd1295-next 146/207] drivers/pci/controller/pcie-rtd129x.c:44:34: error: implicit declaration of function 'FIELD_PREP'
by kernel test robot
tree: https://github.com/afaerber/linux.git rtd1295-next
head: 55ecb5381393882d1f94c4db32e19d19bcd244b4
commit: 92d2b7bb455519e188dcb65fc48cfa81acdda2e5 [146/207] pci: Prepare RTD1295
config: alpha-allyesconfig (attached as .config)
compiler: alpha-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
git checkout 92d2b7bb455519e188dcb65fc48cfa81acdda2e5
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=alpha
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/pci/controller/pcie-rtd129x.c: In function 'rtd129x_pcie_phy_reset':
>> drivers/pci/controller/pcie-rtd129x.c:44:34: error: implicit declaration of function 'FIELD_PREP' [-Werror=implicit-function-declaration]
44 | #define MDIO_CTR_MDIO_RDWR_WRITE FIELD_PREP(MDIO_CTR_MDIO_RDWR, 1)
| ^~~~~~~~~~
drivers/pci/controller/pcie-rtd129x.c:95:29: note: in expansion of macro 'MDIO_CTR_MDIO_RDWR_WRITE'
95 | val = MDIO_CTR_MDIO_SRST | MDIO_CTR_MDIO_RDWR_WRITE;
| ^~~~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
# https://github.com/afaerber/linux/commit/92d2b7bb455519e188dcb65fc48cfa81...
git remote add afaerber https://github.com/afaerber/linux.git
git fetch --no-tags afaerber rtd1295-next
git checkout 92d2b7bb455519e188dcb65fc48cfa81acdda2e5
vim +/FIELD_PREP +44 drivers/pci/controller/pcie-rtd129x.c
41
42 #define MDIO_CTR_MDIO_RDWR BIT(0)
43 #define MDIO_CTR_MDIO_RDWR_READ FIELD_PREP(MDIO_CTR_MDIO_RDWR, 0)
> 44 #define MDIO_CTR_MDIO_RDWR_WRITE FIELD_PREP(MDIO_CTR_MDIO_RDWR, 1)
45 #define MDIO_CTR_MDIO_SRST BIT(1)
46 #define MDIO_CTR_MCLK_RATE GENMASK(3, 2)
47 #define MDIO_CTR_MDIO_RDY BIT(4)
48 #define MDIO_CTR_MDIO_ST GENMASK(6, 5)
49 #define MDIO_CTR_MDIO_BUSY BIT(7)
50 #define MDIO_CTR_PHY_REG GENMASK(12, 8)
51 #define MDIO_CTR_PHY_ADDR GENMASK(15, 13)
52 #define MDIO_CTR_DATA GENMASK(31, 16)
53
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
drivers/video/fbdev/chipsfb.c:334:9: sparse: sparse: incorrect type in argument 1 (different address spaces)
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 9123e3a74ec7b934a4a099e98af6a61c2f80bbf5
commit: 670d0a4b10704667765f7d18f7592993d02783aa sparse: use identifiers to define address spaces
date: 8 weeks ago
config: powerpc64-randconfig-s032-20200817 (attached as .config)
compiler: powerpc-linux-gcc (GCC) 9.3.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# apt-get install sparse
# sparse version: v0.6.2-180-g49f7e13a-dirty
git checkout 670d0a4b10704667765f7d18f7592993d02783aa
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=powerpc64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
>> drivers/video/fbdev/chipsfb.c:334:9: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void *s @@ got char [noderef] __iomem *screen_base @@
drivers/video/fbdev/chipsfb.c:334:9: sparse: expected void *s
>> drivers/video/fbdev/chipsfb.c:334:9: sparse: got char [noderef] __iomem *screen_base
--
>> drivers/soc/fsl/qe/usb.c:46:9: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected unsigned int volatile [noderef] [usertype] __iomem *addr @@ got restricted __be32 [noderef] __iomem * @@
>> drivers/soc/fsl/qe/usb.c:46:9: sparse: expected unsigned int volatile [noderef] [usertype] __iomem *addr
>> drivers/soc/fsl/qe/usb.c:46:9: sparse: got restricted __be32 [noderef] __iomem *
>> drivers/soc/fsl/qe/usb.c:46:9: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected unsigned int const volatile [noderef] [usertype] __iomem *addr @@ got restricted __be32 [noderef] __iomem * @@
>> drivers/soc/fsl/qe/usb.c:46:9: sparse: expected unsigned int const volatile [noderef] [usertype] __iomem *addr
>> drivers/soc/fsl/qe/usb.c:46:9: sparse: got restricted __be32 [noderef] __iomem *
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 670d0a4b10704667765f7d18f7592993d02783aa
vim +334 drivers/video/fbdev/chipsfb.c
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 331
48c68c4f1b5424 drivers/video/chipsfb.c Greg Kroah-Hartman 2012-12-21 332 static void init_chips(struct fb_info *p, unsigned long addr)
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 333 {
8c8709334cec80 drivers/video/chipsfb.c Benjamin Herrenschmidt 2005-06-27 @334 memset(p->screen_base, 0, 0x100000);
8c8709334cec80 drivers/video/chipsfb.c Benjamin Herrenschmidt 2005-06-27 335
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 336 p->fix = chipsfb_fix;
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 337 p->fix.smem_start = addr;
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 338
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 339 p->var = chipsfb_var;
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 340
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 341 p->fbops = &chipsfb_ops;
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 342 p->flags = FBINFO_DEFAULT;
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 343
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 344 fb_alloc_cmap(&p->cmap, 256, 0);
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 345
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 346 chips_hw_init();
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 347 }
^1da177e4c3f41 drivers/video/chipsfb.c Linus Torvalds 2005-04-16 348
:::::: The code at line 334 was first introduced by commit
:::::: 8c8709334cec803368a432a33e0f2e116d48fe07 [PATCH] ppc32: Remove CONFIG_PMAC_PBOOK
:::::: TO: Benjamin Herrenschmidt <benh(a)kernel.crashing.org>
:::::: CC: Linus Torvalds <torvalds(a)ppc970.osdl.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
Re: [RFC PATCH net-next] bridge: Implement MLD Querier wake-up calls / Android bug workaround
by kernel test robot
Hi "Linus,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on net-next/master]
url: https://github.com/0day-ci/linux/commits/Linus-L-ssing/bridge-Implement-M...
base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git bfdd5aaa54b0a44d9df550fe4c9db7e1470a11b8
config: arm64-allyesconfig (attached as .config)
compiler: aarch64-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
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
net/bridge/br_multicast.c: In function 'br_multicast_wakeupcall_check':
>> net/bridge/br_multicast.c:1968:27: warning: ordered comparison of pointer with integer zero [-Wextra]
1968 | if (ipv6_mc_check_icmpv6 < 0)
| ^
# https://github.com/0day-ci/linux/commit/fdde3c8b901df4c7d62fcd4cdd9a66114...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Linus-L-ssing/bridge-Implement-MLD-Querier-wake-up-calls-Android-bug-workaround/20200817-043402
git checkout fdde3c8b901df4c7d62fcd4cdd9a66114758889a
vim +1968 net/bridge/br_multicast.c
1930
1931 static bool br_multicast_wakeupcall_check(struct net_bridge *br,
1932 struct net_bridge_port *port,
1933 struct sk_buff *skb, u16 vid)
1934 {
1935 struct ethhdr *eth = eth_hdr(skb);
1936 const struct ipv6hdr *ip6h;
1937 unsigned int offset, len;
1938 struct icmp6hdr *icmp6h;
1939
1940 /* Wake-up calls to VLANs unsupported for now */
1941 if (!port->wakeupcall_num_rings || vid ||
1942 eth->h_proto != htons(ETH_P_IPV6))
1943 return false;
1944
1945 if (!ether_addr_equal(eth->h_dest, br->dev->dev_addr) ||
1946 is_multicast_ether_addr(eth->h_source) ||
1947 is_zero_ether_addr(eth->h_source))
1948 return false;
1949
1950 offset = skb_network_offset(skb) + sizeof(*ip6h);
1951 if (!pskb_may_pull(skb, offset))
1952 return false;
1953
1954 ip6h = ipv6_hdr(skb);
1955
1956 if (ip6h->version != 6)
1957 return false;
1958
1959 len = offset + ntohs(ip6h->payload_len);
1960 if (skb->len < len || len <= offset)
1961 return false;
1962
1963 if (ip6h->nexthdr != IPPROTO_ICMPV6)
1964 return false;
1965
1966 skb_set_transport_header(skb, offset);
1967
> 1968 if (ipv6_mc_check_icmpv6 < 0)
1969 return false;
1970
1971 icmp6h = (struct icmp6hdr *)skb_transport_header(skb);
1972 if (icmp6h->icmp6_type != ICMPV6_ECHO_REPLY ||
1973 icmp6h->icmp6_dataun.u_echo.identifier != BR_MC_WAKEUP_ID)
1974 return false;
1975
1976 return true;
1977 }
1978
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
drivers/video/fbdev/q40fb.c:105:27: sparse: sparse: incorrect type in assignment (different address spaces)
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 9123e3a74ec7b934a4a099e98af6a61c2f80bbf5
commit: 670d0a4b10704667765f7d18f7592993d02783aa sparse: use identifiers to define address spaces
date: 8 weeks ago
config: m68k-randconfig-s031-20200817 (attached as .config)
compiler: m68k-linux-gcc (GCC) 9.3.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# apt-get install sparse
# sparse version: v0.6.2-180-g49f7e13a-dirty
git checkout 670d0a4b10704667765f7d18f7592993d02783aa
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=m68k
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
>> drivers/video/fbdev/q40fb.c:105:27: sparse: sparse: incorrect type in assignment (different address spaces) @@ expected char [noderef] __iomem *screen_base @@ got char * @@
>> drivers/video/fbdev/q40fb.c:105:27: sparse: expected char [noderef] __iomem *screen_base
drivers/video/fbdev/q40fb.c:105:27: sparse: got char *
drivers/video/fbdev/q40fb.c:136:12: sparse: sparse: symbol 'q40fb_init' was not declared. Should it be static?
--
drivers/macintosh/via-cuda.c:520:24: sparse: sparse: Using plain integer as NULL pointer
>> drivers/macintosh/via-cuda.c:213:9: sparse: sparse: incorrect type in assignment (different address spaces) @@ expected unsigned char volatile [noderef] __iomem *static [toplevel] via @@ got unsigned char volatile [usertype] *extern [addressable] [toplevel] via1 @@
>> drivers/macintosh/via-cuda.c:213:9: sparse: expected unsigned char volatile [noderef] __iomem *static [toplevel] via
drivers/macintosh/via-cuda.c:213:9: sparse: got unsigned char volatile [usertype] *extern [addressable] [toplevel] via1
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 670d0a4b10704667765f7d18f7592993d02783aa
vim +105 drivers/video/fbdev/q40fb.c
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 84
48c68c4f1b5424 drivers/video/q40fb.c Greg Kroah-Hartman 2012-12-21 85 static int q40fb_probe(struct platform_device *dev)
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 86 {
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 87 struct fb_info *info;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 88
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 89 if (!MACH_IS_Q40)
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 90 return -ENXIO;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 91
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 92 /* mapped in q40/config.c */
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 93 q40fb_fix.smem_start = Q40_PHYS_SCREEN_ADDR;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 94
ce303c076baad1 drivers/video/q40fb.c Antonino A. Daplas 2007-07-17 95 info = framebuffer_alloc(sizeof(u32) * 16, &dev->dev);
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 96 if (!info)
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 97 return -ENOMEM;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 98
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 99 info->var = q40fb_var;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 100 info->fix = q40fb_fix;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 101 info->fbops = &q40fb_ops;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 102 info->flags = FBINFO_DEFAULT; /* not as module for now */
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 103 info->pseudo_palette = info->par;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 104 info->par = NULL;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 @105 info->screen_base = (char *) q40fb_fix.smem_start;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 106
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 107 if (fb_alloc_cmap(&info->cmap, 256, 0) < 0) {
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 108 framebuffer_release(info);
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 109 return -ENOMEM;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 110 }
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 111
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 112 master_outb(3, DISPLAY_CONTROL_REG);
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 113
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 114 if (register_framebuffer(info) < 0) {
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 115 printk(KERN_ERR "Unable to register Q40 frame buffer\n");
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 116 fb_dealloc_cmap(&info->cmap);
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 117 framebuffer_release(info);
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 118 return -EINVAL;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 119 }
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 120
31b6780c15a4e3 drivers/video/q40fb.c Joe Perches 2013-09-19 121 fb_info(info, "Q40 frame buffer alive and kicking !\n");
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 122 return 0;
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 123 }
^1da177e4c3f41 drivers/video/q40fb.c Linus Torvalds 2005-04-16 124
:::::: The code at line 105 was first introduced by commit
:::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2
:::::: TO: Linus Torvalds <torvalds(a)ppc970.osdl.org>
:::::: CC: Linus Torvalds <torvalds(a)ppc970.osdl.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month