Re: [PATCH 3/3] scsi:stex.c Add S3/S4 support

2015-09-03 Thread Johannes Thumshirn
Charles Chiou writes: > From f442518879f8f41d103b684046d912eca13844e7 Mon Sep 17 00:00:00 2001 > From: Charles > Date: Wed, 2 Sep 2015 20:54:45 +0800 > Subject: [PATCH 3/3] scsi:stex.c Add S3/S4 support > > Add S3/S4 support, add .suspend and

Re: [v2 PATCH 2/3] scsi:stex.c Add hotplug support

2015-09-03 Thread Johannes Thumshirn
Charles Chiou writes: > From 60e14c245c18cbe0300cfa244334e2850a52a381 Mon Sep 17 00:00:00 2001 > From: Charles > Date: Wed, 2 Sep 2015 20:48:55 +0800 > Subject: [PATCH 2/3] scsi:stex.c Add hotplug support > > 1. Add hotplug support. Pegasus

Re: [v2 PATCH 1/3] scsi:stex.c Support to Pegasus series.

2015-09-03 Thread Johannes Thumshirn
Charles Chiou writes: > From 9d7973dfa05a7785d0eb1e9bcfb0fb6d2c493209 Mon Sep 17 00:00:00 2001 > From: Charles > Date: Wed, 2 Sep 2015 20:41:56 +0800 > Subject: [PATCH 1/3] scsi:stex.c Support to Pegasus series. > > Pegasus is a high

Re: [PATCH 2/2] bfa: Fix incorrect de-reference of pointer

2015-09-03 Thread Ewan Milne
On Thu, 2015-08-13 at 06:41 -0400, anil.gurumur...@qlogic.com wrote: > From: Anil Gurumurthy > > Signed-off-by: Anil Gurumurthy > Tested-by: Sudarsana Kalluru > --- > drivers/scsi/bfa/bfa_ioc.c |2 +- >

Re: [PATCH 1/2] bfa: Fix indentation

2015-09-03 Thread Ewan Milne
On Thu, 2015-08-13 at 06:41 -0400, anil.gurumur...@qlogic.com wrote: > From: Anil Gurumurthy > > Signed-off-by: Anil Gurumurthy > Tested-by : Sudarasana Kalluru > --- > drivers/scsi/bfa/bfa_ioc.c | 22

[PATCH v2] scsi: introduce short_inquiry flag for broken host adapters

2015-09-03 Thread Vitaly Kuznetsov
Some host adapters (e.g. Hyper-V storvsc) are known for not respecting the SPC-2/3/4 requirement for 'INQUIRY data (see table ...) shall contain at least 36 bytes'. As a result we get tons on 'scsi 0:7:1:1: scsi scan: INQUIRY result too short (5), using 36' messages on console. This can be

[PATCH RESEND] libiscsi: Fix iscsi_check_transport_timeouts possible infinite loop

2015-09-03 Thread Sagi Grimberg
From: Ariel Nahum Connection last_ping is not being updated when iscsi_send_nopout fails. Not updating the last_ping will cause firing a timer to a past time (last_ping + ping_tmo < current_time) which triggers an infinite loop of iscsi_check_transport_timeouts() and hogs

Re: [PATCHv3] Update scsi host to use ida for host number

2015-09-03 Thread Lee Duncan
On 09/01/2015 11:48 PM, Christoph Hellwig wrote: > On Tue, Sep 01, 2015 at 05:03:28PM -0700, Lee Duncan wrote: >> +static int host_get_index(int *index) >> +{ >> +int error = -ENOMEM; >> + >> +do { >> +if (!ida_pre_get(_index_ida, GFP_KERNEL)) >> +break; >>

Re: [Patch] scsi_ioctl: support persistent reserve commands through ioctl for non-root user.

2015-09-03 Thread Lee Duncan
On 08/01/2015 02:51 AM, jiang.bi...@zte.com.cn wrote: > scsi_ioctl: support persistent reserve commands through ioctl for > non-root user. > > Scsi persistent reserve commands need to be used for non-root user in > many scenarios. > EPERM error will be returned by sg_io() when

Re: [PATCH RESEND] scsi: Fix sense information setting in fixed sized format

2015-09-03 Thread Ewan Milne
On Thu, 2015-09-03 at 19:49 +0300, Sagi Grimberg wrote: > In fixed size sense format the information field is a four byte > field. > > Signed-off-by: Sagi Grimberg > Reviewed-by: Martin K. Petersen > Reviewed-by: Bart Van Assche

[PATCH RESEND] scsi: Fix sense information setting in fixed sized format

2015-09-03 Thread Sagi Grimberg
In fixed size sense format the information field is a four byte field. Signed-off-by: Sagi Grimberg Reviewed-by: Martin K. Petersen Reviewed-by: Bart Van Assche Reviewed-by: Christoph Hellwig ---

Re: [PATCH 1/3] scsi:stex.c Support to Pegasus series.

2015-09-03 Thread Johannes Thumshirn
Charles Chiou writes: > From 63f806abc43237285a918f559ce6e5e410d90d85 Mon Sep 17 00:00:00 2001 > From: Charles > Date: Wed, 2 Sep 2015 20:41:56 +0800 > Subject: [PATCH 1/3] scsi:stex.c Support to Pegasus series. > > Pegasus is a high

Re: [PATCH 3/3] scsi:stex.c Add S3/S4 support

2015-09-03 Thread Johannes Thumshirn
Charles Chiou writes: > From 7d98f8c500de452277e2700a950b23bf4685ed64 Mon Sep 17 00:00:00 2001 > From: Charles > Date: Wed, 2 Sep 2015 20:54:45 +0800 > Subject: [PATCH 3/3] scsi:stex.c Add S3/S4 support > > Add S3/S4 support, add .suspend and

[PATCH] target: Attach EXTENDED_COPY local I/O descriptors to xcopy_pt_sess

2015-09-03 Thread Nicholas A. Bellinger
From: Nicholas Bellinger This patch is a >= v4.1 regression bug-fix where control CDB emulation logic in commit 38b57f82 now expects a se_cmd->se_sess pointer to exist when determining T10-PI support is to be exposed for initiator host ports. To address this bug, go ahead

Re: [PATCH 06/23] scsi_dh_alua: fixup description of stpg_endio()

2015-09-03 Thread Martin K. Petersen
> "Hannes" == Hannes Reinecke writes: Hannes> Fixup copy-and-paste error in the description of stpg_endio(). Reviewed-by: Martin K. Petersen -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line

Re: [PATCH 07/23] scsi: remove scsi_show_sense_hdr()

2015-09-03 Thread Martin K. Petersen
> "Hannes" == Hannes Reinecke writes: Hannes> Last caller is gone, so remove it. Reviewed-by: Martin K. Petersen -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the

Re: [PATCH 08/23] scsi_dh_alua: use flag for RTPG extended header

2015-09-03 Thread Martin K. Petersen
> "Hannes" == Hannes Reinecke writes: Hannes> We should be using a flag when RTPG extended header is not Hannes> supported, that saves us sending RTPG twice for older arrays. Reviewed-by: Martin K. Petersen -- Martin K. Petersen Oracle Linux

Re: [PATCH 09/23] scsi_dh_alua: use unaligned access macros

2015-09-03 Thread Martin K. Petersen
> "Hannes" == Hannes Reinecke writes: Hannes> Use 'get_unaligned_XX' and 'put_unaligned_XX' instead of Hannes> open-coding it. Reviewed-by: Martin K. Petersen -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list:

Re: [PATCH 10/23] scsi_dh_alua: Pass buffer as function argument

2015-09-03 Thread Martin K. Petersen
> "Hannes" == Hannes Reinecke writes: Hannes> Pass in the buffer as a function argument for submit_rtpg(). Reviewed-by: Martin K. Petersen -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line

Re: [PATCH v3 6/10] megaraid_sas : Chip reset if driver fail to bring ioc ready

2015-09-03 Thread Martin K. Petersen
> "Sumit" == sumit saxena writes: Sumit> This patch will try to do chip reset from driver load time. Sumit> Driver load time, if firmware is not comming to ready state, Sumit> driver try chip reset calling adp_reset() callback. For fusion Sumit> adapter, that call

[v2 PATCH 1/3] scsi:stex.c Support to Pegasus series.

2015-09-03 Thread Charles Chiou
From 9d7973dfa05a7785d0eb1e9bcfb0fb6d2c493209 Mon Sep 17 00:00:00 2001 From: Charles Date: Wed, 2 Sep 2015 20:41:56 +0800 Subject: [PATCH 1/3] scsi:stex.c Support to Pegasus series. Pegasus is a high performace hardware RAID solution designed to unleash the raw

[v2 PATCH 2/3] scsi:stex.c Add hotplug support

2015-09-03 Thread Charles Chiou
From 60e14c245c18cbe0300cfa244334e2850a52a381 Mon Sep 17 00:00:00 2001 From: Charles Date: Wed, 2 Sep 2015 20:48:55 +0800 Subject: [PATCH 2/3] scsi:stex.c Add hotplug support 1. Add hotplug support. Pegasus support surprise removal. To this end, I use

Re: [PATCH v3 9/10] megaraid_sas : Code refactor for use of requestorId

2015-09-03 Thread Martin K. Petersen
> "Sumit" == sumit saxena writes: Sumit> Some of the code changes was proposed by David Binderman Removed Sumit> redudant check of requestorId. Redundant condition: Sumit> instance.requestorId. Check for plasma firmware 1.11 or new Sumit> restructured to support

Re: [PATCH 01/23] scsi_dh_alua: Disable ALUA handling for non-disk devices

2015-09-03 Thread Martin K. Petersen
> "Hannes" == Hannes Reinecke writes: Hannes> Non-disk devices might support ALUA, but the firmware Hannes> implementation is untested and frequently broken. As we're Hannes> don't actually need it disable ALUA support for non-disk device Hannes> for now. Reviewed-by: Martin

Re: [PATCH v3 10/10] megaraid_sas : Version upgrade

2015-09-03 Thread Martin K. Petersen
> "Sumit" == sumit saxena writes: Reviewed-by: Martin K. Petersen -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to

Re: [PATCH v3 8/10] megaraid_sas : Fix validHandles check in io path

2015-09-03 Thread Martin K. Petersen
> "Sumit" == sumit saxena writes: Sumit> Syncro firmware supports round robin IO switch on dual path. For Sumit> this driver use validHandles as a check for dual path. This check Sumit> suppose to be > 1 (not > 2) Sumit> Without this patch, earlier driver code

Re: [PATCH v3 7/10] megaraid_sas : Print critical fw event message

2015-09-03 Thread Martin K. Petersen
> "Sumit" == sumit saxena writes: Sumit> Print firmware event in readable format. This will help user to Sumit> track any critical firmware event without any application Sumit> support. Reviewed-by: Martin K. Petersen -- Martin K.

Re: [PATCH 02/23] scsi_dh_alua: Use vpd_pg83 information

2015-09-03 Thread Martin K. Petersen
> "Hannes" == Hannes Reinecke writes: Hannes> The SCSI device now has the VPD page 0x83 information attached, Hannes> so there is no need to query it again. Reviewed-by: Martin K. Petersen -- Martin K. Petersen Oracle Linux Engineering -- To

Re: [PATCH 03/23] scsi_dh_alua: improved logging

2015-09-03 Thread Martin K. Petersen
> "Hannes" == Hannes Reinecke writes: Hannes> Issue different logging messages if ALUA is not supported or the Hannes> TPGS setting is invalid. Reviewed-by: Martin K. Petersen -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe