linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/22] qla2xxx: Target code enhancemets and feature update
@ 2016-12-06 20:30 Himanshu Madhani
  2016-12-06 20:30 ` [PATCH 01/22] qla2xxx: Fix wrong IOCB type assumption Himanshu Madhani
                   ` (22 more replies)
  0 siblings, 23 replies; 50+ messages in thread
From: Himanshu Madhani @ 2016-12-06 20:30 UTC (permalink / raw)
  To: target-devel, nab; +Cc: giridhar.malavali, linux-scsi, himanshu.madhani

Hi Nic, 

Please consider this series for target-pending. This series is based on
scsi-misc series that was submitted earlier today.

Here's link for that series
 http://marc.info/?l=linux-scsi&m=148105128210090&w=2 

This series contains following changes

o Clean up Target code and tcm_qla2xxx for cleaner interaction.
o Enhanced and strengthen support for T10-DIF/PI.
o Renamed qlini_mode to qlop_mode for better readbility.
o Improve driver capability to manage dual personality 
  (i.e. Initiator and Target) simultaneously.
o Added capability to send non-critical MBX via IOCB path. 
o Some of the pending bug-fixes for target code.

Thanks, 
Himanshu

Himanshu Madhani (3):
  qla2xxx: Include ATIO queue in firmware dump when in target mode
  qla2xxx: Set tcm_qla2xxx version to automatically track qla2xxx
    version.
  qla2xxx: Clear BIT 15 in NVRAM during initialization.

Joe Carnuccio (1):
  qla2xxx: Rerfactor use of sp context

Quinn Tran (18):
  qla2xxx: Fix wrong IOCB type assumption.
  qla2xxx: Add DebugFS node of irq vector cpuid
  qla2xxx: Collect addtional debug data for FW
  qla2xxx: Fix crash due to null pointer access.
  qla2xxx: Refactore target code to remove symbol dependency
  qla2xxx: Refactor T10-DIF PI support
  qla2xxx: Add framework for Async fabric discovery.
  qla2xxx: Refactor session management code.
  qla2xxx: Add Dual mode support in the driver
  qla2xxx: Fix invalid handle erroneous message.
  qla2xxx: Add interrupt polling mechanism
  qla2xxx: Rename qlini_mode parameter
  qla2xxx: Improve RSCN handling in driver
  qla2xxx: Fix slow mem alloc behind lock
  qla2xxx: Reduce exess wait during chip reset
  qla2xxx: Allow relogin to go through if remote login did not finish
  qla2xxx: Improve submission of non critical MB interface.
  qla2xxx: Add check for corrupt ATIO.

 drivers/scsi/qla2xxx/qla_attr.c    |  366 ++++-
 drivers/scsi/qla2xxx/qla_bsg.c     |   21 +-
 drivers/scsi/qla2xxx/qla_dbg.c     |    2 +-
 drivers/scsi/qla2xxx/qla_dbg.h     |    1 +
 drivers/scsi/qla2xxx/qla_def.h     |  287 +++-
 drivers/scsi/qla2xxx/qla_dfs.c     |  237 ++-
 drivers/scsi/qla2xxx/qla_fw.h      |  107 +-
 drivers/scsi/qla2xxx/qla_gbl.h     |   86 +-
 drivers/scsi/qla2xxx/qla_gs.c      |  733 ++++++++-
 drivers/scsi/qla2xxx/qla_init.c    | 1604 +++++++++++++++-----
 drivers/scsi/qla2xxx/qla_inline.h  |   34 +-
 drivers/scsi/qla2xxx/qla_iocb.c    |  172 ++-
 drivers/scsi/qla2xxx/qla_isr.c     |  414 ++++--
 drivers/scsi/qla2xxx/qla_mbx.c     |  553 +++++--
 drivers/scsi/qla2xxx/qla_mid.c     |   15 +-
 drivers/scsi/qla2xxx/qla_mr.c      |   52 +-
 drivers/scsi/qla2xxx/qla_os.c      |  454 ++++--
 drivers/scsi/qla2xxx/qla_target.c  | 2890 +++++++++++++++++++++---------------
 drivers/scsi/qla2xxx/qla_target.h  |  257 ++--
 drivers/scsi/qla2xxx/qla_tmpl.c    |   24 +
 drivers/scsi/qla2xxx/tcm_qla2xxx.c |  753 ++++++++--
 drivers/scsi/qla2xxx/tcm_qla2xxx.h |   16 +-
 22 files changed, 6658 insertions(+), 2420 deletions(-)

-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 50+ messages in thread
* Re: [PATCH 08/22] qla2xxx: Refactore target code to remove symbol dependency
@ 2017-01-17 17:44 Dr. Greg Wettstein
  0 siblings, 0 replies; 50+ messages in thread
From: Dr. Greg Wettstein @ 2017-01-17 17:44 UTC (permalink / raw)
  To: Christoph Hellwig, Tran, Quinn
  Cc: Madhani, Himanshu, target-devel@vger.kernel.org,
	nab@linux-iscsi.org, Malavali, Giridhar,
	linux-scsi@vger.kernel.org

On Dec 16, 12:17am, Christoph Hellwig wrote:
} Subject: Re: [PATCH 08/22] qla2xxx: Refactore target code to remove symbol

Good morning, I hope the week is going well for everyone.

Really late to this thread but am just getting caught d up after the
holidays and putting out fires.

> On Thu, Dec 15, 2016 at 08:50:39PM +0000, Tran, Quinn wrote:
>
> > Christoph, Qlogic was asked to support other Target Stack (SC ST)
> > using the same qla2xxx.ko upstream driver.  I had mentioned this
> > during the last 2015 LSF get together.  Hopefully, that gives a
> > better context.  The understanding that came away was Qlogic do
> > what is needed to provide this support without creating any dead
> > code.

> Right.  And what you're doing here is to create tons of pointless
> and dead code.  If you can support something else without clearly
> making the in-kernel version worse it's all fine.  This patch on the
> other hand is complete crap and totally unacceptable in this form.

It would be helpful to get an architectural overview of how Qlogic is
proposing to interface SCST to the in-kernel Qlogic target driver
code.  Are you going to follow a model of having an analogue of
tcm_qla2xxx for SCST?

We've been running for 3-4 years on the SCST target interface driver
we wrote to allow us to use the in-kernel Qlogic target driver code
with SCST.  We used the model of implementing the interface in
something we referred to as the scst_qla2xxx module and the model
seems solid.  There was a bit of an impedance mismatch but the changes
to the core kernel code were negligible.

Our code is being scheduled for an upgrade to 4.4 and to chase down
what have been reports about issues on 4 GBPS Qlogic cards.  Any
notion of a timeframe on the Cavium/Qlogic based solution?

Have a good day.

Dr. Greg

}-- End of excerpt from Christoph Hellwig

As always,
Dr. G.W. Wettstein, Ph.D.   Enjellic Systems Development, LLC.
4206 N. 19th Ave.           Specializing in information infra-structure
Fargo, ND  58102            development.
PH: 701-281-1686
FAX: 701-281-3949           EMAIL: greg@enjellic.com
------------------------------------------------------------------------------
"Try to remove the color-problem by restarting your computer several
 times."
                                -- Microsoft-Internet Explorer
                                   README.TXT

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

end of thread, other threads:[~2017-01-17 18:45 UTC | newest]

Thread overview: 50+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-06 20:30 [PATCH 00/22] qla2xxx: Target code enhancemets and feature update Himanshu Madhani
2016-12-06 20:30 ` [PATCH 01/22] qla2xxx: Fix wrong IOCB type assumption Himanshu Madhani
2016-12-14 21:09   ` Christoph Hellwig
2016-12-15 17:20     ` Madhani, Himanshu
2016-12-06 20:30 ` [PATCH 02/22] qla2xxx: Include ATIO queue in firmware dump when in target mode Himanshu Madhani
2016-12-06 20:30 ` [PATCH 03/22] qla2xxx: Set tcm_qla2xxx version to automatically track qla2xxx version Himanshu Madhani
2016-12-14 21:12   ` Christoph Hellwig
2016-12-15 17:20     ` Madhani, Himanshu
2016-12-06 20:30 ` [PATCH 04/22] qla2xxx: Clear BIT 15 in NVRAM during initialization Himanshu Madhani
2016-12-06 20:30 ` [PATCH 05/22] qla2xxx: Add DebugFS node of irq vector cpuid Himanshu Madhani
2016-12-14 21:13   ` Christoph Hellwig
2016-12-15 19:32     ` Madhani, Himanshu
2016-12-06 20:30 ` [PATCH 06/22] qla2xxx: Collect addtional debug data for FW Himanshu Madhani
2016-12-06 20:30 ` [PATCH 07/22] qla2xxx: Fix crash due to null pointer access Himanshu Madhani
2016-12-06 20:30 ` [PATCH 08/22] qla2xxx: Refactore target code to remove symbol dependency Himanshu Madhani
2016-12-14 21:18   ` Christoph Hellwig
2016-12-15 19:29     ` Madhani, Himanshu
2016-12-15 20:50       ` Tran, Quinn
2016-12-16  8:17         ` Christoph Hellwig
2016-12-16  8:49         ` Bart Van Assche
2016-12-06 20:30 ` [PATCH 09/22] qla2xxx: Refactor T10-DIF PI support Himanshu Madhani
2016-12-14 21:03   ` Christoph Hellwig
2016-12-06 20:30 ` [PATCH 10/22] qla2xxx: Add framework for Async fabric discovery Himanshu Madhani
2016-12-06 20:30 ` [PATCH 11/22] qla2xxx: Refactor session management code Himanshu Madhani
2016-12-14 21:04   ` Christoph Hellwig
2016-12-06 20:30 ` [PATCH 12/22] qla2xxx: Add Dual mode support in the driver Himanshu Madhani
2016-12-06 20:30 ` [PATCH 13/22] qla2xxx: Fix invalid handle erroneous message Himanshu Madhani
2016-12-06 20:30 ` [PATCH 14/22] qla2xxx: Add interrupt polling mechanism Himanshu Madhani
2016-12-14 21:06   ` Christoph Hellwig
2016-12-15  9:27     ` Bart Van Assche
2016-12-15 17:12       ` Madhani, Himanshu
2016-12-06 20:30 ` [PATCH 15/22] qla2xxx: Rename qlini_mode parameter Himanshu Madhani
2016-12-14 21:07   ` Christoph Hellwig
2016-12-15 17:21     ` Madhani, Himanshu
2016-12-06 20:30 ` [PATCH 16/22] qla2xxx: Improve RSCN handling in driver Himanshu Madhani
2016-12-07 11:32   ` Hannes Reinecke
2016-12-09 19:38     ` Madhani, Himanshu
2016-12-06 20:30 ` [PATCH 17/22] qla2xxx: Rerfactor use of sp context Himanshu Madhani
2016-12-06 20:30 ` [PATCH 18/22] qla2xxx: Fix slow mem alloc behind lock Himanshu Madhani
2016-12-06 20:30 ` [PATCH 19/22] qla2xxx: Reduce exess wait during chip reset Himanshu Madhani
2016-12-06 20:30 ` [PATCH 20/22] qla2xxx: Allow relogin to go through if remote login did not finish Himanshu Madhani
2016-12-07 12:05   ` Hannes Reinecke
2016-12-06 20:30 ` [PATCH 21/22] qla2xxx: Improve submission of non critical MB interface Himanshu Madhani
2016-12-07 12:16   ` Hannes Reinecke
2016-12-06 20:30 ` [PATCH 22/22] qla2xxx: Add check for corrupt ATIO Himanshu Madhani
2016-12-07 12:25   ` Hannes Reinecke
2016-12-12 23:10 ` [PATCH 00/22] qla2xxx: Target code enhancemets and feature update Madhani, Himanshu
2016-12-14 20:24   ` Madhani, Himanshu
2016-12-14 21:19     ` Christoph Hellwig
  -- strict thread matches above, loose matches on Subject: below --
2017-01-17 17:44 [PATCH 08/22] qla2xxx: Refactore target code to remove symbol dependency Dr. Greg Wettstein

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).