Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
openSUSE:Leap:15.5:Update
mdadm.28825
1011-Improvements-for-IMSM_NO_PLATFORM-testing....
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File 1011-Improvements-for-IMSM_NO_PLATFORM-testing.patch of Package mdadm.28825
From a1174f60abe1d8ee6921649892354b79112f5279 Mon Sep 17 00:00:00 2001 From: NeilBrown <neilb@suse.de> Date: Wed, 18 Jan 2023 10:18:53 +1100 Subject: [PATCH] Improvements for IMSM_NO_PLATFORM testing. Factor out IMSM_NO_PLATFORM testing into a single function that caches the result. Allow mdmon to explicitly set the result to "1" so that we don't need the ENV var in the unit file Check if the kernel command line contains "mdadm.imsm.test=1" and in that case assert NO_PLATFORM. This simplifies testing in a virtual machine. Signed-off-by: NeilBrown <neilb@suse.de> --- mdadm.8.in | 5 +++++ mdadm.h | 2 ++ mdmon.c | 6 ++++++ super-intel.c | 45 ++++++++++++++++++++++++++++++++++++++++++--- systemd/mdmon@.service | 3 --- 5 files changed, 55 insertions(+), 6 deletions(-) --- a/mdadm.8.in +++ b/mdadm.8.in @@ -3148,6 +3148,11 @@ environment. This can be useful for tes recovery. You should be aware that interoperability may be compromised by setting this value. +These change can also be suppressed by adding +.B mdadm.imsm.test=1 +to the kernel command line. This makes it easy to test IMSM +code in a virtual machine that doesn't have IMSM virtual hardware. + .TP .B MDADM_GROW_ALLOW_OLD If an array is stopped while it is performing a reshape and that --- a/mdadm.h +++ b/mdadm.h @@ -1163,6 +1163,8 @@ extern struct superswitch super0, super1 extern struct superswitch super_imsm, super_ddf; extern struct superswitch mbr, gpt; +void imsm_set_no_platform(int v); + struct metadata_update { int len; char *buf; --- a/mdmon.c +++ b/mdmon.c @@ -318,6 +318,12 @@ int main(int argc, char *argv[]) argv[0][0] = '@'; } + /* + * mdmon should never complain due to lack of a platform, + * that is mdadm's job if at all. + */ + imsm_set_no_platform(1); + while ((opt = getopt_long(argc, argv, "thaF", options, NULL)) != -1) { switch (opt) { case 'a': --- a/super-intel.c +++ b/super-intel.c @@ -20,6 +20,7 @@ #define HAVE_STDINT_H 1 #include "mdadm.h" #include "mdmon.h" +#include "dlink.h" #include "sha1.h" #include "platform-intel.h" #include <values.h> @@ -597,6 +598,44 @@ static const char *_sys_dev_type[] = { [SYS_DEV_VMD] = "VMD" }; +static int no_platform = -1; + +static int check_no_platform(void) +{ + static const char search[] = "mdadm.imsm.test=1"; + FILE *fp; + + if (no_platform >= 0) + return no_platform; + + if (check_env("IMSM_NO_PLATFORM")) { + no_platform = 1; + return 1; + } + fp = fopen("/proc/cmdline", "r"); + if (fp) { + char *l = conf_line(fp); + char *w = l; + + do { + if (strcmp(w, search) == 0) + no_platform = 1; + w = dl_next(w); + } while (w != l); + free_line(l); + fclose(fp); + if (no_platform >= 0) + return no_platform; + } + no_platform = 0; + return 0; +} + +void imsm_set_no_platform(int v) +{ + no_platform = v; +} + const char *get_sys_dev_type(enum sys_dev_type type) { if (type >= SYS_DEV_MAX) @@ -2542,7 +2581,7 @@ static int detail_platform_imsm(int verb int result=1; if (enumerate_only) { - if (check_env("IMSM_NO_PLATFORM")) + if (check_no_platform()) return 0; list = find_intel_devices(); if (!list) @@ -4549,7 +4588,7 @@ static int find_intel_hba_capability(int devname); return 1; } - if (fd < 0 || check_env("IMSM_NO_PLATFORM")) { + if (fd < 0 || check_no_platform()) { super->orom = NULL; super->hba = NULL; return 0; @@ -10493,7 +10532,7 @@ int open_backup_targets(struct mdinfo *i ******************************************************************************/ int validate_container_imsm(struct mdinfo *info) { - if (check_env("IMSM_NO_PLATFORM")) + if (check_no_platform()) return 0; struct sys_dev *idev; --- a/systemd/mdmon@.service +++ b/systemd/mdmon@.service @@ -11,9 +11,6 @@ DefaultDependencies=no Before=initrd-switch-root.target [Service] -# mdmon should never complain due to lack of a platform, -# that is mdadm's job if at all. -Environment=IMSM_NO_PLATFORM=1 # The mdmon starting in the initramfs (with dracut at least) # cannot see sysfs after root is mounted, so we will have to # 'takeover'. As the '--offroot --takeover' don't hurt when
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor