Re: [bug report] staging: lustre: create striped directory

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 11, 2016 at 04:14:11PM +0000, Wang, Di wrote:
> Hello,
> 
> On 10/11/16, 4:37 AM, "Dan Carpenter" <dan.carpenter@xxxxxxxxxx> wrote:
> 
> >Hello wang di,
> >
> >The patch 2de353860782: "staging: lustre: create striped directory"
> >from Aug 16, 2016, leads to the following static checker warning:
> >
> >	drivers/staging/lustre/lustre/llite/namei.c:294 ll_md_blocking_ast()
> >	warn: 'master_inode' isn't an ERR_PTR
> >
> >drivers/staging/lustre/lustre/llite/namei.c
> >   287  
> >   288                                  hash =
> >cl_fid_build_ino(&lli->lli_pfid,
> >   289            
> >ll_need_32bit_api(ll_i2sbi(inode)));
> >   290  
> >   291                                  master_inode =
> >ilookup5(inode->i_sb, hash,
> >   292            
> >ll_test_inode_by_fid,
> >   293                                                          (void
> >*)&lli->lli_pfid);
> >   294                                  if (master_inode &&
> >!IS_ERR(master_inode)) {
> >                  
> >^^^^^^^^^^^^^^^^^^^^
> >ilookup5() never returns ERR_PTRs.
> 
> Yes, only master_inode != NULL should be enough here.

Please send a patch to remove the other check?

regards,
dan carpenter

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux