public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: peter Bogdanovic <pbog@us.ibm.com>
To: Eric Moore <eric.moore@lsi.com>
Cc: linux-scsi@vger.kernel.org, James.Bottomley@HansenPartnership.com
Subject: Re: [RFC 0/12] mpt2sas v00.100.11.15: 4th revision
Date: Fri, 13 Mar 2009 10:50:55 -0700	[thread overview]
Message-ID: <1236966655.6960.16.camel@gnattop> (raw)
In-Reply-To: <20090309072022.GA18720@lsil.com>


I confirmed the fix for the mpt2sas compilation errors when
SCSI_MPT2SAS_LOGGING wasn't set and the patch of the Med Layer Makefile.
The mpt2sas driver compiles now without error or warning with or without
logging enabled.

I have also had a chance to test the driver with a lsi sas 2108 card and
the ibm ds3200 sas enclosure. I see no errors on when I insert or remove
the driver. I ran block i/o tests for more that 24 hours using disktest
without error. 

Peter


On Mon, 2009-03-09 at 01:20 -0600, Eric Moore wrote:
> * This is new scsi lld device driver from LSI supporting the SAS 2.0
> standard. I have split patchs by filename.
> * Here is list of new 6gb host controllers:
> 
>   LSI SAS2004
>   LSI SAS2008
>   LSI SAS2108
>   LSI SAS2116
> 
> * Here are the changes in the 4th posting of this patch set:
> 
> (1) fix compile errors when SCSI_MPT2SAS_LOGGING is not enabled
> (2) add mpt2sas to the SCSI Mid Layer Makefile
> (3) append mpt2sas_ to the naming of all non-static functions
> (4) fix oops for SMP_PASSTHRU
> (5) doorbell algorithm imported changes from windows driver
> 
> * Here are the changes in the 3rd posting of this patch set:
> 
> (1) add readl following writel from the function that disables
> interrupts
> (2) replace 0xFFFFFFFFFFFFFFFFULL with ~0ULL
> (3) when calling pci_enable_msix, only pass one msix entry (instead of
> 15).
> (4) remove the "current HW implementation uses..... " comment in the
> sources
> (5) merged bug fix for SIGIO/POLLIN notifcation; reported by the
> storlib team.
> 
> * Here are the changes in the 2nd posting of this patch set:
> 
> (1) use little endian types in the mpi headers
> (2) merged in bug fix's from inhouse drivers.
> 
> * Here is a summary of the patchs:
> 
> [PATCH 1/12] mpt2sas_base - Common API layer interface to MPT (Message
> Passing Technology) firmware.
> [PATCH 2/12] mpt2sas_config - Common API for accessing firmware
> configuration pages.
> [PATCH 3/12] mptsas_ctl - Storage Management Module.
> [PATCH 4/12] mpt2sas_scsih - SCSI Host Layer.
> [PATCH 5/12] mpt2sas_transport - Common API for accessing the Kernel
> SAS Transport Layer.
> [PATCH 6/12] mpt2sas_debug - Common Header for debug logging.
> [PATCH 7/12] mpi headers  -  Common MPI (Message Passing Interface)
> Headers.
> [PATCH 8/12] Kconfig: mpt2sas Kernel configuration file.
> [PATCH 9/12] Makefile - mpt2sas driver Makefile.
> [PATCH 10/12] scsiml Kconfig - Adding mptsas to the common scsi ml
> Kconfig.
> [PATCH 11/11] linux/miscdevice.h - Adding suport for
> MPT2SAS_MINOR(221)
> [PATCH 12/12] scsiml Makefile - add mpt2sas to the SCSI Mid Layer
> Makefile
> 
> * diffstat output
> 
>  Kconfig             |   66
>  Makefile            |    7
>  mpi/mpi2.h          | 1067 +++++++++
>  mpi/mpi2_cnfg.h     | 2151 +++++++++++++++++++
>  mpi/mpi2_init.h     |  420 +++
>  mpi/mpi2_ioc.h      | 1295 +++++++++++
>  mpi/mpi2_raid.h     |  295 ++
>  mpi/mpi2_sas.h      |  282 ++
>  mpi/mpi2_tool.h     |  249 ++
>  mpi/mpi2_type.h     |   61
>  mpt2sas_base.c      | 3435 +++++++++++++++++++++++++++++++
>  mpt2sas_base.h      |  779 +++++++
>  mpt2sas_config.c    | 1873 +++++++++++++++++
>  mpt2sas_ctl.c       | 2516 +++++++++++++++++++++++
>  mpt2sas_ctl.h       |  416 +++
>  mpt2sas_debug.h     |  181 +
>  mpt2sas_scsih.c     | 5687
> ++++++++++++++++++++++++++++++++++++++++++++++++++++
>  mpt2sas_transport.c | 1211 +++++++++++
>  18 files changed, 21991 insertions(+)
> 
> Signed-off-by: Eric Moore <eric.moore@lsi.com>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi"
> in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
-- 
Peter Bogdanovic
IBM System x Enablement


      reply	other threads:[~2009-03-13 17:51 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-09  7:20 [RFC 0/12] mpt2sas v00.100.11.15: 4th revision Eric Moore
2009-03-13 17:50 ` peter Bogdanovic [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1236966655.6960.16.camel@gnattop \
    --to=pbog@us.ibm.com \
    --cc=James.Bottomley@HansenPartnership.com \
    --cc=eric.moore@lsi.com \
    --cc=linux-scsi@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox