[linuxppc:next-test 71/81] fs/dlm/lowcomms.c:1551:16: sparse: sparse: context imbalance in 'dlm_lowcomms_new_msg' - wrong count at exit

[linuxppc:next-test 71/81] fs/dlm/lowcomms.c:1551:16: sparse: sparse: context imbalance in 'dlm_lowcomms_new_msg' - wrong count at exit

[kernel test robot]

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

CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Christophe Leroy <christophe.leroy@csgroup.eu>
CC: Michael Ellerman <mpe@ellerman.id.au>

tree:   https://github.com/linuxppc/linux next-test
head:   51ed00e71f0130e0f3534b8e7d78facd16829426
commit: 1e688dd2a3d6759d416616ff07afc4bb836c4213 [71/81] powerpc/bug: Provide better flexibility to WARN_ON/__WARN_FLAGS() with asm goto
:::::: branch date: 9 hours ago
:::::: commit date: 4 days ago
config: powerpc-randconfig-s032-20210818 (attached as .config)
compiler: powerpc-linux-gcc (GCC) 11.2.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-348-gf0e6938b-dirty
        # https://github.com/linuxppc/linux/commit/1e688dd2a3d6759d416616ff07afc4bb836c4213
        git remote add linuxppc https://github.com/linuxppc/linux
        git fetch --no-tags linuxppc next-test
        git checkout 1e688dd2a3d6759d416616ff07afc4bb836c4213
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=powerpc SHELL=/bin/bash fs/dlm/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>


sparse warnings: (new ones prefixed by >>)
>> fs/dlm/lowcomms.c:1551:16: sparse: sparse: context imbalance in 'dlm_lowcomms_new_msg' - wrong count at exit
   fs/dlm/lowcomms.c: note: in included file (through include/linux/notifier.h, include/linux/memory_hotplug.h, include/linux/mmzone.h, ...):
   include/linux/srcu.h:188:9: sparse: sparse: context imbalance in 'dlm_lowcomms_commit_msg' - unexpected unlock

vim +/dlm_lowcomms_new_msg +1551 fs/dlm/lowcomms.c

2874d1a68c4ec5 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1550  
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21 @1551  struct dlm_msg *dlm_lowcomms_new_msg(int nodeid, int len, gfp_t allocation,
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1552  				     char **ppc, void (*cb)(struct dlm_mhandle *mh),
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1553  				     struct dlm_mhandle *mh)
f0747ebf48f362 fs/dlm/lowcomms.c     Alexander Aring   2021-03-01  1554  {
f0747ebf48f362 fs/dlm/lowcomms.c     Alexander Aring   2021-03-01  1555  	struct connection *con;
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1556  	struct dlm_msg *msg;
b38bc9c2b3171f fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1557  	int idx;
f0747ebf48f362 fs/dlm/lowcomms.c     Alexander Aring   2021-03-01  1558  
d10a0b88751a09 fs/dlm/lowcomms.c     Alexander Aring   2021-06-02  1559  	if (len > DLM_MAX_SOCKET_BUFSIZE ||
f0747ebf48f362 fs/dlm/lowcomms.c     Alexander Aring   2021-03-01  1560  	    len < sizeof(struct dlm_header)) {
d10a0b88751a09 fs/dlm/lowcomms.c     Alexander Aring   2021-06-02  1561  		BUILD_BUG_ON(PAGE_SIZE < DLM_MAX_SOCKET_BUFSIZE);
f0747ebf48f362 fs/dlm/lowcomms.c     Alexander Aring   2021-03-01  1562  		log_print("failed to allocate a buffer of size %d", len);
f0747ebf48f362 fs/dlm/lowcomms.c     Alexander Aring   2021-03-01  1563  		WARN_ON(1);
f0747ebf48f362 fs/dlm/lowcomms.c     Alexander Aring   2021-03-01  1564  		return NULL;
fdda387f73947e fs/dlm/lowcomms-tcp.c Patrick Caulfield 2006-11-02  1565  	}
f0747ebf48f362 fs/dlm/lowcomms.c     Alexander Aring   2021-03-01  1566  
b38bc9c2b3171f fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1567  	idx = srcu_read_lock(&connections_srcu);
f0747ebf48f362 fs/dlm/lowcomms.c     Alexander Aring   2021-03-01  1568  	con = nodeid2con(nodeid, allocation);
b38bc9c2b3171f fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1569  	if (!con) {
b38bc9c2b3171f fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1570  		srcu_read_unlock(&connections_srcu, idx);
fdda387f73947e fs/dlm/lowcomms-tcp.c Patrick Caulfield 2006-11-02  1571  		return NULL;
b38bc9c2b3171f fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1572  	}
f0747ebf48f362 fs/dlm/lowcomms.c     Alexander Aring   2021-03-01  1573  
2874d1a68c4ec5 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1574  	msg = dlm_lowcomms_new_msg_con(con, len, allocation, ppc, cb, mh);
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1575  	if (!msg) {
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1576  		srcu_read_unlock(&connections_srcu, idx);
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1577  		return NULL;
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1578  	}
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1579  
b38bc9c2b3171f fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1580  	/* we assume if successful commit must called */
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1581  	msg->idx = idx;
8f2dc78dbc2010 fs/dlm/lowcomms.c     Alexander Aring   2021-05-21  1582  	return msg;
fdda387f73947e fs/dlm/lowcomms-tcp.c Patrick Caulfield 2006-11-02  1583  }
fdda387f73947e fs/dlm/lowcomms-tcp.c Patrick Caulfield 2006-11-02  1584  

:::::: The code at line 1551 was first introduced by commit
:::::: 8f2dc78dbc2010b497bb58e0460cb44c678a3c5b fs: dlm: make buffer handling per msg

:::::: TO: Alexander Aring <aahringo@redhat.com>
:::::: CC: David Teigland <teigland@redhat.com>

---
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: 43253 bytes --]