From: Luis Chamberlain <mcgrof@kernel.org>
To: kdevops@lists.linux.dev, chuck.lever@oracle.com,
da.gomez@samsung.com, jlayton@kernel.org
Cc: mcgrof@kernel.org
Subject: [PATCH 2/3] kdevops: remove duplicate symbols and use scripts/append-makefile-vars.sh
Date: Wed, 4 Sep 2024 16:30:36 -0700 [thread overview]
Message-ID: <20240904233037.1516218-3-mcgrof@kernel.org> (raw)
In-Reply-To: <20240904233037.1516218-1-mcgrof@kernel.org>
A few kdevops kconfig duplicate symbols exist, remove them so to ensure
we can work with the latest linux-next version of kconfig. Additionally
we need to ensure to use scripts/append-makefile-vars.sh for when we
want to work with Makefile inherited variables. The KDEVOPS_PLAYBOOK_DIR
can also just be simplified, while we do provide for variability in
practice we never use this so just this a silent default for now.
Without this we end up in a hung state if we run 'make menuconfig'
with the latest kconfig. Fix this.
Signed-off-by: Luis Chamberlain <mcgrof@kernel.org>
---
kconfigs/Kconfig.ansible_provisioning | 24 ++++++------------------
terraform/aws/Kconfig | 14 --------------
vagrant/Kconfig | 3 ++-
vagrant/Kconfig.opensuse | 22 ----------------------
4 files changed, 8 insertions(+), 55 deletions(-)
diff --git a/kconfigs/Kconfig.ansible_provisioning b/kconfigs/Kconfig.ansible_provisioning
index 82a6f822c3f5..63f04306e7b2 100644
--- a/kconfigs/Kconfig.ansible_provisioning
+++ b/kconfigs/Kconfig.ansible_provisioning
@@ -1,12 +1,6 @@
-if !KDEVOPS_ANSIBLE_PROVISION_ENABLE
-
config KDEVOPS_PLAYBOOK_DIR
string
- default $(shell, echo $KDEVOPS_PLAYBOOKS_DIR)
- help
- The directory where ansible playbooks are found.
-
-endif # KDEVOPS_ANSIBLE_PROVISION_ENABLE
+ default $(shell, scripts/append-makefile-vars.sh $(KDEVOPS_PLAYBOOKS_DIR))
config HAVE_DISTRO_PREFERS_DISABLE_ANSIBLE
bool
@@ -41,7 +35,7 @@ config KDEVOPS_DEVCONFIG_ENABLE_CONSOLE
bool "Enable console setup"
default y
depends on KDEVOPS_DEVCONFIG_ENABLE
- help
+ help
Enable this so that we configure and enable console serial settings
for you on /etc/default/grub.
@@ -50,7 +44,7 @@ if KDEVOPS_DEVCONFIG_ENABLE_CONSOLE
config KDEVOPS_DEVCONFIG_KERNEL_CONSOLE_SETTINGS
string "Kernel command line console kernel parameters to use"
default "console=tty0 console=tty1 console=ttyS0,115200n8"
- help
+ help
Linux kernel console kernel parameters to use. This will be appended
to the GRUB configuration file /etc/default/grub on the variable
GRUB_CMDLINE_LINUX_DEFAULT, if and only if you did not have any
@@ -59,7 +53,7 @@ config KDEVOPS_DEVCONFIG_KERNEL_CONSOLE_SETTINGS
config KDEVOPS_DEVCONFIG_GRUB_SERIAL_COMMAND
string "GRUB specific serial settings"
default "serial --speed=115200 --unit=0 --parity=no --stop=1"
- help
+ help
GRUB console settings to use. This will be appended to the GRUB
configuration file /etc/default/grub on the variable
GRUB_SERIAL_COMMAND, if and only if we did change the your console
@@ -68,7 +62,7 @@ config KDEVOPS_DEVCONFIG_GRUB_SERIAL_COMMAND
config KDEVOPS_GRUB_TIMEOUT
int "GRUB timeout"
default 2
- help
+ help
GRUB timeout to wait for user interaction to let you pick another
kernel or boot options.
@@ -104,15 +98,9 @@ config KDEVOPS_DEVCONFIG_SYSTEMD_WATCHDOG_TIMEOUT_KEXEC
endif # KDEVOPS_DEVCONFIG_ENABLE_SYSTEMD_WATCHDOG
-config KDEVOPS_PLAYBOOK_DIR
- string "The directory for the ansible playbooks to use"
- default $(shell, echo $KDEVOPS_PLAYBOOKS_DIR)
- help
- The directory where ansible playbooks are found.
-
config KDEVOPS_ANSIBLE_INVENTORY_FILE
string "The ansible inventory file to use"
- default $(shell, echo $KDEVOPS_HOSTFILE)
+ default $(shell, scripts/append-makefile-vars.sh $(KDEVOPS_HOSTFILE))
help
The file to use for the ansible inventory.
diff --git a/terraform/aws/Kconfig b/terraform/aws/Kconfig
index dafacaeada15..a94045afcfee 100644
--- a/terraform/aws/Kconfig
+++ b/terraform/aws/Kconfig
@@ -280,13 +280,6 @@ config TERRAFORM_AWS_INSTANCE_IS4GEN_MEDIUM
6 GiB RAM, 1vCPU, 1 x 937 GiB NVMe SSD, 25 Gbps Net, 9.6 Gbps EBS.
https://aws.amazon.com/ec2/instance-types/i4g/
-config TERRAFORM_AWS_INSTANCE_IS4GEN_8XLARGE
- bool "is4gen.8xlarge"
- depends on TARGET_ARCH_ARM64
- help
- 192 GiB RAM, 32 vCPUs, 4 x 7500 GiB NVMe SSD, 50 Gbps Net, 19 Gbps EBS.
- https://aws.amazon.com/ec2/instance-types/i4g/
-
config TERRAFORM_AWS_INSTANCE_IS4GEN_8XLARGE
bool "is4gen.8xlarge"
depends on TARGET_ARCH_ARM64
@@ -308,13 +301,6 @@ config TERRAFORM_AWS_INSTANCE_IM4GN_4XLARGE
64 GiB RAM, 16 vcpus, 1 x 7500 NVMe SSD, 25 Gbps Net, 9.5 Gbps EBS.
https://aws.amazon.com/ec2/instance-types/i4g/
-config TERRAFORM_AWS_INSTANCE_IM4GN_4XLARGE
- bool "im4gn.4xlarge"
- depends on TARGET_ARCH_ARM64
- help
- 64 GiB RAM, 16 vcpus, 1 x 7500 NVMe SSD, 25 Gbps Net, 9.5 Gbps EBS.
- https://aws.amazon.com/ec2/instance-types/i4g/
-
config TERRAFORM_AWS_INSTANCE_I4I_LARGE
bool "i4i.large"
depends on TARGET_ARCH_X86_64
diff --git a/vagrant/Kconfig b/vagrant/Kconfig
index 93480274885e..ecf7bcafe1ad 100644
--- a/vagrant/Kconfig
+++ b/vagrant/Kconfig
@@ -1,4 +1,5 @@
if VAGRANT
+
choice
prompt "Vagrant virtualization technology to use"
default VAGRANT_LIBVIRT_SELECT
@@ -89,7 +90,7 @@ config VIRTUALBOX_EXTRA_DRIVE_FORMAT
default "vmdk" if VIRTUALBOX_EXTRA_DRIVE_VMDK
default "vhd" if VIRTUALBOX_EXTRA_DRIVE_VHD
-endif
+endif # VAGRANT_VIRTUALBOX
config HAVE_SUSE_VAGRANT
bool
diff --git a/vagrant/Kconfig.opensuse b/vagrant/Kconfig.opensuse
index 2ffb1f31c02b..6da402755c26 100644
--- a/vagrant/Kconfig.opensuse
+++ b/vagrant/Kconfig.opensuse
@@ -56,28 +56,6 @@ config VAGRANT_OPENSUSE_X86_64_LEAP_15_3
help
This enables the OpenSUSE Leap 15.3 release.
-config VAGRANT_OPENSUSE_X86_64_LEAP_15_4
- bool "Leap-15.4.x86_64"
- depends on TARGET_ARCH_X86_64
- select HAVE_VAGRANT_BOX_URL
- select HAVE_DISTRO_BTRFS_DISABLES_RAID56 if FSTESTS_BTRFS
- select HAVE_DISTRO_XFS_SUPPORTS_CRC if FSTESTS_XFS
- select HAVE_DISTRO_XFS_SUPPORTS_EXTERNAL_LOG if FSTESTS_XFS
- select HAVE_DISTRO_XFS_SUPPORTS_REFLINKS if FSTESTS_XFS
- select HAVE_DISTRO_XFS_SUPPORTS_BIGBLOCKS if FSTESTS_XFS
- select HAVE_DISTRO_PREFERS_FSTESTS_WATCHDOG
- select HAVE_DISTRO_PREFERS_FSTESTS_WATCHDOG_KILL
- select HAVE_DISTRO_BLKTESTS_TEST_BLOCK if KDEVOPS_WORKFLOW_ENABLE_BLKTESTS
- select HAVE_DISTRO_BLKTESTS_TEST_LOOP if KDEVOPS_WORKFLOW_ENABLE_BLKTESTS
- select HAVE_DISTRO_BLKTESTS_TEST_NBD if KDEVOPS_WORKFLOW_ENABLE_BLKTESTS
- select HAVE_DISTRO_PREFERS_BLKTESTS_WATCHDOG if KDEVOPS_WORKFLOW_ENABLE_BLKTESTS
- select HAVE_DISTRO_PREFERS_BLKTESTS_WATCHDOG_KILL if KDEVOPS_WORKFLOW_ENABLE_BLKTESTS
- select HAVE_DISTRO_PREFERS_REFRESH
- select HAVE_DISTRO_PREFERS_UPDATE
- select HAVE_DISTRO_PREFERS_INSTALL_KDEVTOOLS
- help
- This enables the OpenSUSE Leap 15.4 release.
-
endchoice
if VAGRANT_OPENSUSE_X86_64_TW
--
2.43.0
next prev parent reply other threads:[~2024-09-04 23:30 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-04 23:30 [PATCH 0/3] kdevops: update to latest kconfig Luis Chamberlain
2024-09-04 23:30 ` [PATCH 1/3] scripts/kconfig/kconfig.Makefile: move _refs-default Luis Chamberlain
2024-09-04 23:30 ` Luis Chamberlain [this message]
2024-09-04 23:30 ` [PATCH 3/3] scripts/gen-refs-default.Makefile: support direct make requests Luis Chamberlain
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=20240904233037.1516218-3-mcgrof@kernel.org \
--to=mcgrof@kernel.org \
--cc=chuck.lever@oracle.com \
--cc=da.gomez@samsung.com \
--cc=jlayton@kernel.org \
--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