RE: [PATCH 1/1] scsi: storvsc: Support manual scan of FC hosts on Hyper-V

2016-03-29 Thread KY Srinivasan
> -Original Message- > From: James Bottomley [mailto:james.bottom...@hansenpartnership.com] > Sent: Friday, March 18, 2016 3:41 PM > To: KY Srinivasan ; Martin K. Petersen > > Cc: Christoph Hellwig ; gre...@linuxfoundation.org; > linux-ker...@vger.kernel.org; de...@linuxdriverproject.org

Re: tcm_qla2xxx Add SCSI command jammer/discard capabilty to the tcm_qla2xxx module - revision3

2016-03-29 Thread Bart Van Assche
On 03/29/16 07:42, Laurence Oberman wrote: I have been using this jammer functionality to continue testing the SCSI F/C drivers and recovery for over a year now. Any chance you would agree to ack this so I can get it in now. I last posted to the list last March and it was not picked up. I did l

Re: [PATCH] scsi_transport_fc: Unexport scsi_is_fc_vport()

2016-03-29 Thread Martin K. Petersen
> "Bart" == Bart Van Assche writes: Bart> Running the command "git grep -nHw scsi_is_fc_vport" shows that Bart> this function is only called from inside Bart> scsi_transport_fc.c. Hence unexport this function. Applied to 4.7/scsi-queue. Thanks! -- Martin K. Petersen Oracle Linux Engin

Re: [PATCH v1 1/3] Fix to cleanup aborted IO to avoid device being offlined by mid-layer

2016-03-29 Thread Martin K. Petersen
> "Satish" == Satish Kharat writes: Satish, Satish> If an I/O times out and an abort issued by host, if the abort is Satish> successful we need to set scsi status as DID_ABORT. Or else the Satish> mid-layer error handler which looks for this error code, will Satish> offline the device. Also

Re: [PATCH 0/2] scsi_dh_alua fixes for kernel v4.6

2016-03-29 Thread Martin K. Petersen
> "Bart" == Bart Van Assche writes: Bart> Please consider the two patches in this series for inclusion in Bart> kernel v4.6. These patches are what I came up with while retesting Bart> the v4.6-rc1 scsi_dh_alua handler. The actual patches are: Applied to 4.6/scsi-fixes. -- Martin K. Peters

Re: [PATCH] arm64: dts: apm: Fix compatible string for X-Gene 2 SATA controller DTS node

2016-03-29 Thread Duc Dang
On Tue, Mar 29, 2016 at 2:54 AM, Rameshwar Prasad Sahu wrote: > Fix X-Gene SATA controller compatible string for Merlin board. > > Signed-off-by: Rameshwar Prasad Sahu > Acked-by: Suman Tripathi I will queue this patch into xgene-next tree. > --- > arch/arm64/boot/dts/apm/apm-shadowcat.dtsi |

[Bug 60644] MPT2SAS drops all HDDs when under high I/O

2016-03-29 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=60644 rlung...@yahoo.com changed: What|Removed |Added CC||rlung...@yahoo.com --- Comment #49 fr

Re: [PATCH 2/2] scsi_dh_alua: Fix a recently introduced deadlock

2016-03-29 Thread Ewan D. Milne
On Mon, 2016-03-28 at 11:14 -0700, Bart Van Assche wrote: > While retesting the SRP initiator I ran the command "rmmod mlx4_ib" > while I/O was in progress. That command triggers SCSI device removal > indirectly. Avoid that this action triggers the following deadlock: > > =

Re: [PATCH 1/2] Declare local symbols static

2016-03-29 Thread Ewan D. Milne
On Mon, 2016-03-28 at 11:13 -0700, Bart Van Assche wrote: > Avoid that building with W=1 causes gcc to report warnings about > symbols that have not been declared. > > Signed-off-by: Bart Van Assche > Cc: Hannes Reinecke > --- > drivers/scsi/scsi_sysfs.c | 8 +--- > 1 file changed, 5 insert

tcm_qla2xxx Add SCSI command jammer/discard capabilty to the tcm_qla2xxx module - revision3

2016-03-29 Thread Laurence Oberman
Hello Bart, I have been using this jammer functionality to continue testing the SCSI F/C drivers and recovery for over a year now. Any chance you would agree to ack this so I can get it in now. I last posted to the list last March and it was not picked up. I did look into moving this to upper la

Re: [PATCH v1 3/3] Using rport->dd_data to check rport online instead of rport_lookup.

2016-03-29 Thread Ewan D. Milne
On Fri, 2016-03-18 at 11:22 -0700, Satish Kharat wrote: > When issuing I/O we check if rport is online through libfc > rport_lookup() function which needs to be protected by mutex lock > that cannot acquired in I/O context. The change is to use midlayer > remote port’s dd_data which is preserved un

Re: [PATCH v1 1/3] Fix to cleanup aborted IO to avoid device being offlined by mid-layer

2016-03-29 Thread Ewan D. Milne
On Fri, 2016-03-18 at 11:22 -0700, Satish Kharat wrote: > If an I/O times out and an abort issued by host, if the abort is > successful we need to set scsi status as DID_ABORT. Or else the > mid-layer error handler which looks for this error code, will > offline the device. Also if the original I/O

Re: [PATCH v1 2/3] Cleanup the I/O pending with fw and has timed out and is used to issue LUN reset

