Ros-diffs June 2007

ros-diffs@reactos.org

21 participants
420 discussions

[hpoussin] 26993: Reapply r18676/r18677 (dispatch.S part) on w3seek advice. Fixes vmwinst.exe crash on 2nd stage in qemu
by hpoussin＠svn.reactos.org 08 Jun '07

08 Jun '07

Author: hpoussin Date: Mon Jun 4 18:49:28 2007 New Revision: 26993 URL: http://svn.reactos.org/svn/reactos?rev=26993&view=rev Log: Reapply r18676/r18677 (dispatch.S part) on w3seek advice. Fixes vmwinst.exe crash on 2nd stage in qemu Modified: trunk/reactos/dll/ntdll/main/i386/dispatch.S Modified: trunk/reactos/dll/ntdll/main/i386/dispatch.S URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/ntdll/main/i386/dispat… ============================================================================== --- trunk/reactos/dll/ntdll/main/i386/dispatch.S (original) +++ trunk/reactos/dll/ntdll/main/i386/dispatch.S Mon Jun 4 18:49:28 2007 @@ -211,9 +211,17 @@ mov ecx, [esp+4] mov ebx, [esp] - /* Dispatch the exception */ + /* Call the vectored exception handler */ push ecx push ebx + call _RtlpExecuteVectoredExceptionHandlers@8 + + /* Check for success */ + or al, al + jnz ContinueExecution + + /* Dispatch the exception */ + sub esp, 8 call _RtlDispatchException@8 /* Check for success */

1 0

[hpoussin] 26992: Add a bus number to the PCI resource descriptor
by hpoussin＠svn.reactos.org 08 Jun '07

08 Jun '07

Author: hpoussin Date: Mon Jun 4 14:01:29 2007 New Revision: 26992 URL: http://svn.reactos.org/svn/reactos?rev=26992&view=rev Log: Add a bus number to the PCI resource descriptor Modified: trunk/reactos/boot/freeldr/freeldr/arch/i386/hwpci.c Modified: trunk/reactos/boot/freeldr/freeldr/arch/i386/hwpci.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/arch/… ============================================================================== --- trunk/reactos/boot/freeldr/freeldr/arch/i386/hwpci.c (original) +++ trunk/reactos/boot/freeldr/freeldr/arch/i386/hwpci.c Mon Jun 4 14:01:29 2007 @@ -182,8 +182,8 @@ } /* Set 'Configuration Data' value */ - Size = sizeof(CM_FULL_RESOURCE_DESCRIPTOR) + - Table->Size; + Size = FIELD_OFFSET(CM_FULL_RESOURCE_DESCRIPTOR, PartialResourceList.PartialDescriptors) + + 2 * sizeof(CM_PARTIAL_RESOURCE_DESCRIPTOR) + Table->Size; FullResourceDescriptor = MmAllocateMemory(Size); if (FullResourceDescriptor == NULL) { @@ -196,14 +196,22 @@ memset(FullResourceDescriptor, 0, Size); FullResourceDescriptor->InterfaceType = Isa; FullResourceDescriptor->BusNumber = 0; - FullResourceDescriptor->PartialResourceList.Count = 1; + FullResourceDescriptor->PartialResourceList.Version = 1; + FullResourceDescriptor->PartialResourceList.Revision = 1; + FullResourceDescriptor->PartialResourceList.Count = 2; PartialDescriptor = &FullResourceDescriptor->PartialResourceList.PartialDescriptors[0]; + PartialDescriptor->Type = CmResourceTypeBusNumber; + PartialDescriptor->ShareDisposition = CmResourceShareDeviceExclusive; + PartialDescriptor->u.BusNumber.Start = 0; + PartialDescriptor->u.BusNumber.Length = 1; + + PartialDescriptor = &FullResourceDescriptor->PartialResourceList.PartialDescriptors[1]; PartialDescriptor->Type = CmResourceTypeDeviceSpecific; PartialDescriptor->ShareDisposition = CmResourceShareUndetermined; PartialDescriptor->u.DeviceSpecificData.DataSize = Table->Size; - memcpy((PVOID)((ULONG_PTR)FullResourceDescriptor + sizeof(CM_FULL_RESOURCE_DESCRIPTOR)), + memcpy(&FullResourceDescriptor->PartialResourceList.PartialDescriptors[2], Table, Table->Size);

1 0

[hpoussin] 26991: Copy usetup and welcome apps to the right cdrom directory
by hpoussin＠svn.reactos.org 08 Jun '07

08 Jun '07

Author: hpoussin Date: Mon Jun 4 13:44:46 2007 New Revision: 26991 URL: http://svn.reactos.org/svn/reactos?rev=26991&view=rev Log: Copy usetup and welcome apps to the right cdrom directory Modified: trunk/reactos/base/setup/usetup/usetup.rbuild trunk/reactos/base/setup/welcome/welcome.rbuild Modified: trunk/reactos/base/setup/usetup/usetup.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/setup/usetup/usetup.r… ============================================================================== --- trunk/reactos/base/setup/usetup/usetup.rbuild (original) +++ trunk/reactos/base/setup/usetup/usetup.rbuild Mon Jun 4 13:44:46 2007 @@ -1,5 +1,5 @@ <module name="usetup" type="nativecui" installbase="system32" installname="usetup.exe" allowwarnings="false"> - <bootstrap base="reactos/system32" nameoncd="smss.exe" /> + <bootstrap base="$(CDOUTPUT)/system32" nameoncd="smss.exe" /> <include base="usetup">.</include> <include base="zlib">.</include> <include base="inflib">.</include> Modified: trunk/reactos/base/setup/welcome/welcome.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/setup/welcome/welcome… ============================================================================== --- trunk/reactos/base/setup/welcome/welcome.rbuild (original) +++ trunk/reactos/base/setup/welcome/welcome.rbuild Mon Jun 4 13:44:46 2007 @@ -1,5 +1,5 @@ <module name="welcome" type="win32gui" installbase="system32" installname="welcome.exe"> - <bootstrap base="reactos" /> + <bootstrap base="$(CDOUTPUT)" /> <include base="welcome">.</include> <define name="_WIN32_IE">0x0501</define> <define name="_WIN32_WINNT">0x0501</define>

1 0

[hpoussin] 26990: Use a tag when allocating/freeing some memory
by hpoussin＠svn.reactos.org 08 Jun '07

08 Jun '07

