Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751920AbdHGN4j (ORCPT ); Mon, 7 Aug 2017 09:56:39 -0400 Received: from mail-pg0-f48.google.com ([74.125.83.48]:36960 "EHLO mail-pg0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751396AbdHGN4h (ORCPT ); Mon, 7 Aug 2017 09:56:37 -0400 From: Kashyap Desai References: <1500038560-11231-1-git-send-email-suganath-prabu.subramani@broadcom.com> <20170805135353.GA7526@infradead.org> <1501944129.3649.9.camel@HansenPartnership.com> In-Reply-To: <1501944129.3649.9.camel@HansenPartnership.com> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 14.0 Thread-Index: AQFNq6RnRXsUY5icjNF8Ngs/7MBTPAIHncrBAVKchuEDCFf1qKNQnO6Q Date: Mon, 7 Aug 2017 19:26:34 +0530 Message-ID: <0ada8680ebfeb026662d858ec2b38ef4@mail.gmail.com> Subject: RE: [PATCH v2 00/13] mpt3sas driver NVMe support: To: James Bottomley , Christoph Hellwig , Hannes Reinecke Cc: Suganath Prabu Subramani , martin.petersen@oracle.com, linux-scsi@vger.kernel.org, Sathya Prakash Veerichetty , linux-kernel@vger.kernel.org, Chaitra Basappa , Sreekanth Reddy , linux-nvme@lists.infradead.org Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2505 Lines: 62 > -----Original Message----- > From: James Bottomley [mailto:James.Bottomley@HansenPartnership.com] > Sent: Saturday, August 05, 2017 8:12 PM > To: Christoph Hellwig; Hannes Reinecke > Cc: Suganath Prabu S; martin.petersen@oracle.com; linux- > scsi@vger.kernel.org; Sathya.Prakash@broadcom.com; > kashyap.desai@broadcom.com; linux-kernel@vger.kernel.org; > chaitra.basappa@broadcom.com; sreekanth.reddy@broadcom.com; linux- > nvme@lists.infradead.org > Subject: Re: [PATCH v2 00/13] mpt3sas driver NVMe support: > > On Sat, 2017-08-05 at 06:53 -0700, Christoph Hellwig wrote: > > On Wed, Aug 02, 2017 at 10:14:40AM +0200, Hannes Reinecke wrote: > > > > > > I'm not happy with this approach. > > > NVMe devices should _not_ appear as SCSI devices; this will just > > > confuse matters _and_ will be incompatible with 'normal' NVMe > > > devices. > > > > > > Rather I would like to see the driver to hook into the existing NVMe > > > framework (which essentially means to treat the mpt3sas as a weird > > > NVMe-over-Fabrics HBA), and expose the NVMe devices like any other > > > NVMe HBA. > > > > That doesn't make any sense. The devices behind the mpt adapter don't > > look like NVMe devices at all for the hosts - there are no NVMe > > commands or queues involved at all, they hide behind the same somewhat > > leaky scsi abstraction as other devices behind the mpt controller. > > You might think about what we did for SAS: split the generic handler into > two > pieces, libsas for driving the devices, which mpt didn't need because of > the fat > firmware and the SAS transport class so mpt could at least show the same > sysfs > files as everything else for SAS devices. Ventura generation of controllers are adding connectivity of NVME drives seamlessly and protocol handling is in Firmware. Same as SCSI to ATA translation done in firmware, Ventura controller is doing SCSI to NVME translation and for end user protocol handling is abstracted. This product handles new Transport protocol (NVME) same as ATA and transport is abstracted for end user. NVME pass-through related driver code, it is just a big tunnel for user space application. It is just a basic framework like SATA PASS-Through in existing mpt3sas driver. > > Fortunately for NVMe it's very simple at the moment its just a couple of > host > files and wwid on the devices. > > James > > > > The only additional leak is that the controller now supports NVMe- > > like PRPs in additions to its existing two SGL formats. > >