Re: [PATCH v2] nvme: Print capabilities changes just once

From: Keith Busch
Date: Thu Jun 15 2023 - 14:32:05 EST


On Thu, Jun 15, 2023 at 02:49:03AM -0700, Breno Leitao wrote:
> This current dev_info() could be very verbose and being printed very
> frequently depending on some userspace application sending some specific
> commands.
>
> Just print this message once and skip it until the controller resets.
> Use a controller flag (NVME_CTRL_DIRTY_CAPABILITY) to track if the
> capability needs a reset.

Looks good.

Reviewed-by: Keith Busch <kbusch@xxxxxxxxxx>