Change ownership of the hpsa driver
from Stephen M. Cameron (Hewlett-Packard)
to Don Brace (PMC-Sierra).
Change ownership of the cciss driver
from Mike Miller (Hewlett-Packard)
to Don Brace (PMC-Sierra).
Reviewed-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
of patches for 3.19 or ack this
now, What would you prefer?
---
Don Brace
don.br...@pmcs.com
--
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
From: Robert Elliott elli...@hp.com
RAID-1ADM is unusable with dev_warn called on every command.
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
Reviewed-by: Stephen M. Cameron stephenmcame...@gmail.com
Reviewed-by: Webb Scales web...@hp.com
---
drivers
Clean up issues reported when running sparse.
Signed-off-by: Don Brace don.br...@pmcs.com
Reviewed-by: Webb Scales webb.sca...@hp.com
---
drivers/scsi/hpsa.c | 29 -
drivers/scsi/hpsa.h |6 +++---
2 files changed, 19 insertions(+), 16 deletions(-)
diff --git
This patch set is based on Linus's tree.
The changes are:
- correct warnings from sparse
- updates for some error handling issues
- general code cleanup
- performance enhancements based on removing spin_locks
---
Don Brace (1):
hpsa: Clean up warnings from sparse.
Nicholas Bellinger
of the
userland ABI.
Signed-off-by: Don Brace don.br...@pmcs.com
Reviewed-by: Webb Scales webb.sca...@hp.com
---
drivers/scsi/hpsa.c | 58 ---
drivers/scsi/hpsa_cmd.h | 18 +++
2 files changed, 42 insertions(+), 34 deletions(-)
diff
From: Nicholas Bellinger n...@linux-iscsi.org
There isn't anything in hpsa that requires the host lock to be held
during queuecommand.
Signed-off-by: Don Brace don.br...@pmcs.com
Signed-off-by: Nicholas Bellinger n...@linux-iscsi.org
Reviewed-by: Stephen M. Cameron stephenmcame...@gmail.com
).
Signed-off-by: Don Brace don.br...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Reviewed-by: Webb Scales web...@hp.com
---
drivers/scsi/hpsa.c | 223 +--
drivers/scsi/hpsa_cmd.h | 14 +--
2 files changed, 107 insertions(+), 130 deletions
From: Webb Scales web...@hp.com
Correct the size calculation of the chained SG block
Signed-off-by: Don Brace don.br...@pmcs.com
Signed-off-by: Webb Scales web...@hp.com
Reviewed-by: Stephen M. Cameron stephenmcame...@gmail.com
Reviewed-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c
and physical sizes separately.
Allocate the memory based on the specific data
structure sizes.
Signed-off-by: Don Brace don.br...@pmcs.com
Reviewed-by: Webb Scales webb.sca...@hp.com
---
drivers/scsi/hpsa.c | 14 +++---
drivers/scsi/hpsa_cmd.h |2 +-
2 files changed, 8 insertions(+), 8
From: Stephen M. Cameron stephenmcame...@gmail.com
We were printing a lot of useless information before ultimately
just passing things up to the SCSI mid layer. Just let the
midlayer handle it without LLD chatter.
Signed-off-by: Don Brace don.br...@pmcs.com
Signed-off-by: Stephen M. Cameron
that after pci_request_regions both in the
kdump code and the normal pci_init code.
Remove the comment summarizing what pci_set_master
does, with the incomplete commentary on the impact of
pci_disable_device.
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
From: Stephen M. Cameron stephenmcame...@gmail.com
Use atomics for commands_outstanding instead of protecting with spin locks.
Signed-off-by: Don Brace don.br...@pmcs.com
Signed-off-by: Stephen M. Cameron stephenmcame...@gmail.com
Reviewed-by: Joe Handzik joseph.t.hand...@hp.com
---
drivers
From: Stephen M. Cameron stephenmcame...@gmail.com
In the case of LUN data changing, the driver will
auto rescan and so it's not even true that action is
required.
Remove action required phrases from warning messages and
replace with description phrases.
Signed-off-by: Don Brace don.br
From: Stephen M. Cameron stephenmcame...@gmail.com
It is already using atomic test_and_set_bit to do the
allocation.
There is some microscopic chance of starvation, but it is
so microscopic that it should never happen in reality.
Signed-off-by: Don Brace don.br...@pmcs.com
Reviewed-by: Webb
From: Stephen M. Cameron scame...@beardog.cce.hp.com
Fix a couple of pci id table mistakes:
Subdevice ID 0x3323 missing from product[] table
(another name for HP Smart Storage 1210m)
Bogus 0x1925 subdevice id removed from hpsa_pci_device_id[] (no such thing.)
Signed-off-by: Don Brace
Performance enhancement. Remove spin_locks from the driver.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 39 +--
drivers/scsi/hpsa.h |3 +--
2 files changed, 6 insertions(+), 36
From: Stephen Cameron stephenmcame...@gmail.com
No need to check whether interrupt pending for MSI(X) and
conversely, no need to check whether MSI(X) interrupts are
being used when checking if interrupts are pending.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br
to parity drives.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Webb Scales web...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 318 +--
drivers/scsi/hpsa.h | 14 ++
drivers/scsi/hpsa_cmd.h | 148
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.h |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/scsi/hpsa.h b/drivers/scsi/hpsa.h
index 1856445..aa6cb0b 100644
--- a/drivers/scsi/hpsa.h
+++ b
, and
prevent excessive rescanning of devices.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 72 ++-
drivers/scsi/hpsa.h |1 +
drivers/scsi/hpsa_cmd.h |1 +
3 files
commands internally
in the driver so that we can remove the locks thta protect the
queue that we will no longer have.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |7 +--
drivers/scsi/hpsa.h |2 ++
2 files changed, 7
From: Webb Scales web...@hp.com
Encapsulate the conditional predicate which tests for legacy controllers
in a separate function and rework the code comments.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Webb Scales web...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
From: Robert Elliott elli...@hp.com
Make the function name more descriptive. We use more than
one interrupt.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 11 ++-
1 file
From: Robert Elliott elli...@hp.com
Partial allocation failure wasn't handled correctly
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |6 +-
1 file changed, 5 insertions(+), 1
From: Stephen Cameron stephenmcame...@gmail.com
Cleanup comments to be more specific. Make messages more
informational.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 21
From: Robert Elliott elli...@hp.com
Return the actual error code instead of a generic error code.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |2 +-
1 file changed, 1 insertion
From: Robert Elliott elli...@hp.com
Enhance error reporting.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 14 ++
drivers/scsi/hpsa.h |1 -
2 files changed, 2 insertions(+), 13 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 70f07af..94a82e3 100644
--- a/drivers/scsi/hpsa.c
-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 306 +--
drivers/scsi/hpsa.h | 17 ---
drivers/scsi/hpsa_cmd.h | 10 --
3 files changed, 38 insertions(+), 295 deletions(-)
diff --git
of the ioaccellerated
paths but which have some sort of error condition are
retried down the normal path.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 126 +++
1 file changed, 67
hpsa_drain_accel_commands to use the reference count as the
authoritative indicator of whether a command is allocated instead of
the h-cmd_pool_bits bitmap.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 109
-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 106 ++-
drivers/scsi/hpsa.h |2 -
2 files changed, 31 insertions(+), 77 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 99c32a0..26e3e5b 100644
--- a/drivers
(relevant pointers would be NULL)
but it is potentially confusing.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git
-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |7 +--
drivers/scsi/hpsa.h |1 +
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index c95a20c..72abcf3 100644
it at the end of
hpsa_scan_start() as well.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Webb Scales web...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 40
1 file changed, 16 insertions(+), 24 deletions(-)
diff
hpsa_wait_for_mode_change_ack
and all their callers check/propagate the result.
More consistency in printing errors and whether
dev_err is used.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 40
From: Stephen Cameron stephenmcame...@gmail.com
Printing the address of the command pointer is of little value, change
to print the CDB.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 29 -
1 file
From: Stephen Cameron stephenmcame...@gmail.com
Acking controller events on controllers that do not support
it can cause such controllers to lock up.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Joe Handzik joseph.t.hand...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
accel commands could still
be in flight, ie. perhaps having just been submitted into
hpsa_scsi_ioaccel_raid_map concurrent with -offload_enabled
having just been set to zero.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |4
1
From: Stephen Cameron stephenmcame...@gmail.com
Performance tweak, avoid unnecessary function calls.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |8 +---
drivers/scsi/hpsa.h |5 +
2 files changed, 6 insertions
Add in P840ar model name for gen9
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
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 6568da5..15ef65c 100644
Add in gen9 controller model names
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index
From: Stephen Cameron stephenmcame...@gmail.com
There's no reason for it to be a void *, it should be a struct scsi_cmnd *
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |6 +++---
drivers/scsi/hpsa_cmd.h |2 +-
2
a conditional operation from inside the
loop. The case for which the conditional formerly tested is now executed
(unconditionally) after the loop is exited.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Webb Scales web...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi
-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index afd192d..03fae8a 100644
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -3921,6
...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 03fae8a..834ac78 100644
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -4097,8 +4097,15
- performance enhancements.
- removal of command queueing in driver
- elimination of race conditions around aborts
- change to internal driver workqueues
---
Don Brace (12):
hpsa: correct endian sparse warnings
hpsa: change how SA controllers are reset
hpsa: correct
Correct endiness issues reported by sparse. SA controllers are
little endian. This patch ensures endiness correctness.
Signed-off-by: Don Brace don.br...@pmcs.com
Reviewed-by: Scott Teel scott.t...@pmcs.com
Reviewed-by: Webb Scales web...@hp.com
---
drivers/scsi/hpsa.c | 275
From: Tomas Henzl the...@redhat.com
There is a potential memory leak in hpsa_kdump_hard_reset_controller.
Reviewed-by: Don Brace don.br...@pmcs.com
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Tomas Henzl the...@redhat.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers
scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 92ac76a..85b3d73 100644
--- a/drivers
irq 35 for hpsa1
[ 10.711623] scsi host1: Error handler scsi_eh_1 exiting
[ 10.739170] hpsa: probe of :02:00.0 failed with error -38
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c
From: Robert Elliott elli...@hp.com
Change the function names to have hpsa prefix.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 43 ++-
1
the hpsa_kdump_hard_reset_controller fixes it.
At this point we can't know in which state the card is,
so using SA5_INTR_OFF + SA5_REPLY_INTR_MASK_OFFSET defines directly,
instead of the function the drivers provides, seems to be apropriate.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br
Change how SA controllers are reset by changing PCI power levels.
The hpsa driver was finding the PCI_PM_CTRL_STATE_MASK offset
then reading/writing a bitmask to change the power state. There
are kernel functions that do the same operations. Better to use
the kernel functions.
Signed-off-by: Don
From: Robert Elliott elli...@hp.com
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/hpsa.c b
raid_label[]
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index
We change drive queue depths to match drive reported queue depths.
The name of the SML function was changed from scsi_adjust_queue_depth
changed to scsi_change_queue_depth.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 16
From: Robert Elliott elli...@hp.com
There is nothing worrisome about the Waiting for controller to
respond to no-op print, so use dev_info rather than dev_warn.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
From: Fabian Frederick f...@skynet.be
Remove unused variable in hpsa_free_cmd_pool.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Fabian Frederick f...@skynet.be
Acked-by: Don Brace don.br...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |4
From: Robert Elliott elli...@hp.com
Return the real reason for kdump_hard_reset failure rather
than change them all to -ENODEV.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |5
From: Robert Elliott elli...@hp.com
The queue depth printed at startup is in decimal, so
shouldn't have a 0x prefix.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |2 +-
1 file
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 95d581c..a1cfbd3 100644
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -6831,10 +6831,8 @@ static
kernel module:
Tested-by: Don Brace don.br...@pmcs.com
This patch fixed module removal crashes I was seeing with the hpsa
driver running on 3.19-rc4 kernel. Ran ~200 insmod/rmmod tests with no
issues.
--
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body
From: Webb Scales web...@hp.com
Encapsulate the conditional predicate which tests for legacy controllers
in a separate function and rework the code comments.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Webb Scales web...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
From: Robert Elliott elli...@hp.com
The queue depth printed at startup is in decimal, so
shouldn't have a 0x prefix.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |2 +-
1 file
From: Robert Elliott elli...@hp.com
Return the real reason for kdump_hard_reset failure rather
than change them all to -ENODEV.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |5
Change how SA controllers are reset by changing PCI power levels.
The hpsa driver was finding the PCI_PM_CTRL_STATE_MASK offset
then reading/writing a bitmask to change the power state. There
are kernel functions that do the same operations. Better to use
the kernel functions.
Signed-off-by: Don
scsi_adjust_queue_depth was changed to scsi_change_queue_depth
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi
raid_label[]
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index
scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 92ac76a..85b3d73 100644
--- a/drivers
From: Robert Elliott elli...@hp.com
There is nothing worrisome about the Waiting for controller to
respond to no-op print, so use dev_info rather than dev_warn.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
irq 35 for hpsa1
[ 10.711623] scsi host1: Error handler scsi_eh_1 exiting
[ 10.739170] hpsa: probe of :02:00.0 failed with error -38
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c
Correct endiness issues reported by sparse. SA controllers are
little endian. This patch ensures endiness correctness.
Signed-off-by: Don Brace don.br...@pmcs.com
Reviewed-by: Scott Teel scott.t...@pmcs.com
Reviewed-by: Webb Scales web...@hp.com
---
drivers/scsi/hpsa.c | 275
From: Tomas Henzl the...@redhat.com
There is a potential memory leak in hpsa_kdump_hard_reset_controller.
Reviewed-by: Don Brace don.br...@pmcs.com
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Tomas Henzl the...@redhat.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers
- performance enhancements.
- removal of command queueing in driver
- elimination of race conditions around aborts
- change to internal driver workqueues
---
Don Brace (11):
hpsa: correct endian sparse warnings
hpsa: change how SA controllers are reset
hpsa: correct
the hpsa_kdump_hard_reset_controller fixes it.
At this point we can't know in which state the card is,
so using SA5_INTR_OFF + SA5_REPLY_INTR_MASK_OFFSET defines directly,
instead of the function the drivers provides, seems to be apropriate.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br
From: Robert Elliott elli...@hp.com
Change the function names to have hpsa prefix.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 43 ++-
1
From: Stephen Cameron stephenmcame...@gmail.com
Cleanup comments to be more specific. Make messages more
informational.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 21
From: Robert Elliott elli...@hp.com
Make the function name more descriptive. We use more than
one interrupt.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 11 ++-
1 file
From: Fabian Frederick f...@skynet.be
Remove unused variable in hpsa_free_cmd_pool.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Fabian Frederick f...@skynet.be
Acked-by: Don Brace don.br...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |4
From: Robert Elliott elli...@hp.com
Partial allocation failure wasn't handled correctly
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |6 +-
1 file changed, 5 insertions(+), 1
From: Robert Elliott elli...@hp.com
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/hpsa.c b
From: Robert Elliott elli...@hp.com
Enhance error reporting.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
From: Robert Elliott elli...@hp.com
Return the actual error code instead of a generic error code.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |2 +-
1 file changed, 1 insertion
hpsa_wait_for_mode_change_ack
and all their callers check/propagate the result.
More consistency in printing errors and whether
dev_err is used.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 40
Add in gen9 controller model names
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index
uninterruptible
usleep_range(), which triggers rt_sched timeout errors if the
wait is long.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 15 +--
1 file changed, 9 insertions
a conditional operation from inside the
loop. The case for which the conditional formerly tested is now executed
(unconditionally) after the loop is exited.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Webb Scales web...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi
Add in P840ar model name for gen9
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
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 6568da5..15ef65c 100644
From: Stephen Cameron stephenmcame...@gmail.com
Performance tweak, avoid unnecessary function calls.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |8 +---
drivers/scsi/hpsa.h |5 +
2 files changed, 6 insertions
a driver
specific, per-controller work queue avoids this.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Webb Scales web...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 16 +---
drivers/scsi/hpsa.h |1 +
2 files changed, 14 insertions(+), 3
-by: Robert Elliott elli...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 306 +--
drivers/scsi/hpsa.h | 17 ---
drivers/scsi/hpsa_cmd.h | 10 --
3 files changed, 38 insertions(+), 295 deletions(-)
diff --git
to parity drives.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Webb Scales web...@hp.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 318 +--
drivers/scsi/hpsa.h | 14 ++
drivers/scsi/hpsa_cmd.h | 148
...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 14 ++
drivers/scsi/hpsa.h |1 -
2 files changed, 2 insertions(+), 13 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 70f07af..94a82e3 100644
--- a/drivers/scsi/hpsa.c
-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 106 ++-
drivers/scsi/hpsa.h |2 -
2 files changed, 31 insertions(+), 77 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 99c32a0..26e3e5b 100644
--- a/drivers
commands internally
in the driver so that we can remove the locks thta protect the
queue that we will no longer have.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c |7 +--
drivers/scsi/hpsa.h |2 ++
2 files changed, 7
, and
prevent excessive rescanning of devices.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 60 ---
drivers/scsi/hpsa_cmd.h |1 +
2 files changed, 42 insertions(+), 19 deletions
of the ioaccellerated
paths but which have some sort of error condition are
retried down the normal path.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br...@pmcs.com
---
drivers/scsi/hpsa.c | 126 +++
1 file changed, 67
From: Stephen Cameron stephenmcame...@gmail.com
No need to check whether interrupt pending for MSI(X) and
conversely, no need to check whether MSI(X) interrupts are
being used when checking if interrupts are pending.
Reviewed-by: Scott Teel scott.t...@pmcs.com
Signed-off-by: Don Brace don.br
1 - 100 of 855 matches
Mail list logo