Re: [leon-rdma:rdma-next 58/59] drivers/infiniband/core/uverbs_cmd.c:820:6: warning: variable 'new_mr' is used uninitialized whenever 'if' condition is true

[Leon Romanovsky <leonro@nvidia.com>]

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

On Thu, Nov 05, 2020 at 12:33:39AM +0800, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/leon/linux-rdma.git rdma-next
> head:   e0a81838e0d3c558bfd2776fd6630470c4bdee0e
> commit: ce258f66e947591682a30f874692a72381dbfaf0 [58/59] RDMA/uverbs: Allow drivers to create a new HW object during rereg_mr
> config: x86_64-randconfig-a003-20201104 (attached as .config)
> compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project a6d15d40701ad38f29e4ff93703b3ffa7b204611)
> 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 x86_64 cross compiling tool for clang build
>         # apt-get install binutils-x86-64-linux-gnu
>         # https://git.kernel.org/pub/scm/linux/kernel/git/leon/linux-rdma.git/commit/?id=ce258f66e947591682a30f874692a72381dbfaf0
>         git remote add leon-rdma https://git.kernel.org/pub/scm/linux/kernel/git/leon/linux-rdma.git
>         git fetch --no-tags leon-rdma rdma-next
>         git checkout ce258f66e947591682a30f874692a72381dbfaf0
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All warnings (new ones prefixed by >>):

Thanks

diff --git a/drivers/infiniband/core/uverbs_cmd.c b/drivers/infiniband/core/uverbs_cmd.c
index bea1a1aeea22..cfdf89dd92b2 100644
--- a/drivers/infiniband/core/uverbs_cmd.c
+++ b/drivers/infiniband/core/uverbs_cmd.c
@@ -766,9 +766,9 @@ static int ib_uverbs_rereg_mr(struct uverbs_attr_bundle *attrs)
 	struct ib_mr                *mr;
 	int                          ret;
 	struct ib_uobject	    *uobj;
+	struct ib_mr *new_mr = NULL;
 	struct ib_uobject *new_uobj;
 	struct ib_device *ib_dev;
-	struct ib_mr *new_mr;

 	ret = uverbs_request(attrs, &cmd, sizeof(cmd));
 	if (ret)