From: Sumit Saxena <sumit.saxena@avagotech.com>
To: "Martin K. Petersen" <martin.petersen@oracle.com>
Cc: linux-scsi@vger.kernel.org, thenzl@redhat.com, hch@infradead.org,
jbottomley@parallels.com,
Kashyap Desai <kashyap.desai@avagotech.com>
Subject: RE: [PATCH 1/5] megaraid_sas : Code optimization and refactoring
Date: Mon, 13 Apr 2015 15:38:47 +0530 [thread overview]
Message-ID: <bc4eb5dffe342f41aa5e9dba5afe3d95@mail.gmail.com> (raw)
In-Reply-To: <yq11tjvil9g.fsf@sermon.lab.mkp.net>
>-----Original Message-----
>From: Martin K. Petersen [mailto:martin.petersen@oracle.com]
>Sent: Wednesday, April 08, 2015 4:45 AM
>To: Sumit.Saxena@avagotech.com
>Cc: linux-scsi@vger.kernel.org; thenzl@redhat.com;
>martin.petersen@oracle.com; hch@infradead.org;
>jbottomley@parallels.com; kashyap.desai@avagotech.com
>Subject: Re: [PATCH 1/5] megaraid_sas : Code optimization and refactoring
>
>>>>>> "Sumit" == <Sumit.Saxena@avagotech.com> writes:
>
>Sumit> This patch has code optimization and refactoring in below areas-
>Sumit> 1) Created separate functions for sysPD IOs and non Read Write
>Sumit> LDIOs. 2) Added function for refiring MFI commands in controller
>Sumit> reset path. 3) Enabling interrupt immediately after adapter's
>Sumit> initialization. 4) Added function for setting up IRQs. 5)
>Sumit> Modified few prints for better logging. 6) Modified few macros
>Sumit> not to pass unused parameters. 7) Move internal commands and
>Sumit> SCSI commands reservation in adapter's sepcific functions.
>
>This needs to be 7 patches and not 1.
Ok, I will spilt this patch in 7 patches. Do you have any
feedback/comments for other patches of this series? IF you are ok with
other patches, I will resend the series with patch#1 splitted in 7 patches
and rest of the patches(2,3,4 and 5) will remain unchanged.
>
>--
>Martin K. Petersen Oracle Linux Engineering
next prev parent reply other threads:[~2015-04-13 10:08 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-27 11:55 [PATCH 1/5] megaraid_sas : Code optimization and refactoring Sumit.Saxena
2015-04-07 23:14 ` Martin K. Petersen
2015-04-13 10:08 ` Sumit Saxena [this message]
2015-04-14 1:17 ` Martin K. Petersen
2015-04-14 5:56 ` Christoph Hellwig
2015-04-20 12:38 ` Sumit Saxena
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=bc4eb5dffe342f41aa5e9dba5afe3d95@mail.gmail.com \
--to=sumit.saxena@avagotech.com \
--cc=hch@infradead.org \
--cc=jbottomley@parallels.com \
--cc=kashyap.desai@avagotech.com \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox