https://git.reactos.org/?p=reactos.git;a=commitdiff;h=942b0221e85bdcf5a4bca7865fb4c72ffa105309
commit 942b0221e85bdcf5a4bca7865fb4c72ffa105309 Author: Hermès Bélusca-Maïto <[email protected]> AuthorDate: Mon Mar 20 03:10:14 2023 +0100 Commit: Hermès Bélusca-Maïto <[email protected]> CommitDate: Mon Mar 20 03:10:14 2023 +0100 [NTOS:KDBG] Temporarily HACK-work-around symbols loading by disabling them at BootPhase 0. Of course, now that we **correctly** set the LoadSymbools setting, we attempt loading symbols at BootPhase 0 and everything goes awry! So introduce that hack to fallback to our old behaviour. A proper fix (and removal of the hack) will be done in future commits. --- ntoskrnl/kdbg/kdb_symbols.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/ntoskrnl/kdbg/kdb_symbols.c b/ntoskrnl/kdbg/kdb_symbols.c index 3222c40ce6d..69861f23ed8 100644 --- a/ntoskrnl/kdbg/kdb_symbols.c +++ b/ntoskrnl/kdbg/kdb_symbols.c @@ -344,6 +344,10 @@ BOOLEAN KdbSymInit( _In_ ULONG BootPhase) { +#if 1 // FIXME: This is a workaround HACK!! + static BOOLEAN OrigLoadSymbols = FALSE; +#endif + DPRINT("KdbSymInit() BootPhase=%d\n", BootPhase); if (BootPhase == 0) @@ -398,6 +402,13 @@ KdbSymInit( while (*CommandLine && !isspace(*CommandLine)) ++CommandLine; } + +#if 1 // FIXME: This is a workaround HACK!! +// Save the actual value of LoadSymbols but disable it for BootPhase 0. + OrigLoadSymbols = LoadSymbols; + LoadSymbols = FALSE; + return OrigLoadSymbols; +#endif } else if (BootPhase == 1) { @@ -406,6 +417,11 @@ KdbSymInit( KIRQL OldIrql; PLIST_ENTRY ListEntry; +#if 1 // FIXME: This is a workaround HACK!! +// Now, restore the actual value of LoadSymbols. + LoadSymbols = OrigLoadSymbols; +#endif + /* Do not continue loading symbols if we have less than 96MB of RAM */ if (MmNumberOfPhysicalPages < (96 * 1024 * 1024 / PAGE_SIZE)) LoadSymbols = FALSE;
