Re: [PATCH 1/2] lightnvm: add generic ocssd detection

2017-02-27 Thread Sagi Grimberg
[adding linux-nvme to Cc as the patch changes the nvme driver, despite the subject line] On Sat, Feb 25, 2017 at 08:16:04PM +0100, Matias Bjørling wrote: On 02/25/2017 07:21 PM, Christoph Hellwig wrote: On Fri, Feb 24, 2017 at 06:16:48PM +0100, Matias Bjørling wrote: More implementations of

Re: [PATCH 1/2] lightnvm: add generic ocssd detection

2017-02-27 Thread Keith Busch
On Mon, Feb 27, 2017 at 08:35:06PM +0200, Sagi Grimberg wrote: > > On Sat, Feb 25, 2017 at 08:16:04PM +0100, Matias Bjørling wrote: > > > On 02/25/2017 07:21 PM, Christoph Hellwig wrote: > > > > No way in hell. vs is vendor specific and we absolutely can't overload > > > > it with any sort of mean

Re: [PATCH 1/2] lightnvm: add generic ocssd detection

2017-02-26 Thread Christoph Hellwig
[adding linux-nvme to Cc as the patch changes the nvme driver, despite the subject line] On Sat, Feb 25, 2017 at 08:16:04PM +0100, Matias Bjørling wrote: > On 02/25/2017 07:21 PM, Christoph Hellwig wrote: > > On Fri, Feb 24, 2017 at 06:16:48PM +0100, Matias Bjørling wrote: > > > More implementatio

Re: [PATCH 1/2] lightnvm: add generic ocssd detection

2017-02-25 Thread Matias Bjørling
On 02/25/2017 07:21 PM, Christoph Hellwig wrote: On Fri, Feb 24, 2017 at 06:16:48PM +0100, Matias Bjørling wrote: More implementations of OCSSDs are becoming available. Adding each using pci ids are becoming a hassle. Instead, use a 16 byte string in the vendor-specific area of the identificatio

Re: [PATCH 1/2] lightnvm: add generic ocssd detection

2017-02-25 Thread Christoph Hellwig
On Fri, Feb 24, 2017 at 06:16:48PM +0100, Matias Bjørling wrote: > More implementations of OCSSDs are becoming available. Adding each using > pci ids are becoming a hassle. Instead, use a 16 byte string in the > vendor-specific area of the identification command to identify an > Open-Channel SSD. >

[PATCH 1/2] lightnvm: add generic ocssd detection

2017-02-24 Thread Matias Bjørling
More implementations of OCSSDs are becoming available. Adding each using pci ids are becoming a hassle. Instead, use a 16 byte string in the vendor-specific area of the identification command to identify an Open-Channel SSD. The large string should make the collision probability with other vendor-