Author: hpoussin Date: Mon Jun 4 13:25:04 2007 New Revision: 26990 URL: http://svn.reactos.org/svn/reactos?rev=26990&view=rev Log: Use a tag when allocating/freeing some memory Modified: trunk/reactos/drivers/input/kbdclass/kbdclass.c trunk/reactos/drivers/input/kbdclass/kbdclass.h trunk/reactos/drivers/input/kbdclass/misc.c trunk/reactos/drivers/input/mouclass/misc.c trunk/reactos/drivers/input/mouclass/mouclass.c trunk/reactos/drivers/input/mouclass/mouclass.h trunk/reactos/drivers/input/sermouse/fdo.c trunk/reactos/drivers/input/sermouse/sermouse.c trunk/reactos/drivers/input/sermouse/sermouse.h Modified: trunk/reactos/drivers/input/kbdclass/kbdclass.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/input/kbdclass/kbd… ============================================================================== --- trunk/reactos/drivers/input/kbdclass/kbdclass.c (original) +++ trunk/reactos/drivers/input/kbdclass/kbdclass.c Mon Jun 4 13:25:04 2007 @@ -9,6 +9,16 @@ #define INITGUID #include "kbdclass.h" + +static DRIVER_UNLOAD DriverUnload; +static DRIVER_DISPATCH ClassCreate; +static DRIVER_DISPATCH ClassClose; +static DRIVER_DISPATCH ClassCleanup; +static DRIVER_DISPATCH ClassRead; +static DRIVER_DISPATCH ClassDeviceControl; +static DRIVER_DISPATCH IrpStub; +static DRIVER_ADD_DEVICE ClassAddDevice; +static DRIVER_STARTIO ClassStartIo; static VOID NTAPI DriverUnload(IN PDRIVER_OBJECT DriverObject) @@ -212,11 +222,11 @@ ParametersRegistryKey.Length = 0; ParametersRegistryKey.MaximumLength = RegistryPath->Length + sizeof(L"\\Parameters") + sizeof(UNICODE_NULL); - ParametersRegistryKey.Buffer = ExAllocatePool(PagedPool, ParametersRegistryKey.MaximumLength); + ParametersRegistryKey.Buffer = ExAllocatePoolWithTag(PagedPool, ParametersRegistryKey.MaximumLength, CLASS_TAG); if (!ParametersRegistryKey.Buffer) { - DPRINT("ExAllocatePool() failed\n"); - return STATUS_INSUFFICIENT_RESOURCES; + DPRINT("ExAllocatePoolWithTag() failed\n"); + return STATUS_NO_MEMORY; } RtlCopyUnicodeString(&ParametersRegistryKey, RegistryPath); RtlAppendUnicodeToString(&ParametersRegistryKey, L"\\Parameters"); @@ -276,6 +286,7 @@ Status = STATUS_NO_MEMORY; } + ExFreePoolWithTag(ParametersRegistryKey.Buffer, CLASS_TAG); return Status; } @@ -303,11 +314,11 @@ + DriverExtension->DeviceBaseName.Length /* "KeyboardClass" */ + 4 * sizeof(WCHAR) /* Id between 0 and 9999 */ + sizeof(UNICODE_NULL); /* Final NULL char */ - DeviceNameU.Buffer = ExAllocatePool(PagedPool, DeviceNameU.MaximumLength); + DeviceNameU.Buffer = ExAllocatePoolWithTag(PagedPool, DeviceNameU.MaximumLength, CLASS_TAG); if (!DeviceNameU.Buffer) { - DPRINT("ExAllocatePool() failed\n"); - return STATUS_INSUFFICIENT_RESOURCES; + DPRINT("ExAllocatePoolWithTag() failed\n"); + return STATUS_NO_MEMORY; } Status = RtlAppendUnicodeToString(&DeviceNameU, L"\\Device\\"); if (!NT_SUCCESS(Status)) @@ -348,7 +359,7 @@ cleanup: if (!NT_SUCCESS(Status)) { - ExFreePool(DeviceNameU.Buffer); + ExFreePoolWithTag(DeviceNameU.Buffer, CLASS_TAG); return Status; } @@ -361,11 +372,11 @@ KeInitializeSpinLock(&DeviceExtension->SpinLock); DeviceExtension->ReadIsPending = FALSE; DeviceExtension->InputCount = 0; - DeviceExtension->PortData = ExAllocatePool(NonPagedPool, DeviceExtension->DriverExtension->DataQueueSize * sizeof(KEYBOARD_INPUT_DATA)); + DeviceExtension->PortData = ExAllocatePoolWithTag(NonPagedPool, DeviceExtension->DriverExtension->DataQueueSize * sizeof(KEYBOARD_INPUT_DATA), CLASS_TAG); if (!DeviceExtension->PortData) { - ExFreePool(DeviceNameU.Buffer); - return STATUS_INSUFFICIENT_RESOURCES; + ExFreePoolWithTag(DeviceNameU.Buffer, CLASS_TAG); + return STATUS_NO_MEMORY; } DeviceExtension->DeviceName = DeviceNameU.Buffer; Fdo->Flags |= DO_POWER_PAGABLE; @@ -548,8 +559,7 @@ CONNECT_DATA ConnectData; NTSTATUS Status; - DPRINT("Connecting PortDO %p [%wZ] to ClassDO %p\n", - PortDO, &PortDO->DriverObject->DriverName, ClassDO); + DPRINT("Connecting PortDO %p to ClassDO %p\n", PortDO, ClassDO); KeInitializeEvent(&Event, NotificationEvent, FALSE); @@ -605,8 +615,7 @@ KIRQL OldIrql; NTSTATUS Status; - DPRINT("Destroying PortDO %p [%wZ]\n", - PortDO, &PortDO->DriverObject->DriverName); + DPRINT("Destroying PortDO %p\n", PortDO); DeviceExtension = (PPORT_DEVICE_EXTENSION)PortDO->DeviceExtension; ClassDeviceExtension = DeviceExtension->ClassDO->DeviceExtension; @@ -644,8 +653,8 @@ if (!DriverExtension->ConnectMultiplePorts && DeviceExtension->ClassDO) { - ExFreePool(ClassDeviceExtension->PortData); - ExFreePool((PVOID)ClassDeviceExtension->DeviceName); + ExFreePoolWithTag(ClassDeviceExtension->PortData, CLASS_TAG); + ExFreePoolWithTag((PVOID)ClassDeviceExtension->DeviceName, CLASS_TAG); IoDeleteDevice(DeviceExtension->ClassDO); } @@ -838,11 +847,11 @@ /* Allocate memory */ Size = sizeof(KEY_VALUE_BASIC_INFORMATION) + MAX_PATH; - KeyValueInformation = ExAllocatePool(PagedPool, Size); + KeyValueInformation = ExAllocatePoolWithTag(PagedPool, Size, CLASS_TAG); if (!KeyValueInformation) { - DPRINT("ExAllocatePool() failed\n"); - Status = STATUS_INSUFFICIENT_RESOURCES; + DPRINT("ExAllocatePoolWithTag() failed\n"); + Status = STATUS_NO_MEMORY; goto cleanup; } @@ -893,7 +902,7 @@ DPRINT("IoGetDeviceObjectPointer(%wZ) failed with status 0x%08lx\n", &PortName, Status); continue; } - DPRINT("Legacy driver found: %wZ\n", &PortDeviceObject->DriverObject->DriverName); + DPRINT("Legacy driver found\n"); Status = ClassAddDevice(DriverObject, PortDeviceObject); if (!NT_SUCCESS(Status)) @@ -905,7 +914,7 @@ cleanup: if (KeyValueInformation != NULL) - ExFreePool(KeyValueInformation); + ExFreePoolWithTag(KeyValueInformation, CLASS_TAG); if (hDeviceMapKey != (HANDLE)-1) ZwClose(hDeviceMapKey); if (hPortKey != (HANDLE)-1) Modified: trunk/reactos/drivers/input/kbdclass/kbdclass.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/input/kbdclass/kbd… ============================================================================== --- trunk/reactos/drivers/input/kbdclass/kbdclass.h (original) +++ trunk/reactos/drivers/input/kbdclass/kbdclass.h Mon Jun 4 13:25:04 2007 @@ -2,26 +2,15 @@ #include <kbdmou.h> #include <ntddkbd.h> #include <stdio.h> - -#if defined(__GNUC__) - #include <pseh/pseh.h> - #include <debug.h> -#elif defined(_MSC_VER) - #define DPRINT1 DbgPrint("(%s:%d) ", __FILE__, __LINE__), DbgPrint - #define CHECKPOINT1 DbgPrint("(%s:%d)\n", __FILE__, __LINE__) - #define DPRINT - #define CHECKPOINT - #define _SEH_TRY __try - #define _SEH_HANDLE __except(1) - #define _SEH_END - #define _SEH_GetExceptionCode() GetExceptionCode() -#else - #error Unknown compiler! -#endif +#include <pseh/pseh.h> +#include <debug.h> #define MAX_PATH 260 #define MIN(a, b) ((a) < (b) ? (a) : (b)) + +#define TAG(A, B, C, D) (ULONG)(((A)<<0) + ((B)<<8) + ((C)<<16) + ((D)<<24)) +#define CLASS_TAG TAG('K', 'b', 'd', 'C') typedef enum { @@ -83,10 +72,7 @@ IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp); -NTSTATUS NTAPI -ForwardIrpAndForget( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp); +DRIVER_DISPATCH ForwardIrpAndForget; NTSTATUS DuplicateUnicodeString( Modified: trunk/reactos/drivers/input/kbdclass/misc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/input/kbdclass/mis… ============================================================================== --- trunk/reactos/drivers/input/kbdclass/misc.c (original) +++ trunk/reactos/drivers/input/kbdclass/misc.c Mon Jun 4 13:25:04 2007 @@ -8,6 +8,8 @@ */ #include "kbdclass.h" + +static IO_COMPLETION_ROUTINE ForwardIrpAndWaitCompletion; static NTSTATUS NTAPI ForwardIrpAndWaitCompletion( @@ -35,7 +37,7 @@ KeInitializeEvent(&Event, NotificationEvent, FALSE); IoCopyCurrentIrpStackLocationToNext(Irp); - DPRINT("Calling lower device %p [%wZ]\n", LowerDevice, &LowerDevice->DriverObject->DriverName); + DPRINT("Calling lower device %p\n", LowerDevice); IoSetCompletionRoutine(Irp, ForwardIrpAndWaitCompletion, &Event, TRUE, TRUE, TRUE); Status = IoCallDriver(LowerDevice, Irp); @@ -93,7 +95,7 @@ if (Flags & RTL_DUPLICATE_UNICODE_STRING_NULL_TERMINATE) DestMaxLength += sizeof(UNICODE_NULL); - DestinationString->Buffer = ExAllocatePool(PagedPool, DestMaxLength); + DestinationString->Buffer = ExAllocatePoolWithTag(PagedPool, DestMaxLength, CLASS_TAG); if (DestinationString->Buffer == NULL) return STATUS_NO_MEMORY; Modified: trunk/reactos/drivers/input/mouclass/misc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/input/mouclass/mis… ============================================================================== --- trunk/reactos/drivers/input/mouclass/misc.c (original) +++ trunk/reactos/drivers/input/mouclass/misc.c Mon Jun 4 13:25:04 2007 @@ -8,6 +8,8 @@ */ #include "mouclass.h" + +static IO_COMPLETION_ROUTINE ForwardIrpAndWaitCompletion; static NTSTATUS NTAPI ForwardIrpAndWaitCompletion( @@ -35,7 +37,7 @@ KeInitializeEvent(&Event, NotificationEvent, FALSE); IoCopyCurrentIrpStackLocationToNext(Irp); - DPRINT("Calling lower device %p [%wZ]\n", LowerDevice, &LowerDevice->DriverObject->DriverName); + DPRINT("Calling lower device %p\n", LowerDevice); IoSetCompletionRoutine(Irp, ForwardIrpAndWaitCompletion, &Event, TRUE, TRUE, TRUE); Status = IoCallDriver(LowerDevice, Irp); @@ -93,7 +95,7 @@ if (Flags & RTL_DUPLICATE_UNICODE_STRING_NULL_TERMINATE) DestMaxLength += sizeof(UNICODE_NULL); - DestinationString->Buffer = ExAllocatePool(PagedPool, DestMaxLength); + DestinationString->Buffer = ExAllocatePoolWithTag(PagedPool, DestMaxLength, CLASS_TAG); if (DestinationString->Buffer == NULL) return STATUS_NO_MEMORY; Modified: trunk/reactos/drivers/input/mouclass/mouclass.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/input/mouclass/mou… ============================================================================== --- trunk/reactos/drivers/input/mouclass/mouclass.c (original) +++ trunk/reactos/drivers/input/mouclass/mouclass.c Mon Jun 4 13:25:04 2007 @@ -9,6 +9,16 @@ #define INITGUID #include "mouclass.h" + +static DRIVER_UNLOAD DriverUnload; +static DRIVER_DISPATCH ClassCreate; +static DRIVER_DISPATCH ClassClose; +static DRIVER_DISPATCH ClassCleanup; +static DRIVER_DISPATCH ClassRead; +static DRIVER_DISPATCH ClassDeviceControl; +static DRIVER_DISPATCH IrpStub; +static DRIVER_ADD_DEVICE ClassAddDevice; +static DRIVER_STARTIO ClassStartIo; static VOID NTAPI DriverUnload(IN PDRIVER_OBJECT DriverObject) @@ -189,11 +199,11 @@ ParametersRegistryKey.Length = 0; ParametersRegistryKey.MaximumLength = RegistryPath->Length + sizeof(L"\\Parameters") + sizeof(UNICODE_NULL); - ParametersRegistryKey.Buffer = ExAllocatePool(PagedPool, ParametersRegistryKey.MaximumLength); + ParametersRegistryKey.Buffer = ExAllocatePoolWithTag(PagedPool, ParametersRegistryKey.MaximumLength, CLASS_TAG); if (!ParametersRegistryKey.Buffer) { - DPRINT("ExAllocatePool() failed\n"); - return STATUS_INSUFFICIENT_RESOURCES; + DPRINT("ExAllocatePoolWithTag() failed\n"); + return STATUS_NO_MEMORY; } RtlCopyUnicodeString(&ParametersRegistryKey, RegistryPath); RtlAppendUnicodeToString(&ParametersRegistryKey, L"\\Parameters"); @@ -253,6 +263,7 @@ Status = STATUS_NO_MEMORY; } + ExFreePoolWithTag(ParametersRegistryKey.Buffer, CLASS_TAG); return Status; } @@ -280,11 +291,11 @@ + DriverExtension->DeviceBaseName.Length /* "PointerClass" */ + 4 * sizeof(WCHAR) /* Id between 0 and 9999 */ + sizeof(UNICODE_NULL); /* Final NULL char */ - DeviceNameU.Buffer = ExAllocatePool(PagedPool, DeviceNameU.MaximumLength); + DeviceNameU.Buffer = ExAllocatePoolWithTag(PagedPool, DeviceNameU.MaximumLength, CLASS_TAG); if (!DeviceNameU.Buffer) { - DPRINT("ExAllocatePool() failed\n"); - return STATUS_INSUFFICIENT_RESOURCES; + DPRINT("ExAllocatePoolWithTag() failed\n"); + return STATUS_NO_MEMORY; } Status = RtlAppendUnicodeToString(&DeviceNameU, L"\\Device\\"); if (!NT_SUCCESS(Status)) @@ -325,7 +336,7 @@ cleanup: if (!NT_SUCCESS(Status)) { - ExFreePool(DeviceNameU.Buffer); + ExFreePoolWithTag(DeviceNameU.Buffer, CLASS_TAG); return Status; } @@ -338,11 +349,11 @@ KeInitializeSpinLock(&DeviceExtension->SpinLock); DeviceExtension->ReadIsPending = FALSE; DeviceExtension->InputCount = 0; - DeviceExtension->PortData = ExAllocatePool(NonPagedPool, DeviceExtension->DriverExtension->DataQueueSize * sizeof(MOUSE_INPUT_DATA)); + DeviceExtension->PortData = ExAllocatePoolWithTag(NonPagedPool, DeviceExtension->DriverExtension->DataQueueSize * sizeof(KEYBOARD_INPUT_DATA), CLASS_TAG); if (!DeviceExtension->PortData) { - ExFreePool(DeviceNameU.Buffer); - return STATUS_INSUFFICIENT_RESOURCES; + ExFreePoolWithTag(DeviceNameU.Buffer, CLASS_TAG); + return STATUS_NO_MEMORY; } DeviceExtension->DeviceName = DeviceNameU.Buffer; Fdo->Flags |= DO_POWER_PAGABLE; @@ -524,8 +535,7 @@ CONNECT_DATA ConnectData; NTSTATUS Status; - DPRINT("Connecting PortDO %p [%wZ] to ClassDO %p\n", - PortDO, &PortDO->DriverObject->DriverName, ClassDO); + DPRINT("Connecting PortDO %p to ClassDO %p\n", PortDO, ClassDO); KeInitializeEvent(&Event, NotificationEvent, FALSE); @@ -581,8 +591,7 @@ KIRQL OldIrql; NTSTATUS Status; - DPRINT("Destroying PortDO %p [%wZ]\n", - PortDO, &PortDO->DriverObject->DriverName); + DPRINT("Destroying PortDO %p\n", PortDO); DeviceExtension = (PPORT_DEVICE_EXTENSION)PortDO->DeviceExtension; ClassDeviceExtension = DeviceExtension->ClassDO->DeviceExtension; @@ -620,8 +629,8 @@ if (!DriverExtension->ConnectMultiplePorts && DeviceExtension->ClassDO) { - ExFreePool(ClassDeviceExtension->PortData); - ExFreePool((PVOID)ClassDeviceExtension->DeviceName); + ExFreePoolWithTag(ClassDeviceExtension->PortData, CLASS_TAG); + ExFreePoolWithTag((PVOID)ClassDeviceExtension->DeviceName, CLASS_TAG); IoDeleteDevice(DeviceExtension->ClassDO); } @@ -814,11 +823,11 @@ /* Allocate memory */ Size = sizeof(KEY_VALUE_BASIC_INFORMATION) + MAX_PATH; - KeyValueInformation = ExAllocatePool(PagedPool, Size); + KeyValueInformation = ExAllocatePoolWithTag(PagedPool, Size, CLASS_TAG); if (!KeyValueInformation) { - DPRINT("ExAllocatePool() failed\n"); - Status = STATUS_INSUFFICIENT_RESOURCES; + DPRINT("ExAllocatePoolWithTag() failed\n"); + Status = STATUS_NO_MEMORY; goto cleanup; } @@ -869,7 +878,7 @@ DPRINT("IoGetDeviceObjectPointer(%wZ) failed with status 0x%08lx\n", &PortName, Status); continue; } - DPRINT("Legacy driver found: %wZ\n", &PortDeviceObject->DriverObject->DriverName); + DPRINT("Legacy driver found\n"); Status = ClassAddDevice(DriverObject, PortDeviceObject); if (!NT_SUCCESS(Status)) @@ -881,7 +890,7 @@ cleanup: if (KeyValueInformation != NULL) - ExFreePool(KeyValueInformation); + ExFreePoolWithTag(KeyValueInformation, CLASS_TAG); if (hDeviceMapKey != (HANDLE)-1) ZwClose(hDeviceMapKey); if (hPortKey != (HANDLE)-1) Modified: trunk/reactos/drivers/input/mouclass/mouclass.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/input/mouclass/mou… ============================================================================== --- trunk/reactos/drivers/input/mouclass/mouclass.h (original) +++ trunk/reactos/drivers/input/mouclass/mouclass.h Mon Jun 4 13:25:04 2007 @@ -2,26 +2,15 @@ #include <kbdmou.h> #include <ntddkbd.h> #include <stdio.h> - -#if defined(__GNUC__) - #include <pseh/pseh.h> - #include <debug.h> -#elif defined(_MSC_VER) - #define DPRINT1 DbgPrint("(%s:%d) ", __FILE__, __LINE__), DbgPrint - #define CHECKPOINT1 DbgPrint("(%s:%d)\n", __FILE__, __LINE__) - #define DPRINT - #define CHECKPOINT - #define _SEH_TRY __try - #define _SEH_HANDLE __except(1) - #define _SEH_END - #define _SEH_GetExceptionCode() GetExceptionCode() -#else - #error Unknown compiler! -#endif +#include <pseh/pseh.h> +#include <debug.h> #define MAX_PATH 260 #define MIN(a, b) ((a) < (b) ? (a) : (b)) + +#define TAG(A, B, C, D) (ULONG)(((A)<<0) + ((B)<<8) + ((C)<<16) + ((D)<<24)) +#define CLASS_TAG TAG('M', 'o', 'u', 'C') typedef enum { @@ -83,10 +72,7 @@ IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp); -NTSTATUS NTAPI -ForwardIrpAndForget( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp); +DRIVER_DISPATCH ForwardIrpAndForget; NTSTATUS DuplicateUnicodeString( Modified: trunk/reactos/drivers/input/sermouse/fdo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/input/sermouse/fdo… ============================================================================== --- trunk/reactos/drivers/input/sermouse/fdo.c (original) +++ trunk/reactos/drivers/input/sermouse/fdo.c Mon Jun 4 13:25:04 2007 @@ -186,9 +186,12 @@ PDEVICE_RELATIONS DeviceRelations = NULL; DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_DEVICE_RELATIONS / TargetDeviceRelation\n"); - DeviceRelations = ExAllocatePool(PagedPool, FIELD_OFFSET(DEVICE_RELATIONS, Objects)); + DeviceRelations = ExAllocatePoolWithTag(PagedPool, FIELD_OFFSET(DEVICE_RELATIONS, Objects), SERMOUSE_TAG); if (!DeviceRelations) - Status = STATUS_INSUFFICIENT_RESOURCES; + { + DPRINT("ExAllocatePoolWithTag() failed\n"); + Status = STATUS_NO_MEMORY; + } else { DeviceRelations->Count = 0; Modified: trunk/reactos/drivers/input/sermouse/sermouse.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/input/sermouse/ser… ============================================================================== --- trunk/reactos/drivers/input/sermouse/sermouse.c (original) +++ trunk/reactos/drivers/input/sermouse/sermouse.c Mon Jun 4 13:25:04 2007 @@ -43,11 +43,11 @@ ParametersRegistryKey.Length = 0; ParametersRegistryKey.MaximumLength = RegistryPath->Length + sizeof(L"\\Parameters") + sizeof(UNICODE_NULL); - ParametersRegistryKey.Buffer = ExAllocatePool(PagedPool, ParametersRegistryKey.MaximumLength); + ParametersRegistryKey.Buffer = ExAllocatePoolWithTag(PagedPool, ParametersRegistryKey.MaximumLength, SERMOUSE_TAG); if (!ParametersRegistryKey.Buffer) { - DPRINT("ExAllocatePool() failed\n"); - return STATUS_INSUFFICIENT_RESOURCES; + DPRINT("ExAllocatePoolWithTag() failed\n"); + return STATUS_NO_MEMORY; } RtlCopyUnicodeString(&ParametersRegistryKey, RegistryPath); RtlAppendUnicodeToString(&ParametersRegistryKey, L"\\Parameters"); @@ -80,7 +80,7 @@ Status = STATUS_SUCCESS; } - ExFreePool(ParametersRegistryKey.Buffer); + ExFreePoolWithTag(ParametersRegistryKey.Buffer, SERMOUSE_TAG); return Status; } Modified: trunk/reactos/drivers/input/sermouse/sermouse.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/input/sermouse/ser… ============================================================================== --- trunk/reactos/drivers/input/sermouse/sermouse.h (original) +++ trunk/reactos/drivers/input/sermouse/sermouse.h Mon Jun 4 13:25:04 2007 @@ -2,17 +2,10 @@ #include <kbdmou.h> #include <ntddser.h> #include <ntddmou.h> +#include <debug.h> -#if defined(__GNUC__) - #include <debug.h> -#elif defined(_MSC_VER) - #define DPRINT1 DbgPrint("(%s:%d) ", __FILE__, __LINE__), DbgPrint - #define CHECKPOINT1 DbgPrint("(%s:%d)\n", __FILE__, __LINE__) - #define DPRINT - #define CHECKPOINT -#else - #error Unknown compiler! -#endif +#define TAG(A, B, C, D) (ULONG)(((A)<<0) + ((B)<<8) + ((C)<<16) + ((D)<<24)) +#define SERMOUSE_TAG TAG('S', 'M', 'o', 'u') typedef enum {

