Skip to content

fabrics: Lower log level in __nvmf_add_ctrl#351

Merged
igaw merged 1 commit intolinux-nvme:masterfrom
igaw:fix-log-level
Apr 14, 2022
Merged

fabrics: Lower log level in __nvmf_add_ctrl#351
igaw merged 1 commit intolinux-nvme:masterfrom
igaw:fix-log-level

Conversation

@igaw
Copy link
Copy Markdown
Collaborator

@igaw igaw commented Apr 14, 2022

9df8676 ("Add logging functionality to libnvme") introduced the
logging. The log level for writing to /dev/nvme-fabrics was set on
NOTICE because the kernel reports also status information such
EALREADY which is a soft error.

866c288 ("fabrics: update log level for write failures")
increased the log level without taking this into account.

Drop back to NOTICE level to avoid error message when doing

$ nvme connect-all ....
Failed to write to /dev/nvme-fabrics: Operation already in progress

Signed-off-by: Daniel Wagner dwagner@suse.de

See: linux-nvme/nvme-cli#1505

9df8676 ("Add logging functionality to libnvme") introduced the
logging. The log level for writing to /dev/nvme-fabrics was set on
NOTICE because the kernel reports also status information such
EALREADY which is a soft error.

866c288 ("fabrics: update log level for write failures")
increased the log level without taking this into account.

Drop back to NOTICE level to avoid error message when doing

  $ nvme connect-all ....
  Failed to write to /dev/nvme-fabrics: Operation already in progress

Signed-off-by: Daniel Wagner <dwagner@suse.de>
@igaw igaw merged commit a8e74b3 into linux-nvme:master Apr 14, 2022
@igaw igaw deleted the fix-log-level branch April 14, 2022 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants