diff options
author | murray <murray@FreeBSD.org> | 2001-09-24 10:16:23 +0000 |
---|---|---|
committer | murray <murray@FreeBSD.org> | 2001-09-24 10:16:23 +0000 |
commit | 5db7811e708c1fdf315431ddd6deed09f2cc87b4 (patch) | |
tree | 93adea49800e8fdf86444bd44855929cfc0c2970 /usr.sbin | |
parent | e97a4764bb4efd497473925f8645c948290bd743 (diff) | |
download | FreeBSD-src-5db7811e708c1fdf315431ddd6deed09f2cc87b4.zip FreeBSD-src-5db7811e708c1fdf315431ddd6deed09f2cc87b4.tar.gz |
Add a couple of functions to create persistent variables that will
survive a sysinstall Ctrl-C -> 'Restart'. This fixes another annoying
bug where restarting sysinstall will try to reload kernel modules and
do other external things that have already been done. For now, use
these persistent variables to keep track of module, usbd, and pccardd
initialization.
Bug found by: rwatson
MFC after: 1 week
Diffstat (limited to 'usr.sbin')
-rw-r--r-- | usr.sbin/sade/main.c | 22 | ||||
-rw-r--r-- | usr.sbin/sade/sade.h | 2 | ||||
-rw-r--r-- | usr.sbin/sade/variable.c | 37 | ||||
-rw-r--r-- | usr.sbin/sysinstall/main.c | 22 | ||||
-rw-r--r-- | usr.sbin/sysinstall/sysinstall.h | 2 | ||||
-rw-r--r-- | usr.sbin/sysinstall/variable.c | 37 |
6 files changed, 110 insertions, 12 deletions
diff --git a/usr.sbin/sade/main.c b/usr.sbin/sade/main.c index 19d1c0a..a562029 100644 --- a/usr.sbin/sade/main.c +++ b/usr.sbin/sade/main.c @@ -101,14 +101,24 @@ main(int argc, char **argv) if (DebugFD) dup2(DebugFD, 2); - /* Initialize driver modules */ - moduleInitialize(); + /* Initialize driver modules, if we haven't already done so (ie, + the user hit Ctrl-C -> Restart. */ + if (!pvariable_get("modulesInitialize")) { + moduleInitialize(); + pvariable_set("modulesInitialize=1"); + } - /* Initialize PC-card */ - pccardInitialize(); + /* Initialize PC-card, if we haven't already done so. */ + if (!pvariable_get("pccardInitialize")) { + pccardInitialize(); + pvariable_set("pccardInitialize=1"); + } - /* Initialize USB */ - usbInitialize(); + /* Initialize USB, if we haven't already done so. */ + if (!pvariable_get("usbInitialize")) { + usbInitialize(); + pvariable_set("usbInitialize=1"); + } /* Probe for all relevant devices on the system */ deviceGetAll(); diff --git a/usr.sbin/sade/sade.h b/usr.sbin/sade/sade.h index c6e2b9b..fb764b8 100644 --- a/usr.sbin/sade/sade.h +++ b/usr.sbin/sade/sade.h @@ -772,6 +772,8 @@ extern char *variable_get_value(char *var, char *prompt, int dirty); extern int variable_check(char *data); extern int dump_variables(dialogMenuItem *self); extern void free_variables(void); +extern void pvariable_set(char *var); +extern char *pvariable_get(char *var); /* wizard.c */ extern void slice_wizard(Disk *d); diff --git a/usr.sbin/sade/variable.c b/usr.sbin/sade/variable.c index 2def090..6f896ee 100644 --- a/usr.sbin/sade/variable.c +++ b/usr.sbin/sade/variable.c @@ -8,6 +8,8 @@ * * Copyright (c) 1995 * Jordan Hubbard. All rights reserved. + * Copyright (c) 2001 + * Murray Stokely. All rights reserved. * * Redistribution and use in source and binary forms, with or without * modification, are permitted provided that the following conditions @@ -257,3 +259,38 @@ free_variables(void) VarHead = NULL; } } + +/* + * Persistent variables. The variables modified by these functions + * are not cleared between invocations of sysinstall. This is useful + * to allow the user to completely restart sysinstall, without having + * it load all of the modules again from the installation media which + * are still in memory. + */ + +void +pvariable_set(char *var) +{ + char tmp[1024]; + + if (!var) + msgFatal("NULL variable name & value passed."); + else if (!*var) + msgDebug("Warning: Zero length name & value passed to variable_set()\n"); + /* Add a trivial namespace to whatever name the caller chooses. */ + SAFE_STRCPY(tmp, "SYSINSTALL_PVAR"); + if (index(var, '=') == NULL) + msgFatal("Invalid variable format: %s", var); + strlcat(tmp, var, 1024); + putenv(tmp); +} + +char * +pvariable_get(char *var) +{ + char tmp[1024]; + + SAFE_STRCPY(tmp, "SYSINSTALL_PVAR"); + strlcat(tmp, var, 1024); + return getenv(tmp); +} diff --git a/usr.sbin/sysinstall/main.c b/usr.sbin/sysinstall/main.c index 19d1c0a..a562029 100644 --- a/usr.sbin/sysinstall/main.c +++ b/usr.sbin/sysinstall/main.c @@ -101,14 +101,24 @@ main(int argc, char **argv) if (DebugFD) dup2(DebugFD, 2); - /* Initialize driver modules */ - moduleInitialize(); + /* Initialize driver modules, if we haven't already done so (ie, + the user hit Ctrl-C -> Restart. */ + if (!pvariable_get("modulesInitialize")) { + moduleInitialize(); + pvariable_set("modulesInitialize=1"); + } - /* Initialize PC-card */ - pccardInitialize(); + /* Initialize PC-card, if we haven't already done so. */ + if (!pvariable_get("pccardInitialize")) { + pccardInitialize(); + pvariable_set("pccardInitialize=1"); + } - /* Initialize USB */ - usbInitialize(); + /* Initialize USB, if we haven't already done so. */ + if (!pvariable_get("usbInitialize")) { + usbInitialize(); + pvariable_set("usbInitialize=1"); + } /* Probe for all relevant devices on the system */ deviceGetAll(); diff --git a/usr.sbin/sysinstall/sysinstall.h b/usr.sbin/sysinstall/sysinstall.h index c6e2b9b..fb764b8 100644 --- a/usr.sbin/sysinstall/sysinstall.h +++ b/usr.sbin/sysinstall/sysinstall.h @@ -772,6 +772,8 @@ extern char *variable_get_value(char *var, char *prompt, int dirty); extern int variable_check(char *data); extern int dump_variables(dialogMenuItem *self); extern void free_variables(void); +extern void pvariable_set(char *var); +extern char *pvariable_get(char *var); /* wizard.c */ extern void slice_wizard(Disk *d); diff --git a/usr.sbin/sysinstall/variable.c b/usr.sbin/sysinstall/variable.c index 2def090..6f896ee 100644 --- a/usr.sbin/sysinstall/variable.c +++ b/usr.sbin/sysinstall/variable.c @@ -8,6 +8,8 @@ * * Copyright (c) 1995 * Jordan Hubbard. All rights reserved. + * Copyright (c) 2001 + * Murray Stokely. All rights reserved. * * Redistribution and use in source and binary forms, with or without * modification, are permitted provided that the following conditions @@ -257,3 +259,38 @@ free_variables(void) VarHead = NULL; } } + +/* + * Persistent variables. The variables modified by these functions + * are not cleared between invocations of sysinstall. This is useful + * to allow the user to completely restart sysinstall, without having + * it load all of the modules again from the installation media which + * are still in memory. + */ + +void +pvariable_set(char *var) +{ + char tmp[1024]; + + if (!var) + msgFatal("NULL variable name & value passed."); + else if (!*var) + msgDebug("Warning: Zero length name & value passed to variable_set()\n"); + /* Add a trivial namespace to whatever name the caller chooses. */ + SAFE_STRCPY(tmp, "SYSINSTALL_PVAR"); + if (index(var, '=') == NULL) + msgFatal("Invalid variable format: %s", var); + strlcat(tmp, var, 1024); + putenv(tmp); +} + +char * +pvariable_get(char *var) +{ + char tmp[1024]; + + SAFE_STRCPY(tmp, "SYSINSTALL_PVAR"); + strlcat(tmp, var, 1024); + return getenv(tmp); +} |