1 0

[hpoussin] 26989: Install ISA PNP bus driver Add NO_DRV.HW section, as it is needed by some 3rd party drivers
by hpoussin＠svn.reactos.org 08 Jun '07

08 Jun '07

Author: hpoussin Date: Mon Jun 4 13:17:06 2007 New Revision: 26989 URL: http://svn.reactos.org/svn/reactos?rev=26989&view=rev Log: Install ISA PNP bus driver Add NO_DRV.HW section, as it is needed by some 3rd party drivers Modified: trunk/reactos/media/inf/machine.inf Modified: trunk/reactos/media/inf/machine.inf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/inf/machine.inf?rev=… ============================================================================== --- trunk/reactos/media/inf/machine.inf (original) +++ trunk/reactos/media/inf/machine.inf Mon Jun 4 13:17:06 2007 @@ -9,7 +9,7 @@ Class = System ClassGUID = {4D36E97D-E325-11CE-BFC1-08002BE10318} Provider = %ReactOS% -DriverVer = 10/11/2005,1.01.0.0 +DriverVer = 08/07/2006,1.02 [DestinationDirs] DefaultDestDir = 12 @@ -66,6 +66,8 @@ [NO_DRV.Services] AddService = , 0x00000002 +[NO_DRV.HW] + ;-------------------------- PCI-ISA BUS DRIVER -------------------------- [PCI_ISA_Inst.NT] @@ -75,7 +77,6 @@ isapnp.sys [PCI_ISA_Inst.NT.Services] -;DO NOT INSTALL ISAPNP -> FREEZES SYSTEM ;AddService = isapnp, 0x00000002, isapnp_Service_Inst AddService = , 0x00000002

1 0

[hpoussin] 26988: Rewrite pnproot: - Don't create multiple devices with the same pnp id (Root\UNKNOWN\0000) - Use RtlQueryRegistryValues to read multiple registry values - Keep devices list in memory instead of always reading the registry - Support IRP_MN_QUERY_DEVICE_TEXT/IRP_MN_QUERY_ID/IRP_MN_QUERY_BUS_INFORMATION/IRP_MN_FILTER_RESOURCE_REQUIREMENTS - Better error handling - ...
by hpoussin＠svn.reactos.org 08 Jun '07

08 Jun '07

Author: hpoussin Date: Mon Jun 4 13:15:04 2007 New Revision: 26988 URL: http://svn.reactos.org/svn/reactos?rev=26988&view=rev Log: Rewrite pnproot: - Don't create multiple devices with the same pnp id (Root\UNKNOWN\0000) - Use RtlQueryRegistryValues to read multiple registry values - Keep devices list in memory instead of always reading the registry - Support IRP_MN_QUERY_DEVICE_TEXT/IRP_MN_QUERY_ID/IRP_MN_QUERY_BUS_INFORMATION/IRP_MN_FILTER_RESOURCE_REQUIREMENTS - Better error handling - ... Modified: trunk/reactos/ntoskrnl/io/pnpmgr/pnproot.c Modified: trunk/reactos/ntoskrnl/io/pnpmgr/pnproot.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/pnpmgr/pnproot… ============================================================================== --- trunk/reactos/ntoskrnl/io/pnpmgr/pnproot.c (original) +++ trunk/reactos/ntoskrnl/io/pnpmgr/pnproot.c Mon Jun 4 13:15:04 2007 @@ -5,6 +5,7 @@ * PURPOSE: PnP manager root device * * PROGRAMMERS: Casper S. Hornstrup (chorns(a)users.sourceforge.net) + * Copyright 2007 Hervé Poussineau (hpoussin(a)reactos.org) */ /* INCLUDES ******************************************************************/ @@ -25,19 +26,18 @@ LIST_ENTRY ListEntry; // Physical Device Object of device PDEVICE_OBJECT Pdo; - // Service name - UNICODE_STRING ServiceName; // Device ID UNICODE_STRING DeviceID; // Instance ID UNICODE_STRING InstanceID; // Device description UNICODE_STRING DeviceDescription; - // Boot resource list - PCM_FULL_RESOURCE_DESCRIPTOR BootResourceList; - SIZE_T BootResourceListSize; // Resource requirement list PIO_RESOURCE_REQUIREMENTS_LIST ResourceRequirementsList; + ULONG ResourceRequirementsListSize; + // Associated resource list + PCM_RESOURCE_LIST ResourceList; + ULONG ResourceListSize; } PNPROOT_DEVICE, *PPNPROOT_DEVICE; typedef enum @@ -49,39 +49,26 @@ dsSurpriseRemoved } PNPROOT_DEVICE_STATE; - -#include <pshpack1.h> - typedef struct _PNPROOT_COMMON_DEVICE_EXTENSION { - // Pointer to device object, this device extension is associated with - PDEVICE_OBJECT DeviceObject; // Wether this device extension is for an FDO or PDO BOOLEAN IsFDO; - // Wether the device is removed - BOOLEAN Removed; - // Current device power state for the device - DEVICE_POWER_STATE DevicePowerState; } PNPROOT_COMMON_DEVICE_EXTENSION, *PPNPROOT_COMMON_DEVICE_EXTENSION; - /* Physical Device Object device extension for a child device */ typedef struct _PNPROOT_PDO_DEVICE_EXTENSION { // Common device data PNPROOT_COMMON_DEVICE_EXTENSION Common; - // Device - PPNPROOT_DEVICE Device; + // Informations about the device + PPNPROOT_DEVICE DeviceInfo; } PNPROOT_PDO_DEVICE_EXTENSION, *PPNPROOT_PDO_DEVICE_EXTENSION; - -/* Functional Device Object device extension for the PCI driver device object */ +/* Physical Device Object device extension for the Root bus device object */ typedef struct _PNPROOT_FDO_DEVICE_EXTENSION { // Common device data PNPROOT_COMMON_DEVICE_EXTENSION Common; - // Physical Device Object - PDEVICE_OBJECT Pdo; // Lower device object PDEVICE_OBJECT Ldo; // Current state of the driver @@ -91,43 +78,110 @@ // Number of (not removed) devices in device list ULONG DeviceListCount; // Lock for namespace device list - // FIXME: Use fast mutex instead? - KSPIN_LOCK DeviceListLock; + KGUARDED_MUTEX DeviceListLock; } PNPROOT_FDO_DEVICE_EXTENSION, *PPNPROOT_FDO_DEVICE_EXTENSION; -#include <poppack.h> - - - -PDEVICE_OBJECT PnpRootDeviceObject; - +typedef struct _BUFFER +{ + PVOID *Buffer; + PULONG Length; +} BUFFER, *PBUFFER; + +static PDEVICE_OBJECT PnpRootDeviceObject = NULL; /* FUNCTIONS *****************************************************************/ -/* Physical Device Object routines */ - +static NTSTATUS +LocateChildDevice( + IN PPNPROOT_FDO_DEVICE_EXTENSION DeviceExtension, + IN PCWSTR DeviceId, + IN PCWSTR InstanceId, + OUT PPNPROOT_DEVICE* ChildDevice) +{ + PPNPROOT_DEVICE Device; + UNICODE_STRING DeviceIdU, InstanceIdU; + + RtlInitUnicodeString(&DeviceIdU, DeviceId); + RtlInitUnicodeString(&InstanceIdU, InstanceId); + + LIST_FOR_EACH(Device, &DeviceExtension->DeviceListHead, PNPROOT_DEVICE, ListEntry) + { + if (RtlEqualUnicodeString(&DeviceIdU, &Device->DeviceID, TRUE) + && RtlEqualUnicodeString(&InstanceIdU, &Device->InstanceID, TRUE)) + { + *ChildDevice = Device; + return STATUS_SUCCESS; + } + } + + return STATUS_NO_SUCH_DEVICE; +} + +/* Creates a new PnP device for a legacy driver */ NTSTATUS PnpRootCreateDevice( - PUNICODE_STRING ServiceName, - PDEVICE_OBJECT *PhysicalDeviceObject) -{ + IN PUNICODE_STRING ServiceName, + IN PDEVICE_OBJECT *PhysicalDeviceObject) +{ + PPNPROOT_FDO_DEVICE_EXTENSION DeviceExtension; + UNICODE_STRING UnknownServiceName = RTL_CONSTANT_STRING(L"UNKNOWN"); + PUNICODE_STRING LocalServiceName; PPNPROOT_PDO_DEVICE_EXTENSION PdoDeviceExtension; - PPNPROOT_FDO_DEVICE_EXTENSION DeviceExtension; - PPNPROOT_DEVICE Device; + WCHAR DevicePath[MAX_PATH + 1]; + WCHAR InstancePath[5]; + PPNPROOT_DEVICE Device = NULL; NTSTATUS Status; - - /* This function should be obsoleted soon */ - - DPRINT("Called\n"); - - DeviceExtension = (PPNPROOT_FDO_DEVICE_EXTENSION)PnpRootDeviceObject->DeviceExtension; - - Device = (PPNPROOT_DEVICE)ExAllocatePoolWithTag(PagedPool, sizeof(PNPROOT_DEVICE), TAG_PNP_ROOT); + ULONG i; + + DeviceExtension = PnpRootDeviceObject->DeviceExtension; + KeAcquireGuardedMutex(&DeviceExtension->DeviceListLock); + + if (ServiceName) + LocalServiceName = ServiceName; + else + LocalServiceName = &UnknownServiceName; + + DPRINT("Creating a PnP root device for service '%wZ'\n", LocalServiceName); + + /* Search for a free instance ID */ + _snwprintf(DevicePath, sizeof(DevicePath) / sizeof(WCHAR), L"%s\\LEGACY_%wZ", REGSTR_KEY_ROOTENUM, LocalServiceName); + for (i = 0; i < 9999; i++) + { + _snwprintf(InstancePath, sizeof(InstancePath) / sizeof(WCHAR), L"%04lu", i); + Status = LocateChildDevice(DeviceExtension, DevicePath, InstancePath, &Device); + if (Status == STATUS_NO_SUCH_DEVICE) + break; + } + if (i > 9999) + { + DPRINT1("Too much legacy devices reported for service '%wZ'\n", &LocalServiceName); + Status = STATUS_INSUFFICIENT_RESOURCES; + goto cleanup; + } + + /* FIXME: save the new device to registry? */ + + /* Initialize a PNPROOT_DEVICE structure */ + Device = ExAllocatePoolWithTag(PagedPool, sizeof(PNPROOT_DEVICE), TAG_PNP_ROOT); if (!Device) - return STATUS_INSUFFICIENT_RESOURCES; - - RtlZeroMemory(Device, sizeof(PNPROOT_DEVICE)); - + { + DPRINT("ExAllocatePoolWithTag() failed\n"); + Status = STATUS_NO_MEMORY; + goto cleanup; + } + RtlZeroMemory(Device, sizeof(Device)); + if (!RtlCreateUnicodeString(&Device->DeviceID, DevicePath)) + { + Status = STATUS_NO_MEMORY; + goto cleanup; + } + if (!RtlCreateUnicodeString(&Device->InstanceID, InstancePath)) + { + Status = STATUS_NO_MEMORY; + goto cleanup; + } + + /* Initialize a device object */ Status = IoCreateDevice( PnpRootDeviceObject->DriverObject, sizeof(PNPROOT_PDO_DEVICE_EXTENSION), @@ -136,564 +190,198 @@ FILE_AUTOGENERATED_DEVICE_NAME, FALSE, &Device->Pdo); - if (!NT_SUCCESS(Status)) { - DPRINT("IoCreateDevice() failed with status 0x%X\n", Status); - ExFreePool(Device); - return Status; - } + if (!NT_SUCCESS(Status)) + { + DPRINT("IoCreateDevice() failed with status 0x%08lx\n", Status); + Status = STATUS_NO_MEMORY; + goto cleanup; + } + + PdoDeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)Device->Pdo->DeviceExtension; + RtlZeroMemory(PdoDeviceExtension, sizeof(PNPROOT_PDO_DEVICE_EXTENSION)); + PdoDeviceExtension->Common.IsFDO = FALSE; + PdoDeviceExtension->DeviceInfo = Device; Device->Pdo->Flags |= DO_BUS_ENUMERATED_DEVICE; - Device->Pdo->Flags &= ~DO_DEVICE_INITIALIZING; - //Device->Pdo->Flags |= DO_POWER_PAGABLE; - - PdoDeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)Device->Pdo->DeviceExtension; - - RtlZeroMemory(PdoDeviceExtension, sizeof(PNPROOT_PDO_DEVICE_EXTENSION)); - - PdoDeviceExtension->Common.IsFDO = FALSE; - - PdoDeviceExtension->Common.DeviceObject = Device->Pdo; - - PdoDeviceExtension->Common.DevicePowerState = PowerDeviceD0; - - ExInterlockedInsertTailList( + InsertTailList( &DeviceExtension->DeviceListHead, - &Device->ListEntry, - &DeviceExtension->DeviceListLock); - - DeviceExtension->DeviceListCount++; + &Device->ListEntry); + DeviceExtension->DeviceListCount++; *PhysicalDeviceObject = Device->Pdo; - - return STATUS_SUCCESS; -} - - -NTSTATUS -PdoQueryId( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp, - PIO_STACK_LOCATION IrpSp) -{ - PPNPROOT_PDO_DEVICE_EXTENSION DeviceExtension; - UNICODE_STRING String; - NTSTATUS Status; - - DPRINT("Called\n"); - - DeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - -// Irp->IoStatus.Information = 0; - + DPRINT("Created PDO %p (%wZ\\%wZ)\n", *PhysicalDeviceObject, &Device->DeviceID, &Device->InstanceID); + Device = NULL; Status = STATUS_SUCCESS; - RtlInitUnicodeString(&String, NULL); - - switch (IrpSp->Parameters.QueryId.IdType) { - case BusQueryDeviceID: - if (DeviceExtension->Device) - Status = RtlDuplicateUnicodeString(TRUE, - &DeviceExtension->Device->DeviceID, - &String); - else - Status = RtlCreateUnicodeString(&String, ENUM_NAME_ROOT L"\\LEGACY_UNKNOWN"); - - DPRINT("DeviceID: %wZ\n", &String); - - Irp->IoStatus.Information = (ULONG_PTR)String.Buffer; - break; - - case BusQueryHardwareIDs: - case BusQueryCompatibleIDs: - Status = STATUS_NOT_IMPLEMENTED; - break; - - case BusQueryInstanceID: - if (DeviceExtension->Device) - Status = RtlDuplicateUnicodeString(TRUE, - &DeviceExtension->Device->InstanceID, - &String); - else - Status = RtlCreateUnicodeString(&String, L"0000"); - - DPRINT("InstanceID: %S\n", String.Buffer); - - Irp->IoStatus.Information = (ULONG_PTR)String.Buffer; - break; - - case BusQueryDeviceSerialNumber: - default: - Status = STATUS_NOT_IMPLEMENTED; - } - +cleanup: + KeReleaseGuardedMutex(&DeviceExtension->DeviceListLock); + if (Device) + { + if (Device->Pdo) + IoDeleteDevice(Device->Pdo); + RtlFreeUnicodeString(&Device->DeviceID); + RtlFreeUnicodeString(&Device->InstanceID); + ExFreePoolWithTag(Device, TAG_PNP_ROOT); + } return Status; } - -NTSTATUS -PdoQueryResources( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp, - PIO_STACK_LOCATION IrpSp) -{ - PPNPROOT_PDO_DEVICE_EXTENSION DeviceExtension; - PCM_RESOURCE_LIST ResourceList; - ULONG ResourceListSize = FIELD_OFFSET(CM_RESOURCE_LIST, List); - - DPRINT("Called\n"); - - DeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - - if (!DeviceExtension->Device || DeviceExtension->Device->BootResourceList == NULL) - { - /* Create an empty resource list */ - ResourceList = ExAllocatePool(PagedPool, ResourceListSize); - if (ResourceList == NULL) - return STATUS_INSUFFICIENT_RESOURCES; - - ResourceList->Count = 0; - - Irp->IoStatus.Information = (ULONG_PTR)ResourceList; - } - else - { - /* Copy existing resource list */ - ResourceList = ExAllocatePool(PagedPool, - FIELD_OFFSET(CM_RESOURCE_LIST, List) + DeviceExtension->Device->BootResourceListSize); - if (ResourceList == NULL) - return STATUS_INSUFFICIENT_RESOURCES; - - ResourceList->Count = 1; - RtlCopyMemory( - &ResourceList->List, - DeviceExtension->Device->BootResourceList, - DeviceExtension->Device->BootResourceListSize); - Irp->IoStatus.Information = (ULONG_PTR)ResourceList; - } - - return STATUS_SUCCESS; -} - - -NTSTATUS -PdoQueryResourceRequirements( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp, - PIO_STACK_LOCATION IrpSp) -{ - PPNPROOT_PDO_DEVICE_EXTENSION DeviceExtension; - PIO_RESOURCE_REQUIREMENTS_LIST ResourceList; - ULONG ResourceListSize = FIELD_OFFSET(IO_RESOURCE_REQUIREMENTS_LIST, List); - - DPRINT("Called\n"); - - DeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - - if (!DeviceExtension->Device || DeviceExtension->Device->ResourceRequirementsList == NULL) - { - /* Create an empty resource list */ - ResourceList = ExAllocatePool(PagedPool, ResourceListSize); - if (ResourceList == NULL) - return STATUS_INSUFFICIENT_RESOURCES; - - RtlZeroMemory(ResourceList, ResourceListSize); - ResourceList->ListSize = ResourceListSize; - - Irp->IoStatus.Information = (ULONG_PTR)ResourceList; - } - else - { - /* Copy existing resource requirement list */ - ResourceList = ExAllocatePool(PagedPool, DeviceExtension->Device->ResourceRequirementsList->ListSize); - if (ResourceList == NULL) - return STATUS_INSUFFICIENT_RESOURCES; - - RtlCopyMemory( - ResourceList, - DeviceExtension->Device->ResourceRequirementsList, - DeviceExtension->Device->ResourceRequirementsList->ListSize); - Irp->IoStatus.Information = (ULONG_PTR)ResourceList; - } - - return STATUS_SUCCESS; -} - - -static NTSTATUS -PnpRootPdoQueryCapabilities( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp, - PIO_STACK_LOCATION IrpSp) +static NTSTATUS STDCALL +ForwardIrpAndWaitCompletion( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PVOID Context) +{ + if (Irp->PendingReturned) + KeSetEvent((PKEVENT)Context, IO_NO_INCREMENT, FALSE); + return STATUS_MORE_PROCESSING_REQUIRED; +} + +static NTSTATUS +ForwardIrpAndWait( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp) +{ + PDEVICE_OBJECT LowerDevice; + KEVENT Event; + NTSTATUS Status; + + ASSERT(((PPNPROOT_COMMON_DEVICE_EXTENSION)DeviceObject->DeviceExtension)->IsFDO); + LowerDevice = ((PPNPROOT_FDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension)->Ldo; + + ASSERT(LowerDevice); + + KeInitializeEvent(&Event, NotificationEvent, FALSE); + IoCopyCurrentIrpStackLocationToNext(Irp); + + DPRINT("Calling lower device %p [%wZ]\n", LowerDevice, &LowerDevice->DriverObject->DriverName); + IoSetCompletionRoutine(Irp, ForwardIrpAndWaitCompletion, &Event, TRUE, TRUE, TRUE); + + Status = IoCallDriver(LowerDevice, Irp); + if (Status == STATUS_PENDING) + { + Status = KeWaitForSingleObject(&Event, Suspended, KernelMode, FALSE, NULL); + if (NT_SUCCESS(Status)) + Status = Irp->IoStatus.Status; + } + + return Status; +} + +static NTSTATUS NTAPI +QueryStringCallback( + IN PWSTR ValueName, + IN ULONG ValueType, + IN PVOID ValueData, + IN ULONG ValueLength, + IN PVOID Context, + IN PVOID EntryContext) +{ + PUNICODE_STRING String = (PUNICODE_STRING)EntryContext; + + if (ValueType != REG_SZ || ValueLength == 0 || ValueLength % sizeof(WCHAR) != 0) + return STATUS_SUCCESS; + + String->Buffer = ExAllocatePoolWithTag(PagedPool, ValueLength, TAG_PNP_ROOT); + if (String->Buffer == NULL) + return STATUS_NO_MEMORY; + String->Length = String->MaximumLength = ValueLength; + RtlCopyMemory(String->Buffer, ValueData, ValueLength); + if (ValueLength > 0 && String->Buffer[ValueLength / sizeof(WCHAR) - 1] == L'\0') + String->Length -= sizeof(WCHAR); + return STATUS_SUCCESS; +} + +static NTSTATUS NTAPI +QueryBinaryValueCallback( + IN PWSTR ValueName, + IN ULONG ValueType, + IN PVOID ValueData, + IN ULONG ValueLength, + IN PVOID Context, + IN PVOID EntryContext) +{ + PBUFFER Buffer = (PBUFFER)EntryContext; + PVOID BinaryValue; + + if (ValueLength == 0) + { + *Buffer->Buffer = NULL; + return STATUS_SUCCESS; + } + + BinaryValue = ExAllocatePoolWithTag(PagedPool, ValueLength, TAG_PNP_ROOT); + if (BinaryValue == NULL) + return STATUS_NO_MEMORY; + RtlCopyMemory(BinaryValue, ValueData, ValueLength); + *Buffer->Buffer = BinaryValue; + if (Buffer->Length) *Buffer->Length = ValueLength; + return STATUS_SUCCESS; +} + +static NTSTATUS +EnumerateDevices( + IN PDEVICE_OBJECT DeviceObject) { PPNPROOT_FDO_DEVICE_EXTENSION DeviceExtension; - PDEVICE_CAPABILITIES DeviceCapabilities; - - DPRINT("Called\n"); + OBJECT_ATTRIBUTES ObjectAttributes; + PKEY_BASIC_INFORMATION KeyInfo = NULL, SubKeyInfo = NULL; + UNICODE_STRING KeyName = RTL_CONSTANT_STRING(L"\\Registry\\Machine\\" REGSTR_PATH_SYSTEMENUM "\\" REGSTR_KEY_ROOTENUM); + UNICODE_STRING SubKeyName; + WCHAR DevicePath[MAX_PATH + 1]; + RTL_QUERY_REGISTRY_TABLE QueryTable[4]; + PPNPROOT_DEVICE Device = NULL; + HANDLE KeyHandle = INVALID_HANDLE_VALUE; + HANDLE SubKeyHandle = INVALID_HANDLE_VALUE; + HANDLE DeviceKeyHandle = INVALID_HANDLE_VALUE; + ULONG BufferSize; + ULONG ResultSize; + ULONG Index1, Index2; + BUFFER Buffer1, Buffer2; + NTSTATUS Status = STATUS_UNSUCCESSFUL; + + DPRINT("EnumerateDevices(FDO %p)\n", DeviceObject); DeviceExtension = (PPNPROOT_FDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - DeviceCapabilities = IrpSp->Parameters.DeviceCapabilities.Capabilities; - - if (DeviceCapabilities->Version != 1) - return STATUS_UNSUCCESSFUL; - - DeviceCapabilities->UniqueID = TRUE; - /* FIXME: Fill other fields */ - - return STATUS_SUCCESS; -} - - -/* - * FUNCTION: Handle Plug and Play IRPs for the child device - * ARGUMENTS: - * DeviceObject = Pointer to physical device object of the child device - * Irp = Pointer to IRP that should be handled - * RETURNS: - * Status - */ -NTSTATUS -PnpRootPdoPnpControl( - PDEVICE_OBJECT DeviceObject, - PIRP Irp) -{ - PIO_STACK_LOCATION IrpSp; - NTSTATUS Status; - - DPRINT("Called\n"); - - Status = Irp->IoStatus.Status; - - IrpSp = IoGetCurrentIrpStackLocation(Irp); - - switch (IrpSp->MinorFunction) { -#if 0 - case IRP_MN_QUERY_BUS_INFORMATION: - break; - - case IRP_MN_QUERY_DEVICE_RELATIONS: - /* FIXME: Handle for TargetDeviceRelation */ - break; -#endif - - case IRP_MN_QUERY_ID: - Status = PdoQueryId(DeviceObject, Irp, IrpSp); - break; - - case IRP_MN_QUERY_RESOURCE_REQUIREMENTS: - Status = PdoQueryResourceRequirements(DeviceObject, Irp, IrpSp); - break; - - case IRP_MN_QUERY_RESOURCES: - Status = PdoQueryResources(DeviceObject, Irp, IrpSp); - break; - - case IRP_MN_QUERY_CAPABILITIES: - Status = PnpRootPdoQueryCapabilities(DeviceObject, Irp, IrpSp); - break; - - case IRP_MN_START_DEVICE: - case IRP_MN_QUERY_STOP_DEVICE: - case IRP_MN_CANCEL_STOP_DEVICE: - case IRP_MN_STOP_DEVICE: - case IRP_MN_QUERY_REMOVE_DEVICE: - case IRP_MN_CANCEL_REMOVE_DEVICE: - case IRP_MN_REMOVE_DEVICE: - case IRP_MN_SURPRISE_REMOVAL: - Status = STATUS_SUCCESS; - break; - - default: - DPRINT("Unknown IOCTL 0x%X\n", IrpSp->MinorFunction); - break; - } - - Irp->IoStatus.Status = Status; - IoCompleteRequest(Irp, IO_NO_INCREMENT); - - DPRINT("Leaving. Status 0x%X\n", Status); - - return Status; -} - - -/* - * FUNCTION: Handle power management IRPs for the child device - * ARGUMENTS: - * DeviceObject = Pointer to physical device object of the child device - * Irp = Pointer to IRP that should be handled - * RETURNS: - * Status - */ -NTSTATUS -PnpRootPdoPowerControl( - PDEVICE_OBJECT DeviceObject, - PIRP Irp) -{ - PIO_STACK_LOCATION IrpSp; - NTSTATUS Status; - - DPRINT("Called\n"); - - Status = Irp->IoStatus.Status; - - IrpSp = IoGetCurrentIrpStackLocation(Irp); - - switch (IrpSp->MinorFunction) { - default: - DPRINT("Unknown IOCTL 0x%X\n", IrpSp->MinorFunction); - Status = STATUS_NOT_IMPLEMENTED; - break; - } - - Irp->IoStatus.Status = Status; - IoCompleteRequest(Irp, IO_NO_INCREMENT); - - DPRINT("Leaving. Status 0x%X\n", Status); - - return Status; -} - - -/* Functional Device Object routines */ - -static NTSTATUS -PnpRootReadRegistryBinary( - IN PWSTR KeyName, - IN PWSTR ValueKeyName, - OUT PVOID* Buffer, - OUT SIZE_T* BufferSize OPTIONAL) -{ - OBJECT_ATTRIBUTES ObjectAttributes; - UNICODE_STRING KeyNameU; - UNICODE_STRING ValueKeyNameU; - KEY_VALUE_PARTIAL_INFORMATION Size; - PKEY_VALUE_PARTIAL_INFORMATION Data = NULL; - ULONG DataSize = sizeof(KEY_VALUE_PARTIAL_INFORMATION); - HANDLE KeyHandle; - NTSTATUS Status; - - DPRINT("Called\n"); - - RtlInitUnicodeString(&KeyNameU, KeyName); - RtlInitUnicodeString(&ValueKeyNameU, ValueKeyName); - + KeAcquireGuardedMutex(&DeviceExtension->DeviceListLock); + + BufferSize = sizeof(KEY_BASIC_INFORMATION) + (MAX_PATH + 1) * sizeof(WCHAR); + KeyInfo = ExAllocatePoolWithTag(PagedPool, BufferSize, TAG_PNP_ROOT); + if (!KeyInfo) + { + DPRINT("ExAllocatePoolWithTag() failed\n"); + Status = STATUS_NO_MEMORY; + goto cleanup; + } + SubKeyInfo = ExAllocatePoolWithTag(PagedPool, BufferSize, TAG_PNP_ROOT); + if (!SubKeyInfo) + { + DPRINT("ExAllocatePoolWithTag() failed\n"); + Status = STATUS_NO_MEMORY; + goto cleanup; + } + InitializeObjectAttributes( &ObjectAttributes, - &KeyNameU, + &KeyName, OBJ_CASE_INSENSITIVE, - NULL, /* Root dir */ - NULL); /* Security descriptor */ - Status = ZwOpenKey(&KeyHandle, KEY_READ, &ObjectAttributes); - if (!NT_SUCCESS(Status)) - { - DPRINT("ZwOpenKey() failed (Status 0x%08lx)\n", Status); - return Status; - } - - Status = ZwQueryValueKey( - KeyHandle, - &ValueKeyNameU, - KeyValuePartialInformation, - &Size, DataSize, - &DataSize); - if (Status != STATUS_BUFFER_OVERFLOW) - { - DPRINT("ZwQueryValueKey() failed (Status 0x%08lx)\n", Status); - ZwClose(KeyHandle); - return Status; - } - - while (Status == STATUS_BUFFER_OVERFLOW) - { - if (Data) - ExFreePoolWithTag(Data, TAG_PNP_ROOT); - Data = (PKEY_VALUE_PARTIAL_INFORMATION)ExAllocatePoolWithTag(PagedPool, DataSize, TAG_PNP_ROOT); - if (!Data) - { - DPRINT("ExAllocatePoolWithTag() failed\n", Status); - ZwClose(KeyHandle); - return STATUS_NO_MEMORY; - } - - Status = ZwQueryValueKey( - KeyHandle, - &ValueKeyNameU, - KeyValuePartialInformation, - Data, DataSize, - &DataSize); - if (NT_SUCCESS(Status)) - { - *Buffer = ExAllocatePoolWithTag(PagedPool, Data->DataLength, TAG_PNP_ROOT); - if (BufferSize) *BufferSize = Data->DataLength; - if (!*Buffer) - { - DPRINT("ExAllocatePoolWithTag() failed\n", Status); - ExFreePoolWithTag(Data, TAG_PNP_ROOT); - ZwClose(KeyHandle); - return STATUS_NO_MEMORY; - } - - RtlCopyMemory( - *Buffer, - Data->Data, - Data->DataLength); - break; - } - } - - if (Data) - ExFreePoolWithTag(Data, TAG_PNP_ROOT); - ZwClose(KeyHandle); - - return Status; -} - -NTSTATUS -PnpRootFdoReadDeviceInfo( - PPNPROOT_DEVICE Device) -{ - RTL_QUERY_REGISTRY_TABLE QueryTable[2]; - PUNICODE_STRING DeviceDesc; - WCHAR KeyName[MAX_PATH]; - NTSTATUS Status; - - DPRINT("Called\n"); - - /* Retrieve configuration from Enum key */ - - DeviceDesc = &Device->DeviceDescription; - - wcscpy(KeyName, L"\\Registry\\Machine\\System\\CurrentControlSet\\Enum\\"); - wcscat(KeyName, ENUM_NAME_ROOT); - wcscat(KeyName, L"\\"); - wcscat(KeyName, Device->ServiceName.Buffer); - wcscat(KeyName, L"\\"); - wcscat(KeyName, Device->InstanceID.Buffer); - - DPRINT("KeyName %S\n", KeyName); - - /* 1. Read informations in instance key */ - RtlZeroMemory(QueryTable, sizeof(QueryTable)); - - RtlInitUnicodeString(DeviceDesc, NULL); - - QueryTable[0].Name = L"DeviceDesc"; - QueryTable[0].Flags = RTL_QUERY_REGISTRY_DIRECT; - QueryTable[0].EntryContext = DeviceDesc; - - Status = RtlQueryRegistryValues( - RTL_REGISTRY_ABSOLUTE | RTL_REGISTRY_OPTIONAL, - KeyName, - QueryTable, NULL, NULL); - DPRINT("RtlQueryRegistryValues() returned status 0x%08lx\n", Status); - - if (!NT_SUCCESS(Status)) - { - Device->DeviceDescription.Buffer = NULL; - } - - DPRINT("Got device description: %S\n", DeviceDesc->Buffer); - - /* 2. Read informations in instance key, LogConf subkey */ - RtlZeroMemory(QueryTable, sizeof(QueryTable)); - wcscat(KeyName, L"\\LogConf"); - - Status = PnpRootReadRegistryBinary( - KeyName, - L"BasicConfigVector", - (PVOID*)&Device->ResourceRequirementsList, - NULL); - - DPRINT("PnpRootReadRegistryBinary() returned status 0x%08lx\n", Status); - - if (!NT_SUCCESS(Status)) - { - /* FIXME: */ - } - - Status = PnpRootReadRegistryBinary( - KeyName, - L"BootConfig", - (PVOID*)&Device->BootResourceList, - &Device->BootResourceListSize); - - DPRINT("PnpRootReadRegistryBinary() returned status 0x%08lx\n", Status); - - if (!NT_SUCCESS(Status)) - { - /* FIXME: */ - } - - return STATUS_SUCCESS; -} - - -NTSTATUS -PnpRootFdoEnumerateDevices( - PDEVICE_OBJECT DeviceObject) -{ - PPNPROOT_FDO_DEVICE_EXTENSION DeviceExtension; - OBJECT_ATTRIBUTES ObjectAttributes, SubKeyAttributes; - PKEY_BASIC_INFORMATION KeyInfo, SubKeyInfo; - UNICODE_STRING KeyName = RTL_CONSTANT_STRING(L"\\Registry\\Machine\\System\\CurrentControlSet\\Enum\\" ENUM_NAME_ROOT); - UNICODE_STRING SubKeyName; - PPNPROOT_DEVICE Device; - WCHAR Buffer[MAX_PATH]; - HANDLE KeyHandle, SubKeyHandle; - ULONG BufferSize; - ULONG ResultSize; - NTSTATUS Status; - ULONG Index1, Index2; - - DPRINT("Called\n"); - - DeviceExtension = (PPNPROOT_FDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - - BufferSize = sizeof(KEY_BASIC_INFORMATION) + (MAX_PATH+1) * sizeof(WCHAR); - KeyInfo = ExAllocatePoolWithTag(PagedPool, BufferSize, TAG_PNP_ROOT); - if (!KeyInfo) - { - return STATUS_INSUFFICIENT_RESOURCES; - } - SubKeyInfo = ExAllocatePoolWithTag(PagedPool, BufferSize, TAG_PNP_ROOT); - if (!SubKeyInfo) - { - ExFreePoolWithTag(KeyInfo, TAG_PNP_ROOT); - return STATUS_INSUFFICIENT_RESOURCES; - } - - InitializeObjectAttributes( - &ObjectAttributes, - &KeyName, - OBJ_CASE_INSENSITIVE, - NULL, - NULL); - Status = ZwOpenKey(&KeyHandle, KEY_ENUMERATE_SUB_KEYS, &ObjectAttributes); if (!NT_SUCCESS(Status)) { - DPRINT("ZwOpenKey() failed (Status 0x%08lx)\n", Status); - ExFreePoolWithTag(KeyInfo, TAG_PNP_ROOT); - ExFreePoolWithTag(SubKeyInfo, TAG_PNP_ROOT); - return Status; - } - - /* FIXME: Disabled due to still using the old method of auto loading drivers e.g. - there are more entries in the list than found in the registry as some - drivers are passed on the command line */ -// DeviceExtension->DeviceListCount = 0; + DPRINT("ZwOpenKey(%wZ) failed with status 0x%08lx\n", &KeyName, Status); + goto cleanup; + } /* Devices are sub-sub-keys of 'KeyName'. KeyName is already opened as * KeyHandle. We'll first do a first enumeration to have first level keys, * and an inner one to have the real devices list. */ Index1 = 0; - while (TRUE) { Status = ZwEnumerateKey( @@ -703,30 +391,35 @@ KeyInfo, BufferSize, &ResultSize); - if (!NT_SUCCESS(Status)) + if (Status == STATUS_NO_MORE_ENTRIES) { - DPRINT("ZwEnumerateKey() (Status 0x%08lx)\n", Status); - break; + Status = STATUS_SUCCESS; + break; + } + else if (!NT_SUCCESS(Status)) + { + DPRINT("ZwEnumerateKey() failed with status 0x%08lx\n", Status); + goto cleanup; } /* Terminate the string */ KeyInfo->Name[KeyInfo->NameLength / sizeof(WCHAR)] = 0; - + /* Open the key */ RtlInitUnicodeString(&SubKeyName, KeyInfo->Name); InitializeObjectAttributes( - &SubKeyAttributes, + &ObjectAttributes, &SubKeyName, 0, /* Attributes */ KeyHandle, NULL); /* Security descriptor */ - Status = ZwOpenKey(&SubKeyHandle, KEY_ENUMERATE_SUB_KEYS, &SubKeyAttributes); + Status = ZwOpenKey(&SubKeyHandle, KEY_ENUMERATE_SUB_KEYS, &ObjectAttributes); if (!NT_SUCCESS(Status)) { - DPRINT("ZwOpenKey() failed (Status 0x%08lx)\n", Status); + DPRINT("ZwOpenKey() failed with status 0x%08lx\n", Status); break; } - + /* Enumerate the sub-keys */ Index2 = 0; while (TRUE) @@ -738,125 +431,190 @@ SubKeyInfo, BufferSize, &ResultSize); - if (!NT_SUCCESS(Status)) + if (Status == STATUS_NO_MORE_ENTRIES) + break; + else if (!NT_SUCCESS(Status)) { - DPRINT("ZwEnumerateKey() (Status 0x%08lx)\n", Status); + DPRINT("ZwEnumerateKey() failed with status 0x%08lx\n", Status); break; } - + /* Terminate the string */ SubKeyInfo->Name[SubKeyInfo->NameLength / sizeof(WCHAR)] = 0; - - Device = (PPNPROOT_DEVICE)ExAllocatePoolWithTag(PagedPool, sizeof(PNPROOT_DEVICE), TAG_PNP_ROOT); - if (!Device) + + _snwprintf( + DevicePath, + sizeof(DevicePath) / sizeof(WCHAR), + L"%s\\%s", REGSTR_KEY_ROOTENUM, KeyInfo->Name); + DPRINT("Found device %S\\%s!\n", DevicePath, SubKeyInfo->Name); + if (LocateChildDevice(DeviceExtension, DevicePath, SubKeyInfo->Name, &Device) == STATUS_NO_SUCH_DEVICE) { - /* FIXME: */ - DPRINT("ExAllocatePoolWithTag() failed\n"); - break; + /* Create a PPNPROOT_DEVICE object, and add if in the list of known devices */ + Device = (PPNPROOT_DEVICE)ExAllocatePoolWithTag(PagedPool, sizeof(PNPROOT_DEVICE), TAG_PNP_ROOT); + if (!Device) + { + DPRINT("ExAllocatePoolWithTag() failed\n"); + Status = STATUS_NO_MEMORY; + goto cleanup; + } + RtlZeroMemory(Device, sizeof(PNPROOT_DEVICE)); + + /* Fill device ID and instance ID */ + if (!RtlCreateUnicodeString(&Device->DeviceID, DevicePath)) + { + DPRINT("RtlCreateUnicodeString() failed\n"); + Status = STATUS_NO_MEMORY; + goto cleanup; + } + + if (!RtlCreateUnicodeString(&Device->InstanceID, SubKeyInfo->Name)) + { + DPRINT("RtlCreateUnicodeString() failed\n"); + Status = STATUS_NO_MEMORY; + goto cleanup; + } + + /* Open registry key to fill other informations */ + InitializeObjectAttributes( + &ObjectAttributes, + &Device->InstanceID, + 0, /* Attributes */ + SubKeyHandle, + NULL); /* Security descriptor */ + Status = ZwOpenKey(&DeviceKeyHandle, KEY_READ, &ObjectAttributes); + if (!NT_SUCCESS(Status)) + { + DPRINT("ZwOpenKey() failed with status 0x%08lx\n", Status); + break; + } + + /* Fill other informations */ + Buffer1.Buffer = (PVOID *)&Device->ResourceRequirementsList; + Buffer1.Length = NULL; + Buffer2.Buffer = (PVOID *)&Device->ResourceList; + Buffer2.Length = &Device->ResourceListSize; + RtlZeroMemory(QueryTable, sizeof(QueryTable)); + QueryTable[0].QueryRoutine = QueryStringCallback; + QueryTable[0].Name = L"DeviceDesc"; + QueryTable[0].EntryContext = &Device->DeviceDescription; + QueryTable[1].Flags = RTL_QUERY_REGISTRY_SUBKEY; + QueryTable[1].Name = L"LogConf"; + QueryTable[2].QueryRoutine = QueryBinaryValueCallback; + QueryTable[2].Name = L"BasicConfigVector"; + QueryTable[2].EntryContext = &Buffer1; + QueryTable[2].QueryRoutine = QueryBinaryValueCallback; + QueryTable[2].Name = L"BootConfig"; + QueryTable[2].EntryContext = &Buffer2; + + Status = RtlQueryRegistryValues( + RTL_REGISTRY_HANDLE, + (PCWSTR)DeviceKeyHandle, + QueryTable, + NULL, + NULL); + if (!NT_SUCCESS(Status)) + { + DPRINT("RtlQueryRegistryValues() failed with status 0x%08lx\n", Status); + break; + } + + ZwClose(DeviceKeyHandle); + DeviceKeyHandle = INVALID_HANDLE_VALUE; + + /* Insert the newly created device into the list */ + InsertTailList( + &DeviceExtension->DeviceListHead, + &Device->ListEntry); + DeviceExtension->DeviceListCount++; + Device = NULL; } - RtlZeroMemory(Device, sizeof(PNPROOT_DEVICE)); - - if (!RtlCreateUnicodeString(&Device->ServiceName, KeyInfo->Name)) - { - /* FIXME: */ - DPRINT("RtlCreateUnicodeString() failed\n"); - } - - wcscpy(Buffer, ENUM_NAME_ROOT); - wcscat(Buffer, L"\\"); - wcscat(Buffer, KeyInfo->Name); - - if (!RtlCreateUnicodeString(&Device->DeviceID, Buffer)) - { - /* FIXME: */ - DPRINT("RtlCreateUnicodeString() failed\n"); - } - - DPRINT("Got entry: %S\n", Device->DeviceID.Buffer); - - if (!RtlCreateUnicodeString( - &Device->InstanceID, - SubKeyInfo->Name)) - { - /* FIXME: */ - DPRINT("RtlCreateUnicodeString() failed\n"); - } - - Status = PnpRootFdoReadDeviceInfo(Device); - if (!NT_SUCCESS(Status)) - { - /* FIXME */ - DPRINT("PnpRootFdoReadDeviceInfo() failed with status 0x%08lx\n", Status); - } - - ExInterlockedInsertTailList( - &DeviceExtension->DeviceListHead, - &Device->ListEntry, - &DeviceExtension->DeviceListLock); - - DeviceExtension->DeviceListCount++; - Index2++; } - + ZwClose(SubKeyHandle); + SubKeyHandle = INVALID_HANDLE_VALUE; Index1++; } - ZwClose(KeyHandle); - +cleanup: + if (Device) + { + /* We have a device that has not been added to device list. We need to clean it up */ + /* FIXME */ + ExFreePoolWithTag(Device, TAG_PNP_ROOT); + } + if (DeviceKeyHandle != INVALID_HANDLE_VALUE) + ZwClose(DeviceKeyHandle); + if (SubKeyHandle != INVALID_HANDLE_VALUE) + ZwClose(SubKeyHandle); + if (KeyHandle != INVALID_HANDLE_VALUE) + ZwClose(KeyHandle); ExFreePoolWithTag(KeyInfo, TAG_PNP_ROOT); ExFreePoolWithTag(SubKeyInfo, TAG_PNP_ROOT); - - return STATUS_SUCCESS; -} - - -NTSTATUS -PnpRootQueryBusRelations( + KeReleaseGuardedMutex(&DeviceExtension->DeviceListLock); + return Status; +} + +/* FUNCTION: Handle IRP_MN_QUERY_DEVICE_RELATIONS IRPs for the root bus device object + * ARGUMENTS: + * DeviceObject = Pointer to functional device object of the root bus driver + * Irp = Pointer to IRP that should be handled + * RETURNS: + * Status + */ +static NTSTATUS +PnpRootQueryDeviceRelations( IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp, - IN PIO_STACK_LOCATION IrpSp) + IN PIRP Irp) { PPNPROOT_PDO_DEVICE_EXTENSION PdoDeviceExtension; PPNPROOT_FDO_DEVICE_EXTENSION DeviceExtension; - PDEVICE_RELATIONS Relations; - PPNPROOT_DEVICE Device; + PDEVICE_RELATIONS Relations = NULL, OtherRelations = (PDEVICE_RELATIONS)Irp->IoStatus.Information; + PPNPROOT_DEVICE Device = NULL; + ULONG Size; NTSTATUS Status; - ULONG Size; - ULONG i; - - DPRINT("Called\n"); - - Status = PnpRootFdoEnumerateDevices(DeviceObject); + + DPRINT("PnpRootQueryDeviceRelations(FDO %p, Irp %p)\n", DeviceObject, Irp); + + Status = EnumerateDevices(DeviceObject); if (!NT_SUCCESS(Status)) + { + DPRINT("EnumerateDevices() failed with status 0x%08lx\n", Status); return Status; + } DeviceExtension = (PPNPROOT_FDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - if (Irp->IoStatus.Information) - { - /* FIXME: Another bus driver has already created a DEVICE_RELATIONS - structure so we must merge this structure with our own */ - } - - Size = sizeof(DEVICE_RELATIONS) + sizeof(Relations->Objects) * - (DeviceExtension->DeviceListCount - 1); - + Size = FIELD_OFFSET(DEVICE_RELATIONS, Objects) + sizeof(PDEVICE_OBJECT) * DeviceExtension->DeviceListCount; + if (OtherRelations) + { + /* Another bus driver has already created a DEVICE_RELATIONS + * structure so we must merge this structure with our own */ + + Size += sizeof(PDEVICE_OBJECT) * OtherRelations->Count; + } Relations = (PDEVICE_RELATIONS)ExAllocatePool(PagedPool, Size); if (!Relations) - return STATUS_INSUFFICIENT_RESOURCES; - - Relations->Count = DeviceExtension->DeviceListCount; - - i = 0; - LIST_FOR_EACH(Device,&DeviceExtension->DeviceListHead,PNPROOT_DEVICE, ListEntry) + { + DPRINT("ExAllocatePoolWithTag() failed\n"); + Status = STATUS_NO_MEMORY; + goto cleanup; + } + RtlZeroMemory(Relations, Size); + if (OtherRelations) + { + Relations->Count = OtherRelations->Count; + RtlCopyMemory(Relations->Objects, OtherRelations->Objects, sizeof(PDEVICE_OBJECT) * OtherRelations->Count); + } + + KeAcquireGuardedMutex(&DeviceExtension->DeviceListLock); + LIST_FOR_EACH(Device, &DeviceExtension->DeviceListHead, PNPROOT_DEVICE, ListEntry) { if (!Device->Pdo) { /* Create a physical device object for the - device as it does not already have one */ + * device as it does not already have one */ Status = IoCreateDevice( DeviceObject->DriverObject, sizeof(PNPROOT_PDO_DEVICE_EXTENSION), @@ -867,88 +625,45 @@ &Device->Pdo); if (!NT_SUCCESS(Status)) { - DPRINT("IoCreateDevice() failed with status 0x%X\n", Status); - ExFreePool(Relations); - return Status; + DPRINT("IoCreateDevice() failed with status 0x%08lx\n", Status); + break; } - DPRINT("Created PDO 0x%p\n", Device->Pdo); + PdoDeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)Device->Pdo->DeviceExtension; + RtlZeroMemory(PdoDeviceExtension, sizeof(PNPROOT_PDO_DEVICE_EXTENSION)); + PdoDeviceExtension->Common.IsFDO = FALSE; + PdoDeviceExtension->DeviceInfo = Device; Device->Pdo->Flags |= DO_BUS_ENUMERATED_DEVICE; - Device->Pdo->Flags &= ~DO_DEVICE_INITIALIZING; - - //Device->Pdo->Flags |= DO_POWER_PAGABLE; - - PdoDeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)Device->Pdo->DeviceExtension; - - RtlZeroMemory(PdoDeviceExtension, sizeof(PNPROOT_PDO_DEVICE_EXTENSION)); - - PdoDeviceExtension->Common.IsFDO = FALSE; - - PdoDeviceExtension->Common.DeviceObject = Device->Pdo; - - PdoDeviceExtension->Common.DevicePowerState = PowerDeviceD0; - - PdoDeviceExtension->Device = Device; - - DPRINT("DeviceID: %wZ PDO %p\n", - &PdoDeviceExtension->Device->DeviceID, - Device->Pdo); - - DPRINT("InstanceID: %wZ PDO %p\n", - &PdoDeviceExtension->Device->InstanceID, - Device->Pdo); - - DPRINT("ResourceRequirementsList: %p PDO %p\n", - PdoDeviceExtension->Device->ResourceRequirementsList, - Device->Pdo); } /* Reference the physical device object. The PnP manager will dereference it again when it is no longer needed */ ObReferenceObject(Device->Pdo); - Relations->Objects[i] = Device->Pdo; - - i++; - } - - if (NT_SUCCESS(Status)) - { - Irp->IoStatus.Information = (ULONG_PTR)Relations; - } - else - { - Irp->IoStatus.Information = 0; + Relations->Objects[Relations->Count++] = Device->Pdo; + } + KeReleaseGuardedMutex(&DeviceExtension->DeviceListLock); + + Irp->IoStatus.Information = (ULONG_PTR)Relations; + +cleanup: + if (!NT_SUCCESS(Status)) + { + if (OtherRelations) + ExFreePool(OtherRelations); + if (Relations) + ExFreePool(Relations); + if (Device && Device->Pdo) + { + IoDeleteDevice(Device->Pdo); + Device->Pdo = NULL; + } } return Status; } - - -NTSTATUS -PnpRootQueryDeviceRelations( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp, - IN PIO_STACK_LOCATION IrpSp) -{ - NTSTATUS Status; - - DPRINT("Called\n"); - - switch (IrpSp->Parameters.QueryDeviceRelations.Type) { - case BusRelations: - Status = PnpRootQueryBusRelations(DeviceObject, Irp, IrpSp); - break; - - default: - Status = STATUS_NOT_IMPLEMENTED; - } - - return Status; -} - /* * FUNCTION: Handle Plug and Play IRPs for the root bus device object @@ -958,8 +673,7 @@ * RETURNS: * Status */ -NTSTATUS -STDCALL +static NTSTATUS PnpRootFdoPnpControl( IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp) @@ -968,84 +682,417 @@ PIO_STACK_LOCATION IrpSp; NTSTATUS Status; - DPRINT("Called\n"); - DeviceExtension = (PPNPROOT_FDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - Status = Irp->IoStatus.Status; - IrpSp = IoGetCurrentIrpStackLocation(Irp); - switch (IrpSp->MinorFunction) { - case IRP_MN_QUERY_DEVICE_RELATIONS: - Status = PnpRootQueryDeviceRelations(DeviceObject, Irp, IrpSp); - break; - - case IRP_MN_START_DEVICE: - DeviceExtension->State = dsStarted; - Status = STATUS_SUCCESS; - break; - - case IRP_MN_STOP_DEVICE: - /* Root device cannot be stopped */ - Status = STATUS_UNSUCCESSFUL; - break; - - default: - DPRINT("Unknown IOCTL 0x%X\n", IrpSp->MinorFunction); - Status = STATUS_NOT_IMPLEMENTED; - break; - } - - if (Status != STATUS_PENDING) { + switch (IrpSp->MinorFunction) + { + case IRP_MN_QUERY_DEVICE_RELATIONS: + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_DEVICE_RELATIONS\n"); + Status = PnpRootQueryDeviceRelations(DeviceObject, Irp); + break; + + case IRP_MN_START_DEVICE: + DPRINT("IRP_MJ_PNP / IRP_MN_START_DEVICE\n"); + Status = ForwardIrpAndWait(DeviceObject, Irp); + if (NT_SUCCESS(Status)) + DeviceExtension->State = dsStarted; + Status = STATUS_SUCCESS; + break; + + case IRP_MN_STOP_DEVICE: + DPRINT("IRP_MJ_PNP / IRP_MN_STOP_DEVICE\n"); + /* Root device cannot be stopped */ + Status = STATUS_NOT_SUPPORTED; + break; + + default: + DPRINT("IRP_MJ_PNP / Unknown minor function 0x%lx\n", IrpSp->MinorFunction); + Status = STATUS_NOT_IMPLEMENTED; + break; + } + + if (Status != STATUS_PENDING) + { Irp->IoStatus.Status = Status; IoCompleteRequest(Irp, IO_NO_INCREMENT); } - DPRINT("Leaving. Status 0x%X\n", Status); - return Status; } +static NTSTATUS +PdoQueryDeviceRelations( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PIO_STACK_LOCATION IrpSp) +{ + //PPNPROOT_PDO_DEVICE_EXTENSION DeviceExtension; + PDEVICE_RELATIONS Relations; + DEVICE_RELATION_TYPE RelationType; + NTSTATUS Status = Irp->IoStatus.Status; + + //DeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + RelationType = IrpSp->Parameters.QueryDeviceRelations.Type; + + switch (RelationType) + { + /* FIXME: remove */ + case BusRelations: + { + if (IoGetAttachedDevice(DeviceObject) != DeviceObject) + { + /* We're not alone in the stack */ + PDEVICE_NODE DeviceNode; + DeviceNode = IopGetDeviceNode(IopGetDeviceNode(DeviceObject)->PhysicalDeviceObject); + DPRINT1("Device stack for '%wZ' (%wZ) is misbehaving ; shouldn't receive IRP_MN_QUERY_DEVICE_RELATIONS / BusRelations\n", + &DeviceNode->InstancePath, &DeviceNode->ServiceName); + } + break; + } + + case TargetDeviceRelation: + { + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_DEVICE_RELATIONS / TargetDeviceRelation\n"); + Relations = (PDEVICE_RELATIONS)ExAllocatePool(PagedPool, sizeof(DEVICE_RELATIONS)); + if (!Relations) + { + DPRINT("ExAllocatePoolWithTag() failed\n"); + Status = STATUS_NO_MEMORY; + } + else + { + ObReferenceObject(DeviceObject); + Relations->Count = 1; + Relations->Objects[0] = DeviceObject; + Status = STATUS_SUCCESS; + Irp->IoStatus.Information = (ULONG_PTR)Relations; + } + break; + } + + default: + { + DPRINT1("IRP_MJ_PNP / IRP_MN_QUERY_DEVICE_RELATIONS / unknown relation type 0x%lx\n", RelationType); + } + } + + return Status; +} + +static NTSTATUS +PdoQueryCapabilities( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PIO_STACK_LOCATION IrpSp) +{ + PPNPROOT_FDO_DEVICE_EXTENSION DeviceExtension; + PDEVICE_CAPABILITIES DeviceCapabilities; + + DeviceExtension = (PPNPROOT_FDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + DeviceCapabilities = IrpSp->Parameters.DeviceCapabilities.Capabilities; + + if (DeviceCapabilities->Version != 1) + return STATUS_REVISION_MISMATCH; + + DeviceCapabilities->UniqueID = TRUE; + /* FIXME: Fill other fields */ + + return STATUS_SUCCESS; +} + +static NTSTATUS +PdoQueryResources( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PIO_STACK_LOCATION IrpSp) +{ + PPNPROOT_PDO_DEVICE_EXTENSION DeviceExtension; + PCM_RESOURCE_LIST ResourceList; + + DeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + + if (DeviceExtension->DeviceInfo->ResourceList == NULL) + { + /* Create an empty resource list */ + ResourceList = ExAllocatePool(PagedPool, sizeof(CM_RESOURCE_LIST)); + if (!ResourceList) + return STATUS_NO_MEMORY; + + ResourceList->Count = 0; + + Irp->IoStatus.Information = (ULONG_PTR)ResourceList; + } + else + { + /* Copy existing resource requirement list */ + ResourceList = ExAllocatePool( + PagedPool, + FIELD_OFFSET(CM_RESOURCE_LIST, List) + DeviceExtension->DeviceInfo->ResourceListSize); + if (!ResourceList) + return STATUS_NO_MEMORY; + + ResourceList->Count = 1; + RtlCopyMemory( + &ResourceList->List, + DeviceExtension->DeviceInfo->ResourceList, + DeviceExtension->DeviceInfo->ResourceListSize); + Irp->IoStatus.Information = (ULONG_PTR)ResourceList; + } + + return STATUS_SUCCESS; +} + +static NTSTATUS +PdoQueryResourceRequirements( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PIO_STACK_LOCATION IrpSp) +{ + PPNPROOT_PDO_DEVICE_EXTENSION DeviceExtension; + PIO_RESOURCE_REQUIREMENTS_LIST ResourceList; + ULONG ResourceListSize = FIELD_OFFSET(IO_RESOURCE_REQUIREMENTS_LIST, List); + + DeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + + if (DeviceExtension->DeviceInfo->ResourceRequirementsList == NULL) + { + /* Create an empty resource list */ + ResourceList = ExAllocatePool(PagedPool, ResourceListSize); + if (!ResourceList) + return STATUS_NO_MEMORY; + + RtlZeroMemory(ResourceList, ResourceListSize); + ResourceList->ListSize = ResourceListSize; + + Irp->IoStatus.Information = (ULONG_PTR)ResourceList; + } + else + { + /* Copy existing resource requirement list */ + ResourceList = ExAllocatePool(PagedPool, DeviceExtension->DeviceInfo->ResourceRequirementsList->ListSize); + if (!ResourceList) + return STATUS_NO_MEMORY; + + RtlCopyMemory( + ResourceList, + DeviceExtension->DeviceInfo->ResourceRequirementsList, + DeviceExtension->DeviceInfo->ResourceRequirementsList->ListSize); + Irp->IoStatus.Information = (ULONG_PTR)ResourceList; + } + + return STATUS_SUCCESS; +} + +static NTSTATUS +PdoQueryDeviceText( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PIO_STACK_LOCATION IrpSp) +{ + PPNPROOT_PDO_DEVICE_EXTENSION DeviceExtension; + DEVICE_TEXT_TYPE DeviceTextType; + NTSTATUS Status = Irp->IoStatus.Status; + + DeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + DeviceTextType = IrpSp->Parameters.QueryDeviceText.DeviceTextType; + + switch (DeviceTextType) + { + case DeviceTextDescription: + { + UNICODE_STRING String; + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_DEVICE_TEXT / DeviceTextDescription\n"); + + Status = RtlDuplicateUnicodeString( + RTL_DUPLICATE_UNICODE_STRING_NULL_TERMINATE, + &DeviceExtension->DeviceInfo->DeviceDescription, + &String); + Irp->IoStatus.Information = (ULONG_PTR)String.Buffer; + break; + } + + case DeviceTextLocationInformation: + { + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_DEVICE_TEXT / DeviceTextLocationInformation\n"); + Status = STATUS_NOT_SUPPORTED; + break; + } + + default: + { + DPRINT1("IRP_MJ_PNP / IRP_MN_QUERY_DEVICE_TEXT / unknown query id type 0x%lx\n", DeviceTextType); + } + } + + return Status; +} + +static NTSTATUS +PdoQueryId( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PIO_STACK_LOCATION IrpSp) +{ + PPNPROOT_PDO_DEVICE_EXTENSION DeviceExtension; + BUS_QUERY_ID_TYPE IdType; + NTSTATUS Status = Irp->IoStatus.Status; + + DeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + IdType = IrpSp->Parameters.QueryId.IdType; + + switch (IdType) + { + case BusQueryDeviceID: + { + UNICODE_STRING String; + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_ID / BusQueryDeviceID\n"); + + Status = RtlDuplicateUnicodeString( + RTL_DUPLICATE_UNICODE_STRING_NULL_TERMINATE, + &DeviceExtension->DeviceInfo->DeviceID, + &String); + Irp->IoStatus.Information = (ULONG_PTR)String.Buffer; + break; + } + + case BusQueryHardwareIDs: + case BusQueryCompatibleIDs: + { + /* Optional, do nothing */ + break; + } + + case BusQueryInstanceID: + { + UNICODE_STRING String; + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_ID / BusQueryInstanceID\n"); + + Status = RtlDuplicateUnicodeString( + RTL_DUPLICATE_UNICODE_STRING_NULL_TERMINATE, + &DeviceExtension->DeviceInfo->InstanceID, + &String); + Irp->IoStatus.Information = (ULONG_PTR)String.Buffer; + break; + } + + default: + { + DPRINT1("IRP_MJ_PNP / IRP_MN_QUERY_ID / unknown query id type 0x%lx\n", IdType); + } + } + + return Status; +} + +static NTSTATUS +PdoQueryBusInformation( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PIO_STACK_LOCATION IrpSp) +{ + PPNP_BUS_INFORMATION BusInfo; + NTSTATUS Status; + + BusInfo = (PPNP_BUS_INFORMATION)ExAllocatePoolWithTag(PagedPool, sizeof(PNP_BUS_INFORMATION), TAG_PNP_ROOT); + if (!BusInfo) + Status = STATUS_NO_MEMORY; + else + { + RtlCopyMemory( + &BusInfo->BusTypeGuid, + &GUID_BUS_TYPE_INTERNAL, + sizeof(BusInfo->BusTypeGuid)); + BusInfo->LegacyBusType = PNPBus; + /* We're the only root bus enumerator on the computer */ + BusInfo->BusNumber = 0; + Irp->IoStatus.Information = (ULONG_PTR)BusInfo; + Status = STATUS_SUCCESS; + } + + return Status; +} /* - * FUNCTION: Handle power management IRPs for the root bus device object + * FUNCTION: Handle Plug and Play IRPs for the child device * ARGUMENTS: - * DeviceObject = Pointer to functional device object of the root bus driver + * DeviceObject = Pointer to physical device object of the child device * Irp = Pointer to IRP that should be handled * RETURNS: * Status */ -NTSTATUS -STDCALL -PnpRootFdoPowerControl( +static NTSTATUS +PnpRootPdoPnpControl( IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp) { + PPNPROOT_PDO_DEVICE_EXTENSION DeviceExtension; PIO_STACK_LOCATION IrpSp; NTSTATUS Status; - DPRINT("Called\n"); - + DeviceExtension = (PPNPROOT_PDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + Status = Irp->IoStatus.Status; IrpSp = IoGetCurrentIrpStackLocation(Irp); - switch (IrpSp->MinorFunction) { - default: - DPRINT("Unknown IOCTL 0x%X\n", IrpSp->MinorFunction); - Status = STATUS_NOT_IMPLEMENTED; - break; - } - - if (Status != STATUS_PENDING) { + switch (IrpSp->MinorFunction) + { + case IRP_MN_START_DEVICE: /* 0x00 */ + DPRINT("IRP_MJ_PNP / IRP_MN_START_DEVICE\n"); + Status = STATUS_SUCCESS; + break; + + case IRP_MN_QUERY_DEVICE_RELATIONS: /* 0x07 */ + Status = PdoQueryDeviceRelations(DeviceObject, Irp, IrpSp); + break; + + case IRP_MN_QUERY_CAPABILITIES: /* 0x09 */ + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_CAPABILITIES\n"); + Status = PdoQueryCapabilities(DeviceObject, Irp, IrpSp); + break; + + case IRP_MN_QUERY_RESOURCES: /* 0x0a */ + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_RESOURCES\n"); + Status = PdoQueryResources(DeviceObject, Irp, IrpSp); + break; + + case IRP_MN_QUERY_RESOURCE_REQUIREMENTS: /* 0x0b */ + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_RESOURCE_REQUIREMENTS\n"); + Status = PdoQueryResourceRequirements(DeviceObject, Irp, IrpSp); + break; + + case IRP_MN_QUERY_DEVICE_TEXT: /* 0x0c */ + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_RESOURCE_REQUIREMENTS\n"); + Status = PdoQueryDeviceText(DeviceObject, Irp, IrpSp); + break; + + case IRP_MN_FILTER_RESOURCE_REQUIREMENTS: /* 0x0d */ + DPRINT("IRP_MJ_PNP / IRP_MN_FILTER_RESOURCE_REQUIREMENTS\n"); + break; + + case IRP_MN_QUERY_ID: /* 0x13 */ + Status = PdoQueryId(DeviceObject, Irp, IrpSp); + break; + + case IRP_MN_QUERY_BUS_INFORMATION: /* 0x15 */ + DPRINT("IRP_MJ_PNP / IRP_MN_QUERY_BUS_INFORMATION\n"); + Status = PdoQueryBusInformation(DeviceObject, Irp, IrpSp); + break; + + default: + DPRINT1("IRP_MJ_PNP / Unknown minor function 0x%lx\n", IrpSp->MinorFunction); + Status = STATUS_NOT_IMPLEMENTED; + break; + } + + if (Status != STATUS_PENDING) + { Irp->IoStatus.Status = Status; IoCompleteRequest(Irp, IO_NO_INCREMENT); } - DPRINT("Leaving. Status 0x%X\n", Status); - return Status; } - /* * FUNCTION: Handle Plug and Play IRPs @@ -1055,8 +1102,7 @@ * RETURNS: * Status */ -NTSTATUS -STDCALL +static NTSTATUS NTAPI PnpRootPnpControl( IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp) @@ -1066,49 +1112,13 @@ DeviceExtension = (PPNPROOT_COMMON_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - DPRINT("DeviceObject 0x%p DeviceExtension 0x%p IsFDO %d\n", - DeviceObject, - DeviceExtension, - DeviceExtension->IsFDO); - - if (DeviceExtension->IsFDO) { + if (DeviceExtension->IsFDO) Status = PnpRootFdoPnpControl(DeviceObject, Irp); - } else { + else Status = PnpRootPdoPnpControl(DeviceObject, Irp); - } return Status; } - - -/* - * FUNCTION: Handle power management IRPs - * ARGUMENTS: - * DeviceObject = Pointer to PDO or FDO - * Irp = Pointer to IRP that should be handled - * RETURNS: - * Status - */ -NTSTATUS -STDCALL -PnpRootPowerControl( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp) -{ - PPNPROOT_COMMON_DEVICE_EXTENSION DeviceExtension; - NTSTATUS Status; - - DeviceExtension = (PPNPROOT_COMMON_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - - if (DeviceExtension->IsFDO) { - Status = PnpRootFdoPowerControl(DeviceObject, Irp); - } else { - Status = PnpRootPdoPowerControl(DeviceObject, Irp); - } - - return Status; -} - NTSTATUS STDCALL @@ -1119,7 +1129,14 @@ PPNPROOT_FDO_DEVICE_EXTENSION DeviceExtension; NTSTATUS Status; - DPRINT("Called\n"); + DPRINT("PnpRootAddDevice(DriverObject %p, Pdo %p)\n", DriverObject, PhysicalDeviceObject); + + if (!PhysicalDeviceObject) + { + DPRINT("PhysicalDeviceObject 0x%p\n", PhysicalDeviceObject); + Status = STATUS_INSUFFICIENT_RESOURCES; + KEBUGCHECKEX(PHASE1_INITIALIZATION_FAILED, Status, 0, 0, 0); + } Status = IoCreateDevice( DriverObject, @@ -1129,59 +1146,50 @@ FILE_DEVICE_SECURE_OPEN, TRUE, &PnpRootDeviceObject); - if (!NT_SUCCESS(Status)) { + if (!NT_SUCCESS(Status)) + { + DPRINT("IoCreateDevice() failed with status 0x%08lx\n", Status); KEBUGCHECKEX(PHASE1_INITIALIZATION_FAILED, Status, 0, 0, 0); } + DPRINT("Created FDO %p\n", PnpRootDeviceObject); DeviceExtension = (PPNPROOT_FDO_DEVICE_EXTENSION)PnpRootDeviceObject->DeviceExtension; - RtlZeroMemory(DeviceExtension, sizeof(PNPROOT_FDO_DEVICE_EXTENSION)); DeviceExtension->Common.IsFDO = TRUE; - DeviceExtension->State = dsStopped; - - DeviceExtension->Ldo = IoAttachDeviceToDeviceStack( + InitializeListHead(&DeviceExtension->DeviceListHead); + DeviceExtension->DeviceListCount = 0; + KeInitializeGuardedMutex(&DeviceExtension->DeviceListLock); + + Status = IoAttachDeviceToDeviceStackSafe( PnpRootDeviceObject, - PhysicalDeviceObject); - - if (!PnpRootDeviceObject) { - CPRINT("PnpRootDeviceObject 0x%p\n", PnpRootDeviceObject); + PhysicalDeviceObject, + &DeviceExtension->Ldo); + if (!NT_SUCCESS(Status)) + { + DPRINT("IoAttachDeviceToDeviceStackSafe() failed with status 0x%08lx\n", Status); KEBUGCHECKEX(PHASE1_INITIALIZATION_FAILED, Status, 0, 0, 0); } - if (!PhysicalDeviceObject) { - CPRINT("PhysicalDeviceObject 0x%p\n", PhysicalDeviceObject); - KEBUGCHECKEX(PHASE1_INITIALIZATION_FAILED, Status, 0, 0, 0); - } - - InitializeListHead(&DeviceExtension->DeviceListHead); - - DeviceExtension->DeviceListCount = 0; - - KeInitializeSpinLock(&DeviceExtension->DeviceListLock); - PnpRootDeviceObject->Flags &= ~DO_DEVICE_INITIALIZING; - //PnpRootDeviceObject->Flags |= DO_POWER_PAGABLE; - DPRINT("Done AddDevice()\n"); return STATUS_SUCCESS; } - -NTSTATUS -STDCALL +NTSTATUS NTAPI PnpRootDriverEntry( IN PDRIVER_OBJECT DriverObject, IN PUNICODE_STRING RegistryPath) { - DPRINT("Called\n"); + DPRINT("PnpRootDriverEntry(%p %wZ)\n", DriverObject, RegistryPath); + + DriverObject->DriverExtension->AddDevice = PnpRootAddDevice; DriverObject->MajorFunction[IRP_MJ_PNP] = PnpRootPnpControl; - DriverObject->MajorFunction[IRP_MJ_POWER] = PnpRootPowerControl; - DriverObject->DriverExtension->AddDevice = PnpRootAddDevice; + //DriverObject->MajorFunction[IRP_MJ_POWER] = PnpRootPowerControl; return STATUS_SUCCESS; }

1 0

[hpoussin] 26987: Commit a patch I have in my tree since 1 year. This may be related to bug 2238.
by hpoussin＠svn.reactos.org 08 Jun '07

08 Jun '07

Author: hpoussin Date: Mon Jun 4 13:02:12 2007 New Revision: 26987 URL: http://svn.reactos.org/svn/reactos?rev=26987&view=rev Log: Commit a patch I have in my tree since 1 year. This may be related to bug 2238. Modified: trunk/reactos/subsystems/win32/win32k/ntuser/message.c Modified: trunk/reactos/subsystems/win32/win32k/ntuser/message.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/message.c (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/message.c Mon Jun 4 13:02:12 2007 @@ -1224,7 +1224,7 @@ IntGetCursorLocation(PsGetCurrentThreadWin32Thread()->Desktop->WindowStation, &KernelModeMsg.pt); KeQueryTickCount(&LargeTickCount); - KernelModeMsg.time = LargeTickCount.u.LowPart; + KernelModeMsg.time = LargeTickCount.u.LowPart * (KeQueryTimeIncrement() / 10000); MsqPostMessage(Window->MessageQueue, &KernelModeMsg, NULL != MsgMemoryEntry && 0 != KernelModeMsg.lParam, QS_POSTMESSAGE);

1 0

[hpoussin] 26986: Do not force user to enter an admin password This decision needs to be reconsidered once ReactOS has a better security (lsass...)
by hpoussin＠svn.reactos.org 08 Jun '07

08 Jun '07

Author: hpoussin Date: Mon Jun 4 12:23:45 2007 New Revision: 26986 URL: http://svn.reactos.org/svn/reactos?rev=26986&view=rev Log: Do not force user to enter an admin password This decision needs to be reconsidered once ReactOS has a better security (lsass...) Modified: trunk/reactos/dll/win32/syssetup/wizard.c Modified: trunk/reactos/dll/win32/syssetup/wizard.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/syssetup/wizard.… ============================================================================== --- trunk/reactos/dll/win32/syssetup/wizard.c (original) +++ trunk/reactos/dll/win32/syssetup/wizard.c Mon Jun 4 12:23:45 2007 @@ -650,6 +650,7 @@ return TRUE; } +#if 0 /* Check if admin passwords have been entered */ if ((GetDlgItemText(hwndDlg, IDC_ADMINPASSWORD1, Password1, 15) == 0) || (GetDlgItemText(hwndDlg, IDC_ADMINPASSWORD2, Password2, 15) == 0)) @@ -663,6 +664,10 @@ SetWindowLong(hwndDlg, DWL_MSGRESULT, -1); return TRUE; } +#else + GetDlgItemText(hwndDlg, IDC_ADMINPASSWORD1, Password1, 15); + GetDlgItemText(hwndDlg, IDC_ADMINPASSWORD2, Password2, 15); +#endif /* Check if passwords match */ if (_tcscmp(Password1, Password2)) {

1 0

[greatlrd] 26985: irc : Myrria (from linux xbox project) desc a bug in KiUserExceptionDispatcher and how to fix it. See issue #2279 for more details.
by greatlrd＠svn.reactos.org 08 Jun '07

08 Jun '07

Author: greatlrd Date: Mon Jun 4 09:50:39 2007 New Revision: 26985 URL: http://svn.reactos.org/svn/reactos?rev=26985&view=rev Log: irc : Myrria (from linux xbox project) desc a bug in KiUserExceptionDispatcher and how to fix it. See issue #2279 for more details. Modified: trunk/reactos/dll/ntdll/main/i386/dispatch.S Modified: trunk/reactos/dll/ntdll/main/i386/dispatch.S URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/ntdll/main/i386/dispat… ============================================================================== --- trunk/reactos/dll/ntdll/main/i386/dispatch.S (original) +++ trunk/reactos/dll/ntdll/main/i386/dispatch.S Mon Jun 4 09:50:39 2007 @@ -183,6 +183,30 @@ .globl _KiUserExceptionDispatcher@8 _KiUserExceptionDispatcher@8: + /* clear the direct flag + * text from bug 2279 + * if it not clear it means that if an exception occurs while + * the direction flag is set (typically inside memmove), the + * exception handlers will be called with the direction flag still + * set. The Windows x86-32 and x86-64 ABI requires that the + * direction flag be Calling memset() with a compile-time constant + * size on both GCC and MSVC will result in inlining a "rep stosd" + * instruction. Because of the ABI, they will assume that the + * direction flag is clear and not emit a "cld" instruction. + * Using memset() in an exception handler therefore will + * corrupt memory if the exception occurred during a reverse copy + * such as a forward overlapping memmove(). + * + * For reliability and ease of debugging, please add "cld" to the beginning of + * KiUserExceptionDispatcher. Note that the same will be true of x86-64 whenever + * that happens. This does not affect continuing execution; the CONTEXT of the + * exception has the direction flag set and will be restored upon NtContinue. + * KiUserApcDispatcher and KiUserCallbackDispatcher need to be evaluated for this + * issue. + */ + + cld + /* Save the Context and Exception Records */ mov ecx, [esp+4] mov ebx, [esp]

1 0

[dgorbachev] 26984: - Remove corresponding kernel hack.
by dgorbachev＠svn.reactos.org 08 Jun '07

08 Jun '07

Author: dgorbachev Date: Mon Jun 4 02:12:28 2007 New Revision: 26984 URL: http://svn.reactos.org/svn/reactos?rev=26984&view=rev Log: - Remove corresponding kernel hack. Modified: trunk/reactos/ntoskrnl/config/cmsysini.c Modified: trunk/reactos/ntoskrnl/config/cmsysini.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/config/cmsysini.c… ============================================================================== --- trunk/reactos/ntoskrnl/config/cmsysini.c (original) +++ trunk/reactos/ntoskrnl/config/cmsysini.c Mon Jun 4 02:12:28 2007 @@ -251,26 +251,17 @@ NULL, NULL); - /* The key has been created by mkhive */ - if ((CmpMiniNTBoot) && (CmpShareSystemHives)) - { - Status = NtOpenKey(&KeyHandle, KEY_READ | KEY_WRITE, &ObjectAttributes); - if (!NT_SUCCESS(Status)) return Status; - } - else - { - Status = NtCreateKey(&KeyHandle, - KEY_CREATE_LINK, - &ObjectAttributes, - 0, - NULL, - REG_OPTION_VOLATILE | REG_OPTION_CREATE_LINK, - &Disposition); - if (!NT_SUCCESS(Status)) return Status; - - /* Sanity check */ - ASSERT(Disposition == REG_CREATED_NEW_KEY); - } + Status = NtCreateKey(&KeyHandle, + KEY_CREATE_LINK, + &ObjectAttributes, + 0, + NULL, + REG_OPTION_VOLATILE | REG_OPTION_CREATE_LINK, + &Disposition); + if (!NT_SUCCESS(Status)) return Status; + + /* Sanity check */ + ASSERT(Disposition == REG_CREATED_NEW_KEY); /* Initialize the symbolic link name */ sprintf(Buffer,

1 0

← Newer
1
...
32
33
34
35
36
37
38
...
42
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 June 2007