Ros-diffs November 2009

ros-diffs@reactos.org

23 participants
444 discussions

[sginsberg] 43915: - Fix leftover from 43912: Add KdRefreshDebuggerNotPresent to DDK.

by sginsberg＠svn.reactos.org

Author: sginsberg Date: Mon Nov 2 19:16:12 2009 New Revision: 43915 URL: http://svn.reactos.org/svn/reactos?rev=43915&view=rev Log: - Fix leftover from 43912: Add KdRefreshDebuggerNotPresent to DDK. Modified: trunk/reactos/include/ddk/winddk.h Modified: trunk/reactos/include/ddk/winddk.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ddk/winddk.h?rev=4… ============================================================================== --- trunk/reactos/include/ddk/winddk.h [iso-8859-1] (original) +++ trunk/reactos/include/ddk/winddk.h [iso-8859-1] Mon Nov 2 19:16:12 2009 @@ -10328,6 +10328,13 @@ KdEnableDebugger( VOID); +NTKERNELAPI +BOOLEAN +NTAPI +KdRefreshDebuggerNotPresent( + VOID +); + #if (NTDDI_VERSION >= NTDDI_WS03SP1) NTKERNELAPI NTSTATUS

15 years, 1 month

[sginsberg] 43914: - Fix incorrect comment, spotted by Timo. - Purpouses -> Purposes.

by sginsberg＠svn.reactos.org

Author: sginsberg Date: Mon Nov 2 19:08:49 2009 New Revision: 43914 URL: http://svn.reactos.org/svn/reactos?rev=43914&view=rev Log: - Fix incorrect comment, spotted by Timo. - Purpouses -> Purposes. Modified: trunk/reactos/ntoskrnl/kd64/kdapi.c trunk/reactos/ntoskrnl/ke/i386/irqobj.c Modified: trunk/reactos/ntoskrnl/kd64/kdapi.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd64/kdapi.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/kd64/kdapi.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd64/kdapi.c [iso-8859-1] Mon Nov 2 19:08:49 2009 @@ -671,7 +671,7 @@ ProcessorState.ContextFrame; } - /* Copy it over */ + /* Copy it over to the debugger */ RtlCopyMemory(Data->Buffer, TargetContext, sizeof(CONTEXT)); Data->Length = sizeof(CONTEXT); @@ -724,7 +724,7 @@ ProcessorState.ContextFrame; } - /* Get the context from the PRCB array */ + /* Copy the new context to it */ RtlCopyMemory(TargetContext, Data->Buffer, sizeof(CONTEXT)); /* Finish up */ Modified: trunk/reactos/ntoskrnl/ke/i386/irqobj.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/i386/irqobj.c?… ============================================================================== --- trunk/reactos/ntoskrnl/ke/i386/irqobj.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/ke/i386/irqobj.c [iso-8859-1] Mon Nov 2 19:08:49 2009 @@ -3,7 +3,7 @@ * LICENSE: GPL - See COPYING in the top level directory * FILE: ntoskrnl/ke/i386/irq.c * PURPOSE: Manages the Kernel's IRQ support for external drivers, - * for the purpouses of connecting, disconnecting and setting + * for the purposes of connecting, disconnecting and setting * up ISRs for drivers. The backend behind the Io* Interrupt * routines. * PROGRAMMERS: Alex Ionescu (alex.ionescu(a)reactos.org)

15 years, 1 month

[sginsberg] 43913: - Fix broken HalpQueryPciRegistryInfo: If it wasn't for the fact that it queried a subkey incorrectly ('ConfigurationData' instead of 'Configuration Data') it was setting up the basic structure correctly but forgot to RETURN IT TO THE CALLER. - Fixes PCI detection detection as HalpInitializePciStubs no longer has to rely on a 'no info = PCI 1' assumption, making PCI 2 detection possible. - Add support for querying the CurrentControlSet\Control\PnP\PCI\CardList key and retrievi

by sginsberg＠svn.reactos.org

Author: sginsberg Date: Mon Nov 2 19:05:07 2009 New Revision: 43913 URL: http://svn.reactos.org/svn/reactos?rev=43913&view=rev Log: - Fix broken HalpQueryPciRegistryInfo: If it wasn't for the fact that it queried a subkey incorrectly ('ConfigurationData' instead of 'Configuration Data') it was setting up the basic structure correctly but forgot to RETURN IT TO THE CALLER. - Fixes PCI detection detection as HalpInitializePciStubs no longer has to rely on a 'no info = PCI 1' assumption, making PCI 2 detection possible. - Add support for querying the CurrentControlSet\Control\PnP\PCI\CardList key and retrieving the card list entries (not used yet). Also add the related keys to hivesys.inf so there is something to query. Modified: trunk/reactos/boot/bootdata/hivesys_i386.inf trunk/reactos/hal/halx86/generic/bus/pcibus.c Modified: trunk/reactos/boot/bootdata/hivesys_i386.inf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/bootdata/hivesys_i386… ============================================================================== --- trunk/reactos/boot/bootdata/hivesys_i386.inf [iso-8859-1] (original) +++ trunk/reactos/boot/bootdata/hivesys_i386.inf [iso-8859-1] Mon Nov 2 19:05:07 2009 @@ -782,6 +782,29 @@ HKLM,"SYSTEM\CurrentControlSet\Control\NLS\Locale\Alternate Sorts","00021404",0x00000000,"" HKLM,"SYSTEM\CurrentControlSet\Control\NLS\Locale\Alternate Sorts","00030404",0x00000000,"" +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","Intel 82441FX",0x00030003,\ +01,00,00,00,86,80,37,12,00,00,00,00,00,00,00,00 +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","Intel 82439HX",0x00030003,\ +01,00,00,00,86,80,50,12,00,00,00,00,00,00,00,00 +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","Intel 82439TX",0x00030003,\ +01,00,00,00,86,80,00,71,00,00,00,00,00,00,00,00 +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","Intel 82443LX/EX",0x00030003,\ +01,00,00,00,86,80,80,71,00,00,00,00,00,00,00,00 +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","Intel 82443BX1",0x00030003,\ +01,00,00,00,86,80,90,71,00,00,00,00,00,00,00,00 +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","Intel 82443BX2",0x00030003,\ +01,00,00,00,86,80,92,71,00,00,00,00,00,00,00,00 +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","Intel 82443GX",0x00030003,\ +01,00,00,00,86,80,a0,71,00,00,00,00,00,00,00,00 +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","VIA Tech1",0x00030003,\ +01,00,00,00,06,11,01,05,00,00,00,00,00,00,00,00 +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","VIA Tech2",0x00030003,\ +01,00,00,00,06,11,91,06,00,00,00,00,00,00,00,00 +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","Toshiba",0x00030003,\ +01,00,00,00,79,11,01,06,00,00,00,00,00,00,00,00 +HKLM,"SYSTEM\CurrentControlSet\Control\PnP\Pci\CardList","Ali",0x00030003,\ +01,00,00,00,b9,10,21,15,00,00,00,00,00,00,00,00 + HKLM,"SYSTEM\CurrentControlSet\Control\Print","MajorVersion",0x00010001,2 HKLM,"SYSTEM\CurrentControlSet\Control\Print","MinorVersion",0x00010003,0 HKLM,"SYSTEM\CurrentControlSet\Control\Print","PriorityClass",0x00010003,0 Modified: trunk/reactos/hal/halx86/generic/bus/pcibus.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/hal/halx86/generic/bus/pci… ============================================================================== --- trunk/reactos/hal/halx86/generic/bus/pcibus.c [iso-8859-1] (original) +++ trunk/reactos/hal/halx86/generic/bus/pcibus.c [iso-8859-1] Mon Nov 2 19:05:07 2009 @@ -703,19 +703,24 @@ HalpQueryPciRegistryInfo(VOID) { WCHAR NameBuffer[8]; - OBJECT_ATTRIBUTES ObjectAttributes; + OBJECT_ATTRIBUTES ObjectAttributes; UNICODE_STRING KeyName, ConfigName, IdentName; - HANDLE KeyHandle, BusKeyHandle; + HANDLE KeyHandle, BusKeyHandle, CardListHandle; NTSTATUS Status; - UCHAR KeyBuffer[sizeof(PPCI_REGISTRY_INFO) + 100]; + UCHAR KeyBuffer[sizeof(CM_FULL_RESOURCE_DESCRIPTOR) + 100]; PKEY_VALUE_FULL_INFORMATION ValueInfo = (PVOID)KeyBuffer; + UCHAR PartialKeyBuffer[sizeof(KEY_VALUE_PARTIAL_INFORMATION) + + sizeof(PCI_CARD_DESCRIPTOR)]; + PKEY_VALUE_PARTIAL_INFORMATION PartialValueInfo = (PVOID)PartialKeyBuffer; + KEY_FULL_INFORMATION KeyInformation; ULONG ResultLength; PWSTR Tag; - ULONG i; + ULONG i, ElementCount; PCM_FULL_RESOURCE_DESCRIPTOR FullDescriptor; PCM_PARTIAL_RESOURCE_DESCRIPTOR PartialDescriptor; PPCI_REGISTRY_INFO PciRegInfo; PPCI_REGISTRY_INFO_INTERNAL PciRegistryInfo; + PPCI_CARD_DESCRIPTOR CardDescriptor; /* Setup the object attributes for the key */ RtlInitUnicodeString(&KeyName, @@ -736,7 +741,7 @@ KeyName.MaximumLength = sizeof(NameBuffer); /* Setup the configuration and identifier key names */ - RtlInitUnicodeString(&ConfigName, L"ConfigurationData"); + RtlInitUnicodeString(&ConfigName, L"Configuration Data"); RtlInitUnicodeString(&IdentName, L"Identifier"); /* Keep looping for each ID */ @@ -805,28 +810,120 @@ /* Check if this is our PCI Registry Information */ if (PartialDescriptor->Type == CmResourceTypeDeviceSpecific) { - /* Close the key */ - ZwClose(KeyHandle); - - /* FIXME: Check PnP\PCI\CardList */ - - /* Get the PCI information */ - PciRegInfo = (PPCI_REGISTRY_INFO)(PartialDescriptor + 1); - - /* Allocate the return structure */ - PciRegistryInfo = ExAllocatePoolWithTag(NonPagedPool, - sizeof(PCI_REGISTRY_INFO_INTERNAL), - ' laH'); - if (!PciRegistryInfo) return NULL; - - /* Fill it out */ - PciRegistryInfo->HardwareMechanism = PciRegInfo->HardwareMechanism; - PciRegistryInfo->NoBuses = PciRegInfo->NoBuses; - PciRegistryInfo->MajorRevision = PciRegInfo->MajorRevision; - PciRegistryInfo->MinorRevision = PciRegInfo->MinorRevision; - PciRegistryInfo->ElementCount = 0; - } - } + /* It is, stop searching */ + break; + } + } + + /* Close the key */ + ZwClose(KeyHandle); + + /* Save the PCI information for later */ + PciRegInfo = (PPCI_REGISTRY_INFO)(PartialDescriptor + 1); + + /* Assume no Card List entries */ + ElementCount = 0; + + /* Set up for checking the PCI Card List key */ + RtlInitUnicodeString(&KeyName, + L"\\Registry\\Machine\\System\\CurrentControlSet\\" + L"Control\\PnP\\PCI\\CardList"); + InitializeObjectAttributes(&ObjectAttributes, + &KeyName, + OBJ_CASE_INSENSITIVE, + NULL, + NULL); + + /* Attempt to open it */ + Status = ZwOpenKey(&CardListHandle, KEY_READ, &ObjectAttributes); + if (NT_SUCCESS(Status)) + { + /* It exists, so let's query it */ + Status = ZwQueryKey(CardListHandle, + KeyFullInformation, + &KeyInformation, + sizeof(KEY_FULL_INFORMATION), + &ResultLength); + if (!NT_SUCCESS(Status)) + { + /* Failed to query, so no info */ + PciRegistryInfo = NULL; + } + else + { + /* Allocate the full structure */ + PciRegistryInfo = + ExAllocatePoolWithTag(NonPagedPool, + sizeof(PCI_REGISTRY_INFO_INTERNAL) + + (KeyInformation.Values * + sizeof(PCI_CARD_DESCRIPTOR)), + ' laH'); + if (PciRegistryInfo) + { + /* Get the first card descriptor entry */ + CardDescriptor = (PPCI_CARD_DESCRIPTOR)(PciRegistryInfo + 1); + + /* Loop all the values */ + for (i = 0; i < KeyInformation.Values; i++) + { + /* Attempt to get the value */ + Status = ZwEnumerateValueKey(CardListHandle, + i, + KeyValuePartialInformation, + PartialValueInfo, + sizeof(PartialKeyBuffer), + &ResultLength); + if (!NT_SUCCESS(Status)) + { + /* Something went wrong, stop the search */ + break; + } + + /* Make sure it is correctly sized */ + if (PartialValueInfo->DataLength == sizeof(PCI_CARD_DESCRIPTOR)) + { + /* Sure is, copy it over */ + *CardDescriptor = *(PPCI_CARD_DESCRIPTOR) + PartialValueInfo->Data; + + /* One more Card List entry */ + ElementCount++; + + /* Move to the next descriptor */ + CardDescriptor = (CardDescriptor + 1); + } + } + } + } + + /* Close the Card List key */ + ZwClose(CardListHandle); + } + else + { + /* No key, no Card List */ + PciRegistryInfo = NULL; + } + + /* Check if we failed to get the full structure */ + if (!PciRegistryInfo) + { + /* Just allocate the basic structure then */ + PciRegistryInfo = ExAllocatePoolWithTag(NonPagedPool, + sizeof(PCI_REGISTRY_INFO_INTERNAL), + ' laH'); + if (!PciRegistryInfo) return NULL; + } + + /* Save the info we got */ + PciRegistryInfo->MajorRevision = PciRegInfo->MajorRevision; + PciRegistryInfo->MinorRevision = PciRegInfo->MinorRevision; + PciRegistryInfo->NoBuses = PciRegInfo->NoBuses; + PciRegistryInfo->HardwareMechanism = PciRegInfo->HardwareMechanism; + PciRegistryInfo->ElementCount = ElementCount; + + /* Return it */ + return PciRegistryInfo; } VOID @@ -893,7 +990,7 @@ default: /* Invalid type */ - DbgPrint("HAL: Unnkown PCI type\n"); + DbgPrint("HAL: Unknown PCI type\n"); } /* Loop all possible buses */

15 years, 1 month

[sginsberg] 43912: - Fix support for /CRASHDEBUG and /NODEBUG; we didn't respect those settings properly and would initialize KD at boot even if they were set. - Re-enable the breakpoint in vDbgPrintExWithPrefixInternal() as this works properly now. Without this breakpoint some break-in requests got lost if the break-in occurred when handling a debug print (happened a lot at boot). - Implement Command String support for DbgCommandString() -- we now handle every debug service call. - Implement Nt

by sginsberg＠svn.reactos.org

Author: sginsberg Date: Mon Nov 2 18:45:51 2009 New Revision: 43912 URL: http://svn.reactos.org/svn/reactos?rev=43912&view=rev Log: - Fix support for /CRASHDEBUG and /NODEBUG; we didn't respect those settings properly and would initialize KD at boot even if they were set. - Re-enable the breakpoint in vDbgPrintExWithPrefixInternal() as this works properly now. Without this breakpoint some break-in requests got lost if the break-in occurred when handling a debug print (happened a lot at boot). - Implement Command String support for DbgCommandString() -- we now handle every debug service call. - Implement NtSetDebugFilterState() and NtQueryDebugFilterState() for KD, meaning we now support debug filters properly. - Implement KdRefreshDebuggerNotPresent(), KdChangeOption() and KdPowerTransition(). Stub KdSystemDebugControl() to return error status instead of hanging the system. - Stub the rest of the KD API to print a warning and return a failure packet instead of hanging. - Set and respect KdpContextSent when getting and setting the thread context -- WinDbg doesn't seem to rely on this, but better safe than sorry. - Support MP when getting and setting the thread context too -- if the context is operation is for another processor than the current, just get it through the KiProcessorBlock array. - Initialize the MajorVersion in the KD version block more properly -- the high byte is the major identifier (0 for NT). Add the required DBGKD_MAJOR_TYPES enumeration to wdbgexts.h. - Simplify setting and clearing the InDbgPrint flag in the TEB to minimize the impact on kernel execution; use 2 dedicated routines instead of a generic one. - KdpSymbol doesn't return anything, so don't return an ignore status from KdpReportLoadSymbolsStateChange. - Expose the KdpDefaultRetries and Kd_WIN2000_Mask variables to the registry and add them to KDBG too (unused there). - No reason to implement KdpSysGetVersion per architecture; move it back to the generic code. - Add some ARM offsets to the debugger data block that (N/A on other architectures). - Fix the default size of the DbgPrint log buffer for free builds to save some space. It should be 4 KB for a free build and 32 KB for a checked build. - Move KeDisableInterrupts to cpu.c as it fits here more than in the IRQ support code in irqobj.c. - Use KeDisableInterrupts in KeFreezeExecution instead of checking the x86 EFLAG directly. Modified: trunk/reactos/dll/ntdll/rtl/libsupp.c trunk/reactos/include/ndk/rtlfuncs.h trunk/reactos/include/psdk/wdbgexts.h trunk/reactos/lib/rtl/debug.c trunk/reactos/lib/rtl/rtlp.h trunk/reactos/ntoskrnl/config/cmdata.c trunk/reactos/ntoskrnl/include/internal/kd.h trunk/reactos/ntoskrnl/include/internal/kd64.h trunk/reactos/ntoskrnl/kd/kdmain.c trunk/reactos/ntoskrnl/kd64/amd64/kdsup.c trunk/reactos/ntoskrnl/kd64/arm/kdsup.c trunk/reactos/ntoskrnl/kd64/i386/kdsup.c trunk/reactos/ntoskrnl/kd64/kdapi.c trunk/reactos/ntoskrnl/kd64/kddata.c trunk/reactos/ntoskrnl/kd64/kdinit.c trunk/reactos/ntoskrnl/kd64/kdlock.c trunk/reactos/ntoskrnl/kd64/kdprint.c trunk/reactos/ntoskrnl/kd64/kdtrap.c trunk/reactos/ntoskrnl/ke/i386/cpu.c trunk/reactos/ntoskrnl/ke/i386/irqobj.c trunk/reactos/ntoskrnl/ntoskrnl.pspec 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 [iso-8859-1] (original) +++ trunk/reactos/dll/ntdll/rtl/libsupp.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -41,21 +41,22 @@ BOOLEAN NTAPI -RtlpSetInDbgPrint(IN BOOLEAN NewValue) -{ - /* If we're setting it to false, do it and return */ - if (NewValue == FALSE) - { - NtCurrentTeb()->InDbgPrint = FALSE; - return FALSE; - } - - /* Setting to true; check if it's not already */ +RtlpSetInDbgPrint(VOID) +{ + /* Check if it's already set and return TRUE if so */ if (NtCurrentTeb()->InDbgPrint) return TRUE; /* Set it and return */ NtCurrentTeb()->InDbgPrint = TRUE; return FALSE; +} + +VOID +NTAPI +RtlpClearInDbgPrint(VOID) +{ + /* Clear the flag */ + NtCurrentTeb()->InDbgPrint = FALSE; } KPROCESSOR_MODE Modified: trunk/reactos/include/ndk/rtlfuncs.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ndk/rtlfuncs.h?rev… ============================================================================== --- trunk/reactos/include/ndk/rtlfuncs.h [iso-8859-1] (original) +++ trunk/reactos/include/ndk/rtlfuncs.h [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -2682,6 +2682,13 @@ IN ULONG_PTR ProcessId ); +VOID +NTAPI +DbgCommandString( + IN PCCH Name, + IN PCCH Command +); + // // Generic Table Functions // Modified: trunk/reactos/include/psdk/wdbgexts.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/wdbgexts.h?re… ============================================================================== --- trunk/reactos/include/psdk/wdbgexts.h [iso-8859-1] (original) +++ trunk/reactos/include/psdk/wdbgexts.h [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -26,6 +26,26 @@ #define DBGKD_VERS_FLAG_PARTITIONS 0x0020 #define KDBG_TAG 'GBDK' + +typedef enum _DBGKD_MAJOR_TYPES +{ + DBGKD_MAJOR_NT, + DBGKD_MAJOR_XBOX, + DBGKD_MAJOR_BIG, + DBGKD_MAJOR_EXDI, + DBGKD_MAJOR_NTBD, + DBGKD_MAJOR_EFI, + DBGKD_MAJOR_TNT, + DBGKD_MAJOR_SINGULARITY, + DBGKD_MAJOR_HYPERVISOR, + DBGKD_MAJOR_COUNT +} DBGKD_MAJOR_TYPES; + +// +// The major type is in the high byte +// +#define DBGKD_MAJOR_TYPE(MajorVersion) \ + ((DBGKD_MAJOR_TYPES)((MajorVersion) >> 8)) typedef struct _DBGKD_GET_VERSION32 { Modified: trunk/reactos/lib/rtl/debug.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/debug.c?rev=43912&… ============================================================================== --- trunk/reactos/lib/rtl/debug.c [iso-8859-1] (original) +++ trunk/reactos/lib/rtl/debug.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -69,7 +69,7 @@ } /* For user mode, don't recursively DbgPrint */ - if (RtlpSetInDbgPrint(TRUE)) return STATUS_SUCCESS; + if (RtlpSetInDbgPrint()) return STATUS_SUCCESS; /* Guard against incorrect pointers */ _SEH2_TRY @@ -142,13 +142,13 @@ if (Status == STATUS_BREAKPOINT) { /* Breakpoint */ - //DbgBreakPointWithStatus(DBG_STATUS_CONTROL_C); + DbgBreakPointWithStatus(DBG_STATUS_CONTROL_C); Status = STATUS_SUCCESS; } } - /* In user-mode, remove the InDbgPrint Flag */ - RtlpSetInDbgPrint(FALSE); + /* In user-mode, clear the InDbgPrint Flag */ + RtlpClearInDbgPrint(); /* Return */ return Status; @@ -282,7 +282,7 @@ /* Setup the output string */ Output.Length = strlen(Prompt); - Output.Buffer = (PCH)Prompt; + Output.Buffer = (PCHAR)Prompt; /* Call the system service */ return DebugPrompt(&Output, &Input); @@ -340,7 +340,8 @@ else { /* No data available */ - SymbolInfo.CheckSum = SymbolInfo.SizeOfImage = 0; + SymbolInfo.CheckSum = + SymbolInfo.SizeOfImage = 0; } /* Load the symbols */ @@ -366,3 +367,23 @@ /* Load the symbols */ DebugService2(Name, &SymbolInfo, BREAKPOINT_UNLOAD_SYMBOLS); } + +/* + * @implemented + */ +VOID +NTAPI +DbgCommandString(IN PCCH Name, + IN PCCH Command) +{ + STRING NameString, CommandString; + + /* Setup the strings */ + NameString.Buffer = (PCHAR)Name; + NameString.Length = strlen(Name); + CommandString.Buffer = (PCHAR)Command; + CommandString.Length = strlen(Command); + + /* Send them to the debugger */ + DebugService2(&NameString, &CommandString, BREAKPOINT_COMMAND_STRING); +} Modified: trunk/reactos/lib/rtl/rtlp.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/rtlp.h?rev=43912&r… ============================================================================== --- trunk/reactos/lib/rtl/rtlp.h [iso-8859-1] (original) +++ trunk/reactos/lib/rtl/rtlp.h [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -114,7 +114,15 @@ BOOLEAN NTAPI -RtlpSetInDbgPrint(IN BOOLEAN NewValue); +RtlpSetInDbgPrint( + VOID +); + +VOID +NTAPI +RtlpClearInDbgPrint( + VOID +); /* i386/except.S */ Modified: trunk/reactos/ntoskrnl/config/cmdata.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/config/cmdata.c?r… ============================================================================== --- trunk/reactos/ntoskrnl/config/cmdata.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/config/cmdata.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -780,10 +780,20 @@ { L"Session Manager", L"Debugger Retries", - &DummyData, - NULL, - NULL - }, + &KdpContext.KdpDefaultRetries, + NULL, + NULL + }, + + { + L"Session Manager\\Debug Print Filter", + L"WIN2000", + &Kd_WIN2000_Mask, + NULL, + NULL + }, + + /* TODO: Add the other masks */ { L"WMI", Modified: trunk/reactos/ntoskrnl/include/internal/kd.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/include/internal/… ============================================================================== --- trunk/reactos/ntoskrnl/include/internal/kd.h [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/include/internal/kd.h [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -360,6 +360,8 @@ extern BOOLEAN KdpEarlyBreak; extern PKDEBUG_ROUTINE KiDebugRoutine; +extern KD_CONTEXT KdpContext; +extern ULONG Kd_WIN2000_Mask; #endif #endif /* __INCLUDE_INTERNAL_KERNEL_DEBUGGER_H */ Modified: trunk/reactos/ntoskrnl/include/internal/kd64.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/include/internal/… ============================================================================== --- trunk/reactos/ntoskrnl/include/internal/kd64.h [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/include/internal/kd64.h [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -12,6 +12,15 @@ #define KD_BREAKPOINT_MAX 32 // +// Default size of the DbgPrint log buffer +// +#if DBG +#define KD_DEFAULT_LOG_BUFFER_SIZE 0x8000 +#else +#define KD_DEFAULT_LOG_BUFFER_SIZE 0x1000 +#endif + +// // Breakpoint Status Flags // typedef enum _KDP_BREAKPOINT_FLAGS @@ -192,7 +201,7 @@ NTAPI KdpPrint( IN ULONG ComponentId, - IN ULONG ComponentMask, + IN ULONG Level, IN LPSTR String, IN USHORT Length, IN KPROCESSOR_MODE PreviousMode, @@ -228,8 +237,8 @@ VOID NTAPI KdpCommandString( - IN ULONG Length, - IN LPSTR String, + IN PSTRING NameString, + IN PSTRING CommandString, IN KPROCESSOR_MODE PreviousMode, IN PCONTEXT ContextRecord, IN PKTRAP_FRAME TrapFrame, @@ -239,12 +248,20 @@ // // State Change Notifications // -BOOLEAN +VOID NTAPI KdpReportLoadSymbolsStateChange( IN PSTRING PathName, IN PKD_SYMBOLS_INFO SymbolInfo, IN BOOLEAN Unload, + IN OUT PCONTEXT Context +); + +VOID +NTAPI +KdpReportCommandStringStateChange( + IN PSTRING NameString, + IN PSTRING CommandString, IN OUT PCONTEXT Context ); @@ -311,7 +328,7 @@ ); // -// Architecture dependent support routines +// Low Level Support Routines for the KD API // // @@ -484,12 +501,14 @@ extern KSPIN_LOCK KdpTimeSlipEventLock; extern BOOLEAN KdpPortLocked; extern BOOLEAN KdpControlCPressed; +extern BOOLEAN KdpContextSent; extern KSPIN_LOCK KdpDebuggerLock; extern LARGE_INTEGER KdTimerStop, KdTimerStart, KdTimerDifference; extern ULONG KdComponentTableSize; extern ULONG Kd_WIN2000_Mask; extern PULONG KdComponentTable[104]; -extern CHAR KdpMessageBuffer[4096], KdpPathBuffer[4096]; +extern CHAR KdpMessageBuffer[0x1000], KdpPathBuffer[0x1000]; +extern CHAR KdPrintDefaultCircularBuffer[KD_DEFAULT_LOG_BUFFER_SIZE]; extern BREAKPOINT_ENTRY KdpBreakpointTable[KD_BREAKPOINT_MAX]; extern KD_BREAKPOINT_TYPE KdpBreakpointInstruction; extern BOOLEAN KdpOweBreakpoint; Modified: trunk/reactos/ntoskrnl/kd/kdmain.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd/kdmain.c?rev=4… ============================================================================== --- trunk/reactos/ntoskrnl/kd/kdmain.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd/kdmain.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -20,6 +20,8 @@ BOOLEAN KdpBreakPending = FALSE; BOOLEAN KdPitchDebugger = TRUE; BOOLEAN KdIgnoreUmExceptions = FALSE; +KD_CONTEXT KdpContext; +ULONG Kd_WIN2000_Mask; VOID NTAPI PspDumpThreads(BOOLEAN SystemThreads); typedef struct Modified: trunk/reactos/ntoskrnl/kd64/amd64/kdsup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd64/amd64/kdsup.… ============================================================================== --- trunk/reactos/ntoskrnl/kd64/amd64/kdsup.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd64/amd64/kdsup.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -30,14 +30,6 @@ NTAPI KdpSetContextState(IN PDBGKD_ANY_WAIT_STATE_CHANGE WaitStateChange, IN PCONTEXT Context) -{ - UNIMPLEMENTED; - while (TRUE); -} - -VOID -NTAPI -KdpSysGetVersion(IN PDBGKD_GET_VERSION64 Version) { UNIMPLEMENTED; while (TRUE); Modified: trunk/reactos/ntoskrnl/kd64/arm/kdsup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd64/arm/kdsup.c?… ============================================================================== --- trunk/reactos/ntoskrnl/kd64/arm/kdsup.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd64/arm/kdsup.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -30,14 +30,6 @@ NTAPI KdpSetContextState(IN PDBGKD_ANY_WAIT_STATE_CHANGE WaitStateChange, IN PCONTEXT Context) -{ - UNIMPLEMENTED; - while (TRUE); -} - -VOID -NTAPI -KdpSysGetVersion(IN PDBGKD_GET_VERSION64 Version) { UNIMPLEMENTED; while (TRUE); Modified: trunk/reactos/ntoskrnl/kd64/i386/kdsup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd64/i386/kdsup.c… ============================================================================== --- trunk/reactos/ntoskrnl/kd64/i386/kdsup.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd64/i386/kdsup.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -14,14 +14,6 @@ #include <debug.h> /* FUNCTIONS *****************************************************************/ - -VOID -NTAPI -KdpSysGetVersion(IN PDBGKD_GET_VERSION64 Version) -{ - /* Copy the version block */ - RtlCopyMemory(Version, &KdVersionBlock, sizeof(DBGKD_GET_VERSION64)); -} VOID NTAPI Modified: trunk/reactos/ntoskrnl/kd64/kdapi.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd64/kdapi.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/kd64/kdapi.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd64/kdapi.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -39,14 +39,14 @@ ChunkSize = MMDBG_COPY_MAX_SIZE; } - /* Copy the whole range in page aligned chunks */ + /* Copy the whole range in aligned chunks */ RemainingLength = TotalSize; CopyChunk = 1; while (RemainingLength > 0) { /* * Determine the best chunk size for this round. - * The ideal size is page aligned, isn't larger than the + * The ideal size is aligned, isn't larger than the * the remaining length and respects the chunk limit. */ while (((CopyChunk * 2) <= RemainingLength) && @@ -54,7 +54,7 @@ ((Address & ((CopyChunk * 2) - 1)) == 0)) { /* Increase it */ - CopyChunk = CopyChunk * 2; + CopyChunk *= 2; } /* @@ -148,9 +148,20 @@ IN PSTRING Data, IN PCONTEXT Context) { - /* FIXME: STUB */ - KdpDprintf("KdpSearchMemory called\n"); - while (TRUE); + //PDBGKD_SEARCH_MEMORY SearchMemory = &State->u.SearchMemory; + STRING Header; + + /* TODO */ + KdpDprintf("Memory Search support is unimplemented!\n"); + + /* Send a failure packet */ + State->ReturnStatus = STATUS_UNSUCCESSFUL; + Header.Length = sizeof(DBGKD_MANIPULATE_STATE64); + Header.Buffer = (PCHAR)State; + KdSendPacket(PACKET_TYPE_KD_STATE_MANIPULATE, + &Header, + NULL, + &KdpContext); } VOID @@ -159,9 +170,20 @@ IN PSTRING Data, IN PCONTEXT Context) { - /* FIXME: STUB */ - KdpDprintf("KdpFillMemory called\n"); - while (TRUE); + //PDBGKD_FILL_MEMORY FillMemory = &State->u.FillMemory; + STRING Header; + + /* TODO */ + KdpDprintf("Memory Fill support is unimplemented!\n"); + + /* Send a failure packet */ + State->ReturnStatus = STATUS_UNSUCCESSFUL; + Header.Length = sizeof(DBGKD_MANIPULATE_STATE64); + Header.Buffer = (PCHAR)State; + KdSendPacket(PACKET_TYPE_KD_STATE_MANIPULATE, + &Header, + NULL, + &KdpContext); } VOID @@ -238,9 +260,20 @@ IN PSTRING Data, IN PCONTEXT Context) { - /* FIXME: STUB */ - KdpDprintf("KdpWriteBreakPointEx called\n"); - while (TRUE); + //PDBGKD_BREAKPOINTEX = &State->u.BreakPointEx; + STRING Header; + + /* TODO */ + KdpDprintf("Extended Breakpoint Write support is unimplemented!\n"); + + /* Send a failure packet */ + State->ReturnStatus = STATUS_UNSUCCESSFUL; + Header.Length = sizeof(DBGKD_MANIPULATE_STATE64); + Header.Buffer = (PCHAR)State; + KdSendPacket(PACKET_TYPE_KD_STATE_MANIPULATE, + &Header, + Data, + &KdpContext); return STATUS_UNSUCCESSFUL; } @@ -250,9 +283,20 @@ IN PSTRING Data, IN PCONTEXT Context) { - /* FIXME: STUB */ - KdpDprintf("KdpRestoreBreakPointEx called\n"); - while (TRUE); + //PDBGKD_BREAKPOINTEX = &State->u.BreakPointEx; + STRING Header; + + /* TODO */ + KdpDprintf("Extended Breakpoint Restore support is unimplemented!\n"); + + /* Send a failure packet */ + State->ReturnStatus = STATUS_UNSUCCESSFUL; + Header.Length = sizeof(DBGKD_MANIPULATE_STATE64); + Header.Buffer = (PCHAR)State; + KdSendPacket(PACKET_TYPE_KD_STATE_MANIPULATE, + &Header, + Data, + &KdpContext); } VOID @@ -319,6 +363,14 @@ VOID NTAPI +KdpSysGetVersion(IN PDBGKD_GET_VERSION64 Version) +{ + /* Copy the version block */ + RtlCopyMemory(Version, &KdVersionBlock, sizeof(DBGKD_GET_VERSION64)); +} + +VOID +NTAPI KdpGetVersion(IN PDBGKD_MANIPULATE_STATE64 State) { STRING Header; @@ -596,7 +648,7 @@ IN PCONTEXT Context) { STRING Header; - PVOID ControlStart; + PCONTEXT TargetContext; /* Setup the header */ Header.Length = sizeof(DBGKD_MANIPULATE_STATE64); @@ -610,19 +662,21 @@ if (State->Processor == KeGetCurrentPrcb()->Number) { /* We're just copying our own context */ - ControlStart = Context; + TargetContext = Context; } else { - /* SMP not yet handled */ - KdpDprintf("KdpGetContext: SMP UNHANDLED\n"); - ControlStart = NULL; - while (TRUE); + /* Get the context from the PRCB array */ + TargetContext = &KiProcessorBlock[State->Processor]-> + ProcessorState.ContextFrame; } - /* Copy the memory */ - RtlCopyMemory(Data->Buffer, ControlStart, sizeof(CONTEXT)); + /* Copy it over */ + RtlCopyMemory(Data->Buffer, TargetContext, sizeof(CONTEXT)); Data->Length = sizeof(CONTEXT); + + /* Let the debugger set the context now */ + KdpContextSent = TRUE; /* Finish up */ State->ReturnStatus = STATUS_SUCCESS; @@ -647,7 +701,7 @@ IN PCONTEXT Context) { STRING Header; - PVOID ControlStart; + PCONTEXT TargetContext; /* Setup the header */ Header.Length = sizeof(DBGKD_MANIPULATE_STATE64); @@ -655,24 +709,23 @@ ASSERT(Data->Length == sizeof(CONTEXT)); /* Make sure that this is a valid request */ - if (State->Processor < KeNumberProcessors) + if ((State->Processor < KeNumberProcessors) && (KdpContextSent == TRUE)) { /* Check if the request is for this CPU */ if (State->Processor == KeGetCurrentPrcb()->Number) { /* We're just copying our own context */ - ControlStart = Context; + TargetContext = Context; } else { - /* SMP not yet handled */ - KdpDprintf("KdpSetContext: SMP UNHANDLED\n"); - ControlStart = NULL; - while (TRUE); + /* Get the context from the PRCB array */ + TargetContext = &KiProcessorBlock[State->Processor]-> + ProcessorState.ContextFrame; } - /* Copy the memory */ - RtlCopyMemory(ControlStart, Data->Buffer, sizeof(CONTEXT)); + /* Get the context from the PRCB array */ + RtlCopyMemory(TargetContext, Data->Buffer, sizeof(CONTEXT)); /* Finish up */ State->ReturnStatus = STATUS_SUCCESS; @@ -985,6 +1038,26 @@ &KdpContext); } +VOID +NTAPI +KdpNotSupported(IN PDBGKD_MANIPULATE_STATE64 State) +{ + STRING Header; + + /* Set failure */ + State->ReturnStatus = STATUS_UNSUCCESSFUL; + + /* Setup the packet */ + Header.Length = sizeof(DBGKD_MANIPULATE_STATE64); + Header.Buffer = (PCHAR)State; + + /* Send it */ + KdSendPacket(PACKET_TYPE_KD_STATE_MANIPULATE, + &Header, + NULL, + &KdpContext); +} + KCONTINUE_STATUS NTAPI KdpSendWaitContinue(IN ULONG PacketType, @@ -1002,7 +1075,12 @@ Header.Buffer = (PCHAR)&ManipulateState; Data.MaximumLength = sizeof(KdpMessageBuffer); Data.Buffer = KdpMessageBuffer; - //KdpContextSent = FALSE; + + /* + * Reset the context state to ensure the debugger has received + * the current context before it sets it + */ + KdpContextSent = FALSE; SendPacket: /* Send the Packet */ @@ -1130,38 +1208,20 @@ break; case DbgKdQuerySpecialCallsApi: - - /* FIXME: TODO */ - KdpDprintf("DbgKdQuerySpecialCallsApi called\n"); - while (TRUE); - break; - case DbgKdSetSpecialCallApi: - - /* FIXME: TODO */ - KdpDprintf("DbgKdSetSpecialCallApi called\n"); - while (TRUE); - break; - case DbgKdClearSpecialCallsApi: - /* FIXME: TODO */ - KdpDprintf("DbgKdClearSpecialCallsApi called\n"); - while (TRUE); + /* TODO */ + KdpDprintf("Special Call support is unimplemented!\n"); + KdpNotSupported(&ManipulateState); break; case DbgKdSetInternalBreakPointApi: - - /* FIXME: TODO */ - KdpDprintf("DbgKdSetInternalBreakPointApi called\n"); - while (TRUE); - break; - case DbgKdGetInternalBreakPointApi: - /* FIXME: TODO */ - KdpDprintf("DbgKdGetInternalBreakPointApi called\n"); - while (TRUE); + /* TODO */ + KdpDprintf("Internal Breakpoint support is unimplemented!\n"); + KdpNotSupported(&ManipulateState); break; case DbgKdReadIoSpaceExtendedApi: @@ -1208,16 +1268,16 @@ case DbgKdSwitchProcessor: - /* FIXME: TODO */ - KdpDprintf("DbgKdSwitchProcessor called\n"); - while (TRUE); + /* TODO */ + KdpDprintf("Processor Switch support is unimplemented!\n"); + KdpNotSupported(&ManipulateState); break; case DbgKdPageInApi: - /* FIXME: TODO */ - KdpDprintf("DbgKdPageInApi called\n"); - while (TRUE); + /* TODO */ + KdpDprintf("Page-In support is unimplemented!\n"); + KdpNotSupported(&ManipulateState); break; case DbgKdReadMachineSpecificRegister: @@ -1276,9 +1336,9 @@ case DbgKdSwitchPartition: - /* FIXME: TODO */ - KdpDprintf("DbgKdSwitchPartition called\n"); - while (TRUE); + /* TODO */ + KdpDprintf("Partition Switch support is unimplemented!\n"); + KdpNotSupported(&ManipulateState); break; /* Unsupported Message */ @@ -1299,7 +1359,7 @@ } } -BOOLEAN +VOID NTAPI KdpReportLoadSymbolsStateChange(IN PSTRING PathName, IN PKD_SYMBOLS_INFO SymbolInfo, @@ -1342,8 +1402,7 @@ &PathNameLength); /* Null terminate */ - KdpPathBuffer[PathNameLength] = ANSI_NULL; - PathNameLength++; + KdpPathBuffer[PathNameLength++] = ANSI_NULL; /* Set the path length */ WaitStateChange.u.LoadSymbols.PathNameLength = PathNameLength; @@ -1370,9 +1429,83 @@ ExtraData, Context); } while (Status == ContinueProcessorReselected); - - /* Return status */ - return Status; +} + +VOID +NTAPI +KdpReportCommandStringStateChange(IN PSTRING NameString, + IN PSTRING CommandString, + IN OUT PCONTEXT Context) +{ + STRING Header, Data; + DBGKD_ANY_WAIT_STATE_CHANGE WaitStateChange; + ULONG Length, ActualLength, TotalLength; + KCONTINUE_STATUS Status; + + /* Start wait loop */ + do + { + /* Build the architecture common parts of the message */ + KdpSetCommonState(DbgKdCommandStringStateChange, + Context, + &WaitStateChange); + + /* Set the context */ + KdpSetContextState(&WaitStateChange, Context); + + /* Clear the command string structure */ + RtlZeroMemory(&WaitStateChange.u.CommandString, + sizeof(DBGKD_COMMAND_STRING)); + + /* Normalize name string to max */ + Length = min(128 - 1, NameString->Length); + + /* Copy it to the message buffer */ + KdpCopyMemoryChunks((ULONG_PTR)NameString->Buffer, + KdpMessageBuffer, + Length, + 0, + MMDBG_COPY_UNSAFE, + &ActualLength); + + /* Null terminate and calculate the total length */ + TotalLength = ActualLength; + KdpMessageBuffer[TotalLength++] = ANSI_NULL; + + /* Check if the command string is too long */ + Length = CommandString->Length; + if (Length > (PACKET_MAX_SIZE - + sizeof(DBGKD_ANY_WAIT_STATE_CHANGE) - TotalLength)) + { + /* Use maximum possible size */ + Length = (PACKET_MAX_SIZE - + sizeof(DBGKD_ANY_WAIT_STATE_CHANGE) - TotalLength); + } + + /* Copy it to the message buffer */ + KdpCopyMemoryChunks((ULONG_PTR)CommandString->Buffer, + KdpMessageBuffer + TotalLength, + Length, + 0, + MMDBG_COPY_UNSAFE, + &ActualLength); + + /* Null terminate and calculate the total length */ + TotalLength += ActualLength; + KdpMessageBuffer[TotalLength++] = ANSI_NULL; + + /* Now set up the header and the data */ + Header.Length = sizeof(DBGKD_ANY_WAIT_STATE_CHANGE); + Header.Buffer = (PCHAR)&WaitStateChange; + Data.Length = TotalLength; + Data.Buffer = KdpMessageBuffer; + + /* Send State Change packet and wait for a reply */ + Status = KdpSendWaitContinue(PACKET_TYPE_KD_STATE_CHANGE64, + &Header, + &Data, + Context); + } while (Status == ContinueProcessorReselected); } BOOLEAN @@ -1818,12 +1951,13 @@ IN OUT PULONG ReturnLength, IN KPROCESSOR_MODE PreviousMode) { - /* HACK */ - return STATUS_SUCCESS; + /* Local kernel debugging is not yet supported */ + DbgPrint("KdSystemDebugControl is unimplemented!\n"); + return STATUS_NOT_IMPLEMENTED; } /* - * @unimplemented + * @implemented */ NTSTATUS NTAPI @@ -1834,46 +1968,161 @@ OUT PVOID OutBuffer, OUT PULONG OutBufferNeeded OPTIONAL) { - /* HACK */ + /* Fail if there is no debugger */ + if (KdPitchDebugger) + { + /* No debugger, no options */ + return STATUS_DEBUGGER_INACTIVE; + } + + /* Do we recognize this option? */ + if (Option != KD_OPTION_SET_BLOCK_ENABLE) + { + /* We don't, clear the output length and fail */ + if (OutBufferNeeded) *OutBufferNeeded = 0; + return STATUS_INVALID_INFO_CLASS; + } + + /* Verify parameters */ + if ((InBufferBytes != sizeof(BOOLEAN)) || + (OutBufferBytes != 0) || + (OutBuffer != NULL)) + { + /* Invalid parameters for this option, fail */ + return STATUS_INVALID_PARAMETER; + } + + /* + * Check if the high bit is set, meaning we don't + * allow the debugger to be enabled + */ + if (KdBlockEnable & 0x80) + { + /* Fail regardless of what state the caller tried to set */ + return STATUS_ACCESS_VIOLATION; + } + + /* Set the new block enable state */ + KdBlockEnable = *(PBOOLEAN)InBuffer; + + /* No output buffer required for this option */ + if (OutBufferNeeded) *OutBufferNeeded = 0; + + /* We are done */ return STATUS_SUCCESS; } /* - * @unimplemented + * @implemented */ NTSTATUS NTAPI KdPowerTransition(IN DEVICE_POWER_STATE NewState) { - /* HACK */ - return STATUS_SUCCESS; + /* Check what power state this is */ + if (NewState == PowerDeviceD0) + { + /* Wake up the debug port */ + KdD0Transition(); + return STATUS_SUCCESS; + } + else if ((NewState == PowerDeviceD1) || + (NewState == PowerDeviceD2) || + (NewState == PowerDeviceD3)) + { + /* Power down the debug port */ + KdD3Transition(); + return STATUS_SUCCESS; + } + else + { + /* Invalid state! */ + return STATUS_INVALID_PARAMETER_1; + } } /* - * @unimplemented + * @implemented */ BOOLEAN NTAPI KdRefreshDebuggerNotPresent(VOID) { - /* HACK */ - return KdDebuggerNotPresent; + BOOLEAN Entered, DebuggerNotPresent; + + /* Check if the debugger is completely disabled */ + if (KdPitchDebugger) + { + /* Don't try to refresh then -- fail early */ + return TRUE; + } + + /* Enter the debugger */ + Entered = KdEnterDebugger(NULL, NULL); + + /* + * Attempt to send a string to the debugger to refresh the + * connection state + */ + KdpDprintf("KDTARGET: Refreshing KD connection\n"); + + /* Save the state while we are holding the lock */ + DebuggerNotPresent = KdDebuggerNotPresent; + + /* Exit the debugger and return the state */ + KdExitDebugger(Entered); + return DebuggerNotPresent; } /* - * @unimplemented + * @implemented */ NTSTATUS NTAPI NtQueryDebugFilterState(IN ULONG ComponentId, IN ULONG Level) { - /* HACK */ - return STATUS_SUCCESS; + PULONG Mask; + + /* Check if the ID fits in the component table */ + if (ComponentId < KdComponentTableSize) + { + /* It does, so get the mask from there */ + Mask = KdComponentTable[ComponentId]; + } + else if (ComponentId == MAXULONG) + { + /* + * This is the internal ID used for DbgPrint messages without ID and + * Level. Use the system-wide mask for those. + */ + Mask = &Kd_WIN2000_Mask; + } + else + { + /* Invalid ID, fail */ + return STATUS_INVALID_PARAMETER_1; + } + + /* Convert Level to bit field if necessary */ + if (Level < 32) Level = 1 << Level; + + /* Determine if this Level is filtered out */ + if ((Kd_WIN2000_Mask & Level) || + (*Mask & Level)) + { + /* This mask will get through to the debugger */ + return (NTSTATUS)TRUE; + } + else + { + /* This mask is filtered out */ + return (NTSTATUS)FALSE; + } } /* - * @unimplemented + * @implemented */ NTSTATUS NTAPI @@ -1881,6 +2130,51 @@ IN ULONG Level, IN BOOLEAN State) { - /* HACK */ + PULONG Mask; + + /* Modifying debug filters requires the debug privilege */ + if (!SeSinglePrivilegeCheck(SeDebugPrivilege, + ExGetPreviousMode())) + { + /* Fail */ + return STATUS_ACCESS_DENIED; + } + + /* Check if the ID fits in the component table */ + if (ComponentId < KdComponentTableSize) + { + /* It does, so get the mask from there */ + Mask = KdComponentTable[ComponentId]; + } + else if (ComponentId == MAXULONG) + { + /* + * This is the internal ID used for DbgPrint messages without ID and + * Level. Use the system-wide mask for those. + */ + Mask = &Kd_WIN2000_Mask; + } + else + { + /* Invalid ID, fail */ + return STATUS_INVALID_PARAMETER_1; + } + + /* Convert Level to bit field if required */ + if (Level < 32) Level = 1 << Level; + + /* Check what kind of operation this is */ + if (State) + { + /* Set the Level */ + *Mask |= Level; + } + else + { + /* Remove the Level */ + *Mask &= ~Level; + } + + /* Success */ return STATUS_SUCCESS; } Modified: trunk/reactos/ntoskrnl/kd64/kddata.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd64/kddata.c?rev… ============================================================================== --- trunk/reactos/ntoskrnl/kd64/kddata.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd64/kddata.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -9,7 +9,7 @@ /* INCLUDES ******************************************************************/ #include <ntoskrnl.h> -#include "../mm/arm3/miarm.h" +#include "../mm/ARM3/miarm.h" #define NDEBUG #include <debug.h> @@ -18,27 +18,36 @@ // // Apply the KIPCR WDK workaround for x86 and AMD64 // -#if defined(_M_IX86) || defined(_M_AMD64) +#if defined(_X86_) || defined(_AMD64_) #define KPCR KIPCR #endif -#if defined(_M_IX86) - -#define KPCR_SELF_OFFSET FIELD_OFFSET(KPCR, Self) +#if defined(_X86_) + +#define KPCR_SELF_PCR_OFFSET FIELD_OFFSET(KPCR, Self) #define KPCR_CURRENT_PRCB_OFFSET FIELD_OFFSET(KPCR, Prcb) #define KPCR_CONTAINED_PRCB_OFFSET FIELD_OFFSET(KPCR, PrcbData) - -#elif defined(_M_AMD64) - -#define KPCR_SELF_OFFSET FIELD_OFFSET(KPCR, Self) +#define KPCR_INITIAL_STACK_OFFSET 0 +#define KPCR_STACK_LIMIT_OFFSET 0 +#define KPRCB_PCR_PAGE_OFFSET 0 + +#elif defined(_AMD64_) + +#define KPCR_SELF_PCR_OFFSET FIELD_OFFSET(KPCR, Self) #define KPCR_CURRENT_PRCB_OFFSET FIELD_OFFSET(KPCR, CurrentPrcb) #define KPCR_CONTAINED_PRCB_OFFSET FIELD_OFFSET(KPCR, Prcb) - -#elif defined(_M_ARM) - -#define KPCR_SELF_OFFSET 0 +#define KPCR_INITIAL_STACK_OFFSET 0 +#define KPCR_STACK_LIMIT_OFFSET 0 +#define KPRCB_PCR_PAGE_OFFSET 0 + +#elif defined(_ARM_) + +#define KPCR_SELF_PCR_OFFSET 0 #define KPCR_CURRENT_PRCB_OFFSET FIELD_OFFSET(KPCR, Prcb) #define KPCR_CONTAINED_PRCB_OFFSET 0 +#define KPCR_INITIAL_STACK_OFFSET FIELD_OFFSET(KPCR, InitialStack) +#define KPCR_STACK_LIMIT_OFFSET FIELD_OFFSET(KPCR, StackLimit) +#define KPRCB_PCR_PAGE_OFFSET FIELD_OFFSET(KPRCB, PcrPage) #else #error Unsupported Architecture @@ -53,6 +62,7 @@ BOOLEAN KdpPortLocked; KSPIN_LOCK KdpDebuggerLock; BOOLEAN KdpControlCPressed; +BOOLEAN KdpContextSent; // // Debug Trap Handlers @@ -110,13 +120,13 @@ // // Buffers // -CHAR KdpMessageBuffer[4096]; -CHAR KdpPathBuffer[4096]; +CHAR KdpMessageBuffer[0x1000]; +CHAR KdpPathBuffer[0x1000]; // // KdPrint Buffers // -CHAR KdPrintDefaultCircularBuffer[0x8000]; +CHAR KdPrintDefaultCircularBuffer[KD_DEFAULT_LOG_BUFFER_SIZE]; PCHAR KdPrintWritePointer = KdPrintDefaultCircularBuffer; ULONG KdPrintRolloverCount; PCHAR KdPrintCircularBuffer = KdPrintDefaultCircularBuffer; @@ -486,14 +496,14 @@ {(ULONG_PTR)&KdPrintBufferSize}, {(ULONG_PTR)&KeLoaderBlock}, sizeof(KPCR), - KPCR_SELF_OFFSET, + KPCR_SELF_PCR_OFFSET, KPCR_CURRENT_PRCB_OFFSET, KPCR_CONTAINED_PRCB_OFFSET, 0, 0, - 0, - 0, - 0, + KPCR_INITIAL_STACK_OFFSET, + KPCR_STACK_LIMIT_OFFSET, + KPRCB_PCR_PAGE_OFFSET, FIELD_OFFSET(KPRCB, ProcessorState.SpecialRegisters), #if defined(_M_IX86) // Modified: trunk/reactos/ntoskrnl/kd64/kdinit.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd64/kdinit.c?rev… ============================================================================== --- trunk/reactos/ntoskrnl/kd64/kdinit.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd64/kdinit.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -116,7 +116,7 @@ sizeof(KdDebuggerDataBlock)); /* Fill out the KD Version Block */ - KdVersionBlock.MajorVersion = (USHORT)(NtBuildNumber >> 28); + KdVersionBlock.MajorVersion = (USHORT)((DBGKD_MAJOR_NT << 8) | (NtBuildNumber >> 28)); KdVersionBlock.MinorVersion = (USHORT)(NtBuildNumber & 0xFFFF); #ifdef CONFIG_SMP @@ -161,13 +161,18 @@ /* Assume we'll disable KD */ EnableKd = FALSE; - /* Check for CRASHDEBUG and NODEBUG */ - if (strstr(CommandLine, "CRASHDEBUG")) KdPitchDebugger = FALSE; - if (strstr(CommandLine, "NODEBUG")) KdPitchDebugger = TRUE; - - /* Check if DEBUG was on */ - DebugLine = strstr(CommandLine, "DEBUG"); - if (DebugLine) + /* Check for CRASHDEBUG, NODEBUG and just DEBUG */ + if (strstr(CommandLine, "CRASHDEBUG")) + { + /* Don't enable KD now, but allow it to be enabled later */ + KdPitchDebugger = FALSE; + } + else if (strstr(CommandLine, "NODEBUG")) + { + /* Don't enable KD and don't let it be enabled later */ + KdPitchDebugger = TRUE; + } + else if ((DebugLine = strstr(CommandLine, "DEBUG")) != NULL) { /* Enable KD */ EnableKd = TRUE; Modified: trunk/reactos/ntoskrnl/kd64/kdlock.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd64/kdlock.c?rev… ============================================================================== --- trunk/reactos/ntoskrnl/kd64/kdlock.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd64/kdlock.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -50,10 +50,10 @@ { /* Now get a packet */ if (KdReceivePacket(PACKET_TYPE_KD_POLL_BREAKIN, - NULL, - NULL, - NULL, - NULL) == KdPacketReceived) + NULL, + NULL, + NULL, + NULL) == KdPacketReceived) { /* Successful breakin */ DoBreak = TRUE; @@ -97,10 +97,10 @@ { /* Now get a packet */ if (KdReceivePacket(PACKET_TYPE_KD_POLL_BREAKIN, - NULL, - NULL, - NULL, - NULL) == KdPacketReceived) + NULL, + NULL, + NULL, + NULL) == KdPacketReceived) { /* Successful breakin */ DoBreak = TRUE; Modified: trunk/reactos/ntoskrnl/kd64/kdprint.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd64/kdprint.c?re… ============================================================================== --- trunk/reactos/ntoskrnl/kd64/kdprint.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd64/kdprint.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -123,16 +123,41 @@ VOID NTAPI -KdpCommandString(IN ULONG Length, - IN LPSTR String, +KdpCommandString(IN PSTRING NameString, + IN PSTRING CommandString, IN KPROCESSOR_MODE PreviousMode, IN PCONTEXT ContextRecord, IN PKTRAP_FRAME TrapFrame, IN PKEXCEPTION_FRAME ExceptionFrame) { - /* FIXME */ - KdpDprintf("KdpCommandString called\n"); - while (TRUE); + BOOLEAN Entered; + PKPRCB Prcb = KeGetCurrentPrcb(); + + /* Check if we need to do anything */ + if ((PreviousMode != KernelMode) || (KdDebuggerNotPresent)) return; + + /* Enter the debugger */ + Entered = KdEnterDebugger(TrapFrame, ExceptionFrame); + + /* Save the CPU Control State and save the context */ + KiSaveProcessorControlState(&Prcb->ProcessorState); + RtlCopyMemory(&Prcb->ProcessorState.ContextFrame, + ContextRecord, + sizeof(CONTEXT)); + + /* Send the command string to the debugger */ + KdpReportCommandStringStateChange(NameString, + CommandString, + &Prcb->ProcessorState.ContextFrame); + + /* Restore the processor state */ + RtlCopyMemory(ContextRecord, + &Prcb->ProcessorState.ContextFrame, + sizeof(CONTEXT)); + KiRestoreProcessorControlState(&Prcb->ProcessorState); + + /* Exit the debugger and return */ + KdExitDebugger(Entered); } VOID @@ -147,7 +172,6 @@ { BOOLEAN Entered; PKPRCB Prcb = KeGetCurrentPrcb(); - ULONG Status; /* Check if we need to do anything */ if ((PreviousMode != KernelMode) || (KdDebuggerNotPresent)) return; @@ -162,19 +186,18 @@ sizeof(CONTEXT)); /* Report the new state */ - Status = KdpReportLoadSymbolsStateChange(DllPath, - SymbolInfo, - Unload, - &Prcb->ProcessorState. - ContextFrame); - - /* Now restore the processor state, manually again. */ + KdpReportLoadSymbolsStateChange(DllPath, + SymbolInfo, + Unload, + &Prcb->ProcessorState.ContextFrame); + + /* Restore the processor state */ RtlCopyMemory(ContextRecord, &Prcb->ProcessorState.ContextFrame, sizeof(CONTEXT)); KiRestoreProcessorControlState(&Prcb->ProcessorState); - /* Exit the debugger and clear the CTRL-C state */ + /* Exit the debugger and return */ KdExitDebugger(Entered); } @@ -233,7 +256,7 @@ NTSTATUS NTAPI KdpPrint(IN ULONG ComponentId, - IN ULONG ComponentMask, + IN ULONG Level, IN LPSTR String, IN USHORT Length, IN KPROCESSOR_MODE PreviousMode, @@ -249,14 +272,14 @@ *Status = FALSE; /* Validate the mask */ - if (ComponentMask < 0x20) ComponentMask = 1 << ComponentMask; - if (!(Kd_WIN2000_Mask & ComponentMask) || + if (Level < 32) Level = 1 << Level; + if (!(Kd_WIN2000_Mask & Level) || ((ComponentId < KdComponentTableSize) && - !(*KdComponentTable[ComponentId] & ComponentMask))) + !(*KdComponentTable[ComponentId] & Level))) { /* Mask validation failed */ *Status = TRUE; - return FALSE; + return STATUS_SUCCESS; } /* Normalize the length */ Modified: trunk/reactos/ntoskrnl/kd64/kdtrap.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/kd64/kdtrap.c?rev… ============================================================================== --- trunk/reactos/ntoskrnl/kd64/kdtrap.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/kd64/kdtrap.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -17,7 +17,7 @@ // Retrieves the ComponentId and Level for BREAKPOINT_PRINT // and OutputString and OutputStringLength for BREAKPOINT_PROMPT. // -#if defined(_M_IX86) +#if defined(_X86_) // // EBX/EDI on x86 @@ -25,7 +25,7 @@ #define KdpGetFirstParameter(Context) ((Context)->Ebx) #define KdpGetSecondParameter(Context) ((Context)->Edi) -#elif defined(_M_AMD64) +#elif defined(_AMD64_) // // R8/R9 on AMD64 @@ -33,7 +33,7 @@ #define KdpGetFirstParameter(Context) ((Context)->R8) #define KdpGetSecondParameter(Context) ((Context)->R9) -#elif defined(_M_ARM) +#elif defined(_ARM_) #error Yo Ninjas! @@ -218,9 +218,9 @@ case BREAKPOINT_COMMAND_STRING: /* Call the worker routine */ - KdpCommandString((ULONG)ExceptionRecord-> + KdpCommandString((PSTRING)ExceptionRecord-> ExceptionInformation[1], - (LPSTR)ExceptionRecord-> + (PSTRING)ExceptionRecord-> ExceptionInformation[2], PreviousMode, ContextRecord, Modified: trunk/reactos/ntoskrnl/ke/i386/cpu.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/i386/cpu.c?rev… ============================================================================== --- trunk/reactos/ntoskrnl/ke/i386/cpu.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/ke/i386/cpu.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -915,15 +915,29 @@ BOOLEAN NTAPI +KeDisableInterrupts(VOID) +{ + ULONG Flags; + BOOLEAN Return; + + /* Get EFLAGS and check if the interrupt bit is set */ + Flags = __readeflags(); + Return = (Flags & EFLAGS_INTERRUPT_MASK) ? TRUE: FALSE; + + /* Disable interrupts */ + _disable(); + return Return; +} + +BOOLEAN +NTAPI KeFreezeExecution(IN PKTRAP_FRAME TrapFrame, IN PKEXCEPTION_FRAME ExceptionFrame) { - ULONG Flags; + BOOLEAN Enable; /* Disable interrupts and get previous state */ - Flags = __readeflags(); - //Flags = __getcallerseflags(); - _disable(); + Enable = KeDisableInterrupts(); /* Save freeze flag */ KiFreezeFlag = 4; @@ -932,7 +946,7 @@ KiOldIrql = KeGetCurrentIrql(); /* Return whether interrupts were enabled */ - return (Flags & EFLAGS_INTERRUPT_MASK) ? TRUE: FALSE; + return Enable; } VOID Modified: trunk/reactos/ntoskrnl/ke/i386/irqobj.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/i386/irqobj.c?… ============================================================================== --- trunk/reactos/ntoskrnl/ke/i386/irqobj.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/ke/i386/irqobj.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -3,7 +3,7 @@ * LICENSE: GPL - See COPYING in the top level directory * FILE: ntoskrnl/ke/i386/irq.c * PURPOSE: Manages the Kernel's IRQ support for external drivers, - * for the purpopses of connecting, disconnecting and setting + * for the purpouses of connecting, disconnecting and setting * up ISRs for drivers. The backend behind the Io* Interrupt * routines. * PROGRAMMERS: Alex Ionescu (alex.ionescu(a)reactos.org) @@ -22,22 +22,6 @@ extern ULONG NTAPI KiChainedDispatch2ndLvl(VOID); /* PRIVATE FUNCTIONS *********************************************************/ - -BOOLEAN -NTAPI -KeDisableInterrupts(VOID) -{ - ULONG Flags; - BOOLEAN Return; - - /* Get EFLAGS and check if the interrupt bit is set */ - Flags = __readeflags(); - Return = (Flags & EFLAGS_INTERRUPT_MASK) ? TRUE: FALSE; - - /* Disable interrupts */ - _disable(); - return Return; -} VOID NTAPI Modified: trunk/reactos/ntoskrnl/ntoskrnl.pspec URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ntoskrnl.pspec?re… ============================================================================== --- trunk/reactos/ntoskrnl/ntoskrnl.pspec [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/ntoskrnl.pspec [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -47,7 +47,7 @@ @ stdcall CmUnRegisterCallback(long long) @ stdcall DbgBreakPoint() @ stdcall DbgBreakPointWithStatus(long) -;DbgCommandString +@ stdcall DbgCommandString(ptr ptr) @ stdcall DbgLoadImageSymbols(ptr ptr long) @ cdecl DbgPrint(str) @ cdecl DbgPrintEx(long long str) 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 [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/rtl/libsupp.c [iso-8859-1] Mon Nov 2 18:45:51 2009 @@ -52,10 +52,17 @@ BOOLEAN NTAPI -RtlpSetInDbgPrint(IN BOOLEAN NewValue) -{ - /* This check is meaningless in kernel-mode */ +RtlpSetInDbgPrint(VOID) +{ + /* Nothing to set in kernel mode */ return FALSE; +} + +VOID +NTAPI +RtlpClearInDbgPrint(VOID) +{ + /* Nothing to clear in kernel mode */ } KPROCESSOR_MODE

15 years, 1 month

[tkreuzer] 43911: [OPENGL32] fix TessVertexOutData callback, too.

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Mon Nov 2 17:16:09 2009 New Revision: 43911 URL: http://svn.reactos.org/svn/reactos?rev=43911&view=rev Log: [OPENGL32] fix TessVertexOutData callback, too. Modified: trunk/reactos/dll/win32/opengl32/font.c Modified: trunk/reactos/dll/win32/opengl32/font.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/opengl32/font.c?… ============================================================================== --- trunk/reactos/dll/win32/opengl32/font.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/opengl32/font.c [iso-8859-1] Mon Nov 2 17:16:09 2009 @@ -461,12 +461,12 @@ * Used by tessellator to handle output vertexes. *****************************************************************************/ -VOID CALLBACK TessVertexOutData(FLOAT p[3], GLfloat z) +VOID CALLBACK TessVertexOutData(FLOAT p[3], GLfloat *pz) { GLfloat v[3]; v[0] = (GLfloat) p[0]; v[1] = (GLfloat) p[1]; - v[2] = z; + v[2] = *pz; glVertex3fv(v); }

15 years, 1 month

[khornicek] 43910: - get rid of some cast frenzy, by Physicus

by khornicek＠svn.reactos.org

Author: khornicek Date: Mon Nov 2 15:00:47 2009 New Revision: 43910 URL: http://svn.reactos.org/svn/reactos?rev=43910&view=rev Log: - get rid of some cast frenzy, by Physicus Modified: trunk/reactos/dll/win32/opengl32/font.c Modified: trunk/reactos/dll/win32/opengl32/font.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/opengl32/font.c?… ============================================================================== --- trunk/reactos/dll/win32/opengl32/font.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/opengl32/font.c [iso-8859-1] Mon Nov 2 15:00:47 2009 @@ -647,7 +647,7 @@ v[2] = 0.0; z_value = 0.0f; - gluTessBeginPolygon(tess, (VOID *)*(INT *)&z_value); + gluTessBeginPolygon(tess, &z_value); for (loop = (DWORD) *p++; loop; --loop) { @@ -718,7 +718,7 @@ glNormal3f(0.0f, 0.0f, -1.0f); gluTessNormal(tess, 0.0F, 0.0F, -1.0F); - gluTessBeginPolygon(tess, (VOID *)*(INT *)&thickness); + gluTessBeginPolygon(tess, &thickness); for (loop = (DWORD) *p++; loop; --loop) {

15 years, 1 month

[dreimer] 43909: I proudly introduce my new Toy. The remake command. It fully rebuilds one or several modules without needing to write make MODULE1_CLEAN MODULE1 MODULE2_CLEAN MODULE2 etcetc. Some tidy up work.

by dreimer＠svn.reactos.org

Author: dreimer Date: Mon Nov 2 11:48:16 2009 New Revision: 43909 URL: http://svn.reactos.org/svn/reactos?rev=43909&view=rev Log: I proudly introduce my new Toy. The remake command. It fully rebuilds one or several modules without needing to write make MODULE1_CLEAN MODULE1 MODULE2_CLEAN MODULE2 etcetc. Some tidy up work. Added: trunk/tools/RosBE/RosBE-Windows/Powershell/Remake.ps1 (with props) trunk/tools/RosBE/RosBE-Windows/Root/Remake.cmd (with props) Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/Build.ps1 trunk/tools/RosBE/RosBE-Windows/Powershell/Clean.ps1 trunk/tools/RosBE/RosBE-Windows/Powershell/Config.ps1 trunk/tools/RosBE/RosBE-Windows/Powershell/Help.ps1 trunk/tools/RosBE/RosBE-Windows/Powershell/RosBE.ps1 trunk/tools/RosBE/RosBE-Windows/Powershell/chdefdir.ps1 trunk/tools/RosBE/RosBE-Windows/Powershell/options.ps1 trunk/tools/RosBE/RosBE-Windows/Powershell/reladdr2line.ps1 trunk/tools/RosBE/RosBE-Windows/Powershell/rosbe-gcc-env.ps1 trunk/tools/RosBE/RosBE-Windows/Powershell/sSVN.ps1 trunk/tools/RosBE/RosBE-Windows/Powershell/update.ps1 trunk/tools/RosBE/RosBE-Windows/Root/Help.cmd Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/Build.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/Build.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/Build.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -4,7 +4,6 @@ # FILE: Root/Build.ps1 # PURPOSE: Perform the build of ReactOS. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # # Check if config.template.rbuild is newer than config.rbuild, if it is then Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/Clean.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/Clean.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/Clean.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -4,7 +4,6 @@ # FILE: Root/Clean.ps1 # PURPOSE: Clean the ReactOS source directory. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # $host.ui.RawUI.WindowTitle = "Cleaning..." Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/Config.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/Config.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/Config.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -4,7 +4,6 @@ # FILE: Root/Config.ps1 # PURPOSE: A Basic Config.rbuild Creator for ReactOS. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # $host.ui.RawUI.WindowTitle = "ReactOS Build Configurator" Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/Help.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/Help.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/Help.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -4,7 +4,6 @@ # FILE: Root/Help.ps1 # PURPOSE: Display help for the commands included with the ReactOS Build Environment. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # # Check if we are displaying help on all commands, if so, display it. @@ -51,6 +50,11 @@ " specific bugs in ReactOS." } + if (Test-Path "$_ROSBE_BASEDIR\Remake.ps1") { + " remake [OPTIONS] - Cleans one or several specific modules and" + " immediately rebuilds it cleanly." + } + if (Test-Path "$_ROSBE_BASEDIR\scut.ps1") { " scut [OPTIONS] - List, add, edit, remove, switch and default to" " shortcutted ReactOS source directories." @@ -180,6 +184,13 @@ " ADDRESS - The address to be translated." "" } +} elseif ("$args" -eq "Remake") { + if (Test-Path "$_ROSBE_BASEDIR\Remake.cmd") { + " Usage: remake [OPTIONS]" + " Cleans one or several specific modules and immediately rebuilds it cleanly." + "" + " OPTIONS - One or more Module names to rebuild." + } } elseif ("$args" -eq "scut") { if (Test-Path "$_ROSBE_BASEDIR\scut.ps1") { IEX "&'$_ROSBE_BASEDIR\Tools\scut.exe' --help" Added: trunk/tools/RosBE/RosBE-Windows/Powershell/Remake.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/Remake.ps1 (added) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/Remake.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -1,0 +1,15 @@ +# +# PROJECT: RosBE - ReactOS Build Environment for Windows +# LICENSE: GNU General Public License v2. (see LICENSE.txt) +# FILE: Root/Remake.ps1 +# PURPOSE: Fully re-make one or several modules. +# COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> +# + +$argindex = 0 +while ( "$($args[$argindex])" -ne "") { + $cl = "$($args[$argindex])" + "_clean" + " " + "$($args[$argindex])" + make $cl + $argindex += 1 +} +remove-variable cl Propchange: trunk/tools/RosBE/RosBE-Windows/Powershell/Remake.ps1 ------------------------------------------------------------------------------ svn:eol-style = native Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/RosBE.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/RosBE.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/RosBE.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -5,7 +5,6 @@ # PURPOSE: This script provides/sets up various build environments for # ReactOS. Currently it provides a GCC 4.1.3 build environment. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # $host.ui.RawUI.WindowTitle = "ReactOS Build Environment $_ROSBE_VERSION" @@ -109,6 +108,10 @@ set-alias RADDR2LINE "$_ROSBE_BASEDIR\reladdr2line.ps1" -scope Global } + if (Test-Path "$_ROSBE_BASEDIR\Remake.ps1") { + set-alias REMAKE "$_ROSBE_BASEDIR\Remake.ps1" -scope Global + } + if (Test-Path "$_ROSBE_BASEDIR\scut.ps1") { set-alias SCUT "$_ROSBE_BASEDIR\scut.ps1" -scope Global } Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/chdefdir.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/chdefdir.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/chdefdir.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -5,7 +5,6 @@ # PURPOSE: Tool to change the current working ReactOS source directory # in RosBE. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # $host.ui.RawUI.WindowTitle = "Change the current working ReactOS source directory..." Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/options.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/options.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/options.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -4,7 +4,6 @@ # FILE: Root/options.ps1 # PURPOSE: Starts options.exe and restarts RosBE afterwards. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # $host.ui.RawUI.WindowTitle = "Options" Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/reladdr2line.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/reladdr2line.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/reladdr2line.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -4,7 +4,6 @@ # FILE: Root/reladdr2line.ps1 # PURPOSE: Converts a value to hex and displays it. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # $host.ui.RawUI.WindowTitle = "reladdr2line..." Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/rosbe-gcc-env.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/rosbe-gcc-env.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/rosbe-gcc-env.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -4,7 +4,6 @@ # FILE: Root/rosbe-gcc-env.ps1 # PURPOSE: Set up the GCC 4.x.x build environment. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # # Check if we're switching to the AMD64 architecture Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/sSVN.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/sSVN.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/sSVN.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -4,7 +4,6 @@ # FILE: Root/sSVN.ps1 # PURPOSE: Integrated SVN Client. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # function UP($arg) { Modified: trunk/tools/RosBE/RosBE-Windows/Powershell/update.ps1 URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Powershe… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Powershell/update.ps1 [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Powershell/update.ps1 [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -4,7 +4,6 @@ # FILE: Root/update.ps1 # PURPOSE: RosBE Updater. # COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> -# # $host.ui.RawUI.WindowTitle = "Updating..." Modified: trunk/tools/RosBE/RosBE-Windows/Root/Help.cmd URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Root/Hel… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Root/Help.cmd [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Root/Help.cmd [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -58,6 +58,11 @@ echo specific bugs in ReactOS. ) + if exist "%_ROSBE_BASEDIR%\Remake.cmd" ( + echo remake [OPTIONS] - Cleans one or several specific modules and + echo immediately rebuilds it cleanly. + ) + if exist "%_ROSBE_BASEDIR%\scut.cmd" ( echo scut [OPTIONS] - List, add, edit, remove, switch and default to echo shortcutted ReactOS source directories. @@ -186,6 +191,13 @@ echo ADDRESS - The address to be translated. echo. ) +) else if /i "%1" == "Remake" ( + if exist "%_ROSBE_BASEDIR%\Remake.cmd" ( + echo Usage: remake [OPTIONS] + echo Cleans one or several specific modules and immediately rebuilds it cleanly. + echo. + echo OPTIONS - One or more Module names to rebuild. + ) ) else if /i "%1" == "scut" ( if exist "%_ROSBE_BASEDIR%\scut.cmd" ( "%_ROSBE_BASEDIR%\Tools\scut.exe" --help Added: trunk/tools/RosBE/RosBE-Windows/Root/Remake.cmd URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Root/Rem… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Root/Remake.cmd (added) +++ trunk/tools/RosBE/RosBE-Windows/Root/Remake.cmd [iso-8859-1] Mon Nov 2 11:48:16 2009 @@ -1,0 +1,24 @@ +:: +:: PROJECT: RosBE - ReactOS Build Environment for Windows +:: LICENSE: GNU General Public License v2. (see LICENSE.txt) +:: FILE: Root/Remake.cmd +:: PURPOSE: Fully re-make one or several modules. +:: COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> +:: + +@echo off +if not defined _ROSBE_DEBUG set _ROSBE_DEBUG=0 +if %_ROSBE_DEBUG% == 1 ( + @echo on +) + +title 'Remake %*' started: %TIMERAW% (%ROS_ARCH%) + +:WHILE + if "%1" == "" goto :EOF + call "%_ROSBE_BASEDIR%\Make.cmd" %1_clean %1 + shift /1 + echo. + GOTO :WHILE %* + +title ReactOS Build Environment %_ROSBE_VERSION% Propchange: trunk/tools/RosBE/RosBE-Windows/Root/Remake.cmd ------------------------------------------------------------------------------ svn:eol-style = native

15 years, 1 month

[dreimer] 43908: After renaming some scrips it was forgotten to change the names internally, too. Fix this.

by dreimer＠svn.reactos.org

Author: dreimer Date: Mon Nov 2 11:23:58 2009 New Revision: 43908 URL: http://svn.reactos.org/svn/reactos?rev=43908&view=rev Log: After renaming some scrips it was forgotten to change the names internally, too. Fix this. Modified: trunk/tools/RosBE/RosBE-Windows/Root/Clean.cmd trunk/tools/RosBE/RosBE-Windows/Root/Help.cmd trunk/tools/RosBE/RosBE-Windows/Root/Make.cmd trunk/tools/RosBE/RosBE-Windows/Root/Makex.cmd trunk/tools/RosBE/RosBE-Windows/Root/Renv.cmd trunk/tools/RosBE/RosBE-Windows/Root/raddr2line.cmd Modified: trunk/tools/RosBE/RosBE-Windows/Root/Clean.cmd URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Root/Cle… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Root/Clean.cmd [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Root/Clean.cmd [iso-8859-1] Mon Nov 2 11:23:58 2009 @@ -34,7 +34,7 @@ :WHILE if "%1" == "" goto :EOF - call "%_ROSBE_BASEDIR%\Build.cmd" %1_clean + call "%_ROSBE_BASEDIR%\Make.cmd" %1_clean shift /1 echo. GOTO :WHILE %* Modified: trunk/tools/RosBE/RosBE-Windows/Root/Help.cmd URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Root/Hel… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Root/Help.cmd [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Root/Help.cmd [iso-8859-1] Mon Nov 2 11:23:58 2009 @@ -52,7 +52,7 @@ echo help [COMMAND] - Display the available commands or give help on a echo specific command. - if exist "%_ROSBE_BASEDIR%\reladdr2line.cmd" ( + if exist "%_ROSBE_BASEDIR%\raddr2line.cmd" ( echo raddr2line [OPTIONS] - Translates program addresses into file names and echo line numbers to assist developers with finding echo specific bugs in ReactOS. @@ -173,7 +173,7 @@ echo Usage: help [COMMAND] echo Shows help for the specified command or lists all available commands. ) else if /i "%1" == "raddr2line" ( - if exist "%_ROSBE_BASEDIR%\reladdr2line.cmd" ( + if exist "%_ROSBE_BASEDIR%\raddr2line.cmd" ( echo Usage: raddr2line [FILE] [ADDRESS] echo Translates program addresses into file names and line numbers to assist echo developers with finding specific bugs in ReactOS. If either of the options Modified: trunk/tools/RosBE/RosBE-Windows/Root/Make.cmd URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Root/Mak… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Root/Make.cmd [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Root/Make.cmd [iso-8859-1] Mon Nov 2 11:23:58 2009 @@ -1,7 +1,7 @@ :: :: PROJECT: RosBE - ReactOS Build Environment for Windows :: LICENSE: GNU General Public License v2. (see LICENSE.txt) -:: FILE: Root/Build.cmd +:: FILE: Root/Make.cmd :: PURPOSE: Perform a uniprocessor build :: COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> :: Colin Finck <colin(a)reactos.org> Modified: trunk/tools/RosBE/RosBE-Windows/Root/Makex.cmd URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Root/Mak… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Root/Makex.cmd [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Root/Makex.cmd [iso-8859-1] Mon Nov 2 11:23:58 2009 @@ -1,7 +1,7 @@ :: :: PROJECT: RosBE - ReactOS Build Environment for Windows :: LICENSE: GNU General Public License v2. (see LICENSE.txt) -:: FILE: Root/Build-Multi.cmd +:: FILE: Root/Makex.cmd :: PURPOSE: Perform a multiprocessor build :: COPYRIGHT: Copyright 2009 Daniel Reimer <reimer.daniel(a)freenet.de> :: Colin Finck <colin(a)reactos.org> Modified: trunk/tools/RosBE/RosBE-Windows/Root/Renv.cmd URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Root/Ren… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Root/Renv.cmd [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Root/Renv.cmd [iso-8859-1] Mon Nov 2 11:23:58 2009 @@ -1,7 +1,7 @@ :: :: PROJECT: RosBE - ReactOS Build Environment for Windows :: LICENSE: GNU General Public License v2. (see LICENSE.txt) -:: FILE: Root/renv.cmd +:: FILE: Root/Renv.cmd :: PURPOSE: Shows RosBE's Environment Variables. :: COPYRIGHT: Copyright 2009 Colin Finck <colin(a)reactos.org> :: Modified: trunk/tools/RosBE/RosBE-Windows/Root/raddr2line.cmd URL: http://svn.reactos.org/svn/reactos/trunk/tools/RosBE/RosBE-Windows/Root/rad… ============================================================================== --- trunk/tools/RosBE/RosBE-Windows/Root/raddr2line.cmd [iso-8859-1] (original) +++ trunk/tools/RosBE/RosBE-Windows/Root/raddr2line.cmd [iso-8859-1] Mon Nov 2 11:23:58 2009 @@ -1,7 +1,7 @@ :: :: PROJECT: RosBE - ReactOS Build Environment for Windows :: LICENSE: GNU General Public License v2. (see LICENSE.txt) -:: FILE: Root/reladdr2line.cmd +:: FILE: Root/raddr2line.cmd :: PURPOSE: Converts a value to hex and displays it. :: COPYRIGHT: Copyright 2009 Christoph von Wittich <Christoph_vW(a)reactos.org> :: Daniel Reimer <reimer.daniel(a)freenet.de>

15 years, 1 month

[sserapion] 43907: Fix bad cast.

by sserapion＠svn.reactos.org

Author: sserapion Date: Mon Nov 2 02:34:52 2009 New Revision: 43907 URL: http://svn.reactos.org/svn/reactos?rev=43907&view=rev Log: Fix bad cast. Modified: branches/ros-amd64-bringup/reactos/include/psdk/commctrl.h Modified: branches/ros-amd64-bringup/reactos/include/psdk/commctrl.h URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/inclu… ============================================================================== --- branches/ros-amd64-bringup/reactos/include/psdk/commctrl.h [iso-8859-1] (original) +++ branches/ros-amd64-bringup/reactos/include/psdk/commctrl.h [iso-8859-1] Mon Nov 2 02:34:52 2009 @@ -2802,7 +2802,7 @@ (LPARAM)(HTREEITEM)(hitem)) #define TreeView_EnsureVisible(hwnd, hitem) \ - (BOOL)SNDMSG((hwnd), TVM_ENSUREVISIBLE, 0, (LPARAM)(UINT)(hitem)) + (BOOL)SNDMSG((hwnd), TVM_ENSUREVISIBLE, 0, (LPARAM)(hitem)) #define TreeView_SortChildrenCB(hwnd, psort, recurse) \ (BOOL)SNDMSG((hwnd), TVM_SORTCHILDRENCB, (WPARAM)recurse, \

15 years, 1 month

[sserapion] 43906: Fix rapps build.

by sserapion＠svn.reactos.org

Author: sserapion Date: Mon Nov 2 02:30:09 2009 New Revision: 43906 URL: http://svn.reactos.org/svn/reactos?rev=43906&view=rev Log: Fix rapps build. Modified: branches/ros-amd64-bringup/reactos/base/applications/rapps/toolbar.c Modified: branches/ros-amd64-bringup/reactos/base/applications/rapps/toolbar.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/base/… ============================================================================== --- branches/ros-amd64-bringup/reactos/base/applications/rapps/toolbar.c [iso-8859-1] (original) +++ branches/ros-amd64-bringup/reactos/base/applications/rapps/toolbar.c [iso-8859-1] Mon Nov 2 02:30:09 2009 @@ -20,9 +20,9 @@ /* Toolbar buttons */ static const TBBUTTON Buttons[] = { /* iBitmap, idCommand, fsState, fsStyle, bReserved[2], dwData, iString */ - { 0, ID_INSTALL, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, {0}, 0, (int) szInstallBtn}, - { 1, ID_UNINSTALL, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, {0}, 0, (int) szUninstallBtn}, - { 2, ID_MODIFY, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, {0}, 0, (int) szModifyBtn}, + { 0, ID_INSTALL, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, {0}, 0, (INT_PTR)szInstallBtn}, + { 1, ID_UNINSTALL, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, {0}, 0, (INT_PTR)szUninstallBtn}, + { 2, ID_MODIFY, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, {0}, 0, (INT_PTR)szModifyBtn}, { 5, 0, TBSTATE_ENABLED, BTNS_SEP, {0}, 0, 0}, { 3, ID_REFRESH, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, {0}, 0, 0}, { 5, 0, TBSTATE_ENABLED, BTNS_SEP, {0}, 0, 0},

15 years, 1 month

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs November 2009