public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Joe Perches <joe@perches.com>, Jeff Layton <jlayton@kernel.org>,
	Ilya Dryomov <idryomov@gmail.com>
Cc: kbuild-all@lists.01.org, ceph-devel@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] ceph: Use generic debugging facility
Date: Mon, 17 Aug 2020 07:03:48 +0800	[thread overview]
Message-ID: <202008170710.BoRBUoeo%lkp@intel.com> (raw)
In-Reply-To: <d67377660a317c826a7899e6ffcb5bf686733ddb.1597615399.git.joe@perches.com>

[-- Attachment #1: Type: text/plain, Size: 27145 bytes --]

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-debugging/20200817-060623
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@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/ab0413062c34a692c0480a3237a04534f823e02d
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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 24973 bytes --]

  reply	other threads:[~2020-08-16 23:04 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-16 22:04 [PATCH 0/2] ceph: Neaten debugging Joe Perches
2020-08-16 22:04 ` [PATCH 1/2] ceph: Use generic debugging facility Joe Perches
2020-08-16 23:03   ` kernel test robot [this message]
2020-08-16 23:45     ` Joe Perches
2020-08-16 23:50   ` kernel test robot
2020-08-17  0:13   ` kernel test robot
2020-08-16 22:04 ` [PATCH 2/2] ceph: Remove embedded function names from pr_debug uses Joe Perches

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=202008170710.BoRBUoeo%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=idryomov@gmail.com \
    --cc=jlayton@kernel.org \
    --cc=joe@perches.com \
    --cc=kbuild-all@lists.01.org \
    --cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox