RE: [patch] hpsa: fix an sprintf() overflow in the reset handler

2015-06-18 Thread Don Brace
-Original Message- From: Dan Carpenter [mailto:dan.carpen...@oracle.com] Sent: Thursday, June 04, 2015 9:48 AM To: Don Brace Cc: James E.J. Bottomley; iss_storage...@hp.com; dl Team ESD Storage Dev Support; linux-scsi@vger.kernel.org; kernel-janit...@vger.kernel.org Subject: [patch]

Re: [PATCH 7/8] target_core_alua: disallow READ_CAPACITY when in standby

2015-06-18 Thread Chris Boot
On 18/06/15 10:43, Hannes Reinecke wrote: Strictly speaking SPC doesn't require READ CAPACITY and friends to be supported while in the port is in standby. Hi Hannes, I'd really rather this didn't go away. Yes, strictly speaking SPC doesn't require these commands but Linux in practice does, and

Re: [PATCH 17/20 v1] [SCSI] mpt3sas: Use alloc_ordered_workqueue() API instead of create_singlethread_workqueue() API

2015-06-18 Thread Joe Lawrence
On 06/16/2015 01:37 AM, Sreekanth Reddy wrote: Created a thread using alloc_ordered_workqueue() API in order to process the works from firmware Work-queue sequentially instead of create_singlethread_workqueue() API. Changes in v1: No need to check for backport compatibility in the

RE: [patch] hpsa: fix an sprintf() overflow in the reset handler

2015-06-18 Thread Don Brace
-Original Message- From: walter harms [mailto:wha...@bfs.de] Sent: Thursday, June 04, 2015 10:23 AM To: Dan Carpenter Cc: Don Brace; James E.J. Bottomley; iss_storage...@hp.com; dl Team ESD Storage Dev Support; linux-scsi@vger.kernel.org; kernel- janit...@vger.kernel.org Subject:

some question about scsi cdb security code

2015-06-18 Thread yoma sophian
hi all: is so far SCSI support security cdb out/in commands? if YES, how could I trigger these 2 scsi commands? Could I trigger these commands by usb devices? appreciate your help in advance, -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to

Re: [PATCH 17/20 v1] [SCSI] mpt3sas: Use alloc_ordered_workqueue() API instead of create_singlethread_workqueue() API

2015-06-18 Thread Sreekanth Reddy
On Thu, Jun 18, 2015 at 5:40 PM, Joe Lawrence joe.lawre...@stratus.com wrote: On 06/16/2015 01:37 AM, Sreekanth Reddy wrote: Created a thread using alloc_ordered_workqueue() API in order to process the works from firmware Work-queue sequentially instead of create_singlethread_workqueue() API.

[PATCH] Doc: Docbook: Change wikipedia's URL from http to https in scsi.tmpl

2015-06-18 Thread Masanari Iida
Recently wikipedia announced to secure access to the servers. Now all http access re-route to https. Signed-off-by: Masanari Iida standby2...@gmail.com --- Documentation/DocBook/scsi.tmpl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/DocBook/scsi.tmpl

Re: [PATCH 7/8] target_core_alua: disallow READ_CAPACITY when in standby

2015-06-18 Thread Hannes Reinecke
On 06/18/2015 01:40 PM, Chris Boot wrote: On 18/06/15 10:43, Hannes Reinecke wrote: Strictly speaking SPC doesn't require READ CAPACITY and friends to be supported while in the port is in standby. Hi Hannes, I'd really rather this didn't go away. Yes, strictly speaking SPC doesn't

OOPS: While logging out of iSCSI Session when MultiPath is Enabled

2015-06-18 Thread Sony John
Hi All, We had logged in to the same target through two NIC interfaces. There were 2 iSCSI-Session and multipath -ll showed the details fine. The moment we issue logout command on both the session the below kernel panic happens. If multipath is disabled there is no panic and everything works

Re: OOPS: While logging out of iSCSI Session when MultiPath is Enabled

2015-06-18 Thread Bart Van Assche
On 06/18/15 08:50, Sony John wrote: We had logged in to the same target through two NIC interfaces. There were 2 iSCSI-Session and multipath -ll showed the details fine. The moment we issue logout command on both the session the below kernel panic happens. If multipath is disabled there is no

hpsa static checker issues

2015-06-18 Thread Don Brace
Dan, You had noted three concerns covering: mutex_unlock, decode sense data, and a goto label issue, that were detected during your static checker run. I have patches that address these issues. I am thinking that I need to post these three patches linux-scsi. Is that correct? Thanks, Don

RE: [Patch V2 5/9] [SCSI] aacraid: Tune response path if IsFastPath bit set

2015-06-18 Thread Rajinikanth Pandurangan
Hi, Any review comments on this patch. please let us known if any changes are required. Thanks, -Raj P. -Original Message- From: Rajinikanth Pandurangan Sent: Wednesday, June 10, 2015 6:42 PM To: jbottom...@parallels.com; linux-scsi@vger.kernel.org Cc: aacr...@pmc-sierra.com; Harry

Re: optimal io size / custom alignment

2015-06-18 Thread Martin K. Petersen
Tom == Tom Yan tom.t...@gmail.com writes: Tom No I put it in the wrong way. What I meant was sd vs md. For Tom example, couldn't the scsi disk driver bind the value it reads from Tom the VPD to another variable instead of optimal i/o size, so that Tom this value would be exclusively for RAID

Re: [PATCH 17/20 v1] [SCSI] mpt3sas: Use alloc_ordered_workqueue() API instead of create_singlethread_workqueue() API

2015-06-18 Thread Joe Lawrence
On 06/18/2015 09:06 AM, Sreekanth Reddy wrote: On Thu, Jun 18, 2015 at 5:40 PM, Joe Lawrence joe.lawre...@stratus.com wrote: On 06/16/2015 01:37 AM, Sreekanth Reddy wrote: Created a thread using alloc_ordered_workqueue() API in order to process the works from firmware Work-queue sequentially

Re: [PATCH 04/20] [SCSI] mpt3sas: Remove redundancy code while freeing the controller resources.

2015-06-18 Thread Sreekanth Reddy
Hi, Any other review comments on this patch. please let us known if any changes are required. Thanks, Sreekanth On Mon, Jun 15, 2015 at 4:18 PM, Johannes Thumshirn jthumsh...@suse.de wrote: On Mon, Jun 15, 2015 at 03:56:56PM +0530, Sreekanth Reddy wrote: On Fri, Jun 12, 2015 at 6:10 PM,

Re: [PATCH 16/20] [SCSI] mpt3sas: Added support for customer specific branding

2015-06-18 Thread Sreekanth Reddy
Hi, Any review comments on this patch. please let us known if any changes are required. Thanks, Sreekanth On Fri, Jun 12, 2015 at 3:12 PM, Sreekanth Reddy sreekanth.re...@avagotech.com wrote: Added support for below customer specific brandings VendorID DeviceID SubsystemVendor ID

Re: [PATCH 11/20] [SCSI] mpt3sas: Add branding string support for OEM custom HBA

2015-06-18 Thread Sreekanth Reddy
Hi, Any review comments on this patch. please let us known if any changes are required. Thanks, Sreekanth On Fri, Jun 12, 2015 at 3:12 PM, Sreekanth Reddy sreekanth.re...@avagotech.com wrote: Add the following OEM's branding to the mpt3sas driver. VendorID DeviceID SubsystemVendor ID

Re: [PATCH 06/20] [SCSI] mpt3sas: Provides the physical location of sas drives

2015-06-18 Thread Sreekanth Reddy
Hi, Any other review comments on this patch. please let us known if any changes are required. Thanks, Sreekanth On Fri, Jun 12, 2015 at 3:12 PM, Sreekanth Reddy sreekanth.re...@avagotech.com wrote: This Patch will provide more details of the devices such as slot number, enclosure logical id,

Re: [PATCH 01/20] [SCSI] mpt3sas: Added Combined Reply Queue feature to extend up-to 96 MSIX vector support

2015-06-18 Thread Sreekanth Reddy
Hi, Any other review comments on this patch. please let us known if any changes are required. Thanks, Sreekanth On Fri, Jun 12, 2015 at 4:46 PM, Sreekanth Reddy sreekanth.re...@avagotech.com wrote: Thanks Johannes, we will take care of this point in our current on-development mpt2sas/mpt3sas

Re: [PATCH 02/20] [SCSI] mpt3sas: Get IOC_FACTS information using handshake protocol only after HBA card gets into READY or Operational state.

2015-06-18 Thread Sreekanth Reddy
Hi, Any review comments on this patch. please let us known if any changes are required. Thanks, Sreekanth On Mon, Mar 30, 2015 at 7:25 PM, Sreekanth Reddy sreekanth.re...@avagotech.com wrote: Driver initialization fails if driver tries to send IOC facts request message when the IOC is in

Re: [PATCH 04/20] [SCSI] mpt3sas: Remove redundancy code while freeing the controller resources.

2015-06-18 Thread Johannes Thumshirn
On Fri, Jun 12, 2015 at 03:12:16PM +0530, Sreekanth Reddy wrote: Removed the redundancy code while freeing the controller resources. Signed-off-by: Sreekanth Reddy sreekanth.re...@avagotech.com --- drivers/scsi/mpt3sas/mpt3sas_base.c | 57 + 1 file

Re: [PATCH 10/20] [SCSI] mpt3sas: Add branding string support for OEM's HBA

2015-06-18 Thread Johannes Thumshirn
On Fri, Jun 12, 2015 at 03:12:22PM +0530, Sreekanth Reddy wrote: Added the following Dell branding to the mpt3sas driver. VendorID DeviceID SubsystemVendor ID SubsystemDevice ID Dell Branding String 0x10000x0097 0x1028 0x1F46DELL 12Gbps

Re: [PATCH 14/20 v1] [SCSI] mpt3sas: Complete the SCSI command with DID_RESET status for log_info value 0x0x32010081

2015-06-18 Thread Johannes Thumshirn
On Tue, Jun 16, 2015 at 11:05:24AM +0530, Sreekanth Reddy wrote: For any SCSI command, if the driver receives IOC status = SCSI_IOC_TERMINATED and log info = 0x32010081 then that command will be completed with DID_RESET host status. The definition of this log info value is Virtual IO has

[PATCH 2/8] tcm_loop: Add SAS transport topology

2015-06-18 Thread Hannes Reinecke
As tcm_loop emulates a SAS HBA it should not only attach to the SAS transport template but also populate the SAS device hierarchy. Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/target/loopback/tcm_loop.c | 139 - drivers/target/loopback/tcm_loop.h |

[PATCH 6/8] target: Issue Power-On/Reset UA upon LUN instantiation

2015-06-18 Thread Hannes Reinecke
Whenever a LUN is being assigned we should be sending out an Power-On/Reset UA. Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/target/target_core_device.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/drivers/target/target_core_device.c

[PATCH 4/8] tcm_loop: Send I_T_NEXUS_LOSS_OCCURRED UA

2015-06-18 Thread Hannes Reinecke
If the virtual SAS link is set to 'offline' we should be queueing an I_T_NEXUS_LOSS_OCCURRED UA. Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/target/loopback/tcm_loop.c | 5 + drivers/target/target_core_tpg.c| 17 + include/target/target_core_fabric.h | 1 +

[PATCH 3/8] tcm_loop: Remove SAS vestigies

2015-06-18 Thread Hannes Reinecke
tcm_loop is able to emulate several protocols, so remove last vestigies of the SAS protocol. Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/target/loopback/tcm_loop.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/target/loopback/tcm_loop.c

[PATCH 0/8] tcm_loop updates

2015-06-18 Thread Hannes Reinecke
Hi Nic, As tcm_loop claims to be a SAS HBA I thought it a good idea to hook it into the SAS transport class, so that we have the entire (virtual) SAS topology in sysfs now. And even lsscsi is happy: # lsscsi -H -t [10]tcm_loopback sas:0x6001405cc9332c5a # lsscsi -t [10:0:0:0] disk

[PATCH 1/8] tcm_loop: Hook into SAS transport class

2015-06-18 Thread Hannes Reinecke
If tcm_loop emulates a SAS HBA it should hook into the SAS transport class, too. Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/target/loopback/tcm_loop.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/drivers/target/loopback/tcm_loop.c

[PATCH 7/8] target_core_alua: disallow READ_CAPACITY when in standby

2015-06-18 Thread Hannes Reinecke
Strictly speaking SPC doesn't require READ CAPACITY and friends to be supported while in the port is in standby. Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/target/target_core_alua.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/target/target_core_alua.c

[PATCH 5/8] tcm_loop: Rescan SCSI target on transport online

2015-06-18 Thread Hannes Reinecke
Setting the transport status to 'online' needs to rescan the target, as some devices might have been added while the transport has been offline. Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/target/loopback/tcm_loop.c | 9 + 1 file changed, 9 insertions(+) diff --git

[PATCH 8/8] target: display 'write_protect' attribute for demo-mode LUNs

2015-06-18 Thread Hannes Reinecke
When LUNs are mapped with a demo-mode initiator we're missing the 'write_protect' attribute to set a LUN read-only. Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/target/target_core_device.c | 27 ++ drivers/target/target_core_fabric_configfs.c | 42

Re: [PATCH 10/20] [SCSI] mpt3sas: Add branding string support for OEM's HBA

2015-06-18 Thread Sreekanth Reddy
Hi, Any review comments on this patch. please let us known if any changes are required. Thanks, Sreekanth On Mon, Jun 15, 2015 at 5:30 PM, Sreekanth Reddy sreekanth.re...@avagotech.com wrote: On Mon, Jun 15, 2015 at 5:23 PM, Johannes Thumshirn jthumsh...@suse.de wrote: On Mon, Jun 15, 2015