Re: [PATCH 5/5 v2] nvme: LightNVM support

From: Keith Busch
Date: Thu Apr 16 2015 - 11:52:27 EST


On Thu, 16 Apr 2015, Javier GonzÃlez wrote:
On 16 Apr 2015, at 16:55, Keith Busch <keith.busch@xxxxxxxxx> wrote:

Otherwise it looks pretty good to me, but I think it would be cleaner if
the lightnvm stuff is not mixed in the same file with the standard nvme
command set. We might end up splitting nvme-core in the future anyway
for command sets and transports.

Would you be ok with having nvme-lightnvm for LightNVM specific
commands?

Sounds good to me, but I don't really have a dog in this fight. :)