[PATCH] scsi: break from queue depth adjusting loops when device found

2014-07-03 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Don't loop through all the devices even after finding the one we're looking for Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Robert Elliott elli...@hp.com --- drivers/scsi/scsi_error.c |2 ++ 1 files changed

[PATCH 0/5] hpsa a few small updates for early July 2014

2014-07-03 Thread Stephen M. Cameron
Nothing very big here, just a few small updates for now. I still have a giant ball of patches waiting in the wings, but it is unfortunately not quite ready yet. --- Robert Elliott (1): hpsa: do not unconditionally copy sense data Stephen M. Cameron (4): hpsa: remove online devices

[PATCH 1/5] hpsa: remove online devices from offline device list

2014-07-03 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com When devices come on line, they should be removed from the list of offline devices that are monitored. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Scott Teel scott.t...@hp.com Reviewed-by: Joe Handzik

[PATCH 2/5] hpsa: fix bad -ENOMEM return value in hpsa_big_passthru_ioctl

2014-07-03 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com When copy_from_user fails, return -EFAULT, not -ENOMEM Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reported-by: Robert Elliott elli...@hp.com Reviewed-by: Joe Handzik joseph.t.hand...@hp.com Reviewed-by: Scott Teel scott.t

[PATCH 4/5] hpsa: fix 6-byte READ/WRITE with 0 length data xfer

2014-07-03 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com a 6-byte READ/WRITE CDB with a 0 block data transfer really means a 256 block data transfer. The RAID mapping code failed to handle this case. For 10/12/16 byte READ/WRITEs, 0 just means no data should be transferred, and should not trigger

[PATCH 5/5] hpsa: do not unconditionally copy sense data

2014-07-03 Thread Stephen M. Cameron
From: Robert Elliott elli...@hp.com Signed-off-by: Robert Elliott elli...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 16 1 files changed, 8 insertions(+), 8

[PATCH 3/5] hpsa: make hpsa_init_one return -ENOMEM if allocation of h-lockup_detected fails

2014-07-03 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Joe Handzik joseph.t.hand...@hp.com --- drivers/scsi/hpsa.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers

[PATCH 2 01/24] hpsa: add new Smart Array PCI IDs (May 2014)

2014-05-29 Thread Stephen M. Cameron
From: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Scott Teel scott.t...@hp.com Signed-off-by: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Stephen M. Cameron scame...@beardog.cce.hp.com Cc: sta...@vger.kernel.org

[PATCH 2 00/24] Resend of May 2014 patches for hpsa driver

2014-05-29 Thread Stephen M. Cameron
Not a lot of extensive changes in this set, some new PCI IDs, some small bug fixes, quieting some noisy messages, allowing more reply queues, setting irq affinity hints, and some minor locking improvments. Two patches are slightly changed since they were first sent due to reviewer comments and

[PATCH 2 02/24] hpsa: fix missing check of kzalloc return value

2014-05-29 Thread Stephen M. Cameron
From: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Scott Teel scott.t...@hp.com Signed-off-by: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Stephen M. Cameron scame...@beardog.cce.hp.com Cc: sta...@vger.kernel.org

[PATCH 2 09/24] hpsa: allocate reply queues individually

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Now that we can allocate more than 4 reply queues (up to 64) we shouldn't try to make them share the same allocation but should allocate them separately. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Mike Miller

[PATCH 2 08/24] hpsa: choose number of reply queues more intelligently.

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com No sense having 8 or 16 reply queues if you only have 4 cpus, and likewise no sense limiting to 8 reply queues if you have many more cpus. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Mike Miller michael.mil

[PATCH 2 13/24] hpsa: Rearrange start_io to avoid one unlock/lock sequence in main io path

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Joe Handzik joseph.t.hand...@hp.com --- drivers/scsi/hpsa.c | 24 1 files changed, 16 insertions(+), 8 deletions(-) diff --git a/drivers

