Ros-diffs August 2006

ros-diffs@reactos.org

19 participants
447 discussions

[hpoussin] 23800: Add missing includes in .rc file Sort them in alphabetical order
by hpoussin＠svn.reactos.org 30 Aug '06

30 Aug '06

Author: hpoussin Date: Wed Aug 30 14:40:59 2006 New Revision: 23800 URL: http://svn.reactos.org/svn/reactos?rev=23800&view=rev Log: Add missing includes in .rc file Sort them in alphabetical order Modified: trunk/reactos/base/shell/explorer/explorer_intres.rc Modified: trunk/reactos/base/shell/explorer/explorer_intres.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/explorer/explor… ============================================================================== --- trunk/reactos/base/shell/explorer/explorer_intres.rc (original) +++ trunk/reactos/base/shell/explorer/explorer_intres.rc Wed Aug 30 14:40:59 2006 @@ -107,22 +107,26 @@ "CREATEPROCESS_MANIFEST_RESOURCE_ID RT_MANIFEST DISCARDABLE PURE ""explorer.exe.manifest""\r\n" "#endif\r\n" "\r\n" + "#include ""explorer-cz.rc""\r\n" + "#include ""explorer-de.rc""\r\n" "#include ""explorer-en.rc""\r\n" - "#include ""explorer-de.rc""\r\n" "#include ""explorer-es.rc""\r\n" - "#include ""explorer-sv.rc""\r\n" + "#include ""explorer-fr.rc""\r\n" + "#include ""explorer-gr.rc""\r\n" + "#include ""explorer-hu.rc""\r\n" + "#include ""explorer-it.rc""\r\n" + "#include ""explorer-jp.rc""\r\n" + "#include ""explorer-nl.rc""\r\n" + "#include ""explorer-no.rc""\r\n" "#include ""explorer-pl.rc""\r\n" "#include ""explorer-pt.rc""\r\n" - "#include ""explorer-cz.rc""\r\n" "#include ""explorer-ro.rc""\r\n" "#include ""explorer-ru.rc""\r\n" - "#include ""explorer-jp.rc""\r\n" - "#include ""explorer-hu.rc""\r\n" + "#include ""explorer-sv.rc""\r\n" "#include ""explorer-uk.rc""\r\n" - "#include ""explorer-nl.rc""\r\n" - "#include ""explorer-it.rc""\r\n" "\r\n" "#ifndef __WINDRES__\r\n" + "#include ""explorer-bn.rc""\r\n" "#include ""explorer-cn.rc""\r\n" "#endif\r\n" "\0"

1 0

[greatlrd] 23799: Implement RtlRandomEx by copy RtlRandom code. ms have two different implement, One random is faster that other, (in some doc I read (maybe from osr) some year ago), the RtlRandomEx is not document in the free ddk/sdk, but it is include in ddk/ifs kit, according the doc.
by greatlrd＠svn.reactos.org 30 Aug '06

30 Aug '06

Author: greatlrd Date: Wed Aug 30 14:13:01 2006 New Revision: 23799 URL: http://svn.reactos.org/svn/reactos?rev=23799&view=rev Log: Implement RtlRandomEx by copy RtlRandom code. ms have two different implement, One random is faster that other, (in some doc I read (maybe from osr) some year ago), the RtlRandomEx is not document in the free ddk/sdk, but it is include in ddk/ifs kit, according the doc. Modified: trunk/reactos/lib/rtl/random.c Modified: trunk/reactos/lib/rtl/random.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/random.c?rev=23799… ============================================================================== --- trunk/reactos/lib/rtl/random.c (original) +++ trunk/reactos/lib/rtl/random.c Wed Aug 30 14:13:01 2006 @@ -83,16 +83,26 @@ } /* -* @unimplemented +* @implemented */ ULONG NTAPI -RtlRandomEx( - PULONG Seed +RtlRandomEx( IN OUT PULONG Seed ) { - UNIMPLEMENTED; - return 0; + ULONG Rand; + int Pos; + ULONG Result; + + PAGED_CODE_RTL(); + + Rand = (*Seed * 0x7fffffed + 0x7fffffc3) % 0x7fffffff; + *Seed = (Rand * 0x7fffffed + 0x7fffffc3) % 0x7fffffff; + Pos = *Seed & 0x7f; + Result = SavedValue[Pos]; + SavedValue[Pos] = Rand; + + return Result; }

1 0

[janderwald] 23798: * remove useless cast
by janderwald＠svn.reactos.org 30 Aug '06

30 Aug '06

Author: janderwald Date: Wed Aug 30 13:45:08 2006 New Revision: 23798 URL: http://svn.reactos.org/svn/reactos?rev=23798&view=rev Log: * remove useless cast Modified: trunk/reactos/dll/win32/gdi32/objects/font.c Modified: trunk/reactos/dll/win32/gdi32/objects/font.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/gdi32/objects/fo… ============================================================================== --- trunk/reactos/dll/win32/gdi32/objects/font.c (original) +++ trunk/reactos/dll/win32/gdi32/objects/font.c Wed Aug 30 13:45:08 2006 @@ -252,8 +252,8 @@ if (Unicode) { Ret = ((FONTENUMPROCW) EnumProc)( - (LPLOGFONTW)&Info[i].EnumLogFontEx, - (LPTEXTMETRICW)&Info[i].NewTextMetricEx, + &Info[i].EnumLogFontEx, + &Info[i].NewTextMetricEx, Info[i].FontType, lParam); } else @@ -268,8 +268,8 @@ NewTextMetricExW2A(&NewTextMetricExA, &Info[i].NewTextMetricEx); Ret = ((FONTENUMPROCA) EnumProc)( - (LPLOGFONTA)&EnumLogFontExA, - (LPTEXTMETRICA)&NewTextMetricExA, + &EnumLogFontExA, + &NewTextMetricExA, Info[i].FontType, lParam); } }

1 0

[janderwald] 23797: - fix definition of FONTENUMPROC
by janderwald＠svn.reactos.org 30 Aug '06

30 Aug '06

Author: janderwald Date: Wed Aug 30 13:17:47 2006 New Revision: 23797 URL: http://svn.reactos.org/svn/reactos?rev=23797&view=rev Log: - fix definition of FONTENUMPROC Modified: trunk/reactos/include/psdk/wingdi.h Modified: trunk/reactos/include/psdk/wingdi.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/wingdi.h?rev=… ============================================================================== --- trunk/reactos/include/psdk/wingdi.h (original) +++ trunk/reactos/include/psdk/wingdi.h Wed Aug 30 13:17:47 2006 @@ -2525,8 +2525,8 @@ typedef BOOL (CALLBACK *ABORTPROC)(HDC,int); typedef int (CALLBACK *MFENUMPROC)(HDC,HANDLETABLE*,METARECORD*,int,LPARAM); typedef int (CALLBACK *ENHMFENUMPROC)(HDC,HANDLETABLE*,ENHMETARECORD*,int,LPARAM); -typedef int (CALLBACK *OLDFONTENUMPROCA)(const LOGFONTA*,const TEXTMETRICA*,DWORD,LPARAM); -typedef int (CALLBACK *OLDFONTENUMPROCW)(const LOGFONTW*,const TEXTMETRICW*,DWORD,LPARAM); +typedef int (CALLBACK *OLDFONTENUMPROCA)(ENUMLOGFONTEXA*,NEWTEXTMETRICEXA*,DWORD,LPARAM); +typedef int (CALLBACK *OLDFONTENUMPROCW)(ENUMLOGFONTEXW*,NEWTEXTMETRICEXW*,DWORD,LPARAM); typedef OLDFONTENUMPROCA FONTENUMPROCA; typedef OLDFONTENUMPROCW FONTENUMPROCW; typedef int (CALLBACK *ICMENUMPROCA)(LPSTR,LPARAM);

1 0

[ion] 23796: - Update kernel fun.
by ion＠svn.reactos.org 30 Aug '06

30 Aug '06

Author: ion Date: Wed Aug 30 10:56:06 2006 New Revision: 23796 URL: http://svn.reactos.org/svn/reactos?rev=23796&view=rev Log: - Update kernel fun. Modified: trunk/reactos/ntoskrnl/KrnlFun.c Modified: trunk/reactos/ntoskrnl/KrnlFun.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/KrnlFun.c?rev=237… ============================================================================== --- trunk/reactos/ntoskrnl/KrnlFun.c (original) +++ trunk/reactos/ntoskrnl/KrnlFun.c Wed Aug 30 10:56:06 2006 @@ -25,14 +25,14 @@ // - Use Object Type Mutex/Lock. // // Ke: -// - Sanitize some context fields during conversions // - Add PSEH handler when an exception occurs in an exception (KiCopyExceptionRecord). +// - Get rid of KiRosPrintAddress and use KiDumpParameterImages instead. // - Forward exceptions to user-mode debugger. +// - Sanitize some context fields during conversions. // - Implement stack fault and segment fault handlers. // - Implement kernel-mode GPF handler, possibly fixing below: // - Figure out why ES/DS gets messed up in VMWare, when doing KiServiceExit only, // and only when called from user-mode, and returning to user-mode. -// - Figure out what the DEBUGEIP hack is for and how it can be moved away. // - Add DR macro/save and VM macro/save. // - Implement KiCallbackReturn, KiGetTickCount, KiRaiseAssertion. //

1 0

[ion] 23795: - Implement RtlCaptureStackBackTrace and RtlWalkFrameChain and add definitions to ntifs.h - Get rid of KeRosGetStackFrames and replace usage by RtlWalkFrameChain or RtlCaptureStackBackTrace, depending. - Remove this entry from Kernel Fun.
by ion＠svn.reactos.org 30 Aug '06

30 Aug '06

Author: ion Date: Wed Aug 30 10:52:10 2006 New Revision: 23795 URL: http://svn.reactos.org/svn/reactos?rev=23795&view=rev Log: - Implement RtlCaptureStackBackTrace and RtlWalkFrameChain and add definitions to ntifs.h - Get rid of KeRosGetStackFrames and replace usage by RtlWalkFrameChain or RtlCaptureStackBackTrace, depending. - Remove this entry from Kernel Fun. Modified: trunk/reactos/dll/ntdll/rtl/libsupp.c trunk/reactos/include/ddk/ntifs.h trunk/reactos/lib/rtl/exception.c trunk/reactos/lib/rtl/rtlp.h trunk/reactos/ntoskrnl/KrnlFun.c trunk/reactos/ntoskrnl/ke/bug.c trunk/reactos/ntoskrnl/mm/ppool.c trunk/reactos/ntoskrnl/ntoskrnl.def trunk/reactos/ntoskrnl/rtl/libsupp.c Modified: trunk/reactos/dll/ntdll/rtl/libsupp.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/ntdll/rtl/libsupp.c?re… ============================================================================== --- trunk/reactos/dll/ntdll/rtl/libsupp.c (original) +++ trunk/reactos/dll/ntdll/rtl/libsupp.c Wed Aug 30 10:52:10 2006 @@ -171,6 +171,18 @@ IN ULONG Size) { /* Exception logging is not done in user-mode */ +} + +BOOLEAN +NTAPI +RtlpCaptureStackLimits(IN ULONG_PTR Ebp, + IN ULONG_PTR *StackBegin, + IN ULONG_PTR *StackEnd) +{ + /* FIXME: Verify */ + *StackBegin = (ULONG_PTR)NtCurrentTeb()->Tib.StackLimit; + *StackEnd = (ULONG_PTR)NtCurrentTeb()->Tib.StackBase; + return TRUE; } /* RTL Atom Tables ************************************************************/ Modified: trunk/reactos/include/ddk/ntifs.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ddk/ntifs.h?rev=23… ============================================================================== --- trunk/reactos/include/ddk/ntifs.h (original) +++ trunk/reactos/include/ddk/ntifs.h Wed Aug 30 10:52:10 2006 @@ -3676,6 +3676,16 @@ ); NTSYSAPI +USHORT +NTAPI +RtlCaptureStackBackTrace ( + IN ULONG FramesToSkip, + IN ULONG FramesToCapture, + OUT PVOID *BackTrace, + OUT PULONG BackTraceHash OPTIONAL +); + +NTSYSAPI NTSTATUS NTAPI RtlCompressBuffer ( Modified: trunk/reactos/lib/rtl/exception.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/exception.c?rev=23… ============================================================================== --- trunk/reactos/lib/rtl/exception.c (original) +++ trunk/reactos/lib/rtl/exception.c Wed Aug 30 10:52:10 2006 @@ -110,8 +110,92 @@ } /* -* @unimplemented -*/ + * @implemented + */ +ULONG +NTAPI +RtlWalkFrameChain(OUT PVOID *Callers, + IN ULONG Count, + IN ULONG Flags) +{ + PULONG Stack, NewStack; + ULONG Eip; + ULONG_PTR StackBegin, StackEnd; + BOOLEAN Result, StopSearch = FALSE; + ULONG i = 0; + + /* Get current EBP */ +#if defined __GNUC__ + __asm__("mov %%ebp, %0" : "=r" (Stack) : ); +#elif defined(_MSC_VER) + __asm mov Stack, ebp +#endif + + /* Set it as the stack begin limit as well */ + StackBegin = (ULONG_PTR)Stack; + + /* Check if we're called for non-logging mode */ + if (!Flags) + { + /* Get the actual safe limits */ + Result = RtlpCaptureStackLimits((ULONG_PTR)Stack, + &StackBegin, + &StackEnd); + if (!Result) return 0; + } + + /* Loop the frames */ + for (i = 0; i < Count; i++) + { + /* Check if we're past the stack */ + if ((ULONG_PTR)Stack >= StackEnd) break; + + /* Check if this is the first entry */ +#if 0 + if (!i) + { + if ((ULONG_PTR)Stack != StackBegin) break; + } + else + { + if ((ULONG_PTR)Stack == StackBegin) break; + } +#endif + + /* Make sure there's enough frames */ + if ((StackEnd - (ULONG_PTR)Stack) < (2 * sizeof(ULONG_PTR))) break; + + /* Get new stack and EIP */ + NewStack = (PULONG)Stack[0]; + Eip = Stack[1]; + + /* Check if the new pointer is above the oldone and past the end */ + if (!((Stack < NewStack) && ((ULONG_PTR)NewStack < StackEnd))) + { + /* Stop searching after this entry */ + StopSearch = TRUE; + } + + /* Also make sure that the EIP isn't a stack address */ + if ((StackBegin < Eip) && (Eip < StackEnd)) break; + + /* Save this frame */ + Callers[i] = (PVOID)Eip; + + /* Check if we should continue */ + if (StopSearch) break; + + /* Move to the next stack */ + Stack = NewStack; + } + + /* Return frames parsed */ + return i; +} + +/* + * @implemented + */ USHORT NTAPI RtlCaptureStackBackTrace(IN ULONG FramesToSkip, @@ -119,21 +203,39 @@ OUT PVOID *BackTrace, OUT PULONG BackTraceHash OPTIONAL) { - UNIMPLEMENTED; - return 0; -} - -/* -* @unimplemented -*/ -ULONG -NTAPI -RtlWalkFrameChain(OUT PVOID *Callers, - IN ULONG Count, - IN ULONG Flags) -{ - UNIMPLEMENTED; - return 0; + PVOID Frames[2 * 64]; + ULONG FrameCount; + ULONG Hash = 0, i; + + /* Skip a frame for the caller */ + FramesToSkip++; + + /* Don't go past the limit */ + if ((FramesToCapture + FramesToSkip) >= 128) return 0; + + /* Do the back trace */ + FrameCount = RtlWalkFrameChain(Frames, FramesToCapture + FramesToSkip, 0); + + /* Make sure we're not skipping all of them */ + if (FrameCount <= FramesToSkip) return 0; + + /* Loop all the frames */ + for (i = 0; i < FramesToCapture; i++) + { + /* Don't go past the limit */ + if ((FramesToSkip + i) >= FrameCount) break; + + /* Save this entry and hash it */ + BackTrace[i] = Frames[FramesToSkip + i]; + Hash += PtrToUlong(BackTrace[i]); + } + + /* Write the hash */ + if (BackTraceHash) *BackTraceHash = Hash; + + /* Clear the other entries and return count */ + RtlFillMemoryUlong(Frames, 128, 0); + return (USHORT)i; } /* Modified: trunk/reactos/lib/rtl/rtlp.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/rtlp.h?rev=23795&r… ============================================================================== --- trunk/reactos/lib/rtl/rtlp.h (original) +++ trunk/reactos/lib/rtl/rtlp.h Wed Aug 30 10:52:10 2006 @@ -32,6 +32,14 @@ KPROCESSOR_MODE NTAPI RtlpGetMode(VOID); + +BOOLEAN +NTAPI +RtlpCaptureStackLimits( + IN ULONG_PTR Ebp, + IN ULONG_PTR *StackBegin, + IN ULONG_PTR *StackEnd +); NTSTATUS NTAPI Modified: trunk/reactos/ntoskrnl/KrnlFun.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/KrnlFun.c?rev=237… ============================================================================== --- trunk/reactos/ntoskrnl/KrnlFun.c (original) +++ trunk/reactos/ntoskrnl/KrnlFun.c Wed Aug 30 10:52:10 2006 @@ -25,7 +25,6 @@ // - Use Object Type Mutex/Lock. // // Ke: -// - Clean up exp.c (remove all stack functions and use RtlWalkFrameChain/RtlCaptureStackBacktrace) // - Sanitize some context fields during conversions // - Add PSEH handler when an exception occurs in an exception (KiCopyExceptionRecord). // - Forward exceptions to user-mode debugger. Modified: trunk/reactos/ntoskrnl/ke/bug.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/bug.c?rev=2379… ============================================================================== --- trunk/reactos/ntoskrnl/ke/bug.c (original) +++ trunk/reactos/ntoskrnl/ke/bug.c Wed Aug 30 10:52:10 2006 @@ -83,83 +83,22 @@ return(FALSE); } -ULONG -NTAPI -KeRosGetStackFrames(PULONG Frames, - ULONG FrameCount) -{ - ULONG Count = 0; - PULONG StackBase, StackEnd, Frame; - MEMORY_BASIC_INFORMATION mbi; - ULONG ResultLength = sizeof(mbi); - NTSTATUS Status; - - _SEH_TRY - { -#if defined __GNUC__ - __asm__("mov %%ebp, %0" : "=r" (Frame) : ); -#elif defined(_MSC_VER) - __asm mov [Frame], ebp -#endif - - Status = MiQueryVirtualMemory ( - (HANDLE)-1, - Frame, - MemoryBasicInformation, - &mbi, - sizeof(mbi), - &ResultLength ); - if ( !NT_SUCCESS(Status) ) - { - DPRINT1("Can't get stack frames: MiQueryVirtualMemory() failed: %x\n", Status ); - return 0; - } - - StackBase = Frame; - StackEnd = (PULONG)((ULONG_PTR)mbi.BaseAddress + mbi.RegionSize); - - while ( Count < FrameCount && Frame >= StackBase && Frame < StackEnd ) - { - Frames[Count++] = Frame[1]; - StackBase = Frame; - Frame = (PULONG)Frame[0]; - } - } - _SEH_HANDLE - { - } - _SEH_END; - return Count; -} - -VOID -NTAPI -KeDumpStackFrames(PULONG Frame) -{ - /* Just call the extended version */ - KeRosDumpStackFrames(Frame, 0); -} - VOID NTAPI KeRosDumpStackFrames(IN PULONG Frame OPTIONAL, IN ULONG FrameCount OPTIONAL) { ULONG Frames[32]; - ULONG Count, i, Addr; - - /* Don't let anyone ask more then 32 frames */ - if (FrameCount > 32) return; + ULONG i, Addr; /* If the caller didn't ask, assume 32 frames */ if (!FrameCount) FrameCount = 32; - /* Get the current frames, and make sure caller didn't ask too many */ - Count = KeRosGetStackFrames(Frames, FrameCount); // <= should be replaced with Rtl - if (FrameCount > Count) FrameCount = Count; + /* Get the current frames */ + FrameCount = RtlCaptureStackBackTrace(2, FrameCount, (PVOID*)Frames, NULL); /* Now loop them (skip the two. One for the dumper, one for the caller) */ - for (i = 2; i < FrameCount; i++) + for (i = 0; i < FrameCount; i++) { /* Get the EIP */ Addr = Frames[i]; @@ -177,9 +116,6 @@ /* Print it out */ if (!KeRosPrintAddress((PVOID)Addr)) DbgPrint("<%X>", Addr); - - /* Break out of invalid addresses */ - if (Addr == 0 || Addr == 0xDEADBEEF) break; /* Go to the next frame */ DbgPrint("\n"); Modified: trunk/reactos/ntoskrnl/mm/ppool.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ppool.c?rev=23… ============================================================================== --- trunk/reactos/ntoskrnl/mm/ppool.c (original) +++ trunk/reactos/ntoskrnl/mm/ppool.c Wed Aug 30 10:52:10 2006 @@ -34,7 +34,7 @@ #define R_PRINT_ADDRESS(addr) KeRosPrintAddress(addr) #define R_PANIC() KeBugCheck(0) #define R_DEBUG DbgPrint -#define R_GET_STACK_FRAMES(ptr,cnt) KeRosGetStackFrames(ptr,cnt) +#define R_GET_STACK_FRAMES(ptr,cnt) RtlWalkFrameChain((PVOID*)ptr,cnt, 0) #include "rpoolmgr.h" Modified: trunk/reactos/ntoskrnl/ntoskrnl.def URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ntoskrnl.def?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/ntoskrnl.def (original) +++ trunk/reactos/ntoskrnl/ntoskrnl.def Wed Aug 30 10:52:10 2006 @@ -977,7 +977,6 @@ READ_REGISTER_BUFFER_USHORT@12 KiRosPrintAddress@4 KeRosDumpStackFrames@8 -KeRosGetStackFrames@8 RtlAbsoluteToSelfRelativeSD@12 RtlAddAccessAllowedAce@16 RtlAddAce@20 Modified: trunk/reactos/ntoskrnl/rtl/libsupp.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/rtl/libsupp.c?rev… ============================================================================== --- trunk/reactos/ntoskrnl/rtl/libsupp.c (original) +++ trunk/reactos/ntoskrnl/rtl/libsupp.c Wed Aug 30 10:52:10 2006 @@ -186,6 +186,34 @@ /* Not in DPC stack */ return FALSE; +} + +BOOLEAN +NTAPI +RtlpCaptureStackLimits(IN ULONG_PTR Ebp, + IN ULONG_PTR *StackBegin, + IN ULONG_PTR *StackEnd) +{ + PKTHREAD Thread = KeGetCurrentThread(); + + /* FIXME: Super native implementation */ + + /* FIXME: ROS HACK */ + if (!Thread) return FALSE; + + /* Start with defaults */ + *StackBegin = Thread->StackLimit; + *StackEnd = (ULONG_PTR)Thread->StackBase; + + /* Check if we seem to be on the DPC stack */ + if ((*StackBegin > Ebp) || (Ebp > *StackEnd)) + { + /* FIXME: TODO */ + ASSERT(FALSE); + } + + /* Return success */ + return TRUE; } /* RTL Atom Tables ************************************************************/

1 0

[ion] 23794: - Add about two dozen new bugcheck codes and strings. - Major cleanup of exp.c to move out deprecated and old code. - Use KdDebuggerNotPresent to determine if KDBG is there or not. KdDebuggerEnable is now set whenever booting with debugging -- even if only serial port or screen debugging. - Complete cleanup and major rewrite/improvement of bugcheck code: - Moved stack dump routines to bug.c and cleaned them up. Next step is to remove most of them and implement them properly as part
by ion＠svn.reactos.org 30 Aug '06

30 Aug '06

Author: ion Date: Wed Aug 30 09:22:56 2006 New Revision: 23794 URL: http://svn.reactos.org/svn/reactos?rev=23794&view=rev Log: - Add about two dozen new bugcheck codes and strings. - Major cleanup of exp.c to move out deprecated and old code. - Use KdDebuggerNotPresent to determine if KDBG is there or not. KdDebuggerEnable is now set whenever booting with debugging -- even if only serial port or screen debugging. - Complete cleanup and major rewrite/improvement of bugcheck code: - Moved stack dump routines to bug.c and cleaned them up. Next step is to remove most of them and implement them properly as part of RtlCaptureStackBackTrace and RtlWalkFrameChain. - Use InbvDisplayString for everything now. BSODs will only truly show on screen. (however, part of the KeBugCheckWithTf improvements, a special "if debugger enabled" BSOD will also be displayed for debugging purposes). - Do checksum calculation in bugcheck callbacks. - Detect and display which drivers belong to which EIP. - Implemented code to select the proper verbose BSOD message depending on the bug check code. - Also implemented code to detect a variety of "known" bug check codes and to recover more data on them, and/or modify them accordingly. Sadly ROS doesn't yet make use of most of these codes. - Factored out and split some code into individual routines. - Look and functionality is now identical to Windows XP. - Stack trace will only be shown if KDBG isn't connected. If KDBG is connected you can do "bt" anyway (which is more powerful). Modified: trunk/reactos/ntoskrnl/deprecated/irq.c trunk/reactos/ntoskrnl/include/internal/ke.h trunk/reactos/ntoskrnl/kd/kdinit.c trunk/reactos/ntoskrnl/kd/kdmain.c trunk/reactos/ntoskrnl/ke/bug.c trunk/reactos/ntoskrnl/ke/i386/exp.c trunk/reactos/ntoskrnl/ke/i386/v86vdm.c trunk/reactos/ntoskrnl/ntoskrnl.mc Modified: trunk/reactos/ntoskrnl/deprecated/irq.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/deprecated/irq.c?… ============================================================================== --- trunk/reactos/ntoskrnl/deprecated/irq.c (original) +++ trunk/reactos/ntoskrnl/deprecated/irq.c Wed Aug 30 09:22:56 2006 @@ -142,7 +142,6 @@ #endif #define TAG_ISR_LOCK TAG('I', 'S', 'R', 'L') -extern IDT_DESCRIPTOR KiIdt[256]; /* FUNCTIONS ****************************************************************/ @@ -162,8 +161,8 @@ */ for (i=0;i<NR_IRQS;i++) { - KiIdt[IRQ_BASE+i].a=(irq_handler[i]&0xffff)+(KGDT_R0_CODE<<16); - KiIdt[IRQ_BASE+i].b=(irq_handler[i]&0xffff0000)+PRESENT+ + ((IDT_DESCRIPTOR*)&KiIdt[IRQ_BASE+i])->a=(irq_handler[i]&0xffff)+(KGDT_R0_CODE<<16); + ((IDT_DESCRIPTOR*)&KiIdt[IRQ_BASE+i])->b=(irq_handler[i]&0xffff0000)+PRESENT+ I486_INTERRUPT_GATE; #ifdef CONFIG_SMP for (j = 0; j < MAXIMUM_PROCESSORS; j++) Modified: trunk/reactos/ntoskrnl/include/internal/ke.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/include/internal/… ============================================================================== --- trunk/reactos/ntoskrnl/include/internal/ke.h (original) +++ trunk/reactos/ntoskrnl/include/internal/ke.h Wed Aug 30 09:22:56 2006 @@ -75,6 +75,7 @@ extern ULONG KeI386EFlagsAndMaskV86; extern ULONG KeI386EFlagsOrMaskV86; extern BOOLEAN KeI386VirtualIntExtensions; +extern KIDTENTRY KiIdt[]; /* MACROS *************************************************************************/ @@ -145,6 +146,8 @@ #define IOPM_OFFSET FIELD_OFFSET(KTSS, IoMaps[0].IoMap) +#define SIZE_OF_FX_REGISTERS 32 + /* INTERNAL KERNEL FUNCTIONS ************************************************/ /* threadsch.c ********************************************************************/ @@ -679,14 +682,6 @@ VOID STDCALL -KiDumpTrapFrame( - PKTRAP_FRAME Tf, - ULONG ExceptionNr, - ULONG cr2 -); - -VOID -STDCALL KeFlushCurrentTb(VOID); VOID @@ -757,6 +752,12 @@ VOID ); +VOID +NTAPI +KiFlushNPXState( + IN FLOATING_SAVE_AREA *SaveArea +); + #include "ke_x.h" #endif /* __NTOSKRNL_INCLUDE_INTERNAL_KE_H */ Modified: trunk/reactos/ntoskrnl/kd/kdinit.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd/kdinit.c?rev=2… ============================================================================== --- trunk/reactos/ntoskrnl/kd/kdinit.c (original) +++ trunk/reactos/ntoskrnl/kd/kdinit.c Wed Aug 30 09:22:56 2006 @@ -74,6 +74,7 @@ #ifdef KDBG /* Get the KDBG Settings and enable it */ KdDebuggerEnabled = TRUE; + KdDebuggerNotPresent = FALSE; KdpDebugMode.Gdb = TRUE; KdbpGetCommandLineSettings((PCHAR)LoaderBlock->CommandLine); #endif @@ -183,6 +184,7 @@ p2 += 10; p2 = KdpGetDebugMode(p2); p2 = KdpGetWrapperDebugMode(p2, LoaderBlock); + KdDebuggerEnabled = TRUE; } /* Check for early breakpoint */ else if (!_strnicmp(p2, "BREAK", 5)) Modified: trunk/reactos/ntoskrnl/kd/kdmain.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd/kdmain.c?rev=2… ============================================================================== --- trunk/reactos/ntoskrnl/kd/kdmain.c (original) +++ trunk/reactos/ntoskrnl/kd/kdmain.c Wed Aug 30 09:22:56 2006 @@ -105,8 +105,8 @@ BOOLEAN FirstChance, BOOLEAN Gdb) { - /* Get out of here if the Debugger isn't enabled */ - if (!KdDebuggerEnabled) return kdHandleException; + /* Get out of here if the Debugger isn't connected */ + if (KdDebuggerNotPresent) return kdHandleException; /* FIXME: * Right now, the GDB wrapper seems to handle exceptions differntly Modified: trunk/reactos/ntoskrnl/ke/bug.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/bug.c?rev=2379… ============================================================================== --- trunk/reactos/ntoskrnl/ke/bug.c (original) +++ trunk/reactos/ntoskrnl/ke/bug.c Wed Aug 30 09:22:56 2006 @@ -1,15 +1,12 @@ /* - * COPYRIGHT: See COPYING in the top level directory - * PROJECT: ReactOS kernel + * PROJECT: ReactOS Kernel + * LICENSE: GPL - See COPYING in the top level directory * FILE: ntoskrnl/ke/bug.c - * PURPOSE: Graceful system shutdown if a bug is detected - * - * PROGRAMMERS: Alex Ionescu - Rewrote Bugcheck Routines and implemented Reason Callbacks. - * David Welch (welch(a)cwcom.net) - * Phillip Susi + * PURPOSE: Bugcheck Support + * PROGRAMMERS: Alex Ionescu (alex.ionescu(a)reactos.org) */ -/* INCLUDES *****************************************************************/ +/* INCLUDES ******************************************************************/ #include <ntoskrnl.h> #define NDEBUG @@ -29,15 +26,168 @@ extern FAST_MUTEX KernelAddressSpaceLock; -/* GLOBALS ******************************************************************/ - -static LIST_ENTRY BugcheckCallbackListHead = {NULL,NULL}; -static LIST_ENTRY BugcheckReasonCallbackListHead = {NULL,NULL}; -static ULONG InBugCheck; -static PRTL_MESSAGE_RESOURCE_DATA KiBugCodeMessages; -static ULONG KeBugCheckCount = 1; - -/* FUNCTIONS *****************************************************************/ +/* GLOBALS *******************************************************************/ + +LIST_ENTRY BugcheckCallbackListHead = {NULL,NULL}; +LIST_ENTRY BugcheckReasonCallbackListHead = {NULL,NULL}; +ULONG KeBugCheckActive, KeBugCheckOwner; +LONG KeBugCheckOwnerRecursionCount; +PRTL_MESSAGE_RESOURCE_DATA KiBugCodeMessages; +ULONG KeBugCheckCount = 1; +ULONG KiHardwareTrigger; +PUNICODE_STRING KiBugCheckDriver; +ULONG_PTR KiBugCheckData[5]; + +typedef PCHAR +(NTAPI *PKE_BUGCHECK_UNICODE_TO_ANSI)( + IN PUNICODE_STRING Unicode, + IN PCHAR Ansi, + IN ULONG Length +); + +/* PRIVATE FUNCTIONS *********************************************************/ + +BOOLEAN +NTAPI +KiRosPrintAddress(PVOID address) +{ + PLIST_ENTRY current_entry; + PLDR_DATA_TABLE_ENTRY current; + extern LIST_ENTRY ModuleListHead; + ULONG_PTR RelativeAddress; + ULONG i = 0; + + do + { + current_entry = ModuleListHead.Flink; + + while (current_entry != &ModuleListHead) + { + current = CONTAINING_RECORD(current_entry, + LDR_DATA_TABLE_ENTRY, + InLoadOrderLinks); + + if (address >= (PVOID)current->DllBase && + address < (PVOID)((ULONG_PTR)current->DllBase + + current->SizeOfImage)) + { + RelativeAddress = (ULONG_PTR)address - + (ULONG_PTR)current->DllBase; + DbgPrint("<%wZ: %x>", &current->FullDllName, RelativeAddress); + return(TRUE); + } + current_entry = current_entry->Flink; + } + } while(++i <= 1); + + return(FALSE); +} + +ULONG +NTAPI +KeRosGetStackFrames(PULONG Frames, + ULONG FrameCount) +{ + ULONG Count = 0; + PULONG StackBase, StackEnd, Frame; + MEMORY_BASIC_INFORMATION mbi; + ULONG ResultLength = sizeof(mbi); + NTSTATUS Status; + + _SEH_TRY + { +#if defined __GNUC__ + __asm__("mov %%ebp, %0" : "=r" (Frame) : ); +#elif defined(_MSC_VER) + __asm mov [Frame], ebp +#endif + + Status = MiQueryVirtualMemory ( + (HANDLE)-1, + Frame, + MemoryBasicInformation, + &mbi, + sizeof(mbi), + &ResultLength ); + if ( !NT_SUCCESS(Status) ) + { + DPRINT1("Can't get stack frames: MiQueryVirtualMemory() failed: %x\n", Status ); + return 0; + } + + StackBase = Frame; + StackEnd = (PULONG)((ULONG_PTR)mbi.BaseAddress + mbi.RegionSize); + + while ( Count < FrameCount && Frame >= StackBase && Frame < StackEnd ) + { + Frames[Count++] = Frame[1]; + StackBase = Frame; + Frame = (PULONG)Frame[0]; + } + } + _SEH_HANDLE + { + } + _SEH_END; + return Count; +} + +VOID +NTAPI +KeDumpStackFrames(PULONG Frame) +{ + /* Just call the extended version */ + KeRosDumpStackFrames(Frame, 0); +} + +VOID +NTAPI +KeRosDumpStackFrames(IN PULONG Frame OPTIONAL, + IN ULONG FrameCount OPTIONAL) +{ + ULONG Frames[32]; + ULONG Count, i, Addr; + + /* Don't let anyone ask more then 32 frames */ + if (FrameCount > 32) return; + + /* If the caller didn't ask, assume 32 frames */ + if (!FrameCount) FrameCount = 32; + + /* Get the current frames, and make sure caller didn't ask too many */ + Count = KeRosGetStackFrames(Frames, FrameCount); // <= should be replaced with Rtl + if (FrameCount > Count) FrameCount = Count; + + /* Now loop them (skip the two. One for the dumper, one for the caller) */ + for (i = 2; i < FrameCount; i++) + { + /* Get the EIP */ + Addr = Frames[i]; + + /* If we had a custom frame, make sure we've reached it first */ + if ((Frame) && (Frame[1] == Addr)) + { + Frame = NULL; + } + else if (Frame) + { + /* Skip this entry */ + continue; + } + + /* Print it out */ + if (!KeRosPrintAddress((PVOID)Addr)) DbgPrint("<%X>", Addr); + + /* Break out of invalid addresses */ + if (Addr == 0 || Addr == 0xDEADBEEF) break; + + /* Go to the next frame */ + DbgPrint("\n"); + } + + /* Finish the output */ + DbgPrint("\n"); +} VOID INIT_FUNCTION @@ -52,7 +202,6 @@ /* Initialize Callbadk Listhead and State */ InitializeListHead(&BugcheckCallbackListHead); InitializeListHead(&BugcheckReasonCallbackListHead); - InBugCheck = 0; /* Cache the Bugcheck Message Strings. Prepare the Lookup Data */ ResourceInfo.Type = 11; @@ -66,31 +215,795 @@ &ResourceDataEntry); /* Make sure it worked */ - if (NT_SUCCESS(Status)) { - - DPRINT("Found Bugcheck Resource Data!\n"); - + if (NT_SUCCESS(Status)) + { /* Now actually get a pointer to it */ Status = LdrAccessResource((PVOID)KERNEL_BASE, ResourceDataEntry, (PVOID*)&BugCheckData, NULL); - - /* Make sure it worked */ - if (NT_SUCCESS(Status)) { - - DPRINT("Got Pointer to Bugcheck Resource Data!\n"); - KiBugCodeMessages = BugCheckData; - } - } -} + if (NT_SUCCESS(Status)) KiBugCodeMessages = BugCheckData; + } +} + +VOID +NTAPI +KeGetBugMessageText(IN ULONG BugCheckCode, + OUT PANSI_STRING OutputString OPTIONAL) +{ + ULONG i; + ULONG IdOffset; + ULONG_PTR MessageEntry; + PCHAR BugCode; + + /* Find the message. This code is based on RtlFindMesssage */ + for (i = 0; i < KiBugCodeMessages->NumberOfBlocks; i++) + { + /* Check if the ID Matches */ + if ((BugCheckCode >= KiBugCodeMessages->Blocks[i].LowId) && + (BugCheckCode <= KiBugCodeMessages->Blocks[i].HighId)) + { + /* Get Offset to Entry */ + MessageEntry = KiBugCodeMessages->Blocks[i].OffsetToEntries + + (ULONG_PTR)KiBugCodeMessages; + IdOffset = BugCheckCode - KiBugCodeMessages->Blocks[i].LowId; + + /* Get offset to ID */ + for (i = 0; i < IdOffset; i++) + { + /* Advance in the Entries */ + MessageEntry += ((PRTL_MESSAGE_RESOURCE_ENTRY)MessageEntry)-> + Length; + } + + /* Get the final Code */ + BugCode = ((PRTL_MESSAGE_RESOURCE_ENTRY)MessageEntry)->Text; + i = strlen(BugCode); + + /* Return it in the OutputString */ + if (OutputString) + { + OutputString->Buffer = BugCode; + OutputString->Length = i + 1; + OutputString->MaximumLength = i + 1; + } + else + { + /* Direct Output to Screen */ + InbvDisplayString(BugCode); + InbvDisplayString("\r"); + break; + } + } + } +} + +VOID +NTAPI +KiDoBugCheckCallbacks(VOID) +{ + PKBUGCHECK_CALLBACK_RECORD CurrentRecord; + PLIST_ENTRY ListHead, NextEntry, LastEntry; + ULONG_PTR Checksum; + + /* First make sure that the list is Initialized... it might not be */ + ListHead = &BugcheckCallbackListHead; + if ((ListHead->Flink) && (ListHead->Blink)) + { + /* Loop the list */ + LastEntry = ListHead; + NextEntry = ListHead->Flink; + while (NextEntry != ListHead) + { + /* Get the reord */ + CurrentRecord = CONTAINING_RECORD(NextEntry, + KBUGCHECK_CALLBACK_RECORD, + Entry); + + /* Validate it */ + if (CurrentRecord->Entry.Blink != LastEntry) return; + Checksum = (ULONG_PTR)CurrentRecord->CallbackRoutine; + Checksum += (ULONG_PTR)CurrentRecord->Buffer; + Checksum += (ULONG_PTR)CurrentRecord->Length; + Checksum += (ULONG_PTR)CurrentRecord->Component; + + /* Make sure it's inserted and valitdated */ + if ((CurrentRecord->State == BufferInserted) && + (CurrentRecord->Checksum == Checksum)) + { + /* Call the routine */ + CurrentRecord->State = BufferStarted; + (CurrentRecord->CallbackRoutine)(CurrentRecord->Buffer, + CurrentRecord->Length); + CurrentRecord->State = BufferFinished; + } + + /* Go to the next entry */ + LastEntry = NextEntry; + NextEntry = NextEntry->Flink; + } + } +} + +VOID +NTAPI +KiBugCheckDebugBreak(IN ULONG StatusCode) +{ + /* If KDBG isn't connected, freeze the CPU, otherwise, break */ + if (KdDebuggerNotPresent) for (;;) Ke386HaltProcessor(); + DbgBreakPointWithStatus(StatusCode); +} + +PVOID +NTAPI +KiPcToFileHeader(IN PVOID Eip, + OUT PLDR_DATA_TABLE_ENTRY *LdrEntry, + IN BOOLEAN DriversOnly, + OUT PBOOLEAN InKernel) +{ + ULONG i = 0; + PVOID ImageBase, EipBase = NULL; + PLDR_DATA_TABLE_ENTRY Entry; + PLIST_ENTRY ListHead, NextEntry; + extern LIST_ENTRY ModuleListHead; + + /* Assume no */ + *InKernel = FALSE; + + /* Set list pointers and make sure it's valid */ + ListHead = &ModuleListHead; + NextEntry = ListHead->Flink; + if (NextEntry) + { + /* Start loop */ + while (NextEntry != ListHead) + { + /* Increase entry */ + i++; + + /* Check if this is a kernel entry and we only want drivers */ + if ((i <= 2) && (DriversOnly == TRUE)) + { + /* Skip it */ + NextEntry = NextEntry->Flink; + continue; + } + + /* Get the loader entry */ + Entry = CONTAINING_RECORD(NextEntry, + LDR_DATA_TABLE_ENTRY, + InLoadOrderLinks); + + /* Move to the next entry */ + NextEntry = NextEntry->Flink; + ImageBase = Entry->DllBase; + + /* Check if this is the right one */ + if (((ULONG_PTR)Eip >= (ULONG_PTR)Entry->DllBase) && + ((ULONG_PTR)Eip < ((ULONG_PTR)Entry->DllBase + Entry->SizeOfImage))) + { + /* Return this entry */ + *LdrEntry = Entry; + EipBase = ImageBase; + + /* Check if this was a kernel or HAL entry */ + if (i <= 2) *InKernel = TRUE; + break; + } + } + } + + /* Return the base address */ + return EipBase; +} + +PCHAR +NTAPI +KeBugCheckUnicodeToAnsi(IN PUNICODE_STRING Unicode, + OUT PCHAR Ansi, + IN ULONG Length) +{ + PCHAR p; + PWCHAR pw; + ULONG i; + + /* Set length and normalize it */ + i = Unicode->Length / sizeof(WCHAR); + i = min(i, Length - 1); + + /* Set source and destination, and copy */ + pw = Unicode->Buffer; + p = Ansi; + while (i--) *p++ = (CHAR)*pw++; + + /* Null terminate and return */ + *p = ANSI_NULL; + return Ansi; +} + +VOID +NTAPI +KiDumpParameterImages(IN PCHAR Message, + IN PULONG_PTR Parameters, + IN ULONG ParameterCount, + IN PKE_BUGCHECK_UNICODE_TO_ANSI ConversionRoutine) +{ + ULONG i; + BOOLEAN InSystem; + PLDR_DATA_TABLE_ENTRY LdrEntry; + PVOID ImageBase; + PUNICODE_STRING DriverName; + CHAR AnsiName[32]; + PIMAGE_NT_HEADERS NtHeader; + ULONG TimeStamp; + BOOLEAN FirstRun = TRUE; + + /* Loop parameters */ + for (i = 0; i < ParameterCount; i++) + { + /* Get the base for this parameter */ + ImageBase = KiPcToFileHeader((PVOID)Parameters[i], + &LdrEntry, + FALSE, + &InSystem); + if (!ImageBase) + { + /* Driver wasn't found, check for unloaded driver */ + DriverName = NULL; // FIXME: ROS can't + if (!DriverName) continue; + + /* Convert the driver name */ + ImageBase = (PVOID)Parameters[i]; + ConversionRoutine(DriverName, AnsiName, sizeof(AnsiName)); + } + else + { + /* Get the NT Headers and Timestamp */ + NtHeader = RtlImageNtHeader(LdrEntry->DllBase); + TimeStamp = NtHeader->FileHeader.TimeDateStamp; + + /* Convert the driver name */ + DriverName = &LdrEntry->BaseDllName; + ConversionRoutine(&LdrEntry->BaseDllName, + AnsiName, + sizeof(AnsiName)); + } + + /* Format driver name */ + sprintf(Message, + "%s** %12s - Address %p base at %p, DateStamp %08lx\n", + FirstRun ? "\r\n*":"*", + AnsiName, + (PVOID)Parameters[i], + ImageBase, + TimeStamp); + + /* Check if we only had one parameter */ + if (ParameterCount <= 1) + { + /* Then just save the name */ + KiBugCheckDriver = DriverName; + } + else + { + /* Otherwise, display the message */ + InbvDisplayString(Message); + } + + /* Loop again */ + FirstRun = FALSE; + } +} + +VOID +NTAPI +KiDisplayBlueScreen(IN ULONG MessageId, + IN BOOLEAN IsHardError, + IN PCHAR HardErrCaption OPTIONAL, + IN PCHAR HardErrMessage OPTIONAL, + IN PCHAR Message) +{ + CHAR AnsiName[75]; + + /* FIXMEs: Use inbv to clear, fill and write to screen. */ + + /* Check if this is a hard error */ + if (IsHardError) + { + /* Display caption and message */ + if (HardErrCaption) InbvDisplayString(HardErrCaption); + if (HardErrMessage) InbvDisplayString(HardErrMessage); + return; + } + + /* Begin the display */ + InbvDisplayString("\r\n"); + + /* Print out initial message */ + KeGetBugMessageText(BUGCHECK_MESSAGE_INTRO, NULL); + InbvDisplayString("\r\n\r\n"); + + /* Check if we have a driver */ + if (KiBugCheckDriver) + { + /* Print out into to driver name */ + KeGetBugMessageText(BUGCODE_ID_DRIVER, NULL); + + /* Convert and print out driver name */ + KeBugCheckUnicodeToAnsi(KiBugCheckDriver, AnsiName, sizeof(AnsiName)); + InbvDisplayString(" "); + InbvDisplayString(AnsiName); + InbvDisplayString("\r\n\r\n"); + } + + /* Check if this is the generic message */ + if (MessageId == BUGCODE_PSS_MESSAGE) + { + /* It is, so get the bug code string as well */ + KeGetBugMessageText(KiBugCheckData[0], NULL); + InbvDisplayString("\r\n\r\n"); + } + + /* Print second introduction message */ + KeGetBugMessageText(PSS_MESSAGE_INTRO, NULL); + InbvDisplayString("\r\n\r\n"); + + /* Get the bug code string */ + KeGetBugMessageText(MessageId, NULL); + InbvDisplayString("\r\n\r\n"); + + /* Print message for technical information */ + KeGetBugMessageText(BUGCHECK_TECH_INFO, NULL); + + /* Show the techincal Data */ + sprintf(AnsiName, + "\r\n\r\n*** STOP: 0x%08lX (0x%p,0x%p,0x%p,0x%p)\r\n\r\n", + KiBugCheckData[0], + (PVOID)KiBugCheckData[1], + (PVOID)KiBugCheckData[2], + (PVOID)KiBugCheckData[3], + (PVOID)KiBugCheckData[4]); + InbvDisplayString(AnsiName); + + /* Check if we have a driver*/ + if (KiBugCheckDriver) + { + /* Display technical driver data */ + InbvDisplayString(Message); + } + else + { + /* Dump parameter information */ + KiDumpParameterImages(Message, + (PVOID)&KiBugCheckData[1], + 4, + KeBugCheckUnicodeToAnsi); + } +} + +VOID +NTAPI +KeBugCheckWithTf(IN ULONG BugCheckCode, + IN ULONG_PTR BugCheckParameter1, + IN ULONG_PTR BugCheckParameter2, + IN ULONG_PTR BugCheckParameter3, + IN ULONG_PTR BugCheckParameter4, + IN PKTRAP_FRAME TrapFrame) +{ + PKPRCB Prcb = KeGetCurrentPrcb(); + CONTEXT Context; + ULONG MessageId; + CHAR AnsiName[128]; + BOOLEAN IsSystem, IsHardError = FALSE; + PCHAR HardErrCaption = NULL, HardErrMessage = NULL; + PVOID Eip = NULL, Memory; + PVOID DriverBase; + PLDR_DATA_TABLE_ENTRY LdrEntry; + PULONG_PTR HardErrorParameters; + KIRQL OldIrql; + + /* Set active bugcheck */ + KeBugCheckActive = TRUE; + KiBugCheckDriver = NULL; + + /* Check if this is power failure simulation */ + if (BugCheckCode == POWER_FAILURE_SIMULATE) + { + /* Call the Callbacks and reboot */; + KiDoBugCheckCallbacks(); + HalReturnToFirmware(HalRebootRoutine); + } + + /* Save the IRQL and set hardware trigger */ + Prcb->DebuggerSavedIRQL = KeGetCurrentIrql(); + InterlockedIncrement(&KiHardwareTrigger); + + /* Capture the CPU Context */ + RtlCaptureContext(&Prcb->ProcessorState.ContextFrame); + Context = Prcb->ProcessorState.ContextFrame; + + /* FIXME: Call the Watchdog if it's regsitered */ + + /* Check which bugcode this is */ + switch (BugCheckCode) + { + /* These bug checks already have detailed messages, keep them */ + case UNEXPECTED_KERNEL_MODE_TRAP: + case DRIVER_CORRUPTED_EXPOOL: + case ACPI_BIOS_ERROR: + case ACPI_BIOS_FATAL_ERROR: + case THREAD_STUCK_IN_DEVICE_DRIVER: + case DATA_BUS_ERROR: + case FAT_FILE_SYSTEM: + case NO_MORE_SYSTEM_PTES: + case INACCESSIBLE_BOOT_DEVICE: + case KMODE_EXCEPTION_NOT_HANDLED: + + /* Keep the same code */ + MessageId = BugCheckCode; + break; + + /* Check if this is a kernel-mode exception */ + case KERNEL_MODE_EXCEPTION_NOT_HANDLED: + + /* Use the generic text message */ + MessageId = KMODE_EXCEPTION_NOT_HANDLED; + + /* File-system errors */ + case NTFS_FILE_SYSTEM: + + /* Use the generic message for FAT */ + MessageId = FAT_FILE_SYSTEM; + + /* Check if this is a coruption of the Mm's Pool */ + case DRIVER_CORRUPTED_MMPOOL: + + /* Use generic corruption message */ + MessageId = DRIVER_CORRUPTED_EXPOOL; + + /* Check if this is a signature check failure */ + case STATUS_SYSTEM_IMAGE_BAD_SIGNATURE: + + /* Use the generic corruption message */ + MessageId = BUGCODE_PSS_MESSAGE_SIGNATURE; + + /* All other codes */ + default: + + /* Use the default bugcheck message */ + MessageId = BUGCODE_PSS_MESSAGE; + } + + /* Save bugcheck data */ + KiBugCheckData[0] = BugCheckCode; + KiBugCheckData[1] = BugCheckParameter1; + KiBugCheckData[2] = BugCheckParameter2; + KiBugCheckData[3] = BugCheckParameter3; + KiBugCheckData[4] = BugCheckParameter4; + + /* Now check what bugcheck this is */ + switch (BugCheckCode) + { + /* Invalid access to R/O memory or Unhandled KM Exception */ + case KERNEL_MODE_EXCEPTION_NOT_HANDLED: + case ATTEMPTED_WRITE_TO_READONLY_MEMORY: + case ATTEMPTED_EXECUTE_OF_NOEXECUTE_MEMORY: + + /* Check if we have a trap frame */ + if (!TrapFrame) + { + /* Use parameter 3 as a trap frame, if it exists */ + if (BugCheckParameter3) TrapFrame = (PVOID)BugCheckParameter3; + } + + /* Check if we got one now and if we need to get EIP */ + if ((TrapFrame) && + (BugCheckCode != KERNEL_MODE_EXCEPTION_NOT_HANDLED)) + { + /* Get EIP */ + Eip = (PVOID)TrapFrame->Eip; + } + break; + + /* Wrong IRQL */ + case IRQL_NOT_LESS_OR_EQUAL: + + /* + * The NT kernel has 3 special sections: + * MISYSPTE, POOLMI and POOLCODE. The bug check code can + * determine in which of these sections this bugcode happened + * and provide a more detailed analysis. For now, we don't. + */ + + /* Eip is in parameter 4 */ + Eip = (PVOID)BugCheckParameter4; + + /* Get the driver base */ + DriverBase = KiPcToFileHeader(Eip, &LdrEntry, FALSE, &IsSystem); + if (IsSystem) + { + /* + * The error happened inside the kernel or HAL. + * Get the memory address that was being referenced. + */ + Memory = (PVOID)BugCheckParameter1; + + /* Find to which driver it belongs */ + DriverBase = KiPcToFileHeader(Memory, + &LdrEntry, + TRUE, + &IsSystem); + if (DriverBase) + { + /* Get the driver name and update the bug code */ + KiBugCheckDriver = &LdrEntry->BaseDllName; + KiBugCheckData[0] = DRIVER_PORTION_MUST_BE_NONPAGED; + } + else + { + /* Find the driver that unloaded at this address */ + KiBugCheckDriver = NULL; // FIXME: ROS can't locate + + /* Check if the cause was an unloaded driver */ + if (KiBugCheckDriver) + { + /* Update bug check code */ + KiBugCheckData[0] = + SYSTEM_SCAN_AT_RAISED_IRQL_CAUGHT_IMPROPER_DRIVER_UNLOAD; + } + } + } + else + { + /* Update the bug check code */ + KiBugCheckData[0] = DRIVER_IRQL_NOT_LESS_OR_EQUAL; + } + + /* Clear EIP so we don't look it up later */ + Eip = NULL; + break; + + /* Hard error */ + case FATAL_UNHANDLED_HARD_ERROR: + + /* Copy bug check data from hard error */ + HardErrorParameters = (PULONG_PTR)BugCheckParameter2; + KiBugCheckData[0] = BugCheckParameter1; + KiBugCheckData[1] = HardErrorParameters[0]; + KiBugCheckData[2] = HardErrorParameters[1]; + KiBugCheckData[3] = HardErrorParameters[2]; + KiBugCheckData[4] = HardErrorParameters[3]; + + /* Remember that this is hard error and set the caption/message */ + IsHardError = TRUE; + HardErrCaption = (PCHAR)BugCheckParameter3; + HardErrMessage = (PCHAR)BugCheckParameter4; + break; + + /* Page fault */ + case PAGE_FAULT_IN_NONPAGED_AREA: + + /* Assume no driver */ + DriverBase = NULL; + + /* Check if we have a trap frame */ + if (!TrapFrame) + { + /* We don't, use parameter 3 if possible */ + if (BugCheckParameter3) TrapFrame = (PVOID)BugCheckParameter3; + } + + /* Check if we have a frame now */ + if (TrapFrame) + { + /* Get EIP */ + Eip = (PVOID)TrapFrame->Eip; + + /* Find out if was in the kernel or drivers */ + DriverBase = KiPcToFileHeader(Eip, &LdrEntry, FALSE, &IsSystem); + } + + /* + * Now we should check if this happened in: + * 1) Special Pool 2) Free Special Pool 3) Session Pool + * and update the bugcheck code appropriately. + */ + + /* Check if we had a driver base */ + if (DriverBase) + { + /* Find the driver that unloaded at this address */ + KiBugCheckDriver = NULL; // FIXME: ROS can't locate + + /* Check if the cause was an unloaded driver */ + if (KiBugCheckDriver) + { + KiBugCheckData[0] = + DRIVER_UNLOADED_WITHOUT_CANCELLING_PENDING_OPERATIONS; + } + } + break; + + /* Check if the driver forgot to unlock pages */ + case DRIVER_LEFT_LOCKED_PAGES_IN_PROCESS: + + /* EIP is in parameter 1 */ + Eip = (PVOID)BugCheckParameter1; + break; + + /* Check if the driver consumed too many PTEs */ + case DRIVER_USED_EXCESSIVE_PTES: + + /* Driver base is in parameter 1 */ + DriverBase = (PVOID)BugCheckParameter1; + KiBugCheckDriver = &LdrEntry->BaseDllName; + break; + + /* Check if the driver has a stuck thread */ + case THREAD_STUCK_IN_DEVICE_DRIVER: + + /* The name is in Parameter 3 */ + KiBugCheckDriver = (PVOID)BugCheckParameter3; + break; + + /* Anything else */ + default: + break; + } + + /* Do we have a driver name? */ + if (KiBugCheckDriver) + { + /* Convert it to ANSI */ + KeBugCheckUnicodeToAnsi(KiBugCheckDriver, AnsiName, sizeof(AnsiName)); + } + else + { + /* Do we have an EIP? */ + if (Eip) + { + /* Dump image name */ + KiDumpParameterImages(AnsiName, + (PULONG_PTR)&Eip, + 1, + KeBugCheckUnicodeToAnsi); + } + } + + /* FIXME: Check if we need to save the context for KD */ + + /* Check if a debugger is connected */ + if ((BugCheckCode != MANUALLY_INITIATED_CRASH) && (KdDebuggerEnabled)) + { + /* Crash on the debugger console */ + DbgPrint("\n*** Fatal System Error: 0x%08lx\n" + " (0x%p,0x%p,0x%p,0x%p)\n\n", + KiBugCheckData[0], + KiBugCheckData[1], + KiBugCheckData[2], + KiBugCheckData[3], + KiBugCheckData[4]); + + /* Check if the debugger isn't currently connected */ + if (!KdDebuggerNotPresent) + { + /* Check if we have a driver to blame */ + if (KiBugCheckDriver) + { + /* Dump it */ + DbgPrint("Driver at fault: %s.\n", AnsiName); + } + + /* Check if this was a hard error */ + if (IsHardError) + { + /* Print caption and message */ + if (HardErrCaption) DbgPrint(HardErrCaption); + if (HardErrMessage) DbgPrint(HardErrMessage); + } + + /* Break in the debugger */ + KiBugCheckDebugBreak(DBG_STATUS_BUGCHECK_FIRST); + } + else + { + /* + * ROS HACK. + * Ok, so debugging is enabled, but KDBG isn't there. + * We'll manually dump the stack for the user. + */ + KeRosDumpStackFrames(NULL, 0); + } + } + + /* Switching back to the blue screen so we print messages on it */ + HalReleaseDisplayOwnership(); + + /* Raise IRQL to HIGH_LEVEL */ + Ke386DisableInterrupts(); + KeRaiseIrql(HIGH_LEVEL, &OldIrql); + + /* Unlock the Kernel Adress Space if we own it */ + if (KernelAddressSpaceLock.Owner == KeGetCurrentThread()) + { + MmUnlockAddressSpace(MmGetKernelAddressSpace()); + } + + /* Avoid recursion */ + if (!InterlockedDecrement(&KeBugCheckCount)) + { + /* Set CPU that is bug checking now */ + KeBugCheckOwner = Prcb->Number; + +#ifdef CONFIG_SMP + /* Freeze the other CPUs */ + for (i = 0; i < KeNumberProcessors; i++) + { + if (i != (LONG)KeGetCurrentProcessorNumber()) + { + /* Send the IPI and give them one second to catch up */ + KiIpiSendRequest(1 << i, IPI_FREEZE); + KeStallExecutionProcessor(1000000); + } + } +#endif + + /* Display the BSOD */ + KiDisplayBlueScreen(MessageId, + IsHardError, + HardErrCaption, + HardErrMessage, + AnsiName); + + /* FIXME: Enable debugger if it was pending */ + + /* Print the last line */ + InbvDisplayString("\r\n"); + + /* Save the context */ + Prcb->ProcessorState.ContextFrame = Context; + + /* FIXME: Support Triage Dump */ + + /* Write the crash dump */ + MmDumpToPagingFile(KiBugCheckData[4], + KiBugCheckData[0], + KiBugCheckData[1], + KiBugCheckData[2], + KiBugCheckData[3], + TrapFrame); + } + + /* Increase recursioun count */ + KeBugCheckOwnerRecursionCount++; + if (KeBugCheckOwnerRecursionCount == 2) + { + /* Break in the debugger */ + KiBugCheckDebugBreak(DBG_STATUS_BUGCHECK_SECOND); + } + else if (KeBugCheckOwnerRecursionCount > 2) + { + /* Halt the CPU */ + for (;;) Ke386HaltProcessor(); + } + + /* Call the Callbacks */ + KiDoBugCheckCallbacks(); + + /* FIXME: Call Watchdog if enabled */ + + /* Attempt to break in the debugger (otherwise halt CPU) */ + KiBugCheckDebugBreak(DBG_STATUS_BUGCHECK_SECOND); +} + +/* PUBLIC FUNCTIONS **********************************************************/ /* * @implemented */ BOOLEAN -STDCALL -KeDeregisterBugCheckCallback(PKBUGCHECK_CALLBACK_RECORD CallbackRecord) +NTAPI +KeDeregisterBugCheckCallback(IN PKBUGCHECK_CALLBACK_RECORD CallbackRecord) { KIRQL OldIrql; BOOLEAN Status = FALSE; @@ -99,12 +1012,11 @@ KeRaiseIrql(HIGH_LEVEL, &OldIrql); /* Check the Current State */ - if (CallbackRecord->State == BufferInserted) { - + if (CallbackRecord->State == BufferInserted) + { /* Reset state and remove from list */ CallbackRecord->State = BufferEmpty; RemoveEntryList(&CallbackRecord->Entry); - Status = TRUE; } @@ -117,8 +1029,9 @@ * @implemented */ BOOLEAN -STDCALL -KeDeregisterBugCheckReasonCallback(IN PKBUGCHECK_REASON_CALLBACK_RECORD CallbackRecord) +NTAPI +KeDeregisterBugCheckReasonCallback( + IN PKBUGCHECK_REASON_CALLBACK_RECORD CallbackRecord) { KIRQL OldIrql; BOOLEAN Status = FALSE; @@ -127,12 +1040,11 @@ KeRaiseIrql(HIGH_LEVEL, &OldIrql); /* Check the Current State */ - if (CallbackRecord->State == BufferInserted) { - + if (CallbackRecord->State == BufferInserted) + { /* Reset state and remove from list */ CallbackRecord->State = BufferEmpty; RemoveEntryList(&CallbackRecord->Entry); - Status = TRUE; } @@ -145,12 +1057,12 @@ * @implemented */ BOOLEAN -STDCALL -KeRegisterBugCheckCallback(PKBUGCHECK_CALLBACK_RECORD CallbackRecord, - PKBUGCHECK_CALLBACK_ROUTINE CallbackRoutine, - PVOID Buffer, - ULONG Length, - PUCHAR Component) +NTAPI +KeRegisterBugCheckCallback(IN PKBUGCHECK_CALLBACK_RECORD CallbackRecord, + IN PKBUGCHECK_CALLBACK_ROUTINE CallbackRoutine, + IN PVOID Buffer, + IN ULONG Length, + IN PUCHAR Component) { KIRQL OldIrql; BOOLEAN Status = FALSE; @@ -159,8 +1071,8 @@ KeRaiseIrql(HIGH_LEVEL, &OldIrql); /* Check the Current State first so we don't double-register */ - if (CallbackRecord->State == BufferEmpty) { - + if (CallbackRecord->State == BufferEmpty) + { /* Set the Callback Settings and insert into the list */ CallbackRecord->Length = Length; CallbackRecord->Buffer = Buffer; @@ -168,7 +1080,6 @@ CallbackRecord->CallbackRoutine = CallbackRoutine; CallbackRecord->State = BufferInserted; InsertTailList(&BugcheckCallbackListHead, &CallbackRecord->Entry); - Status = TRUE; } @@ -181,11 +1092,12 @@ * @implemented */ BOOLEAN -STDCALL -KeRegisterBugCheckReasonCallback(IN PKBUGCHECK_REASON_CALLBACK_RECORD CallbackRecord, - IN PKBUGCHECK_REASON_CALLBACK_ROUTINE CallbackRoutine, - IN KBUGCHECK_CALLBACK_REASON Reason, - IN PUCHAR Component) +NTAPI +KeRegisterBugCheckReasonCallback( + IN PKBUGCHECK_REASON_CALLBACK_RECORD CallbackRecord, + IN PKBUGCHECK_REASON_CALLBACK_ROUTINE CallbackRoutine, + IN KBUGCHECK_CALLBACK_REASON Reason, + IN PUCHAR Component) { KIRQL OldIrql; BOOLEAN Status = FALSE; @@ -194,15 +1106,15 @@ KeRaiseIrql(HIGH_LEVEL, &OldIrql); /* Check the Current State first so we don't double-register */ - if (CallbackRecord->State == BufferEmpty) { - + if (CallbackRecord->State == BufferEmpty) + { /* Set the Callback Settings and insert into the list */ CallbackRecord->Component = Component; CallbackRecord->CallbackRoutine = CallbackRoutine; CallbackRecord->State = BufferInserted; CallbackRecord->Reason = Reason; - InsertTailList(&BugcheckReasonCallbackListHead, &CallbackRecord->Entry); - + InsertTailList(&BugcheckReasonCallbackListHead, + &CallbackRecord->Entry); Status = TRUE; } @@ -211,285 +1123,18 @@ return Status; } -VOID -STDCALL -KeGetBugMessageText(ULONG BugCheckCode, PANSI_STRING OutputString) -{ - ULONG i; - ULONG IdOffset; - ULONG_PTR MessageEntry; - PCHAR BugCode; - - /* Find the message. This code is based on RtlFindMesssage -- Alex */ - for (i = 0; i < KiBugCodeMessages->NumberOfBlocks; i++) - { - /* Check if the ID Matches */ - if ((BugCheckCode >= KiBugCodeMessages->Blocks[i].LowId) && - (BugCheckCode <= KiBugCodeMessages->Blocks[i].HighId)) - { - /* Get Offset to Entry */ - MessageEntry = (ULONG_PTR)KiBugCodeMessages + KiBugCodeMessages->Blocks[i].OffsetToEntries; - IdOffset = BugCheckCode - KiBugCodeMessages->Blocks[i].LowId; - - /* Get offset to ID */ - for (i = 0; i < IdOffset; i++) - { - /* Advance in the Entries */ - MessageEntry += ((PRTL_MESSAGE_RESOURCE_ENTRY)MessageEntry)->Length; - } - - /* Get the final Code */ - BugCode = ((PRTL_MESSAGE_RESOURCE_ENTRY)MessageEntry)->Text; - - /* Return it in the OutputString */ - if (OutputString) - { - OutputString->Buffer = BugCode; - OutputString->Length = strlen(BugCode) + 1; - OutputString->MaximumLength = strlen(BugCode) + 1; - } - else - { - /* Direct Output to Screen */ - CHAR BugString[100]; - sprintf(BugString, "%s\n", BugCode); - InbvDisplayString(BugString); - break; - } - } - } -} - -VOID -STDCALL -KiDoBugCheckCallbacks(VOID) -{ - PKBUGCHECK_CALLBACK_RECORD CurrentRecord; - PLIST_ENTRY ListHead; - - /* FIXME: Check Checksum and add support for WithReason Callbacks */ - - /* First make sure that the list is Initialized... it might not be */ - ListHead = &BugcheckCallbackListHead; - if (ListHead->Flink && ListHead->Blink) { - - /* Loop the list */ - LIST_FOR_EACH(CurrentRecord, ListHead, KBUGCHECK_CALLBACK_RECORD, Entry) - { - /* Make sure it's inserted */ - if (CurrentRecord->State == BufferInserted) { - - /* Call the routine */ - CurrentRecord->State = BufferStarted; - (CurrentRecord->CallbackRoutine)(CurrentRecord->Buffer, - CurrentRecord->Length); - CurrentRecord->State = BufferFinished; - } - } - } -} - -VOID -STDCALL -KeBugCheckWithTf(ULONG BugCheckCode, - ULONG BugCheckParameter1, - ULONG BugCheckParameter2, - ULONG BugCheckParameter3, - ULONG BugCheckParameter4, - PKTRAP_FRAME Tf) -{ - KIRQL OldIrql; - BOOLEAN GotExtendedCrashInfo = FALSE; - PVOID Address = 0; - PLDR_DATA_TABLE_ENTRY CurrentModule = NULL; - extern LIST_ENTRY ModuleListHead; -#if 0 - CHAR PrintString[100]; -#endif - /* Make sure we're switching back to the blue screen and print messages on it */ - HalReleaseDisplayOwnership(); - if (!KdpDebugMode.Screen) - { - /* Enable screen debug mode */ - KdpDebugMode.Screen = TRUE; - InitRoutines[0](&DispatchTable[0], 0); - } - - /* Try to find out who did this. For this, we need a Trap Frame. - * Note: Some special BSODs pass the Frame/EIP as a Param. MSDN has the - * info so it eventually needs to be supported. - */ - if (Tf) - { - /* For now, get Address from EIP */ - Address = (PVOID)Tf->Eip; - - /* Try to get information on the module */ - LIST_FOR_EACH(CurrentModule, &ModuleListHead, LDR_DATA_TABLE_ENTRY, InLoadOrderLinks) - { - /* Check if this is the right one */ - if ((Address != NULL && (Address >= (PVOID)CurrentModule->DllBase && - Address < (PVOID)((ULONG_PTR)CurrentModule->DllBase + CurrentModule->SizeOfImage)))) - { - /* We got it */ - GotExtendedCrashInfo = TRUE; - break; - } - } - } - - /* Raise IRQL to HIGH_LEVEL */ - Ke386DisableInterrupts(); - KeRaiseIrql(HIGH_LEVEL, &OldIrql); - - /* Unload the Kernel Adress Space if we own it */ - if (KernelAddressSpaceLock.Owner == KeGetCurrentThread()) - MmUnlockAddressSpace(MmGetKernelAddressSpace()); - - /* FIXMEs: Use inbv to clear, fill and write to screen. */ - - /* Show the STOP Message */ -#if 0 - InbvDisplayString("A problem has been detected and ReactOS has been shut down to prevent " - "damage to your computer.\n\n"); -#else - DbgPrint("A problem has been detected and ReactOS has been shut down to prevent " - "damage to your computer.\n\n"); -#endif - /* Show the module name of who caused this */ - if (GotExtendedCrashInfo) - { -#if 0 - sprintf(PrintString, - "The problem seems to be caused by the following file: %wZ\n\n", - &CurrentModule->BaseDllName); - InbvDisplayString(PrintString); -#else - DbgPrint("The problem seems to be caused by the following file: %wZ\n\n", - &CurrentModule->BaseDllName); -#endif - } - - /* Find the Bug Code String */ - KeGetBugMessageText(BugCheckCode, NULL); - - /* Show the techincal Data */ -#if 0 - sprintf(PrintString, - "Technical information:\n\n*** STOP: 0x%08lX (0x%p,0x%p,0x%p,0x%p)\n\n", - BugCheckCode, - (PVOID)BugCheckParameter1, - (PVOID)BugCheckParameter2, - (PVOID)BugCheckParameter3, - (PVOID)BugCheckParameter4); - InbvDisplayString(PrintString); -#else - DbgPrint("Technical information:\n\n*** STOP: 0x%08lX (0x%p,0x%p,0x%p,0x%p)\n\n", - BugCheckCode, - (PVOID)BugCheckParameter1, - (PVOID)BugCheckParameter2, - (PVOID)BugCheckParameter3, - (PVOID)BugCheckParameter4); -#endif - /* Show the module name and more data of who caused this */ - if (GotExtendedCrashInfo) - { -#if 0 - sprintf(PrintString, - "*** %wZ - Address 0x%p base at 0x%p, DateStamp 0x%x\n\n", - &CurrentModule->BaseDllName, - Address, - (PVOID)CurrentModule->DllBase, - 0); - InbvDisplayString(PrintString); -#else - DbgPrint("*** %wZ - Address 0x%p base at 0x%p, DateStamp 0x%x\n\n", - &CurrentModule->BaseDllName, - Address, - (PVOID)CurrentModule->DllBase, - 0); -#endif - } - - /* There can only be one Bugcheck per Bootup */ - if (!InterlockedDecrement((PLONG)&KeBugCheckCount)) - { -#ifdef CONFIG_SMP - LONG i; - /* Freeze the other CPUs */ - for (i = 0; i < KeNumberProcessors; i++) - { - if (i != (LONG)KeGetCurrentProcessorNumber()) - { - /* Send the IPI and give them one second to catch up */ - KiIpiSendRequest(1 << i, IPI_FREEZE); - KeStallExecutionProcessor(1000000); - } - } -#endif - /* Check if we got a Trap Frame */ - if (Tf) - { - /* Dump it */ - KiDumpTrapFrame(Tf, BugCheckParameter1, BugCheckParameter2); - } - else - { - /* We can only dump the frames */ -#if defined(__GNUC__) - KeDumpStackFrames((PULONG)__builtin_frame_address(0)); -#elif defined(_MSC_VER) - __asm push ebp - __asm call KeDumpStackFrames - __asm add esp, 4 -#else -#error Unknown compiler for inline assembler -#endif - } - - /* Call the Callbacks */; - KiDoBugCheckCallbacks(); - - /* Dump the BSOD to the Paging File */ - MmDumpToPagingFile(BugCheckCode, - BugCheckParameter1, - BugCheckParameter2, - BugCheckParameter3, - BugCheckParameter4, - Tf); - - /* Wake up the Debugger */ - if (KdDebuggerEnabled) - { - Ke386EnableInterrupts(); - DbgBreakPointWithStatus(DBG_STATUS_BUGCHECK_SECOND); - Ke386DisableInterrupts(); - } - } - - /* Halt this CPU now */ - for (;;) Ke386HaltProcessor(); -} - /* * @implemented - * - * FUNCTION: Brings the system down in a controlled manner when an - * inconsistency that might otherwise cause corruption has been detected - * ARGUMENTS: - * BugCheckCode = Specifies the reason for the bug check - * BugCheckParameter[1-4] = Additional information about bug - * RETURNS: Doesn't */ VOID -STDCALL -KeBugCheckEx(ULONG BugCheckCode, - ULONG BugCheckParameter1, - ULONG BugCheckParameter2, - ULONG BugCheckParameter3, - ULONG BugCheckParameter4) -{ - /* Call the Trap Frame version without a Trap Frame */ +NTAPI +KeBugCheckEx(IN ULONG BugCheckCode, + IN ULONG_PTR BugCheckParameter1, + IN ULONG_PTR BugCheckParameter2, + IN ULONG_PTR BugCheckParameter3, + IN ULONG_PTR BugCheckParameter4) +{ + /* Call the internal API */ KeBugCheckWithTf(BugCheckCode, BugCheckParameter1, BugCheckParameter2, @@ -500,18 +1145,13 @@ /* * @implemented - * - * FUNCTION: Brings the system down in a controlled manner when an - * inconsistency that might otherwise cause corruption has been detected - * ARGUMENTS: - * BugCheckCode = Specifies the reason for the bug check - * RETURNS: Doesn't */ VOID -STDCALL +NTAPI KeBugCheck(ULONG BugCheckCode) { - KeBugCheckEx(BugCheckCode, 0, 0, 0, 0); + /* Call the internal API */ + KeBugCheckWithTf(BugCheckCode, 0, 0, 0, 0, NULL); } /* EOF */ Modified: trunk/reactos/ntoskrnl/ke/i386/exp.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/i386/exp.c?rev… ============================================================================== --- trunk/reactos/ntoskrnl/ke/i386/exp.c (original) +++ trunk/reactos/ntoskrnl/ke/i386/exp.c Wed Aug 30 09:22:56 2006 @@ -1,214 +1,40 @@ /* - * COPYRIGHT: See COPYING in the top level directory * PROJECT: ReactOS Kernel + * LICENSE: GPL - See COPYING in the top level directory * FILE: ntoskrnl/ke/i386/exp.c - * PURPOSE: Exception Support Code - * PROGRAMMERS: Alex Ionescu (alex(a)relsoft.net) + * PURPOSE: Exception Dispatching and Context<->Trap Frame Conversion + * PROGRAMMERS: Alex Ionescu (alex.ionescu(a)reactos.org) * Gregor Anich - * David Welch (welch(a)cwcom.net) * Skywing (skywing(a)valhallalegends.com) */ -/* INCLUDES *****************************************************************/ +/* INCLUDES ******************************************************************/ #include <ntoskrnl.h> - #define NDEBUG -#include <internal/debug.h> - -#if defined (ALLOC_PRAGMA) -#pragma alloc_text(INIT, KeInitExceptions) -#endif - -#define SIZE_OF_FX_REGISTERS 32 +#include <debug.h> + +/* FUNCTIONS *****************************************************************/ VOID -NTAPI -Ki386AdjustEsp0( - IN PKTRAP_FRAME TrapFrame -); - -VOID -NTAPI -KiFlushNPXState( - IN FLOATING_SAVE_AREA *SaveArea -); - -extern KIDTENTRY KiIdt[]; - -/* GLOBALS *****************************************************************/ - -#define FLAG_IF (1<<9) - -#define _STR(x) #x -#define STR(x) _STR(x) - -#ifndef ARRAY_SIZE -# define ARRAY_SIZE(x) (sizeof (x) / sizeof (x[0])) -#endif - -extern ULONG init_stack; -extern ULONG init_stack_top; - -extern BOOLEAN Ke386NoExecute; - -static char *ExceptionTypeStrings[] = - { - "Divide Error", - "Debug Trap", - "NMI", - "Breakpoint", - "Overflow", - "BOUND range exceeded", - "Invalid Opcode", - "No Math Coprocessor", - "Double Fault", - "Unknown(9)", - "Invalid TSS", - "Segment Not Present", - "Stack Segment Fault", - "General Protection", - "Page Fault", - "Reserved(15)", - "Math Fault", - "Alignment Check", - "Machine Check", - "SIMD Fault" - }; - -NTSTATUS ExceptionToNtStatus[] = - { - STATUS_INTEGER_DIVIDE_BY_ZERO, - STATUS_SINGLE_STEP, - STATUS_ACCESS_VIOLATION, - STATUS_BREAKPOINT, - STATUS_INTEGER_OVERFLOW, - STATUS_ARRAY_BOUNDS_EXCEEDED, - STATUS_ILLEGAL_INSTRUCTION, - STATUS_FLOAT_INVALID_OPERATION, - STATUS_ACCESS_VIOLATION, - STATUS_ACCESS_VIOLATION, - STATUS_ACCESS_VIOLATION, - STATUS_ACCESS_VIOLATION, - STATUS_STACK_OVERFLOW, - STATUS_ACCESS_VIOLATION, - STATUS_ACCESS_VIOLATION, - STATUS_ACCESS_VIOLATION, /* RESERVED */ - STATUS_FLOAT_INVALID_OPERATION, /* Should not be used, the FPU can give more specific info */ - STATUS_DATATYPE_MISALIGNMENT, - STATUS_ACCESS_VIOLATION, - STATUS_FLOAT_MULTIPLE_TRAPS, - }; - -/* FUNCTIONS ****************************************************************/ - -BOOLEAN STDCALL -KiRosPrintAddress(PVOID address) -{ - PLIST_ENTRY current_entry; - PLDR_DATA_TABLE_ENTRY current; - extern LIST_ENTRY ModuleListHead; - ULONG_PTR RelativeAddress; - ULONG i = 0; - - do - { - current_entry = ModuleListHead.Flink; - - while (current_entry != &ModuleListHead) - { - current = - CONTAINING_RECORD(current_entry, LDR_DATA_TABLE_ENTRY, InLoadOrderLinks); - - if (address >= (PVOID)current->DllBase && - address < (PVOID)((ULONG_PTR)current->DllBase + current->SizeOfImage)) - { - RelativeAddress = (ULONG_PTR) address - (ULONG_PTR) current->DllBase; - DbgPrint("<%wZ: %x>", &current->FullDllName, RelativeAddress); - return(TRUE); - } - current_entry = current_entry->Flink; - } - - address = (PVOID)((ULONG_PTR)address & ~(ULONG_PTR)MmSystemRangeStart); - } while(++i <= 1); - - return(FALSE); -} - -VOID -NTAPI -KiDumpTrapFrame(PKTRAP_FRAME Tf, ULONG Parameter1, ULONG Parameter2) -{ - ULONG cr3_; - ULONG StackLimit; - ULONG Esp0; - ULONG ExceptionNr = (ULONG)Tf->DbgArgMark; - ULONG cr2 = (ULONG)Tf->DbgArgPointer; - - Esp0 = (ULONG)Tf; - - /* - * Print out the CPU registers - */ - if (ExceptionNr < ARRAY_SIZE(ExceptionTypeStrings)) - { - DbgPrint("%s Exception: %d(%x)\n", ExceptionTypeStrings[ExceptionNr], - ExceptionNr, Tf->ErrCode&0xffff); - } - else - { - DbgPrint("Exception: %d(%x)\n", ExceptionNr, Tf->ErrCode&0xffff); - } - DbgPrint("Processor: %d CS:EIP %x:%x ", KeGetCurrentProcessorNumber(), - Tf->SegCs&0xffff, Tf->Eip); - KeRosPrintAddress((PVOID)Tf->Eip); - DbgPrint("\n"); - Ke386GetPageTableDirectory(cr3_); - DbgPrint("cr2 %x cr3 %x ", cr2, cr3_); - DbgPrint("Proc: %x ",PsGetCurrentProcess()); - if (PsGetCurrentProcess() != NULL) - { - DbgPrint("Pid: %x <", PsGetCurrentProcess()->UniqueProcessId); - DbgPrint("%.16s> ", PsGetCurrentProcess()->ImageFileName); - } - if (PsGetCurrentThread() != NULL) - { - DbgPrint("Thrd: %x Tid: %x", - PsGetCurrentThread(), - PsGetCurrentThread()->Cid.UniqueThread); - } - DbgPrint("\n"); - DbgPrint("DS %x ES %x FS %x GS %x\n", Tf->SegDs&0xffff, Tf->SegEs&0xffff, - Tf->SegFs&0xffff, Tf->SegGs&0xfff); - DbgPrint("EAX: %.8x EBX: %.8x ECX: %.8x\n", Tf->Eax, Tf->Ebx, Tf->Ecx); - DbgPrint("EDX: %.8x EBP: %.8x ESI: %.8x ESP: %.8x\n", Tf->Edx, - Tf->Ebp, Tf->Esi, Esp0); - DbgPrint("EDI: %.8x EFLAGS: %.8x ", Tf->Edi, Tf->EFlags); - if ((Tf->SegCs&0xffff) == KGDT_R0_CODE) - { - DbgPrint("kESP %.8x ", Esp0); - if (PsGetCurrentThread() != NULL) - { - DbgPrint("kernel stack base %x\n", - PsGetCurrentThread()->Tcb.StackLimit); - - } - } - - if (PsGetCurrentThread() != NULL) - { - StackLimit = (ULONG)PsGetCurrentThread()->Tcb.StackBase; - } - else - { - StackLimit = (ULONG)init_stack_top; - } - - /* - * Dump the stack frames - */ - KeDumpStackFrames((PULONG)Tf->Ebp); +INIT_FUNCTION +NTAPI +KeInitExceptions(VOID) +{ + ULONG i; + USHORT FlippedSelector; + extern KIDTENTRY KiIdt[]; + + /* Loop the IDT */ + for (i = 0; i <= MAXIMUM_IDTVECTOR; i ++) + { + /* Save the current Selector */ + FlippedSelector = KiIdt[i].Selector; + + /* Flip Selector and Extended Offset */ + KiIdt[i].Selector = KiIdt[i].ExtendedOffset; + KiIdt[i].ExtendedOffset = FlippedSelector; + } } ULONG @@ -752,177 +578,6 @@ VOID NTAPI -KeDumpStackFrames(PULONG Frame) -{ - PULONG StackBase, StackEnd; - MEMORY_BASIC_INFORMATION mbi; - ULONG ResultLength = sizeof(mbi); - NTSTATUS Status; - - DbgPrint("Frames:\n"); - _SEH_TRY - { - Status = MiQueryVirtualMemory ( - (HANDLE)-1, - Frame, - MemoryBasicInformation, - &mbi, - sizeof(mbi), - &ResultLength ); - if ( !NT_SUCCESS(Status) ) - { - DPRINT1("Can't dump stack frames: MiQueryVirtualMemory() failed: %x\n", Status ); - return; - } - - StackBase = Frame; - StackEnd = (PULONG)((ULONG_PTR)mbi.BaseAddress + mbi.RegionSize); - - while ( Frame >= StackBase && Frame < StackEnd ) - { - ULONG Addr = Frame[1]; - if (!KeRosPrintAddress((PVOID)Addr)) - DbgPrint("<%X>", Addr); - if ( Addr == 0 || Addr == 0xDEADBEEF ) - break; - StackBase = Frame; - Frame = (PULONG)Frame[0]; - DbgPrint("\n"); - } - } - _SEH_HANDLE - { - } - _SEH_END; - DbgPrint("\n"); -} - -VOID STDCALL -KeRosDumpStackFrames ( PULONG Frame, ULONG FrameCount ) -{ - ULONG i=0; - PULONG StackBase, StackEnd; - MEMORY_BASIC_INFORMATION mbi; - ULONG ResultLength = sizeof(mbi); - NTSTATUS Status; - - DbgPrint("Frames: "); - _SEH_TRY - { - if ( !Frame ) - { -#if defined __GNUC__ - __asm__("mov %%ebp, %0" : "=r" (Frame) : ); -#elif defined(_MSC_VER) - __asm mov [Frame], ebp -#endif - //Frame = (PULONG)Frame[0]; // step out of KeRosDumpStackFrames - } - - Status = MiQueryVirtualMemory ( - (HANDLE)-1, - Frame, - MemoryBasicInformation, - &mbi, - sizeof(mbi), - &ResultLength ); - if ( !NT_SUCCESS(Status) ) - { - DPRINT1("Can't dump stack frames: MiQueryVirtualMemory() failed: %x\n", Status ); - return; - } - - StackBase = Frame; - StackEnd = (PULONG)((ULONG_PTR)mbi.BaseAddress + mbi.RegionSize); - - while ( Frame >= StackBase && Frame < StackEnd && i++ < FrameCount ) - { - ULONG Addr = Frame[1]; - if (!KeRosPrintAddress((PVOID)Addr)) - DbgPrint("<%X>", Addr); - if ( Addr == 0 || Addr == 0xDEADBEEF ) - break; - StackBase = Frame; - Frame = (PULONG)Frame[0]; - DbgPrint(" "); - } - } - _SEH_HANDLE - { - } - _SEH_END; - DbgPrint("\n"); -} - -ULONG STDCALL -KeRosGetStackFrames ( PULONG Frames, ULONG FrameCount ) -{ - ULONG Count = 0; - PULONG StackBase, StackEnd, Frame; - MEMORY_BASIC_INFORMATION mbi; - ULONG ResultLength = sizeof(mbi); - NTSTATUS Status; - - _SEH_TRY - { -#if defined __GNUC__ - __asm__("mov %%ebp, %0" : "=r" (Frame) : ); -#elif defined(_MSC_VER) - __asm mov [Frame], ebp -#endif - - Status = MiQueryVirtualMemory ( - (HANDLE)-1, - Frame, - MemoryBasicInformation, - &mbi, - sizeof(mbi), - &ResultLength ); - if ( !NT_SUCCESS(Status) ) - { - DPRINT1("Can't get stack frames: MiQueryVirtualMemory() failed: %x\n", Status ); - return 0; - } - - StackBase = Frame; - StackEnd = (PULONG)((ULONG_PTR)mbi.BaseAddress + mbi.RegionSize); - - while ( Count < FrameCount && Frame >= StackBase && Frame < StackEnd ) - { - Frames[Count++] = Frame[1]; - StackBase = Frame; - Frame = (PULONG)Frame[0]; - } - } - _SEH_HANDLE - { - } - _SEH_END; - return Count; -} - -VOID -INIT_FUNCTION -NTAPI -KeInitExceptions(VOID) -{ - ULONG i; - USHORT FlippedSelector; - - /* Loop the IDT */ - for (i = 0; i <= MAXIMUM_IDTVECTOR; i ++) - { - /* Save the current Selector */ - FlippedSelector = KiIdt[i].Selector; - - /* Flip Selector and Extended Offset */ - KiIdt[i].Selector = KiIdt[i].ExtendedOffset; - KiIdt[i].ExtendedOffset = FlippedSelector; - } -} - -VOID -NTAPI KiDispatchException(PEXCEPTION_RECORD ExceptionRecord, PKEXCEPTION_FRAME ExceptionFrame, PKTRAP_FRAME TrapFrame, @@ -1091,10 +746,10 @@ NTAPI KeRaiseUserException(IN NTSTATUS ExceptionCode) { - ULONG OldEip; - PKTHREAD Thread = KeGetCurrentThread(); - - /* Make sure we can access the TEB */ + ULONG OldEip; + PKTHREAD Thread = KeGetCurrentThread(); + + /* Make sure we can access the TEB */ _SEH_TRY { Thread->Teb->ExceptionCode = ExceptionCode; Modified: trunk/reactos/ntoskrnl/ke/i386/v86vdm.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/i386/v86vdm.c?… ============================================================================== --- trunk/reactos/ntoskrnl/ke/i386/v86vdm.c (original) +++ trunk/reactos/ntoskrnl/ke/i386/v86vdm.c Wed Aug 30 09:22:56 2006 @@ -86,7 +86,8 @@ KeSetSystemAffinityThread(1); /* Make sure there's space for two IOPMs, then copy & clear the current */ - //ASSERT(((PKGDTENTRY)&KeGetPcr()->GDT[KGDT_TSS / 8])->LimitLow >= (0x2000 + IOPM_OFFSET - 1)); + //ASSERT(((PKGDTENTRY)&KeGetPcr()->GDT[KGDT_TSS / 8])->LimitLow >= + // (0x2000 + IOPM_OFFSET - 1)); RtlMoveMemory(Ki386IopmSaveArea, &Tss->IoMaps[0].IoMap, PAGE_SIZE * 2); RtlZeroMemory(&Tss->IoMaps[0].IoMap, PAGE_SIZE * 2); Modified: trunk/reactos/ntoskrnl/ntoskrnl.mc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ntoskrnl.mc?rev=2… ============================================================================== --- trunk/reactos/ntoskrnl/ntoskrnl.mc (original) +++ trunk/reactos/ntoskrnl/ntoskrnl.mc Wed Aug 30 09:22:56 2006 @@ -21,6 +21,58 @@ ; ; message definitions ; +MessageId=0x7F +Severity=Warning +Facility=System +SymbolicName=BUGCHECK_MESSAGE_INTRO +Language=English +A problem has been detected and ReactOS has been shut down to prevent damage +to your computer. +. + +MessageId=0x80 +Severity=Warning +Facility=System +SymbolicName=BUGCODE_ID_DRIVER +Language=English +The problem seems to be caused by the following file: + +. + +MessageId=0x81 +Severity=Warning +Facility=System +SymbolicName=PSS_MESSAGE_INTRO +Language=English +If this is the first time you've seen this Stop error screen, +restart your computer. If this screen appears again, follow +these steps: + +. + +MessageId=0x82 +Severity=Warning +Facility=System +SymbolicName=BUGCODE_PSS_MESSAGE +Language=English +Check to make sure any new hardware or software is properly installed. +If this is a new installation, ask your hardware or software manufacturer +for any ReactOS updates you might need. + +If problems continue, disable or remove any newly installed hardware +or software. Disable BIOS memory options such as caching or shadowing. +If you need to use Safe Mode to remove or disable components, restart +your computer, press F8 to select Advanced Startup Options, and then +select Safe Mode. +. + +MessageId=0x83 +Severity=Warning +Facility=System +SymbolicName=BUGCHECK_TECH_INFO +Language=English +Technical information: +. MessageId=0x0 Severity=Success @@ -267,7 +319,16 @@ Facility=System SymbolicName=KMODE_EXCEPTION_NOT_HANDLED Language=English -KMODE_EXCEPTION_NOT_HANDLED +Check to be sure you have adequate disk space. If a driver is +identified in the Stop message, disable the driver or check +with the manufacturer for driver updates. Try changing video +adapters. + +Check with your hardware vendor for any BIOS updates. Disable +BIOS memory options such as caching or shadowing. If you need +to use Safe Mode to remove or disable components, restart your +computer, press F8 to select Advanced Startup Options, and then +select Safe Mode. . MessageId=0x1F @@ -307,7 +368,10 @@ Facility=System SymbolicName=FAT_FILE_SYSTEM Language=English -FAT_FILE_SYSTEM +Disable or uninstall any anti-virus, disk defragmentation +or backup utilities. Check your hard drive configuration, +and check for any updated drivers. Run CHKDSK /F to check +for hard drive corruption, and then restart your computer. . MessageId=0x24 @@ -395,7 +459,15 @@ Facility=System SymbolicName=DATA_BUS_ERROR Language=English -DATA_BUS_ERROR +Run system diagnostics supplied by your hardware manufacturer. +In particular, run a memory check, and check for faulty or +mismatched memory. Try changing video adapters. + +Check with your hardware vendor for any BIOS updates. Disable +BIOS memory options such as caching or shadowing. If you need +to use Safe Mode to remove or disable components, restart your +computer, press F8 to select Advanced Startup Options, and then +select Safe Mode. . MessageId=0x2F @@ -502,7 +574,12 @@ Facility=System SymbolicName=NO_MORE_SYSTEM_PTES Language=English -NO_MORE_SYSTEM_PTES +Remove any recently installed software including backup +utilities or disk-intensive applications. + +If you need to use Safe Mode to remove or disable components, +restart your computer, press F8 to select Advanced Startup +Options, and then select Safe Mode. . MessageId=0x40 @@ -581,7 +658,13 @@ STREAMS_INTERNAL_ERROR . - +MessageId=0x4C +Severity=Success +Facility=System +SymbolicName=FATAL_UNHANDLED_HARD_ERROR +Language=English +FATAL_UNHANDLED_HARD_ERROR +. MessageId=0x4D Severity=Success @@ -942,7 +1025,11 @@ Facility=System SymbolicName=INACCESSIBLE_BOOT_DEVICE Language=English -INACCESSIBLE_BOOT_DEVICE +Check for viruses on your computer. Remove any newly installed +hard drives or hard drive controllers. Check your hard drive +to make sure it is properly configured and terminated. +Run CHKDSK /F to check for hard drive corruption, and then +restart your computer. . @@ -960,7 +1047,14 @@ Facility=System SymbolicName=UNEXPECTED_KERNEL_MODE_TRAP Language=English -UNEXPECTED_KERNEL_MODE_TRAP +Run a system diagnostic utility supplied by your hardware manufacturer. +In particular, run a memory check, and check for faulty or mismatched +memory. Try changing video adapters. + +Disable or remove any newly installed hardware and drivers. Disable or +remove any newly installed software. If you need to use Safe Mode to +remove or disable components, restart your computer, press F8 to select +Advanced Startup Options, and then select Safe Mode. . MessageId=0x7F @@ -974,6 +1068,14 @@ MessageId=0x80 Severity=Success Facility=System +SymbolicName=KERNEL_MODE_EXCEPTION_NOT_HANDLED +Language=English +KERNEL_MODE_EXCEPTION_NOT_HANDLED +. + +MessageId=0x8E +Severity=Success +Facility=System SymbolicName=SPIN_LOCK_INIT_FAILURE Language=English SPIN_LOCK_INIT_FAILURE @@ -995,6 +1097,22 @@ INVALID_WORK_QUEUE_ITEM . +MessageId=0xA5 +Severity=Success +Facility=System +SymbolicName=ACPI_BIOS_ERROR +Language=English +The BIOS in this system is not fully ACPI compliant. Please contact your +system vendor for an updated BIOS. +. + +MessageId=0xBE +Severity=Success +Facility=System +SymbolicName=ATTEMPTED_WRITE_TO_READONLY_MEMORY +Language=English +ATTEMPTED_WRITE_TO_READONLY_MEMORY + MessageId=0xC2 Severity=Success Facility=System @@ -1003,6 +1121,110 @@ BAD_POOL_CALLER . +MessageId=0xC3 +Severity=Success +Facility=System +SymbolicName=BUGCODE_PSS_MESSAGE_SIGNATURE +Language=English +A system file that is owned by ReactOS was replaced by an application +running on your system. The operating system detected this and tried to +verify the validity of the file's signature. The operating system found that +the file signature is not valid and put the original, correct file back +so that your operating system will continue to function properly. +. + +MessageId=0xC5 +Severity=Success +Facility=System +SymbolicName=DRIVER_CORRUPTED_EXPOOL +Language=English +A device driver has pool. + +Check to make sure any new hardware or software is properly installed. +If this is a new installation, ask your hardware or software manufacturer +for any ReactOS updates you might need. + +Run the driver verifier against any new (or suspect) drivers. +If that doesn't reveal the corrupting driver, try enabling special pool. +Both of these features are intended to catch the corruption at an earlier +point where the offending driver can be identified. + +If you need to use Safe Mode to remove or disable components, +restart your computer, press F8 to select Advanced Startup Options, +and then select Safe Mode. +. + +MessageId=0xCB +Severity=Success +Facility=System +SymbolicName=DRIVER_LEFT_LOCKED_PAGES_IN_PROCESS +Language=English +DRIVER_LEFT_LOCKED_PAGES_IN_PROCESS +. + +MessageId=0xCE +Severity=Success +Facility=System +SymbolicName=DRIVER_UNLOADED_WITHOUT_CANCELLING_PENDING_OPERATIONS +Language=English +DRIVER_UNLOADED_WITHOUT_CANCELLING_PENDING_OPERATIONS +. + +MessageId=0xD0 +Severity=Success +Facility=System +SymbolicName=DRIVER_CORRUPTED_MMPOOL +Language=English +DRIVER_CORRUPTED_MMPOOL +. + +MessageId=0xD1 +Severity=Success +Facility=System +SymbolicName=DRIVER_IRQL_NOT_LESS_OR_EQUAL +Language=English +DRIVER_IRQL_NOT_LESS_OR_EQUAL +. + +MessageId=0xD3 +Severity=Success +Facility=System +SymbolicName=DRIVER_PORTION_MUST_BE_NONPAGED +Language=English +DRIVER_PORTION_MUST_BE_NONPAGED +. + +MessageId=0xD8 +Severity=Success +Facility=System +SymbolicName=DRIVER_USED_EXCESSIVE_PTES +Language=English +DRIVER_USED_EXCESSIVE_PTES +. + +MessageId=0xD4 +Severity=Success +Facility=System +SymbolicName=SYSTEM_SCAN_AT_RAISED_IRQL_CAUGHT_IMPROPER_DRIVER_UNLOAD +Language=English +SYSTEM_SCAN_AT_RAISED_IRQL_CAUGHT_IMPROPER_DRIVER_UNLOAD +. + +MessageId=0xE0 +Severity=Success +Facility=System +SymbolicName=ACPI_BIOS_FATAL_ERROR +Language=English + +Your computer (BIOS) has reported that a component in your system is faulty and +has prevented ReactOS from operating. You can determine which component is +faulty by running the diagnostic disk or tool that came with your computer. + +If you do not have this tool, you must contact your system vendor and report +this error message to them. They will be able to assist you in correcting this +hardware problem thereby allowing ReactOS to operate. +. + MessageId=0xE1 Severity=Success Facility=System @@ -1033,6 +1255,14 @@ SymbolicName=WORKER_INVALID Language=English WORKER_INVALID +. + +MessageId=0xE5 +Severity=Success +Facility=System +SymbolicName=POWER_FAILURE_SIMULATE +Language=English +POWER_FAILURE_SIMULATE . MessageId=0xFA @@ -1101,6 +1331,19 @@ ACTIVE_EX_WORKER_THREAD_TERMINATION . +MessageId=0xEA +Severity=Success +Facility=System +SymbolicName=THREAD_STUCK_IN_DEVICE_DRIVER +Language=English + +The device driver got stuck in an infinite loop. This usually indicates +problem with the device itself or with the device driver programming the +hardware incorrectly. + +Please check with your hardware device vendor for any driver updates. +. + MessageId=0xEF Severity=Success Facility=System

1 0

[ion] 23793: - Jesus Christ... will Hell freeze over before someone finally listens and implements a proper PE loader in Freeloader?
by ion＠svn.reactos.org 30 Aug '06

30 Aug '06

Author: ion Date: Wed Aug 30 05:22:41 2006 New Revision: 23793 URL: http://svn.reactos.org/svn/reactos?rev=23793&view=rev Log: - Jesus Christ... will Hell freeze over before someone finally listens and implements a proper PE loader in Freeloader? Modified: trunk/reactos/ntoskrnl/ke/main.c Modified: trunk/reactos/ntoskrnl/ke/main.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/main.c?rev=237… ============================================================================== --- trunk/reactos/ntoskrnl/ke/main.c (original) +++ trunk/reactos/ntoskrnl/ke/main.c Wed Aug 30 05:22:41 2006 @@ -71,6 +71,8 @@ #if defined (ALLOC_PRAGMA) #pragma alloc_text(INIT, _main) #endif + +extern LDR_DATA_TABLE_ENTRY HalModuleObject; /* FUNCTIONS ****************************************************************/ @@ -240,6 +242,17 @@ (PVOID)KERNEL_BASE, &DriverSize); + // + // + // HACK HACK HACK WHEN WILL YOU PEOPLE FIX FREELDR?!?!?! + // FREELDR SENDS US AN ***INVALID*** HAL PE HEADER!!! + // WE READ IT IN LdrInitModuleManagement ABOVE!!! + // WE SET .SizeOfImage TO A *GARBAGE* VALUE!!! + // + // This dirty hack fixes it, and should make symbol lookup work too. + // + HalModuleObject.SizeOfImage = RtlImageNtHeader((PVOID)HalModuleObject.DllBase)->OptionalHeader.SizeOfImage; + /* Increase the last kernel address with the size of HAL */ LastKernelAddress += PAGE_ROUND_UP(DriverSize);

