public inbox for kdevops@lists.linux.dev
 help / color / mirror / Atom feed
From: cel@kernel.org
To: <kdevops@lists.linux.dev>
Cc: Chuck Lever <chuck.lever@oracle.com>
Subject: [PATCH 1/4] bringup.goals: Make siw settings more generic
Date: Mon,  3 Jun 2024 12:06:55 -0400	[thread overview]
Message-ID: <20240603160658.4073204-2-cel@kernel.org> (raw)
In-Reply-To: <20240603160658.4073204-1-cel@kernel.org>

From: Chuck Lever <chuck.lever@oracle.com>

I'm about to introduce another option for software RDMA. Rename
the siw Makefile and Kconfig options to make way for this new
option.

Since only one type of emulated RDMA device can be attached to an
Ethernet device at a time, make the Kconfig logic choose between
no device, siw, or, when it's added, rxe.

NB: This patch replaces CONFIG_KDEVOPS_SETUP_SIW. If you have
enabled siw in your configuration, please visit the config setting
for software RDMA after applying this patch.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
---
 Makefile                                |  2 +-
 kconfigs/Kconfig.bringup.goals          | 26 +++++++++++++++++++------
 scripts/{siw.Makefile => rdma.Makefile} |  2 +-
 3 files changed, 22 insertions(+), 8 deletions(-)
 rename scripts/{siw.Makefile => rdma.Makefile} (79%)

diff --git a/Makefile b/Makefile
index 266eccd90111..20c2081118a9 100644
--- a/Makefile
+++ b/Makefile
@@ -104,7 +104,7 @@ ifeq (y,$(CONFIG_WORKFLOWS))
 include workflows/Makefile
 endif # CONFIG_WORKFLOWS
 
-include scripts/siw.Makefile
+include scripts/rdma.Makefile
 include scripts/ktls.Makefile
 include scripts/nfsd.Makefile
 include scripts/smbd.Makefile
diff --git a/kconfigs/Kconfig.bringup.goals b/kconfigs/Kconfig.bringup.goals
index e6d3b1815f3b..e8629ebea5e0 100644
--- a/kconfigs/Kconfig.bringup.goals
+++ b/kconfigs/Kconfig.bringup.goals
@@ -77,16 +77,30 @@ config KDEVOPS_TRY_INSTALL_KDEV_TOOLS
 	  most kernel developers might prefer to have installed on target
 	  systems.
 
-config KDEVOPS_SETUP_SIW
-	bool "Configure soft iWARP on all test systems"
-	default n
+choice
+	prompt "Configure an RDMA device"
+	default KDEVOPS_SETUP_RDMA_NONE
 	help
-	  Configure software-emulated RDMA on all target hosts via the
+	  Provision an RDMA device on target nodes. The default is to
+	  provision no additional RDMA devices.
+
+config KDEVOPS_SETUP_RDMA_NONE
+	bool "none"
+	help
+	  Do not provision an additional RDMA device on target nodes.
+	  This is the default behavior.
+
+config KDEVOPS_SETUP_RDMA_SIW
+	bool "siw"
+	help
+	  Provision software-emulated RDMA on target nodes via the
 	  kernel siw.ko driver. siw can be used with any standard
 	  Ethernet device, including virtio-net devices.
 
-	  Because siw is currently not enabled in most Linux distro
-	  kernels, this option defaults to N.
+	  To use this device driver, enable the CONFIG_RDMA_SIW
+	  option in your test kernels.
+
+endchoice
 
 config KDEVOPS_SETUP_KTLS
 	bool "Configure ktls on the hosts with self-signed CA"
diff --git a/scripts/siw.Makefile b/scripts/rdma.Makefile
similarity index 79%
rename from scripts/siw.Makefile
rename to scripts/rdma.Makefile
index d09d1a2f82ac..d7cdf0062638 100644
--- a/scripts/siw.Makefile
+++ b/scripts/rdma.Makefile
@@ -1,4 +1,4 @@
-ifeq (y,$(CONFIG_KDEVOPS_SETUP_SIW))
+ifeq (y,$(CONFIG_KDEVOPS_SETUP_RDMA_SIW))
 
 siw:
 	$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --extra-vars=@./extra_vars.yaml \
-- 
2.45.1


  reply	other threads:[~2024-06-03 16:07 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-06-03 16:06 [PATCH 0/4] Add rxe as a second software RDMA option cel
2024-06-03 16:06 ` cel [this message]
2024-06-03 16:06 ` [PATCH 2/4] playbooks: Add a playbook to set up rxe on target nodes cel
2024-06-03 16:06 ` [PATCH 3/4] Kconfig: Add support for testing RDMA with software RoCE cel
2024-06-03 16:54   ` Zhu Yanjun
2024-06-07 13:10   ` Zhu Yanjun
2024-06-03 16:06 ` [PATCH 4/4] Update help text that mentions SIW cel
2024-06-05 17:50 ` [PATCH 0/4] Add rxe as a second software RDMA option Chuck Lever III

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240603160658.4073204-2-cel@kernel.org \
    --to=cel@kernel.org \
    --cc=chuck.lever@oracle.com \
    --cc=kdevops@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox