Ros-diffs February 2012

ros-diffs@reactos.org

22 participants
577 discussions

[tkreuzer] 55456: [NTOSKRNL] - Implement MiSynchronizeSystemPde, which does what its name suggests, synchronize a system PDE and is an improved replacement (with a more proper name) for MiCheckPdeF...

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Mon Feb 6 10:46:52 2012 New Revision: 55456 URL: http://svn.reactos.org/svn/reactos?rev=55456&view=rev Log: [NTOSKRNL] - Implement MiSynchronizeSystemPde, which does what its name suggests, synchronize a system PDE and is an improved replacement (with a more proper name) for MiCheckPdeForPagedPool - Move some code to avoid an additional check - Call MiResolveDemandZeroFault directy instead of creating a demand zero PDE and then calling MiDispatchFault, which after a lot of checks will finally do the same Modified: trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c Modified: trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/pagfault.… ============================================================================== --- trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c [iso-8859-1] Mon Feb 6 10:46:52 2012 @@ -100,6 +100,28 @@ } } +#if (_MI_PAGING_LEVELS == 2) +BOOLEAN +FORCEINLINE +MiSynchronizeSystemPde(PMMPDE PointerPde) +{ + MMPDE SystemPde; + ULONG Index; + + /* Get the Index from the PDE */ + Index = ((ULONG_PTR)PointerPde & (SYSTEM_PD_SIZE - 1)) / sizeof(MMPTE); + + /* Copy the PDE from the double-mapped system page directory */ + SystemPde = MmSystemPagePtes[Index]; + *PointerPde = SystemPde; + + /* Make sure we re-read the PDE and PTE */ + KeMemoryBarrierWithoutFence(); + + /* Return, if we had success */ + return (BOOLEAN)SystemPde.u.Hard.Valid; +} + NTSTATUS FASTCALL MiCheckPdeForPagedPool(IN PVOID Address) @@ -159,6 +181,7 @@ // return Status; } +#endif VOID NTAPI @@ -699,28 +722,12 @@ /* Check if the PDE is invalid */ if (PointerPde->u.Hard.Valid == 0) { - // - // Debug spew (eww!) - // - DPRINT("Invalid PDE\n"); #if (_MI_PAGING_LEVELS == 2) - // - // Handle mapping in "Special" PDE directoreis - // - MiCheckPdeForPagedPool(Address); -#endif - // - // Now we SHOULD be good - // - if (PointerPde->u.Hard.Valid == 0) + /* Sync this PDE and check, if that made it valid */ + if (!MiSynchronizeSystemPde(PointerPde)) +#endif { - // - // FIXFIX: Do the S-LIST hack - // - - // - // Kill the system - // + /* PDE (still) not valid, kill the system */ KeBugCheckEx(PAGE_FAULT_IN_NONPAGED_AREA, (ULONG_PTR)Address, StoreInstruction, @@ -824,20 +831,20 @@ (ULONG_PTR)TrapInformation, 1); } - } - - /* Check for demand page */ - if ((StoreInstruction) && !(ProtoPte) && !(TempPte.u.Hard.Valid)) - { - /* Get the protection code */ - if (!(TempPte.u.Soft.Protection & MM_READWRITE)) + + /* Check for demand page */ + if ((StoreInstruction) && !(TempPte.u.Hard.Valid)) { - /* Bad boy, bad boy, whatcha gonna do, whatcha gonna do when ARM3 comes for you! */ - KeBugCheckEx(ATTEMPTED_WRITE_TO_READONLY_MEMORY, - (ULONG_PTR)Address, - TempPte.u.Long, - (ULONG_PTR)TrapInformation, - 14); + /* Get the protection code */ + if (!(TempPte.u.Soft.Protection & MM_READWRITE)) + { + /* Bugcheck the system! */ + KeBugCheckEx(ATTEMPTED_WRITE_TO_READONLY_MEMORY, + (ULONG_PTR)Address, + TempPte.u.Long, + (ULONG_PTR)TrapInformation, + 14); + } } } @@ -889,21 +896,15 @@ /* Right now, we expect a valid protection mask on the VAD */ ASSERT(ProtectionCode != MM_NOACCESS); - /* Make the PDE demand-zero */ - MI_WRITE_INVALID_PDE(PointerPde, DemandZeroPde); - /* And go dispatch the fault on the PDE. This should handle the demand-zero */ #if MI_TRACE_PFNS UserPdeFault = TRUE; #endif - Status = MiDispatchFault(TRUE, - PointerPte, - (PMMPTE)PointerPde, - NULL, - FALSE, - PsGetCurrentProcess(), - TrapInformation, - NULL); + /* Resolve a demand zero fault */ + Status = MiResolveDemandZeroFault(PointerPte, + MM_READWRITE, + CurrentProcess, + MM_NOIRQL); #if MI_TRACE_PFNS UserPdeFault = FALSE; #endif @@ -930,15 +931,14 @@ return STATUS_PAGE_FAULT_DEMAND_ZERO; } - /* Get protection and check if it's a prototype PTE */ - ProtectionCode = (ULONG)TempPte.u.Soft.Protection; + /* Make sure it's not a prototype PTE */ ASSERT(TempPte.u.Soft.Prototype == 0); /* Check for non-demand zero PTE */ if (TempPte.u.Long != 0) { /* This is a page fault, check for valid protection */ - ASSERT(ProtectionCode != 0x100); + ASSERT(TempPte.u.Soft.Protection != 0x100); /* FIXME: Run MiAccessCheck */

12 years, 10 months

[ion] 55455: [SMSS]: Nuke the environment code and instead inherit SMSS2's, which does all the right things. #if 0 all the non-used code, and nuke support for the "debug" subsystem as well as the c...

by ion＠svn.reactos.org

Author: ion Date: Mon Feb 6 09:29:33 2012 New Revision: 55455 URL: http://svn.reactos.org/svn/reactos?rev=55455&view=rev Log: [SMSS]: Nuke the environment code and instead inherit SMSS2's, which does all the right things. #if 0 all the non-used code, and nuke support for the "debug" subsystem as well as the custom "SmQuery" APIs. [SMSS2]: Use SMSS2's environment when creating SMSS. Modified: trunk/reactos/base/system/smss/client.c trunk/reactos/base/system/smss/debug.c trunk/reactos/base/system/smss/init.c trunk/reactos/base/system/smss/initdosdev.c trunk/reactos/base/system/smss/initenv.c trunk/reactos/base/system/smss/initmv.c trunk/reactos/base/system/smss/initobdir.c trunk/reactos/base/system/smss/initpage.c trunk/reactos/base/system/smss/initreg.c trunk/reactos/base/system/smss/initrun.c trunk/reactos/base/system/smss/initwkdll.c trunk/reactos/base/system/smss/print.c trunk/reactos/base/system/smss/smapi.c trunk/reactos/base/system/smss/smapiexec.c trunk/reactos/base/system/smss/smapiquery.c trunk/reactos/base/system/smss2/smss.c Modified: trunk/reactos/base/system/smss/client.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/client.c?… ============================================================================== --- trunk/reactos/base/system/smss/client.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/client.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -500,7 +500,7 @@ RtlLeaveCriticalSection (& SmpClientDirectory.Lock); return Status; } - +#if 0 /* === Utilities for SmQryInfo === */ /********************************************************************** @@ -566,5 +566,5 @@ RtlLeaveCriticalSection (& SmpClientDirectory.Lock); return Status; } - +#endif /* EOF */ Modified: trunk/reactos/base/system/smss/debug.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/debug.c?r… ============================================================================== --- trunk/reactos/base/system/smss/debug.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/debug.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -12,6 +12,7 @@ #define NDEBUG #include <debug.h> +#if 0 /* GLOBALS ***********************************************************/ @@ -164,5 +165,7 @@ return STATUS_SUCCESS; } +#endif + /* EOF */ Modified: trunk/reactos/base/system/smss/init.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/init.c?re… ============================================================================== --- trunk/reactos/base/system/smss/init.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/init.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -30,7 +30,7 @@ // {TRUE, SmInitDosDevices, "create dos device links"}, // {TRUE, SmRunBootApplications, "run boot applications"}, // {TRUE, SmProcessFileRenameList, "process the file rename list"}, - {FALSE, SmUpdateEnvironment, "update environment variables"}, +// {FALSE, SmUpdateEnvironment, "update environment variables"}, // {FALSE, SmLoadKnownDlls, "preload system DLLs"}, // {TRUE, SmCreatePagingFiles, "create paging files"}, // {TRUE, SmInitializeRegistry, "initialize the registry"}, Modified: trunk/reactos/base/system/smss/initdosdev.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/initdosde… ============================================================================== --- trunk/reactos/base/system/smss/initdosdev.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/initdosdev.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -11,6 +11,8 @@ #define NDEBUG #include <debug.h> + +#if 0 static NTSTATUS NTAPI SmpDosDevicesQueryRoutine(PWSTR ValueName, @@ -89,4 +91,6 @@ return(Status); } +#endif + /* EOF */ Modified: trunk/reactos/base/system/smss/initenv.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/initenv.c… ============================================================================== --- trunk/reactos/base/system/smss/initenv.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/initenv.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -22,9 +22,11 @@ NTSTATUS SmCreateEnvironment(VOID) { - return RtlCreateEnvironment(FALSE, &SmSystemEnvironment); -} - + return RtlCreateEnvironment(TRUE, &SmSystemEnvironment); +} + + +#if 0 static NTSTATUS SmpSetEnvironmentVariable(IN PVOID Context, @@ -60,7 +62,6 @@ DPRINT("ValueData '%S'\n", (PWSTR)ValueData); return SmpSetEnvironmentVariable(Context,ValueName,(PWSTR)ValueData); } - NTSTATUS SmSetEnvironmentVariables(VOID) @@ -280,14 +281,13 @@ return Status; } - /********************************************************************** * Set environment variables from registry */ NTSTATUS SmUpdateEnvironment(VOID) { - RTL_QUERY_REGISTRY_TABLE QueryTable[2]; + //RTL_QUERY_REGISTRY_TABLE QueryTable[2]; WCHAR ValueBuffer[MAX_PATH]; NTSTATUS Status; #ifndef NDEBUG @@ -343,5 +343,5 @@ return Status; } - +#endif /* EOF */ Modified: trunk/reactos/base/system/smss/initmv.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/initmv.c?… ============================================================================== --- trunk/reactos/base/system/smss/initmv.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/initmv.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -12,6 +12,7 @@ #define NDEBUG #include <debug.h> +#if 0 /* FUNCTIONS *****************************************************************/ @@ -446,4 +447,6 @@ return (STATUS_SUCCESS); } +#endif + /* EOF */ Modified: trunk/reactos/base/system/smss/initobdir.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/initobdir… ============================================================================== --- trunk/reactos/base/system/smss/initobdir.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/initobdir.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -12,6 +12,7 @@ #define NDEBUG #include <debug.h> +#if 0 static NTSTATUS NTAPI SmpObjectDirectoryQueryRoutine(PWSTR ValueName, ULONG ValueType, @@ -69,5 +70,6 @@ return(Status); } +#endif /* EOF */ Modified: trunk/reactos/base/system/smss/initpage.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/initpage.… ============================================================================== --- trunk/reactos/base/system/smss/initpage.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/initpage.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -11,6 +11,8 @@ #define NDEBUG #include <debug.h> + +#if 0 #define GIGABYTE (1024 * 1024 * 1024) /* One Gigabyte */ @@ -520,6 +522,6 @@ return Status; } - +#endif /* EOF */ Modified: trunk/reactos/base/system/smss/initreg.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/initreg.c… ============================================================================== --- trunk/reactos/base/system/smss/initreg.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/initreg.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -12,6 +12,7 @@ #define NDEBUG #include <debug.h> +#if 0 NTSTATUS SmInitializeRegistry(VOID) { @@ -20,5 +21,6 @@ /* Load remaining registry hives */ return NtInitializeRegistry(CM_BOOT_FLAG_SMSS); } +#endif /* EOF */ Modified: trunk/reactos/base/system/smss/initrun.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/initrun.c… ============================================================================== --- trunk/reactos/base/system/smss/initrun.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/initrun.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -12,6 +12,7 @@ #define NDEBUG #include <debug.h> +#if 0 //HANDLE Children[2] = {0, 0}; /* csrss, winlogon */ @@ -223,6 +224,6 @@ return(Status); } - +#endif /* EOF */ Modified: trunk/reactos/base/system/smss/initwkdll.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/initwkdll… ============================================================================== --- trunk/reactos/base/system/smss/initwkdll.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/initwkdll.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -12,6 +12,7 @@ #define NDEBUG #include <debug.h> +#if 0 static NTSTATUS NTAPI SmpKnownDllsQueryRoutine(PWSTR ValueName, ULONG ValueType, @@ -236,6 +237,6 @@ return Status; } - +#endif /* EOF */ Modified: trunk/reactos/base/system/smss/print.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/print.c?r… ============================================================================== --- trunk/reactos/base/system/smss/print.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/print.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -12,6 +12,7 @@ #define NDEBUG #include <debug.h> +#if 0 VOID NTAPI DisplayString(LPCWSTR lpwString) { UNICODE_STRING us; @@ -39,5 +40,6 @@ NtDisplayString(&UnicodeString); RtlFreeUnicodeString (&UnicodeString); } +#endif /* EOF */ Modified: trunk/reactos/base/system/smss/smapi.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/smapi.c?r… ============================================================================== --- trunk/reactos/base/system/smss/smapi.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/smapi.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -35,7 +35,7 @@ SmInvalid, /* obsolete */ SmInvalid, /* unknown */ SmExecPgm, /* smapiexec.c */ - SmQryInfo /* smapyqry.c */ + SmInvalid /* smapyqry.c */ }; /* TODO: optimize this address computation (it should be done Modified: trunk/reactos/base/system/smss/smapiexec.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/smapiexec… ============================================================================== --- trunk/reactos/base/system/smss/smapiexec.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/smapiexec.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -263,23 +263,9 @@ else { /* - * OK, the definition is empty, but check - * if it is the name of an embedded subsystem. + * Badly defined subsystem. Check the registry! */ - if(0 == _wcsicmp(L"DEBUG", Name)) - { - /* - * Initialize the embedded DBGSS. - */ - Request->SmHeader.Status = SmInitializeDbgSs(); - } - else - { - /* - * Badly defined subsystem. Check the registry! - */ - Request->SmHeader.Status = STATUS_NOT_FOUND; - } + Request->SmHeader.Status = STATUS_NOT_FOUND; } } else { /* It couldn't lookup the Name! */ Modified: trunk/reactos/base/system/smss/smapiquery.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/smapiquer… ============================================================================== --- trunk/reactos/base/system/smss/smapiquery.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/smapiquery.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -12,7 +12,7 @@ #define NDEBUG #include <debug.h> - +#if 0 /********************************************************************** * SmQryInfo/1 API */ @@ -50,6 +50,6 @@ } return Status; } - +#endif /* EOF */ Modified: trunk/reactos/base/system/smss2/smss.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss2/smss.c?r… ============================================================================== --- trunk/reactos/base/system/smss2/smss.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss2/smss.c [iso-8859-1] Mon Feb 6 09:29:33 2012 @@ -305,7 +305,7 @@ SIZE_T Size; PWSTR p; UNICODE_STRING NullString = RTL_CONSTANT_STRING(L""); - UNICODE_STRING SmssName, Environment, SystemDriveString, DebugString; + UNICODE_STRING SmssName, DebugString; PVOID EnvironmentPtr = NULL; PRTL_USER_PROCESS_INFORMATION ProcessInformation; PRTL_USER_PROCESS_PARAMETERS ProcessParams = NULL; @@ -436,28 +436,7 @@ NtInitialUserProcessBuffer); /* Create the environment string */ - RtlInitEmptyUnicodeString(&Environment, - ProcessParams->Environment, - (USHORT)Size); - - /* Append the DLL path to it */ - RtlAppendUnicodeToString(&Environment, L"Path="); - RtlAppendUnicodeStringToString(&Environment, &ProcessParams->DllPath); - RtlAppendUnicodeStringToString(&Environment, &NullString); - - /* Create the system drive string */ - SystemDriveString = SmpSystemRoot; - SystemDriveString.Length = 2 * sizeof(WCHAR); - - /* Append it to the environment */ - RtlAppendUnicodeToString(&Environment, L"SystemDrive="); - RtlAppendUnicodeStringToString(&Environment, &SystemDriveString); - RtlAppendUnicodeStringToString(&Environment, &NullString); - - /* Append the system root to the environment */ - RtlAppendUnicodeToString(&Environment, L"SystemRoot="); - RtlAppendUnicodeStringToString(&Environment, &SmpSystemRoot); - RtlAppendUnicodeStringToString(&Environment, &NullString); + ProcessParams->Environment = SmpDefaultEnvironment; /* Create SMSS process */ SmssName = ProcessParams->ImagePathName;

12 years, 10 months

[ion] 55453: [SMSS2]: Implement SmpProcessFileRenames and actually create the paging file now. All initialization work is now done by SMSS2. [SMSS]: Neuter away most of the code and only keep SMSS ...

by ion＠svn.reactos.org

Author: ion Date: Mon Feb 6 09:06:38 2012 New Revision: 55453 URL: http://svn.reactos.org/svn/reactos?rev=55453&view=rev Log: [SMSS2]: Implement SmpProcessFileRenames and actually create the paging file now. All initialization work is now done by SMSS2. [SMSS]: Neuter away most of the code and only keep SMSS around for managing the API port. Next step is to make the SMSS2 API port go into use. Modified: trunk/reactos/base/system/smss/init.c trunk/reactos/base/system/smss2/pagefile.c trunk/reactos/base/system/smss2/sminit.c Modified: trunk/reactos/base/system/smss/init.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss/init.c?re… ============================================================================== --- trunk/reactos/base/system/smss/init.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss/init.c [iso-8859-1] Mon Feb 6 09:06:38 2012 @@ -23,16 +23,16 @@ PCHAR ErrorMessage; } InitRoutine [] = { {TRUE, SmCreateHeap, "create private heap, aborting"}, - {TRUE, SmCreateObjectDirectories, "create object directories"}, +// {TRUE, SmCreateObjectDirectories, "create object directories"}, {TRUE, SmCreateApiPort, "create \\SmApiPort"}, {TRUE, SmCreateEnvironment, "create the system environment"}, - {TRUE, SmSetEnvironmentVariables, "set system environment variables"}, - {TRUE, SmInitDosDevices, "create dos device links"}, - {TRUE, SmRunBootApplications, "run boot applications"}, - {TRUE, SmProcessFileRenameList, "process the file rename list"}, +// {TRUE, SmSetEnvironmentVariables, "set system environment variables"}, +// {TRUE, SmInitDosDevices, "create dos device links"}, +// {TRUE, SmRunBootApplications, "run boot applications"}, +// {TRUE, SmProcessFileRenameList, "process the file rename list"}, {FALSE, SmUpdateEnvironment, "update environment variables"}, - {FALSE, SmLoadKnownDlls, "preload system DLLs"}, - {TRUE, SmCreatePagingFiles, "create paging files"}, +// {FALSE, SmLoadKnownDlls, "preload system DLLs"}, +// {TRUE, SmCreatePagingFiles, "create paging files"}, // {TRUE, SmInitializeRegistry, "initialize the registry"}, {TRUE, SmInitializeClientManagement, "initialize client management"}, {TRUE, SmLoadSubsystems, "load subsystems"} Modified: trunk/reactos/base/system/smss2/pagefile.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss2/pagefile… ============================================================================== --- trunk/reactos/base/system/smss2/pagefile.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss2/pagefile.c [iso-8859-1] Mon Feb 6 09:06:38 2012 @@ -451,11 +451,9 @@ IN ULONG Priority) { NTSTATUS Status; - DPRINT1("Should request pagefile: %wZ with size %I64x and %I64x\n", Name, MinSize->QuadPart, MaxSize->QuadPart); /* Tell the kernel to create the pagefile */ - Status = STATUS_SUCCESS; - //Status = NtCreatePagingFile(Name, MinSize, MaxSize, Priority); + Status = NtCreatePagingFile(Name, MinSize, MaxSize, Priority); if (NT_SUCCESS(Status)) { DPRINT1("SMSS:PFILE: NtCreatePagingFile (%wZ, %I64X, %I64X) succeeded. \n", Modified: trunk/reactos/base/system/smss2/sminit.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss2/sminit.c… ============================================================================== --- trunk/reactos/base/system/smss2/sminit.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss2/sminit.c [iso-8859-1] Mon Feb 6 09:06:38 2012 @@ -1827,7 +1827,211 @@ NTAPI SmpProcessFileRenames(VOID) { - return STATUS_SUCCESS; + BOOLEAN OldState, HavePrivilege; + NTSTATUS Status; + HANDLE FileHandle, OtherFileHandle; + FILE_INFORMATION_CLASS InformationClass; + OBJECT_ATTRIBUTES ObjectAttributes; + IO_STATUS_BLOCK IoStatusBlock; + UNICODE_STRING FileString; + FILE_BASIC_INFORMATION BasicInfo; + FILE_DISPOSITION_INFORMATION DeleteInformation; + PFILE_RENAME_INFORMATION Buffer; + PLIST_ENTRY Head, NextEntry; + PSMP_REGISTRY_VALUE RegEntry; + PWCHAR FileName; + ULONG ValueLength, Length; + + /* Give us access to restore any files we want */ + Status = RtlAdjustPrivilege(SE_RESTORE_PRIVILEGE, TRUE, FALSE, &OldState); + if (NT_SUCCESS(Status)) HavePrivilege = TRUE; + + /* Process pending files to rename */ + Head = &SmpFileRenameList; + while (!IsListEmpty(Head)) + { + /* Get this entry */ + NextEntry = RemoveHeadList(Head); + RegEntry = CONTAINING_RECORD(NextEntry, SMP_REGISTRY_VALUE, Entry); + DPRINT1("Processing PFRO: %wZ/%wZ\n", &RegEntry->Value, &RegEntry->Name); + + /* Skip past the '@' marker */ + if (!(RegEntry->Value.Length) && (*RegEntry->Name.Buffer == L'@')) + { + RegEntry->Name.Length -= sizeof(UNICODE_NULL); + RegEntry->Name.Buffer++; + } + + /* Open the file for delete access */ + InitializeObjectAttributes(&ObjectAttributes, + &RegEntry->Name, + OBJ_CASE_INSENSITIVE, + NULL, + NULL); + Status = NtOpenFile(&OtherFileHandle, + DELETE | SYNCHRONIZE, + &ObjectAttributes, + &IoStatusBlock, + FILE_SHARE_READ | FILE_SHARE_WRITE, + FILE_SYNCHRONOUS_IO_NONALERT); + if (!NT_SUCCESS(Status)) goto Quickie; + + /* Check if it's a rename or just a delete */ + ValueLength = RegEntry->Value.Length; + if (!ValueLength) + { + /* Just a delete, set up the class, length and buffer */ + InformationClass = FileDispositionInformation; + Length = sizeof(DeleteInformation); + Buffer = (PFILE_RENAME_INFORMATION)&DeleteInformation; + + /* Set the delete disposition */ + DeleteInformation.DeleteFile = TRUE; + } + else + { + /* This is a rename, setup the class and length */ + InformationClass = FileRenameInformation; + Length = ValueLength + sizeof(FILE_RENAME_INFORMATION); + + /* Skip past the special markers */ + FileName = RegEntry->Value.Buffer; + if ((*FileName == L'!') || (*FileName == L'@')) + { + FileName++; + Length -= sizeof(UNICODE_NULL); + } + + /* Now allocate the buffer for the rename information */ + Buffer = RtlAllocateHeap(RtlGetProcessHeap(), SmBaseTag, Length); + if (Buffer) + { + /* Setup the buffer to point to the filename, and copy it */ + Buffer->RootDirectory = NULL; + Buffer->FileNameLength = Length - sizeof(FILE_RENAME_INFORMATION); + Buffer->ReplaceIfExists = FileName != RegEntry->Value.Buffer; + RtlCopyMemory(Buffer->FileName, FileName, Buffer->FileNameLength); + } + else + { + /* Fail */ + Status = STATUS_NO_MEMORY; + } + } + + /* Check if everything is okay till here */ + if (NT_SUCCESS(Status)) + { + /* Now either rename or delete the file as requested */ + Status = NtSetInformationFile(OtherFileHandle, + &IoStatusBlock, + Buffer, + Length, + InformationClass); + + /* Check if we seem to have failed because the file was readonly */ + if ((NT_SUCCESS(Status) && + (InformationClass == FileRenameInformation) && + (Status == STATUS_OBJECT_NAME_COLLISION) && + (Buffer->ReplaceIfExists))) + { + /* Open the file for write attribute access this time... */ + DPRINT1("\nSMSS: %wZ => %wZ failed - Status == %x, Possible readonly target\n", + &RegEntry->Name, + &RegEntry->Value, + STATUS_OBJECT_NAME_COLLISION); + FileString.Length = RegEntry->Value.Length - sizeof(WCHAR); + FileString.MaximumLength = RegEntry->Value.MaximumLength - sizeof(WCHAR); + FileString.Buffer = FileName; + InitializeObjectAttributes(&ObjectAttributes, + &FileString, + OBJ_CASE_INSENSITIVE, + NULL, + NULL); + Status = NtOpenFile(&FileHandle, + FILE_WRITE_ATTRIBUTES | SYNCHRONIZE, + &ObjectAttributes, + &IoStatusBlock, + FILE_SHARE_READ | FILE_SHARE_WRITE, + FILE_SYNCHRONOUS_IO_NONALERT); + if (!NT_SUCCESS(Status)) + { + /* That didn't work, so bail out */ + DPRINT1(" SMSS: Open Existing file Failed - Status == %x\n", + Status); + } + else + { + /* Now remove the read-only attribute from the file */ + DPRINT1(" SMSS: Open Existing Success\n"); + RtlZeroMemory(&BasicInfo, sizeof(BasicInfo)); + BasicInfo.FileAttributes = FILE_ATTRIBUTE_NORMAL; + Status = NtSetInformationFile(FileHandle, + &IoStatusBlock, + &BasicInfo, + sizeof(BasicInfo), + FileBasicInformation); + NtClose(FileHandle); + if (!NT_SUCCESS(Status)) + { + /* That didn't work, bail out */ + DPRINT1(" SMSS: Set To NORMAL Failed - Status == %x\n", + Status); + } + else + { + /* Now that the file is no longer read-only, delete! */ + DPRINT1(" SMSS: Set To NORMAL OK\n"); + Status = NtSetInformationFile(OtherFileHandle, + &IoStatusBlock, + Buffer, + Length, + FileRenameInformation); + if (!NT_SUCCESS(Status)) + { + /* That failed too! */ + DPRINT1(" SMSS: Re-Rename Failed - Status == %x\n", + Status); + } + else + { + /* Everything ok */ + DPRINT1(" SMSS: Re-Rename Worked OK\n"); + } + } + } + } + } + + /* Close the file handle and check the operation result */ + NtClose(OtherFileHandle); +Quickie: + if (!NT_SUCCESS(Status)) + { + /* We totally failed */ + DPRINT1("SMSS: %wZ => %wZ failed - Status == %x\n", + &RegEntry->Name, &RegEntry->Value, Status); + } + else if (RegEntry->Value.Length) + { + /* We succeed with a rename */ + DPRINT1("SMSS: %wZ (renamed to) %wZ\n", &RegEntry->Name, &RegEntry->Value); + } + else + { + /* We suceeded with a delete */ + DPRINT1("SMSS: %wZ (deleted)\n", &RegEntry->Name); + } + + /* Now free this entry and keep going */ + if (RegEntry->AnsiValue) RtlFreeHeap(RtlGetProcessHeap(), 0, RegEntry->AnsiValue); + if (RegEntry->Value.Buffer) RtlFreeHeap(RtlGetProcessHeap(), 0, RegEntry->Value.Buffer); + RtlFreeHeap(RtlGetProcessHeap(), 0, RegEntry); + } + + /* Put back the restore privilege if we had requested it, and return */ + if (HavePrivilege) RtlAdjustPrivilege(SE_RESTORE_PRIVILEGE, FALSE, FALSE, &OldState); + return Status; } NTSTATUS

12 years, 10 months

[ion] 55452: [NTOSKRNL]: Finish implementation of NtAreMappedFilesTheSame based on what winetests revealed. Should fix all 20+ winetests in this area, on top of making Ldr suck less.

by ion＠svn.reactos.org

Author: ion Date: Mon Feb 6 08:17:03 2012 New Revision: 55452 URL: http://svn.reactos.org/svn/reactos?rev=55452&view=rev Log: [NTOSKRNL]: Finish implementation of NtAreMappedFilesTheSame based on what winetests revealed. Should fix all 20+ winetests in this area, on top of making Ldr suck less. Modified: trunk/reactos/ntoskrnl/mm/ARM3/section.c Modified: trunk/reactos/ntoskrnl/mm/ARM3/section.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/section.c… ============================================================================== --- trunk/reactos/ntoskrnl/mm/ARM3/section.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/ARM3/section.c [iso-8859-1] Mon Feb 6 08:17:03 2012 @@ -1387,8 +1387,6 @@ PMEMORY_AREA MemoryArea1, MemoryArea2; PROS_SECTION_OBJECT Section1, Section2; - DPRINT1("Is image: %p the same as file: %p?\n", File1MappedAsAnImage, File2MappedAsFile); - /* Lock address space */ AddressSpace = MmGetCurrentAddressSpace(); MmLockAddressSpace(AddressSpace); @@ -1398,7 +1396,6 @@ if (!MemoryArea1) { /* Fail, the address does not exist */ - DPRINT1("Invalid address\n"); MmUnlockAddressSpace(AddressSpace); return STATUS_INVALID_ADDRESS; } @@ -1407,7 +1404,6 @@ if (MemoryArea1->Type != MEMORY_AREA_SECTION_VIEW) { /* Fail, the address is not a section */ - DPRINT1("Invalid address (not a section)\n"); MmUnlockAddressSpace(AddressSpace); return STATUS_CONFLICTING_ADDRESSES; } @@ -1416,7 +1412,6 @@ Section1 = MemoryArea1->Data.SectionData.Section; if (Section1->FileObject == NULL) { - DPRINT1("No file object\n"); MmUnlockAddressSpace(AddressSpace); return STATUS_CONFLICTING_ADDRESSES; } @@ -1426,7 +1421,6 @@ if (!MemoryArea2) { /* Fail, the address does not exist */ - DPRINT1("Invalid address\n"); MmUnlockAddressSpace(AddressSpace); return STATUS_INVALID_ADDRESS; } @@ -1435,7 +1429,6 @@ if (MemoryArea2->Type != MEMORY_AREA_SECTION_VIEW) { /* Fail, the address is not a section */ - DPRINT1("Invalid address (not a section)\n"); MmUnlockAddressSpace(AddressSpace); return STATUS_CONFLICTING_ADDRESSES; } @@ -1444,23 +1437,17 @@ Section2 = MemoryArea2->Data.SectionData.Section; if (Section2->FileObject == NULL) { - DPRINT1("No file object\n"); MmUnlockAddressSpace(AddressSpace); return STATUS_CONFLICTING_ADDRESSES; } - /* These dbgprints should allow me to see what should w ecompare in ROS's section implementation once the winetests are run... for now lie and say they're not equal. */ - DPRINT1("FO1/2: %p %p\n", Section1->FileObject, Section2->FileObject); - DPRINT1("SOP: %p %p\n", - Section1->FileObject->SectionObjectPointer, - Section2->FileObject->SectionObjectPointer); - DPRINT1("SCM: %p %p\n", - Section1->FileObject->SectionObjectPointer->SharedCacheMap, - Section2->FileObject->SectionObjectPointer->SharedCacheMap); - DPRINT1("ISO: %p %p\n", - Section1->FileObject->SectionObjectPointer->ImageSectionObject, - Section2->FileObject->SectionObjectPointer->ImageSectionObject); - DPRINT1("SISO: %p %p\n", Section1->ImageSection, Section2->ImageSection); + /* The shared cache map seems to be the same if both of these are equal */ + if (Section1->FileObject->SectionObjectPointer->SharedCacheMap == + Section2->FileObject->SectionObjectPointer->SharedCacheMap) + { + MmUnlockAddressSpace(AddressSpace); + return STATUS_SUCCESS; + } /* Unlock address space */ MmUnlockAddressSpace(AddressSpace);

12 years, 10 months

[janderwald] 55451: [DISK] - Add more structures to ntdddisk.h - Fix more compile errors in disk_new - disk_new builds but fails to link as classpnp import lib is broken

by janderwald＠svn.reactos.org

Author: janderwald Date: Mon Feb 6 08:15:21 2012 New Revision: 55451 URL: http://svn.reactos.org/svn/reactos?rev=55451&view=rev Log: [DISK] - Add more structures to ntdddisk.h - Fix more compile errors in disk_new - disk_new builds but fails to link as classpnp import lib is broken Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.c branches/usb-bringup-trunk/drivers/storage/class/disk_new/diskwmi.c branches/usb-bringup-trunk/drivers/storage/class/disk_new/enum.c branches/usb-bringup-trunk/drivers/storage/classpnp/CMakeLists.txt branches/usb-bringup-trunk/include/psdk/ntdddisk.h Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.c [iso-8859-1] Mon Feb 6 08:15:21 2012 @@ -1201,6 +1201,7 @@ break; } +#if(_WIN32_WINNT >= 0x0500) case IOCTL_DISK_GET_WRITE_CACHE_STATE: { PDISK_WRITE_CACHE_STATE writeCacheState = (PDISK_WRITE_CACHE_STATE)Irp->AssociatedIrp.SystemBuffer; @@ -1248,6 +1249,7 @@ status = STATUS_SUCCESS; break; } +#endif case SMART_GET_VERSION: { Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/diskwmi.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/class/disk_new/diskwmi.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/class/disk_new/diskwmi.c [iso-8859-1] Mon Feb 6 08:15:21 2012 @@ -281,7 +281,7 @@ // // Enable SMART // -_inline NTSTATUS +__inline NTSTATUS DiskEnableSmart( PFUNCTIONAL_DEVICE_EXTENSION FdoExtension ) @@ -302,7 +302,7 @@ // // Disable SMART // -_inline NTSTATUS +__inline NTSTATUS DiskDisableSmart( PFUNCTIONAL_DEVICE_EXTENSION FdoExtension ) @@ -322,7 +322,7 @@ // // Enable Attribute Autosave // -_inline NTSTATUS +__inline NTSTATUS DiskEnableSmartAttributeAutosave( PFUNCTIONAL_DEVICE_EXTENSION FdoExtension ) @@ -342,7 +342,7 @@ // // Disable Attribute Autosave // -_inline NTSTATUS +__inline NTSTATUS DiskDisableSmartAttributeAutosave( PFUNCTIONAL_DEVICE_EXTENSION FdoExtension ) @@ -362,7 +362,7 @@ // // Initialize execution of SMART online diagnostics // -_inline NTSTATUS +__inline NTSTATUS DiskExecuteSmartDiagnostics( PFUNCTIONAL_DEVICE_EXTENSION FdoExtension, UCHAR Subcommand Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/enum.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/class/disk_new/enum.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/class/disk_new/enum.c [iso-8859-1] Mon Feb 6 08:15:21 2012 @@ -646,7 +646,7 @@ ULONG partitionOrdinal; ULONG newPartitionNumber; - PPARTITION_INFORMATION_EX partitionEntry; + PPARTITION_INFORMATION_EX partitionEntry = NULL; PDISK_DATA pdoData; PARTITION_STYLE partitionStyle; Modified: branches/usb-bringup-trunk/drivers/storage/classpnp/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/classpnp/CMakeLists.txt [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/classpnp/CMakeLists.txt [iso-8859-1] Mon Feb 6 08:15:21 2012 @@ -42,4 +42,5 @@ target_link_libraries(classpnp ${PSEH_LIB} libcntpr) set_module_type(classpnp kernelmodedriver) add_importlibs(classpnp hal ntoskrnl) +add_importlib_target(class.spec) add_cd_file(TARGET classpnp DESTINATION reactos/system32/drivers NO_CAB FOR all) Modified: branches/usb-bringup-trunk/include/psdk/ntdddisk.h URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/include/psdk/… ============================================================================== --- branches/usb-bringup-trunk/include/psdk/ntdddisk.h [iso-8859-1] (original) +++ branches/usb-bringup-trunk/include/psdk/ntdddisk.h [iso-8859-1] Mon Feb 6 08:15:21 2012 @@ -581,6 +581,29 @@ #if(_WIN32_WINNT >= 0x0500) + +typedef struct _CREATE_DISK_GPT +{ + GUID DiskId; + ULONG MaxPartitionCount; +} CREATE_DISK_GPT, *PCREATE_DISK_GPT; + +typedef struct _CREATE_DISK_MBR +{ + ULONG Signature; +} CREATE_DISK_MBR, *PCREATE_DISK_MBR; + + +typedef struct _CREATE_DISK +{ + PARTITION_STYLE PartitionStyle; + union { + CREATE_DISK_MBR Mbr; + CREATE_DISK_GPT Gpt; + }; +} CREATE_DISK, *PCREATE_DISK; + + typedef enum { EqualPriority, KeepPrefetchedData,

12 years, 10 months

[cgutman] 55450: [USBEHCI] - Implement ResetController and call it before setting up the controller - Don't disable 64-bit addressing until the BIOS gives up control of the controller - Comment out...

by cgutman＠svn.reactos.org

Author: cgutman Date: Mon Feb 6 07:41:20 2012 New Revision: 55450 URL: http://svn.reactos.org/svn/reactos?rev=55450&view=rev Log: [USBEHCI] - Implement ResetController and call it before setting up the controller - Don't disable 64-bit addressing until the BIOS gives up control of the controller - Comment out manually clearing the SMI interrupt and the BIOS owned flag by request of janderwald - EHCI no longer hangs for me on real hardware during controller initialization Modified: branches/usb-bringup-trunk/drivers/usb/usbehci_new/hardware.cpp Modified: branches/usb-bringup-trunk/drivers/usb/usbehci_new/hardware.cpp URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/usb/u… ============================================================================== --- branches/usb-bringup-trunk/drivers/usb/usbehci_new/hardware.cpp [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/usb/usbehci_new/hardware.cpp [iso-8859-1] Mon Feb 6 07:41:20 2012 @@ -567,17 +567,6 @@ LARGE_INTEGER Timeout; // - // check caps - // - if (m_Capabilities.HCCParams.CurAddrBits) - { - // - // disable 64-bit addressing - // - EHCI_WRITE_REGISTER_ULONG(EHCI_CTRLDSSEGMENT, 0x0); - } - - // // are extended caps supported // ExtendedCapsSupport = (m_Capabilities.HCCParamsLong >> EHCI_ECP_SHIFT) & EHCI_ECP_MASK; @@ -648,7 +637,7 @@ // DPRINT1("[EHCI] acquired ownership\n"); } - +#if 0 // // explictly clear the bios owned flag 2.1.7 // @@ -660,15 +649,11 @@ // Caps = 4; m_BusInterface.SetBusData(m_BusInterface.Context, PCI_WHICHSPACE_CONFIG, &Caps, ExtendedCapsSupport+4, sizeof(ULONG)); - - +#endif } } } - - -#if 1 // // Stop the controller if its running // @@ -678,7 +663,22 @@ DPRINT1("Stopping Controller %x\n", UsbSts); StopController(); } -#endif + + // + // Reset the controller + // + ResetController(); + + // + // check caps + // + if (m_Capabilities.HCCParams.CurAddrBits) + { + // + // disable 64-bit addressing + // + EHCI_WRITE_REGISTER_ULONG(EHCI_CTRLDSSEGMENT, 0x0); + } // // Enable Interrupts and start execution @@ -692,7 +692,6 @@ DPRINT1("Interrupt Mask %x\n", Status); ASSERT((Status & Mask) == Mask); - // // Assign the SyncList Register @@ -827,8 +826,28 @@ NTSTATUS CUSBHardwareDevice::ResetController(void) { - UNIMPLEMENTED - return STATUS_NOT_IMPLEMENTED; + EHCI_USBCMD_CONTENT UsbCmd; + ULONG FailSafe; + + GetCommandRegister(&UsbCmd); + UsbCmd.HCReset = TRUE; + SetCommandRegister(&UsbCmd); + + for (FailSafe = 100; FailSafe > 1; FailSafe--) + { + KeStallExecutionProcessor(100); + GetCommandRegister(&UsbCmd); + if (!UsbCmd.HCReset) + break; + } + + if (UsbCmd.HCReset) + { + DPRINT1("EHCI ERROR: Controller is not responding to reset request!\n"); + return STATUS_UNSUCCESSFUL; + } + + return STATUS_SUCCESS; } NTSTATUS

12 years, 10 months

[janderwald] 55449: [DDK] - More header fixes for disk_new

by janderwald＠svn.reactos.org

Author: janderwald Date: Mon Feb 6 07:35:22 2012 New Revision: 55449 URL: http://svn.reactos.org/svn/reactos?rev=55449&view=rev Log: [DDK] - More header fixes for disk_new Modified: branches/usb-bringup-trunk/include/psdk/ntdddisk.h Modified: branches/usb-bringup-trunk/include/psdk/ntdddisk.h URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/include/psdk/… ============================================================================== --- branches/usb-bringup-trunk/include/psdk/ntdddisk.h [iso-8859-1] (original) +++ branches/usb-bringup-trunk/include/psdk/ntdddisk.h [iso-8859-1] Mon Feb 6 07:35:22 2012 @@ -178,8 +178,17 @@ #define SMART_SEND_DRIVE_COMMAND \ CTL_CODE(IOCTL_DISK_BASE, 0x0021, METHOD_BUFFERED, FILE_READ_ACCESS | FILE_WRITE_ACCESS) -#define IOCTL_DISK_UPDATE_PROPERTIES \ - CTL_CODE(IOCTL_DISK_BASE, 0x50, METHOD_BUFFERED, FILE_ANY_ACCESS) +#if(_WIN32_WINNT >= 0x0500) +#define IOCTL_DISK_UPDATE_DRIVE_SIZE CTL_CODE(IOCTL_DISK_BASE, 0x0032, METHOD_BUFFERED, FILE_READ_ACCESS | FILE_WRITE_ACCESS) +#define IOCTL_DISK_GROW_PARTITION CTL_CODE(IOCTL_DISK_BASE, 0x0034, METHOD_BUFFERED, FILE_READ_ACCESS | FILE_WRITE_ACCESS) +#define IOCTL_DISK_GET_CACHE_INFORMATION CTL_CODE(IOCTL_DISK_BASE, 0x0035, METHOD_BUFFERED, FILE_READ_ACCESS) +#define IOCTL_DISK_SET_CACHE_INFORMATION CTL_CODE(IOCTL_DISK_BASE, 0x0036, METHOD_BUFFERED, FILE_READ_ACCESS | FILE_WRITE_ACCESS) +#define IOCTL_DISK_GET_WRITE_CACHE_STATE CTL_CODE(IOCTL_DISK_BASE, 0x0037, METHOD_BUFFERED, FILE_READ_ACCESS) +#define IOCTL_DISK_DELETE_DRIVE_LAYOUT CTL_CODE(IOCTL_DISK_BASE, 0x0040, METHOD_BUFFERED, FILE_READ_ACCESS | FILE_WRITE_ACCESS) +#define IOCTL_DISK_UPDATE_PROPERTIES CTL_CODE(IOCTL_DISK_BASE, 0x0050, METHOD_BUFFERED, FILE_ANY_ACCESS) +#define IOCTL_DISK_FORMAT_DRIVE CTL_CODE(IOCTL_DISK_BASE, 0x00f3, METHOD_BUFFERED, FILE_READ_ACCESS | FILE_WRITE_ACCESS) +#define IOCTL_DISK_SENSE_DEVICE CTL_CODE(IOCTL_DISK_BASE, 0x00f8, METHOD_BUFFERED, FILE_ANY_ACCESS) +#endif #define PARTITION_ENTRY_UNUSED 0x00 #define PARTITION_FAT_12 0x01 @@ -465,38 +474,6 @@ ULONG Length; } VERIFY_INFORMATION, *PVERIFY_INFORMATION; -typedef enum { - EqualPriority, - KeepPrefetchedData, - KeepReadData -} DISK_CACHE_RETENTION_PRIORITY; - -typedef struct _DISK_CACHE_INFORMATION { - BOOLEAN ParametersSavable; - BOOLEAN ReadCacheEnabled; - BOOLEAN WriteCacheEnabled; - DISK_CACHE_RETENTION_PRIORITY ReadRetentionPriority; - DISK_CACHE_RETENTION_PRIORITY WriteRetentionPriority; - USHORT DisablePrefetchTransferLength; - BOOLEAN PrefetchScalar; - _ANONYMOUS_UNION union { - struct { - USHORT Minimum; - USHORT Maximum; - USHORT MaximumBlocks; - } ScalarPrefetch; - struct { - USHORT Minimum; - USHORT Maximum; - } BlockPrefetch; - } DUMMYUNIONNAME; -} DISK_CACHE_INFORMATION, *PDISK_CACHE_INFORMATION; - -typedef struct _DISK_GROW_PARTITION { - ULONG PartitionNumber; - LARGE_INTEGER BytesToGrow; -} DISK_GROW_PARTITION, *PDISK_GROW_PARTITION; - /* GETVERSIONINPARAMS.fCapabilities constants */ #define CAP_ATA_ID_CMD 1 #define CAP_ATAPI_ID_CMD 2 @@ -602,6 +579,52 @@ } MAPPED_ADDRESS, *PMAPPED_ADDRESS; + +#if(_WIN32_WINNT >= 0x0500) +typedef enum { + EqualPriority, + KeepPrefetchedData, + KeepReadData +} DISK_CACHE_RETENTION_PRIORITY; + +typedef enum _DISK_WRITE_CACHE_STATE { + DiskWriteCacheNormal, + DiskWriteCacheForceDisable, + DiskWriteCacheDisableNotSupported +} DISK_WRITE_CACHE_STATE, *PDISK_WRITE_CACHE_STATE; + + +typedef struct _DISK_CACHE_INFORMATION +{ + BOOLEAN ParametersSavable; + BOOLEAN ReadCacheEnabled; + BOOLEAN WriteCacheEnabled; + DISK_CACHE_RETENTION_PRIORITY ReadRetentionPriority; + DISK_CACHE_RETENTION_PRIORITY WriteRetentionPriority; + USHORT DisablePrefetchTransferLength; + BOOLEAN PrefetchScalar; + union { + struct { + USHORT Minimum; + USHORT Maximum; + USHORT MaximumBlocks; + } ScalarPrefetch; + + struct { + USHORT Minimum; + USHORT Maximum; + } BlockPrefetch; + }; + +} DISK_CACHE_INFORMATION, *PDISK_CACHE_INFORMATION; + +typedef struct _DISK_GROW_PARTITION { + ULONG PartitionNumber; + LARGE_INTEGER BytesToGrow; +} DISK_GROW_PARTITION, *PDISK_GROW_PARTITION; +#endif + + #ifdef __cplusplus } #endif

12 years, 10 months

[ion] 55448: [NTOSKRNL]: Partly implement NtAreMappedFilesTheSame... [NTDLL]: Fix (although I'm not sure quite why) LdrpCheckForLoadedDll which was crashing now that I've fixed PAGE_EXECUTE section...

by ion＠svn.reactos.org

Author: ion Date: Mon Feb 6 06:53:28 2012 New Revision: 55448 URL: http://svn.reactos.org/svn/reactos?rev=55448&view=rev Log: [NTOSKRNL]: Partly implement NtAreMappedFilesTheSame... [NTDLL]: Fix (although I'm not sure quite why) LdrpCheckForLoadedDll which was crashing now that I've fixed PAGE_EXECUTE sections with only FILE_EXECUTE handles. Ironically, this meant that LdrpCheckForLoadedDll never worked until my previous fix some revisions ago, it always returned FALSE. This should fix KVM/QEMU crashes... Modified: trunk/reactos/dll/ntdll/ldr/ldrutils.c trunk/reactos/ntoskrnl/mm/ARM3/section.c Modified: trunk/reactos/dll/ntdll/ldr/ldrutils.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/ntdll/ldr/ldrutils.c?r… ============================================================================== --- trunk/reactos/dll/ntdll/ldr/ldrutils.c [iso-8859-1] (original) +++ trunk/reactos/dll/ntdll/ldr/ldrutils.c [iso-8859-1] Mon Feb 6 06:53:28 2012 @@ -2204,12 +2204,7 @@ { /* Headers match too! Finally ask the kernel to compare mapped files */ Status = ZwAreMappedFilesTheSame(CurEntry->DllBase, ViewBase); - if (!NT_SUCCESS(Status)) - { - /* Almost identical, but not quite, keep trying */ - _SEH2_YIELD(continue;) - } - else + if (NT_SUCCESS(Status)) { /* This is our entry!, unmap and return success */ *LdrEntry = CurEntry; Modified: trunk/reactos/ntoskrnl/mm/ARM3/section.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/section.c… ============================================================================== --- trunk/reactos/ntoskrnl/mm/ARM3/section.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/ARM3/section.c [iso-8859-1] Mon Feb 6 06:53:28 2012 @@ -1383,8 +1383,88 @@ NtAreMappedFilesTheSame(IN PVOID File1MappedAsAnImage, IN PVOID File2MappedAsFile) { - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; + PVOID AddressSpace; + PMEMORY_AREA MemoryArea1, MemoryArea2; + PROS_SECTION_OBJECT Section1, Section2; + + DPRINT1("Is image: %p the same as file: %p?\n", File1MappedAsAnImage, File2MappedAsFile); + + /* Lock address space */ + AddressSpace = MmGetCurrentAddressSpace(); + MmLockAddressSpace(AddressSpace); + + /* Locate the memory area for the process by address */ + MemoryArea1 = MmLocateMemoryAreaByAddress(AddressSpace, File1MappedAsAnImage); + if (!MemoryArea1) + { + /* Fail, the address does not exist */ + DPRINT1("Invalid address\n"); + MmUnlockAddressSpace(AddressSpace); + return STATUS_INVALID_ADDRESS; + } + + /* Check if it's a section view (RosMm section) or ARM3 section */ + if (MemoryArea1->Type != MEMORY_AREA_SECTION_VIEW) + { + /* Fail, the address is not a section */ + DPRINT1("Invalid address (not a section)\n"); + MmUnlockAddressSpace(AddressSpace); + return STATUS_CONFLICTING_ADDRESSES; + } + + /* Get the section pointer to the SECTION_OBJECT */ + Section1 = MemoryArea1->Data.SectionData.Section; + if (Section1->FileObject == NULL) + { + DPRINT1("No file object\n"); + MmUnlockAddressSpace(AddressSpace); + return STATUS_CONFLICTING_ADDRESSES; + } + + /* Locate the memory area for the process by address */ + MemoryArea2 = MmLocateMemoryAreaByAddress(AddressSpace, File2MappedAsFile); + if (!MemoryArea2) + { + /* Fail, the address does not exist */ + DPRINT1("Invalid address\n"); + MmUnlockAddressSpace(AddressSpace); + return STATUS_INVALID_ADDRESS; + } + + /* Check if it's a section view (RosMm section) or ARM3 section */ + if (MemoryArea2->Type != MEMORY_AREA_SECTION_VIEW) + { + /* Fail, the address is not a section */ + DPRINT1("Invalid address (not a section)\n"); + MmUnlockAddressSpace(AddressSpace); + return STATUS_CONFLICTING_ADDRESSES; + } + + /* Get the section pointer to the SECTION_OBJECT */ + Section2 = MemoryArea2->Data.SectionData.Section; + if (Section2->FileObject == NULL) + { + DPRINT1("No file object\n"); + MmUnlockAddressSpace(AddressSpace); + return STATUS_CONFLICTING_ADDRESSES; + } + + /* These dbgprints should allow me to see what should w ecompare in ROS's section implementation once the winetests are run... for now lie and say they're not equal. */ + DPRINT1("FO1/2: %p %p\n", Section1->FileObject, Section2->FileObject); + DPRINT1("SOP: %p %p\n", + Section1->FileObject->SectionObjectPointer, + Section2->FileObject->SectionObjectPointer); + DPRINT1("SCM: %p %p\n", + Section1->FileObject->SectionObjectPointer->SharedCacheMap, + Section2->FileObject->SectionObjectPointer->SharedCacheMap); + DPRINT1("ISO: %p %p\n", + Section1->FileObject->SectionObjectPointer->ImageSectionObject, + Section2->FileObject->SectionObjectPointer->ImageSectionObject); + DPRINT1("SISO: %p %p\n", Section1->ImageSection, Section2->ImageSection); + + /* Unlock address space */ + MmUnlockAddressSpace(AddressSpace); + return STATUS_NOT_SAME_DEVICE; } /*

12 years, 10 months

[janderwald] 55447: [DISK_NEW] - Fix compilation of new disk driver - Patch by Alex Ionescu

by janderwald＠svn.reactos.org

Author: janderwald Date: Mon Feb 6 06:50:19 2012 New Revision: 55447 URL: http://svn.reactos.org/svn/reactos?rev=55447&view=rev Log: [DISK_NEW] - Fix compilation of new disk driver - Patch by Alex Ionescu Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/CMakeLists.txt branches/usb-bringup-trunk/drivers/storage/class/disk_new/data.c branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.c branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.h branches/usb-bringup-trunk/drivers/storage/class/disk_new/diskwmi.c branches/usb-bringup-trunk/drivers/storage/class/disk_new/geometry.c branches/usb-bringup-trunk/drivers/storage/class/disk_new/pnp.c Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/class/disk_new/CMakeLists.txt [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/class/disk_new/CMakeLists.txt [iso-8859-1] Mon Feb 6 06:50:19 2012 @@ -1,13 +1,14 @@ -remove_definitions(-D_WIN32_WINNT=0x502) -add_definitions(-D_WIN32_WINNT=0x600) - include_directories(..) add_library(disk SHARED data.c disk.c diskwmi.c enum.c geometry.c part.c pnp.c disk.rc) +if(ARCH MATCHES i386) + add_target_compile_flags(disk "-mrtd -fno-builtin -Wno-unused-variable -Wno-pointer-sign") +endif() + set_module_type(disk kernelmodedriver) add_importlibs(disk - class2 + classpnp scsiport ntoskrnl hal) Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/data.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/class/disk_new/data.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/class/disk_new/data.c [iso-8859-1] Mon Feb 6 06:50:19 2012 @@ -60,15 +60,20 @@ { NULL , NULL , NULL, 0x0 } }; +// +// ======== ROS DIFF ======== +// Added MediaTypes in their own brace nesting level +// ======== ROS DIFF ======== +// DISK_MEDIA_TYPES_LIST const DiskMediaTypes[] = { - { "COMPAQ" , "PD-1 LF-1094" , NULL, 1, 1, PC_5_RW , 0 , 0 , 0 }, - { "HP" , NULL , NULL, 2, 2, MO_5_WO , MO_5_RW, 0 , 0 }, - { "iomega" , "jaz" , NULL, 1, 1, IOMEGA_JAZ , 0 , 0 , 0 }, - { "IOMEGA" , "ZIP" , NULL, 1, 1, IOMEGA_ZIP , 0 , 0 , 0 }, - { "PINNACLE", "Apex 4.6GB" , NULL, 3, 2, PINNACLE_APEX_5_RW, MO_5_RW, MO_5_WO, 0 }, - { "SONY" , "SMO-F541" , NULL, 2, 2, MO_5_WO , MO_5_RW, 0 , 0 }, - { "SONY" , "SMO-F551" , NULL, 2, 2, MO_5_WO , MO_5_RW, 0 , 0 }, - { NULL , NULL , NULL, 0, 0, 0 , 0 , 0 , 0 } + { "COMPAQ" , "PD-1 LF-1094" , NULL, 1, 1, {PC_5_RW , 0 , 0 , 0 }}, + { "HP" , NULL , NULL, 2, 2, {MO_5_WO , MO_5_RW, 0 , 0 }}, + { "iomega" , "jaz" , NULL, 1, 1, {IOMEGA_JAZ , 0 , 0 , 0 }}, + { "IOMEGA" , "ZIP" , NULL, 1, 1, {IOMEGA_ZIP , 0 , 0 , 0 }}, + { "PINNACLE", "Apex 4.6GB" , NULL, 3, 2, {PINNACLE_APEX_5_RW, MO_5_RW, MO_5_WO, 0 }}, + { "SONY" , "SMO-F541" , NULL, 2, 2, {MO_5_WO , MO_5_RW, 0 , 0 }}, + { "SONY" , "SMO-F551" , NULL, 2, 2, {MO_5_WO , MO_5_RW, 0 , 0 }}, + { NULL , NULL , NULL, 0, 0, {0 , 0 , 0 , 0 }} }; #ifdef ALLOC_DATA_PRAGMA Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.c [iso-8859-1] Mon Feb 6 06:50:19 2012 @@ -173,6 +173,7 @@ NTSTATUS +NTAPI DriverEntry( IN PDRIVER_OBJECT DriverObject, IN PUNICODE_STRING RegistryPath @@ -1304,7 +1305,7 @@ // buffer = (PUCHAR)srbControl; - (ULONG_PTR)buffer += srbControl->HeaderLength; + buffer += srbControl->HeaderLength; // // Ensure correct target is set in the cmd parameters. @@ -1341,7 +1342,7 @@ if (NT_SUCCESS(status)) { buffer = (PUCHAR)srbControl; - (ULONG_PTR)buffer += srbControl->HeaderLength; + buffer += srbControl->HeaderLength; RtlMoveMemory (Irp->AssociatedIrp.SystemBuffer, buffer, sizeof(GETVERSIONINPARAMS)); @@ -1444,7 +1445,7 @@ // buffer = (PUCHAR)srbControl; - (ULONG_PTR)buffer += srbControl->HeaderLength; + buffer += srbControl->HeaderLength; // // Ensure correct target is set in the cmd parameters. @@ -1494,7 +1495,7 @@ // buffer = (PUCHAR)srbControl; - (ULONG_PTR)buffer += srbControl->HeaderLength; + buffer += srbControl->HeaderLength; if (NT_SUCCESS(status)) { @@ -1640,7 +1641,7 @@ // buffer = (PUCHAR)srbControl; - (ULONG_PTR)buffer += srbControl->HeaderLength; + buffer += srbControl->HeaderLength; // // Ensure correct target is set in the cmd parameters. @@ -1690,7 +1691,7 @@ // buffer = (PUCHAR)srbControl; - (ULONG_PTR)buffer += srbControl->HeaderLength; + buffer += srbControl->HeaderLength; // // Update the return buffer size based on the sub-command. @@ -1828,7 +1829,7 @@ // blockDescriptor = (PMODE_PARAMETER_BLOCK)modeData; - (ULONG_PTR)blockDescriptor += sizeof(MODE_PARAMETER_HEADER); + blockDescriptor = (PMODE_PARAMETER_BLOCK)((ULONG_PTR)blockDescriptor + sizeof(MODE_PARAMETER_HEADER)); // // Do some validation. @@ -2165,7 +2166,7 @@ IoMarkIrpPending(Irp); IoQueueWorkItem(Context->WorkItem, - DiskIoctlVerify, + (PIO_WORKITEM_ROUTINE)DiskIoctlVerify, DelayedWorkQueue, Context); @@ -3521,7 +3522,7 @@ ((PMODE_PARAMETER_HEADER)buffer)->BlockDescriptorLength = sizeof(MODE_PARAMETER_BLOCK); - (PULONG)blockDescriptor = (buffer + 1); + blockDescriptor = (PMODE_PARAMETER_BLOCK)(buffer + 1); // // Set size @@ -3888,7 +3889,7 @@ if (workItem) { IoQueueWorkItem(workItem, - DisableWriteCache, + (PIO_WORKITEM_ROUTINE)DisableWriteCache, CriticalWorkQueue, workItem); } Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.h URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.h [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/class/disk_new/disk.h [iso-8859-1] Mon Feb 6 06:50:19 2012 @@ -19,6 +19,9 @@ Revision History: --*/ +#undef _WIN32_WINNT +#define _WIN32_WINNT 0x0501 +#define NTDDI_VERSION NTDDI_WINXP #include "ntddk.h" #include "scsi.h" Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/diskwmi.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/class/disk_new/diskwmi.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/class/disk_new/diskwmi.c [iso-8859-1] Mon Feb 6 06:50:19 2012 @@ -546,7 +546,7 @@ // buffer = (PUCHAR)SrbControl; - (ULONG_PTR)buffer += sizeof(SRB_IO_CONTROL); + buffer += sizeof(SRB_IO_CONTROL); cmdInParameters = (PSENDCMDINPARAMS)buffer; cmdOutParameters = (PSENDCMDOUTPARAMS)buffer; @@ -641,11 +641,16 @@ break; } + default: + controlCode = 0; + break; + } } else if (Command == ID_CMD) { controlCode = IOCTL_SCSI_MINIPORT_IDENTIFY; lengthNeeded = IDENTIFY_BUFFER_SIZE + sizeof(SENDCMDOUTPARAMS) -1; } else { + controlCode = 0; ASSERT(FALSE); } @@ -1393,8 +1398,10 @@ retry = TRUE; } - if (retry && ((ULONG)(ULONG_PTR)irpStack->Parameters.Others.Argument4)--) - { + if (retry && irpStack->Parameters.Others.Argument4) + { + irpStack->Parameters.Others.Argument4 = + (PVOID)((ULONG_PTR)irpStack->Parameters.Others.Argument4 - 1); // // Retry request. @@ -2798,6 +2805,7 @@ // case ReadLogSectors: { + sizeNeeded = 0; if (diskData->FailurePredictionCapability == FailurePredictionSmart) { @@ -2853,6 +2861,7 @@ // ); case WriteLogSectors: { + sizeNeeded = 0; if (diskData->FailurePredictionCapability == FailurePredictionSmart) { @@ -2911,6 +2920,7 @@ // uint32 ReturnCode); case ExecuteSelfTest: { + sizeNeeded = 0; if (diskData->FailurePredictionCapability == FailurePredictionSmart) { Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/geometry.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/class/disk_new/geometry.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/class/disk_new/geometry.c [iso-8859-1] Mon Feb 6 06:50:19 2012 @@ -1002,7 +1002,7 @@ KeInitializeEvent(&event, SynchronizationEvent, FALSE); IoSetCompletionRoutine(irp, - ClassSignalCompletion, + (PIO_COMPLETION_ROUTINE)ClassSignalCompletion, &event, TRUE, TRUE, @@ -1348,7 +1348,7 @@ { ULONG i; BOOLEAN found; - PDISK_DETECT_INFO diskInfo; + PDISK_DETECT_INFO diskInfo = NULL; PDISK_DATA diskData = FdoExtension->CommonExtension.DriverData; PAGED_CODE (); Modified: branches/usb-bringup-trunk/drivers/storage/class/disk_new/pnp.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/stora… ============================================================================== --- branches/usb-bringup-trunk/drivers/storage/class/disk_new/pnp.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/storage/class/disk_new/pnp.c [iso-8859-1] Mon Feb 6 06:50:19 2012 @@ -424,7 +424,7 @@ HalExamineMBR(fdoExtension->CommonExtension.DeviceObject, fdoExtension->DiskGeometry.BytesPerSector, (ULONG)0x54, - &dmSkew); + (PVOID*)&dmSkew); if (dmSkew) {

12 years, 10 months

[tkreuzer] 55446: [NTOSKRNL] In PeFmtCreateSection don't use the ImageBase field of the ImageOptionalHeader directly, since it's different between PE32 and PE64. Use the ImageSectionObject->ImageBa...

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Mon Feb 6 00:29:50 2012 New Revision: 55446 URL: http://svn.reactos.org/svn/reactos?rev=55446&view=rev Log: [NTOSKRNL] In PeFmtCreateSection don't use the ImageBase field of the ImageOptionalHeader directly, since it's different between PE32 and PE64. Use the ImageSectionObject->ImageBase field instead, which already contains the correct value. Modified: trunk/reactos/ntoskrnl/mm/section.c Modified: trunk/reactos/ntoskrnl/mm/section.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/section.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/mm/section.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/section.c [iso-8859-1] Mon Feb 6 00:29:50 2012 @@ -402,7 +402,7 @@ break; } - /* PE32+ */ + /* PE64 */ case IMAGE_NT_OPTIONAL_HDR64_MAGIC: { const IMAGE_OPTIONAL_HEADER64 * pioh64OptHeader; @@ -463,7 +463,7 @@ if(RTL_CONTAINS_FIELD(piohOptHeader, cbOptHeaderSize, AddressOfEntryPoint)) { - ImageSectionObject->EntryPoint = piohOptHeader->ImageBase + + ImageSectionObject->EntryPoint = ImageSectionObject->ImageBase + piohOptHeader->AddressOfEntryPoint; }

12 years, 10 months

← Newer
1
...
46
47
48
49
50
51
52
...
58
Older →

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 February 2012