1 0

[hpoussin] 23792: Implement RegNtPreEnumerateValueKey/RegNtPostEnumerateValueKey registry callbacks. Add missing callbacks in some paths Fix a few typos
by hpoussin＠svn.reactos.org 29 Aug '06

29 Aug '06

Author: hpoussin Date: Wed Aug 30 03:01:52 2006 New Revision: 23792 URL: http://svn.reactos.org/svn/reactos?rev=23792&view=rev Log: Implement RegNtPreEnumerateValueKey/RegNtPostEnumerateValueKey registry callbacks. Add missing callbacks in some paths Fix a few typos Modified: trunk/reactos/ntoskrnl/cm/ntfunc.c Modified: trunk/reactos/ntoskrnl/cm/ntfunc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/cm/ntfunc.c?rev=2… ============================================================================== --- trunk/reactos/ntoskrnl/cm/ntfunc.c (original) +++ trunk/reactos/ntoskrnl/cm/ntfunc.c Wed Aug 30 03:01:52 2006 @@ -359,6 +359,9 @@ Status = CmiCallRegisteredCallbacks(RegNtPreCreateKey, &PreCreateKeyInfo); if (!NT_SUCCESS(Status)) { + PostCreateKeyInfo.Object = NULL; + PostCreateKeyInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostCreateKey, &PostCreateKeyInfo); goto Cleanup; } @@ -612,11 +615,11 @@ PostOperationInfo.Object = (PVOID)KeyObject; DeleteKeyInfo.Object = (PVOID)KeyObject; - Status = CmiCallRegisteredCallbacks(RegNtPreSetValueKey, &DeleteKeyInfo); + Status = CmiCallRegisteredCallbacks(RegNtPreDeleteKey, &DeleteKeyInfo); if (!NT_SUCCESS(Status)) { PostOperationInfo.Status = Status; - CmiCallRegisteredCallbacks(RegNtDeleteKey, &PostOperationInfo); + CmiCallRegisteredCallbacks(RegNtPostDeleteKey, &PostOperationInfo); ObDereferenceObject(KeyObject); return Status; } @@ -728,9 +731,11 @@ EnumerateKeyInfo.Length = Length; EnumerateKeyInfo.ResultLength = ResultLength; - Status = CmiCallRegisteredCallbacks(RegNtEnumerateKey, &EnumerateKeyInfo); - if (!NT_SUCCESS(Status)) - { + Status = CmiCallRegisteredCallbacks(RegNtPreEnumerateKey, &EnumerateKeyInfo); + if (!NT_SUCCESS(Status)) + { + PostOperationInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostEnumerateKey, &PostOperationInfo); ObDereferenceObject(KeyObject); return Status; } @@ -752,7 +757,7 @@ ExReleaseResourceLite(&CmiRegistryLock); KeLeaveCriticalRegion(); PostOperationInfo.Status = STATUS_NO_MORE_ENTRIES; - CmiCallRegisteredCallbacks(RegNtPostDeleteKey, &PostOperationInfo); + CmiCallRegisteredCallbacks(RegNtPostEnumerateKey, &PostOperationInfo); ObDereferenceObject(KeyObject); DPRINT("No more volatile entries\n"); return STATUS_NO_MORE_ENTRIES; @@ -990,6 +995,8 @@ PCM_KEY_VALUE ValueCell; PVOID DataCell; ULONG NameSize, DataSize; + REG_ENUMERATE_VALUE_KEY_INFORMATION EnumerateValueKeyInfo; + REG_POST_OPERATION_INFORMATION PostOperationInfo; PKEY_VALUE_BASIC_INFORMATION ValueBasicInformation; PKEY_VALUE_PARTIAL_INFORMATION ValuePartialInformation; PKEY_VALUE_FULL_INFORMATION ValueFullInformation; @@ -1017,6 +1024,23 @@ return Status; } + PostOperationInfo.Object = (PVOID)KeyObject; + EnumerateValueKeyInfo.Object = (PVOID)KeyObject; + EnumerateValueKeyInfo.Index = Index; + EnumerateValueKeyInfo.KeyValueInformationClass = KeyValueInformationClass; + EnumerateValueKeyInfo.KeyValueInformation = KeyValueInformation; + EnumerateValueKeyInfo.Length = Length; + EnumerateValueKeyInfo.ResultLength = ResultLength; + + Status = CmiCallRegisteredCallbacks(RegNtPreEnumerateValueKey, &EnumerateValueKeyInfo); + if (!NT_SUCCESS(Status)) + { + PostOperationInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostEnumerateValueKey, &PostOperationInfo); + ObDereferenceObject(KeyObject); + return Status; + } + /* Acquire hive lock */ KeEnterCriticalRegion(); ExAcquireResourceSharedLite(&CmiRegistryLock, TRUE); @@ -1038,6 +1062,8 @@ ExReleaseResourceLite(&CmiRegistryLock); KeLeaveCriticalRegion(); ObDereferenceObject(KeyObject); + PostOperationInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostEnumerateValueKey, &PostOperationInfo); return Status; } @@ -1217,6 +1243,8 @@ ExReleaseResourceLite(&CmiRegistryLock); KeLeaveCriticalRegion(); ObDereferenceObject(KeyObject); + PostOperationInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostEnumerateValueKey, &PostOperationInfo); return Status; } @@ -1351,6 +1379,9 @@ Status = CmiCallRegisteredCallbacks(RegNtPreOpenKey, &PreOpenKeyInfo); if (!NT_SUCCESS(Status)) { + PostOpenKeyInfo.Object = NULL; + PostOpenKeyInfo.Status = Status; + CmiCallRegisteredCallbacks (RegNtPostOpenKey, &PostOpenKeyInfo); ObpReleaseCapturedAttributes(&ObjectCreateInfo); if (ObjectName.Buffer) ObpReleaseCapturedName(&ObjectName); return Status; @@ -1475,9 +1506,11 @@ QueryKeyInfo.Length = Length; QueryKeyInfo.ResultLength = ResultLength; - Status = CmiCallRegisteredCallbacks(RegNtQueryKey, &QueryKeyInfo); - if (!NT_SUCCESS(Status)) - { + Status = CmiCallRegisteredCallbacks(RegNtPreQueryKey, &QueryKeyInfo); + if (!NT_SUCCESS(Status)) + { + PostOperationInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostQueryKey, &PostOperationInfo); ObDereferenceObject(KeyObject); return Status; } @@ -1704,6 +1737,8 @@ Status = CmiCallRegisteredCallbacks(RegNtPreQueryValueKey, &QueryValueKeyInfo); if (!NT_SUCCESS(Status)) { + PostOperationInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostQueryValueKey, &PostOperationInfo); ObDereferenceObject(KeyObject); return Status; } @@ -1962,6 +1997,8 @@ Status = CmiCallRegisteredCallbacks(RegNtPreSetValueKey, &SetValueKeyInfo); if (!NT_SUCCESS(Status)) { + PostOperationInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostSetValueKey, &PostOperationInfo); ObDereferenceObject(KeyObject); return Status; } @@ -2141,6 +2178,9 @@ Status = CmiCallRegisteredCallbacks(RegNtPreDeleteValueKey, &DeleteValueKeyInfo); if (!NT_SUCCESS(Status)) { + PostOperationInfo.Object = (PVOID)KeyObject; + PostOperationInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostDeleteValueKey, &PostOperationInfo); ReleaseCapturedUnicodeString(&CapturedValueName, PreviousMode); Fail: @@ -2411,6 +2451,8 @@ Status = CmiCallRegisteredCallbacks(RegNtPreQueryMultipleValueKey, &QueryMultipleValueKeyInfo); if (!NT_SUCCESS(Status)) { + PostOperationInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostQueryMultipleValueKey, &PostOperationInfo); ObDereferenceObject(KeyObject); return Status; } @@ -2658,9 +2700,11 @@ SetInformationKeyInfo.KeySetInformation = KeyInformation; SetInformationKeyInfo.KeySetInformationLength = KeyInformationLength; - Status = CmiCallRegisteredCallbacks(RegNtSetInformationKey, &SetInformationKeyInfo); - if (!NT_SUCCESS(Status)) - { + Status = CmiCallRegisteredCallbacks(RegNtPreSetInformationKey, &SetInformationKeyInfo); + if (!NT_SUCCESS(Status)) + { + PostOperationInfo.Status = Status; + CmiCallRegisteredCallbacks(RegNtPostSetInformationKey, &PostOperationInfo); ObDereferenceObject (KeyObject); return Status; }

1 0

[turner] 23791: Better rand() implementation. This should fix the problem where you get 0 every time you use rand()
by turner＠svn.reactos.org 29 Aug '06

29 Aug '06

Author: turner Date: Wed Aug 30 02:27:48 2006 New Revision: 23791 URL: http://svn.reactos.org/svn/reactos?rev=23791&view=rev Log: Better rand() implementation. This should fix the problem where you get 0 every time you use rand() Modified: trunk/reactos/lib/crt/misc/tls.c trunk/reactos/lib/crt/stdlib/rand.c Modified: trunk/reactos/lib/crt/misc/tls.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/crt/misc/tls.c?rev=237… ============================================================================== --- trunk/reactos/lib/crt/misc/tls.c (original) +++ trunk/reactos/lib/crt/misc/tls.c Wed Aug 30 02:27:48 2006 @@ -13,6 +13,8 @@ ThreadData->tdoserrno = 0; ThreadData->fpecode = 0; + + ThreadData->tnext = 1; /* FIXME: init more thread local data */ Modified: trunk/reactos/lib/crt/stdlib/rand.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/crt/stdlib/rand.c?rev=… ============================================================================== --- trunk/reactos/lib/crt/stdlib/rand.c (original) +++ trunk/reactos/lib/crt/stdlib/rand.c Wed Aug 30 02:27:48 2006 @@ -11,7 +11,7 @@ { PTHREADDATA ThreadData = GetThreadData(); - ThreadData->tnext = ThreadData->tnext * 0x5deece66dLL + 11; + ThreadData->tnext = ThreadData->tnext * 0x5deece66dLL + 2531011; return (int)((ThreadData->tnext >> 16) & RAND_MAX); }

1 0

← Newer
1
2
3
4
5
6
7
8
9
...
45
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs August 2006