From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: "Yang, Bo" <Bo.Yang@lsi.com>
Cc: "'linux-scsi@vger.kernel.org'" <linux-scsi@vger.kernel.org>,
"'akpm@osdl.org'" <akpm@osdl.org>,
"'linux-kernel@vger.kernel.org'" <linux-kernel@vger.kernel.org>,
Tomas Henzl <thenzl@redhat.com>
Subject: Re: [PATCH 8/9] scsi: megaraid_sas - Driver take some workloads from FW
Date: Fri, 19 Nov 2010 11:53:01 -0600 [thread overview]
Message-ID: <1290189181.23556.77.camel@mulgrave.site> (raw)
In-Reply-To: <4B6A08C587958942AA3002690DD4F8C30106A96223@cosmail02.lsi.com>
On Fri, 2010-11-19 at 10:43 -0700, Yang, Bo wrote:
> Driver added the new feature to take some of the workloads from FW to increase
> performance of the iMR controller. FW assigns the read cmds back to driver
> which will increase the performance for megaraid sas iMR controller.
Just on a process note: you don't have to cc me at every known
address ... I do read linux-scsi ...
My first take on this patch is that you're basically passing data back
to the kernel for RAID-N parity calculations. This effectively makes
your RAID one of those pseudo HW ones. In which case, why not simply
abandon the HW raid piece and have it all done by DM/MD, which are well
optimised for all types of RAID? The reason for asking is that we're
trying to reduce the number of in-kernel raid implementations and this
is going in the wrong direction.
James
next prev parent reply other threads:[~2010-11-19 17:53 UTC|newest]
Thread overview: 70+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-10 2:34 [PATCH 1/12] scsi: megaraid_sas - Online Controller Reset - I: Enable Online Controller Reset Bit in register related functions Yang, Bo
2010-06-10 2:34 ` Yang, Bo
2010-06-18 21:18 ` Yinghai Lu
2010-06-18 21:18 ` Yinghai Lu
2010-06-18 21:21 ` Yang, Bo
2010-06-18 21:21 ` Yang, Bo
2010-06-18 21:37 ` Yinghai Lu
2010-06-18 21:37 ` Yinghai Lu
2010-06-18 21:56 ` Yang, Bo
2010-06-18 21:56 ` Yang, Bo
2010-06-20 14:54 ` James Bottomley
2010-06-20 15:01 ` James Bottomley
2010-06-20 19:27 ` Yang, Bo
2010-06-20 19:27 ` Yang, Bo
2010-06-20 19:47 ` James Bottomley
2010-06-20 20:19 ` Yang, Bo
2010-06-20 20:19 ` Yang, Bo
2010-06-20 20:48 ` James Bottomley
2010-06-21 15:35 ` Yang, Bo
2010-06-21 15:35 ` Yang, Bo
2010-07-20 13:08 ` Yang, Bo
2010-07-20 13:08 ` Yang, Bo
2010-07-30 18:53 ` Yang, Bo
2010-07-30 18:53 ` Yang, Bo
2010-07-31 17:46 ` James Bottomley
2010-08-05 20:44 ` Yang, Bo
2010-08-05 20:44 ` Yang, Bo
2010-08-06 0:36 ` James Bottomley
2010-08-12 15:20 ` Yang, Bo
2010-08-12 15:20 ` Yang, Bo
2010-08-16 14:22 ` Yang, Bo
2010-08-16 14:22 ` Yang, Bo
2010-08-16 15:06 ` James Bottomley
2010-08-05 19:47 ` Yang, Bo
2010-08-05 19:47 ` Yang, Bo
2010-11-19 16:33 ` [PATCH 1/9] scsi: megaraid_sas - Change the time format during the driver online controller reset Yang, Bo
2010-11-19 16:33 ` Yang, Bo
2010-11-19 16:53 ` [PATCH 2/9] scsi: megaraid_sas - Driver only report tape drive, JBOD and logic drives Yang, Bo
2010-11-19 16:53 ` Yang, Bo
2010-11-19 17:03 ` [PATCH 3/9] scsi: megaraid_sas - Driver will scan device after receive event MR_EVT_LD_STATE_CHANGE Yang, Bo
2010-11-19 17:03 ` Yang, Bo
2010-11-19 17:08 ` [PATCH 4/9] scsi: megaraid_sas - Driver provide the version to FW Yang, Bo
2010-11-19 17:08 ` Yang, Bo
2010-11-19 17:23 ` [PATCH 5/9] scsi: megaraid_sas - report different io sectors for ieee support Yang, Bo
2010-11-19 17:23 ` Yang, Bo
2010-11-19 17:29 ` [PATCH 6/9] scsi: megaraid_sas - Add the online reset to iMR chip Yang, Bo
2010-11-19 17:29 ` Yang, Bo
2010-11-19 17:37 ` [PATCH 7/9] scsi: megaraid_sas - Driver created the self work queue for OCR Yang, Bo
2010-11-19 17:37 ` Yang, Bo
2010-11-19 17:43 ` [PATCH 8/9] scsi: megaraid_sas - Driver take some workloads from FW Yang, Bo
2010-11-19 17:43 ` Yang, Bo
2010-11-19 17:53 ` James Bottomley [this message]
2010-11-24 15:22 ` Yang, Bo
2010-11-24 15:22 ` Yang, Bo
2010-11-30 15:57 ` Tomas Henzl
2010-11-30 16:11 ` [PATCH 7/9] scsi: megaraid_sas - Driver created the self work queue for OCR Tomas Henzl
2010-11-30 16:11 ` Tomas Henzl
2010-11-30 18:05 ` Yang, Bo
2010-11-30 18:05 ` Yang, Bo
2010-12-01 12:19 ` Tomas Henzl
2010-12-01 12:19 ` Tomas Henzl
2010-12-01 15:55 ` Yang, Bo
2010-12-01 15:55 ` Yang, Bo
2010-11-30 15:36 ` [PATCH 6/9] scsi: megaraid_sas - Add the online reset to iMR chip Tomas Henzl
2010-11-30 15:36 ` Tomas Henzl
2010-11-19 17:50 ` [PATCH 9/9] scsi: megaraid_sas - Documentation and driver version update Yang, Bo
2010-11-19 17:50 ` Yang, Bo
2010-12-09 17:07 ` [PATCH 2/9] scsi: megaraid_sas - Driver only report tape drive, JBOD and logic drives James Bottomley
2010-11-30 15:21 ` [PATCH 1/9] scsi: megaraid_sas - Change the time format during the driver online controller reset Tomas Henzl
2010-11-30 15:21 ` Tomas Henzl
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=1290189181.23556.77.camel@mulgrave.site \
--to=james.bottomley@hansenpartnership.com \
--cc=Bo.Yang@lsi.com \
--cc=akpm@osdl.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=thenzl@redhat.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.