All of lore.kernel.org
 help / color / mirror / Atom feed
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

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.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: 3079 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(a)lists.01.org

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

WARNING: multiple messages have this Message-ID (diff)
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, Alexey Gladkov <legion@kernel.org>,
	Alexander Viro <viro@zeniv.linux.org.uk>,
	Kees Cook <keescook@chromium.org>,
	Linux Containers <containers@lists.linux-foundation.org>,
	Linux FS Devel <linux-fsdevel@vger.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 --]

  reply	other threads:[~2021-03-10 19:42 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-10 18:19 [PATCH v5 0/5] proc: Relax check of mount visibility Alexey Gladkov
2021-03-10 18:19 ` Alexey Gladkov
2021-03-10 18:19 ` [PATCH v5 1/5] docs: proc: add documentation about mount restrictions Alexey Gladkov
2021-03-10 18:19   ` Alexey Gladkov
2021-03-10 18:19 ` [PATCH v5 2/5] proc: Show /proc/self/net only for CAP_NET_ADMIN Alexey Gladkov
2021-03-10 18:19   ` Alexey Gladkov
2021-03-10 18:19 ` [PATCH v5 3/5] proc: Disable cancellation of subset=pid option Alexey Gladkov
2021-03-10 18:19   ` Alexey Gladkov
2021-03-10 18:19 ` [PATCH v5 4/5] proc: Relax check of mount visibility Alexey Gladkov
2021-03-10 18:19   ` Alexey Gladkov
2021-03-10 19:42   ` kernel test robot [this message]
2021-03-10 19:42     ` kernel test robot
2021-03-10 19:42     ` kernel test robot
2021-03-10 18:20 ` [PATCH v5 5/5] docs: proc: add documentation about relaxing visibility restrictions Alexey Gladkov
2021-03-10 18:20   ` Alexey Gladkov
2021-03-10 20:07 ` [PATCH v5 0/5] proc: Relax check of mount visibility Alexey Gladkov
2021-03-10 20:07   ` Alexey Gladkov

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=202103110340.VKEE0q8m-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=containers@lists.linux-foundation.org \
    --cc=ebiederm@xmission.com \
    --cc=gladkov.alexey@gmail.com \
    --cc=kbuild-all@lists.01.org \
    --cc=keescook@chromium.org \
    --cc=legion@kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=viro@zeniv.linux.org.uk \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.