[PATCH 2 06/24] hpsa: use gcc aligned attribute instead of manually padding structs

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Webb Scales webb.sca...@hp.com --- drivers/scsi/hpsa.c |3 --- drivers/scsi/hpsa_cmd.h | 33 ++--- 2 files changed, 6 insertions

[PATCH 2 15/24] hpsa: kill annoying messages about SSD Smart Path retries

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com There's nothing the user can or should do about these messages, the commands are retried down the normal RAID path, and the messages just flood the logs and sap performance. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com

[PATCH 2 03/24] hpsa: remove unused fields from struct ctlr_info

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com The fields major, max_outstanding, and usage_count of struct ctlr_info were not used for anything. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Mike Miller michael.mil...@canonical.com Reviewed-by: Webb Scales

[PATCH 2 05/24] hpsa: change doorbell reset delay to ten seconds

2014-05-29 Thread Stephen M. Cameron
...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 1df9a8a..2220b32 100644

[PATCH 2 10/24] hpsa: set irq affinity hints to route MSI-X vectors across CPUs

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Mike Miller michael.mil...@canonical.com Reviewed-by: Scott Teel scott.t...@hp.com --- drivers/scsi/hpsa.c | 17 - 1 files changed, 16 insertions

[PATCH 2 04/24] hpsa: allow passthru ioctls to work with bidirectional commands

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Treat the the data direction bits as a bit mask allowing both READ and WRITE at the same time instead of testing for equality to see if it's a exclusively a READ or a WRITE. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com

[PATCH 2 21/24] hpsa: remove messages about volume status VPD inquiry page not supported

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com They are annoying and do not help anyone. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Justin Lindley justin.lind...@hp.com Reviewed-by: Mike Miller michael.mil...@hp.com --- drivers/scsi/hpsa.c | 12

[PATCH 2 19/24] hpsa: fix memory leak in hpsa_hba_mode_enabled

2014-05-29 Thread Stephen M. Cameron
From: Joe Handzik joseph.t.hand...@hp.com And while we're at it fix a magic number Signed-off-by: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |7

[PATCH 2 20/24] hpsa: do not ignore failure of sense controller parameters command

2014-05-29 Thread Stephen M. Cameron
From: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |8 +--- 1 files changed, 5 insertions(+), 3

[PATCH 2 14/24] hpsa: define extended_report_lun_entry data structure

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Joe Handzik joseph.t.hand...@hp.com --- drivers/scsi/hpsa.c | 21 - drivers/scsi/hpsa_cmd.h | 12 +++- 2 files changed, 19

[PATCH 2 24/24] hpsa: fix handling of hpsa_volume_offline return value

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Make return value an int instead of an unsigned char so that we do not lose negative error return values. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Webb Scales webb.sca...@hp.com Reported-by: Dan Carpenter

[PATCH 2 18/24] hpsa: report check condition even if no sense data present for ioaccel2 mode

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com It shouldn't happen that we get a check condition with no sense data, but if it does, we shouldn't just drop the check condition on the floor. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Justin Lindley

[PATCH 2 23/24] hpsa: return -ENOMEM not -1 on kzalloc failure in hpsa_get_device_id

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reported-by: Dan Carpenter dan.carpen...@oracle.com Reviewed-by: Mike Miller michael.mil...@canonical.com --- drivers/scsi/hpsa.c |2 +- 1 files changed, 1 insertions(+), 1

[PATCH 2 16/24] hpsa: fix event filtering to prevent excessive rescans with old firmware

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com CTLR_STATE_CHANGE_EVENT and CTLR_STATE_CHANGE_EVENT_REDUNDANT_CNTRL do not require rescans to be initiated. Current firmware filters out these events already, but some out of date firmware doesn't, so the driver needs to filter them out too

[PATCH 2 22/24] hpsa: fix bad comparison of signed with unsigned in hpsa_update_scsi_devices

2014-05-29 Thread Stephen M. Cameron
M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reported-by: Dan Carpenter dan.carpen...@oracle.com --- drivers/scsi/hpsa.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index

[PATCH 2 12/24] hpsa: avoid unnecessary readl on every command submission

2014-05-29 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com for controllers which support either of the ioaccel transport methods. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reviewed-by: Mike Miller michael.mil...@canonical.com Reviewed-by: Joe Handzik joseph.t.hand...@hp.com

[PATCH 2/3] hpsa: return -ENOMEM not -1 on kzalloc failure in hpsa_get_device_id

2014-05-21 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reported-by: Dan Carpenter dan.carpen...@oracle.com --- drivers/scsi/hpsa.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers

[PATCH 1/3] hpsa: fix bad comparison of signed with unsigned in hpsa_update_scsi_devices

2014-05-21 Thread Stephen M. Cameron
M. Cameron scame...@beardog.cce.hp.com Reported-by: Dan Carpenter dan.carpen...@oracle.com --- drivers/scsi/hpsa.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 654e341..c5b24e6 100644 --- a/drivers/scsi/hpsa.c +++ b

[PATCH 3/3] hpsa: fix handling of hpsa_volume_offline return value

2014-05-21 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Make return value an int instead of an unsigned char so that we do not lose negative error return values. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Reported-by: Dan Carpenter dan.carpen...@oracle.com --- drivers/scsi

[PATCH 1/2] hpsa: fix memory leak in hpsa_hba_mode_enabled

2014-05-15 Thread Stephen M. Cameron
From: Joe Handzik joseph.t.hand...@hp.com And while we're at it fix a magic number Signed-off-by: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |7 ++- drivers/scsi/hpsa.h |1 + 2 files changed, 7 insertions

[PATCH 2/2] hpsa: do not ignore failure of sense controller parameters command

2014-05-15 Thread Stephen M. Cameron
From: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi

[PATCH 01/19] hpsa: add new Smart Array PCI IDs (May 2014)

2014-05-08 Thread Stephen M. Cameron
From: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Scott Teel scott.t...@hp.com Signed-off-by: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Cc: sta...@vger.kernel.org --- drivers/scsi/hpsa.c | 12 1 files changed, 12

[PATCH 12/19] hpsa: avoid unnecessary readl on every command submission

2014-05-08 Thread Stephen M. Cameron
From: root r...@rhel6u2e-s1.ldev.net for controllers which support either of the ioaccel transport methods. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |7 +++ drivers/scsi/hpsa.h | 15 ++- 2 files changed, 21 insertions(+), 1

[PATCH 11/19] hpsa: use per-cpu variable for lockup_detected

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Avoid excessive locking by using per-cpu variable for lockup_detected Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 80 ++- drivers/scsi/hpsa.h |2

[PATCH 14/19] hpsa: define extended_report_lun_entry data structure

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 21 - drivers/scsi/hpsa_cmd.h | 12 +++- 2 files changed, 19 insertions(+), 14 deletions(-) diff --git a/drivers/scsi

[PATCH 09/19] hpsa: allocate reply queues individually

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Now that we can allocate more than 4 reply queues (up to 64) we shouldn't try to make them share the same allocation but should allocate them separately. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c

[PATCH 16/19] hpsa: fix event filtering to prevent excessive rescans with old firmware

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com CTLR_STATE_CHANGE_EVENT and CTLR_STATE_CHANGE_EVENT_REDUNDANT_CNTRL do not require rescans to be initiated. Current firmware filters out these events already, but some out of date firmware doesn't, so the driver needs to filter them out too

[PATCH 15/19] hpsa: kill annoying messages about SSD Smart Path retries

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com There's nothing the user can or should do about these messages, the commands are retried down the normal RAID path, and the messages just flood the logs and sap performance. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com

[PATCH 10/19] hpsa: set irq affinity hints to route MSI-X vectors across CPUs

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 17 - 1 files changed, 16 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 9c44f26..e8090e2

[PATCH 18/19] hpsa: report check condition even if no sense data present for ioaccel2 mode

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com It shouldn't happen that we get a check condition with no sense data, but if it does, we shouldn't just drop the check condition on the floor. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |7

[PATCH 00/19] hpsa driver updates May 8, 2014

2014-05-08 Thread Stephen M. Cameron
Not a lot of extensive changes in this set, some new PCI IDs, some small bug fixes, quieting some noisy messages, allowing more reply queues, setting irq affinity hints, and some minor locking improvments. drivers/scsi/hpsa.c | 266 +--

[PATCH 07/19] hpsa: remove dev_dbg() calls from hot paths

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com They are not completely free of cost when disabled and when enabled emitting debug output for every command submitted produces far too much output to be useful. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi

[PATCH 17/19] hpsa: remove bad unlikely annotation from device list updating code

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 8359884..9d4a0bd 100644

[PATCH 08/19] hpsa: choose number of reply queues more intelligently.

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com No sense having 8 or 16 reply queues if you only have 4 cpus, and likewise no sense limiting to 8 reply queues if you have many more cpus. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |2

[PATCH 04/19] hpsa: allow passthru ioctls to work with bidirectional commands

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Treat the the data direction bits as a bit mask allowing both READ and WRITE at the same time instead of testing for equality to see if it's a exclusively a READ or a WRITE. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com

[PATCH 06/19] hpsa: use gcc aligned attribute instead of manually padding structs

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |3 --- drivers/scsi/hpsa_cmd.h | 33 ++--- 2 files changed, 6 insertions(+), 30 deletions(-) diff --git a/drivers

[PATCH 19/19] hpsa: remove messages about volume status VPD inquiry page not supported

2014-05-08 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com They are annoying and do not help anyone. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 12 +++- 1 files changed, 3 insertions(+), 9 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers

[PATCH] hpsa: fix bug in figure_lunaddrbytes()

2014-03-20 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Depending on whether we use extended or normal CISS_REPORT_LUNS command, the returned data is either 8 or 24 bytes per LUN, and the code needs to take this into account instead of just always assuming 24 bytes per LUN. This would cause

[PATCH] hpsa: update driver version to 3.4.4-1

2014-03-13 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index eb9af2f..9a6e4a2 100644

[PATCH 4/6] hpsa: remove unused struct request from CommandList

2014-02-21 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa_cmd.h |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/hpsa_cmd.h b/drivers/scsi/hpsa_cmd.h index 8026d2e..50388f1

[PATCH 3/6] hpsa: increase the probability of a reported success after a device reset

2014-02-21 Thread Stephen M. Cameron
From: Tomas Henzl the...@redhat.com rc is set in the loop, and it isn't set back to zero anywhere this patch fixes it Signed-off-by: Tomas Henzl the...@redhat.com --- drivers/scsi/hpsa.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/hpsa.c

[PATCH 03/35] hpsa: add support for 'fastpath' i/o

2014-02-18 Thread Stephen M. Cameron
From: Matt Gates matthew.ga...@hp.com For certain i/o's to certain devices (unmasked physical disks) we can bypass the RAID stack firmware and do the i/o to the device directly and it will be faster. Signed-off-by: Matt Gates matthew.ga...@hp.com Signed-off-by: Stephen M. Cameron scame

[PATCH 00/35] hpsa February 2014 driver updates

2014-02-18 Thread Stephen M. Cameron
hpsa: retry certain ioaccel error cases on the RAID path hpsa: update source file copyrights hpsa: add controller base data-at-rest encryption compatibility ioaccel2 Stephen M. Cameron (17): hpsa: only allow REQ_TYPE_FS to use fast path hpsa: add ioaccell mode 1 RAID

[PATCH 04/35] hpsa: only allow REQ_TYPE_FS to use fast path

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com When commands sent down the fast path fail, they must be re-tried down the normal RAID path. We do this by kicking i/o's back to the scsi mid layer with a DID_SOFT_ERROR status, which causes them to be retried. This won't work for SG_IO's

[PATCH 02/35] hpsa: mark last scatter gather element as the last

2014-02-18 Thread Stephen M. Cameron
From: Matt Gates matt.ga...@hp.com This is normally optional, but for SSD Smart Path support (in subsequent patches) it is required. Signed-off-by: Matt Gates matthew.ga...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |8 drivers/scsi

