[reactos] 02/02: [FREELDR] Add the possibility to change the boot load options from TXTSETUP.SIF at runtime from the boot selection menu or from FREELDR.INI.
4 Nov
2020
4 Nov
'20
6:59 p.m.
https://git.reactos.org/?p=reactos.git;a=commitdiff;h=4792c007aafa9739c76ff…
commit 4792c007aafa9739c76fff31cb3c19ffdc7a456d
Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org>
AuthorDate: Tue Nov 3 01:42:09 2020 +0100
Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org>
CommitDate: Wed Nov 4 19:59:28 2020 +0100
[FREELDR] Add the possibility to change the boot load options from TXTSETUP.SIF
at runtime from the boot selection menu or from FREELDR.INI.
CORE-17350, CORE-9023
For a proper override of the options by new user options, specify the
/SIFOPTIONSOVERRIDE switch in addition. Otherwise, user options are
merged with those retrieved from TXTSETUP.SIF, with priority given to
the former ones.
- Update the documentation for the 'ReactOSSetup' OS type in the
FREELDR.INI file template.
- Use a different prompt in the custom boot options editor for the
'ReactOSSetup' OS type, with adequate explanation.
- Get rid of the ReactOS-specific TXTSETUP.SIF 'DbgOsLoadOptions' value,
and use instead the Windows-compatible 'SetupDebugOptions' value that
is added to the other load options when debugging is to be enabled.
---
boot/bootdata/txtsetup.sif | 6 +-
boot/freeldr/FREELDR.INI | 24 +-
boot/freeldr/freeldr/custom.c | 9 +-
boot/freeldr/freeldr/ntldr/ntldropts.c | 80 ++++++
boot/freeldr/freeldr/ntldr/ntldropts.h | 7 +
boot/freeldr/freeldr/ntldr/setupldr.c | 429 ++++++++++++++++++++++++++++++---
6 files changed, 511 insertions(+), 44 deletions(-)
diff --git a/boot/bootdata/txtsetup.sif b/boot/bootdata/txtsetup.sif
index 2b7b04002e2..2ddb0512ed3 100644
--- a/boot/bootdata/txtsetup.sif
+++ b/boot/bootdata/txtsetup.sif
@@ -212,10 +212,10 @@ Cabinet=reactos.cab
[SetupData]
DefaultPath = \ReactOS
+SetupDebugOptions = "/DEBUG /KDSERIAL /DEBUGPORT=COM1 /FIRSTCHANCE"
+;SetupDebugOptions = "/DEBUG /SOS /DEBUGPORT=SCREEN"
+;SetupDebugOptions = "/DEBUG /DEBUGPORT=BOCHS"
OsLoadOptions = "/NOGUIBOOT /NODEBUG"
-DbgOsLoadOptions = "/NOGUIBOOT /KDSERIAL /DEBUGPORT=COM1 /FIRSTCHANCE"
-;DbgOsLoadOptions = "/SOS /DEBUGPORT=SCREEN"
-;DbgOsLoadOptions = "/NOGUIBOOT /DEBUGPORT=BOCHS"
[NLS]
AnsiCodepage = c_1252.nls
diff --git a/boot/freeldr/FREELDR.INI b/boot/freeldr/FREELDR.INI
index 0bd420d06dc..48561fc4475 100644
--- a/boot/freeldr/FREELDR.INI
+++ b/boot/freeldr/FREELDR.INI
@@ -115,14 +115,14 @@
; will only be used to set the boot partition.
; Initrd - specifies the optional init ramdisk file name to be used.
; The same remarks about the path as for "Kernel" remain valid.
-; CommandLine - specifies the command line options for the kernel.
+; CommandLine - specifies the boot load options for the kernel.
; ["Windows(NT40|2003)" OSType] Section Commands:
;
; SystemPath - specifies the system root path (must be a valid ARC path):
; multi(0)disk(0)rdisk(0)partition(1)\reactos
; multi(0)disk(0)fdisk(0)
-; Options - specifies the command line options for the kernel being booted.
+; Options - specifies the boot load options for the kernel.
; Kernel - specifies the kernel file name (default: ntoskrnl.exe)
; Hal - specifies the HAL file name (default: hal.dll)
;
@@ -133,8 +133,24 @@
; ["ReactOSSetup" OSType] Section Commands:
;
-; No options defined for the moment. This OS type is used to tell FreeLdr
-; to start the SETUP portion of NT / ReactOS.
+; This OS type is used to start the SETUP portion of NT / ReactOS.
+;
+; SystemPath - specifies the system root path (must be a valid ARC path):
+; multi(0)disk(0)rdisk(0)partition(1)\reactos
+; multi(0)disk(0)fdisk(0)
+; Options - specifies extra boot load options for the kernel and the
+; setup environment (see REMARK 2 below).
+;
+; REMARK: Contrary to the "Windows" type, this OS type does not support
separate
+; "Kernel" and "Hal" values. Instead, these values must be specified
using the
+; NT-compatible "/HAL=filename" and "/KERNEL=filename" option
switches.
+;
+; REMARK 2: The SETUP portion retrieves the default boot load options from the
+; TXTSETUP.SIF file present in the installation source (section "[SetupData]",
+; values "OsLoadOptions" and "SetupDebugOptions"). Thus, any options
specified
+; using the "Options" value will supplement those already obtained from the
+; TXTSETUP.SIF file, unless the "/SIFOPTIONSOVERRIDE" option switch is
specified,
+; in which case the "Options" value overrides those from TXTSETUP.SIF.
[FREELOADER]
diff --git a/boot/freeldr/freeldr/custom.c b/boot/freeldr/freeldr/custom.c
index 7cfd38c8d40..2422107d58e 100644
--- a/boot/freeldr/freeldr/custom.c
+++ b/boot/freeldr/freeldr/custom.c
@@ -36,7 +36,8 @@ const CHAR BootDrivePrompt[] = "Enter the boot
drive.\n\nExamples:\nfd0 - first
const CHAR BootPartitionPrompt[] = "Enter the boot partition.\n\nEnter 0 for the
active (bootable) partition.";
const CHAR ARCPathPrompt[] = "Enter the boot ARC
path.\n\nExamples:\nmulti(0)disk(0)rdisk(0)partition(1)\nmulti(0)disk(0)fdisk(0)";
const CHAR ReactOSSystemPathPrompt[] = "Enter the path to your ReactOS system
directory.\n\nExamples:\n\\REACTOS\n\\ROS";
-const CHAR ReactOSOptionsPrompt[] = "Enter the options you want passed to the
kernel.\n\nExamples:\n/DEBUG /DEBUGPORT=COM1 /BAUDRATE=115200\n/FASTDETECT /SOS
/NOGUIBOOT\n/BASEVIDEO /MAXMEM=64\n/KERNEL=NTKRNLMP.EXE /HAL=HALMPS.DLL";
+const CHAR ReactOSOptionsPrompt[] = "Enter the load options you want passed to the
kernel.\n\nExamples:\n/DEBUG /DEBUGPORT=COM1 /BAUDRATE=115200\n/FASTDETECT /SOS
/NOGUIBOOT\n/BASEVIDEO /MAXMEM=64\n/KERNEL=NTKRNLMP.EXE /HAL=HALMPS.DLL";
+const CHAR ReactOSSetupOptionsPrompt[] = "Enter additional load options you want
passed to the ReactOS Setup.\nThese options will supplement those obtained from the
TXTSETUP.SIF\nfile, unless you also specify the /SIFOPTIONSOVERRIDE option
switch.\n\nExample:\n/NOGUIBOOT /DEBUG /DEBUGPORT=COM1 /BAUDRATE=115200";
const CHAR CustomBootPrompt[] = "Press ENTER to boot your custom boot setup.";
/* FUNCTIONS ******************************************************************/
@@ -659,7 +660,7 @@ EditCustomBootReactOS(
return;
}
- if (!UiEditBox(ReactOSOptionsPrompt, ReactOSOptions, sizeof(ReactOSOptions)))
+ if (!UiEditBox(IsSetup ? ReactOSSetupOptionsPrompt : ReactOSOptionsPrompt,
ReactOSOptions, sizeof(ReactOSOptions)))
return;
/* Modify the settings values and return if we were in edit mode */
@@ -686,7 +687,9 @@ EditCustomBootReactOS(
return;
/* Construct the ReactOS ARC system path */
- ConstructArcPath(ReactOSARCPath, ReactOSSystemPath,
DriveMapGetBiosDriveNumber(BootDriveString), atoi(BootPartitionString));
+ ConstructArcPath(ReactOSARCPath, ReactOSSystemPath,
+ DriveMapGetBiosDriveNumber(BootDriveString),
+ atoi(BootPartitionString));
/* Add the system path */
if (!IniAddSettingValueToSection(SectionId, "SystemPath", ReactOSARCPath))
diff --git a/boot/freeldr/freeldr/ntldr/ntldropts.c
b/boot/freeldr/freeldr/ntldr/ntldropts.c
index 2173924e224..ee3eb344cde 100644
--- a/boot/freeldr/freeldr/ntldr/ntldropts.c
+++ b/boot/freeldr/freeldr/ntldr/ntldropts.c
@@ -131,3 +131,83 @@ NtLdrGetOption(
{
return NtLdrGetOptionEx(Options, OptionName, NULL);
}
+
+/*
+ * Appends or prepends new options to the ones originally contained
+ * in the buffer pointed by LoadOptions, of maximum size BufferSize.
+ */
+VOID
+NtLdrAddOptions(
+ IN OUT PSTR LoadOptions,
+ IN ULONG BufferSize,
+ IN BOOLEAN Append,
+ IN PCSTR NewOptions OPTIONAL)
+{
+ ULONG OptionsLength;
+ ULONG NewOptsLength;
+ BOOLEAN AddSeparator;
+
+ if (!LoadOptions || (BufferSize == 0))
+ return;
+ // ASSERT(strlen(LoadOptions) + 1 <= BufferSize);
+
+ if (!NewOptions || !*NewOptions)
+ return;
+
+ if (Append)
+ {
+ OptionsLength = (ULONG)strlen(LoadOptions);
+ OptionsLength = min(OptionsLength, BufferSize-1);
+
+ /* Add a whitespace separator if needed */
+ if (OptionsLength != 0 &&
+ (LoadOptions[OptionsLength-1] != ' ') &&
+ (LoadOptions[OptionsLength-1] != '\t') &&
+ (*NewOptions != '\0') &&
+ (*NewOptions != ' ') &&
+ (*NewOptions != '\t'))
+ {
+ RtlStringCbCatA(LoadOptions, BufferSize * sizeof(CHAR), " ");
+ }
+
+ /* Append the options */
+ RtlStringCbCatA(LoadOptions, BufferSize * sizeof(CHAR), NewOptions);
+ }
+ else
+ {
+ NewOptsLength = (ULONG)strlen(NewOptions);
+ NewOptsLength = min(NewOptsLength, BufferSize-1);
+
+ /* Add a whitespace separator if needed */
+ AddSeparator = FALSE;
+ if (NewOptsLength != 0 &&
+ (NewOptions[NewOptsLength-1] != ' ') &&
+ (NewOptions[NewOptsLength-1] != '\t') &&
+ (*LoadOptions != '\0') &&
+ (*LoadOptions != ' ') &&
+ (*LoadOptions != '\t'))
+ {
+ AddSeparator = TRUE;
+ ++NewOptsLength;
+ }
+
+ /*
+ * Move the original load options forward (possibly truncating them
+ * at the end if the buffer is not large enough) to make place for
+ * the options to prepend.
+ */
+ OptionsLength = (ULONG)strlen(LoadOptions) + 1;
+ OptionsLength = min(OptionsLength, BufferSize - NewOptsLength);
+ RtlMoveMemory(LoadOptions + NewOptsLength,
+ LoadOptions,
+ OptionsLength * sizeof(CHAR));
+ /* NULL-terminate */
+ (LoadOptions + NewOptsLength)[OptionsLength-1] = '\0';
+
+ /* Restore the new options length back to its original value */
+ if (AddSeparator) --NewOptsLength;
+ /* Prepend the options and add the whitespace separator if needed */
+ strncpy(LoadOptions, NewOptions, NewOptsLength);
+ if (AddSeparator) LoadOptions[NewOptsLength] = ' ';
+ }
+}
diff --git a/boot/freeldr/freeldr/ntldr/ntldropts.h
b/boot/freeldr/freeldr/ntldr/ntldropts.h
index aab539ce92d..9b92eac8e74 100644
--- a/boot/freeldr/freeldr/ntldr/ntldropts.h
+++ b/boot/freeldr/freeldr/ntldr/ntldropts.h
@@ -29,3 +29,10 @@ PCSTR
NtLdrGetOption(
IN PCSTR Options,
IN PCSTR OptionName);
+
+VOID
+NtLdrAddOptions(
+ IN OUT PSTR LoadOptions,
+ IN ULONG BufferSize,
+ IN BOOLEAN Append,
+ IN PCSTR NewOptions OPTIONAL);
diff --git a/boot/freeldr/freeldr/ntldr/setupldr.c b/boot/freeldr/freeldr/ntldr/setupldr.c
index 548e4b7c21b..e80d15e03a6 100644
--- a/boot/freeldr/freeldr/ntldr/setupldr.c
+++ b/boot/freeldr/freeldr/ntldr/setupldr.c
@@ -15,8 +15,6 @@
#include <debug.h>
DBG_DEFAULT_CHANNEL(WINDOWS);
-#define TAG_BOOT_OPTIONS 'pOtB'
-
// TODO: Move to .h
VOID
AllocateAndInitLPB(
@@ -175,6 +173,264 @@ SetupLdrScanBootDrivers(PLIST_ENTRY BootDriverListHead, HINF
InfHandle, PCSTR Se
/* SETUP STARTER **************************************************************/
+/*
+ * Update the options in the buffer pointed by LoadOptions, of maximum size
+ * BufferSize, by first removing any specified options, and then adding any
+ * other ones.
+ *
+ * OptionsToAdd is a NULL-terminated array of string buffer pointers that
+ * specify the options to be added into LoadOptions. Whether they are
+ * prepended or appended to LoadOptions is controlled via the Append
+ * parameter. The options are added in the order specified by the array.
+ *
+ * OptionsToRemove is a NULL-terminated array of string buffer pointers that
+ * specify the options to remove from LoadOptions. Specifying also there
+ * any options to add, has the effect of removing from LoadOptions any
+ * duplicates of the options to be added, before adding them later into
+ * LoadOptions. The options are removed in the order specified by the array.
+ *
+ * The options string buffers in the OptionsToRemove array have the format:
+ * "/option1 /option2[=] ..."
+ *
+ * An option in the OptionsToRemove list with a trailing '=' or ':'
designates
+ * an option in LoadOptions with user-specific data appended after the sign.
+ * When such an option is being removed from LoadOptions, all the appended
+ * data is also removed until the next option.
+ */
+VOID
+NtLdrUpdateLoadOptions(
+ IN OUT PSTR LoadOptions,
+ IN ULONG BufferSize,
+ IN BOOLEAN Append,
+ IN PCSTR OptionsToAdd[] OPTIONAL,
+ IN PCSTR OptionsToRemove[] OPTIONAL)
+{
+ PCSTR NextOptions, NextOpt;
+ PSTR Options, Option;
+ ULONG NextOptLength;
+ ULONG OptionLength;
+
+ if (!LoadOptions || (BufferSize == 0))
+ return;
+ // ASSERT(strlen(LoadOptions) + 1 <= BufferSize);
+
+ /* Loop over the options to remove */
+ for (; OptionsToRemove && *OptionsToRemove; ++OptionsToRemove)
+ {
+ NextOptions = *OptionsToRemove;
+ while ((NextOpt = NtLdrGetNextOption(&NextOptions, &NextOptLength)))
+ {
+ /* Scan the load options */
+ Options = LoadOptions;
+ while ((Option = (PSTR)NtLdrGetNextOption((PCSTR*)&Options,
&OptionLength)))
+ {
+ /*
+ * Check whether the option to find exactly matches the current
+ * load option, or is a prefix thereof if this is an option with
+ * appended data.
+ */
+ if ((OptionLength >= NextOptLength) &&
+ (_strnicmp(Option, NextOpt, NextOptLength) == 0))
+ {
+ if ((OptionLength == NextOptLength) ||
+ (NextOpt[NextOptLength-1] == '=') ||
+ (NextOpt[NextOptLength-1] == ':'))
+ {
+ /* Eat any skipped option or whitespace separators */
+ while ((Option > LoadOptions) &&
+ (Option[-1] == '/' ||
+ Option[-1] == ' ' ||
+ Option[-1] == '\t'))
+ {
+ --Option;
+ }
+
+ /* If the option was not preceded by a whitespace
+ * separator, insert one and advance the pointer. */
+ if ((Option > LoadOptions) &&
+ (Option[-1] != ' ') &&
+ (Option[-1] != '\t') &&
+ (*Options != '\0') /* &&
+ ** Not necessary since NtLdrGetNextOption() **
+ ** stripped any leading separators. **
+ (*Options != ' ') &&
+ (*Options != '\t') */)
+ {
+ *Option++ = ' ';
+ }
+
+ /* Move the remaining options back, erasing the current one */
+ ASSERT(Option <= Options);
+ RtlMoveMemory(Option,
+ Options,
+ (strlen(Options) + 1) * sizeof(CHAR));
+
+ /* Reset the iterator */
+ Options = Option;
+ }
+ }
+ }
+ }
+ }
+
+ /* Now loop over the options to add */
+ for (; OptionsToAdd && *OptionsToAdd; ++OptionsToAdd)
+ {
+ NtLdrAddOptions(LoadOptions,
+ BufferSize,
+ Append,
+ *OptionsToAdd);
+ }
+}
+
+
+/*
+ * List of options and their corresponding higher priority ones,
+ * that are either checked before any other ones, or whose name
+ * includes another option name as a subset (e.g. NODEBUG vs. DEBUG).
+ * See also https://geoffchappell.com/notes/windows/boot/editoptions.htm
+ */
+static const struct
+{
+ PCSTR Options;
+ PCSTR ExtraOptions;
+ PCSTR HigherPriorOptions;
+} HighPriorOptionsMap[] =
+{
+ /* NODEBUG has a higher precedence than DEBUG */
+ {"/DEBUG/DEBUG=", NULL, "/NODEBUG"},
+
+ /* When using SCREEN debug port, we need boot video */
+ {"/DEBUGPORT=SCREEN", NULL, "/NOGUIBOOT"},
+
+ /* DETECTHAL has a higher precedence than HAL= or KERNEL= */
+ {"/HAL=/KERNEL=", NULL, "/DETECTHAL"},
+
+ /* NOPAE has a higher precedence than PAE */
+ {"/PAE", NULL, "/NOPAE"},
+
+ /* NOEXECUTE(=) has a higher precedence than EXECUTE */
+ {"/EXECUTE", "/NOEXECUTE=ALWAYSOFF",
"/NOEXECUTE/NOEXECUTE="},
+ /* NOEXECUTE(=) options are self-excluding and
+ * some have higher precedence than others. */
+ {"/NOEXECUTE/NOEXECUTE=", NULL, "/NOEXECUTE/NOEXECUTE="},
+
+ /* SAFEBOOT(:) options are self-excluding */
+ {"/SAFEBOOT/SAFEBOOT:", NULL, "/SAFEBOOT/SAFEBOOT:"},
+};
+
+#define TAG_BOOT_OPTIONS 'pOtB'
+
+VOID
+NtLdrGetHigherPriorityOptions(
+ IN PCSTR BootOptions,
+ OUT PSTR* ExtraOptions,
+ OUT PSTR* HigherPriorityOptions)
+{
+ ULONG i;
+ PCSTR NextOptions, NextOpt;
+ ULONG NextOptLength;
+ SIZE_T ExtraOptsSize = 0;
+ SIZE_T HighPriorOptsSize = 0;
+
+ /* Masks specifying the presence (TRUE) or absence (FALSE) of the options */
+ BOOLEAN Masks[RTL_NUMBER_OF(HighPriorOptionsMap)];
+
+ /* Just return if we cannot return anything */
+ if (!ExtraOptions && !HigherPriorityOptions)
+ return;
+
+ if (ExtraOptions)
+ *ExtraOptions = NULL;
+ if (HigherPriorityOptions)
+ *HigherPriorityOptions = NULL;
+
+ /* Just return if no initial options were given */
+ if (!BootOptions || !*BootOptions)
+ return;
+
+ /* Determine the presence of the colliding options, and the
+ * maximum necessary sizes for the pointers to be allocated. */
+ RtlZeroMemory(Masks, sizeof(Masks));
+ for (i = 0; i < RTL_NUMBER_OF(HighPriorOptionsMap); ++i)
+ {
+ /* Loop over the given options to search for */
+ NextOptions = HighPriorOptionsMap[i].Options;
+ while ((NextOpt = NtLdrGetNextOption(&NextOptions, &NextOptLength)))
+ {
+ /* If any of these options are present... */
+ if (NtLdrGetOptionExN(BootOptions, NextOpt, NextOptLength, NULL))
+ {
+ /* ... set the mask, retrieve the sizes and stop looking for these
options */
+ Masks[i] = TRUE;
+ if (ExtraOptions && HighPriorOptionsMap[i].ExtraOptions)
+ {
+ ExtraOptsSize += strlen(HighPriorOptionsMap[i].ExtraOptions) *
sizeof(CHAR);
+ }
+ if (HigherPriorityOptions &&
HighPriorOptionsMap[i].HigherPriorOptions)
+ {
+ HighPriorOptsSize +=
strlen(HighPriorOptionsMap[i].HigherPriorOptions) * sizeof(CHAR);
+ }
+ break;
+ }
+ }
+ }
+ /* Count the NULL-terminator */
+ if (ExtraOptions)
+ ExtraOptsSize += sizeof(ANSI_NULL);
+ if (HigherPriorityOptions)
+ HighPriorOptsSize += sizeof(ANSI_NULL);
+
+ /* Allocate the string pointers */
+ if (ExtraOptions)
+ {
+ *ExtraOptions = FrLdrHeapAlloc(ExtraOptsSize, TAG_BOOT_OPTIONS);
+ if (!*ExtraOptions)
+ return;
+ }
+ if (HigherPriorityOptions)
+ {
+ *HigherPriorityOptions = FrLdrHeapAlloc(HighPriorOptsSize, TAG_BOOT_OPTIONS);
+ if (!*HigherPriorityOptions)
+ {
+ if (ExtraOptions)
+ {
+ FrLdrHeapFree(*ExtraOptions, TAG_BOOT_OPTIONS);
+ *ExtraOptions = NULL;
+ }
+ return;
+ }
+ }
+
+ /* Initialize the strings */
+ if (ExtraOptions)
+ *(*ExtraOptions) = '\0';
+ if (HigherPriorityOptions)
+ *(*HigherPriorityOptions) = '\0';
+
+ /* Go through the masks that determine the options to check */
+ for (i = 0; i < RTL_NUMBER_OF(HighPriorOptionsMap); ++i)
+ {
+ if (Masks[i])
+ {
+ /* Retrieve the strings */
+ if (ExtraOptions && HighPriorOptionsMap[i].ExtraOptions)
+ {
+ RtlStringCbCatA(*ExtraOptions,
+ ExtraOptsSize,
+ HighPriorOptionsMap[i].ExtraOptions);
+ }
+ if (HigherPriorityOptions &&
HighPriorOptionsMap[i].HigherPriorOptions)
+ {
+ RtlStringCbCatA(*HigherPriorityOptions,
+ HighPriorOptsSize,
+ HighPriorOptionsMap[i].HigherPriorOptions);
+ }
+ }
+ }
+}
+
+
ARC_STATUS
LoadReactOSSetup(
IN ULONG Argc,
@@ -196,9 +452,8 @@ LoadReactOSSetup(
PSETUP_LOADER_BLOCK SetupBlock;
CHAR BootPath[MAX_PATH];
CHAR FilePath[MAX_PATH];
- CHAR BootOptions2[256];
- PSTR BootOptions;
- PCSTR LoadOptions;
+ CHAR UserBootOptions[256];
+ PCSTR BootOptions;
static PCSTR SourcePaths[] =
{
@@ -287,20 +542,22 @@ LoadReactOSSetup(
TRACE("BootPath: '%s'\n", BootPath);
- /* Retrieve the boot options */
- *BootOptions2 = ANSI_NULL;
- ArgValue = GetArgumentValue(Argc, Argv, "Options");
- if (ArgValue && *ArgValue)
- RtlStringCbCopyA(BootOptions2, sizeof(BootOptions2), ArgValue);
+ /*
+ * Retrieve the boot options. Any options present here will supplement or
+ * override those that will be specified in TXTSETUP.SIF's OsLoadOptions.
+ */
+ BootOptions = GetArgumentValue(Argc, Argv, "Options");
+ if (!BootOptions)
+ BootOptions = "";
- TRACE("BootOptions: '%s'\n", BootOptions2);
+ TRACE("BootOptions: '%s'\n", BootOptions);
/* Check if a RAM disk file was given */
- FileName = (PSTR)NtLdrGetOptionEx(BootOptions2, "RDPATH=",
&FileNameLength);
+ FileName = (PSTR)NtLdrGetOptionEx(BootOptions, "RDPATH=",
&FileNameLength);
if (FileName && (FileNameLength > 7))
{
/* Load the RAM disk */
- Status = RamDiskInitialize(FALSE, BootOptions2, SystemPartition);
+ Status = RamDiskInitialize(FALSE, BootOptions, SystemPartition);
if (Status != ESUCCESS)
{
FileName += 7; FileNameLength -= 7;
@@ -335,34 +592,138 @@ LoadReactOSSetup(
TRACE("BootPath: '%s', SystemPath: '%s'\n", BootPath,
SystemPath);
- /* Get load options - debug and non-debug */
- if (!InfFindFirstLine(InfHandle, "SetupData", "OsLoadOptions",
&InfContext))
- {
- ERR("Failed to find 'SetupData/OsLoadOptions'\n");
- return EINVAL;
- }
+ // UseLocalSif = NtLdrGetOption(BootOptions, "USELOCALSIF");
- if (!InfGetDataField(&InfContext, 1, &LoadOptions))
+ if (NtLdrGetOption(BootOptions, "SIFOPTIONSOVERRIDE"))
{
- ERR("Failed to get load options\n");
- return EINVAL;
- }
+ PCSTR OptionsToRemove[2] = {"SIFOPTIONSOVERRIDE", NULL};
-#if DBG
- /* Get debug load options and use them */
- if (InfFindFirstLine(InfHandle, "SetupData", "DbgOsLoadOptions",
&InfContext))
- {
- PCSTR DbgLoadOptions;
+ /* Do not use any load options from TXTSETUP.SIF, but
+ * use instead those passed from the command line. */
+ RtlStringCbCopyA(UserBootOptions, sizeof(UserBootOptions), BootOptions);
- if (InfGetDataField(&InfContext, 1, &DbgLoadOptions))
- LoadOptions = DbgLoadOptions;
+ /* Remove the private switch from the options */
+ NtLdrUpdateLoadOptions(UserBootOptions,
+ sizeof(UserBootOptions),
+ FALSE,
+ NULL,
+ OptionsToRemove);
+
+ BootOptions = UserBootOptions;
}
+ else // if (!*BootOptions || NtLdrGetOption(BootOptions, "SIFOPTIONSADD"))
+ {
+ PCSTR LoadOptions = NULL;
+ PCSTR DbgLoadOptions = NULL;
+ PSTR ExtraOptions, HigherPriorityOptions;
+ PSTR OptionsToAdd[3];
+ PSTR OptionsToRemove[4];
+
+ /* Load the options from TXTSETUP.SIF */
+ if (InfFindFirstLine(InfHandle, "SetupData", "OsLoadOptions",
&InfContext))
+ {
+ if (!InfGetDataField(&InfContext, 1, &LoadOptions))
+ WARN("Failed to get load options\n");
+ }
+
+#if !DBG
+ /* Non-debug mode: get the debug load options only if /DEBUG was specified
+ * in the Argv command-line options (was e.g. added to the options when
+ * the user selected "Debugging Mode" in the advanced boot menu). */
+ if (NtLdrGetOption(BootOptions, "DEBUG") ||
+ NtLdrGetOption(BootOptions, "DEBUG="))
+ {
+#else
+ /* Debug mode: always get the debug load options */
#endif
+ if (InfFindFirstLine(InfHandle, "SetupData",
"SetupDebugOptions", &InfContext))
+ {
+ if (!InfGetDataField(&InfContext, 1, &DbgLoadOptions))
+ WARN("Failed to get debug load options\n");
+ }
+ /* If none was found, default to enabling debugging */
+ if (!DbgLoadOptions)
+ DbgLoadOptions = "/DEBUG";
+#if !DBG
+ }
+#endif
+
+ /* Initialize the load options with those from TXTSETUP.SIF */
+ *UserBootOptions = ANSI_NULL;
+ if (LoadOptions && *LoadOptions)
+ RtlStringCbCopyA(UserBootOptions, sizeof(UserBootOptions), LoadOptions);
- /* Copy LoadOptions (original string will be freed) */
- BootOptions = FrLdrTempAlloc(strlen(LoadOptions) + 1, TAG_BOOT_OPTIONS);
- ASSERT(BootOptions);
- strcpy(BootOptions, LoadOptions);
+ /* Merge the debug load options if any */
+ if (DbgLoadOptions)
+ {
+ RtlZeroMemory(OptionsToAdd, sizeof(OptionsToAdd));
+ RtlZeroMemory(OptionsToRemove, sizeof(OptionsToRemove));
+
+ /*
+ * Retrieve any option patterns that we should remove from the
+ * SIF load options because they are of higher precedence than
+ * those specified in the debug load options to be added.
+ * Also always remove NODEBUG (even if the debug load options
+ * do not contain explicitly the DEBUG option), since we want
+ * to have debugging enabled if possible.
+ */
+ OptionsToRemove[0] = "/NODEBUG";
+ NtLdrGetHigherPriorityOptions(DbgLoadOptions,
+ &ExtraOptions,
+ &HigherPriorityOptions);
+ OptionsToAdd[1] = (ExtraOptions ? ExtraOptions : "");
+ OptionsToRemove[1] = (HigherPriorityOptions ? HigherPriorityOptions :
"");
+
+ /*
+ * Prepend the debug load options, so that in case it contains
+ * redundant options with respect to the SIF load options, the
+ * former can take precedence over the latter.
+ */
+ OptionsToAdd[0] = (PSTR)DbgLoadOptions;
+ OptionsToRemove[2] = (PSTR)DbgLoadOptions;
+ NtLdrUpdateLoadOptions(UserBootOptions,
+ sizeof(UserBootOptions),
+ FALSE,
+ (PCSTR*)OptionsToAdd,
+ (PCSTR*)OptionsToRemove);
+
+ if (ExtraOptions)
+ FrLdrHeapFree(ExtraOptions, TAG_BOOT_OPTIONS);
+ if (HigherPriorityOptions)
+ FrLdrHeapFree(HigherPriorityOptions, TAG_BOOT_OPTIONS);
+ }
+
+ RtlZeroMemory(OptionsToAdd, sizeof(OptionsToAdd));
+ RtlZeroMemory(OptionsToRemove, sizeof(OptionsToRemove));
+
+ /*
+ * Retrieve any option patterns that we should remove from the
+ * SIF load options because they are of higher precedence than
+ * those specified in the options to be added.
+ */
+ NtLdrGetHigherPriorityOptions(BootOptions,
+ &ExtraOptions,
+ &HigherPriorityOptions);
+ OptionsToAdd[1] = (ExtraOptions ? ExtraOptions : "");
+ OptionsToRemove[0] = (HigherPriorityOptions ? HigherPriorityOptions :
"");
+
+ /* Finally, prepend the user-specified options that
+ * take precedence over those from TXTSETUP.SIF. */
+ OptionsToAdd[0] = (PSTR)BootOptions;
+ OptionsToRemove[1] = (PSTR)BootOptions;
+ NtLdrUpdateLoadOptions(UserBootOptions,
+ sizeof(UserBootOptions),
+ FALSE,
+ (PCSTR*)OptionsToAdd,
+ (PCSTR*)OptionsToRemove);
+
+ if (ExtraOptions)
+ FrLdrHeapFree(ExtraOptions, TAG_BOOT_OPTIONS);
+ if (HigherPriorityOptions)
+ FrLdrHeapFree(HigherPriorityOptions, TAG_BOOT_OPTIONS);
+
+ BootOptions = UserBootOptions;
+ }
TRACE("BootOptions: '%s'\n", BootOptions);
1726
days inactive
1726
days old
0 comments
1 participants
participants (1)
-
Hermès Bélusca-Maïto