From: James Bottomley <James.Bottomley@suse.de>
To: Dan Williams <dan.j.williams@intel.com>
Cc: linux-scsi <linux-scsi@vger.kernel.org>,
linux-kernel <linux-kernel@vger.kernel.org>,
Dave Jiang <dave.jiang@intel.com>,
Ed Nadolski <edmund.nadolski@intel.com>,
Ed Ciechanowski <ed.ciechanowski@intel.com>,
Jacek Danecki <jacek.danecki@intel.com>,
Jeff Skirvin <jeffrey.d.skirvin@intel.com>
Subject: Re: isci for 2.6.39?
Date: Thu, 10 Mar 2011 18:10:19 -0600 [thread overview]
Message-ID: <1299802219.11933.175.camel@mulgrave.site> (raw)
In-Reply-To: <1299801757.8641.21.camel@dwillia2-linux>
On Thu, 2011-03-10 at 16:02 -0800, Dan Williams wrote:
> Given the impending opening of the 2.6.39 merge window I would like to
> discuss the options for merging the isci driver. Review has been
> intermittent which is understandable given the size and flux of the
> codebase. It has received a good amount of cleanups, but additional
> review issues and cleanups are still all too easy to spot. I know you
> have expressed reservations about taking scsi drivers through -staging
> in the past [1], so I would like to propose an alternative. Merge the
> driver into scsi-misc but with a -staging style TODO file that tracks
> the review issues. If the TODO file is not addressed by the 2.6.40
> window the driver would be moved to -staging. This has the benefit of
> keeping the driver under your purview and expected location, but still
> have the imminent prospect of being de-staged to ensure the community's
> concerns are ultimately addressed. We fully intend to maintain the
> current momentum on the driver cleanup effort and be ready in advance of
> 2.6.40.
Given that you only posted the core files today, I think it's a bit late
for the 2.6.39 merge window, which will be opening in under a week.
You could send it all to staging ... it's just that pretty much seems to
guarantee no SCSI review, which is what you seem to need at the moment.
> As a side note I'm still looking for a disposition for:
> "libsas: flush initial device discovery before completing ->scan_finished()" [2]
> "libsas: fix/amend device gone notification in sas_deform_port()" [3]
I thought I commented ... I'll go back and dig them out of email.
James
next prev parent reply other threads:[~2011-03-11 0:10 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-11 0:02 isci for 2.6.39? Dan Williams
2011-03-11 0:10 ` James Bottomley [this message]
2011-03-11 0:40 ` Dan Williams
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=1299802219.11933.175.camel@mulgrave.site \
--to=james.bottomley@suse.de \
--cc=dan.j.williams@intel.com \
--cc=dave.jiang@intel.com \
--cc=ed.ciechanowski@intel.com \
--cc=edmund.nadolski@intel.com \
--cc=jacek.danecki@intel.com \
--cc=jeffrey.d.skirvin@intel.com \
--cc=linux-kernel@vger.kernel.org \
--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