[PATCH 1/2] nvme-cli: add fields into identify controller data structure

Guan Junxiong guanjunxiong at huawei.com
Mon Jun 12 18:31:53 PDT 2017


Hi, Christoph

On 2017/6/13 0:11, Christoph Hellwig wrote:
> On Fri, Jun 02, 2017 at 10:25:11PM -0700, Christoph Hellwig wrote:
>> On Fri, Jun 02, 2017 at 11:57:17PM +0800, 管俊雄 wrote:
>>> Hi, Christoph
>>>
>>> On Friday, June 02, 2017 03:46 PM, Christoph Hellwig wrote:
>>>> Please also add them to the kernel copy of the header so that they
>>>> stay in sync.  Except for that his looks fine to me:
>>>>
>>>
>>> Here is the patch which adds those fields into nvme_id_ctrl structure
>>> in the kernel copy of the header. Does it looks good?
>>
>> Thanks, this looks great.  Please send it out formally as a separate
>> mail and your signoff.
> 
> Can I get your Signed-off-by?  Just noticed that it didn't seem to
> be provided anywhere when trying to apply it.
> 
Sure. An updated patch was sent as a separate formal patch a few minutes ago.
Future patches will not be lack of my Signed-off-by.
Thanks

Cheers,
Guan
.






More information about the Linux-nvme mailing list