Ros-diffs

ros-diffs@reactos.org

81390 discussions

[cgutman] 56049: [NEWCC] - Hold the PFN lock when referencing pages
by cgutman＠svn.reactos.org 05 Mar '12

05 Mar '12

Author: cgutman Date: Mon Mar 5 19:29:39 2012 New Revision: 56049 URL: http://svn.reactos.org/svn/reactos?rev=56049&view=rev Log: [NEWCC] - Hold the PFN lock when referencing pages Modified: trunk/reactos/ntoskrnl/cache/section/data.c trunk/reactos/ntoskrnl/cache/section/fault.c Modified: trunk/reactos/ntoskrnl/cache/section/data.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/cache/section/dat… ============================================================================== --- trunk/reactos/ntoskrnl/cache/section/data.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/cache/section/data.c [iso-8859-1] Mon Mar 5 19:29:39 2012 @@ -104,8 +104,12 @@ PMEMORY_AREA MemoryArea; PMM_SECTION_SEGMENT Segment; LARGE_INTEGER FileOffset = *FileOffsetPtr, End, FirstMapped; + KIRQL OldIrql; + DPRINT("MiZeroFillSection(Address %x,Offset %x,Length %x)\n", Address, FileOffset.LowPart, Length); + AddressSpace = MmGetKernelAddressSpace(); + MmLockAddressSpace(AddressSpace); MemoryArea = MmLocateMemoryAreaByAddress(AddressSpace, Address); MmUnlockAddressSpace(AddressSpace); @@ -140,7 +144,10 @@ MmSetPageEntrySectionSegment(Segment, &FileOffset, MAKE_PFN_SSE(Page)); Address = ((PCHAR)MemoryArea->StartingAddress) + FileOffset.QuadPart - FirstMapped.QuadPart; + OldIrql = KeAcquireQueuedSpinLock(LockQueuePfnLock); MmReferencePage(Page); + KeReleaseQueuedSpinLock(LockQueuePfnLock, OldIrql); + MmCreateVirtualMapping(NULL, Address, PAGE_READWRITE, &Page, 1); MmInsertRmap(Page, NULL, Address); } @@ -175,6 +182,7 @@ LARGE_INTEGER FileOffset; PFN_NUMBER Page; PPFN_NUMBER Pages; + KIRQL OldIrql; DPRINT("MiFlushMappedSection(%x,%08x,%x,%d,%s:%d)\n", BaseAddress, BaseOffset->LowPart, FileSize, WriteData, File, Line); @@ -222,7 +230,9 @@ (MmIsDirtyPageRmap(Page) || IS_DIRTY_SSE(Entry)) && FileOffset.QuadPart < FileSize->QuadPart) { + OldIrql = KeAcquireQueuedSpinLock(LockQueuePfnLock); MmReferencePage(Page); + KeReleaseQueuedSpinLock(LockQueuePfnLock, OldIrql); Pages[(PageAddress - BeginningAddress) >> PAGE_SHIFT] = Entry; } else Modified: trunk/reactos/ntoskrnl/cache/section/fault.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/cache/section/fau… ============================================================================== --- trunk/reactos/ntoskrnl/cache/section/fault.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/cache/section/fault.c [iso-8859-1] Mon Mar 5 19:29:39 2012 @@ -73,6 +73,7 @@ ULONG Entry; ULONG Attributes; PEPROCESS Process = MmGetAddressSpaceOwner(AddressSpace); + KIRQL OldIrql; DPRINT("Not Present: %p %p (%p-%p)\n", AddressSpace, Address, MemoryArea->StartingAddress, MemoryArea->EndingAddress); @@ -175,7 +176,9 @@ KeBugCheck(CACHE_MANAGER); } + OldIrql = KeAcquireQueuedSpinLock(LockQueuePfnLock); MmReferencePage(Page); + KeReleaseQueuedSpinLock(LockQueuePfnLock, OldIrql); Status = MmCreateVirtualMapping(Process, Address, Attributes, &Page, 1); if (NT_SUCCESS(Status))

1 0

[jgardou] 56048: [CABINET] - Really use random temporary file name
by jgardou＠svn.reactos.org 05 Mar '12

05 Mar '12

Author: jgardou Date: Mon Mar 5 19:18:19 2012 New Revision: 56048 URL: http://svn.reactos.org/svn/reactos?rev=56048&view=rev Log: [CABINET] - Really use random temporary file name Modified: trunk/reactos/tools/cabman/cabinet.cxx Modified: trunk/reactos/tools/cabman/cabinet.cxx URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/cabman/cabinet.cxx?r… ============================================================================== --- trunk/reactos/tools/cabman/cabinet.cxx [iso-8859-1] (original) +++ trunk/reactos/tools/cabman/cabinet.cxx [iso-8859-1] Mon Mar 5 19:18:19 2012 @@ -115,7 +115,7 @@ } -ULONG CCFDATAStorage::Create(const char* FileName) +ULONG CCFDATAStorage::Create() /* * FUNCTION: Creates the file * ARGUMENTS: @@ -124,13 +124,16 @@ * Status of operation */ { +#if defined(_WIN32) + char tmpPath[MAX_PATH]; +#endif ASSERT(!FileCreated); #if defined(_WIN32) - if (GetTempPath(MAX_PATH, FullName) == 0) + if (GetTempPath(MAX_PATH, tmpPath) == 0) return CAB_STATUS_CANNOT_CREATE; - - strcat(FullName, FileName); + if(GetTempFileName(tmpPath, "cab", 0, FullName) == 0) + return CAB_STATUS_CANNOT_CREATE; /* Create file, overwrite if it already exists */ FileHandle = CreateFile(FullName, // Create this file @@ -1529,7 +1532,7 @@ return CAB_STATUS_NOMEMORY; } - Status = ScratchFile->Create("~CAB.tmp"); + Status = ScratchFile->Create(); CreateNewFolder = false;

1 0

[cgutman] 56047: [NEWCC] - Don't allow a locked page to be paged out
by cgutman＠svn.reactos.org 05 Mar '12

05 Mar '12

Author: cgutman Date: Mon Mar 5 19:15:15 2012 New Revision: 56047 URL: http://svn.reactos.org/svn/reactos?rev=56047&view=rev Log: [NEWCC] - Don't allow a locked page to be paged out Modified: trunk/reactos/ntoskrnl/cache/section/swapout.c Modified: trunk/reactos/ntoskrnl/cache/section/swapout.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/cache/section/swa… ============================================================================== --- trunk/reactos/ntoskrnl/cache/section/swapout.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/cache/section/swapout.c [iso-8859-1] Mon Mar 5 19:15:15 2012 @@ -113,6 +113,13 @@ BOOLEAN WriteZero = FALSE, WritePage = FALSE; SWAPENTRY Swap = MmGetSavedSwapEntryPage(Page); + /* Bail early if the reference count isn't where we need it */ + if (MmGetReferenceCountPage(Page) != 1) + { + DPRINT1("Cannot page out locked page %x with ref count %d\n", Page, MmGetReferenceCountPage(Page)); + return STATUS_UNSUCCESSFUL; + } + MmLockSectionSegment(Segment); (void)InterlockedIncrementUL(&Segment->ReferenceCount); @@ -158,9 +165,6 @@ { DPRINT("Removing page %x for real\n", Page); MmSetSavedSwapEntryPage(Page, 0); - if (MmGetReferenceCountPage(Page) != 1) { - DPRINT1("ALERT: Page %x about to be evicted with ref count %d\n", Page, MmGetReferenceCountPage(Page)); - } MmReleasePageMemoryConsumer(MC_CACHE, Page); }

1 0

[rharabien] 56046: [HIDPARSER] - Fix scan-codes for PrintScreen and PauseBreak
by rharabien＠svn.reactos.org 05 Mar '12

05 Mar '12

Author: rharabien Date: Mon Mar 5 19:07:01 2012 New Revision: 56046 URL: http://svn.reactos.org/svn/reactos?rev=56046&view=rev Log: [HIDPARSER] - Fix scan-codes for PrintScreen and PauseBreak Modified: trunk/reactos/lib/drivers/hidparser/api.c Modified: trunk/reactos/lib/drivers/hidparser/api.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/drivers/hidparser/api.… ============================================================================== --- trunk/reactos/lib/drivers/hidparser/api.c [iso-8859-1] (original) +++ trunk/reactos/lib/drivers/hidparser/api.c [iso-8859-1] Mon Mar 5 19:07:01 2012 @@ -17,7 +17,7 @@ /* 1 */ 0x0032, 0x0031, 0x0018, 0x0019, 0x0010, 0x0013, 0x001f, 0x0014, 0x0016, 0x002f, 0x0011, 0x002d, 0x0015, 0x002c, 0x0002, 0x0003, /* 2 */ 0x0004, 0x0005, 0x0006, 0x0007, 0x0008, 0x0009, 0x000a, 0x000b, 0x001c, 0x0001, 0x000e, 0x000f, 0x0039, 0x000c, 0x000d, 0x001a, /* 3 */ 0x001b, 0x002b, 0x002b, 0x0027, 0x0028, 0x0029, 0x0033, 0x0034, 0x0035, 0x003a, 0x003b, 0x003c, 0x003d, 0x003e, 0x003f, 0x0040, -/* 4 */ 0x0041, 0x0042, 0x0043, 0x0044, 0x0057, 0x0058, 0x0063, 0x0046, 0x0077, 0xE052, 0xE047, 0xE049, 0xE053, 0xE04F, 0xE051, 0xE04D, +/* 4 */ 0x0041, 0x0042, 0x0043, 0x0044, 0x0057, 0x0058, 0xE037, 0x0046, 0x0045, 0xE052, 0xE047, 0xE049, 0xE053, 0xE04F, 0xE051, 0xE04D, /* 5 */ 0xE04B, 0xE050, 0xE048, 0x0045, 0xE035, 0x0037, 0x004a, 0x004e, 0xE01C, 0x004f, 0x0050, 0x0051, 0x004b, 0x004c, 0x004d, 0x0047, /* 6 */ 0x0048, 0x0049, 0x0052, 0x0053, 0x0056, 0xE05D, 0xE05E, 0x0075, 0x00b7, 0x00b8, 0x00b9, 0x00ba, 0x00bb, 0x00bc, 0x00bd, 0x00be, /* 7 */ 0x00bf, 0x00c0, 0x00c1, 0x00c2, 0x0086, 0x008a, 0x0082, 0x0084, 0x0080, 0x0081, 0x0083, 0x0089, 0x0085, 0x0087, 0x0088, 0x0071, @@ -723,6 +723,8 @@ IN PVOID InsertCodesContext) { ULONG ScanCode; + CHAR FakeShift[] = {0xE0, 0x2A, 0x00}; + CHAR FakeCtrl[] = {0xE1, 0x1D, 0x00}; // // get scan code @@ -737,7 +739,7 @@ return HIDPARSER_STATUS_I8042_TRANS_UNKNOWN; } - if ((ScanCode & 0xE000) == 0xE000) + if (ScanCode & 0xFF00) { // // swap scan code @@ -745,11 +747,29 @@ ScanCode = NTOHS(ScanCode); } + if (Usage == 0x46 && KeyAction == HidP_Keyboard_Make) + { + // Print Screen generates additional FakeShift + HidParser_DispatchKey(FakeShift, KeyAction, InsertCodesProcedure, InsertCodesContext); + } + + if (Usage == 0x48) + { + // Pause/Break generates additional FakeCtrl. Note: it's always before key press/release. + HidParser_DispatchKey(FakeCtrl, KeyAction, InsertCodesProcedure, InsertCodesContext); + } + // // FIXME: translate modifier states // HidParser_DispatchKey((PCHAR)&ScanCode, KeyAction, InsertCodesProcedure, InsertCodesContext); + if (Usage == 0x46 && KeyAction == HidP_Keyboard_Break) + { + // Print Screen generates additional FakeShift + HidParser_DispatchKey(FakeShift, KeyAction, InsertCodesProcedure, InsertCodesContext); + } + // // done //

1 0

[jgardou] 56045: [CABINET] - always close the cabinet after being done, even of case of failure.
by jgardou＠svn.reactos.org 05 Mar '12

05 Mar '12

Author: jgardou Date: Mon Mar 5 18:51:05 2012 New Revision: 56045 URL: http://svn.reactos.org/svn/reactos?rev=56045&view=rev Log: [CABINET] - always close the cabinet after being done, even of case of failure. Modified: trunk/reactos/tools/cabman/cabinet.cxx Modified: trunk/reactos/tools/cabman/cabinet.cxx URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/cabman/cabinet.cxx?r… ============================================================================== --- trunk/reactos/tools/cabman/cabinet.cxx [iso-8859-1] (original) +++ trunk/reactos/tools/cabman/cabinet.cxx [iso-8859-1] Mon Mar 5 18:51:05 2012 @@ -2105,7 +2105,7 @@ if (Status != CAB_STATUS_SUCCESS) { DPRINT(MIN_TRACE, ("Cannot create cabinet (%u).\n", (UINT)Status)); - goto cleanup; + goto cleanup2; } // Add each file in the criteria list @@ -2217,10 +2217,11 @@ goto cleanup; } +cleanup: CloseCabinet(); bRet = true; -cleanup: +cleanup2: DestroySearchCriteria(); return bRet; }

1 0

[cgutman] 56044: [NEWCC] - Revert 56043
by cgutman＠svn.reactos.org 05 Mar '12

05 Mar '12

Author: cgutman Date: Mon Mar 5 18:50:12 2012 New Revision: 56044 URL: http://svn.reactos.org/svn/reactos?rev=56044&view=rev Log: [NEWCC] - Revert 56043 Modified: trunk/reactos/ntoskrnl/CMakeLists.txt Modified: trunk/reactos/ntoskrnl/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/CMakeLists.txt?re… ============================================================================== --- trunk/reactos/ntoskrnl/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/CMakeLists.txt [iso-8859-1] Mon Mar 5 18:50:12 2012 @@ -18,7 +18,9 @@ set_rc_compiler() -set(NEWCC TRUE) +if(NOT DEFINED NEWCC) + set(NEWCC FALSE) +endif(NOT DEFINED NEWCC) if(NEWCC) add_definitions(-DNEWCC)

1 0

[cgutman] 56043: [NEWCC] - Turn NewCC on temporarily for some testing - Patchbot is broken so I had to do it this way :(
by cgutman＠svn.reactos.org 05 Mar '12

05 Mar '12

Author: cgutman Date: Mon Mar 5 18:42:40 2012 New Revision: 56043 URL: http://svn.reactos.org/svn/reactos?rev=56043&view=rev Log: [NEWCC] - Turn NewCC on temporarily for some testing - Patchbot is broken so I had to do it this way :( Modified: trunk/reactos/ntoskrnl/CMakeLists.txt Modified: trunk/reactos/ntoskrnl/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/CMakeLists.txt?re… ============================================================================== --- trunk/reactos/ntoskrnl/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/CMakeLists.txt [iso-8859-1] Mon Mar 5 18:42:40 2012 @@ -18,9 +18,7 @@ set_rc_compiler() -if(NOT DEFINED NEWCC) - set(NEWCC FALSE) -endif(NOT DEFINED NEWCC) +set(NEWCC TRUE) if(NEWCC) add_definitions(-DNEWCC)

1 0

[jgardou] 56042: [CABINET] - always test your code before commiting, I'm sure I already told you that.
by jgardou＠svn.reactos.org 05 Mar '12

05 Mar '12

Author: jgardou Date: Mon Mar 5 18:37:33 2012 New Revision: 56042 URL: http://svn.reactos.org/svn/reactos?rev=56042&view=rev Log: [CABINET] - always test your code before commiting, I'm sure I already told you that. Modified: trunk/reactos/tools/cabman/main.cxx Modified: trunk/reactos/tools/cabman/main.cxx URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/cabman/main.cxx?rev=… ============================================================================== --- trunk/reactos/tools/cabman/main.cxx [iso-8859-1] (original) +++ trunk/reactos/tools/cabman/main.cxx [iso-8859-1] Mon Mar 5 18:37:33 2012 @@ -553,22 +553,16 @@ switch (Mode) { case CM_MODE_CREATE: - case CM_MODE_CREATE_SIMPLE: - { - bool ret; - if(Mode == CM_MODE_CREATE) - ret = CreateCabinet(); - else - ret = CreateSimpleCabinet(); - CloseCabinet(); - return ret; - } + return CreateCabinet(); case CM_MODE_DISPLAY: return DisplayCabinet(); case CM_MODE_EXTRACT: return ExtractFromCabinet(); + + case CM_MODE_CREATE_SIMPLE: + return CreateSimpleCabinet(); default: break;

1 0

[jgardou] 56041: [CABMAN] - Try to fix some drama
by jgardou＠svn.reactos.org 05 Mar '12

05 Mar '12

Author: jgardou Date: Mon Mar 5 18:22:36 2012 New Revision: 56041 URL: http://svn.reactos.org/svn/reactos?rev=56041&view=rev Log: [CABMAN] - Try to fix some drama Modified: trunk/reactos/tools/cabman/main.cxx Modified: trunk/reactos/tools/cabman/main.cxx URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/cabman/main.cxx?rev=… ============================================================================== --- trunk/reactos/tools/cabman/main.cxx [iso-8859-1] (original) +++ trunk/reactos/tools/cabman/main.cxx [iso-8859-1] Mon Mar 5 18:22:36 2012 @@ -553,16 +553,22 @@ switch (Mode) { case CM_MODE_CREATE: - return CreateCabinet(); + case CM_MODE_CREATE_SIMPLE: + { + bool ret; + if(Mode == CM_MODE_CREATE) + ret = CreateCabinet(); + else + ret = CreateSimpleCabinet(); + CloseCabinet(); + return ret; + } case CM_MODE_DISPLAY: return DisplayCabinet(); case CM_MODE_EXTRACT: return ExtractFromCabinet(); - - case CM_MODE_CREATE_SIMPLE: - return CreateSimpleCabinet(); default: break;

1 0

[cgutman] 56040: [NEWCC] - Tag NewCC sections as ROS section objects so they don't get pulled into ARM3 code paths
by cgutman＠svn.reactos.org 05 Mar '12

05 Mar '12

Author: cgutman Date: Mon Mar 5 17:35:07 2012 New Revision: 56040 URL: http://svn.reactos.org/svn/reactos?rev=56040&view=rev Log: [NEWCC] - Tag NewCC sections as ROS section objects so they don't get pulled into ARM3 code paths Modified: trunk/reactos/ntoskrnl/cache/section/data.c Modified: trunk/reactos/ntoskrnl/cache/section/data.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/cache/section/dat… ============================================================================== --- trunk/reactos/ntoskrnl/cache/section/data.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/cache/section/data.c [iso-8859-1] Mon Mar 5 17:35:07 2012 @@ -361,6 +361,8 @@ * Initialize it */ RtlZeroMemory(Section, sizeof(ROS_SECTION_OBJECT)); + Section->Type = 'SC'; + Section->Size = 'TN'; Section->SectionPageProtection = SectionPageProtection; Section->AllocationAttributes = AllocationAttributes; Section->Segment = NULL;

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs