Re: [PATCH] qla2xxx: Protect access to qpair members with qpair->qp_lock (fwd)

[Julia Lawall]

Please check on whether an unlock is neeed before line 1965.

julia

---------- Forwarded message ----------
Date: Fri, 23 Jun 2017 15:23:00 +0800
From: kbuild test robot <fengguang.wu@xxxxxxxxx>
To: kbuild@xxxxxx
Cc: Julia Lawall <julia.lawall@xxxxxxx>
Subject: Re: [PATCH] qla2xxx: Protect access to qpair members with
    qpair->qp_lock

CC: kbuild-all@xxxxxx
In-Reply-To: <20170622134325.26931-1-jthumshirn@xxxxxxx>

Hi Johannes,

[auto build test WARNING on scsi/for-next]
[also build test WARNING on v4.12-rc6 next-20170622]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Johannes-Thumshirn/qla2xxx-Protect-access-to-qpair-members-with-qpair-qp_lock/20170623-123844
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
:::::: branch date: 3 hours ago
:::::: commit date: 3 hours ago

>> drivers/scsi/qla2xxx/qla_iocb.c:1965:3-9: preceding lock on line 1952

git remote add linux-review https://github.com/0day-ci/linux
git remote update linux-review
git checkout 4a35d720268dbe9ac016957a3c4fc644398d68ba
vim +1965 drivers/scsi/qla2xxx/qla_iocb.c

d74595278 Michael Hernandez  2016-12-12  1946  	/* Only process protection or >16 cdb in this routine */
d74595278 Michael Hernandez  2016-12-12  1947  	if (scsi_get_prot_op(cmd) == SCSI_PROT_NORMAL) {
d74595278 Michael Hernandez  2016-12-12  1948  		if (cmd->cmd_len <= 16)
d74595278 Michael Hernandez  2016-12-12  1949  			return qla2xxx_start_scsi_mq(sp);
d74595278 Michael Hernandez  2016-12-12  1950  	}
d74595278 Michael Hernandez  2016-12-12  1951
4a35d7202 Johannes Thumshirn 2017-06-22 @1952  	spin_lock_irqsave(&qpair->qp_lock, flags);
4a35d7202 Johannes Thumshirn 2017-06-22  1953
d74595278 Michael Hernandez  2016-12-12  1954  	/* Setup qpair pointers */
d74595278 Michael Hernandez  2016-12-12  1955  	rsp = qpair->rsp;
d74595278 Michael Hernandez  2016-12-12  1956  	req = qpair->req;
d74595278 Michael Hernandez  2016-12-12  1957
d74595278 Michael Hernandez  2016-12-12  1958  	/* So we know we haven't pci_map'ed anything yet */
d74595278 Michael Hernandez  2016-12-12  1959  	tot_dsds = 0;
d74595278 Michael Hernandez  2016-12-12  1960
d74595278 Michael Hernandez  2016-12-12  1961  	/* Send marker if required */
d74595278 Michael Hernandez  2016-12-12  1962  	if (vha->marker_needed != 0) {
d74595278 Michael Hernandez  2016-12-12  1963  		if (qla2x00_marker(vha, req, rsp, 0, 0, MK_SYNC_ALL) !=
d74595278 Michael Hernandez  2016-12-12  1964  		    QLA_SUCCESS)
d74595278 Michael Hernandez  2016-12-12 @1965  			return QLA_FUNCTION_FAILED;
d74595278 Michael Hernandez  2016-12-12  1966  		vha->marker_needed = 0;
d74595278 Michael Hernandez  2016-12-12  1967  	}
d74595278 Michael Hernandez  2016-12-12  1968

:::::: The code at line 1965 was first introduced by commit
:::::: d74595278f4ab192af66d9e60a9087464638beee scsi: qla2xxx: Add multiple queue pair functionality.

:::::: TO: Michael Hernandez <michael.hernandez@xxxxxxxxxx>
:::::: CC: Martin K. Petersen <martin.petersen@xxxxxxxxxx>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation