Re: [ammarfaizi2-block:paulmck/linux-rcu/dev 28/28] kernel/rcu/tree_nocb.h:1213:3: warning: misleading indentation; statement is not part of the previous 'if'

From: Joel Fernandes
Date: Wed Apr 20 2022 - 12:40:08 EST


On Wed, Apr 20, 2022 at 9:49 AM Paul E. McKenney <paulmck@xxxxxxxxxx> wrote:
>
> On Wed, Apr 20, 2022 at 05:57:46PM +0800, kernel test robot wrote:
> > tree: https://github.com/ammarfaizi2/linux-block paulmck/linux-rcu/dev
> > head: ad68d8bf085421e4f22731c8b64c0b774a154e06
> > commit: ad68d8bf085421e4f22731c8b64c0b774a154e06 [28/28] rcu/nocb: Add an option to offload all CPUs on boot
> > config: riscv-randconfig-r024-20220420 (https://download.01.org/0day-ci/archive/20220420/202204201732.oTzYh0Xo-lkp@xxxxxxxxx/config)
> > compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project bac6cd5bf85669e3376610cfc4c4f9ca015e7b9b)
> > 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
> > # install riscv cross compiling tool for clang build
> > # apt-get install binutils-riscv64-linux-gnu
> > # https://github.com/ammarfaizi2/linux-block/commit/ad68d8bf085421e4f22731c8b64c0b774a154e06
> > git remote add ammarfaizi2-block https://github.com/ammarfaizi2/linux-block
> > git fetch --no-tags ammarfaizi2-block paulmck/linux-rcu/dev
> > git checkout ad68d8bf085421e4f22731c8b64c0b774a154e06
> > # save the config file
> > mkdir build_dir && cp config build_dir/.config
> > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash kernel/
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@xxxxxxxxx>
> >
> > All warnings (new ones prefixed by >>):
>
> I am dropping this commit, and thank you for your testing efforts!

I discussed privately with Paul and things are a bit late for -next as
other testing efforts are underway. So we'll revisit this patch in the
future. Just wanted to clarify to the onlooker that Paul isn't
dropping it because of a NAK or anything ;-)

Thanks,

- Joel