lkml.org 
[lkml]   [2023]   [Jun]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2] nvme: Print capabilities changes just once
On Thu, Jun 15, 2023 at 02:49:03AM -0700, Breno Leitao wrote:
> if (effects & NVME_CMD_EFFECTS_CCC) {
> - dev_info(ctrl->device,
> + if (!test_and_set_bit(NVME_CTRL_DIRTY_CAPABILITY,
> + &ctrl->flags)) {
> + dev_info(ctrl->device,
> "controller capabilities changed, reset may be required to take effect.\n");
> + }
> }

Thanks, patch applied to nvme-6.5.

A question for Christoph and future consideration: I *think* the
complications that had the driver stop refreshing the controller
attributes are largely mitigated by the fact we now handle effects
after the request is freed. Perhaps I'm missing something. Is there
a risk in just bringing that feature back into the driver?

\
 
 \ /
  Last update: 2023-06-21 18:47    [W:0.335 / U:0.224 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site