linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 00/19] be2iscsi driver updates
@ 2012-10-19 23:11 John Soni Jose
  2012-10-24  7:01 ` Mike Christie
  0 siblings, 1 reply; 2+ messages in thread
From: John Soni Jose @ 2012-10-19 23:11 UTC (permalink / raw)
  To: michaelc, linux-scsi; +Cc: John Soni Jose

 V2 patches for be2iscsi driver updates. Fixed the review comments
 with this patch. These patches were generated aganist scsi-misc branch.

  be2iscsi: Fix the issue with soft reset.
  be2iscsi: Display Completion Event string instead of Opcode
  be2iscsi: Fix memory leak in control path of driver
  be2iscsi: Fix max supported EQ count to 8.
  be2iscsi: Display driver name and version in device attribute
  be2iscsi: Issue an FLR when driver is loaded
  be2iscsi: Fix kernel panic in blk_iopoll disable mode.
  be2iscsi: Fix return value and typo.
  be2iscsi: Fix driver support for an adapter.
  be2iscsi: Fix max EQ supported by the driver.
  be2iscsi: Fix support for handling CQ_CREATE V2 version.
  be2iscsi: Fix support for V2 version of WRB.
  be2iscsi: Fix session update context with V2 version.
  be2iscsi: Fix Task Completion Event handling
  be2iscsi: Fix issue of displaying adapter family.
  be2iscsi: Fix the copyright information
  be2iscsi: Fix for MBX timeout issue
  be2iscsi: Fix Unrecoverable Error Detection
  be2iscsi: Bump the driver version

 drivers/scsi/be2iscsi/be.h       |    7 +-
 drivers/scsi/be2iscsi/be_cmds.c  |  236 +++++++--
 drivers/scsi/be2iscsi/be_cmds.h  |   93 ++++-
 drivers/scsi/be2iscsi/be_iscsi.c |  124 ++---
 drivers/scsi/be2iscsi/be_iscsi.h |    2 +-
 drivers/scsi/be2iscsi/be_main.c  | 1068 ++++++++++++++++++++++++--------------
 drivers/scsi/be2iscsi/be_main.h  |  152 +++++-
 drivers/scsi/be2iscsi/be_mgmt.c  |  424 +++++++++++++---
 drivers/scsi/be2iscsi/be_mgmt.h  |   23 +-
 9 files changed, 1547 insertions(+), 582 deletions(-)

-- 
1.7.2


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH v2 00/19] be2iscsi driver updates
  2012-10-19 23:11 [PATCH v2 00/19] be2iscsi driver updates John Soni Jose
@ 2012-10-24  7:01 ` Mike Christie
  0 siblings, 0 replies; 2+ messages in thread
From: Mike Christie @ 2012-10-24  7:01 UTC (permalink / raw)
  To: John Soni Jose; +Cc: linux-scsi

On 10/19/2012 06:11 PM, John Soni Jose wrote:
>  V2 patches for be2iscsi driver updates. Fixed the review comments
>  with this patch. These patches were generated aganist scsi-misc branch.
> 
>   be2iscsi: Fix the issue with soft reset.
>   be2iscsi: Display Completion Event string instead of Opcode
>   be2iscsi: Fix memory leak in control path of driver
>   be2iscsi: Fix max supported EQ count to 8.
>   be2iscsi: Display driver name and version in device attribute
>   be2iscsi: Issue an FLR when driver is loaded
>   be2iscsi: Fix kernel panic in blk_iopoll disable mode.
>   be2iscsi: Fix return value and typo.
>   be2iscsi: Fix driver support for an adapter.
>   be2iscsi: Fix max EQ supported by the driver.
>   be2iscsi: Fix support for handling CQ_CREATE V2 version.
>   be2iscsi: Fix support for V2 version of WRB.
>   be2iscsi: Fix session update context with V2 version.
>   be2iscsi: Fix Task Completion Event handling
>   be2iscsi: Fix issue of displaying adapter family.
>   be2iscsi: Fix the copyright information
>   be2iscsi: Fix for MBX timeout issue
>   be2iscsi: Fix Unrecoverable Error Detection
>   be2iscsi: Bump the driver version
> 

Patches look ok overall. Just those 2 issues which might be ok with James.

Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-10-24  7:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-19 23:11 [PATCH v2 00/19] be2iscsi driver updates John Soni Jose
2012-10-24  7:01 ` Mike Christie

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).