2016-03-29 Thread Ewan D. Milne
On Fri, 2016-03-18 at 11:22 -0700, Satish Kharat wrote: > In case of LUN reset, the device reset command is issued with one of > the I/Os that has timed out on that LUN. The change is to also return > this I/O with error status set to DID_RESET. In case when the reset > is issued using the sg_reset

RE: [RESEND] [PATCH v11 3/6] added support for DesignWare Controller

2016-03-29 Thread Winkler, Tomas
diff --git a/drivers/scsi/ufs/ufshcd.h b/drivers/scsi/ufs/ufshcd.h index 2570d94..8b0cdf0 100644 --- a/drivers/scsi/ufs/ufshcd.h +++ b/drivers/scsi/ufs/ufshcd.h @@ -262,6 +262,7 @@ struct ufs_pwr_mode_info { * @suspend: called during host controller PM callback * @resume: called during host c

Re: [RESEND] [PATCH v11 2/6] added UFS 2.0 capabilities

2016-03-29 Thread Joao Pinto
On 3/29/2016 12:33 PM, Winkler, Tomas wrote: > Hi Thomas, > > On 3/29/2016 11:41 AM, Winkler, Tomas wrote: >> >> Adding UFS 2.0 support to the UFS core driver. >> >> Signed-off-by: Joao Pinto >> Acked-by: Arnd Bergmann >> Acked-by: Rob Herring >> --- >> Changes v8->v11: >> - Nothing changed (ju

RE: [RESEND] [PATCH v11 2/6] added UFS 2.0 capabilities

2016-03-29 Thread Winkler, Tomas
Hi Thomas, On 3/29/2016 11:41 AM, Winkler, Tomas wrote: > > Adding UFS 2.0 support to the UFS core driver. > > Signed-off-by: Joao Pinto > Acked-by: Arnd Bergmann > Acked-by: Rob Herring > --- > Changes v8->v11: > - Nothing changed (just to keep up with patch set version). > Changes v7->v8: >

Re: [RESEND] [PATCH v11 2/6] added UFS 2.0 capabilities

2016-03-29 Thread Joao Pinto
Hi Thomas, On 3/29/2016 11:41 AM, Winkler, Tomas wrote: > > Adding UFS 2.0 support to the UFS core driver. > > Signed-off-by: Joao Pinto > Acked-by: Arnd Bergmann > Acked-by: Rob Herring > --- > Changes v8->v11: > - Nothing changed (just to keep up with patch set version). > Changes v7->v8: >

[PATCH] arm64: dts: apm: Fix compatible string for X-Gene 2 SATA controller DTS node

2016-03-29 Thread Rameshwar Prasad Sahu
Fix X-Gene SATA controller compatible string for Merlin board. Signed-off-by: Rameshwar Prasad Sahu Acked-by: Suman Tripathi --- arch/arm64/boot/dts/apm/apm-shadowcat.dtsi |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/arm64/boot/dts/apm/apm-shadowcat.dtsi b

RE: [RESEND] [PATCH v11 2/6] added UFS 2.0 capabilities

2016-03-29 Thread Winkler, Tomas
Adding UFS 2.0 support to the UFS core driver. Signed-off-by: Joao Pinto Acked-by: Arnd Bergmann Acked-by: Rob Herring --- Changes v8->v11: - Nothing changed (just to keep up with patch set version). Changes v7->v8: - Added "jedec, ufs-2.0" to the ufschd-platform compatibility strings Changes

[PATCH] sd: fixup capacity calculation for 4k drives

2016-03-29 Thread Hannes Reinecke
in sd_read_capacity() the sdkp->capacity field changes its meaning: after the call to read_capacity_XX() it carries the _unscaled_ values, making the comparison between the original value and the new value always false for drives with a sector size != 512. Signed-off-by: Hannes Reinecke --- driv

Re: [PATCH] scsi: Add intermediate STARGET_REMOVE state to scsi_target_state

2016-03-29 Thread Christoph Hellwig
On Thu, Mar 24, 2016 at 11:42:44AM -0400, Ewan D. Milne wrote: > This looks fine. Do we still need 90a88d6ef (scsi: fix soft lockup in > scsi_remove_target() on module removal) or can that be reverted now, > since the STARGET_REMOVE state will allow the iteration to continue? Yes, that stale poin

Re: [PATCH 1/2] Declare local symbols static

2016-03-29 Thread Christoph Hellwig
On Mon, Mar 28, 2016 at 11:13:16AM -0700, Bart Van Assche wrote: > Avoid that building with W=1 causes gcc to report warnings about > symbols that have not been declared. > > Signed-off-by: Bart Van Assche > Cc: Hannes Reinecke Looks fine, Reviewed-by: Christoph Hellwig -- To unsubscribe from

Re: [PATCH 2/2] scsi_dh_alua: Fix a recently introduced deadlock

2016-03-29 Thread Christoph Hellwig
This indeed looks like the only place where we took pg->lock without disabling irqs. Looks fine, Reviewed-by: Christoph Hellwig -- 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.kerne

Re: [PATCH] scsi: Add intermediate STARGET_REMOVE state to scsi_target_state

2016-03-29 Thread Johannes Thumshirn
On Donnerstag, 24. März 2016 11:42:44 CEST Ewan D. Milne wrote: > On Thu, 2016-03-24 at 10:56 +0100, Johannes Thumshirn wrote: > > The target state machine only knows 'STARGET_DEL', which is set once > > scsi_target_destroy() is called. > > However, by that time the structure is still part of the _