Compare commits
No commits in common. "c8" and "c9-beta" have entirely different histories.
2
.gitignore
vendored
2
.gitignore
vendored
@ -1 +1 @@
|
|||||||
SOURCES/v1.16.tar.gz
|
SOURCES/nvme-cli-2.9.1.tar.gz
|
||||||
|
|||||||
1
.nvme-cli.metadata
Normal file
1
.nvme-cli.metadata
Normal file
@ -0,0 +1 @@
|
|||||||
|
33eaae0a2334451553952ac701f8568a52fd2a98 SOURCES/nvme-cli-2.9.1.tar.gz
|
||||||
@ -0,0 +1,20 @@
|
|||||||
|
diff --git a/fabrics.c b/fabrics.c
|
||||||
|
index 871c20ede093..90c3e5d6c602 100644
|
||||||
|
--- a/fabrics.c
|
||||||
|
+++ b/fabrics.c
|
||||||
|
@@ -643,6 +643,7 @@ char *nvmf_hostid_from_hostnqn(const char *hostnqn)
|
||||||
|
|
||||||
|
void nvmf_check_hostid_and_hostnqn(const char *hostid, const char *hostnqn, unsigned int verbose)
|
||||||
|
{
|
||||||
|
+#if 0
|
||||||
|
_cleanup_free_ char *hostid_from_file = NULL;
|
||||||
|
_cleanup_free_ char *hostid_from_hostnqn = NULL;
|
||||||
|
|
||||||
|
@@ -665,6 +666,7 @@ void nvmf_check_hostid_and_hostnqn(const char *hostid, const char *hostnqn, unsi
|
||||||
|
fprintf(stderr,
|
||||||
|
"warning: use hostid which does not match uuid in hostnqn\n");
|
||||||
|
}
|
||||||
|
+#endif
|
||||||
|
}
|
||||||
|
|
||||||
|
int nvmf_discover(const char *desc, int argc, char **argv, bool connect)
|
||||||
@ -1,32 +0,0 @@
|
|||||||
From f74ac1b841b0aae73969debaed0444f0ecb03dba Mon Sep 17 00:00:00 2001
|
|
||||||
From: Martin George <marting@netapp.com>
|
|
||||||
Date: Mon, 15 Nov 2021 16:00:47 +0530
|
|
||||||
Subject: [PATCH] fabrics: fix 'nvme connect' segfault if transport type is
|
|
||||||
omitted
|
|
||||||
|
|
||||||
Check if the transport type is available before dereferencing
|
|
||||||
it in discovery_trsvcid().
|
|
||||||
|
|
||||||
Fixes: 362c90f ("fabrics: add default port number for NVMe/TCP I/O
|
|
||||||
controllers")
|
|
||||||
Signed-off-by: Martin George <marting@netapp.com>
|
|
||||||
---
|
|
||||||
fabrics.c | 2 ++
|
|
||||||
1 file changed, 2 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/fabrics.c b/fabrics.c
|
|
||||||
index d691191..adca89b 100644
|
|
||||||
--- a/fabrics.c
|
|
||||||
+++ b/fabrics.c
|
|
||||||
@@ -1070,6 +1070,8 @@ static void set_discovery_kato(struct fabrics_config *cfg)
|
|
||||||
|
|
||||||
static void discovery_trsvcid(struct fabrics_config *fabrics_cfg, bool discover)
|
|
||||||
{
|
|
||||||
+ if (!fabrics_cfg->transport)
|
|
||||||
+ return;
|
|
||||||
if (!strcmp(fabrics_cfg->transport, "tcp")) {
|
|
||||||
if (discover) {
|
|
||||||
/* Default port for NVMe/TCP discovery controllers */
|
|
||||||
--
|
|
||||||
2.27.0
|
|
||||||
|
|
||||||
@ -1,29 +0,0 @@
|
|||||||
From ff8f9b6f59b574fda24a071fb0af4381c0cc6d9f Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Thu, 9 Dec 2021 12:21:51 +0100
|
|
||||||
Subject: [PATCH 1/6] fabrics: fix a buffer overrun
|
|
||||||
|
|
||||||
the uuid buffer size must be at least 37 bytes to avoid
|
|
||||||
corrupting the memory
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
fabrics.c | 2 +-
|
|
||||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/fabrics.c b/fabrics.c
|
|
||||||
index adca89b..0766729 100644
|
|
||||||
--- a/fabrics.c
|
|
||||||
+++ b/fabrics.c
|
|
||||||
@@ -883,7 +883,7 @@ static char *hostnqn_generate_systemd(void)
|
|
||||||
|
|
||||||
static char *hostnqn_read_dmi(void)
|
|
||||||
{
|
|
||||||
- char uuid[16];
|
|
||||||
+ char uuid[37];
|
|
||||||
char *ret = NULL;
|
|
||||||
|
|
||||||
if (uuid_from_dmi(uuid) < 0)
|
|
||||||
--
|
|
||||||
2.27.0
|
|
||||||
|
|
||||||
@ -0,0 +1,76 @@
|
|||||||
|
From 42f788edccb954fe8b54c43809523e603c99f63c Mon Sep 17 00:00:00 2001
|
||||||
|
From: Maurizio Lombardi <mlombard@redhat.com>
|
||||||
|
Date: Wed, 22 May 2024 15:06:18 +0200
|
||||||
|
Subject: [PATCH] nvme: telemetry: report the correct error if the ioctl()
|
||||||
|
fails.
|
||||||
|
|
||||||
|
It's wrong to assume that if the ioctl() returns a non-zero number
|
||||||
|
then the errno variable is set.
|
||||||
|
The ioctl() might return an NVMe Status error to inform the caller
|
||||||
|
that the requested log page is not supported, in that case errno is left
|
||||||
|
untouched.
|
||||||
|
|
||||||
|
The original code didn't handle this case and returned "-errno" even when
|
||||||
|
the latter was zero. The caller interpreted this as a successful operation
|
||||||
|
and this might lead to improperly dereferencing the log page pointer.
|
||||||
|
|
||||||
|
$ nvme telemetry-log /dev/nvme0 --output-file=telemetry_log.bin
|
||||||
|
ERROR: get_telemetry_log: : write failed with error : Bad address
|
||||||
|
|
||||||
|
Fix this bug by returning the NVMe status if errno is zero:
|
||||||
|
|
||||||
|
$ nvme telemetry-log /dev/nvme0 --output-file=telemetry_log.bin
|
||||||
|
NVMe status: Invalid Log Page: The log page indicated is invalid(0x109)
|
||||||
|
Failed to acquire telemetry log 265!
|
||||||
|
|
||||||
|
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
||||||
|
---
|
||||||
|
nvme.c | 18 ++++++++++++++----
|
||||||
|
1 file changed, 14 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/nvme.c b/nvme.c
|
||||||
|
index d722b91e5ed8..226413bc0eda 100644
|
||||||
|
--- a/nvme.c
|
||||||
|
+++ b/nvme.c
|
||||||
|
@@ -717,7 +717,10 @@ static int get_log_telemetry_ctrl(struct nvme_dev *dev, bool rae, size_t size,
|
||||||
|
err = nvme_cli_get_log_telemetry_ctrl(dev, rae, 0, size, log);
|
||||||
|
if (err) {
|
||||||
|
free(log);
|
||||||
|
- return -errno;
|
||||||
|
+ if (errno)
|
||||||
|
+ return -errno;
|
||||||
|
+ else
|
||||||
|
+ return err;
|
||||||
|
}
|
||||||
|
|
||||||
|
*buf = log;
|
||||||
|
@@ -737,7 +740,10 @@ static int get_log_telemetry_host(struct nvme_dev *dev, size_t size,
|
||||||
|
err = nvme_cli_get_log_telemetry_host(dev, 0, size, log);
|
||||||
|
if (err) {
|
||||||
|
free(log);
|
||||||
|
- return -errno;
|
||||||
|
+ if (errno)
|
||||||
|
+ return -errno;
|
||||||
|
+ else
|
||||||
|
+ return err;
|
||||||
|
}
|
||||||
|
|
||||||
|
*buf = log;
|
||||||
|
@@ -757,8 +763,12 @@ static int __create_telemetry_log_host(struct nvme_dev *dev,
|
||||||
|
return -ENOMEM;
|
||||||
|
|
||||||
|
err = nvme_cli_get_log_create_telemetry_host(dev, log);
|
||||||
|
- if (err)
|
||||||
|
- return -errno;
|
||||||
|
+ if (err) {
|
||||||
|
+ if (errno)
|
||||||
|
+ return -errno;
|
||||||
|
+ else
|
||||||
|
+ return err;
|
||||||
|
+ }
|
||||||
|
|
||||||
|
err = parse_telemetry_da(dev, da, log, size);
|
||||||
|
if (err)
|
||||||
|
--
|
||||||
|
2.43.0
|
||||||
|
|
||||||
@ -1,36 +0,0 @@
|
|||||||
From 821935cccfcad6b18da78d42f9ddf49f4cbe0b8e Mon Sep 17 00:00:00 2001
|
|
||||||
From: Daniel Wagner <dwagner@suse.de>
|
|
||||||
Date: Thu, 9 Dec 2021 15:40:26 +0100
|
|
||||||
Subject: [PATCH 2/6] bash: Fix nvme completion
|
|
||||||
|
|
||||||
Signed-off-by: Daniel Wagner <dwagner@suse.de>
|
|
||||||
---
|
|
||||||
completions/bash-nvme-completion.sh | 5 ++---
|
|
||||||
1 file changed, 2 insertions(+), 3 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/completions/bash-nvme-completion.sh b/completions/bash-nvme-completion.sh
|
|
||||||
index 41938d7..902da01 100644
|
|
||||||
--- a/completions/bash-nvme-completion.sh
|
|
||||||
+++ b/completions/bash-nvme-completion.sh
|
|
||||||
@@ -101,8 +101,7 @@ _cmds="list list-subsys id-ctrl id-ns \
|
|
||||||
connect disconnect disconnect-all gen-hostnqn \
|
|
||||||
show-hostnqn dir-receive dir-send virt-mgmt \
|
|
||||||
rpmb boot-part-log fid-support-effects-log \
|
|
||||||
- supported-log-pages lockdown"
|
|
||||||
- supported-log-pages list-endgrp"
|
|
||||||
+ supported-log-pages lockdown list-endgrp"
|
|
||||||
|
|
||||||
# Add plugins:
|
|
||||||
for plugin in "${!_plugin_subcmds[@]}"; do
|
|
||||||
@@ -1368,7 +1367,7 @@ plugin_ymtc_opts () {
|
|
||||||
opts+=" --namespace-id= -n --raw-binary -b"
|
|
||||||
;;
|
|
||||||
"help")
|
|
||||||
- opts+=NO_OPTS
|
|
||||||
+ opts+=$NO_OPTS
|
|
||||||
;;
|
|
||||||
esac
|
|
||||||
|
|
||||||
--
|
|
||||||
2.27.0
|
|
||||||
|
|
||||||
@ -0,0 +1,63 @@
|
|||||||
|
From fac45ce03315479b0d30e71316e669d9d4ffb31c Mon Sep 17 00:00:00 2001
|
||||||
|
From: Greg Joyce <gjoyce@linux.ibm.com>
|
||||||
|
Date: Fri, 10 May 2024 14:23:11 -0500
|
||||||
|
Subject: [PATCH] sed: only re-read partition table after unlock.
|
||||||
|
|
||||||
|
The partition table was being re-read after both lock and
|
||||||
|
unlock operations. The re-read would fail with an error
|
||||||
|
message after a lock since the partition was no longer readable.
|
||||||
|
|
||||||
|
Signed-off-by: Greg Joyce <gjoyce@linux.ibm.com>
|
||||||
|
---
|
||||||
|
plugins/sed/sedopal_cmd.c | 27 ++++++++++++++-------------
|
||||||
|
1 file changed, 14 insertions(+), 13 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/plugins/sed/sedopal_cmd.c b/plugins/sed/sedopal_cmd.c
|
||||||
|
index 21ebd360c219..221f62b15186 100644
|
||||||
|
--- a/plugins/sed/sedopal_cmd.c
|
||||||
|
+++ b/plugins/sed/sedopal_cmd.c
|
||||||
|
@@ -251,8 +251,21 @@ int sedopal_cmd_lock(int fd)
|
||||||
|
*/
|
||||||
|
int sedopal_cmd_unlock(int fd)
|
||||||
|
{
|
||||||
|
+ int rc;
|
||||||
|
+
|
||||||
|
+ rc = sedopal_lock_unlock(fd, OPAL_RW);
|
||||||
|
+
|
||||||
|
+ /*
|
||||||
|
+ * If the unlock was successful, force a re-read of the
|
||||||
|
+ * partition table. Return rc of unlock operation.
|
||||||
|
+ */
|
||||||
|
+ if (rc == 0) {
|
||||||
|
+ if (ioctl(fd, BLKRRPART, 0) != 0)
|
||||||
|
+ fprintf(stderr,
|
||||||
|
+ "Warning: failed re-reading partition\n");
|
||||||
|
+ }
|
||||||
|
|
||||||
|
- return sedopal_lock_unlock(fd, OPAL_RW);
|
||||||
|
+ return rc;
|
||||||
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
|
@@ -275,18 +288,6 @@ int sedopal_lock_unlock(int fd, int lock_state)
|
||||||
|
if (rc != 0)
|
||||||
|
fprintf(stderr,
|
||||||
|
"Error: failed locking or unlocking - %d\n", rc);
|
||||||
|
-
|
||||||
|
- /*
|
||||||
|
- * If the unlock was successful, force a re-read of the
|
||||||
|
- * partition table.
|
||||||
|
- */
|
||||||
|
- if (rc == 0) {
|
||||||
|
- rc = ioctl(fd, BLKRRPART, 0);
|
||||||
|
- if (rc != 0)
|
||||||
|
- fprintf(stderr,
|
||||||
|
- "Error: failed re-reading partition\n");
|
||||||
|
- }
|
||||||
|
-
|
||||||
|
return rc;
|
||||||
|
}
|
||||||
|
|
||||||
|
--
|
||||||
|
2.43.0
|
||||||
|
|
||||||
@ -1,39 +0,0 @@
|
|||||||
From 45223fd3c5a13da9209c4f44d7593cb42cab94fb Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Tue, 14 Dec 2021 10:09:42 +0100
|
|
||||||
Subject: [PATCH 3/6] nvme: do not leak an open file handle
|
|
||||||
|
|
||||||
We performed a "sec_fd=open(cfg.file)" earlier, so we should not
|
|
||||||
overwrite the handle.
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
nvme.c | 9 +--------
|
|
||||||
1 file changed, 1 insertion(+), 8 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/nvme.c b/nvme.c
|
|
||||||
index 862f9b6..5beeac7 100644
|
|
||||||
--- a/nvme.c
|
|
||||||
+++ b/nvme.c
|
|
||||||
@@ -4379,17 +4379,10 @@ static int sec_send(int argc, char **argv, struct command *cmd, struct plugin *p
|
|
||||||
fprintf(stderr, "WARNING: --tl not dword aligned; unaligned bytes may be truncated\n");
|
|
||||||
|
|
||||||
if (strlen(cfg.file) == 0) {
|
|
||||||
+ close(sec_fd);
|
|
||||||
sec_fd = STDIN_FILENO;
|
|
||||||
sec_size = cfg.tl;
|
|
||||||
} else {
|
|
||||||
- sec_fd = open(cfg.file, O_RDONLY);
|
|
||||||
- if (sec_fd < 0) {
|
|
||||||
- fprintf(stderr, "Failed to open %s: %s\n",
|
|
||||||
- cfg.file, strerror(errno));
|
|
||||||
- err = -EINVAL;
|
|
||||||
- goto close_fd;
|
|
||||||
- }
|
|
||||||
-
|
|
||||||
err = fstat(sec_fd, &sb);
|
|
||||||
if (err < 0) {
|
|
||||||
perror("fstat");
|
|
||||||
--
|
|
||||||
2.27.0
|
|
||||||
|
|
||||||
@ -1,35 +0,0 @@
|
|||||||
From b580886dbc322b2a71d6c0c1b71f9fca7b750b02 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Tue, 14 Dec 2021 09:58:09 +0100
|
|
||||||
Subject: [PATCH 4/6] nvme-topology: fix memory leaks in
|
|
||||||
nvme_logical_block_size_from_ns_char()
|
|
||||||
|
|
||||||
Free the allocated strings before returning from the function
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
nvme-topology.c | 5 ++++-
|
|
||||||
1 file changed, 4 insertions(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/nvme-topology.c b/nvme-topology.c
|
|
||||||
index 0a22f6b..6be2b09 100644
|
|
||||||
--- a/nvme-topology.c
|
|
||||||
+++ b/nvme-topology.c
|
|
||||||
@@ -686,10 +686,13 @@ int nvme_logical_block_size_from_ns_char(const char *dev)
|
|
||||||
return -EINVAL;
|
|
||||||
|
|
||||||
s = nvme_get_ctrl_attr(path, "logical_block_size");
|
|
||||||
+ free(path);
|
|
||||||
if (!s)
|
|
||||||
return -EINVAL;
|
|
||||||
|
|
||||||
- return atoi(s);
|
|
||||||
+ ret = atoi(s);
|
|
||||||
+ free(s);
|
|
||||||
+ return ret;
|
|
||||||
}
|
|
||||||
|
|
||||||
void *mmap_registers(const char *dev)
|
|
||||||
--
|
|
||||||
2.27.0
|
|
||||||
|
|
||||||
@ -1,37 +0,0 @@
|
|||||||
From 58c23ceb12d16756b2222a55d1d9dc5f34bb4fa4 Mon Sep 17 00:00:00 2001
|
|
||||||
From: James Smart <jsmart2021@gmail.com>
|
|
||||||
Date: Fri, 17 Dec 2021 11:32:32 -0800
|
|
||||||
Subject: [PATCH 5/6] nvme-cli: nvmf-connect@.service: Remove matching from
|
|
||||||
default syntax
|
|
||||||
|
|
||||||
commit 53aab69a0add added the "--matching" argument to the systemd
|
|
||||||
connect script that issues connect-all to a discovery controller. When
|
|
||||||
this argument is used, only discovery log entries whose target port
|
|
||||||
traddr's match the traddr of the discovery controller will be connected
|
|
||||||
to. This eliminates the ability to do referrals by the discovery
|
|
||||||
controller.
|
|
||||||
|
|
||||||
Revert the commit so that the "--matching" argument is not default
|
|
||||||
behavior.
|
|
||||||
|
|
||||||
Signed-off-by: James Smart <jsmart2021@gmail.com>
|
|
||||||
CC: Martin Wilck <mwilck@suse.com>
|
|
||||||
Signed-off-by: Daniel Wagner <dwagner@suse.de>
|
|
||||||
Link: https://lore.kernel.org/r/20211217193232.29034-1-jsmart2021@gmail.com
|
|
||||||
---
|
|
||||||
nvmf-autoconnect/systemd/nvmf-connect@.service | 2 +-
|
|
||||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/nvmf-autoconnect/systemd/nvmf-connect@.service b/nvmf-autoconnect/systemd/nvmf-connect@.service
|
|
||||||
index 5fbf081..c60f146 100644
|
|
||||||
--- a/nvmf-autoconnect/systemd/nvmf-connect@.service
|
|
||||||
+++ b/nvmf-autoconnect/systemd/nvmf-connect@.service
|
|
||||||
@@ -11,4 +11,4 @@ Requires=nvmf-connect.target
|
|
||||||
[Service]
|
|
||||||
Type=simple
|
|
||||||
Environment="CONNECT_ARGS=%i"
|
|
||||||
-ExecStart=/bin/sh -c "nvme connect-all --matching --quiet `/bin/echo -e '${CONNECT_ARGS}'`"
|
|
||||||
+ExecStart=/bin/sh -c "nvme connect-all --quiet `/bin/echo -e '${CONNECT_ARGS}'`"
|
|
||||||
--
|
|
||||||
2.27.0
|
|
||||||
|
|
||||||
@ -1,86 +0,0 @@
|
|||||||
From 1264c6323937c4a0342174fdd9be5a66ab1eaf24 Mon Sep 17 00:00:00 2001
|
|
||||||
From: James Smart <jsmart2021@gmail.com>
|
|
||||||
Date: Fri, 17 Dec 2021 14:20:22 -0800
|
|
||||||
Subject: [PATCH 6/6] nvme-cli: Make connect-all matching be case insensitive
|
|
||||||
|
|
||||||
The comparison routine that checks discovery controller traddr with a
|
|
||||||
discovery log traddr uses a simple strncmp. For FC, which kicks off
|
|
||||||
connect-all requests vay systemd, the nvme-fc transport will build
|
|
||||||
traddr strings with lower case hexadecimal. Some FC discovery
|
|
||||||
controllers return traddr strings with upper case hexadecimal. There
|
|
||||||
was is no rqmt in the NVME-FC spec that it be upper or lower case.
|
|
||||||
Given the case difference, the connect-all fails the match logic and
|
|
||||||
doesn't connect to storage.
|
|
||||||
|
|
||||||
Revise the traddr comparison routine to duplicate the strings and
|
|
||||||
convert them to lower case for comparison.
|
|
||||||
|
|
||||||
Signed-off-by: James Smart <jsmart2021@gmail.com>
|
|
||||||
Signed-off-by: Daniel Wagner <dwagner@suse.de>
|
|
||||||
Link: https://lore.kernel.org/r/20211217222022.30516-1-jsmart2021@gmail.com
|
|
||||||
---
|
|
||||||
fabrics.c | 26 +++++++++++++++++++++++++-
|
|
||||||
1 file changed, 25 insertions(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/fabrics.c b/fabrics.c
|
|
||||||
index 0766729..a1e2593 100644
|
|
||||||
--- a/fabrics.c
|
|
||||||
+++ b/fabrics.c
|
|
||||||
@@ -34,6 +34,7 @@
|
|
||||||
#include <stddef.h>
|
|
||||||
#include <syslog.h>
|
|
||||||
#include <time.h>
|
|
||||||
+#include <ctype.h>
|
|
||||||
|
|
||||||
#include <sys/types.h>
|
|
||||||
#include <arpa/inet.h>
|
|
||||||
@@ -681,6 +682,12 @@ static int space_strip_len(int max, const char *str)
|
|
||||||
return i + 1;
|
|
||||||
}
|
|
||||||
|
|
||||||
+static void strtolower(char *str)
|
|
||||||
+{
|
|
||||||
+ for ( ; *str; str++)
|
|
||||||
+ *str = tolower(*str);
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
static void print_discovery_log(struct nvmf_disc_rsp_page_hdr *log, int numrec,
|
|
||||||
int instance)
|
|
||||||
{
|
|
||||||
@@ -1385,7 +1392,9 @@ static bool cargs_match_found(struct nvmf_disc_rsp_page_entry *entry)
|
|
||||||
|
|
||||||
static bool should_connect(struct nvmf_disc_rsp_page_entry *entry)
|
|
||||||
{
|
|
||||||
+ char *dctrl_traddr, *log_traddr;
|
|
||||||
int len;
|
|
||||||
+ bool connect = true;
|
|
||||||
|
|
||||||
if (cargs_match_found(entry))
|
|
||||||
return false;
|
|
||||||
@@ -1398,7 +1407,22 @@ static bool should_connect(struct nvmf_disc_rsp_page_entry *entry)
|
|
||||||
return true;
|
|
||||||
|
|
||||||
len = space_strip_len(NVMF_TRADDR_SIZE, entry->traddr);
|
|
||||||
- return !strncmp(fabrics_cfg.traddr, entry->traddr, len);
|
|
||||||
+
|
|
||||||
+ dctrl_traddr = strdup(fabrics_cfg.traddr);
|
|
||||||
+ log_traddr = strndup(entry->traddr, len);
|
|
||||||
+ if (!dctrl_traddr || !log_traddr)
|
|
||||||
+ goto free_exit;
|
|
||||||
+
|
|
||||||
+ strtolower(dctrl_traddr);
|
|
||||||
+ strtolower(log_traddr);
|
|
||||||
+
|
|
||||||
+ connect = (strlen(dctrl_traddr) == len) &&
|
|
||||||
+ !strcmp(dctrl_traddr, log_traddr);
|
|
||||||
+
|
|
||||||
+free_exit:
|
|
||||||
+ free(log_traddr);
|
|
||||||
+ free(dctrl_traddr);
|
|
||||||
+ return connect;
|
|
||||||
}
|
|
||||||
|
|
||||||
static int connect_ctrls(struct nvmf_disc_rsp_page_hdr *log, int numrec)
|
|
||||||
--
|
|
||||||
2.27.0
|
|
||||||
|
|
||||||
@ -1,59 +0,0 @@
|
|||||||
From db50dbf5692325cfef8fb77e56e1e44af83a022e Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Wed, 29 Jun 2022 16:47:30 +0200
|
|
||||||
Subject: [PATCH 1/7] nvme-cli: nvme gen-hostnqn use partition UUID on IBM
|
|
||||||
POWER
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
nvme.c | 27 +++++++++++++++++++++++++--
|
|
||||||
1 file changed, 25 insertions(+), 2 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/nvme.c b/nvme.c
|
|
||||||
index 5beeac78..0535ed2b 100644
|
|
||||||
--- a/nvme.c
|
|
||||||
+++ b/nvme.c
|
|
||||||
@@ -6516,6 +6516,26 @@ static int admin_passthru(int argc, char **argv, struct command *cmd, struct plu
|
|
||||||
return passthru(argc, argv, NVME_IOCTL_ADMIN_CMD, 1, desc, cmd);
|
|
||||||
}
|
|
||||||
|
|
||||||
+#define PATH_UUID_IBM "/proc/device-tree/ibm,partition-uuid"
|
|
||||||
+
|
|
||||||
+static int uuid_from_device_tree(char *system_uuid)
|
|
||||||
+{
|
|
||||||
+ ssize_t len;
|
|
||||||
+ int f;
|
|
||||||
+
|
|
||||||
+ f = open(PATH_UUID_IBM, O_RDONLY);
|
|
||||||
+ if (f < 0)
|
|
||||||
+ return -ENXIO;
|
|
||||||
+
|
|
||||||
+ memset(system_uuid, 0, 37);
|
|
||||||
+ len = read(f, system_uuid, 37 - 1);
|
|
||||||
+ close(f);
|
|
||||||
+ if (len < 0)
|
|
||||||
+ return -ENXIO;
|
|
||||||
+
|
|
||||||
+ return strlen(system_uuid) ? 0 : -ENXIO;
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
static int gen_hostnqn_cmd(int argc, char **argv, struct command *command, struct plugin *plugin)
|
|
||||||
{
|
|
||||||
int ret;
|
|
||||||
@@ -6525,8 +6545,11 @@ static int gen_hostnqn_cmd(int argc, char **argv, struct command *command, struc
|
|
||||||
#endif
|
|
||||||
|
|
||||||
ret = uuid_from_dmi(uuid_str);
|
|
||||||
- if (ret < 0)
|
|
||||||
- ret = uuid_from_systemd(uuid_str);
|
|
||||||
+ if (ret < 0) {
|
|
||||||
+ ret = uuid_from_device_tree(uuid_str);
|
|
||||||
+ if (ret < 0)
|
|
||||||
+ ret = uuid_from_systemd(uuid_str);
|
|
||||||
+ }
|
|
||||||
#ifdef LIBUUID
|
|
||||||
if (ret < 0) {
|
|
||||||
uuid_generate_random(uuid);
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
@ -1,41 +0,0 @@
|
|||||||
From f491884513bfba7bb007428e4664c2dfda21d424 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Wed, 29 Jun 2022 17:33:10 +0200
|
|
||||||
Subject: [PATCH 2/7] Add new events support in PEL
|
|
||||||
|
|
||||||
Add two new events support in header file.
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
linux/nvme.h | 8 ++++++++
|
|
||||||
1 file changed, 8 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/linux/nvme.h b/linux/nvme.h
|
|
||||||
index 7a925c7a..8b4c6833 100644
|
|
||||||
--- a/linux/nvme.h
|
|
||||||
+++ b/linux/nvme.h
|
|
||||||
@@ -862,6 +862,12 @@ struct nvme_thermal_exc_event {
|
|
||||||
__u8 threshold;
|
|
||||||
};
|
|
||||||
|
|
||||||
+/* persistent event type 0Bh */
|
|
||||||
+struct nvme_set_feature_event {
|
|
||||||
+ __le32 layout;
|
|
||||||
+ __le32 cdw_mem[0];
|
|
||||||
+};
|
|
||||||
+
|
|
||||||
/* persistent event entry head */
|
|
||||||
struct nvme_persistent_event_entry_head {
|
|
||||||
__u8 etype;
|
|
||||||
@@ -909,6 +915,8 @@ enum nvme_persistent_event_types {
|
|
||||||
NVME_FORMAT_COMPLETION_EVENT = 0x08,
|
|
||||||
NVME_SANITIZE_START_EVENT = 0x09,
|
|
||||||
NVME_SANITIZE_COMPLETION_EVENT = 0x0a,
|
|
||||||
+ NVME_SET_FEATURE_EVENT = 0x0b,
|
|
||||||
+ NVME_TELEMETRY_CRT = 0x0c,
|
|
||||||
NVME_THERMAL_EXCURSION_EVENT = 0x0d
|
|
||||||
};
|
|
||||||
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
@ -1,146 +0,0 @@
|
|||||||
From a0db61992c14b34fcf6787b957d5d5d0e77c6f33 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Wed, 29 Jun 2022 18:00:44 +0200
|
|
||||||
Subject: [PATCH 3/7] nvme-cli: Decode "Supported Events Bitmap" in PEL header
|
|
||||||
|
|
||||||
"Supported Events Bitmap" in PEL header shows what events
|
|
||||||
are supported in current nvme devices.
|
|
||||||
|
|
||||||
Persistent Event Log for device: nvme0n1
|
|
||||||
Action for Persistent Event Log: 0
|
|
||||||
..
|
|
||||||
..
|
|
||||||
..
|
|
||||||
Supported Events Bitmap:
|
|
||||||
Support SMART/Health Log Snapshot Event(0x1)
|
|
||||||
Support Firmware Commit Event(0x2)
|
|
||||||
Support Timestamp Change Event(0x3)
|
|
||||||
Support Power-on or Reset Event(0x4)
|
|
||||||
Support NVM Subsystem Hardware Error Event(0x5)
|
|
||||||
Support Change Namespace Event(0x6)
|
|
||||||
Support Format NVM Start Event(0x7)
|
|
||||||
Support Format NVM Completion Event(0x8)
|
|
||||||
Support Sanitize Start Event(0x9)
|
|
||||||
Support Sanitize Completion Event(0xa)
|
|
||||||
Support Set Feature Event(0xb)
|
|
||||||
Support Set Telemetry CRT Event(0xc)
|
|
||||||
Support Thermal Excursion Event(0xd)
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
nvme-print.c | 52 ++++++++++++++++++++++++++++++++++++++++++++++------
|
|
||||||
nvme.c | 4 ++--
|
|
||||||
2 files changed, 48 insertions(+), 8 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/nvme-print.c b/nvme-print.c
|
|
||||||
index f631b347..21524a50 100755
|
|
||||||
--- a/nvme-print.c
|
|
||||||
+++ b/nvme-print.c
|
|
||||||
@@ -1015,6 +1015,26 @@ void nvme_show_predictable_latency_event_agg_log(
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
+const char *nvme_pel_event_to_string(int type)
|
|
||||||
+{
|
|
||||||
+ switch (type) {
|
|
||||||
+ case NVME_SMART_HEALTH_EVENT: return "SMART/Health Log Snapshot Event(0x1)";
|
|
||||||
+ case NVME_FW_COMMIT_EVENT: return "Firmware Commit Event(0x2)";
|
|
||||||
+ case NVME_TIMESTAMP_EVENT: return "Timestamp Change Event(0x3)";
|
|
||||||
+ case NVME_POWER_ON_RESET_EVENT: return "Power-on or Reset Event(0x4)";
|
|
||||||
+ case NVME_NSS_HW_ERROR_EVENT: return "NVM Subsystem Hardware Error Event(0x5)";
|
|
||||||
+ case NVME_CHANGE_NS_EVENT: return "Change Namespace Event(0x6)";
|
|
||||||
+ case NVME_FORMAT_START_EVENT: return "Format NVM Start Event(0x7)";
|
|
||||||
+ case NVME_FORMAT_COMPLETION_EVENT: return "Format NVM Completion Event(0x8)";
|
|
||||||
+ case NVME_SANITIZE_START_EVENT: return "Sanitize Start Event(0x9)";
|
|
||||||
+ case NVME_SANITIZE_COMPLETION_EVENT: return "Sanitize Completion Event(0xa)";
|
|
||||||
+ case NVME_SET_FEATURE_EVENT: return "Set Feature Event(0xb)";
|
|
||||||
+ case NVME_TELEMETRY_CRT: return "Set Telemetry CRT Event(0xc)";
|
|
||||||
+ case NVME_THERMAL_EXCURSION_EVENT: return "Thermal Excursion Event(0xd)";
|
|
||||||
+ default: return NULL;
|
|
||||||
+ }
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
static const char *nvme_show_nss_hw_error(__u16 error_code)
|
|
||||||
{
|
|
||||||
switch (error_code) {
|
|
||||||
@@ -1043,6 +1063,29 @@ static const char *nvme_show_nss_hw_error(__u16 error_code)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
+static void add_bitmap(int i, __u8 seb, struct json_object *root, int json_flag)
|
|
||||||
+{
|
|
||||||
+ char evt_str[50];
|
|
||||||
+ char key[128];
|
|
||||||
+
|
|
||||||
+ for (int bit = 0; bit < 8; bit++) {
|
|
||||||
+ if (nvme_pel_event_to_string(bit + i * 8)) {
|
|
||||||
+ if (json_flag == 1) {
|
|
||||||
+ sprintf(key, "bitmap_%x", (bit + i * 8));
|
|
||||||
+ if ((seb >> bit) & 0x1)
|
|
||||||
+ snprintf(evt_str, sizeof(evt_str), "Support %s",
|
|
||||||
+ nvme_pel_event_to_string(bit + i * 8));
|
|
||||||
+ json_object_add_value_string(root, key, evt_str);
|
|
||||||
+ } else {
|
|
||||||
+ if (nvme_pel_event_to_string(bit + i * 8))
|
|
||||||
+ if ((seb >> bit) & 0x1)
|
|
||||||
+ printf(" Support %s\n",
|
|
||||||
+ nvme_pel_event_to_string(bit + i * 8));
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
void json_persistent_event_log(void *pevent_log_info, __u32 size)
|
|
||||||
{
|
|
||||||
struct json_object *root;
|
|
||||||
@@ -1112,9 +1155,7 @@ void json_persistent_event_log(void *pevent_log_info, __u32 size)
|
|
||||||
for (int i = 0; i < 32; i++) {
|
|
||||||
if (pevent_log_head->supp_event_bm[i] == 0)
|
|
||||||
continue;
|
|
||||||
- sprintf(key, "bitmap_%d", i);
|
|
||||||
- json_object_add_value_uint(root, key,
|
|
||||||
- pevent_log_head->supp_event_bm[i]);
|
|
||||||
+ add_bitmap(i, pevent_log_head->supp_event_bm[i], root, 1);
|
|
||||||
}
|
|
||||||
} else {
|
|
||||||
printf("No log data can be shown with this log len at least " \
|
|
||||||
@@ -1440,12 +1481,11 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
le32_to_cpu(pevent_log_head->rci));
|
|
||||||
if(human)
|
|
||||||
nvme_show_persistent_event_log_rci(pevent_log_head->rci);
|
|
||||||
- printf("Supported Events Bitmap: ");
|
|
||||||
+ printf("Supported Events Bitmap: \n");
|
|
||||||
for (int i = 0; i < 32; i++) {
|
|
||||||
if (pevent_log_head->supp_event_bm[i] == 0)
|
|
||||||
continue;
|
|
||||||
- printf("BitMap[%d] is 0x%x\n", i,
|
|
||||||
- pevent_log_head->supp_event_bm[i]);
|
|
||||||
+ add_bitmap(i, pevent_log_head->supp_event_bm[i], NULL, 0);
|
|
||||||
}
|
|
||||||
} else {
|
|
||||||
printf("No log data can be shown with this log len at least " \
|
|
||||||
diff --git a/nvme.c b/nvme.c
|
|
||||||
index 0535ed2b..44c53e6b 100644
|
|
||||||
--- a/nvme.c
|
|
||||||
+++ b/nvme.c
|
|
||||||
@@ -4808,7 +4808,7 @@ static int dsm(int argc, char **argv, struct command *cmd, struct plugin *plugin
|
|
||||||
|
|
||||||
nc = argconfig_parse_comma_sep_array(cfg.ctx_attrs, ctx_attrs, ARRAY_SIZE(ctx_attrs));
|
|
||||||
nb = argconfig_parse_comma_sep_array(cfg.blocks, nlbs, ARRAY_SIZE(nlbs));
|
|
||||||
- ns = argconfig_parse_comma_sep_array_long(cfg.slbas, slbas, ARRAY_SIZE(slbas));
|
|
||||||
+ ns = argconfig_parse_comma_sep_array_long(cfg.slbas, (long long unsigned int *)slbas, ARRAY_SIZE(slbas));
|
|
||||||
nr = max(nc, max(nb, ns));
|
|
||||||
if (!nr || nr > 256) {
|
|
||||||
fprintf(stderr, "No range definition provided\n");
|
|
||||||
@@ -4963,7 +4963,7 @@ static int copy(int argc, char **argv, struct command *cmd, struct plugin *plugi
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
- copy = nvme_setup_copy_range(nlbs, slbas, eilbrts, elbatms, elbats, nr);
|
|
||||||
+ copy = nvme_setup_copy_range(nlbs, (__u64 *)slbas, eilbrts, elbatms, elbats, nr);
|
|
||||||
if (!copy) {
|
|
||||||
fprintf(stderr, "failed to allocate payload\n");
|
|
||||||
errno = ENOMEM;
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
@ -1,141 +0,0 @@
|
|||||||
From 6c916ee527e14bf4c4640ffeef8efdd891ba9ef4 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Thu, 30 Jun 2022 16:17:44 +0200
|
|
||||||
Subject: [PATCH 4/7] nvme-cli: Adds event number in persistent event entries
|
|
||||||
|
|
||||||
Persistent Event Entries:
|
|
||||||
Event Number: 0
|
|
||||||
Event Type: SMART/Health Log Snapshot Event(0x1)
|
|
||||||
Event Type Revision: 1
|
|
||||||
Event Header Length: 21
|
|
||||||
Controller Identifier: 66
|
|
||||||
Event Timestamp: 44392
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
nvme-print.c | 30 ++++++++++++++++--------------
|
|
||||||
1 file changed, 16 insertions(+), 14 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/nvme-print.c b/nvme-print.c
|
|
||||||
index 21524a50..3b82b174 100755
|
|
||||||
--- a/nvme-print.c
|
|
||||||
+++ b/nvme-print.c
|
|
||||||
@@ -1174,8 +1174,9 @@ void json_persistent_event_log(void *pevent_log_info, __u32 size)
|
|
||||||
break;
|
|
||||||
valid_attrs = json_create_object();
|
|
||||||
|
|
||||||
- json_object_add_value_uint(valid_attrs, "event_type",
|
|
||||||
- pevent_entry_head->etype);
|
|
||||||
+ json_object_add_value_uint(valid_attrs, "event_number", i);
|
|
||||||
+ json_object_add_value_string(valid_attrs, "event_type",
|
|
||||||
+ nvme_pel_event_to_string(pevent_entry_head->etype));
|
|
||||||
json_object_add_value_uint(valid_attrs, "event_type_rev",
|
|
||||||
pevent_entry_head->etype_rev);
|
|
||||||
json_object_add_value_uint(valid_attrs, "event_header_len",
|
|
||||||
@@ -1505,7 +1506,8 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
le16_to_cpu(pevent_entry_head->el)) >= size)
|
|
||||||
break;
|
|
||||||
|
|
||||||
- printf("Event Type: %u\n", pevent_entry_head->etype);
|
|
||||||
+ printf("Event Number: %u\n", i);
|
|
||||||
+ printf("Event Type: %s\n", nvme_pel_event_to_string(pevent_entry_head->etype));
|
|
||||||
printf("Event Type Revision: %u\n", pevent_entry_head->etype_rev);
|
|
||||||
printf("Event Header Length: %u\n", pevent_entry_head->ehl);
|
|
||||||
printf("Controller Identifier: %u\n",
|
|
||||||
@@ -1521,12 +1523,12 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
switch (pevent_entry_head->etype) {
|
|
||||||
case NVME_SMART_HEALTH_EVENT:
|
|
||||||
smart_event = pevent_log_info + offset;
|
|
||||||
- printf("Smart Health Event: \n");
|
|
||||||
+ printf("Smart Health Event Entry: \n");
|
|
||||||
nvme_show_smart_log(smart_event, NVME_NSID_ALL, devname, flags);
|
|
||||||
break;
|
|
||||||
case NVME_FW_COMMIT_EVENT:
|
|
||||||
fw_commit_event = pevent_log_info + offset;
|
|
||||||
- printf("FW Commit Event: \n");
|
|
||||||
+ printf("FW Commit Event Entry: \n");
|
|
||||||
printf("Old Firmware Revision: %"PRIu64"\n",
|
|
||||||
le64_to_cpu(fw_commit_event->old_fw_rev));
|
|
||||||
printf("New Firmware Revision: %"PRIu64"\n",
|
|
||||||
@@ -1543,7 +1545,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
break;
|
|
||||||
case NVME_TIMESTAMP_EVENT:
|
|
||||||
ts_change_event = pevent_log_info + offset;
|
|
||||||
- printf("Time Stamp Change Event: \n");
|
|
||||||
+ printf("Time Stamp Change Event Entry: \n");
|
|
||||||
printf("Previous Timestamp: %"PRIu64"\n",
|
|
||||||
le64_to_cpu(ts_change_event->previous_timestamp));
|
|
||||||
printf("Milliseconds Since Reset: %"PRIu64"\n",
|
|
||||||
@@ -1555,7 +1557,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
|
|
||||||
por_info_list = por_info_len / sizeof(*por_event);
|
|
||||||
|
|
||||||
- printf("Power On Reset Event: \n");
|
|
||||||
+ printf("Power On Reset Event Entry: \n");
|
|
||||||
fw_rev = pevent_log_info + offset;
|
|
||||||
printf("Firmware Revision: %"PRIu64"\n", le64_to_cpu(*fw_rev));
|
|
||||||
printf("Reset Information List: \n");
|
|
||||||
@@ -1578,13 +1580,13 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
break;
|
|
||||||
case NVME_NSS_HW_ERROR_EVENT:
|
|
||||||
nss_hw_err_event = pevent_log_info + offset;
|
|
||||||
- printf("NVM Subsystem Hardware Error Event Code: %u, %s\n",
|
|
||||||
+ printf("NVM Subsystem Hardware Error Event Code Entry: %u, %s\n",
|
|
||||||
le16_to_cpu(nss_hw_err_event->nss_hw_err_event_code),
|
|
||||||
nvme_show_nss_hw_error(nss_hw_err_event->nss_hw_err_event_code));
|
|
||||||
break;
|
|
||||||
case NVME_CHANGE_NS_EVENT:
|
|
||||||
ns_event = pevent_log_info + offset;
|
|
||||||
- printf("Change Namespace Event: \n");
|
|
||||||
+ printf("Change Namespace Event Entry: \n");
|
|
||||||
printf("Namespace Management CDW10: %u\n",
|
|
||||||
le32_to_cpu(ns_event->nsmgt_cdw10));
|
|
||||||
printf("Namespace Size: %"PRIu64"\n",
|
|
||||||
@@ -1603,7 +1605,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
break;
|
|
||||||
case NVME_FORMAT_START_EVENT:
|
|
||||||
format_start_event = pevent_log_info + offset;
|
|
||||||
- printf("Format NVM Start Event: \n");
|
|
||||||
+ printf("Format NVM Start Event Entry: \n");
|
|
||||||
printf("Namespace Identifier: %u\n",
|
|
||||||
le32_to_cpu(format_start_event->nsid));
|
|
||||||
printf("Format NVM Attributes: %u\n",
|
|
||||||
@@ -1613,7 +1615,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
break;
|
|
||||||
case NVME_FORMAT_COMPLETION_EVENT:
|
|
||||||
format_cmpln_event = pevent_log_info + offset;
|
|
||||||
- printf("Format NVM Completion Event: \n");
|
|
||||||
+ printf("Format NVM Completion Event Entry: \n");
|
|
||||||
printf("Namespace Identifier: %u\n",
|
|
||||||
le32_to_cpu(format_cmpln_event->nsid));
|
|
||||||
printf("Smallest Format Progress Indicator: %u\n",
|
|
||||||
@@ -1627,7 +1629,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
break;
|
|
||||||
case NVME_SANITIZE_START_EVENT:
|
|
||||||
sanitize_start_event = pevent_log_info + offset;
|
|
||||||
- printf("Sanitize Start Event: \n");
|
|
||||||
+ printf("Sanitize Start Event Entry: \n");
|
|
||||||
printf("SANICAP: %u\n", sanitize_start_event->sani_cap);
|
|
||||||
printf("Sanitize CDW10: %u\n",
|
|
||||||
le32_to_cpu(sanitize_start_event->sani_cdw10));
|
|
||||||
@@ -1636,7 +1638,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
break;
|
|
||||||
case NVME_SANITIZE_COMPLETION_EVENT:
|
|
||||||
sanitize_cmpln_event = pevent_log_info + offset;
|
|
||||||
- printf("Sanitize Completion Event: \n");
|
|
||||||
+ printf("Sanitize Completion Event Entry: \n");
|
|
||||||
printf("Sanitize Progress: %u\n",
|
|
||||||
le16_to_cpu(sanitize_cmpln_event->sani_prog));
|
|
||||||
printf("Sanitize Status: %u\n",
|
|
||||||
@@ -1646,7 +1648,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
break;
|
|
||||||
case NVME_THERMAL_EXCURSION_EVENT:
|
|
||||||
thermal_exc_event = pevent_log_info + offset;
|
|
||||||
- printf("Thermal Excursion Event: \n");
|
|
||||||
+ printf("Thermal Excursion Event Entry: \n");
|
|
||||||
printf("Over Temperature: %u\n", thermal_exc_event->over_temp);
|
|
||||||
printf("Threshold: %u\n", thermal_exc_event->threshold);
|
|
||||||
break;
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
@ -1,113 +0,0 @@
|
|||||||
From 7e858ab53eb23083205d75f590ca2c2b256c2a7b Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Thu, 30 Jun 2022 16:37:40 +0200
|
|
||||||
Subject: [PATCH 5/7] nvme-cli: Adds readable firmware level in persistent
|
|
||||||
|
|
||||||
For example, In Firmware Commit Event:
|
|
||||||
|
|
||||||
Old Firmware Revision: 3617604718875264338
|
|
||||||
New Firmware Revision: 3833777500989048146
|
|
||||||
|
|
||||||
changes to
|
|
||||||
Old Firmware Revision: 3617604718875264338 (REV.SP42)
|
|
||||||
New Firmware Revision: 3833777500989048146 (REV.SP45)
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
nvme-print.c | 43 ++++++++++++++++++++++++++++---------------
|
|
||||||
1 file changed, 28 insertions(+), 15 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/nvme-print.c b/nvme-print.c
|
|
||||||
index 3b82b174..74ecd95a 100755
|
|
||||||
--- a/nvme-print.c
|
|
||||||
+++ b/nvme-print.c
|
|
||||||
@@ -104,12 +104,15 @@ const char *nvme_cmd_to_string(int admin, __u8 opcode)
|
|
||||||
return "Unknown";
|
|
||||||
}
|
|
||||||
|
|
||||||
-static char *fw_to_string(__u8 fw[])
|
|
||||||
+static const char *fw_to_string(char *c)
|
|
||||||
{
|
|
||||||
- static char frs[9];
|
|
||||||
+ static char ret[9];
|
|
||||||
+ int i;
|
|
||||||
|
|
||||||
- snprintf(frs, sizeof(frs), "%-.*s", 8, fw);
|
|
||||||
- return frs;
|
|
||||||
+ for (i = 0; i < 8; i++)
|
|
||||||
+ ret[i] = c[i] >= '!' && c[i] <= '~' ? c[i] : '.';
|
|
||||||
+ ret[i] = '\0';
|
|
||||||
+ return ret;
|
|
||||||
}
|
|
||||||
|
|
||||||
static const char *get_sanitize_log_sstat_status_str(__u16 status)
|
|
||||||
@@ -1094,6 +1097,7 @@ void json_persistent_event_log(void *pevent_log_info, __u32 size)
|
|
||||||
__u32 offset, por_info_len, por_info_list;
|
|
||||||
__u64 *fw_rev;
|
|
||||||
char key[128];
|
|
||||||
+ char fw_str[50];
|
|
||||||
struct nvme_smart_log *smart_event;
|
|
||||||
struct nvme_fw_commit_event *fw_commit_event;
|
|
||||||
struct nvme_time_stamp_change_event *ts_change_event;
|
|
||||||
@@ -1266,10 +1270,14 @@ void json_persistent_event_log(void *pevent_log_info, __u32 size)
|
|
||||||
break;
|
|
||||||
case NVME_FW_COMMIT_EVENT:
|
|
||||||
fw_commit_event = pevent_log_info + offset;
|
|
||||||
- json_object_add_value_uint(valid_attrs, "old_fw_rev",
|
|
||||||
- le64_to_cpu(fw_commit_event->old_fw_rev));
|
|
||||||
- json_object_add_value_uint(valid_attrs, "new_fw_rev",
|
|
||||||
- le64_to_cpu(fw_commit_event->new_fw_rev));
|
|
||||||
+ snprintf(fw_str, sizeof(fw_str), "%"PRIu64" (%s)",
|
|
||||||
+ le64_to_cpu(fw_commit_event->old_fw_rev),
|
|
||||||
+ fw_to_string((char *)&fw_commit_event->old_fw_rev));
|
|
||||||
+ json_object_add_value_string(valid_attrs, "old_fw_rev", fw_str);
|
|
||||||
+ snprintf(fw_str, sizeof(fw_str), "%"PRIu64" (%s)",
|
|
||||||
+ le64_to_cpu(fw_commit_event->new_fw_rev),
|
|
||||||
+ fw_to_string((char *)&fw_commit_event->new_fw_rev));
|
|
||||||
+ json_object_add_value_string(valid_attrs, "new_fw_rev", fw_str);
|
|
||||||
json_object_add_value_uint(valid_attrs, "fw_commit_action",
|
|
||||||
fw_commit_event->fw_commit_action);
|
|
||||||
json_object_add_value_uint(valid_attrs, "fw_slot",
|
|
||||||
@@ -1297,8 +1305,10 @@ void json_persistent_event_log(void *pevent_log_info, __u32 size)
|
|
||||||
por_info_list = por_info_len / sizeof(*por_event);
|
|
||||||
|
|
||||||
fw_rev = pevent_log_info + offset;
|
|
||||||
- json_object_add_value_uint(valid_attrs, "fw_rev",
|
|
||||||
- le64_to_cpu(*fw_rev));
|
|
||||||
+ snprintf(fw_str, sizeof(fw_str), "%"PRIu64" (%s)",
|
|
||||||
+ le64_to_cpu(*fw_rev),
|
|
||||||
+ fw_to_string((char *)fw_rev));
|
|
||||||
+ json_object_add_value_string(valid_attrs, "fw_rev", fw_str);
|
|
||||||
for (int i = 0; i < por_info_list; i++) {
|
|
||||||
por_event = pevent_log_info + offset +
|
|
||||||
sizeof(*fw_rev) + i * sizeof(*por_event);
|
|
||||||
@@ -1529,10 +1539,12 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
case NVME_FW_COMMIT_EVENT:
|
|
||||||
fw_commit_event = pevent_log_info + offset;
|
|
||||||
printf("FW Commit Event Entry: \n");
|
|
||||||
- printf("Old Firmware Revision: %"PRIu64"\n",
|
|
||||||
- le64_to_cpu(fw_commit_event->old_fw_rev));
|
|
||||||
- printf("New Firmware Revision: %"PRIu64"\n",
|
|
||||||
- le64_to_cpu(fw_commit_event->new_fw_rev));
|
|
||||||
+ printf("Old Firmware Revision: %"PRIu64" (%s)\n",
|
|
||||||
+ le64_to_cpu(fw_commit_event->old_fw_rev),
|
|
||||||
+ fw_to_string((char *)&fw_commit_event->old_fw_rev));
|
|
||||||
+ printf("New Firmware Revision: %"PRIu64" (%s)\n",
|
|
||||||
+ le64_to_cpu(fw_commit_event->new_fw_rev),
|
|
||||||
+ fw_to_string((char *)&fw_commit_event->new_fw_rev));
|
|
||||||
printf("FW Commit Action: %u\n",
|
|
||||||
fw_commit_event->fw_commit_action);
|
|
||||||
printf("FW Slot: %u\n", fw_commit_event->fw_slot);
|
|
||||||
@@ -1559,7 +1571,8 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
|
|
||||||
printf("Power On Reset Event Entry: \n");
|
|
||||||
fw_rev = pevent_log_info + offset;
|
|
||||||
- printf("Firmware Revision: %"PRIu64"\n", le64_to_cpu(*fw_rev));
|
|
||||||
+ printf("Firmware Revision: %"PRIu64" (%s)\n", le64_to_cpu(*fw_rev),
|
|
||||||
+ fw_to_string((char *)fw_rev));
|
|
||||||
printf("Reset Information List: \n");
|
|
||||||
|
|
||||||
for (int i = 0; i < por_info_list; i++) {
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
@ -1,69 +0,0 @@
|
|||||||
From 859b8ee99a725bdcfbb7c8352c50449d126cffc4 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Thu, 30 Jun 2022 16:41:59 +0200
|
|
||||||
Subject: [PATCH 6/7] nvme-cli: Add support set feature event in PEL
|
|
||||||
|
|
||||||
Add "Set Feature" event in PEL.
|
|
||||||
|
|
||||||
Persistent Event Entries:
|
|
||||||
Event Number: 0
|
|
||||||
Event Type: Set Feature Event(0xb)
|
|
||||||
Event Type Revision: 1
|
|
||||||
Event Header Length: 21
|
|
||||||
Controller Identifier: 65
|
|
||||||
Event Timestamp: 564587204146155
|
|
||||||
Vendor Specific Information Length: 0
|
|
||||||
Event Length: 16
|
|
||||||
Set Feature Event Entry:
|
|
||||||
Set Feature ID :0x7 (Number of Queues), value:0x270027
|
|
||||||
Number of IO Completion Queues Allocated (NCQA): 40
|
|
||||||
Number of IO Submission Queues Allocated (NSQA): 40
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
nvme-print.c | 15 +++++++++++++++
|
|
||||||
1 file changed, 15 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/nvme-print.c b/nvme-print.c
|
|
||||||
index 74ecd95a..37011dbe 100755
|
|
||||||
--- a/nvme-print.c
|
|
||||||
+++ b/nvme-print.c
|
|
||||||
@@ -1436,6 +1436,8 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
{
|
|
||||||
__u32 offset, por_info_len, por_info_list;
|
|
||||||
__u64 *fw_rev;
|
|
||||||
+ int fid, cdw11, dword_cnt;
|
|
||||||
+ unsigned char *mem_buf = NULL;
|
|
||||||
struct nvme_smart_log *smart_event;
|
|
||||||
struct nvme_fw_commit_event *fw_commit_event;
|
|
||||||
struct nvme_time_stamp_change_event *ts_change_event;
|
|
||||||
@@ -1446,6 +1448,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
struct nvme_format_nvm_compln_event *format_cmpln_event;
|
|
||||||
struct nvme_sanitize_start_event *sanitize_start_event;
|
|
||||||
struct nvme_sanitize_compln_event *sanitize_cmpln_event;
|
|
||||||
+ struct nvme_set_feature_event *set_feat_event;
|
|
||||||
struct nvme_thermal_exc_event *thermal_exc_event;
|
|
||||||
struct nvme_persistent_event_log_head *pevent_log_head;
|
|
||||||
struct nvme_persistent_event_entry_head *pevent_entry_head;
|
|
||||||
@@ -1659,6 +1662,18 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
printf("Completion Information: %u\n",
|
|
||||||
le16_to_cpu(sanitize_cmpln_event->cmpln_info));
|
|
||||||
break;
|
|
||||||
+ case NVME_SET_FEATURE_EVENT:
|
|
||||||
+ set_feat_event = pevent_log_info + offset;
|
|
||||||
+ printf("Set Feature Event Entry: \n");
|
|
||||||
+ dword_cnt = set_feat_event->layout & 0x03;
|
|
||||||
+ fid = le32_to_cpu(set_feat_event->cdw_mem[0]) & 0x000f;
|
|
||||||
+ cdw11 = le32_to_cpu(set_feat_event->cdw_mem[1]);
|
|
||||||
+ if (((set_feat_event->layout & 0xff) >> 2) != 0)
|
|
||||||
+ mem_buf = (unsigned char *)(set_feat_event + 4 + dword_cnt * 4);
|
|
||||||
+ printf("Set Feature ID :%#02x (%s), value:%#08x\n", fid,
|
|
||||||
+ nvme_feature_to_string(fid), cdw11);
|
|
||||||
+ nvme_feature_show_fields(fid, cdw11, mem_buf);
|
|
||||||
+ break;
|
|
||||||
case NVME_THERMAL_EXCURSION_EVENT:
|
|
||||||
thermal_exc_event = pevent_log_info + offset;
|
|
||||||
printf("Thermal Excursion Event Entry: \n");
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
@ -1,29 +0,0 @@
|
|||||||
From 44b23d2daf246db0ac09a4a79a9a5161843a4936 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Thu, 30 Jun 2022 16:44:07 +0200
|
|
||||||
Subject: [PATCH 7/7] nvme-cli: Add support Telemetry CRT in PEL
|
|
||||||
|
|
||||||
Add "Telemetry CRT" event in PEL.
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
nvme-print.c | 3 +++
|
|
||||||
1 file changed, 3 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/nvme-print.c b/nvme-print.c
|
|
||||||
index 37011dbe..992b9b83 100755
|
|
||||||
--- a/nvme-print.c
|
|
||||||
+++ b/nvme-print.c
|
|
||||||
@@ -1674,6 +1674,9 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
|
|
||||||
nvme_feature_to_string(fid), cdw11);
|
|
||||||
nvme_feature_show_fields(fid, cdw11, mem_buf);
|
|
||||||
break;
|
|
||||||
+ case NVME_TELEMETRY_CRT:
|
|
||||||
+ d(pevent_log_info + offset, 512, 16, 1);
|
|
||||||
+ break;
|
|
||||||
case NVME_THERMAL_EXCURSION_EVENT:
|
|
||||||
thermal_exc_event = pevent_log_info + offset;
|
|
||||||
printf("Thermal Excursion Event Entry: \n");
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
@ -1,26 +0,0 @@
|
|||||||
From b3db6509e1f70bdde9205ab97bd87fbb94a49f0a Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Fri, 15 Jul 2022 11:10:59 +0200
|
|
||||||
Subject: [PATCH] fix firmware log page frs variable sign
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
linux/nvme.h | 2 +-
|
|
||||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/linux/nvme.h b/linux/nvme.h
|
|
||||||
index 8b4c6833..2c402688 100644
|
|
||||||
--- a/linux/nvme.h
|
|
||||||
+++ b/linux/nvme.h
|
|
||||||
@@ -1539,7 +1539,7 @@ struct nvme_error_log_page {
|
|
||||||
struct nvme_firmware_log_page {
|
|
||||||
__u8 afi;
|
|
||||||
__u8 resv[7];
|
|
||||||
- __u8 frs[7][8];
|
|
||||||
+ char frs[7][8];
|
|
||||||
__u8 resv2[448];
|
|
||||||
};
|
|
||||||
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
@ -1,19 +0,0 @@
|
|||||||
From f9b6c2100db88e9f317f15f17faaed59b725ac9b Mon Sep 17 00:00:00 2001
|
|
||||||
From: Keith Busch <kbusch@kernel.org>
|
|
||||||
Date: Tue, 24 Aug 2021 11:49:16 -0700
|
|
||||||
Subject: [PATCH] fix file permissions (nvme-print.c)
|
|
||||||
|
|
||||||
It's not executable.
|
|
||||||
|
|
||||||
Signed-off-by: Keith Busch <kbusch@kernel.org>
|
|
||||||
---
|
|
||||||
nvme-print.c | 0
|
|
||||||
1 file changed, 0 insertions(+), 0 deletions(-)
|
|
||||||
mode change 100755 => 100644 nvme-print.c
|
|
||||||
|
|
||||||
diff --git a/nvme-print.c b/nvme-print.c
|
|
||||||
old mode 100755
|
|
||||||
new mode 100644
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
@ -1,65 +0,0 @@
|
|||||||
From ad9f35c0bf8a71c6a4b7586d7553b8e9d171f48e Mon Sep 17 00:00:00 2001
|
|
||||||
From: Daniel Wagner <dwagner@suse.de>
|
|
||||||
Date: Tue, 7 Mar 2023 10:24:01 +0100
|
|
||||||
Subject: [PATCH] fabrics: Fix ordering for auto connect services
|
|
||||||
|
|
||||||
In order to be able to mount file systems via /etc/fstab we have to
|
|
||||||
make sure that the corresponding auto connect services have been
|
|
||||||
executed. Because the mounting of the local filesystem happens very
|
|
||||||
early in the boot we have to carefully sort these service file into the
|
|
||||||
boot process.
|
|
||||||
|
|
||||||
First, we have to disable the DefaultDependency as this will
|
|
||||||
automatically add dependency on sysinit.target which is too late (after
|
|
||||||
local mounts). Though without the default dependency we have to provide
|
|
||||||
a Before and After conditions.
|
|
||||||
|
|
||||||
The Before is simple as we have a local-fs-pre target. The After
|
|
||||||
is a bit tricky as there are no targets available.
|
|
||||||
|
|
||||||
Because the whole autoconnect machinery depends on udev events being
|
|
||||||
delivered we place the service after systemd-udevd has been started.
|
|
||||||
|
|
||||||
Link: https://www.freedesktop.org/software/systemd/man/bootup.html
|
|
||||||
Reported-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Reported-by: Wen Xiong <wenxiong@linux.ibm.com>
|
|
||||||
Tested-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Tested-by: Wen Xiong <wenxiong@linux.ibm.com>
|
|
||||||
Signed-off-by: Daniel Wagner <dwagner@suse.de>
|
|
||||||
---
|
|
||||||
nvmf-autoconnect/systemd/nvmefc-boot-connections.service.in | 3 +++
|
|
||||||
nvmf-autoconnect/systemd/nvmf-connect@.service.in | 4 +++-
|
|
||||||
2 files changed, 6 insertions(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/nvmf-autoconnect/systemd/nvmefc-boot-connections.service b/nvmf-autoconnect/systemd/nvmefc-boot-connections.service
|
|
||||||
index 33ab8c1f..7036625c 100644
|
|
||||||
--- a/nvmf-autoconnect/systemd/nvmefc-boot-connections.service
|
|
||||||
+++ b/nvmf-autoconnect/systemd/nvmefc-boot-connections.service
|
|
||||||
@@ -1,6 +1,9 @@
|
|
||||||
[Unit]
|
|
||||||
Description=Auto-connect to subsystems on FC-NVME devices found during boot
|
|
||||||
ConditionPathExists=/sys/class/fc/fc_udev_device/nvme_discovery
|
|
||||||
+DefaultDependencies=no
|
|
||||||
+After=systemd-udevd.service
|
|
||||||
+Before=local-fs-pre.target
|
|
||||||
|
|
||||||
[Service]
|
|
||||||
Type=oneshot
|
|
||||||
diff --git a/nvmf-autoconnect/systemd/nvmf-connect@.service b/nvmf-autoconnect/systemd/nvmf-connect@.service
|
|
||||||
index 90f774c5..dd245ee6 100644
|
|
||||||
--- a/nvmf-autoconnect/systemd/nvmf-connect@.service
|
|
||||||
+++ b/nvmf-autoconnect/systemd/nvmf-connect@.service
|
|
||||||
@@ -4,7 +4,9 @@
|
|
||||||
|
|
||||||
[Unit]
|
|
||||||
Description=NVMf auto-connect scan upon nvme discovery controller Events
|
|
||||||
-After=syslog.target
|
|
||||||
+DefaultDependencies=no
|
|
||||||
+After=systemd-udevd.service
|
|
||||||
+Before=local-fs-pre.target
|
|
||||||
PartOf=nvmf-connect.target
|
|
||||||
Requires=nvmf-connect.target
|
|
||||||
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
@ -1,35 +0,0 @@
|
|||||||
From 00909e8bc7e5a1b7a1129f8e18c60eedca0248f7 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
Date: Mon, 17 Apr 2023 15:39:56 +0200
|
|
||||||
Subject: [PATCH] nvme: fix rnlpt to_string() values.
|
|
||||||
|
|
||||||
"Reservation Notification Log Page Type" values do not start
|
|
||||||
from 0x1 but from 0x0.
|
|
||||||
|
|
||||||
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
|
|
||||||
---
|
|
||||||
nvme-print.c | 8 ++++----
|
|
||||||
1 file changed, 4 insertions(+), 4 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/nvme-print.c b/nvme-print.c
|
|
||||||
index 8bd2174f..1ed34572 100644
|
|
||||||
--- a/nvme-print.c
|
|
||||||
+++ b/nvme-print.c
|
|
||||||
@@ -610,10 +610,10 @@ void nvme_show_lba_status_log(void *lba_status, __u32 size,
|
|
||||||
static const char *resv_notif_to_string(__u8 type)
|
|
||||||
{
|
|
||||||
switch (type) {
|
|
||||||
- case 0x1: return "Empty Log Page";
|
|
||||||
- case 0x2: return "Registration Preempted";
|
|
||||||
- case 0x3: return "Reservation Released";
|
|
||||||
- case 0x4: return "Reservation Preempted";
|
|
||||||
+ case 0x0: return "Empty Log Page";
|
|
||||||
+ case 0x1: return "Registration Preempted";
|
|
||||||
+ case 0x2: return "Reservation Released";
|
|
||||||
+ case 0x3: return "Reservation Preempted";
|
|
||||||
default: return "Reserved";
|
|
||||||
}
|
|
||||||
}
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
||||||
5
SOURCES/99-nvme-nbft-connect.sh
Normal file
5
SOURCES/99-nvme-nbft-connect.sh
Normal file
@ -0,0 +1,5 @@
|
|||||||
|
#!/bin/bash
|
||||||
|
|
||||||
|
if [[ "$1" == nbft* ]] && [[ "$2" == "up" ]]; then
|
||||||
|
systemctl start nvmf-connect-nbft.service
|
||||||
|
fi
|
||||||
15
SOURCES/99-nvme-nbft-no-ignore-carrier.conf
Normal file
15
SOURCES/99-nvme-nbft-no-ignore-carrier.conf
Normal file
@ -0,0 +1,15 @@
|
|||||||
|
# Boot from NVMe over TCP (NBFT)
|
||||||
|
#
|
||||||
|
# For NVMe/TCP connections that provide namespaces containing rootfs
|
||||||
|
# it is crucial to react on carrier events and reconnect any missing
|
||||||
|
# NVMe/TCP connections as defined in the ACPI NBFT table. A custom
|
||||||
|
# /usr/lib/NetworkManager/dispatcher.d/99-nvme-nbft-connect.sh hook
|
||||||
|
# will respawn nvmf-connect-nbft.service on such occasion.
|
||||||
|
|
||||||
|
[device-nbft-no-ignore-carrier]
|
||||||
|
|
||||||
|
# only affects nbft0, nbft1, ... interfaces
|
||||||
|
match-device=interface-name:nbft*
|
||||||
|
|
||||||
|
# react on link up/down events
|
||||||
|
ignore-carrier=no
|
||||||
596
SOURCES/nvme-cli-2.10-nbft-discovery.patch
Normal file
596
SOURCES/nvme-cli-2.10-nbft-discovery.patch
Normal file
@ -0,0 +1,596 @@
|
|||||||
|
From ca58ecc64852fc413a04f41d953914720d05459f Mon Sep 17 00:00:00 2001
|
||||||
|
From: Tomas Bzatek <tbzatek@redhat.com>
|
||||||
|
Date: Wed, 24 Apr 2024 17:46:19 +0200
|
||||||
|
Subject: [PATCH 1/5] fabrics: Make some symbols public
|
||||||
|
|
||||||
|
Needed for nbft, useful to have public.
|
||||||
|
|
||||||
|
Signed-off-by: Tomas Bzatek <tbzatek@redhat.com>
|
||||||
|
---
|
||||||
|
fabrics.c | 24 ++++++++++++------------
|
||||||
|
fabrics.h | 6 ++++++
|
||||||
|
2 files changed, 18 insertions(+), 12 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/fabrics.c b/fabrics.c
|
||||||
|
index 0b70d290a9..a82a56c7b0 100644
|
||||||
|
--- a/fabrics.c
|
||||||
|
+++ b/fabrics.c
|
||||||
|
@@ -195,9 +195,9 @@ static nvme_ctrl_t __create_discover_ctrl(nvme_root_t r, nvme_host_t h,
|
||||||
|
return c;
|
||||||
|
}
|
||||||
|
|
||||||
|
-static nvme_ctrl_t create_discover_ctrl(nvme_root_t r, nvme_host_t h,
|
||||||
|
- struct nvme_fabrics_config *cfg,
|
||||||
|
- struct tr_config *trcfg)
|
||||||
|
+nvme_ctrl_t nvmf_create_discover_ctrl(nvme_root_t r, nvme_host_t h,
|
||||||
|
+ struct nvme_fabrics_config *cfg,
|
||||||
|
+ struct tr_config *trcfg)
|
||||||
|
{
|
||||||
|
_cleanup_free_ struct nvme_id_ctrl *id = NULL;
|
||||||
|
nvme_ctrl_t c;
|
||||||
|
@@ -378,8 +378,7 @@ static int __discover(nvme_ctrl_t c, struct nvme_fabrics_config *defcfg,
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
-static char *get_default_trsvcid(const char *transport,
|
||||||
|
- bool discovery_ctrl)
|
||||||
|
+char *nvmf_get_default_trsvcid(const char *transport, bool discovery_ctrl)
|
||||||
|
{
|
||||||
|
if (!transport)
|
||||||
|
return NULL;
|
||||||
|
@@ -465,7 +464,7 @@ static int discover_from_conf_file(nvme_root_t r, nvme_host_t h,
|
||||||
|
goto next;
|
||||||
|
|
||||||
|
if (!trsvcid)
|
||||||
|
- trsvcid = get_default_trsvcid(transport, true);
|
||||||
|
+ trsvcid = nvmf_get_default_trsvcid(transport, true);
|
||||||
|
|
||||||
|
struct tr_config trcfg = {
|
||||||
|
.subsysnqn = subsysnqn,
|
||||||
|
@@ -485,7 +484,7 @@ static int discover_from_conf_file(nvme_root_t r, nvme_host_t h,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
- c = create_discover_ctrl(r, h, &cfg, &trcfg);
|
||||||
|
+ c = nvmf_create_discover_ctrl(r, h, &cfg, &trcfg);
|
||||||
|
if (!c)
|
||||||
|
goto next;
|
||||||
|
|
||||||
|
@@ -549,7 +548,8 @@ static int discover_from_json_config_file(nvme_root_t r, nvme_host_t h,
|
||||||
|
|
||||||
|
trsvcid = nvme_ctrl_get_trsvcid(c);
|
||||||
|
if (!trsvcid || !strcmp(trsvcid, ""))
|
||||||
|
- trsvcid = get_default_trsvcid(transport, true);
|
||||||
|
+ trsvcid = nvmf_get_default_trsvcid(transport,
|
||||||
|
+ true);
|
||||||
|
|
||||||
|
if (force)
|
||||||
|
subsysnqn = nvme_ctrl_get_subsysnqn(c);
|
||||||
|
@@ -579,7 +579,7 @@ static int discover_from_json_config_file(nvme_root_t r, nvme_host_t h,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
- cn = create_discover_ctrl(r, h, &cfg, &trcfg);
|
||||||
|
+ cn = nvmf_create_discover_ctrl(r, h, &cfg, &trcfg);
|
||||||
|
if (!cn)
|
||||||
|
continue;
|
||||||
|
|
||||||
|
@@ -806,7 +806,7 @@ int nvmf_discover(const char *desc, int argc, char **argv, bool connect)
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!trsvcid)
|
||||||
|
- trsvcid = get_default_trsvcid(transport, true);
|
||||||
|
+ trsvcid = nvmf_get_default_trsvcid(transport, true);
|
||||||
|
|
||||||
|
struct tr_config trcfg = {
|
||||||
|
.subsysnqn = subsysnqn,
|
||||||
|
@@ -876,7 +876,7 @@ int nvmf_discover(const char *desc, int argc, char **argv, bool connect)
|
||||||
|
}
|
||||||
|
if (!c) {
|
||||||
|
/* No device or non-matching device, create a new controller */
|
||||||
|
- c = create_discover_ctrl(r, h, &cfg, &trcfg);
|
||||||
|
+ c = nvmf_create_discover_ctrl(r, h, &cfg, &trcfg);
|
||||||
|
if (!c) {
|
||||||
|
if (errno != ENVME_CONNECT_IGNORED)
|
||||||
|
fprintf(stderr,
|
||||||
|
@@ -1005,7 +1005,7 @@ int nvmf_connect(const char *desc, int argc, char **argv)
|
||||||
|
if (hostkey)
|
||||||
|
nvme_host_set_dhchap_key(h, hostkey);
|
||||||
|
if (!trsvcid)
|
||||||
|
- trsvcid = get_default_trsvcid(transport, false);
|
||||||
|
+ trsvcid = nvmf_get_default_trsvcid(transport, false);
|
||||||
|
|
||||||
|
struct tr_config trcfg = {
|
||||||
|
.subsysnqn = subsysnqn,
|
||||||
|
diff --git a/fabrics.h b/fabrics.h
|
||||||
|
index c16df60472..aec305dc31 100644
|
||||||
|
--- a/fabrics.h
|
||||||
|
+++ b/fabrics.h
|
||||||
|
@@ -18,5 +18,11 @@ extern int nvmf_disconnect(const char *desc, int argc, char **argv);
|
||||||
|
extern int nvmf_disconnect_all(const char *desc, int argc, char **argv);
|
||||||
|
extern int nvmf_config(const char *desc, int argc, char **argv);
|
||||||
|
extern int nvmf_dim(const char *desc, int argc, char **argv);
|
||||||
|
+extern nvme_ctrl_t nvmf_create_discover_ctrl(nvme_root_t r, nvme_host_t h,
|
||||||
|
+ struct nvme_fabrics_config *cfg,
|
||||||
|
+ struct tr_config *trcfg);
|
||||||
|
+extern char *nvmf_get_default_trsvcid(const char *transport,
|
||||||
|
+ bool discovery_ctrl);
|
||||||
|
+
|
||||||
|
|
||||||
|
#endif
|
||||||
|
|
||||||
|
From 996fd5554157949ed77154eecd0f8236590cc716 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Tomas Bzatek <tbzatek@redhat.com>
|
||||||
|
Date: Wed, 24 Apr 2024 17:47:02 +0200
|
||||||
|
Subject: [PATCH 2/5] util/cleanup: Add cleanup for struct nvme_fabrics_uri
|
||||||
|
|
||||||
|
libnvme cleanup definitions are not part of public API.
|
||||||
|
|
||||||
|
Signed-off-by: Tomas Bzatek <tbzatek@redhat.com>
|
||||||
|
---
|
||||||
|
util/cleanup.h | 8 ++++++++
|
||||||
|
1 file changed, 8 insertions(+)
|
||||||
|
|
||||||
|
diff -up nvme-cli-2.9.1/util/cleanup.h.bak nvme-cli-2.9.1/util/cleanup.h
|
||||||
|
--- nvme-cli-2.9.1/util/cleanup.h.bak 2024-06-21 15:53:35.839852234 +0200
|
||||||
|
+++ nvme-cli-2.9.1/util/cleanup.h 2024-05-03 16:03:42.000000000 +0200
|
||||||
|
@@ -34,4 +34,12 @@ static inline void close_file(int *f)
|
||||||
|
}
|
||||||
|
#define _cleanup_file_ __cleanup__(close_file)
|
||||||
|
|
||||||
|
+static inline void free_uri(struct nvme_fabrics_uri **uri)
|
||||||
|
+{
|
||||||
|
+ if (*uri)
|
||||||
|
+ nvme_free_uri(*uri);
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+#define _cleanup_uri_ __cleanup__(free_uri)
|
||||||
|
+
|
||||||
|
#endif
|
||||||
|
|
||||||
|
From 2f79015b83faff4947d29fb172d6f92d81f078e2 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Tomas Bzatek <tbzatek@redhat.com>
|
||||||
|
Date: Wed, 24 Apr 2024 17:52:46 +0200
|
||||||
|
Subject: [PATCH 3/5] nbft: Perform actual discovery
|
||||||
|
|
||||||
|
This adds actual discovery support for Discovery Descriptor records.
|
||||||
|
|
||||||
|
SSNS records are connected first. Discovery Descriptor records
|
||||||
|
are checked for any existing (back-)reference from SSNS records
|
||||||
|
and are skipped if so. It is assumed in such case that the pre-OS
|
||||||
|
driver has succeeded in discovery and filled SSNS records
|
||||||
|
accordingly.
|
||||||
|
|
||||||
|
In case no SSNS record references the particular Discovery
|
||||||
|
record, an actual discovery is performed.
|
||||||
|
|
||||||
|
Signed-off-by: Tomas Bzatek <tbzatek@redhat.com>
|
||||||
|
---
|
||||||
|
nbft.c | 232 ++++++++++++++++++++++++++++++++++++++++++++++++++++++---
|
||||||
|
nbft.h | 2 +-
|
||||||
|
2 files changed, 223 insertions(+), 11 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/nbft.c b/nbft.c
|
||||||
|
index 7ff8765a80..995b31ca54 100644
|
||||||
|
--- a/nbft.c
|
||||||
|
+++ b/nbft.c
|
||||||
|
@@ -7,6 +7,7 @@
|
||||||
|
|
||||||
|
#include <libnvme.h>
|
||||||
|
|
||||||
|
+#include "common.h"
|
||||||
|
#include "nvme.h"
|
||||||
|
#include "nbft.h"
|
||||||
|
#include "fabrics.h"
|
||||||
|
@@ -77,12 +78,38 @@ void free_nbfts(struct list_head *nbft_list)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
+static bool validate_uri(struct nbft_info_discovery *dd,
|
||||||
|
+ struct nvme_fabrics_uri *uri)
|
||||||
|
+{
|
||||||
|
+ if (!uri) {
|
||||||
|
+ fprintf(stderr,
|
||||||
|
+ "Discovery Descriptor %d: failed to parse URI %s\n",
|
||||||
|
+ dd->index, dd->uri);
|
||||||
|
+ return false;
|
||||||
|
+ }
|
||||||
|
+ if (strcmp(uri->scheme, "nvme") != 0) {
|
||||||
|
+ fprintf(stderr,
|
||||||
|
+ "Discovery Descriptor %d: unsupported scheme '%s'\n",
|
||||||
|
+ dd->index, uri->scheme);
|
||||||
|
+ return false;
|
||||||
|
+ }
|
||||||
|
+ if (!uri->protocol || strcmp(uri->protocol, "tcp") != 0) {
|
||||||
|
+ fprintf(stderr,
|
||||||
|
+ "Discovery Descriptor %d: unsupported transport '%s'\n",
|
||||||
|
+ dd->index, uri->protocol);
|
||||||
|
+ return false;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ return true;
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
/* returns 0 for success or negative errno otherwise */
|
||||||
|
static int do_connect(nvme_root_t r,
|
||||||
|
nvme_host_t h,
|
||||||
|
+ struct nvmf_disc_log_entry *e,
|
||||||
|
struct nbft_info_subsystem_ns *ss,
|
||||||
|
struct tr_config *trcfg,
|
||||||
|
- const struct nvme_fabrics_config *cfg,
|
||||||
|
+ struct nvme_fabrics_config *cfg,
|
||||||
|
enum nvme_print_flags flags,
|
||||||
|
unsigned int verbose)
|
||||||
|
{
|
||||||
|
@@ -111,6 +138,12 @@ static int do_connect(nvme_root_t r,
|
||||||
|
nvme_init_logging(r, -1, false, false);
|
||||||
|
}
|
||||||
|
|
||||||
|
+ if (e) {
|
||||||
|
+ if (e->trtype == NVMF_TRTYPE_TCP &&
|
||||||
|
+ e->tsas.tcp.sectype != NVMF_TCP_SECTYPE_NONE)
|
||||||
|
+ cfg->tls = true;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
errno = 0;
|
||||||
|
ret = nvmf_add_ctrl(h, c, cfg);
|
||||||
|
|
||||||
|
@@ -145,10 +178,114 @@ static int do_connect(nvme_root_t r,
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
+static int do_discover(struct nbft_info_discovery *dd,
|
||||||
|
+ nvme_root_t r,
|
||||||
|
+ nvme_host_t h,
|
||||||
|
+ nvme_ctrl_t c,
|
||||||
|
+ struct nvme_fabrics_config *defcfg,
|
||||||
|
+ struct tr_config *deftrcfg,
|
||||||
|
+ enum nvme_print_flags flags,
|
||||||
|
+ unsigned int verbose)
|
||||||
|
+{
|
||||||
|
+ struct nvmf_discovery_log *log = NULL;
|
||||||
|
+ int i;
|
||||||
|
+ int ret;
|
||||||
|
+
|
||||||
|
+ struct nvme_get_discovery_args args = {
|
||||||
|
+ .c = c,
|
||||||
|
+ .args_size = sizeof(args),
|
||||||
|
+ .max_retries = 10 /* MAX_DISC_RETRIES */,
|
||||||
|
+ .result = 0,
|
||||||
|
+ .timeout = NVME_DEFAULT_IOCTL_TIMEOUT,
|
||||||
|
+ .lsp = 0,
|
||||||
|
+ };
|
||||||
|
+
|
||||||
|
+ log = nvmf_get_discovery_wargs(&args);
|
||||||
|
+ if (!log) {
|
||||||
|
+ fprintf(stderr,
|
||||||
|
+ "Discovery Descriptor %d: failed to get discovery log: %s\n",
|
||||||
|
+ dd->index, nvme_strerror(errno));
|
||||||
|
+ return -errno;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ for (i = 0; i < le64_to_cpu(log->numrec); i++) {
|
||||||
|
+ struct nvmf_disc_log_entry *e = &log->entries[i];
|
||||||
|
+ nvme_ctrl_t cl;
|
||||||
|
+ int tmo = defcfg->keep_alive_tmo;
|
||||||
|
+
|
||||||
|
+ struct tr_config trcfg = {
|
||||||
|
+ .subsysnqn = e->subnqn,
|
||||||
|
+ .transport = nvmf_trtype_str(e->trtype),
|
||||||
|
+ .traddr = e->traddr,
|
||||||
|
+ .host_traddr = deftrcfg->host_traddr,
|
||||||
|
+ .host_iface = deftrcfg->host_iface,
|
||||||
|
+ .trsvcid = e->trsvcid,
|
||||||
|
+ };
|
||||||
|
+
|
||||||
|
+ if (e->subtype == NVME_NQN_CURR)
|
||||||
|
+ continue;
|
||||||
|
+
|
||||||
|
+ /* Already connected ? */
|
||||||
|
+ cl = lookup_ctrl(h, &trcfg);
|
||||||
|
+ if (cl && nvme_ctrl_get_name(cl))
|
||||||
|
+ continue;
|
||||||
|
+
|
||||||
|
+ /* Skip connect if the transport types don't match */
|
||||||
|
+ if (strcmp(nvme_ctrl_get_transport(c),
|
||||||
|
+ nvmf_trtype_str(e->trtype)))
|
||||||
|
+ continue;
|
||||||
|
+
|
||||||
|
+ if (e->subtype == NVME_NQN_DISC) {
|
||||||
|
+ nvme_ctrl_t child;
|
||||||
|
+
|
||||||
|
+ child = nvmf_connect_disc_entry(h, e, defcfg, NULL);
|
||||||
|
+ do_discover(dd, r, h, child, defcfg, &trcfg,
|
||||||
|
+ flags, verbose);
|
||||||
|
+ nvme_disconnect_ctrl(child);
|
||||||
|
+ nvme_free_ctrl(child);
|
||||||
|
+ } else {
|
||||||
|
+ ret = do_connect(r, h, e, NULL, &trcfg,
|
||||||
|
+ defcfg, flags, verbose);
|
||||||
|
+
|
||||||
|
+ /*
|
||||||
|
+ * With TCP/DHCP, it can happen that the OS
|
||||||
|
+ * obtains a different local IP address than the
|
||||||
|
+ * firmware had. Retry without host_traddr.
|
||||||
|
+ */
|
||||||
|
+ if (ret == -ENVME_CONNECT_ADDRNOTAVAIL &&
|
||||||
|
+ !strcmp(trcfg.transport, "tcp") &&
|
||||||
|
+ strlen(dd->hfi->tcp_info.dhcp_server_ipaddr) > 0) {
|
||||||
|
+ const char *htradr = trcfg.host_traddr;
|
||||||
|
+
|
||||||
|
+ trcfg.host_traddr = NULL;
|
||||||
|
+ ret = do_connect(r, h, e, NULL, &trcfg,
|
||||||
|
+ defcfg, flags, verbose);
|
||||||
|
+
|
||||||
|
+ if (ret == 0 && verbose >= 1)
|
||||||
|
+ fprintf(stderr,
|
||||||
|
+ "Discovery Descriptor %d: connect with host_traddr=\"%s\" failed, success after omitting host_traddr\n",
|
||||||
|
+ dd->index,
|
||||||
|
+ htradr);
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ if (ret)
|
||||||
|
+ fprintf(stderr, "Discovery Descriptor %d: no controller found\n",
|
||||||
|
+ dd->index);
|
||||||
|
+ if (ret == -ENOMEM)
|
||||||
|
+ break;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ defcfg->keep_alive_tmo = tmo;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ free(log);
|
||||||
|
+ return 0;
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
char *hostnqn_sys, char *hostid_sys,
|
||||||
|
const char *desc, bool connect,
|
||||||
|
- const struct nvme_fabrics_config *cfg, char *nbft_path,
|
||||||
|
+ struct nvme_fabrics_config *cfg, char *nbft_path,
|
||||||
|
enum nvme_print_flags flags, unsigned int verbose)
|
||||||
|
{
|
||||||
|
char *hostnqn = NULL, *hostid = NULL, *host_traddr = NULL;
|
||||||
|
@@ -158,6 +295,7 @@ int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
struct nbft_file_entry *entry = NULL;
|
||||||
|
struct nbft_info_subsystem_ns **ss;
|
||||||
|
struct nbft_info_hfi *hfi;
|
||||||
|
+ struct nbft_info_discovery **dd;
|
||||||
|
|
||||||
|
if (!connect)
|
||||||
|
/* to do: print discovery-type info from NBFT tables */
|
||||||
|
@@ -192,15 +330,15 @@ int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
if (!h)
|
||||||
|
goto out_free;
|
||||||
|
|
||||||
|
+ /* Subsystem Namespace Descriptor List */
|
||||||
|
for (ss = entry->nbft->subsystem_ns_list; ss && *ss; ss++)
|
||||||
|
for (i = 0; i < (*ss)->num_hfis; i++) {
|
||||||
|
hfi = (*ss)->hfis[i];
|
||||||
|
|
||||||
|
- if (!cfg->host_traddr) {
|
||||||
|
- host_traddr = NULL;
|
||||||
|
- if (!strncmp((*ss)->transport, "tcp", 3))
|
||||||
|
- host_traddr = hfi->tcp_info.ipaddr;
|
||||||
|
- }
|
||||||
|
+ host_traddr = NULL;
|
||||||
|
+ if (!cfg->host_traddr &&
|
||||||
|
+ !strncmp((*ss)->transport, "tcp", 3))
|
||||||
|
+ host_traddr = hfi->tcp_info.ipaddr;
|
||||||
|
|
||||||
|
struct tr_config trcfg = {
|
||||||
|
.subsysnqn = (*ss)->subsys_nqn,
|
||||||
|
@@ -211,7 +349,7 @@ int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
.trsvcid = (*ss)->trsvcid,
|
||||||
|
};
|
||||||
|
|
||||||
|
- ret = do_connect(r, h, *ss, &trcfg,
|
||||||
|
+ ret = do_connect(r, h, NULL, *ss, &trcfg,
|
||||||
|
cfg, flags, verbose);
|
||||||
|
|
||||||
|
/*
|
||||||
|
@@ -220,11 +358,11 @@ int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
* firmware had. Retry without host_traddr.
|
||||||
|
*/
|
||||||
|
if (ret == -ENVME_CONNECT_ADDRNOTAVAIL &&
|
||||||
|
- !strcmp((*ss)->transport, "tcp") &&
|
||||||
|
+ !strcmp(trcfg.transport, "tcp") &&
|
||||||
|
strlen(hfi->tcp_info.dhcp_server_ipaddr) > 0) {
|
||||||
|
trcfg.host_traddr = NULL;
|
||||||
|
|
||||||
|
- ret = do_connect(r, h, *ss, &trcfg,
|
||||||
|
+ ret = do_connect(r, h, NULL, *ss, &trcfg,
|
||||||
|
cfg, flags, verbose);
|
||||||
|
|
||||||
|
if (ret == 0 && verbose >= 1)
|
||||||
|
@@ -241,6 +379,80 @@ int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
if (ret == -ENOMEM)
|
||||||
|
goto out_free;
|
||||||
|
}
|
||||||
|
+
|
||||||
|
+ /* Discovery Descriptor List */
|
||||||
|
+ for (dd = entry->nbft->discovery_list; dd && *dd; dd++) {
|
||||||
|
+ nvme_ctrl_t c;
|
||||||
|
+ bool linked = false;
|
||||||
|
+ _cleanup_uri_ struct nvme_fabrics_uri *uri = NULL;
|
||||||
|
+ _cleanup_free_ char *trsvcid = NULL;
|
||||||
|
+
|
||||||
|
+ /* only perform discovery when no SSNS record references it */
|
||||||
|
+ for (ss = entry->nbft->subsystem_ns_list; ss && *ss; ss++)
|
||||||
|
+ if ((*ss)->discovery &&
|
||||||
|
+ (*ss)->discovery->index == (*dd)->index &&
|
||||||
|
+ /* unavailable boot attempts are not discovered
|
||||||
|
+ * and may get transferred along with a well-known
|
||||||
|
+ * discovery NQN into an SSNS record.
|
||||||
|
+ */
|
||||||
|
+ strcmp((*ss)->subsys_nqn, NVME_DISC_SUBSYS_NAME) != 0) {
|
||||||
|
+ linked = true;
|
||||||
|
+ break;
|
||||||
|
+ }
|
||||||
|
+ if (linked)
|
||||||
|
+ continue;
|
||||||
|
+
|
||||||
|
+ hfi = (*dd)->hfi;
|
||||||
|
+ uri = nvme_parse_uri((*dd)->uri);
|
||||||
|
+ if (!validate_uri(*dd, uri))
|
||||||
|
+ continue;
|
||||||
|
+
|
||||||
|
+ host_traddr = NULL;
|
||||||
|
+ if (!cfg->host_traddr &&
|
||||||
|
+ !strncmp(uri->protocol, "tcp", 3))
|
||||||
|
+ host_traddr = hfi->tcp_info.ipaddr;
|
||||||
|
+ if (uri->port > 0) {
|
||||||
|
+ if (asprintf(&trsvcid, "%d", uri->port) < 0) {
|
||||||
|
+ errno = ENOMEM;
|
||||||
|
+ goto out_free;
|
||||||
|
+ }
|
||||||
|
+ } else
|
||||||
|
+ trsvcid = strdup(nvmf_get_default_trsvcid(uri->protocol, true));
|
||||||
|
+
|
||||||
|
+ struct tr_config trcfg = {
|
||||||
|
+ .subsysnqn = NVME_DISC_SUBSYS_NAME,
|
||||||
|
+ .transport = uri->protocol,
|
||||||
|
+ .traddr = uri->host,
|
||||||
|
+ .host_traddr = host_traddr,
|
||||||
|
+ .host_iface = NULL,
|
||||||
|
+ .trsvcid = trsvcid,
|
||||||
|
+ };
|
||||||
|
+
|
||||||
|
+ c = nvmf_create_discover_ctrl(r, h, cfg, &trcfg);
|
||||||
|
+ if (!c && errno == ENVME_CONNECT_ADDRNOTAVAIL &&
|
||||||
|
+ !strcmp(trcfg.transport, "tcp") &&
|
||||||
|
+ strlen(hfi->tcp_info.dhcp_server_ipaddr) > 0) {
|
||||||
|
+ trcfg.host_traddr = NULL;
|
||||||
|
+ c = nvmf_create_discover_ctrl(r, h, cfg, &trcfg);
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ if (!c) {
|
||||||
|
+ fprintf(stderr,
|
||||||
|
+ "Discovery Descriptor %d: failed to add discovery controller: %s\n",
|
||||||
|
+ (*dd)->index,
|
||||||
|
+ nvme_strerror(errno));
|
||||||
|
+ if (errno == ENOMEM)
|
||||||
|
+ goto out_free;
|
||||||
|
+ continue;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ ret = do_discover(*dd, r, h, c, cfg, &trcfg,
|
||||||
|
+ flags, verbose);
|
||||||
|
+ nvme_disconnect_ctrl(c);
|
||||||
|
+ nvme_free_ctrl(c);
|
||||||
|
+ if (ret == -ENOMEM)
|
||||||
|
+ goto out_free;
|
||||||
|
+ }
|
||||||
|
}
|
||||||
|
out_free:
|
||||||
|
free_nbfts(&nbft_list);
|
||||||
|
diff --git a/nbft.h b/nbft.h
|
||||||
|
index 0f7e33cee5..5dfb8704fd 100644
|
||||||
|
--- a/nbft.h
|
||||||
|
+++ b/nbft.h
|
||||||
|
@@ -15,5 +15,5 @@ void free_nbfts(struct list_head *nbft_list);
|
||||||
|
extern int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
char *hostnqn_sys, char *hostid_sys,
|
||||||
|
const char *desc, bool connect,
|
||||||
|
- const struct nvme_fabrics_config *cfg, char *nbft_path,
|
||||||
|
+ struct nvme_fabrics_config *cfg, char *nbft_path,
|
||||||
|
enum nvme_print_flags flags, unsigned int verbose);
|
||||||
|
|
||||||
|
From 1074da1dbbbef229bf7e42a3cb644ab5b9745267 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Tomas Bzatek <tbzatek@redhat.com>
|
||||||
|
Date: Thu, 25 Apr 2024 15:54:53 +0200
|
||||||
|
Subject: [PATCH 4/5] nbft: Skip SSNS records pointing to well-known discovery
|
||||||
|
NQN
|
||||||
|
|
||||||
|
Depending on a pre-OS implementation, boot attempts pointing
|
||||||
|
to the well-known discovery NQN may get transformed in
|
||||||
|
an SSNS record (and marked as 'unavailable') in case
|
||||||
|
the discovery cannot be performed. Otherwise the NBFT
|
||||||
|
table should be populated by discovered records instead.
|
||||||
|
|
||||||
|
Signed-off-by: Tomas Bzatek <tbzatek@redhat.com>
|
||||||
|
---
|
||||||
|
nbft.c | 9 +++++++++
|
||||||
|
1 file changed, 9 insertions(+)
|
||||||
|
|
||||||
|
diff --git a/nbft.c b/nbft.c
|
||||||
|
index 995b31ca54..9d1834d0db 100644
|
||||||
|
--- a/nbft.c
|
||||||
|
+++ b/nbft.c
|
||||||
|
@@ -335,6 +335,15 @@ int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
for (i = 0; i < (*ss)->num_hfis; i++) {
|
||||||
|
hfi = (*ss)->hfis[i];
|
||||||
|
|
||||||
|
+ /* Skip discovery NQN records */
|
||||||
|
+ if (strcmp((*ss)->subsys_nqn, NVME_DISC_SUBSYS_NAME) == 0) {
|
||||||
|
+ if (verbose >= 1)
|
||||||
|
+ fprintf(stderr,
|
||||||
|
+ "SSNS %d points to well-known discovery NQN, skipping\n",
|
||||||
|
+ (*ss)->index);
|
||||||
|
+ continue;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
host_traddr = NULL;
|
||||||
|
if (!cfg->host_traddr &&
|
||||||
|
!strncmp((*ss)->transport, "tcp", 3))
|
||||||
|
|
||||||
|
From 8b4c30014ed17fa226a34a3dd92167f122ccec42 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Tomas Bzatek <tbzatek@redhat.com>
|
||||||
|
Date: Mon, 20 May 2024 17:43:01 +0200
|
||||||
|
Subject: [PATCH 5/5] nbft: Reuse existing discovery controller
|
||||||
|
|
||||||
|
Attempt to look up and use existing (persistent) discovery
|
||||||
|
controller.
|
||||||
|
|
||||||
|
Signed-off-by: Tomas Bzatek <tbzatek@redhat.com>
|
||||||
|
---
|
||||||
|
nbft.c | 21 +++++++++++++++------
|
||||||
|
1 file changed, 15 insertions(+), 6 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/nbft.c b/nbft.c
|
||||||
|
index 9d1834d0db..8c03a1f53b 100644
|
||||||
|
--- a/nbft.c
|
||||||
|
+++ b/nbft.c
|
||||||
|
@@ -393,6 +393,7 @@ int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
for (dd = entry->nbft->discovery_list; dd && *dd; dd++) {
|
||||||
|
nvme_ctrl_t c;
|
||||||
|
bool linked = false;
|
||||||
|
+ bool persistent = false;
|
||||||
|
_cleanup_uri_ struct nvme_fabrics_uri *uri = NULL;
|
||||||
|
_cleanup_free_ char *trsvcid = NULL;
|
||||||
|
|
||||||
|
@@ -437,12 +438,19 @@ int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
.trsvcid = trsvcid,
|
||||||
|
};
|
||||||
|
|
||||||
|
- c = nvmf_create_discover_ctrl(r, h, cfg, &trcfg);
|
||||||
|
- if (!c && errno == ENVME_CONNECT_ADDRNOTAVAIL &&
|
||||||
|
- !strcmp(trcfg.transport, "tcp") &&
|
||||||
|
- strlen(hfi->tcp_info.dhcp_server_ipaddr) > 0) {
|
||||||
|
- trcfg.host_traddr = NULL;
|
||||||
|
+ /* Lookup existing discovery controller */
|
||||||
|
+ c = lookup_ctrl(h, &trcfg);
|
||||||
|
+ if (c && nvme_ctrl_get_name(c))
|
||||||
|
+ persistent = true;
|
||||||
|
+
|
||||||
|
+ if (!c) {
|
||||||
|
c = nvmf_create_discover_ctrl(r, h, cfg, &trcfg);
|
||||||
|
+ if (!c && errno == ENVME_CONNECT_ADDRNOTAVAIL &&
|
||||||
|
+ !strcmp(trcfg.transport, "tcp") &&
|
||||||
|
+ strlen(hfi->tcp_info.dhcp_server_ipaddr) > 0) {
|
||||||
|
+ trcfg.host_traddr = NULL;
|
||||||
|
+ c = nvmf_create_discover_ctrl(r, h, cfg, &trcfg);
|
||||||
|
+ }
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!c) {
|
||||||
|
@@ -457,7 +465,8 @@ int discover_from_nbft(nvme_root_t r, char *hostnqn_arg, char *hostid_arg,
|
||||||
|
|
||||||
|
ret = do_discover(*dd, r, h, c, cfg, &trcfg,
|
||||||
|
flags, verbose);
|
||||||
|
- nvme_disconnect_ctrl(c);
|
||||||
|
+ if (!persistent)
|
||||||
|
+ nvme_disconnect_ctrl(c);
|
||||||
|
nvme_free_ctrl(c);
|
||||||
|
if (ret == -ENOMEM)
|
||||||
|
goto out_free;
|
||||||
@ -1,13 +0,0 @@
|
|||||||
--- nvme-cli-1.9/Makefile.orig 2020-05-27 14:04:56.259961135 -0400
|
|
||||||
+++ nvme-cli-1.9/Makefile 2020-05-27 14:05:22.796185371 -0400
|
|
||||||
@@ -125,8 +125,8 @@
|
|
||||||
$(INSTALL) -m 644 -T ./etc/discovery.conf.in $(DESTDIR)$(SYSCONFDIR)/nvme/discovery.conf; \
|
|
||||||
fi
|
|
||||||
|
|
||||||
-install-spec: install-bin install-man install-bash-completion install-zsh-completion install-etc install-systemd install-udev install-dracut
|
|
||||||
-install: install-spec install-hostparams
|
|
||||||
+install-spec: install-bin install-man install-bash-completion install-zsh-completion install-systemd install-udev install-dracut
|
|
||||||
+install: install-spec
|
|
||||||
|
|
||||||
nvme.spec: nvme.spec.in NVME-VERSION-FILE
|
|
||||||
sed -e 's/@@VERSION@@/$(NVME_VERSION)/g' < $< > $@+
|
|
||||||
@ -1,103 +1,86 @@
|
|||||||
#%%global commit0 bdbb4da0979fbdc079cf98410cdb31cf799e83b3
|
#%%global commit0 bdbb4da0979fbdc079cf98410cdb31cf799e83b3
|
||||||
#%%global shortcommit0 %%(c=%%{commit0}; echo ${c:0:7})
|
#%%global shortcommit0 %%(c=%%{commit0}; echo ${c:0:7})
|
||||||
|
|
||||||
|
%global nmlibdir %{_prefix}/lib/NetworkManager
|
||||||
|
|
||||||
Name: nvme-cli
|
Name: nvme-cli
|
||||||
Version: 1.16
|
Version: 2.9.1
|
||||||
Release: 9%{?dist}
|
Release: 6%{?dist}
|
||||||
Summary: NVMe management command line interface
|
Summary: NVMe management command line interface
|
||||||
|
|
||||||
License: GPLv2+
|
License: GPL-2.0-only
|
||||||
URL: https://github.com/linux-nvme/nvme-cli
|
URL: https://github.com/linux-nvme/nvme-cli
|
||||||
Source0: https://github.com/linux-nvme/%{name}/archive/v%{version}.tar.gz
|
Source0: %{url}/archive/v%{version}/%{name}-%{version}.tar.gz
|
||||||
|
Source1: 99-nvme-nbft-connect.sh
|
||||||
|
Source2: 99-nvme-nbft-no-ignore-carrier.conf
|
||||||
|
|
||||||
Patch0: nvme-cli-makefile-dont-install-host-params-patch
|
Patch0: 0001-Revert-fabrics-Use-corresponding-hostid-when-hostnqn.patch
|
||||||
Patch1: 0001-fabrics-fix-nvme-connect-segfault-if-transport-type-.patch
|
Patch1: 0002-nvme-telemetry-report-the-correct-error-if-the-ioctl.patch
|
||||||
Patch2: 0002-fabrics-fix-a-buffer-overrun.patch
|
# https://issues.redhat.com/browse/RHEL-37601
|
||||||
Patch3: 0003-bash-Fix-nvme-completion.patch
|
Patch2: nvme-cli-2.10-nbft-discovery.patch
|
||||||
Patch4: 0004-nvme-do-not-leak-an-open-file-handle.patch
|
Patch3: 0003-sed-only-re-read-partition-table-after-unlock.patch
|
||||||
Patch5: 0005-nvme-topology-fix-memory-leaks-in-nvme_logical_block.patch
|
|
||||||
Patch6: 0006-nvme-cli-nvmf-connect-.service-Remove-matching-from-.patch
|
|
||||||
Patch7: 0007-nvme-cli-Make-connect-all-matching-be-case-insensiti.patch
|
|
||||||
Patch8: 0008-nvme-cli-nvme-gen-hostnqn-use-partition-UUID-on-IBM-.patch
|
|
||||||
Patch9: 0009-Add-new-events-support-in-PEL.patch
|
|
||||||
Patch10: 0010-nvme-cli-Decode-Supported-Events-Bitmap-in-PEL-heade.patch
|
|
||||||
Patch11: 0011-nvme-cli-Adds-event-number-in-persistent-event-entri.patch
|
|
||||||
Patch12: 0012-nvme-cli-Adds-readable-firmware-level-in-persistent.patch
|
|
||||||
Patch13: 0013-nvme-cli-Add-support-set-feature-event-in-PEL.patch
|
|
||||||
Patch14: 0014-nvme-cli-Add-support-Telemetry-CRT-in-PEL.patch
|
|
||||||
Patch15: 0015-fix-firmware-log-page-frs-variable-sign.patch
|
|
||||||
Patch16: 0016-fix-file-permissions-nvme-print.c.patch
|
|
||||||
Patch17: 0017-fabrics-Fix-ordering-for-auto-connect-services.patch
|
|
||||||
Patch18: 0018-nvme-fix-rnlpt-to_string-values.patch
|
|
||||||
|
|
||||||
|
BuildRequires: meson >= 0.50.0
|
||||||
|
BuildRequires: gcc gcc-c++
|
||||||
BuildRequires: libuuid-devel
|
BuildRequires: libuuid-devel
|
||||||
BuildRequires: gcc
|
|
||||||
BuildRequires: systemd-devel
|
BuildRequires: systemd-devel
|
||||||
|
BuildRequires: systemd-rpm-macros
|
||||||
|
BuildRequires: zlib-devel
|
||||||
|
BuildRequires: libnvme-devel >= 1.9-2
|
||||||
|
BuildRequires: json-c-devel >= 0.14
|
||||||
|
BuildRequires: asciidoc
|
||||||
|
BuildRequires: xmlto
|
||||||
|
|
||||||
Requires: util-linux
|
Requires: util-linux
|
||||||
|
|
||||||
%description
|
%description
|
||||||
nvme-cli provides NVM-Express user space tooling for Linux.
|
nvme-cli provides NVM-Express user space tooling for Linux.
|
||||||
|
|
||||||
%prep
|
%prep
|
||||||
#%%setup -qn %%{name}-%%{commit0}
|
%autosetup -p1 -n %{name}-%{version}
|
||||||
%setup -q
|
|
||||||
%patch0 -p1
|
|
||||||
%patch1 -p1
|
|
||||||
%patch2 -p1
|
|
||||||
%patch3 -p1
|
|
||||||
%patch4 -p1
|
|
||||||
%patch5 -p1
|
|
||||||
%patch6 -p1
|
|
||||||
%patch7 -p1
|
|
||||||
%patch8 -p1
|
|
||||||
%patch9 -p1
|
|
||||||
%patch10 -p1
|
|
||||||
%patch11 -p1
|
|
||||||
%patch12 -p1
|
|
||||||
%patch13 -p1
|
|
||||||
%patch14 -p1
|
|
||||||
%patch15 -p1
|
|
||||||
%patch16 -p1
|
|
||||||
%patch17 -p1
|
|
||||||
%patch18 -p1
|
|
||||||
|
|
||||||
%build
|
%build
|
||||||
|
%meson -Dudevrulesdir=%{_udevrulesdir} -Dsystemddir=%{_unitdir} -Ddocs=all -Ddocs-build=true -Dhtmldir=%{_pkgdocdir}
|
||||||
# CFLAGS on the command line breaks the build. It works okay as an
|
%meson_build
|
||||||
# environment variable, though. See:
|
|
||||||
# https://github.com/linux-nvme/nvme-cli/pull/480
|
|
||||||
CFLAGS="%{optflags}" make PREFIX=/usr LDFLAGS="%{__global_ldflags}" %{?_smp_mflags}
|
|
||||||
|
|
||||||
|
|
||||||
%install
|
%install
|
||||||
%make_install PREFIX=/usr UDEVDIR="%{_udevrulesdir}/.." SYSTEMDDIR="%{_unitdir}/.."
|
%meson_install
|
||||||
mkdir -p %{buildroot}%{_sysconfdir}/nvme
|
mkdir -p $RPM_BUILD_ROOT%{nmlibdir}/dispatcher.d
|
||||||
|
mkdir -p $RPM_BUILD_ROOT%{nmlibdir}/conf.d
|
||||||
# hostid and hostnqn are supposed to be unique per machine. We obviously
|
%{__install} -pm 755 %{SOURCE1} $RPM_BUILD_ROOT%{nmlibdir}/dispatcher.d/
|
||||||
# can't package them.
|
%{__install} -pm 644 %{SOURCE2} $RPM_BUILD_ROOT%{nmlibdir}/conf.d/
|
||||||
#rm -f %{buildroot}%{_sysconfdir}/nvme/hostid
|
|
||||||
#rm -f %{buildroot}%{_sysconfdir}/nvme/hostnqn
|
|
||||||
|
|
||||||
# Do not install the dracut rule yet. See rhbz 1742764
|
# Do not install the dracut rule yet. See rhbz 1742764
|
||||||
|
# Do we want to keep this here? Now that we have boot support for nvme/fc + tcp?
|
||||||
rm -f %{buildroot}/usr/lib/dracut/dracut.conf.d/70-nvmf-autoconnect.conf
|
rm -f %{buildroot}/usr/lib/dracut/dracut.conf.d/70-nvmf-autoconnect.conf
|
||||||
|
|
||||||
|
# Move html docs into the right place
|
||||||
|
mv %{buildroot}%{_pkgdocdir}/nvme %{buildroot}%{_pkgdocdir}/html
|
||||||
|
rm -rf %{buildroot}%{_pkgdocdir}/nvme
|
||||||
|
|
||||||
%files
|
%files
|
||||||
%license LICENSE
|
%license LICENSE
|
||||||
%doc README.md
|
%doc README.md
|
||||||
|
%doc %{_pkgdocdir}/*
|
||||||
%{_sbindir}/nvme
|
%{_sbindir}/nvme
|
||||||
%{_mandir}/man1/nvme*.gz
|
%{_mandir}/man1/nvme*.gz
|
||||||
%{_datadir}/bash-completion/completions/nvme
|
%{_datadir}/bash-completion/completions/nvme
|
||||||
%{_datadir}/zsh/site-functions/_nvme
|
%{_datadir}/zsh/site-functions/_nvme
|
||||||
%dir %{_sysconfdir}/nvme
|
%dir %{_sysconfdir}/nvme
|
||||||
|
%config(noreplace) %{_sysconfdir}/nvme/discovery.conf
|
||||||
%{_unitdir}/nvmefc-boot-connections.service
|
%{_unitdir}/nvmefc-boot-connections.service
|
||||||
%{_unitdir}/nvmf-autoconnect.service
|
%{_unitdir}/nvmf-autoconnect.service
|
||||||
|
%{_unitdir}/nvmf-connect-nbft.service
|
||||||
%{_unitdir}/nvmf-connect.target
|
%{_unitdir}/nvmf-connect.target
|
||||||
%{_unitdir}/nvmf-connect@.service
|
%{_unitdir}/nvmf-connect@.service
|
||||||
|
%{_udevrulesdir}/65-persistent-net-nbft.rules
|
||||||
%{_udevrulesdir}/70-nvmf-autoconnect.rules
|
%{_udevrulesdir}/70-nvmf-autoconnect.rules
|
||||||
%{_udevrulesdir}/71-nvmf-iopolicy-netapp.rules
|
%{_udevrulesdir}/71-nvmf-netapp.rules
|
||||||
# Do not install the dracut rule yet. See rhbz 1742764
|
# Do not install the dracut rule yet. See rhbz 1742764
|
||||||
|
# Is this still true? Now that we support nvme-of boot, do we want to install this file?
|
||||||
# /usr/lib/dracut/dracut.conf.d/70-nvmf-autoconnect.conf
|
# /usr/lib/dracut/dracut.conf.d/70-nvmf-autoconnect.conf
|
||||||
|
%{nmlibdir}/dispatcher.d/99-nvme-nbft-connect.sh
|
||||||
|
%{nmlibdir}/conf.d/99-nvme-nbft-no-ignore-carrier.conf
|
||||||
|
|
||||||
%post
|
%post
|
||||||
if [ $1 -eq 1 ] || [ $1 -eq 2 ]; then
|
if [ $1 -eq 1 ] || [ $1 -eq 2 ]; then
|
||||||
@ -105,7 +88,7 @@ if [ $1 -eq 1 ] || [ $1 -eq 2 ]; then
|
|||||||
echo $(nvme gen-hostnqn) > %{_sysconfdir}/nvme/hostnqn
|
echo $(nvme gen-hostnqn) > %{_sysconfdir}/nvme/hostnqn
|
||||||
fi
|
fi
|
||||||
if [ ! -s %{_sysconfdir}/nvme/hostid ]; then
|
if [ ! -s %{_sysconfdir}/nvme/hostid ]; then
|
||||||
uuidgen > %{_sysconfdir}/nvme/hostid
|
echo $(nvme show-hostnqn | sed 's/^.*uuid://') > %{_sysconfdir}/nvme/hostid
|
||||||
fi
|
fi
|
||||||
|
|
||||||
# apply udev and systemd changes that we did
|
# apply udev and systemd changes that we did
|
||||||
@ -118,51 +101,115 @@ if [ $1 -eq 1 ] || [ $1 -eq 2 ]; then
|
|||||||
fi
|
fi
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
* Thu Apr 20 2023 Maurizio Lombardi <mlombard@redhat.com> - 1.16-9
|
* Thu Aug 22 2024 Tomas Bzatek <tbzatek@redhat.com> - 2.9.1-6
|
||||||
- Fix BZ #2187288
|
- Install NetworkManager override for nbft interfaces
|
||||||
|
- Rename reconnect NetworkManager hook to 99-nvme-nbft-connect.sh
|
||||||
|
|
||||||
* Wed Apr 05 2023 Maurizio Lombardi <mlombard@redhat.com> - 1.16-8
|
* Tue Aug 06 2024 Maurizio Lombardi <mlombard@redhat.com> - 2.9.1-5
|
||||||
- Fix BZ #1954185
|
- Fix RHEL-38372
|
||||||
|
|
||||||
* Wed Nov 09 2022 Maurizio Lombardi <mlombard@redhat.com> - 1.16-7
|
* Wed Jul 24 2024 Tomas Bzatek <tbzatek@redhat.com> - 2.9.1-4
|
||||||
- Do not re-enable the nvmefc-boot-connections service if
|
- Backport NBFT discovery support (RHEL-37601)
|
||||||
we are just upgrading the package.
|
|
||||||
|
|
||||||
* Thu Oct 13 2022 Maurizio Lombardi <mlombard@redhat.com> - 1.16-6
|
* Wed June 19 2024 Maurizio Lombardi <mlombard@redhat.com> - 2.9.1-3
|
||||||
- Fix a file permission
|
- Fix RHEL-36139
|
||||||
|
|
||||||
* Fri Jul 15 2022 Maurizio Lombardi <mlombard@redhat.com> - 1.16-5
|
* Tue May 07 2024 Tomas Bzatek <tbzatek@redhat.com> - 2.9.1-2
|
||||||
- Fix a compiler warning
|
- Install custom nvmf-connect-nbft.sh NetworkManager hook (RHEL-18912)
|
||||||
|
|
||||||
* Tue Jul 12 2022 Maurizio Lombardi <mlombard@redhat.com> - 1.16-4
|
* Tue May 07 2024 Maurizio Lombardi <mlombard@redhat.com> - 2.9.1-1
|
||||||
- Merge fixes for PowerPC
|
- Update to version 2.9.1
|
||||||
|
|
||||||
* Fri Jan 07 2022 Maurizio Lombardi <mlombard@redhat.com> - 1.16-3
|
* Wed Apr 03 2024 Maurizio Lombardi <mlombard@redhat.com> - 2.8-1
|
||||||
- Merge a few bugfixes
|
- Update to version 2.8
|
||||||
|
|
||||||
* Mon Nov 29 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.16-2
|
* Thu Feb 22 2024 Maurizio Lombardi <mlombard@redhat.com> - 2.6-5
|
||||||
- Update to version 1.16
|
- Fix for RHEL-13107
|
||||||
|
|
||||||
* Thu Jul 22 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.14-2
|
* Mon Nov 13 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.6-4
|
||||||
- Merge various bugfixes
|
- Fix for RHEL-16216 (revert 1:1 mapping between hostnqn and hostid)
|
||||||
|
|
||||||
* Wed Apr 28 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.14-1
|
* Wed Nov 08 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.6-3
|
||||||
- Update to version v1.14
|
- Fixes for RHEL-12566
|
||||||
|
|
||||||
* Tue Apr 20 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.13-4
|
* Mon Nov 06 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.6-2
|
||||||
- Rebuild to mark bz1949462 as fixed
|
- Rebuild for side-tag
|
||||||
|
|
||||||
* Fri Apr 16 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.13-3
|
* Tue Oct 31 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.6-1
|
||||||
- KATO values fixes for various controllers
|
- Update to version 2.6
|
||||||
|
|
||||||
* Wed Apr 14 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.13-2
|
* Mon Aug 21 2023 John Meneghini <jmeneghi@redhat.com> - 2.4-10
|
||||||
- Add dependency to util-linux and fix the post install script
|
- JIRA: https://issues.redhat.com/browse/RHEL-1492
|
||||||
|
|
||||||
* Wed Apr 14 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.13-1
|
* Thu Aug 10 2023 John Meneghini <jmeneghi@redhat.com> - 2.4-9
|
||||||
- Update to 1.13
|
- JIRA: https://issues.redhat.com/browse/RHEL-1147
|
||||||
|
|
||||||
* Tue Jun 16 2020 Fedora Release Monitoring <release-monitoring@fedoraproject.org> - 1.12-1
|
* Mon Jul 17 2023 John Meneghini <jmeneghi@redhat.com> - 2.4-8
|
||||||
- Update to 1.12 (#1827581)
|
- Fix BZ#2223436
|
||||||
|
|
||||||
|
* Wed Jun 14 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.4-7
|
||||||
|
- Fix BZ#2210656
|
||||||
|
|
||||||
|
* Tue May 30 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.4-6
|
||||||
|
- Rebuild for #2208399
|
||||||
|
|
||||||
|
* Thu May 25 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.4-5
|
||||||
|
- Fix SSTAT print (BZ2208399)
|
||||||
|
|
||||||
|
* Tue May 16 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.4-4
|
||||||
|
- Add support to NBFT (BZ2188518)
|
||||||
|
|
||||||
|
* Fri May 12 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.4-3
|
||||||
|
- Fix a warning when building the package BZ2195897
|
||||||
|
|
||||||
|
* Wed May 03 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.4-2
|
||||||
|
- Fix a bogus changelog date BZ2186074
|
||||||
|
|
||||||
|
* Mon Apr 03 2023 Maurizio Lombardi <mlombard@redhat.com> - 2.4-1
|
||||||
|
- Update to version v2.4
|
||||||
|
|
||||||
|
* Thu Nov 10 2022 Maurizio Lombardi <mlombard@redhat.com> - 2.1.2-2
|
||||||
|
- Do not re-enable nvmefc-boot-connections when the package gets updated
|
||||||
|
|
||||||
|
* Mon Sep 26 2022 Maurizio Lombardi <mlombard@redhat.com> - 2.1.2-1
|
||||||
|
- Update to version v2.1.2
|
||||||
|
|
||||||
|
* Mon Aug 29 2022 Maurizio Lombardi <mlombard@redhat.com> - 2.0-4
|
||||||
|
- Fix BZ2104945
|
||||||
|
|
||||||
|
* Fri Jul 15 2022 Maurizio Lombardi <mlombard@redhat.com> - 2.0-3
|
||||||
|
- Fix BZ2105742
|
||||||
|
|
||||||
|
* Thu Jun 16 2022 Maurizio Lombardi <mlombard@redhat.com> - 2.0-2
|
||||||
|
- Fix the gating tests
|
||||||
|
|
||||||
|
* Wed Apr 27 2022 Maurizio Lombardi <mlombard@redhat.com> - 2.0-1
|
||||||
|
- Upgrade to version 2.0
|
||||||
|
|
||||||
|
* Mon Feb 07 2022 Maurizio Lombardi <mlombard@redhat.com> - 1.16-3
|
||||||
|
- Add a few bugfixes
|
||||||
|
|
||||||
|
* Mon Dec 13 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.16-2
|
||||||
|
- Update to the latest version
|
||||||
|
|
||||||
|
* Thu Dec 09 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.14-4
|
||||||
|
- Fix handling of the ctrl_loss_tmo parameter
|
||||||
|
|
||||||
|
* Mon Aug 09 2021 Mohan Boddu <mboddu@redhat.com>
|
||||||
|
- Rebuilt for IMA sigs, glibc 2.34, aarch64 flags
|
||||||
|
Related: rhbz#1991688
|
||||||
|
|
||||||
|
* Mon Jun 14 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.14-2
|
||||||
|
- Fix for bz1962422 (nvme flush failed from from v5.13-rc1)
|
||||||
|
|
||||||
|
* Mon May 03 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.14-1
|
||||||
|
- Update to the latest upstream version
|
||||||
|
|
||||||
|
* Fri Apr 16 2021 Mohan Boddu <mboddu@redhat.com>
|
||||||
|
- Rebuilt for RHEL 9 BETA on Apr 15th 2021. Related: rhbz#1947937
|
||||||
|
|
||||||
|
* Fri Mar 19 2021 Maurizio Lombardi <mlombard@redhat.com> - 1.12-1
|
||||||
|
- Update to 1.13 and add postin scriptlet
|
||||||
|
|
||||||
* Sat Apr 25 2020 luto@kernel.org - 1.11.1-1
|
* Sat Apr 25 2020 luto@kernel.org - 1.11.1-1
|
||||||
- Update to 1.11
|
- Update to 1.11
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user