[PATCH 0/9] staging: unisys: visorhba: fix scsi task mgmt completion handling and other cleanups

David Kershner david.kershner at unisys.com
Thu May 12 13:14:39 UTC 2016


On 5/2 Dan Carpenter pointed out some sloppy u64 <--> pointer casting we
were doing in visorhba, but this turned out to be the tip of an iceberg.
Our handling of scsi task mgmt completions turned out to be buggy. 

This patchset switches visorhba to use idr_alloc() / idr_find() so that we
can cleanly pass handles back-and-forth from/to the IO partition instead
of raw pointers.

This patchset simplifies and enhances the visorhba debugfs interface, and
deletes a global array made no-longer-necessary by the cleanup.


David Binder (4):
  staging: unisys: visornic: simplify visornic if statements
  staging: unisys: visornic: cleanup error handling
  staging: unisys: visorhba: return 0 literal
  staging: unisys: visornic: check for error instead of success

Tim Sell (5):
  staging: unisys: visorhba: delete processing of vdiskmgmt commands
  staging: unisys: visorhba: correct scsi task mgmt completion handling
  staging: unisys: visorhba: remove unused (and broken) logic
  staging: unisys: visorhba: simplify and enhance debugfs interface
  staging: unisys: visorhba: visorhbas_open[] no longer used, so deleted

 drivers/staging/unisys/visorhba/visorhba_main.c | 384 +++++++++++-------------
 drivers/staging/unisys/visornic/visornic_main.c |  53 ++--
 2 files changed, 209 insertions(+), 228 deletions(-)

-- 
1.9.1



More information about the devel mailing list