mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Bhumika Goyal <bhumirks@gmail.com>
Cc: kbuild-all@01.org, julia.lawall@lip6.fr, mfasheh@versity.com,
	jlbec@evilplan.org, ocfs2-devel@oss.oracle.com,
	linux-kernel@vger.kernel.org, Bhumika Goyal <bhumirks@gmail.com>
Subject: Re: [PATCH] ocfs2/cluster: make config_item_type const
Date: Sat, 14 Oct 2017 20:14:49 +0800	[thread overview]
Message-ID: <201710142029.1LNtJPOZ%fengguang.wu@intel.com> (raw)
In-Reply-To: <1507798192-20528-1-git-send-email-bhumirks@gmail.com>

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

Hi Bhumika,

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

url:    https://github.com/0day-ci/linux/commits/Bhumika-Goyal/ocfs2-cluster-make-config_item_type-const/20171014-185701
config: tile-allyesconfig (attached as .config)
compiler: tilegx-linux-gcc (GCC) 4.6.2
reproduce:
        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
        make.cross ARCH=tile 

All warnings (new ones prefixed by >>):

   fs/ocfs2/cluster/nodemanager.c: In function 'o2nm_node_group_make_item':
   fs/ocfs2/cluster/nodemanager.c:573:2: warning: passing argument 3 of 'config_item_init_type_name' discards 'const' qualifier from pointer target type [enabled by default]
   include/linux/configfs.h:73:13: note: expected 'struct config_item_type *' but argument is of type 'const struct config_item_type *'
   fs/ocfs2/cluster/nodemanager.c: In function 'o2nm_cluster_group_make_group':
   fs/ocfs2/cluster/nodemanager.c:681:9: warning: passing argument 3 of 'config_group_init_type_name' discards 'const' qualifier from pointer target type [enabled by default]
   include/linux/configfs.h:102:13: note: expected 'struct config_item_type *' but argument is of type 'const struct config_item_type *'
   fs/ocfs2/cluster/nodemanager.c:685:9: warning: passing argument 3 of 'config_group_init_type_name' discards 'const' qualifier from pointer target type [enabled by default]
   include/linux/configfs.h:102:13: note: expected 'struct config_item_type *' but argument is of type 'const struct config_item_type *'
   fs/ocfs2/cluster/nodemanager.c: In function 'o2nm_depend_item':
   fs/ocfs2/cluster/nodemanager.c:743:2: warning: passing argument 1 of 'configfs_depend_item' discards 'const' qualifier from pointer target type [enabled by default]
   include/linux/configfs.h:269:5: note: expected 'struct configfs_subsystem *' but argument is of type 'const struct configfs_subsystem *'
   fs/ocfs2/cluster/nodemanager.c: In function 'exit_o2nm':
   fs/ocfs2/cluster/nodemanager.c:785:2: warning: passing argument 1 of 'configfs_unregister_subsystem' discards 'const' qualifier from pointer target type [enabled by default]
   include/linux/configfs.h:253:6: note: expected 'struct configfs_subsystem *' but argument is of type 'const struct configfs_subsystem *'
   fs/ocfs2/cluster/nodemanager.c: In function 'init_o2nm':
   fs/ocfs2/cluster/nodemanager.c:808:2: warning: passing argument 1 of 'config_group_init' discards 'const' qualifier from pointer target type [enabled by default]
   include/linux/configfs.h:101:13: note: expected 'struct config_group *' but argument is of type 'const struct config_group *'
>> fs/ocfs2/cluster/nodemanager.c:809:2: warning: passing argument 1 of '__mutex_init' discards 'const' qualifier from pointer target type [enabled by default]
   include/linux/mutex.h:133:13: note: expected 'struct mutex *' but argument is of type 'const struct mutex *'
   fs/ocfs2/cluster/nodemanager.c:810:2: warning: passing argument 1 of 'configfs_register_subsystem' discards 'const' qualifier from pointer target type [enabled by default]
   include/linux/configfs.h:252:5: note: expected 'struct configfs_subsystem *' but argument is of type 'const struct configfs_subsystem *'
   fs/ocfs2/cluster/nodemanager.c:820:2: warning: passing argument 1 of 'configfs_unregister_subsystem' discards 'const' qualifier from pointer target type [enabled by default]
   include/linux/configfs.h:253:6: note: expected 'struct configfs_subsystem *' but argument is of type 'const struct configfs_subsystem *'

vim +809 fs/ocfs2/cluster/nodemanager.c

