NVMe Add Support for Solid State Media Log Page
PATCH 1/1 NVMe Add Support for Solid State Media Log Page
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 1/1 NVMe Add Support for Solid State Media Log Page
Quoted 2 lines · Jeff Lien
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 1/1 NVMe Add Support for Solid State Media Log Page
This was a customer request. I?ll check to see if they are aware that the component will no longer be supported.
Quoted 4 lines · Keith Busch
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 1/1 NVMe Add Support for Solid State Media Log Page
Quoted 1 line · David Darrington
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 ...
Quoted 10 lines · Keith Busch
Quoted 5 lines
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
Christoph Hellwig
maintainerPATCH 1/1 NVMe Add Support for Solid State Media Log Page
Quoted 6 lines · Matthew Wilcox
Quoted 1 line · David Darrington
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..