lkml.org 
[lkml]   [2021]   [Aug]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v2] nvme: update keep alive interval when kato is modified
Date
On 8/25/21 5:42 AM JST, Sagi Grimberg wrote:
> > + if (cmd->opcode == nvme_admin_set_features &&
> > + (cmd->cdw10 & 0xFF) == NVME_FEAT_KATO) {
> > + /* ms -> s */
>
> no need for this comment.

Thanks for your comments. I will remove this /* ms -> s*/ comment.

> > + unsigned int new_kato = DIV_ROUND_UP(cmd->cdw11, 1000);
> > +
> > + nvme_update_keep_alive(ctrl, new_kato);
>
> I think you can now inline nvme_update_keep_alive here, no need to keep
> it in a function.

Does this mean the section below needs to be moved from core routine
to nvme_user_cmd_post function?
> > + dev_info(ctrl->device,
> > + "keep alive commands interval on the host is updated from %u ms to %u ms\n",
> > + ctrl->kato * 1000 / 2, new_kato * 1000 / 2);
> > +
> > + nvme_stop_keep_alive(ctrl);
> > + ctrl->kato = new_kato;
> > + nvme_start_keep_alive(ctrl);

Thanks.

\
 
 \ /
  Last update: 2021-08-25 10:27    [W:0.117 / U:0.252 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site