| Message ID | 1494348626-62378-1-git-send-email-bryantly@linux.vnet.ibm.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Tue, 2017-05-09 at 11:50 -0500, Bryant G. Ly wrote: > This patch is dependent on: > 'commit 25e78531268e ("ibmvscsis: Do not send aborted task response")' > This patch cleans up some pointers after usage. > > Signed-off-by: Bryant G. Ly <bryantly@linux.vnet.ibm.com> > Reviewed-by: Michael Cyr <mikecyr@linux.vnet.ibm.com> > Cc: <stable@vger.kernel.org> # v4.8+ > --- > drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c | 3 +++ > 1 file changed, 3 insertions(+) Applied, with a more verbose commit message. Thanks for splitting this out into an incremental patch. -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 5/9/17 10:45 PM, Nicholas A. Bellinger wrote: > On Tue, 2017-05-09 at 11:50 -0500, Bryant G. Ly wrote: >> This patch is dependent on: >> 'commit 25e78531268e ("ibmvscsis: Do not send aborted task response")' >> This patch cleans up some pointers after usage. >> >> Signed-off-by: Bryant G. Ly <bryantly@linux.vnet.ibm.com> >> Reviewed-by: Michael Cyr <mikecyr@linux.vnet.ibm.com> >> Cc: <stable@vger.kernel.org> # v4.8+ >> --- >> drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c | 3 +++ >> 1 file changed, 3 insertions(+) > Applied, with a more verbose commit message. > > Thanks for splitting this out into an incremental patch. > No problem, did you end up forgetting it once you reverted Bart's VERIFY/WRITE VERIFY patch? It seems to be missing from your for-next now. -Bryant -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, 2017-05-11 at 10:40 -0500, Bryant G. Ly wrote: > On 5/9/17 10:45 PM, Nicholas A. Bellinger wrote: > > > On Tue, 2017-05-09 at 11:50 -0500, Bryant G. Ly wrote: > >> This patch is dependent on: > >> 'commit 25e78531268e ("ibmvscsis: Do not send aborted task response")' > >> This patch cleans up some pointers after usage. > >> > >> Signed-off-by: Bryant G. Ly <bryantly@linux.vnet.ibm.com> > >> Reviewed-by: Michael Cyr <mikecyr@linux.vnet.ibm.com> > >> Cc: <stable@vger.kernel.org> # v4.8+ > >> --- > >> drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c | 3 +++ > >> 1 file changed, 3 insertions(+) > > Applied, with a more verbose commit message. > > > > Thanks for splitting this out into an incremental patch. > > > No problem, did you end up forgetting it once you reverted Bart's VERIFY/WRITE VERIFY patch? > > It seems to be missing from your for-next now. > Yep, going to include it post -rc1 with your latest patch. -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c b/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c index d390325..ee64241 100644 --- a/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c +++ b/drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c @@ -1170,6 +1170,8 @@ static struct ibmvscsis_cmd *ibmvscsis_get_free_cmd(struct scsi_info *vscsi) cmd = list_first_entry_or_null(&vscsi->free_cmd, struct ibmvscsis_cmd, list); if (cmd) { + if (cmd->abort_cmd) + cmd->abort_cmd = NULL; cmd->flags &= ~(DELAY_SEND); list_del(&cmd->list); cmd->iue = iue; @@ -1774,6 +1776,7 @@ static void ibmvscsis_send_messages(struct scsi_info *vscsi) if (cmd->abort_cmd) { retry = true; cmd->abort_cmd->flags &= ~(DELAY_SEND); + cmd->abort_cmd = NULL; } /*
This patch is dependent on: 'commit 25e78531268e ("ibmvscsis: Do not send aborted task response")' This patch cleans up some pointers after usage. Signed-off-by: Bryant G. Ly <bryantly@linux.vnet.ibm.com> Reviewed-by: Michael Cyr <mikecyr@linux.vnet.ibm.com> Cc: <stable@vger.kernel.org> # v4.8+ --- drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c | 3 +++ 1 file changed, 3 insertions(+)