Re: [PATCH v5 4/5] proc: Relax check of mount visibility

From: kernel test robot <lkp@intel.com>
To: Alexey Gladkov <gladkov.alexey@gmail.com>,
	LKML <linux-kernel@vger.kernel.org>,
	"Eric W . Biederman" <ebiederm@xmission.com>
Cc: kbuild-all@lists.01.org, Kees Cook <keescook@chromium.org>,
	Linux Containers <containers@lists.linux-foundation.org>,
	Alexander Viro <viro@zeniv.linux.org.uk>,
	Linux FS Devel <linux-fsdevel@vger.kernel.org>,
	Alexey Gladkov <legion@kernel.org>
Subject: Re: [PATCH v5 4/5] proc: Relax check of mount visibility
Date: Thu, 11 Mar 2021 03:42:11 +0800	[thread overview]
Message-ID: <202103110340.VKEE0q8m-lkp@intel.com> (raw)
In-Reply-To: <eeba37cf1db2fb81bd4509efce9063156ebb83dc.1615400395.git.gladkov.alexey@gmail.com>

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

Hi Alexey,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linux/master]
[also build test ERROR on kees/for-next/pstore linus/master v5.12-rc2 next-20210310]
[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/Alexey-Gladkov/proc-Relax-check-of-mount-visibility/20210311-022252
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 144c79ef33536b4ecb4951e07dbc1f2b7fa99d32
config: powerpc-randconfig-s032-20210309 (attached as .config)
compiler: powerpc64-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.3-262-g5e674421-dirty
        # https://github.com/0day-ci/linux/commit/57a1fff647a507e103bbe22d67c6fe6b54c6a088
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Alexey-Gladkov/proc-Relax-check-of-mount-visibility/20210311-022252
        git checkout 57a1fff647a507e103bbe22d67c6fe6b54c6a088
        # 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=powerpc 

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 >>):

   fs/proc/root.c: In function 'proc_reconfigure':
>> fs/proc/root.c:233:11: error: 'fs_info' undeclared (first use in this function); did you mean 'qc_info'?
     233 |  put_cred(fs_info->mounter_cred);
         |           ^~~~~~~
         |           qc_info
   fs/proc/root.c:233:11: note: each undeclared identifier is reported only once for each function it appears in

vim +233 fs/proc/root.c

97412950b10e64 Vasiliy Kulikov 2012-01-10  226  
66f592e2ece038 David Howells   2018-11-01  227  static int proc_reconfigure(struct fs_context *fc)
97412950b10e64 Vasiliy Kulikov 2012-01-10  228  {
66f592e2ece038 David Howells   2018-11-01  229  	struct super_block *sb = fc->root->d_sb;
02b9984d640873 Theodore Ts'o   2014-03-13  230  
02b9984d640873 Theodore Ts'o   2014-03-13  231  	sync_filesystem(sb);
66f592e2ece038 David Howells   2018-11-01  232  
b84f25e5938b65 Alexey Gladkov  2021-03-10 @233  	put_cred(fs_info->mounter_cred);
b84f25e5938b65 Alexey Gladkov  2021-03-10  234  	fs_info->mounter_cred = get_cred(fc->cred);
b84f25e5938b65 Alexey Gladkov  2021-03-10  235  
57a1fff647a507 Alexey Gladkov  2021-03-10  236  	return proc_apply_options(sb, fc, current_user_ns());
97412950b10e64 Vasiliy Kulikov 2012-01-10  237  }
97412950b10e64 Vasiliy Kulikov 2012-01-10  238  

---
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: 28560 bytes --]

[-- Attachment #3: Type: text/plain, Size: 171 bytes --]

_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/containers