0c83ed8e Kurt Hackel       2005-12-15  740  
14829422 Joel Becker       2007-06-14  741  int o2nm_depend_item(struct config_item *item)
14829422 Joel Becker       2007-06-14  742  {
14829422 Joel Becker       2007-06-14 @743  	return configfs_depend_item(&o2nm_cluster_group.cs_subsys, item);
14829422 Joel Becker       2007-06-14  744  }
14829422 Joel Becker       2007-06-14  745  
14829422 Joel Becker       2007-06-14  746  void o2nm_undepend_item(struct config_item *item)
14829422 Joel Becker       2007-06-14  747  {
9a9e3415 Krzysztof Opasiak 2015-12-11  748  	configfs_undepend_item(item);
14829422 Joel Becker       2007-06-14  749  }
14829422 Joel Becker       2007-06-14  750  
16c6a4f2 Joel Becker       2007-06-19  751  int o2nm_depend_this_node(void)
16c6a4f2 Joel Becker       2007-06-19  752  {
16c6a4f2 Joel Becker       2007-06-19  753  	int ret = 0;
16c6a4f2 Joel Becker       2007-06-19  754  	struct o2nm_node *local_node;
16c6a4f2 Joel Becker       2007-06-19  755  
16c6a4f2 Joel Becker       2007-06-19  756  	local_node = o2nm_get_node_by_num(o2nm_this_node());
16c6a4f2 Joel Becker       2007-06-19  757  	if (!local_node) {
16c6a4f2 Joel Becker       2007-06-19  758  		ret = -EINVAL;
16c6a4f2 Joel Becker       2007-06-19  759  		goto out;
16c6a4f2 Joel Becker       2007-06-19  760  	}
16c6a4f2 Joel Becker       2007-06-19  761  
16c6a4f2 Joel Becker       2007-06-19  762  	ret = o2nm_depend_item(&local_node->nd_item);
16c6a4f2 Joel Becker       2007-06-19  763  	o2nm_node_put(local_node);
16c6a4f2 Joel Becker       2007-06-19  764  
16c6a4f2 Joel Becker       2007-06-19  765  out:
16c6a4f2 Joel Becker       2007-06-19  766  	return ret;
16c6a4f2 Joel Becker       2007-06-19  767  }
16c6a4f2 Joel Becker       2007-06-19  768  
16c6a4f2 Joel Becker       2007-06-19  769  void o2nm_undepend_this_node(void)
16c6a4f2 Joel Becker       2007-06-19  770  {
16c6a4f2 Joel Becker       2007-06-19  771  	struct o2nm_node *local_node;
16c6a4f2 Joel Becker       2007-06-19  772  
16c6a4f2 Joel Becker       2007-06-19  773  	local_node = o2nm_get_node_by_num(o2nm_this_node());
16c6a4f2 Joel Becker       2007-06-19  774  	BUG_ON(!local_node);
16c6a4f2 Joel Becker       2007-06-19  775  
16c6a4f2 Joel Becker       2007-06-19  776  	o2nm_undepend_item(&local_node->nd_item);
16c6a4f2 Joel Becker       2007-06-19  777  	o2nm_node_put(local_node);
16c6a4f2 Joel Becker       2007-06-19  778  }
16c6a4f2 Joel Becker       2007-06-19  779  
16c6a4f2 Joel Becker       2007-06-19  780  
0c83ed8e Kurt Hackel       2005-12-15  781  static void __exit exit_o2nm(void)
0c83ed8e Kurt Hackel       2005-12-15  782  {
0c83ed8e Kurt Hackel       2005-12-15  783  	/* XXX sync with hb callbacks and shut down hb? */
0c83ed8e Kurt Hackel       2005-12-15  784  	o2net_unregister_hb_callbacks();
0c83ed8e Kurt Hackel       2005-12-15  785  	configfs_unregister_subsystem(&o2nm_cluster_group.cs_subsys);
0c83ed8e Kurt Hackel       2005-12-15  786  	o2cb_sys_shutdown();
0c83ed8e Kurt Hackel       2005-12-15  787  
0c83ed8e Kurt Hackel       2005-12-15  788  	o2net_exit();
87d3d3f3 Sunil Mushran     2008-12-17  789  	o2hb_exit();
0c83ed8e Kurt Hackel       2005-12-15  790  }
0c83ed8e Kurt Hackel       2005-12-15  791  
0c83ed8e Kurt Hackel       2005-12-15  792  static int __init init_o2nm(void)
0c83ed8e Kurt Hackel       2005-12-15  793  {
0c83ed8e Kurt Hackel       2005-12-15  794  	int ret = -1;
0c83ed8e Kurt Hackel       2005-12-15  795  
87d3d3f3 Sunil Mushran     2008-12-17  796  	ret = o2hb_init();
87d3d3f3 Sunil Mushran     2008-12-17  797  	if (ret)
87d3d3f3 Sunil Mushran     2008-12-17  798  		goto out;
2309e9e0 Sunil Mushran     2008-04-14  799  
2309e9e0 Sunil Mushran     2008-04-14  800  	ret = o2net_init();
2309e9e0 Sunil Mushran     2008-04-14  801  	if (ret)
87d3d3f3 Sunil Mushran     2008-12-17  802  		goto out_o2hb;
0c83ed8e Kurt Hackel       2005-12-15  803  
0c83ed8e Kurt Hackel       2005-12-15  804  	ret = o2net_register_hb_callbacks();
0c83ed8e Kurt Hackel       2005-12-15  805  	if (ret)
3878f110 Joel Becker       2008-05-30  806  		goto out_o2net;
0c83ed8e Kurt Hackel       2005-12-15  807  
0c83ed8e Kurt Hackel       2005-12-15  808  	config_group_init(&o2nm_cluster_group.cs_subsys.su_group);
e6bd07ae Joel Becker       2007-07-06 @809  	mutex_init(&o2nm_cluster_group.cs_subsys.su_mutex);
0c83ed8e Kurt Hackel       2005-12-15  810  	ret = configfs_register_subsystem(&o2nm_cluster_group.cs_subsys);
0c83ed8e Kurt Hackel       2005-12-15  811  	if (ret) {
0c83ed8e Kurt Hackel       2005-12-15  812  		printk(KERN_ERR "nodemanager: Registration returned %d\n", ret);
0c83ed8e Kurt Hackel       2005-12-15  813  		goto out_callbacks;
0c83ed8e Kurt Hackel       2005-12-15  814  	}
0c83ed8e Kurt Hackel       2005-12-15  815  
0c83ed8e Kurt Hackel       2005-12-15  816  	ret = o2cb_sys_init();
0c83ed8e Kurt Hackel       2005-12-15  817  	if (!ret)
0c83ed8e Kurt Hackel       2005-12-15  818  		goto out;
0c83ed8e Kurt Hackel       2005-12-15  819  
0c83ed8e Kurt Hackel       2005-12-15  820  	configfs_unregister_subsystem(&o2nm_cluster_group.cs_subsys);
0c83ed8e Kurt Hackel       2005-12-15  821  out_callbacks:
0c83ed8e Kurt Hackel       2005-12-15  822  	o2net_unregister_hb_callbacks();
895928b8 Jeff Mahoney      2006-02-21  823  out_o2net:
895928b8 Jeff Mahoney      2006-02-21  824  	o2net_exit();
87d3d3f3 Sunil Mushran     2008-12-17  825  out_o2hb:
87d3d3f3 Sunil Mushran     2008-12-17  826  	o2hb_exit();
0c83ed8e Kurt Hackel       2005-12-15  827  out:
0c83ed8e Kurt Hackel       2005-12-15  828  	return ret;
0c83ed8e Kurt Hackel       2005-12-15  829  }
0c83ed8e Kurt Hackel       2005-12-15  830  

:::::: The code at line 809 was first introduced by commit
:::::: e6bd07aee739566803425acdbf5cdb29919164e1 configfs: Convert subsystem semaphore to mutex

:::::: TO: Joel Becker <joel.becker@oracle.com>
:::::: CC: Mark Fasheh <mark.fasheh@oracle.com>

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

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

  parent reply	other threads:[~2017-10-14 12:15 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-12  8:49 Bhumika Goyal
2017-10-14 11:50 ` kbuild test robot
2017-10-14 12:14 ` kbuild test robot [this message]
2017-10-16  3:26   ` [Ocfs2-devel] " Gang He
2017-10-16  4:09     ` Fengguang Wu
2017-10-16  5:48     ` Julia Lawall

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=201710142029.1LNtJPOZ%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=bhumirks@gmail.com \
    --cc=jlbec@evilplan.org \
    --cc=julia.lawall@lip6.fr \
    --cc=kbuild-all@01.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mfasheh@versity.com \
    --cc=ocfs2-devel@oss.oracle.com \
    /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

all inboxes | Powered by JetHome®