[PATCH 00/21] hpsa updates

2015-10-24 Thread Don Brace
t out scsi_add_device scsi_remove_device calls - add in SAS transport class for HBA devices - add in physical target resets - eliminate fake lun0 enclosures - add polling for PT RAID devices - disable report lun data caching - bumped driver version to match in-box driver --- Don Brace (10):

[PATCH 06/21] hpsa: fix physical target reset

2015-10-24 Thread Don Brace
From: Scott Teel Set reset type in device_reset_handler to do either logical unit reset for logical devices, or physical target reset, for physical devices. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c

[PATCH 04/21] hpsa: correct transfer length for 6 byte read/write commands

2015-10-24 Thread Don Brace
handle block counts of 0. Cleanup block and block count calculations. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 19 ++- 1 file changed, 6 insertions(+), 13 deletions(-) diff --git a

[PATCH 01/21] hpsa: correct calls to dev_printk

2015-10-24 Thread Don Brace
was seeing issues passing level into hpsa_show_dev_msg and then using the level as an arguement to dev_printk. For now, switch to dev_warn. Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 41 - 1 file changed, 24 insertions(+), 17 deletions(-) diff

[PATCH 05/21] hpsa: fix hpsa_adjust_hpsa_scsi_table

2015-10-24 Thread Don Brace
Fix a NULL pointer issue in the driver when devices are removed during a reset. Signed-off-by: Don Brace --- drivers/block/cciss.h |1 + drivers/scsi/hpsa.c | 16 drivers/scsi/hpsa.h |1 + 3 files changed, 18 insertions(+) diff --git a/drivers/block/cciss.h b

[PATCH 09/21] hpsa: simplify check for device exposure

2015-10-24 Thread Don Brace
From: Kevin Barnett remove macros and cleanup device exposure checking Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 25 +++-- drivers/scsi/hpsa.h |6 +- 2 files changed, 12

[PATCH 10/21] hpsa: simplify update scsi devices

2015-10-24 Thread Don Brace
From: Kevin Barnett remove repeated calculation that checks for physical or logical devices. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 23 ++- drivers/scsi/hpsa.h |1 + 2 files

[PATCH 08/21] hpsa: correct ioaccel2 sg chain len

2015-10-24 Thread Don Brace
Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index d42063d..ff34c8e 100644 --- a/drivers

[PATCH 07/21] hpsa: correct check for non-disk devices

2015-10-24 Thread Don Brace
. Change the code to use the structure elements directly since this MACRO is only used in one location. Reported-by: Dan Carpenter Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 28

[PATCH 14/21] hpsa: move scsi_add_device and scsi_remove_device calls to new function

2015-10-24 Thread Don Brace
From: Kevin Barnett preparation for adding the sas transport class Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 62 ++- 1 file changed, 36 insertions

[PATCH 12/21] hpsa: enhance hpsa_get_device_id

2015-10-24 Thread Don Brace
use an index into vpd data for SAS/SATA drives Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 23 --- 1 file changed, 20 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/hpsa.c b

[PATCH 11/21] hpsa: add function is_logical_device

2015-10-24 Thread Don Brace
From: Kevin Barnett simplify checking for logical/physical devices Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a

[PATCH 16/21] hpsa: eliminate fake lun0 enclosures

2015-10-24 Thread Don Brace
, then the target scan would stop, and move to the next target. Lun0 enclosure device was added to prevent sparsely-numbered LUNs from being missed. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hps

[PATCH 13/21] hpsa: refactor hpsa_figure_bus_target_lun

2015-10-24 Thread Don Brace
From: Kevin Barnett setup for sas transport. Need to set the bus and target accordingly. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 25 - drivers/scsi/hpsa.h |5 + 2

[PATCH 17/21] hpsa: add discovery polling for PT RAID devices.

2015-10-24 Thread Don Brace
rom rescan_controller_worker (every 30 seconds). If the data from report_luns is different than last time (binary compare), execute a full rescan via update_scsi_devices. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |

[PATCH 19/21] hpsa: enhance device messages

2015-10-24 Thread Don Brace
Reviewed-by: Justin Lindley Reviewed-by: Scott Teel Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 46 +- 1 file changed, 41 insertions(+), 5 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index

[PATCH 20/21] hpsa: add in sas transport class

2015-10-24 Thread Don Brace
From: Kevin Barnett Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 537 +-- drivers/scsi/hpsa.h | 27 ++ drivers/scsi/hpsa_cmd.h | 14 + 3 files

[PATCH 15/21] External array LUNs must use target and lun numbers assigned by the

2015-10-24 Thread Don Brace
he number of luns from "Report LUNs" in excess of those reported by 'ID controller' are therefore the PTRAID LUNS. We can now remove function is_ext_target, and the 'white list' array of supported model names. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Review

[PATCH 18/21] hpsa: disable report lun data caching

2015-10-24 Thread Don Brace
ignal from the external array to the smart array when lun configuration changes, and thus when driver requests report luns, it may be stale data. Use diag options to turn off RPL data caching. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don

[PATCH 21/21] hpsa: bump the driver version

2015-10-24 Thread Don Brace
Reviewed-by: Justin Lindley Reviewed-by: Scott Teel Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 04799a8..5f74eaa 100644 --- a/drivers/scsi

Re: [PATCH 19/21] hpsa: enhance device messages

2015-10-26 Thread Don Brace
On 10/24/2015 03:19 PM, kbuild test robot wrote: Hi Don, [auto build test ERROR on scsi/for-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base] url: https://github.com/0day-ci/linux/commits/Don-Brace/hpsa-updates/20151025-040012 config: x

Re: [PATCH 01/21] hpsa: correct calls to dev_printk

2015-10-26 Thread Don Brace
On 10/26/2015 07:52 AM, Tomas Henzl wrote: On 24.10.2015 21:52, Don Brace wrote: was seeing issues passing level into hpsa_show_dev_msg and then using the level as an arguement to dev_printk. For now, switch to dev_warn. Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 41

Re: [PATCH 03/21] hpsa: abandon rescans on memory alloaction failures.

2015-10-26 Thread Don Brace
On 10/26/2015 09:24 AM, Tomas Henzl wrote: On 24.10.2015 21:52, Don Brace wrote: Abandon and reschedule rescan process only if device inquiries fail due to mem alloc failures, which are likely to occur for all devices. Otherwise, skip device if inquiry fails for other reasons, and continue

Re: [PATCH 05/21] hpsa: fix hpsa_adjust_hpsa_scsi_table

2015-10-26 Thread Don Brace
On 10/26/2015 09:55 AM, Tomas Henzl wrote: On 24.10.2015 21:52, Don Brace wrote: Fix a NULL pointer issue in the driver when devices are removed during a reset. Signed-off-by: Don Brace --- drivers/block/cciss.h |1 + drivers/scsi/hpsa.c | 16 drivers/scsi

Re: [PATCH] scsi: hpsa: fix multiple issues in path_info_show

2015-10-28 Thread Don Brace
On 10/27/2015 05:16 PM, Rasmus Villemoes wrote: I'm not familiar with this code, but path_info_show() (added in 8270b86243658 "hpsa: add sysfs entry path_info to show box and bay information") seems to be broken in multiple ways. First, there's 817 return snprintf(buf, output_len+1, "%s%s%s%

[PATCH 1 04/25] hpsa: fix null device issues

2015-10-28 Thread Don Brace
Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 25 - 1 file changed, 20 insertions(+), 5 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 6b6e9bb..3fe8a18 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c @@ -1136,7 +1136,7

[PATCH 1 01/25] hpsa: stop zeroing reset_cmds_out and ioaccel_cmds_out during rescan

2015-10-28 Thread Don Brace
pulling the rug out from under the reset handler likewise for ioaccel_cmds_out Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 40669f8..e60c652 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers

[PATCH 1 02/25] hpsa: remove unused hpsa_tag_discard_error_bits

2015-10-28 Thread Don Brace
This function is no longer used. Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index e60c652..864fb03 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c

[PATCH 1 00/25] hpsa updates

2015-10-28 Thread Don Brace
anged driver initiated rescan messages - changed atomic_t to int from ctlr_info.reset_in_progress - added hpsa-stop-zeroing-reset-cmds-out-during-rescan - added hpsa-remove-unused-function - added hpsa-fix-multiple-issues-in-path_info_show --- Don Brace (13): hpsa: stop zeroing reset_cmd

[PATCH 1 05/25] hpsa: allow driver requested rescans

2015-10-28 Thread Don Brace
Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 17 +++-- drivers/scsi/hpsa.h |1 + 2 files changed, 16 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/hpsa.c

[PATCH 1 06/25] hpsa: abandon rescans on memory alloaction failures.

2015-10-28 Thread Don Brace
Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 24 +++- 1 file changed, 19 insertions(+), 5 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index a3f671c..ecc6ada 100644 --- a/drivers/scsi

[PATCH 1 03/25] hpsa: check for null arguments to dev_printk

2015-10-28 Thread Don Brace
Check for NULLs. Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |6 ++ drivers/scsi/hpsa.h |2 +- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 864fb03..6b6e9bb 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi

[PATCH 1 10/25] hpsa: correct check for non-disk devices

2015-10-28 Thread Don Brace
. Change the code to use the structure elements directly since this MACRO is only used in one location. Reported-by: Dan Carpenter Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 28

[PATCH 1 12/25] hpsa: simplify check for device exposure

2015-10-28 Thread Don Brace
From: Kevin Barnett remove macros and cleanup device exposure checking Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 25 +++-- drivers/scsi/hpsa.h |6 +- 2 files changed, 12

[PATCH 1 09/25] hpsa: fix physical target reset

2015-10-28 Thread Don Brace
From: Scott Teel Set reset type in device_reset_handler to do either logical unit reset for logical devices, or physical target reset, for physical devices. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c

[PATCH 1 14/25] hpsa: add function is_logical_device

2015-10-28 Thread Don Brace
From: Kevin Barnett simplify checking for logical/physical devices Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a

[PATCH 1 13/25] hpsa: simplify update scsi devices

2015-10-28 Thread Don Brace
From: Kevin Barnett remove repeated calculation that checks for physical or logical devices. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 23 ++- drivers/scsi/hpsa.h |1 + 2 files

[PATCH 1 11/25] hpsa: correct ioaccel2 sg chain len

2015-10-28 Thread Don Brace
Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 3973fb3..ffc0551 100644 --- a/drivers

[PATCH 1 08/25] hpsa: fix hpsa_adjust_hpsa_scsi_table

2015-10-28 Thread Don Brace
Fix a NULL pointer issue in the driver when devices are removed during a reset. Signed-off-by: Don Brace --- drivers/block/cciss.h |1 + drivers/scsi/hpsa.c | 16 drivers/scsi/hpsa.h |1 + 3 files changed, 18 insertions(+) diff --git a/drivers/block/cciss.h b

[PATCH 1 17/25] hpsa: move scsi_add_device and scsi_remove_device calls to new function

2015-10-28 Thread Don Brace
From: Kevin Barnett preparation for adding the sas transport class Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 65 +++ 1 file changed, 39 insertions

[PATCH 1 16/25] hpsa: refactor hpsa_figure_bus_target_lun

2015-10-28 Thread Don Brace
From: Kevin Barnett setup for sas transport. Need to set the bus and target accordingly. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 25 - drivers/scsi/hpsa.h |5 + 2

[PATCH 1 07/25] hpsa: correct transfer length for 6 byte read/write commands

2015-10-28 Thread Don Brace
handle block counts of 0. Cleanup block and block count calculations. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 19 ++- 1 file changed, 6 insertions(+), 13

[PATCH 1 15/25] hpsa: enhance hpsa_get_device_id

2015-10-28 Thread Don Brace
use an index into vpd data for SAS/SATA drives Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 23 --- 1 file changed, 20 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/hpsa.c b

[PATCH 1 22/25] hpsa: enhance device messages

2015-10-28 Thread Don Brace
Reviewed-by: Justin Lindley Reviewed-by: Scott Teel Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 45 - 1 file changed, 40 insertions(+), 5 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index

[PATCH 1 19/25] hpsa: eliminate fake lun0 enclosures

2015-10-28 Thread Don Brace
, then the target scan would stop, and move to the next target. Lun0 enclosure device was added to prevent sparsely-numbered LUNs from being missed. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hps

[PATCH 1 21/25] hpsa: disable report lun data caching

2015-10-28 Thread Don Brace
ignal from the external array to the smart array when lun configuration changes, and thus when driver requests report luns, it may be stale data. Use diag options to turn off RPL data caching. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don

[PATCH 1 20/25] hpsa: add discovery polling for PT RAID devices.

2015-10-28 Thread Don Brace
rom rescan_controller_worker (every 30 seconds). If the data from report_luns is different than last time (binary compare), execute a full rescan via update_scsi_devices. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |

[PATCH 1 18/25] External array LUNs must use target and lun numbers assigned by the

2015-10-28 Thread Don Brace
he number of luns from "Report LUNs" in excess of those reported by 'ID controller' are therefore the PTRAID LUNS. We can now remove function is_ext_target, and the 'white list' array of supported model names. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Review

[PATCH 1 23/25] hpsa: fix multiple issues in path_info_show

2015-10-28 Thread Don Brace
d time any longer. Using scnprintf ensures that output_len always represents the number of bytes actually written to the buffer, so we'll report the proper amount to the upper layer. Signed-off-by: Rasmus Villemoes Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 38 +

[PATCH 1 25/25] hpsa: bump the driver version

2015-10-28 Thread Don Brace
Reviewed-by: Justin Lindley Reviewed-by: Scott Teel Reviewed-by: Kevin Barnett Reviewed-by: Gerry Morong Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index ca38a00..1b43157

[PATCH 1 24/25] hpsa: add in sas transport class

2015-10-28 Thread Don Brace
From: Kevin Barnett Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 535 +-- drivers/scsi/hpsa.h | 27 ++ drivers/scsi/hpsa_cmd.h | 14 + 3 files

Re: [PATCH 1 02/25] hpsa: remove unused hpsa_tag_discard_error_bits

2015-10-29 Thread Don Brace
On 10/29/2015 09:37 AM, Manoj Kumar wrote: Don: See comment below. - Manoj Kumar On 10/28/2015 5:04 PM, Don Brace wrote: This function is no longer used. diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c +#define HPSA_PERF_ERROR_BITS ((1 << DIRECT_LOOKUP_SHIF

Re: [PATCH 1 09/25] hpsa: fix physical target reset

2015-10-29 Thread Don Brace
On 10/29/2015 09:30 AM, Tomas Henzl wrote: On 28.10.2015 23:05, Don Brace wrote: From: Scott Teel Set reset type in device_reset_handler to do either logical unit reset for logical devices, or physical target reset, for physical devices. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley

Re: [PATCH 1 13/25] hpsa: simplify update scsi devices

2015-10-29 Thread Don Brace
On 10/29/2015 11:43 AM, Matthew R. Ochs wrote: On Oct 28, 2015, at 5:05 PM, Don Brace wrote: From: Kevin Barnett remove repeated calculation that checks for physical or logical devices. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don

Re: [PATCH 1 17/25] hpsa: move scsi_add_device and scsi_remove_device calls to new function

2015-10-29 Thread Don Brace
On 10/29/2015 12:21 PM, Matthew R. Ochs wrote: On Oct 28, 2015, at 5:06 PM, Don Brace wrote: From: Kevin Barnett preparation for adding the sas transport class Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c

Re: [PATCH 1 20/25] hpsa: add discovery polling for PT RAID devices.

2015-10-30 Thread Don Brace
On 10/29/2015 03:59 PM, Matthew R. Ochs wrote: On Oct 29, 2015, at 3:51 PM, Don Brace wrote: On 10/29/2015 03:20 PM, Matthew R. Ochs wrote: On Oct 28, 2015, at 5:06 PM, Don Brace wrote: From: Scott Teel There are problems with getting configuration change notification in pass-through

Re: [PATCH 1 03/25] hpsa: check for null arguments to dev_printk

2015-10-30 Thread Don Brace
On 10/30/2015 02:47 AM, Hannes Reinecke wrote: On 10/28/2015 11:04 PM, Don Brace wrote: Check for NULLs. Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |6 ++ drivers/scsi/hpsa.h |2 +- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/scsi/hpsa.c b

Re: [PATCH 1 06/25] hpsa: abandon rescans on memory alloaction failures.

2015-10-30 Thread Don Brace
On 10/30/2015 02:53 AM, Hannes Reinecke wrote: On 10/28/2015 11:05 PM, Don Brace wrote: Abandon and reschedule rescan process only if device inquiries fail due to mem alloc failures, which are likely to occur for all devices. Otherwise, skip device if inquiry fails for other reasons, and

Re: [PATCH 1 08/25] hpsa: fix hpsa_adjust_hpsa_scsi_table

2015-10-30 Thread Don Brace
On 10/30/2015 02:57 AM, Hannes Reinecke wrote: On 10/28/2015 11:05 PM, Don Brace wrote: Fix a NULL pointer issue in the driver when devices are removed during a reset. Signed-off-by: Don Brace --- drivers/block/cciss.h |1 + drivers/scsi/hpsa.c | 16 drivers

Re: [PATCH 1 15/25] hpsa: enhance hpsa_get_device_id

2015-10-30 Thread Don Brace
On 10/30/2015 03:08 AM, Hannes Reinecke wrote: On 10/28/2015 11:06 PM, Don Brace wrote: use an index into vpd data for SAS/SATA drives Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 23

Re: [PATCH 1 21/25] hpsa: disable report lun data caching

2015-10-30 Thread Don Brace
On 10/30/2015 09:25 AM, Tomas Henzl wrote: On 28.10.2015 23:06, Don Brace wrote: From: Scott Teel When external target arrays are present, disable the firmware's normal behavior of returning a cached copy of the report lun data, and force it to collect new data each time we request a r

Re: [PATCH 1 24/25] hpsa: add in sas transport class

2015-10-30 Thread Don Brace
On 10/30/2015 03:07 PM, Matthew R. Ochs wrote: On Oct 28, 2015, at 5:06 PM, Don Brace wrote: From: Kevin Barnett Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 535

Re: [PATCH 1 22/25] hpsa: enhance device messages

2015-11-02 Thread Don Brace
On 10/30/2015 09:32 AM, Tomas Henzl wrote: RAID_UNKNOWN is used in few other places - raid_level_show for example, raid_level_show handles physical devices by using "N/A", otherwise it displays the RAID level for logical devices. Other functions avoid the use of raid_type when the drive is n

[PATCH v2 00/27] hpsa updates

2015-11-04 Thread Don Brace
rt-class to check return code from hpsa_add_sas_host --- Don Brace (15): hpsa: remove unused parameter hostno hpsa: stop zeroing reset_cmds_out and ioaccel_cmds_out during rescan hpsa: remove unused hpsa_tag_discard_error_bits hpsa: change devtype to unsigned hpsa:

[PATCH v2 01/27] hpsa: remove unused parameter hostno

2015-11-04 Thread Don Brace
This parameter was once used before scan_start was defined but now it is no longer used. Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 30 ++ 1 file changed, 14 insertions(+), 16 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index

[PATCH v2 06/27] hpsa: fix null device issues

2015-11-04 Thread Don Brace
Reviewed-by: Tomas Henzl Reviewed-by: Manoj Kumar Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 22 +- 1 file changed, 17 insertions(+), 5 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index b64913f..4468166 100644 --- a/drivers/scsi/hpsa.c

[PATCH v2 04/27] hpsa: change devtype to unsigned

2015-11-04 Thread Don Brace
This member is used in calls to scsi_device_type. It should be unsigned since the kernel checks for upper bounds and it should never be negative. Suggested-by: Tomas Henzl Suggested-by: Hannes Reinecke Suggested-by: Matthew R. Ochs Signed-off-by: Don Brace --- drivers/scsi/hpsa.h |2

[PATCH v2 03/27] hpsa: remove unused hpsa_tag_discard_error_bits

2015-11-04 Thread Don Brace
This function is no longer used. Reviewed-by: Tomas Henzl Reviewed-by: Hannes Reinecke Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 11 +-- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 4085401..5a996273 100644

[PATCH v2 02/27] hpsa: stop zeroing reset_cmds_out and ioaccel_cmds_out during rescan

2015-11-04 Thread Don Brace
pulling the rug out from under the reset handler likewise for ioaccel_cmds_out Reviewed-by: Hannes Reinecke Reviewed-by: Tomas Henzl Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 815dc7f

[PATCH v2 08/27] hpsa: abandon rescans on memory alloaction failures.

2015-11-04 Thread Don Brace
Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 24 +++- 1 file changed, 19 insertions(+), 5 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index f010b8e..ce0341a 100644 --- a/drivers/scsi

[PATCH v2 05/27] hpsa: check for null arguments to dev_printk

2015-11-04 Thread Don Brace
Check for NULLs. Reviewed-by: Tomas Henzl Reviewed-by: Manoj Kumar Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 5a996273..b64913f 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi

[PATCH v2 09/27] hpsa: correct transfer length for 6 byte read/write commands

2015-11-04 Thread Don Brace
handle block counts of 0. Cleanup block and block count calculations. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Reviewed-by: Hannes Reinecke Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 19 ++- 1 file

[PATCH v2 07/27] hpsa: allow driver requested rescans

2015-11-04 Thread Don Brace
Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Reviewed-by; Hannes Reinecke Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 17 +++-- drivers/scsi/hpsa.h |1 + 2 files changed, 16 insertions(+), 2 deletions

[PATCH v2 11/27] hpsa: fix physical target reset

2015-11-04 Thread Don Brace
Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 34 -- drivers/scsi/hpsa.h |1 + 2 files changed, 29 insertions(+), 6 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 382cb5d..1607b6d 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers

[PATCH v2 10/27] hpsa: fix hpsa_adjust_hpsa_scsi_table

2015-11-04 Thread Don Brace
Fix a NULL pointer issue in the driver when devices are removed during a reset. Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 16 drivers/scsi/hpsa.h |1 + 2 files changed, 17 insertions(+) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index ae9968b

[PATCH v2 18/27] hpsa: refactor hpsa_figure_bus_target_lun

2015-11-04 Thread Don Brace
From: Kevin Barnett setup for sas transport. Need to set the bus and target accordingly. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Hannes Reinecke Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 25 - drivers

[PATCH v2 19/27] hpsa: move scsi_add_device and scsi_remove_device calls to new function

2015-11-04 Thread Don Brace
From: Kevin Barnett preparation for adding the sas transport class Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Reviewed-by: Hannes Reinecke Reviewed-by: Matthew R. Ochs Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 71

[PATCH v2 20/27] hpsa: generalize external arrays

2015-11-04 Thread Don Brace
odel names. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 139 ++- drivers/scsi/hpsa.h |1 drivers/scsi/hpsa_cmd.h | 11 3 files changed, 125 insertion

[PATCH v2 17/27] hpsa: enhance hpsa_get_device_id

2015-11-04 Thread Don Brace
use an index into vpd data for SAS/SATA drives Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Reviewed-by: Matthew R. Ochs Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 23 --- 1 file changed, 20 insertions

[PATCH v2 13/27] hpsa: correct ioaccel2 sg chain len

2015-11-04 Thread Don Brace
Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Reviewed-by: Hannes Reinecke Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers

[PATCH v2 12/27] hpsa: correct check for non-disk devices

2015-11-04 Thread Don Brace
. Change the code to use the structure elements directly since this MACRO is only used in one location. Reported-by: Dan Carpenter Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Reviewed-by: Hannes Reinecke Signed-off-by: Don Brace

[PATCH v2 16/27] hpsa: add function is_logical_device

2015-11-04 Thread Don Brace
From: Kevin Barnett simplify checking for logical/physical devices Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Reviewed-by: Matthew R. Ochs Reviewed-by: Hannes Reinecke Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 19

[PATCH v2 14/27] hpsa: simplify check for device exposure

2015-11-04 Thread Don Brace
From: Kevin Barnett remove macros and cleanup device exposure checking Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Reviewed-by: Hannes Reinecke Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 25

[PATCH v2 15/27] hpsa: simplify update scsi devices

2015-11-04 Thread Don Brace
From: Kevin Barnett remove repeated calculation that checks for physical or logical devices. Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Reviewed-by: Matthew R. Ochs Reviewed-by: Hannes Reinecke Signed-off-by: Don Brace

[PATCH v2 21/27] hpsa: eliminate fake lun0 enclosures

2015-11-04 Thread Don Brace
-by: Tomas Henzl Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 68 --- 1 file changed, 68 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 1a67d63..c843dd2 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c @

[PATCH v2 24/27] hpsa: enhance device messages

2015-11-04 Thread Don Brace
Reviewed-by: Justin Lindley Reviewed-by: Scott Teel Reviewed-by: Kevin Barnett Reviewed-by: Tomas Henzl Reviewed-by: Hannes Reinecke Reviewed-by: Matthew R. Ochs Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 45 - 1 file changed, 40

[PATCH v2 22/27] hpsa: add discovery polling for PT RAID devices.

2015-11-04 Thread Don Brace
chs Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 68 +++ drivers/scsi/hpsa.h |2 ++ 2 files changed, 70 insertions(+) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index c843dd2..acfbbb2 100644 --- a/drivers/scsi/hpsa.c +++ b/driv

[PATCH v2 25/27] hpsa: fix multiple issues in path_info_show

2015-11-04 Thread Don Brace
d time any longer. Using scnprintf ensures that output_len always represents the number of bytes actually written to the buffer, so we'll report the proper amount to the upper layer. Reviewed-by: Hannes Reinecke Reviewed-by: Tomas Henzl Reviewed-by: Matthew R. Ochs Signed-off-by: Rasmus Villem

[PATCH v2 23/27] hpsa: disable report lun data caching

2015-11-04 Thread Don Brace
necke Reviewed-by: Matthew R. Ochs Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 85 +++ drivers/scsi/hpsa_cmd.h |3 ++ 2 files changed, 88 insertions(+) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index acfbbb2..91e7005 1

[PATCH v2 27/27] hpsa: bump the driver version

2015-11-04 Thread Don Brace
Reviewed-by: Justin Lindley Reviewed-by: Scott Teel Reviewed-by: Kevin Barnett Reviewed-by: Gerry Morong Reviewed-by: Hannes Reinecke Reviewed-by: Matthew R. Ochs Signed-off-by: Don Brace --- drivers/scsi/hpsa.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a

[PATCH v2 26/27] hpsa: add in sas transport class

2015-11-04 Thread Don Brace
From: Kevin Barnett Reviewed-by: Scott Teel Reviewed-by: Justin Lindley Reviewed-by: Kevin Barnett Reviewed-by: Hannes Reinecke Reviewed-by: Tomas Henzl Reviewed-by: Matthew R. Ochs Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 550

Re: [PATCH v2 20/27] hpsa: generalize external arrays

2015-11-05 Thread Don Brace
On 11/05/2015 10:23 AM, Matthew R. Ochs wrote: On Nov 4, 2015, at 3:51 PM, Don Brace wrote: From: Scott Teel External array LUNs must use target and lun numbers assigned by the external array. So the driver must treat these differently from local LUNs when assigning lun/target. LUN's &

Re: [PATCH v2 00/27] hpsa updates

2015-11-06 Thread Don Brace
Please repost, I'll sign off. Thank-you On 11/06/2015 08:30 AM, Tomas Henzl wrote: Please add to this series the "hpsa: move lockup_detected attribute to host attr" http://www.spinics.net/lists/linux-scsi/msg88129.html. Maybe the maintainer could add it too, or should I repost the patch? Than

Re: [PATCH RESEND] hpsa: move lockup_detected attribute to host attr

2015-11-06 Thread Don Brace
On 11/06/2015 09:24 AM, Tomas Henzl wrote: This patch fixes a 'general protection fault' issue by moving the attribute to where it was likely meant. Signed-off-by: Tomas Henzl Signed-off-by: Don Brace --- drivers/scsi/hpsa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [patch] hpsa: logical vs bitwise AND typo

2015-11-12 Thread Don Brace
goto out; errout: Reviewed-by: Johannes Thumshirn -- 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 Signed-off-by: Don Bra

RE: [PATCH] scsi: hpsa: fix multiple issues in path_info_show

2015-11-18 Thread Don Brace
> -Original Message- > From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi- > ow...@vger.kernel.org] On Behalf Of Rasmus Villemoes > Sent: Saturday, November 14, 2015 9:24 AM > To: Don Brace > Cc: Joe Handzik; James E.J. Bottomley; Kevin Barnett; Scott

Re: [PATCH] scsi: hpsa: select CONFIG_SCSI_SAS_ATTR

2015-11-20 Thread Don Brace
Signed-off-by: Don Brace And...Thanks! On 11/19/2015 04:04 PM, Arnd Bergmann wrote: The hpsa driver recently started using the sas transport class, but it does not ensure that the corresponding code is actually built, which may lead to a link error: drivers/built-in.o: In function

[PATCH 2/3] smartpqi: add ofa support

2018-12-18 Thread Don Brace
resources. - Driver calls SIS/PQI initialization and rescans the device list. - Driver will resume the traffic to RAID/HBA path. Reviewed-by: Murthy Bhat Signed-off-by: Mahesh Rajashekhara Signed-off-by: Don Brace --- drivers/scsi/smartpqi/smartpqi.h | 76 drivers/scsi/smartpqi

[PATCH 1/3] smartpqi: increase fw status register read timeout

2018-12-18 Thread Don Brace
Reviewed-by: Ajish Koshy Reviewed-by: Murthy Bhat Reviewed-by: Dave Carroll Reviewed-by: Kevin Barnett Signed-off-by: Mahesh Rajashekhara Signed-off-by: Don Brace --- drivers/scsi/smartpqi/smartpqi_sis.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/smartpqi

[PATCH 3/3] smartpqi: update driver version

2018-12-18 Thread Don Brace
- need to bump up the driver version because of the OFA patch and the fw status register read timeout patch. Reviewed-by: Gerry Morong Signed-off-by: Don Brace --- drivers/scsi/smartpqi/smartpqi_init.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi

<    1   2   3   4   5   6   7   8   9   10   >