Re: [PATCHv2 1/2] ibmvscsi: Abort init sequence during error recovery
On 05/27/2014 08:55 AM, Nathan Fontenot wrote: On 05/23/2014 10:52 AM, Brian King wrote: If a CRQ reset is triggered for some reason while in the middle of performing VSCSI adapter initialization, we don't want to call the done function for the initialization MAD commands as this will only result in two threads attempting initialization at the same time, resulting in failures. Signed-off-by: Brian King brk...@linux.vnet.ibm.com Acked-by: Nathan Fontenot nf...@linux.vnet.ibm.com --- drivers/scsi/ibmvscsi/ibmvscsi.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -puN drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix drivers/scsi/ibmvscsi/ibmvscsi.c --- linux/drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix 2014-05-23 10:36:04.0 -0500 +++ linux-bjking1/drivers/scsi/ibmvscsi/ibmvscsi.c 2014-05-23 10:50:03.0 -0500 @@ -797,7 +797,8 @@ static void purge_requests(struct ibmvsc evt-hostdata-dev); if (evt-cmnd_done) evt-cmnd_done(evt-cmnd); -} else if (evt-done) +} else if (evt-done evt-crq.format != VIOSRP_MAD_FORMAT + evt-iu.srp.login_req.opcode != SRP_LOGIN_REQ) evt-done(evt); free_event_struct(evt-hostdata-pool, evt); spin_lock_irqsave(hostdata-host-host_lock, flags); _ Hi James, Are these two patches in your queue? They both fix rather nasty issues that have caused kernel crashes in our testing. Thanks, Brian -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCHv2 1/2] ibmvscsi: Abort init sequence during error recovery
On Fri, 2014-06-13 at 10:30 -0500, Brian King wrote: On 05/27/2014 08:55 AM, Nathan Fontenot wrote: On 05/23/2014 10:52 AM, Brian King wrote: If a CRQ reset is triggered for some reason while in the middle of performing VSCSI adapter initialization, we don't want to call the done function for the initialization MAD commands as this will only result in two threads attempting initialization at the same time, resulting in failures. Signed-off-by: Brian King brk...@linux.vnet.ibm.com Acked-by: Nathan Fontenot nf...@linux.vnet.ibm.com --- drivers/scsi/ibmvscsi/ibmvscsi.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -puN drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix drivers/scsi/ibmvscsi/ibmvscsi.c --- linux/drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix 2014-05-23 10:36:04.0 -0500 +++ linux-bjking1/drivers/scsi/ibmvscsi/ibmvscsi.c 2014-05-23 10:50:03.0 -0500 @@ -797,7 +797,8 @@ static void purge_requests(struct ibmvsc evt-hostdata-dev); if (evt-cmnd_done) evt-cmnd_done(evt-cmnd); - } else if (evt-done) + } else if (evt-done evt-crq.format != VIOSRP_MAD_FORMAT + evt-iu.srp.login_req.opcode != SRP_LOGIN_REQ) evt-done(evt); free_event_struct(evt-hostdata-pool, evt); spin_lock_irqsave(hostdata-host-host_lock, flags); _ Hi James, Are these two patches in your queue? They both fix rather nasty issues that have caused kernel crashes in our testing. Should they be cc'd to stable, or are they recently introduced bugs? James -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCHv2 1/2] ibmvscsi: Abort init sequence during error recovery
On 06/13/2014 10:49 AM, James Bottomley wrote: On Fri, 2014-06-13 at 10:30 -0500, Brian King wrote: On 05/27/2014 08:55 AM, Nathan Fontenot wrote: On 05/23/2014 10:52 AM, Brian King wrote: If a CRQ reset is triggered for some reason while in the middle of performing VSCSI adapter initialization, we don't want to call the done function for the initialization MAD commands as this will only result in two threads attempting initialization at the same time, resulting in failures. Signed-off-by: Brian King brk...@linux.vnet.ibm.com Acked-by: Nathan Fontenot nf...@linux.vnet.ibm.com --- drivers/scsi/ibmvscsi/ibmvscsi.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -puN drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix drivers/scsi/ibmvscsi/ibmvscsi.c --- linux/drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix 2014-05-23 10:36:04.0 -0500 +++ linux-bjking1/drivers/scsi/ibmvscsi/ibmvscsi.c 2014-05-23 10:50:03.0 -0500 @@ -797,7 +797,8 @@ static void purge_requests(struct ibmvsc evt-hostdata-dev); if (evt-cmnd_done) evt-cmnd_done(evt-cmnd); - } else if (evt-done) + } else if (evt-done evt-crq.format != VIOSRP_MAD_FORMAT + evt-iu.srp.login_req.opcode != SRP_LOGIN_REQ) evt-done(evt); free_event_struct(evt-hostdata-pool, evt); spin_lock_irqsave(hostdata-host-host_lock, flags); _ Hi James, Are these two patches in your queue? They both fix rather nasty issues that have caused kernel crashes in our testing. Should they be cc'd to stable, or are they recently introduced bugs? They should probably go to stable as well as they've been around a while. Thanks, Brian -- Brian King Power Linux I/O IBM Linux Technology Center -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCHv2 1/2] ibmvscsi: Abort init sequence during error recovery
On 05/23/2014 10:52 AM, Brian King wrote: If a CRQ reset is triggered for some reason while in the middle of performing VSCSI adapter initialization, we don't want to call the done function for the initialization MAD commands as this will only result in two threads attempting initialization at the same time, resulting in failures. Signed-off-by: Brian King brk...@linux.vnet.ibm.com Acked-by: Nathan Fontenot nf...@linux.vnet.ibm.com --- drivers/scsi/ibmvscsi/ibmvscsi.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -puN drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix drivers/scsi/ibmvscsi/ibmvscsi.c --- linux/drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix 2014-05-23 10:36:04.0 -0500 +++ linux-bjking1/drivers/scsi/ibmvscsi/ibmvscsi.c2014-05-23 10:50:03.0 -0500 @@ -797,7 +797,8 @@ static void purge_requests(struct ibmvsc evt-hostdata-dev); if (evt-cmnd_done) evt-cmnd_done(evt-cmnd); - } else if (evt-done) + } else if (evt-done evt-crq.format != VIOSRP_MAD_FORMAT +evt-iu.srp.login_req.opcode != SRP_LOGIN_REQ) evt-done(evt); free_event_struct(evt-hostdata-pool, evt); spin_lock_irqsave(hostdata-host-host_lock, flags); _ -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCHv2 1/2] ibmvscsi: Abort init sequence during error recovery
On 05/23/14 17:52, Brian King wrote: If a CRQ reset is triggered for some reason while in the middle of performing VSCSI adapter initialization, we don't want to call the done function for the initialization MAD commands as this will only result in two threads attempting initialization at the same time, resulting in failures. Signed-off-by: Brian King brk...@linux.vnet.ibm.com --- drivers/scsi/ibmvscsi/ibmvscsi.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -puN drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix drivers/scsi/ibmvscsi/ibmvscsi.c --- linux/drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix 2014-05-23 10:36:04.0 -0500 +++ linux-bjking1/drivers/scsi/ibmvscsi/ibmvscsi.c2014-05-23 10:50:03.0 -0500 @@ -797,7 +797,8 @@ static void purge_requests(struct ibmvsc evt-hostdata-dev); if (evt-cmnd_done) evt-cmnd_done(evt-cmnd); - } else if (evt-done) + } else if (evt-done evt-crq.format != VIOSRP_MAD_FORMAT +evt-iu.srp.login_req.opcode != SRP_LOGIN_REQ) evt-done(evt); free_event_struct(evt-hostdata-pool, evt); spin_lock_irqsave(hostdata-host-host_lock, flags); Hello Brian, Does this mean that the SRP-over-CRQ protocol is still in use for communication between partitions ? Had you already noticed the tgt infrastructure removal patch series (http://thread.gmane.org/gmane.linux.scsi/90119) ? Feedback would be appreciated. Thanks, Bart. -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCHv2 1/2] ibmvscsi: Abort init sequence during error recovery
On 05/23/2014 11:03 AM, Bart Van Assche wrote: On 05/23/14 17:52, Brian King wrote: If a CRQ reset is triggered for some reason while in the middle of performing VSCSI adapter initialization, we don't want to call the done function for the initialization MAD commands as this will only result in two threads attempting initialization at the same time, resulting in failures. Signed-off-by: Brian King brk...@linux.vnet.ibm.com --- drivers/scsi/ibmvscsi/ibmvscsi.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -puN drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix drivers/scsi/ibmvscsi/ibmvscsi.c --- linux/drivers/scsi/ibmvscsi/ibmvscsi.c~ibmvscsi_purge_fix 2014-05-23 10:36:04.0 -0500 +++ linux-bjking1/drivers/scsi/ibmvscsi/ibmvscsi.c 2014-05-23 10:50:03.0 -0500 @@ -797,7 +797,8 @@ static void purge_requests(struct ibmvsc evt-hostdata-dev); if (evt-cmnd_done) evt-cmnd_done(evt-cmnd); -} else if (evt-done) +} else if (evt-done evt-crq.format != VIOSRP_MAD_FORMAT + evt-iu.srp.login_req.opcode != SRP_LOGIN_REQ) evt-done(evt); free_event_struct(evt-hostdata-pool, evt); spin_lock_irqsave(hostdata-host-host_lock, flags); Hello Brian, Does this mean that the SRP-over-CRQ protocol is still in use for communication between partitions ? Had you already noticed the tgt infrastructure removal patch series (http://thread.gmane.org/gmane.linux.scsi/90119) ? Feedback would be appreciated. It is indeed still supported. I've sent a response on that other thread with clarification. -Brian -- Brian King Power Linux I/O IBM Linux Technology Center -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html