Skip to contentSkip to conversation

fabrics: add option to override drivers queue depth also for connect-all command

PATCH New Johannes Thumshirn 2 participants 3 messages May 11, 2017 – May 18, 2017

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 fabrics: add option to override drivers queue depth also for connect-all command

Currently it is not possible to override the fabrics drivers default queue depth with the nvme userspace utility's 'connect-all' command, but only when using the 'connect' command.

Add an option to override the drivers default queue depth for NVMe over fabrics with the 'connect-all' command as we did previouslt with the 'connect' command.

Signed-off-by Johannes Thumshirn <jthumshirn at suse.de>
---
 Documentation/nvme-connect-all.txt | 5 +++++
 fabrics.c                          | 1 +
 2 files changed, 6 insertions(+)
Documentation/nvme-connect-all.txt +5 -0
index b4b3e40..03adac5 100644
@@ -81,6 +81,11 @@ OPTIONS
 	and dump it to a raw binary file. By default 'nvme connect-all' will
 	dump the output to stdout.
 
+-Q <#>::
+--queue-size=<#>::
+	Overrides the default number of elements in the I/O queues created
+	by the driver.
+
 EXAMPLES
 --------
 * Connect to all records returned by the Discover Controller with IP4 address
fabrics.c +1 -0
index a826ecc..bbcca47 100644
@@ -782,6 +782,7 @@ int discover(const char *desc, int argc, char **argv, bool connect)
 		{"trsvcid",     's', "LIST", CFG_STRING, &cfg.trsvcid,     required_argument, "transport service id (e.g. IP port)" },
 		{"host-traddr", 'w', "LIST", CFG_STRING, &cfg.host_traddr, required_argument, "host traddr (e.g. FC WWN's)" },
 		{"hostnqn",     'q', "LIST", CFG_STRING, &cfg.hostnqn,     required_argument, "user-defined hostnqn (if default not used)" },
+		{"queue-size",  'Q', "LIST", CFG_STRING, &cfg.queue_size,  required_argument, "number of io queue elements to use (default 128)" },
 		{"raw",         'r', "LIST", CFG_STRING, &cfg.raw,         required_argument, "raw output file" },
 		{NULL},
 	};
Signature · 1 line
2.12.0
#1 lore raw

PATCH 1/1 fabrics: add option to override drivers queue depth also for connect-all command

Quoted 10 lines · Johannes Thumshirn

Currently it is not possible to override the fabrics drivers default queue depth with the nvme userspace utility's 'connect-all' command, but only when using the 'connect' command.

Add an option to override the drivers default queue depth for NVMe over fabrics with the 'connect-all' command as we did previouslt with the 'connect' command.

Signed-off-by Johannes Thumshirn <jthumshirn at suse.de>

---

Keith, ping?

Signature · 6 lines
Johannes Thumshirn                                          Storage
jthumshirn at suse.de                                +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 N?rnberg
GF: Felix Imend?rffer, Jane Smithard, Graham Norton
HRB 21284 (AG N?rnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
#2 lore raw

PATCH 1/1 fabrics: add option to override drivers queue depth also for connect-all command

Quoted 11 lines · Johannes Thumshirn
Quoted 10 lines · Johannes Thumshirn

Currently it is not possible to override the fabrics drivers default queue depth with the nvme userspace utility's 'connect-all' command, but only when using the 'connect' command.

Add an option to override the drivers default queue depth for NVMe over fabrics with the 'connect-all' command as we did previouslt with the 'connect' command.

Signed-off-by Johannes Thumshirn <jthumshirn at suse.de>

---

Thanks, Johnanes, looks good, and applied to the repo.