[cifs:for-next 9/9] fs/cifs/cifsfs.c:219:28: sparse: incorrect type in assignment (different base types)

[Steve French]

fixed now - cifs-2.6.git for-next updated.  Aurelien reviewed.
On Fri, Jun 22, 2018 at 4:12 PM kbuild test robot <lkp at intel.com> wrote:
>
> tree:   git://git.samba.org/sfrench/cifs-2.6.git for-next
> head:   37a0fd808c8d8c2ad7d10275068d1c36dde14daf
> commit: 37a0fd808c8d8c2ad7d10275068d1c36dde14daf [9/9] smb3: fill in statfs fsid and correct namelen
> reproduce:
>         # apt-get install sparse
>         git checkout 37a0fd808c8d8c2ad7d10275068d1c36dde14daf
>         make ARCH=x86_64 allmodconfig
>         make C=1 CF=-D__CHECK_ENDIAN__
>
>
> sparse warnings: (new ones prefixed by >>)
>
>    fs/cifs/cifsglob.h:1703:41: sparse: symbol 'cifs_tcp_ses_list' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1714:41: sparse: symbol 'cifs_tcp_ses_lock' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1726:28: sparse: symbol 'GlobalCurrentXid' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1727:28: sparse: symbol 'GlobalTotalActiveXid' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1728:28: sparse: symbol 'GlobalMaxActiveXid' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1729:26: sparse: symbol 'GlobalMid_Lock' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1734:24: sparse: symbol 'sesInfoAllocCount' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1735:24: sparse: symbol 'tconInfoAllocCount' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1736:24: sparse: symbol 'tcpSesAllocCount' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1737:24: sparse: symbol 'tcpSesReconnectCount' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1738:24: sparse: symbol 'tconInfoReconnectCount' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1741:24: sparse: symbol 'bufAllocCount' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1743:24: sparse: symbol 'totBufAllocCount' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1744:24: sparse: symbol 'totSmBufAllocCount' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1746:24: sparse: symbol 'smBufAllocCount' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1747:24: sparse: symbol 'midCount' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1763:30: sparse: symbol 'uidtree' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1764:30: sparse: symbol 'gidtree' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1765:26: sparse: symbol 'siduidlock' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1766:26: sparse: symbol 'sidgidlock' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1767:30: sparse: symbol 'siduidtree' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1768:30: sparse: symbol 'sidgidtree' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1769:26: sparse: symbol 'uidsidlock' was not declared. Should it be static?
>    fs/cifs/cifsglob.h:1770:26: sparse: symbol 'gidsidlock' was not declared. Should it be static?
> >> fs/cifs/cifsfs.c:219:28: sparse: incorrect type in assignment (different base types) @@    expected int [signed] <noident> @@    got restricted __le6int [signed] <noident> @@
>    fs/cifs/cifsfs.c:219:28:    expected int [signed] <noident>
>    fs/cifs/cifsfs.c:219:28:    got restricted __le64 [usertype] vol_create_time
>
> vim +219 fs/cifs/cifsfs.c
>
>    199
>    200  static int
>    201  cifs_statfs(struct dentry *dentry, struct kstatfs *buf)
>    202  {
>    203          struct super_block *sb = dentry->d_sb;
>    204          struct cifs_sb_info *cifs_sb = CIFS_SB(sb);
>    205          struct cifs_tcon *tcon = cifs_sb_master_tcon(cifs_sb);
>    206          struct TCP_Server_Info *server = tcon->ses->server;
>    207          unsigned int xid;
>    208          int rc = 0;
>    209
>    210          xid = get_xid();
>    211
>    212          if (le32_to_cpu(tcon->fsAttrInfo.MaxPathNameComponentLength) > 0)
>    213                  buf->f_namelen =
>    214                         le32_to_cpu(tcon->fsAttrInfo.MaxPathNameComponentLength);
>    215          else
>    216                  buf->f_namelen = PATH_MAX;
>    217
>    218          buf->f_fsid.val[0] = tcon->vol_serial_number;
>  > 219          buf->f_fsid.val[1] = tcon->vol_create_time; /* random, see man statfs */
>    220
>    221          buf->f_files = 0;       /* undefined */
>    222          buf->f_ffree = 0;       /* unlimited */
>    223
>    224          if (server->ops->queryfs)
>    225                  rc = server->ops->queryfs(xid, tcon, buf);
>    226
>    227          free_xid(xid);
>    228          return 0;
>    229  }
>    230
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
> --
> To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Thanks,

Steve