linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: <Don.Brace@microchip.com>
To: <mironov.ivan@gmail.com>, <linux-scsi@vger.kernel.org>
Cc: <linux-kernel@vger.kernel.org>, <don.brace@microsemi.com>,
	<jejb@linux.vnet.ibm.com>, <martin.petersen@oracle.com>,
	<esc.storagedev@microsemi.com>
Subject: RE: [PATCH 0/6] Add support of the HBA mode on HP Smart Array P410i controllers
Date: Mon, 17 Dec 2018 16:32:10 +0000	[thread overview]
Message-ID: <BYAPR11MB285502EFDD672C228F9FFB73E1BC0@BYAPR11MB2855.namprd11.prod.outlook.com> (raw)
In-Reply-To: <449fb880e94d478fb35e01da93c5b1fa1732f7d8.camel@gmail.com>

-----Original Message-----
From: Ivan Mironov [mailto:mironov.ivan@gmail.com] 
Sent: Friday, December 14, 2018 2:30 PM
To: Don Brace - C33706 <Don.Brace@microchip.com>; linux-scsi@vger.kernel.org
Cc: linux-kernel@vger.kernel.org; don.brace@microsemi.com; jejb@linux.vnet.ibm.com; martin.petersen@oracle.com; esc.storagedev@microsemi.com
Subject: Re: [PATCH 0/6] Add support of the HBA mode on HP Smart Array P410i controllers

EXTERNAL EMAIL


On Fri, 2018-12-14 at 19:38 +0000, Don.Brace@microchip.com wrote:
> NAKing this series.
> - The P410 controllers do not fully support HBA mode and we do not support adding it to the hpsa driver.
>

Could you please elaborate on what exactly you mean by "do not fully support HBA mode"?

Don> HBA mode may have started on this controller family, but was not fully developed.
Don> Regardless of the server, It's not a generic offering and should not be accepted into the general driver space.
Don> Adding this to kernel.org means it will find its way into other Linux vendors and we do not want this either.

HBA mode looks supported on P410i (at least to some degree) because:
*) There is an official method to enable/disable it on these controllers. Only for itanium-based servers unfortunately.
*) This method is mentioned in various documents from HP regarding Integrity servers.
*) Controller reacts on the change of corresponging bit in nvram_flags by immediately removing any configured RAID arrays and by refusing to configure new arrays. Which is totally expected for HBA mode.
*) HBA mode works for me on P410i with patched driver.


      reply	other threads:[~2018-12-17 16:32 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-14 13:21 [PATCH 0/6] Add support of the HBA mode on HP Smart Array P410i controllers Ivan Mironov
2018-12-14 13:21 ` [PATCH 1/6] scsi: hpsa: Add function to check if device is a disk or a zoned device Ivan Mironov
2018-12-14 13:26   ` Johannes Thumshirn
2018-12-14 13:21 ` [PATCH 2/6] scsi: hpsa: Support HBA mode on HP Smart Array P410i controllers Ivan Mironov
2018-12-14 13:21 ` [PATCH 3/6] scsi: hpsa: Add/mask existing devices on rescan if visibility changes Ivan Mironov
2018-12-14 13:21 ` [PATCH 4/6] scsi: hpsa: Ignore HBA flag from NVRAM if logical devices exist Ivan Mironov
2018-12-14 13:21 ` [PATCH 5/6] scsi: hpsa: Name more fields in "struct bmic_identify_controller" Ivan Mironov
2018-12-14 13:21 ` [PATCH 6/6] scsi: hpsa: Do not use HBA flag from NVRAM if HBA is not supported Ivan Mironov
2018-12-14 19:38 ` [PATCH 0/6] Add support of the HBA mode on HP Smart Array P410i controllers Don.Brace
2018-12-14 20:30   ` Ivan Mironov
2018-12-17 16:32     ` Don.Brace [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=BYAPR11MB285502EFDD672C228F9FFB73E1BC0@BYAPR11MB2855.namprd11.prod.outlook.com \
    --to=don.brace@microchip.com \
    --cc=don.brace@microsemi.com \
    --cc=esc.storagedev@microsemi.com \
    --cc=jejb@linux.vnet.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=martin.petersen@oracle.com \
    --cc=mironov.ivan@gmail.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;
as well as URLs for NNTP newsgroup(s).