[PATCH 07/35] hpsa: update raid offload status on device rescan

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com When rescanning for logical drives, store information about whather raid offload is enabled for each logical drive, and update the driver's internal record of this. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers

[PATCH 05/35] hpsa: fix task management for mode-1 ioaccell path

2014-02-18 Thread Stephen M. Cameron
michael.mil...@canonical.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 27 +-- 1 files changed, 21 insertions(+), 6 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index f1ebb8c..0377692 100644 --- a/drivers

[PATCH 10/35] hpsa: add hp_ssd_smart_path_enabled sysfs attribute

2014-02-18 Thread Stephen M. Cameron
From: Scott Teel scott.t...@hp.com Signed-off-by: Scott Teel scott.t...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 25 + 1 files changed, 25 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi

[PATCH 19/35] hpsa: add task management for ioaccel mode 2

2014-02-18 Thread Stephen M. Cameron
-by: Mike Miller michael.mil...@canonical.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 166 ++- drivers/scsi/hpsa_cmd.h |7 +- 2 files changed, 167 insertions(+), 6 deletions(-) diff --git a/drivers/scsi/hpsa.c

[PATCH 01/35] hpsa: use extended report luns command for HP SSD SmartPath

2014-02-18 Thread Stephen M. Cameron
i/o's by bypassing the RAID stack code in the controller firmware. Signed-off-by: Matt Gates matthew.ga...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 13 - drivers/scsi/hpsa_cmd.h |1 + 2 files changed, 9 insertions(+), 5

[PATCH 14/35] hpsa: do ioaccel mode 2 resource allocations

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Mike Miller michael.mil...@canonical.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 50 ++ drivers/scsi/hpsa.h |3 +++ 2 files changed

[PATCH 12/35] hpsa: add ioaccel mode 2 structure definitions

2014-02-18 Thread Stephen M. Cameron
From: Mike Miller mi...@beardog.cce.hp.com Signed-off-by: Mike Miller michael.mil...@canonical.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 22 ++ drivers/scsi/hpsa.h |8 drivers/scsi/hpsa_cmd.h | 99

[PATCH 09/35] hpsa: do not rescan controllers known to be locked up

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com * Do not check event bits on locked up controllers to see if they need to be rescanned. * Do not initiate any device rescans on controllers which are known to be locked up. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com

[PATCH 11/35] hpsa: complain if physical or logical aborts are not supported

2014-02-18 Thread Stephen M. Cameron
From: Scott Teel scott.t...@hp.com Signed-off-by: Scott Teel scott.t...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 31ce73f

[PATCH 17/35] hpsa: get ioaccel mode 2 i/o working

2014-02-18 Thread Stephen M. Cameron
From: Scott Teel scott.t...@hp.com Signed-off-by: Scott Teel scott.t...@hp.com Signed-off-by: Joe Handzik joseph.t.hand...@hp.com Signed-off-by: Mike Miller michael.mil...@canonical.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 325

[PATCH 06/35] hpsa: add ioaccell mode 1 RAID offload support.

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com This enables sending i/o's destined for RAID logical drives which can be serviced by a single physical disk down a different, faster i/o path directly to physical drives for certain logical volumes on SSDs bypassing the Smart Array RAID stack

[PATCH 08/35] hpsa: poll controller to detect device change event

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com For shared SAS configurations, hosts need to poll Smart Arrays periodically in order to be able to detect configuration changes such as logical drives being added or removed from remote hosts. A register on the controller indicates when

[PATCH 20/35] hpsa: make device update copy the raid map also

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Otherwise we could wind up using incorrect raid map data, and then very bad things would likely happen. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |3 +++ 1 files changed, 3 insertions(+), 0

[PATCH 21/35] hpsa: complete the ioaccel raidmap code

2014-02-18 Thread Stephen M. Cameron
From: Scott Teel scott.t...@hp.com Load balance across members of a N-way mirror set, and handle the meta-RAID levels: R10, R50, R60. Signed-off-by: Scott Teel scott.t...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 174

[PATCH 23/35] hpsa: rescan devices on ioaccel2 error

2014-02-18 Thread Stephen M. Cameron
will not duplicate on the standard path, so the driver could get stuck in non-ioaccel mode. To avoid that, we allow driver to request a rescan during the next run of the rescan thread. Signed-off-by: Scott Teel scott.t...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c

[PATCH 26/35] hpsa: retry certain ioaccel error cases on the RAID path

2014-02-18 Thread Stephen M. Cameron
From: Scott Teel scott.t...@hp.com Change the handling of HP SSD Smart Path errors with status: 0x02 CHECK CONDITION 0x08 BUSY 0x18 RESERVATION CONFLICT 0x40 TASK ABORTED So that they get retried on the RAID Path. Signed-off-by: Scott Teel scott.t...@hp.com Acked-by: Stephen M. Cameron

[PATCH 16/35] hpsa: initialize controller to perform io accelerator mode 2

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Scott Teel scott.t...@hp.com --- drivers/scsi/hpsa.c | 111 --- drivers/scsi/hpsa.h |1 drivers/scsi

[PATCH 15/35] hpsa: get physical device handles for io accel mode 2 as well as mode 1

2014-02-18 Thread Stephen M. Cameron
From: Mike MIller mi...@beardog.cce.hp.com Signed-off-by: Mike MIller michael.mil...@canonical.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c

[PATCH 32/35] hpsa add sysfs debug switch for raid map debugging messages

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Scott Teel scott.t...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 64 +++ drivers/scsi/hpsa.h |1 + 2 files changed, 65

[PATCH 24/35] hpsa: allow VPD page zero to be queried

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Code was confused and assumed that page zero was not VPD page and all non-zero pages were VPD pages. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 17 + 1 files changed, 9 insertions

[PATCH 30/35] hpsa: only do device rescan for certain events

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Do no rescan on every events -- way too many rescans are triggered if we don't filter the events. Limit rescans to be triggered by the following set of events: * controller state change * enclosure hot plug * physical drive state change

[PATCH 25/35] hpsa: do not inquire for unsupported ioaccel status vpd page

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 45 + drivers/scsi/hpsa_cmd.h |2 ++ 2 files changed, 47 insertions(+), 0 deletions(-) diff --git

[PATCH 18/35] hpsa: teach hpsa_device_reset to do either target or lun reset

2014-02-18 Thread Stephen M. Cameron
From: Scott Teel scott.t...@hp.com Signed-off-by: Scott Teel scott.t...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 10 ++ 1 files changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index

[PATCH 22/35] hpsa: allow user to disable accelerated i/o path

2014-02-18 Thread Stephen M. Cameron
/acciopath_status To check state: cat /sys/class/scsi_host/hostid/acciopath_status Signed-off-by: Scott Teel scott.t...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- Documentation/ABI/testing/sysfs-class-scsi_host | 16 drivers/scsi/hpsa.c

[PATCH 13/35] hpsa: Acknowledge controller events in ioaccell mode 2 as well as mode 1

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |3 ++- drivers/scsi/hpsa_cmd.h |1 + 2 files changed, 3 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c

[PATCH 27/35] hpsa: update source file copyrights

2014-02-18 Thread Stephen M. Cameron
From: Scott Teel scott.t...@hp.com Signed-off-by: Scott Teel scott.t...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c |2 +- drivers/scsi/hpsa.h |2 +- drivers/scsi/hpsa_cmd.h |2 +- 3 files changed, 3 insertions(+), 3 deletions

[PATCH 29/35] hpsa: when switching out of accel mode await only accel command completions

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Don't wait for *all* commands to complete, only for accelerated mode commands. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 22 -- 1 files changed, 16 insertions(+), 6 deletions

[PATCH 31/35] hpsa: improve error messages for driver initiated commands

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com On encountering unexpected error conditions from driver initiated commands, print something useful like CDB and sense data rather than something useless like the kernel virtual address of the command buffer. Signed-off-by: Stephen M. Cameron

[PATCH 28/35] hpsa: add controller base data-at-rest encryption compatibility ioaccel2

2014-02-18 Thread Stephen M. Cameron
encryption flag, data encryption key index, and calculate tweak value from request's logical block address. Signed-off-by: Scott Teel scott.t...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 179

[PATCH 34/35] hpsa: Add support for a few HP Storage controllers

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 9410482..c8de330 100644

[PATCH 35/35] hpsa: fixup MSI-X registration

2014-02-18 Thread Stephen M. Cameron
: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/hpsa.c | 31 +-- 1 files changed, 17 insertions(+), 14 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index c8de330..9f593b9 100644

[PATCH 33/35] pci: add HP/3PAR vendor id to pci_ids.h

2014-02-18 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- include/linux/pci_ids.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h index 97fbecd..0154d9b 100644

[PATCH 02/41] hpsa: revert hide logical drives with format in progress from linux

2014-01-15 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com This relies on a kernel thread which I wish to replace with a work queue based solution. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 50 ++ drivers

[PATCH 03/41] hpsa: use workqueue instead of kernel thread for lockup detection

2014-01-15 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Much simpler and avoids races starting/stopping the thread. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 103 +-- drivers/scsi/hpsa.h |3 + 2

[PATCH 01/41] hpsa: revert bring logical drives online when format completes

2014-01-15 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com This relies on a kernel thread which I wish to remove and replace with a work queue based solution. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 128

[PATCH 05/41] hpsa: enable unit attention reporting

2014-01-15 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com This used to be the default, but at some point the firmware guys changed the default and I failed to notice. Now to get unit attention notifications, you must twiddle a bit indicating you want them. Signed-off-by: Stephen M. Cameron scame

[PATCH 06/41] hpsa: do not require board not ready status after hard reset

2014-01-15 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com Immediately following a hard board reset, There are some mandatory delays during which we must not access the board and during which we might miss the not ready status, therefore it is a mistake to look for and expect to see the not ready

[PATCH 08/41] hpsa: use extended report luns command for HP SSD SmartPath

2014-01-15 Thread Stephen M. Cameron
i/o's by bypassing the RAID stack code in the controller firmware. Signed-off-by: Matt Gates matthew.ga...@hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 13 - drivers/scsi/hpsa_cmd.h |1 + 2 files changed, 9 insertions(+), 5

[PATCH 12/41] hpsa: fix task management for mode-1 ioaccell path

2014-01-15 Thread Stephen M. Cameron
...@beardog.cce.hp.com Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers/scsi/hpsa.c | 27 +-- 1 files changed, 21 insertions(+), 6 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 81545a7..9b4f5ff 100644 --- a/drivers/scsi

[PATCH 07/41] hpsa: allow SCSI mid layer to handle unit attention

2014-01-15 Thread Stephen M. Cameron
From: Matt Gates matthew.ga...@hp.com We were clobbering the SCSI status and setting cmd-result = DID_SOFT_ERROR 16; to get a retry, but better to let the mid layer handle the unit attention. Signed-off-by: Matt Gates matthew.ga...@hp.com Acked-by: Stephen M. Cameron scame...@beardog.cce.hp.com

[PATCH 10/41] hpsa: add support for 'fastpath' i/o

2014-01-15 Thread Stephen M. Cameron
From: Matt Gates matthew.ga...@hp.com For certain i/o's to certain devices (unmasked physical disks) we can bypass the RAID stack firmware and do the i/o to the device directly and it will be faster. Signed-off-by: Matt Gates matthew.ga...@hp.com Signed-off-by: Stephen M. Cameron scame

[PATCH 14/41] hpsa: update raid offload status on device rescan

2014-01-15 Thread Stephen M. Cameron
From: Stephen M. Cameron scame...@beardog.cce.hp.com When rescanning for logical drives, store information about whather raid offload is enabled for each logical drive, and update the driver's internal record of this. Signed-off-by: Stephen M. Cameron scame...@beardog.cce.hp.com --- drivers

  1   2   >