[PATCH] NVMe Add Support for Solid State Media Log Page

[PATCH] NVMe Add Support for Solid State Media Log Page

[Jeff Lien]

Attached the subject matter patch for review.  






----------------------------------------------------------
Jeff Lien
Linux Device Driver Development
Device Host Apps and Drivers
Western Digital Corporation
e. ?jeff.lien at hgst.com
o.? +1-507-322-2416
m. +1-507-273-9124


1456350886356_PastedImage







      
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-NVMe-Add-Support-for-Solid-State-Media-Log-Page.patch
Type: application/octet-stream
Size: 3808 bytes
Desc: 0001-NVMe-Add-Support-for-Solid-State-Media-Log-Page.patch
URL: <http://lists.infradead.org/pipermail/linux-nvme/attachments/20160318/a048ed3b/attachment.obj>

[PATCH] NVMe Add Support for Solid State Media Log Page

[Keith Busch]

On Fri, Mar 18, 2016@03:43:29PM +0000, Jeff Lien wrote:
> 
> Attached the subject matter patch for review.  

The translation itself looks fine, but are we applying new SCSI
translations upstream anymore? The entire component is disabled by
default, I believe with the intention of removing it in the future.

[PATCH] NVMe Add Support for Solid State Media Log Page

[David Darrington]

This was a customer request. I?ll check to see if they are aware that the component will no longer be supported.



On 3/18/16, 11:56 AM, "Keith Busch" <keith.busch@intel.com> wrote:

>The translation itself looks fine, but are we applying new SCSI
>translations upstream anymore? The entire component is disabled by
>default, I believe with the intention of removing it in the future.
>

[PATCH] NVMe Add Support for Solid State Media Log Page

[Matthew Wilcox]

On Fri, Mar 18, 2016@06:04:06PM +0000, David Darrington wrote:
> This was a customer request. I?ll check to see if they are aware that the component will no longer be supported.

"will no longer be supported" is Christoph's idea.  I think there are far
more users of this feature than he realises, and distros will continue
to enable it.  Much as we might want to get rid of it ...

> On 3/18/16, 11:56 AM, "Keith Busch" <keith.busch@intel.com> wrote:
> 
> >The translation itself looks fine, but are we applying new SCSI
> >translations upstream anymore? The entire component is disabled by
> >default, I believe with the intention of removing it in the future.
> >
> _______________________________________________
> Linux-nvme mailing list
> Linux-nvme at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-nvme

[PATCH] NVMe Add Support for Solid State Media Log Page

[Christoph Hellwig]

On Fri, Mar 18, 2016@02:19:46PM -0400, Matthew Wilcox wrote:
> On Fri, Mar 18, 2016@06:04:06PM +0000, David Darrington wrote:
> > This was a customer request. I?ll check to see if they are aware that the component will no longer be supported.
> 
> "will no longer be supported" is Christoph's idea.  I think there are far
> more users of this feature than he realises, and distros will continue
> to enable it.  Much as we might want to get rid of it ...

I kept it around as a non-default to not break existing users.  But
I don't think we should add new feature to it.  I think we should
have a usespace library that application can use instead.  Any chance
we could the original Intel code for the translation licensed under
a permissive license for it?  It know very similar code it available
under a BSD license for the Windows driver, but it it would take a
while to clean it up..