[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20171016040951.uqwsroiip57m6nnr@wfg-t540p.sh.intel.com>
Date: Mon, 16 Oct 2017 12:09:51 +0800
From: Fengguang Wu <lkp@...el.com>
To: Gang He <ghe@...e.com>
Cc: kbuild-all@...org, bhumirks@...il.com, julia.lawall@...6.fr,
ocfs2-devel@....oracle.com, linux-kernel@...r.kernel.org
Subject: Re: [Ocfs2-devel] [PATCH] ocfs2/cluster: make config_item_type const
Hi Gang,
On Sun, Oct 15, 2017 at 09:26:00PM -0600, Gang He wrote:
>Hello Intel Kbuild team,
>
>You just upgrade GCC version when compiling the latest kernel?
This report comes from a pretty old gcc:
config: tile-allyesconfig (attached as .config)
compiler: tilegx-linux-gcc (GCC) 4.6.2
Thanks,
Fengguang
>Since these code looks a little old, I am sure that the related contributors are still work on OCFS2 project.
>
>
>Thanks
>Gang
>
>
>>>>
>> 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://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_0day-2Dci_l
>> inux_commits_Bhumika-2DGoyal_ocfs2-2Dcluster-2Dmake-2Dconfig-5Fitem-5Ftype-2Dconst_2
>> 0171014-2D185701&d=DwIBAg&c=RoP1YumCXCgaWHvlZYR8PQcxBKCX5YTpkKY057SbK10&r=f4ohdmG
>> rYxZejY77yzx3eNgTHb1ZAfZytktjHqNVzc8&m=8Ca3nktkbfrjbR5Spv4mvmdj2s377WohnG6h6z1
>> WB7E&s=hFVfDC8STTMTIDNtdipf8BlwR_V8RJG6an_l-r9MKyA&e=
>> config: tile-allyesconfig (attached as .config)
>> compiler: tilegx-linux-gcc (GCC) 4.6.2
>> reproduce:
>> wget
>> https://urldefense.proofpoint.com/v2/url?u=https-3A__raw.githubusercontent.c
>> om_intel_lkp-2Dtests_master_sbin_make.cross&d=DwIBAg&c=RoP1YumCXCgaWHvlZYR8PQcxB
>> KCX5YTpkKY057SbK10&r=f4ohdmGrYxZejY77yzx3eNgTHb1ZAfZytktjHqNVzc8&m=8Ca3nktkbfrj
>> bR5Spv4mvmdj2s377WohnG6h6z1WB7E&s=f5lUGF9NCtygTaG6AR0HkjhG7fOwD5nKFLaYCvPpju0&e
>> = -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@...cle.com>
>> :::::: CC: Mark Fasheh <mark.fasheh@...cle.com>
>>
>> ---
>> 0-DAY kernel test infrastructure Open Source Technology Center
>> https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.01.org_pipermail_k
>> build-2Dall&d=DwIBAg&c=RoP1YumCXCgaWHvlZYR8PQcxBKCX5YTpkKY057SbK10&r=f4ohdmGrYxZe
>> jY77yzx3eNgTHb1ZAfZytktjHqNVzc8&m=8Ca3nktkbfrjbR5Spv4mvmdj2s377WohnG6h6z1WB7E&s
>> =nLlW0xEmimMcnOene83E4zhz60ab1-1iXKtP_w_e8cQ&e= Intel
>> Corporation
>
Powered by blists - more mailing lists