Ros-diffs

ros-diffs@reactos.org

81390 discussions

[hbelusca] 68591: [NTVDM]: Fix the return code of 2 functions.
by hbelusca＠svn.reactos.org 01 Aug '15

01 Aug '15

Author: hbelusca Date: Sat Aug 1 21:40:58 2015 New Revision: 68591 URL: http://svn.reactos.org/svn/reactos?rev=68591&view=rev Log: [NTVDM]: Fix the return code of 2 functions. Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.c [iso-8859-1] Sat Aug 1 21:40:58 2015 @@ -90,7 +90,7 @@ return (HandleEntry != NULL && HandleEntry->Allocated); } -static USHORT EmsFree(USHORT Handle) +static UCHAR EmsFree(USHORT Handle) { PLIST_ENTRY Entry; PEMS_HANDLE HandleEntry = GetHandleRecord(Handle); @@ -171,7 +171,7 @@ return (PEMS_PAGE)CONTAINING_RECORD(Entry, EMS_PAGE, Entry); } -static USHORT EmsMap(USHORT Handle, UCHAR PhysicalPage, USHORT LogicalPage) +static UCHAR EmsMap(USHORT Handle, UCHAR PhysicalPage, USHORT LogicalPage) { PEMS_PAGE PageEntry; PEMS_HANDLE HandleEntry = GetHandleRecord(Handle);

1 0

[ekohl] 68590: [PARPORT] Remove DPRINTs from PdoWrite. CORE-9942 #resolve
by ekohl＠svn.reactos.org 01 Aug '15

01 Aug '15

Author: ekohl Date: Sat Aug 1 19:41:24 2015 New Revision: 68590 URL: http://svn.reactos.org/svn/reactos?rev=68590&view=rev Log: [PARPORT] Remove DPRINTs from PdoWrite. CORE-9942 #resolve Modified: trunk/reactos/drivers/parallel/parport/pdo.c Modified: trunk/reactos/drivers/parallel/parport/pdo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/parallel/parport/p… ============================================================================== --- trunk/reactos/drivers/parallel/parport/pdo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/parallel/parport/pdo.c [iso-8859-1] Sat Aug 1 19:41:24 2015 @@ -114,15 +114,8 @@ DPRINT("Length: %lu\n", IoStack->Parameters.Write.Length); DPRINT("Buffer: %p\n", Buffer); - if (Buffer != NULL) - { - DPRINT("%s\n", Buffer); - } - for (i = 0; i < IoStack->Parameters.Write.Length; i++) { - DPRINT("%lu: %c\n", i, Buffer[i]); - ulCount = 0; do

1 0

[hbelusca] 68589: [NTVDM]: Use the while() construct for looping into doubly-linked lists where we can remove items from it in the middle of the loop code.
by hbelusca＠svn.reactos.org 01 Aug '15

01 Aug '15

Author: hbelusca Date: Sat Aug 1 18:56:44 2015 New Revision: 68589 URL: http://svn.reactos.org/svn/reactos?rev=68589&view=rev Log: [NTVDM]: Use the while() construct for looping into doubly-linked lists where we can remove items from it in the middle of the loop code. Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.c trunk/reactos/subsystems/mvdm/ntvdm/clock.c trunk/reactos/subsystems/mvdm/ntvdm/clock.h Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.c [iso-8859-1] Sat Aug 1 18:56:44 2015 @@ -91,11 +91,11 @@ // FIXME: Check! What to do? if (RequestSize == 0) DPRINT1("Requesting UMA descriptor with null size?!\n"); - for (Entry = UmaDescriptorList.Flink; - Entry != &UmaDescriptorList; - Entry = Entry->Flink) + Entry = UmaDescriptorList.Flink; + while (Entry != &UmaDescriptorList) { UmaDesc = (PUMA_DESCRIPTOR)CONTAINING_RECORD(Entry, UMA_DESCRIPTOR, Entry); + Entry = Entry->Flink; /* Only check free descriptors */ if (UmaDesc->Type != UMA_FREE) continue; @@ -209,11 +209,11 @@ PUMA_DESCRIPTOR UmaDesc, PrevDesc = NULL, NextDesc = NULL; PUMA_DESCRIPTOR FoundUmaDesc = NULL; - for (Entry = UmaDescriptorList.Flink; - Entry != &UmaDescriptorList; - Entry = Entry->Flink) + Entry = UmaDescriptorList.Flink; + while (Entry != &UmaDescriptorList) { UmaDesc = (PUMA_DESCRIPTOR)CONTAINING_RECORD(Entry, UMA_DESCRIPTOR, Entry); + Entry = Entry->Flink; /* Search for the descriptor in the list */ if (UmaDesc->Start == Address && UmaDesc->Type == UMA_UMB) @@ -267,11 +267,11 @@ // FIXME: Check! What to do? if (RequestSize == 0) DPRINT1("Resizing UMA descriptor %04X to null size?!\n", Segment); - for (Entry = UmaDescriptorList.Flink; - Entry != &UmaDescriptorList; - Entry = Entry->Flink) + Entry = UmaDescriptorList.Flink; + while (Entry != &UmaDescriptorList) { UmaDesc = (PUMA_DESCRIPTOR)CONTAINING_RECORD(Entry, UMA_DESCRIPTOR, Entry); + Entry = Entry->Flink; /* Only get the maximum size of free descriptors */ if (UmaDesc->Type == UMA_FREE) Modified: trunk/reactos/subsystems/mvdm/ntvdm/clock.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/cloc… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/clock.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/clock.c [iso-8859-1] Sat Aug 1 18:56:44 2015 @@ -65,6 +65,7 @@ extern BOOLEAN CpuRunning; UINT i; PLIST_ENTRY Entry; + PHARDWARE_TIMER Timer; while (VdmRunning && CpuRunning) { @@ -81,10 +82,13 @@ ++Cycles; } - for (Entry = Timers.Flink; Entry != &Timers; Entry = Entry->Flink) + Entry = Timers.Flink; + while (Entry != &Timers) { ULONGLONG Ticks = (ULONGLONG)-1; - PHARDWARE_TIMER Timer = CONTAINING_RECORD(Entry, HARDWARE_TIMER, Link); + + Timer = CONTAINING_RECORD(Entry, HARDWARE_TIMER, Link); + Entry = Entry->Flink; ASSERT((Timer->EnableCount > 0) && (Timer->Flags & HARDWARE_TIMER_ENABLED)); @@ -128,12 +132,13 @@ { PHARDWARE_TIMER Timer; - Timer = RtlAllocateHeap(RtlGetProcessHeap(), 0, sizeof(*Timer)); + Timer = RtlAllocateHeap(RtlGetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*Timer)); if (Timer == NULL) return NULL; Timer->Flags = Flags & ~HARDWARE_TIMER_ENABLED; Timer->EnableCount = 0; Timer->Callback = Callback; + Timer->LastTick.QuadPart = 0; SetHardwareTimerDelay(Timer, Delay); if (Flags & HARDWARE_TIMER_ENABLED) EnableHardwareTimer(Timer); Modified: trunk/reactos/subsystems/mvdm/ntvdm/clock.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/cloc… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/clock.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/clock.h [iso-8859-1] Sat Aug 1 18:56:44 2015 @@ -27,8 +27,8 @@ ULONG Flags; LONG EnableCount; ULONGLONG Delay; + LARGE_INTEGER LastTick; PHARDWARE_TIMER_PROC Callback; - LARGE_INTEGER LastTick; } HARDWARE_TIMER, *PHARDWARE_TIMER; /* FUNCTIONS ******************************************************************/

1 0

[ekohl] 68588: [PARPORT] - Add support for IRP_MJ_READ. - IRP_MJ_READ and IRP_MJ_WRITE return STATUS_NOT_SUPPORTED for FDO devices.
by ekohl＠svn.reactos.org 01 Aug '15

01 Aug '15

Author: ekohl Date: Sat Aug 1 18:02:45 2015 New Revision: 68588 URL: http://svn.reactos.org/svn/reactos?rev=68588&view=rev Log: [PARPORT] - Add support for IRP_MJ_READ. - IRP_MJ_READ and IRP_MJ_WRITE return STATUS_NOT_SUPPORTED for FDO devices. Modified: trunk/reactos/drivers/parallel/parport/fdo.c trunk/reactos/drivers/parallel/parport/parport.c trunk/reactos/drivers/parallel/parport/parport.h trunk/reactos/drivers/parallel/parport/pdo.c Modified: trunk/reactos/drivers/parallel/parport/fdo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/parallel/parport/f… ============================================================================== --- trunk/reactos/drivers/parallel/parport/fdo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/parallel/parport/fdo.c [iso-8859-1] Sat Aug 1 18:02:45 2015 @@ -473,15 +473,29 @@ NTSTATUS NTAPI +FdoRead(IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp) +{ + DPRINT("FdoRead()\n"); + + Irp->IoStatus.Information = 0; + Irp->IoStatus.Status = STATUS_NOT_SUPPORTED; + IoCompleteRequest(Irp, IO_NO_INCREMENT); + return STATUS_NOT_SUPPORTED; +} + + +NTSTATUS +NTAPI FdoWrite(IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp) { DPRINT("FdoWrite()\n"); Irp->IoStatus.Information = 0; - Irp->IoStatus.Status = STATUS_SUCCESS; + Irp->IoStatus.Status = STATUS_NOT_SUPPORTED; IoCompleteRequest(Irp, IO_NO_INCREMENT); - return STATUS_SUCCESS; + return STATUS_NOT_SUPPORTED; } Modified: trunk/reactos/drivers/parallel/parport/parport.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/parallel/parport/p… ============================================================================== --- trunk/reactos/drivers/parallel/parport/parport.c [iso-8859-1] (original) +++ trunk/reactos/drivers/parallel/parport/parport.c [iso-8859-1] Sat Aug 1 18:02:45 2015 @@ -68,6 +68,19 @@ static NTSTATUS NTAPI +DispatchRead(IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp) +{ + if (((PFDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension)->Common.IsFDO) + return FdoRead(DeviceObject, Irp); + else + return PdoRead(DeviceObject, Irp); +} + + +static +NTSTATUS +NTAPI DispatchWrite(IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp) { @@ -122,7 +135,7 @@ DriverObject->MajorFunction[IRP_MJ_CREATE] = DispatchCreate; DriverObject->MajorFunction[IRP_MJ_CLOSE] = DispatchClose; DriverObject->MajorFunction[IRP_MJ_CLEANUP] = DispatchCleanup; -// DriverObject->MajorFunction[IRP_MJ_READ] = DispatchRead; + DriverObject->MajorFunction[IRP_MJ_READ] = DispatchRead; DriverObject->MajorFunction[IRP_MJ_WRITE] = DispatchWrite; // DriverObject->MajorFunction[IRP_MJ_DEVICE_CONTROL] = DispatchDeviceControl; // DriverObject->MajorFunction[IRP_MJ_QUERY_INFORMATION] = DispatchQueryInformation; Modified: trunk/reactos/drivers/parallel/parport/parport.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/parallel/parport/p… ============================================================================== --- trunk/reactos/drivers/parallel/parport/parport.h [iso-8859-1] (original) +++ trunk/reactos/drivers/parallel/parport/parport.h [iso-8859-1] Sat Aug 1 18:02:45 2015 @@ -88,6 +88,11 @@ NTSTATUS NTAPI +FdoRead(IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp); + +NTSTATUS +NTAPI FdoWrite(IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp); @@ -135,6 +140,11 @@ NTSTATUS NTAPI +PdoRead(IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp); + +NTSTATUS +NTAPI PdoWrite(IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp); Modified: trunk/reactos/drivers/parallel/parport/pdo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/parallel/parport/p… ============================================================================== --- trunk/reactos/drivers/parallel/parport/pdo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/parallel/parport/pdo.c [iso-8859-1] Sat Aug 1 18:02:45 2015 @@ -68,6 +68,20 @@ IN PIRP Irp) { DPRINT("PdoCleanup()\n"); + + Irp->IoStatus.Information = 0; + Irp->IoStatus.Status = STATUS_SUCCESS; + IoCompleteRequest(Irp, IO_NO_INCREMENT); + return STATUS_SUCCESS; +} + + +NTSTATUS +NTAPI +PdoRead(IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp) +{ + DPRINT("PdoRead()\n"); Irp->IoStatus.Information = 0; Irp->IoStatus.Status = STATUS_SUCCESS;

1 0

[hbelusca] 68587: [MSPAINT_NEW] Fix build on MSVC by rewriting the code in *STANDARD* C++ !! (and not into some strange idiom called "GCC-C++"). I suggest also to write a proper class for dynamical...
by hbelusca＠svn.reactos.org 01 Aug '15

01 Aug '15

Author: hbelusca Date: Sat Aug 1 17:30:17 2015 New Revision: 68587 URL: http://svn.reactos.org/svn/reactos?rev=68587&view=rev Log: [MSPAINT_NEW] Fix build on MSVC by rewriting the code in *STANDARD* C++ !! (and not into some strange idiom called "GCC-C++"). I suggest also to write a proper class for dynamically-allocated (resource) strings instead of either having the static arrays of hardcoded sizes, or being tempted to use non-standard constructs as the one I just saw. (and btw, instead of defining a new "SIZEOF()" macro, there is one which already exists in the PSDK called "ARRAYSIZE()" which just does the correct job). Modified: trunk/reactos/base/applications/mspaint_new/winproc.cpp Modified: trunk/reactos/base/applications/mspaint_new/winproc.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/winproc.cpp [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/winproc.cpp [iso-8859-1] Sat Aug 1 17:30:17 2015 @@ -196,14 +196,11 @@ { if (!imageModel.IsImageSaved()) { - TCHAR* tempptr; - TCHAR programname[LoadString(hProgInstance, IDS_PROGRAMNAME, (LPTSTR)&tempptr, 0) + 1]; - CopyMemory(programname, tempptr, sizeof(programname) - sizeof(TCHAR)); - programname[SIZEOF(programname) - 1] = (TCHAR)'\0'; - TCHAR saveprompttext[LoadString(hProgInstance, IDS_SAVEPROMPTTEXT, (LPTSTR)&tempptr, 0) + 1]; - CopyMemory(saveprompttext, tempptr, sizeof(saveprompttext) - sizeof(TCHAR)); - saveprompttext[SIZEOF(saveprompttext) - 1] = (TCHAR)'\0'; - TCHAR temptext[SIZEOF(saveprompttext) + _tcslen(filename)]; + TCHAR programname[20]; + TCHAR saveprompttext[100]; + TCHAR temptext[500]; + LoadString(hProgInstance, IDS_PROGRAMNAME, programname, SIZEOF(programname)); + LoadString(hProgInstance, IDS_SAVEPROMPTTEXT, saveprompttext, SIZEOF(saveprompttext)); _stprintf(temptext, saveprompttext, filename); switch (MessageBox(temptext, programname, MB_YESNOCANCEL | MB_ICONQUESTION)) {

1 0

[hbelusca] 68586: [NTVDM] EMS: - Introduce (and use) helpers for checking validity of EMS handles. - Do not hardcode the EMS page frame segment, but instead allow it to be changed (for now, it is s...
by hbelusca＠svn.reactos.org 01 Aug '15

01 Aug '15

Author: hbelusca Date: Sat Aug 1 17:07:07 2015 New Revision: 68586 URL: http://svn.reactos.org/svn/reactos?rev=68586&view=rev Log: [NTVDM] EMS: - Introduce (and use) helpers for checking validity of EMS handles. - Do not hardcode the EMS page frame segment, but instead allow it to be changed (for now, it is still set to a default value; in the future, via some configuration file or via the registry). This is done by acquiring a UMB block (see after). XMS: - Implement functions 0x01 "Request HMA" and 0x02 "Release HMA". - Fix the return value of functions 0x04 "Global disable A20" and 0x08 "Query free Extended Memory"; simplify code of function 0x0B "Move EMB". - Halfplement function 0x0F "Reallocate Extended Memory Block" in the simple case of size reduction (size expansion is left to the programmer as an exercise :PP ) - Rewrite the UMB provider support (functions 0x10, 0x11, 0x12) by calling the Upper Memory Area manager helpers (see after) (this is closer to reality: UMBs are either provided by XMS driver itself, or by an EMS driver which hooks into the XMS driver chain -- as it is done with MS' himem+EMM386; sometimes all that stuff is contained inside one driver only --) instead of calling back into DOS. This is the DOS which calls XMS for getting the UMB blocks and initializing them! (and not the other way around as it was done in r68001!). NTVDM: - Introduce an "Upper Memory Area manager" which maintains a list of areas of upper memory (>= A000:0000 and <= FFFF:000F) that can be used as RAM blocks. It is intended to work closely with the NTVDM memory manager and be used by XMS for getting possible free UMBs, and by VDDs for implementing the VDDInclude/ExcludeMem APIs (which adds/remove blocks in/from the UMB pool; those are unaccessible to DOS if those APIs are called after NTVDM have been started, but are accessible by XMS). DOS: - Add a helper function for detecting early DOS arena corruptions (for debugging purposes only). - Make the DOS memory manager really UMB-compatible. This means: * not hardcoding the start of the UMB chain; * getting all the available UMB blocks from XMS and initializing them, marking the reserved blocks as read-only (with a correct header; reserved blocks are eg. VGA memory area, ROM blocks...). There is room for improvements obviously (see the FIXMEs in the code). Used documentation is mentioned in comments in the code. This commit should fix quite some apps, as well as it fixes corruptions of loaded ROMs in upper memory: that's how I came into working on fixing the UMB support. In other words, during those two last weeks, I was like in: http://i.imgur.com/zoWpqEB.gifv CORE-9969 #resolve Added: trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.c (with props) trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.h (with props) Modified: trunk/reactos/subsystems/mvdm/ntvdm/CMakeLists.txt trunk/reactos/subsystems/mvdm/ntvdm/bios/bios.c trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.c trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.h trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.c trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.h trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/himem.c trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/himem.h trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/memory.c trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/memory.h trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/process.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/CMak… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/CMakeLists.txt [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -14,6 +14,7 @@ bios/bios.c bios/kbdbios.c bios/rom.c + bios/umamgr.c bios/vidbios.c cpu/bop.c cpu/callback.c @@ -40,8 +41,8 @@ dos/dos32krnl/himem.c dos/dos32krnl/memory.c dos/dos32krnl/process.c + dos/dem.c dos/mouse32.c - dos/dem.c clock.c emulator.c int32.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios.c [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -17,8 +17,8 @@ #include "bios.h" #include "bios32/bios32.h" - #include "rom.h" +#include "umamgr.h" #include "io.h" #include "hardware/cmos.h" @@ -87,22 +87,22 @@ DisplayMessage(L"First bytes at 0x%p: 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x\n" L"3 last bytes at 0x%p: 0x%02x 0x%02x 0x%02x", - BiosLocation, - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 0), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 1), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 2), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 3), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 4), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 5), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 6), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 7), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 8), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 9), + BiosLocation, + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 0), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 1), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 2), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 3), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 4), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 5), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 6), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 7), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 8), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + 9), - (PVOID)((ULONG_PTR)BiosLocation + BiosSize - 2), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + BiosSize - 2), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + BiosSize - 1), - *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + BiosSize - 0)); + (PVOID)((ULONG_PTR)BiosLocation + BiosSize - 2), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + BiosSize - 2), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + BiosSize - 1), + *(PCHAR)((ULONG_PTR)REAL_TO_PHYS(BiosLocation) + BiosSize - 0)); DisplayMessage(L"POST at 0x%p: 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x", TO_LINEAR(getCS(), getIP()), @@ -135,12 +135,21 @@ // /* Enable interrupts */ // setIF(1); + /* Initialize the Upper Memory Area Manager */ + if (!UmaMgrInitialize()) + { + wprintf(L"FATAL: Failed to initialize the UMA manager.\n"); + return FALSE; + } + return Success; } VOID BiosCleanup(VOID) { + UmaMgrCleanup(); + if (Bios32Loaded) Bios32Cleanup(); } Added: trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.c (added) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.c [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -0,0 +1,625 @@ +/* + * COPYRIGHT: GPLv2+ - See COPYING in the top level directory + * PROJECT: ReactOS Virtual DOS Machine + * FILE: umamgr.c + * PURPOSE: Upper Memory Area Manager + * PROGRAMMERS: Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) + * + * NOTE: The UMA Manager is used by the DOS XMS Driver (UMB Provider part), + * indirectly by the DOS EMS Driver, and by VDD memory management functions. + */ + +/* INCLUDES *******************************************************************/ + +#define NDEBUG + +#include "ntvdm.h" +#include "emulator.h" +#include "memory.h" + +#include "umamgr.h" + +/* PRIVATE VARIABLES **********************************************************/ + +typedef struct _UMA_DESCRIPTOR +{ + LIST_ENTRY Entry; + ULONG Start; + ULONG Size; + UMA_DESC_TYPE Type; +} UMA_DESCRIPTOR, *PUMA_DESCRIPTOR; + +/* + * Sorted list of UMA descriptors. + * + * The descriptor list is (and must always be) sorted by memory addresses, + * and all consecutive free descriptors are always merged together, so that + * free ones are always separated from each other by descriptors of other types. + * + * TODO: Add the fact that no blocks of size == 0 are allowed. + */ +static LIST_ENTRY UmaDescriptorList = { &UmaDescriptorList, &UmaDescriptorList }; + +/* PRIVATE FUNCTIONS **********************************************************/ + +static PUMA_DESCRIPTOR +CreateUmaDescriptor(IN OUT PLIST_ENTRY ListHead, + IN ULONG Address, + IN ULONG Size, + IN UMA_DESC_TYPE Type) +{ + PUMA_DESCRIPTOR UmaDesc; + + ASSERT(Size > 0); + + UmaDesc = RtlAllocateHeap(RtlGetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*UmaDesc)); + if (!UmaDesc) return NULL; + + UmaDesc->Start = Address; + UmaDesc->Size = Size; + UmaDesc->Type = Type; + + /* + * We use the trick of http://www.osronline.com/article.cfm?article=499 to insert + * the new descriptor just after the current entry that we specify via 'ListHead'. + * If 'ListHead' is NULL then we insert the descriptor at the tail of 'UmaDescriptorList' + * (which is equivalent to inserting it at the head of 'UmaDescriptorList.Blink'). + */ + if (ListHead == NULL) ListHead = UmaDescriptorList.Blink; + InsertHeadList(ListHead, &UmaDesc->Entry); + + return UmaDesc; +} + +static VOID FreeUmaDescriptor(PUMA_DESCRIPTOR UmaDesc) +{ + RemoveEntryList(&UmaDesc->Entry); + RtlFreeHeap(RtlGetProcessHeap(), 0, UmaDesc); +} + +/* PUBLIC FUNCTIONS ***********************************************************/ + +BOOLEAN UmaDescReserve(IN OUT PUSHORT Segment, IN OUT PUSHORT Size) +{ + ULONG Address = (*Segment << 4); // Convert segment number into address. + ULONG RequestSize = (*Size << 4); // Convert size in paragraphs into size in bytes. + ULONG MaxSize = 0; + PLIST_ENTRY Entry; + PUMA_DESCRIPTOR UmaDesc, NewUmaDesc; + PUMA_DESCRIPTOR FoundUmaDesc = NULL; + + // FIXME: Check! What to do? + if (RequestSize == 0) DPRINT1("Requesting UMA descriptor with null size?!\n"); + + for (Entry = UmaDescriptorList.Flink; + Entry != &UmaDescriptorList; + Entry = Entry->Flink) + { + UmaDesc = (PUMA_DESCRIPTOR)CONTAINING_RECORD(Entry, UMA_DESCRIPTOR, Entry); + + /* Only check free descriptors */ + if (UmaDesc->Type != UMA_FREE) continue; + + /* Update the maximum descriptor size */ + if (UmaDesc->Size > MaxSize) MaxSize = UmaDesc->Size; + + /* Descriptor too small, continue... */ + if (UmaDesc->Size < RequestSize) continue; + + /* Do we want to reserve the descriptor at a precise address? */ + if (Address) + { + /* If the descriptor ends before the desired region, try again */ + if (UmaDesc->Start + UmaDesc->Size <= Address) continue; + + /* + * If we have a descriptor, but one of its boundaries crosses the + * desired region (it starts after the desired region, or ends + * before the end of the desired region), this means that there + * is already something inside the region, so that we cannot + * allocate the region here. Bail out. + */ + if (UmaDesc->Start > Address || + UmaDesc->Start + UmaDesc->Size < Address + RequestSize) + { + goto Fail; + } + + /* We now have a free descriptor that overlaps our desired region: split it */ + + /* + * Here, UmaDesc->Start == Address or UmaDesc->Start < Address, + * in which case we need to split the descriptor to the left. + */ + if (UmaDesc->Start < Address) + { + /* Create a new free descriptor and insert it after the current one */ + NewUmaDesc = CreateUmaDescriptor(&UmaDesc->Entry, + Address, + UmaDesc->Size - (Address - UmaDesc->Start), + UmaDesc->Type); // UMA_FREE + if (!NewUmaDesc) + { + DPRINT1("CreateUmaDescriptor failed, UMA descriptor list possibly corrupted!\n"); + goto Fail; + } + + /* Reduce the size of the splitted left descriptor */ + UmaDesc->Size = (Address - UmaDesc->Start); + + /* Current descriptor is now the new created one */ + UmaDesc = NewUmaDesc; + } + + /* Here, UmaDesc->Start == Address */ + } + + /* Descriptor of large enough size: split it to the right if needed */ + // FIXME: It might be needed to consider a minimum size starting which we need to split. + // if (UmaDesc->Size - RequestSize > (3 << 4)) + if (UmaDesc->Size > RequestSize) + { + /* + * Create a new free descriptor and insert it after the current one. + * Because consecutive free descriptors are always merged together, + * the descriptor following 'UmaDesc' cannot be free, so that this + * new free descriptor does not need to be merged with some others. + */ + NewUmaDesc = CreateUmaDescriptor(&UmaDesc->Entry, + UmaDesc->Start + RequestSize, + UmaDesc->Size - RequestSize, + UmaDesc->Type); // UMA_FREE + if (!NewUmaDesc) + { + DPRINT1("CreateUmaDescriptor failed, UMA descriptor list possibly corrupted!\n"); + goto Fail; + } + + /* Reduce the size of the splitted left descriptor */ + UmaDesc->Size = RequestSize; + } + + /* We have a descriptor of correct size, initialize it */ + UmaDesc->Type = UMA_UMB; + FoundUmaDesc = UmaDesc; + break; + } + + if (FoundUmaDesc) + { + /* Returned address is a segment and size is in paragraphs */ + *Segment = (FoundUmaDesc->Start >> 4); + *Size = (FoundUmaDesc->Size >> 4); + return TRUE; + } + else + { +Fail: + /* Returned address is a segment and size is in paragraphs */ + *Segment = 0x0000; + *Size = (MaxSize >> 4); + return FALSE; + } +} + +BOOLEAN UmaDescRelease(IN USHORT Segment) +{ + ULONG Address = (Segment << 4); // Convert segment number into address. + PLIST_ENTRY Entry, PrevEntry, NextEntry; + PUMA_DESCRIPTOR UmaDesc, PrevDesc = NULL, NextDesc = NULL; + PUMA_DESCRIPTOR FoundUmaDesc = NULL; + + for (Entry = UmaDescriptorList.Flink; + Entry != &UmaDescriptorList; + Entry = Entry->Flink) + { + UmaDesc = (PUMA_DESCRIPTOR)CONTAINING_RECORD(Entry, UMA_DESCRIPTOR, Entry); + + /* Search for the descriptor in the list */ + if (UmaDesc->Start == Address && UmaDesc->Type == UMA_UMB) + { + /* We found it */ + FoundUmaDesc = UmaDesc; + break; + } + } + + if (FoundUmaDesc) + { + FoundUmaDesc->Type = UMA_FREE; + + /* Combine free descriptors adjacent to this one */ + PrevEntry = FoundUmaDesc->Entry.Blink; + NextEntry = FoundUmaDesc->Entry.Flink; + + if (PrevEntry != &UmaDescriptorList) + PrevDesc = (PUMA_DESCRIPTOR)CONTAINING_RECORD(PrevEntry, UMA_DESCRIPTOR, Entry); + if (NextEntry != &UmaDescriptorList) + NextDesc = (PUMA_DESCRIPTOR)CONTAINING_RECORD(NextEntry, UMA_DESCRIPTOR, Entry); + + if (NextDesc && NextDesc->Type == FoundUmaDesc->Type) // UMA_FREE + { + FoundUmaDesc->Size += NextDesc->Size; + FreeUmaDescriptor(NextDesc); + } + + if (PrevDesc && PrevDesc->Type == FoundUmaDesc->Type) // UMA_FREE + { + PrevDesc->Size += FoundUmaDesc->Size; + FreeUmaDescriptor(FoundUmaDesc); + } + + return TRUE; + } + + return FALSE; +} + +BOOLEAN UmaDescReallocate(IN USHORT Segment, IN OUT PUSHORT Size) +{ + ULONG Address = (Segment << 4); // Convert segment number into address. + ULONG RequestSize = (*Size << 4); // Convert size in paragraphs into size in bytes. + ULONG MaxSize = 0; + PLIST_ENTRY Entry, NextEntry; + PUMA_DESCRIPTOR UmaDesc, NextDesc = NULL, NewUmaDesc; + PUMA_DESCRIPTOR FoundUmaDesc = NULL; + + // FIXME: Check! What to do? + if (RequestSize == 0) DPRINT1("Resizing UMA descriptor %04X to null size?!\n", Segment); + + for (Entry = UmaDescriptorList.Flink; + Entry != &UmaDescriptorList; + Entry = Entry->Flink) + { + UmaDesc = (PUMA_DESCRIPTOR)CONTAINING_RECORD(Entry, UMA_DESCRIPTOR, Entry); + + /* Only get the maximum size of free descriptors */ + if (UmaDesc->Type == UMA_FREE) + { + /* Update the maximum descriptor size */ + if (UmaDesc->Size > MaxSize) MaxSize = UmaDesc->Size; + } + + /* Search for the descriptor in the list */ + if (UmaDesc->Start == Address && UmaDesc->Type == UMA_UMB) + { + /* We found it */ + FoundUmaDesc = UmaDesc; + break; + } + } + + if (FoundUmaDesc) + { + /* If we do not resize anything, just quit with success */ + if (FoundUmaDesc->Size == RequestSize) goto Success; + + NextEntry = FoundUmaDesc->Entry.Flink; + + if (NextEntry != &UmaDescriptorList) + NextDesc = (PUMA_DESCRIPTOR)CONTAINING_RECORD(NextEntry, UMA_DESCRIPTOR, Entry); + + /* Check for reduction or enlargement */ + if (FoundUmaDesc->Size > RequestSize) + { + /* Reduction */ + + /* + * Check if the next descriptor is free, in which case we + * extend it, otherwise we create a new free descriptor. + */ + if (NextDesc && NextDesc->Type == UMA_FREE) + { + /* Yes it is, expand its size and move it down */ + NextDesc->Size += (FoundUmaDesc->Size - RequestSize); + NextDesc->Start -= (FoundUmaDesc->Size - RequestSize); + } + else + { + // FIXME: It might be needed to consider a minimum size starting which we need to split. + // if (FoundUmaDesc->Size - RequestSize > (3 << 4)) + + /* Create a new free descriptor and insert it after the current one */ + NewUmaDesc = CreateUmaDescriptor(&FoundUmaDesc->Entry, + FoundUmaDesc->Start + RequestSize, + FoundUmaDesc->Size - RequestSize, + FoundUmaDesc->Type); + if (!NewUmaDesc) + { + DPRINT1("CreateUmaDescriptor failed, UMA descriptor list possibly corrupted!\n"); + MaxSize = 0; + goto Fail; + } + } + } + else // if (FoundUmaDesc->Size <= RequestSize) + { + /* Enlargement */ + + /* Check whether the next descriptor is free and large enough for merging */ + if (NextDesc && NextDesc->Type == UMA_FREE && + FoundUmaDesc->Size + NextDesc->Size >= RequestSize) + { + /* Yes it is, reduce its size and move it up, and enlarge the reallocated descriptor */ + NextDesc->Size -= (RequestSize - FoundUmaDesc->Size); + NextDesc->Start += (RequestSize - FoundUmaDesc->Size); + + if (NextDesc->Size == 0) FreeUmaDescriptor(NextDesc); + } + else + { + MaxSize = 0; + goto Fail; + } + } + + /* Finally, resize the descriptor */ + FoundUmaDesc->Size = RequestSize; + +Success: + /* Returned size is in paragraphs */ + *Size = (FoundUmaDesc->Size >> 4); + return TRUE; + } + else + { +Fail: + /* Returned size is in paragraphs */ + *Size = (MaxSize >> 4); + return FALSE; + } +} + +BOOLEAN UmaMgrInitialize(VOID) +{ +// See bios/rom.h +#define ROM_AREA_START 0xE0000 +#define ROM_AREA_END 0xFFFFF + +#define OPTION_ROM_SIGNATURE 0xAA55 + + PUMA_DESCRIPTOR UmaDesc = NULL; + ULONG StartAddress = 0; + ULONG Size = 0; + UMA_DESC_TYPE Type = UMA_FREE; + + UINT i; + + ULONG Start, End; + ULONG Increment; + + ULONG Address; + + // ULONG RomStart[] = {}; + // ULONG RomEnd[] = {}; + ULONG RomBoundaries[] = {0xA0000, 0xC0000, /*0xC8000, 0xE0000,*/ 0xF0000, 0x100000}; + ULONG SizeIncrement[] = {0x20000, 0x00800, /*0x00800, 0x10000,*/ 0x10000, 0x0000 }; + + // InitializeListHead(&UmaDescriptorList); + + /* NOTE: There must be always one UMA descriptor at least */ + // FIXME: Maybe it should be a static object? + + for (i = 0; i < ARRAYSIZE(RomBoundaries) - 1; i++) + { + Start = RomBoundaries[i]; // RomStart[i] + End = RomBoundaries[i+1]; // RomEnd[i] + Increment = SizeIncrement[i]; + + for (Address = Start; Address < End; Address += Increment) + { + if (StartAddress == 0) + { + /* Initialize data for a new descriptor */ + StartAddress = Address; + Size = 0; + Type = UMA_FREE; + } + + /* Is it a normal system zone/user-excluded zone? */ + if (Address >= 0xA0000 && Address < 0xC0000) + { + // StartAddress = Address; + Size = Increment; + Type = UMA_SYSTEM; + + /* Create descriptor */ + UmaDesc = CreateUmaDescriptor(NULL, StartAddress, Size, Type); + if (!UmaDesc) return FALSE; + + StartAddress = 0; + continue; + } + /* Is it the PC ROM BIOS? */ + else if (Address >= 0xF0000) + { + // StartAddress = Address; + Size = 0x10000; // Increment; + Type = UMA_ROM; + + /* Create descriptor */ + UmaDesc = CreateUmaDescriptor(NULL, StartAddress, Size, Type); + if (!UmaDesc) return FALSE; + + StartAddress = 0; + continue; + } + /* Is it an option ROM? */ + else if (Address >= 0xC0000 && Address < 0xF0000) + { + ULONG RomSize; + ULONG PrevRomAddress = 0; + ULONG PrevRomSize = 0; + + // while (Address < 0xF0000) + for (; Address < 0xF0000; Address += Increment) + { + +#if 0 // FIXME: Is this block, better here... + { + // We may be looping inside a ROM block, if: Type == 2 and: + // (StartAddress <= Address &&) StartAddress + Size > Address. + // In this case, do nothing (do not increase size either) + // But if we are now outside of a ROM block, then we need + // to create the previous block, and initialize a new empty one! + // (and following the next passages, increase its size). + + // if (StartAddress < Address && Type != 2) + if (Type == UMA_ROM && StartAddress + Size > Address) + { + /* We are inside ROM, do nothing */ + } + else if (Type == UMA_ROM && StartAddress + Size <= Address) + { + /* We finished a ROM descriptor */ + + /* Create descriptor */ + UmaDesc = CreateUmaDescriptor(NULL, StartAddress, Size, Type); + if (!UmaDesc) return FALSE; + + StartAddress = 0; + // goto Restart; + } + else if (Type != UMA_ROM) + { + Size += Increment; + } + } +#endif + +Restart: + /// if (Address >= 0xE0000) { Increment = 0x10000; } + + if (StartAddress == 0) + { + /* Initialize data for a new descriptor */ + StartAddress = Address; + Size = 0; + Type = UMA_FREE; + + PrevRomAddress = 0; + PrevRomSize = 0; + } + + if (*(PUSHORT)REAL_TO_PHYS(Address) == OPTION_ROM_SIGNATURE) + { + /* + * If this is an adapter ROM (Start: C8000, End: E0000), + * its reported size is stored in byte 2 of the ROM. + * + * If this is an expansion ROM (Start: E0000, End: F0000), + * its real length is 64kB. + */ + RomSize = *(PUCHAR)REAL_TO_PHYS(Address + 2) * 512; + // if (Address >= 0xE0000) RomSize = 0x10000; + if (Address >= 0xE0000) { RomSize = 0x10000; Increment = RomSize; } + + DPRINT1("ROM present @ address 0x%p\n", Address); + + if (StartAddress != 0 && Size != 0 && + StartAddress + Size <= Address) + { + /* Finish old descriptor */ + UmaDesc = CreateUmaDescriptor(NULL, StartAddress, Size, Type); + if (!UmaDesc) return FALSE; + } + + /* + * We may have overlapping ROMs, when PrevRomAddress + PrevRomSize > RomAddress. + * They must be put inside the same UMA descriptor. + */ + if (PrevRomAddress + PrevRomSize > /*Rom*/Address) + { + // Overlapping ROM + + /* + * PrevRomAddress remains the same, but adjust + * PrevRomSize (ROM descriptors merging). + */ + PrevRomSize = max(PrevRomSize, RomSize + Address - PrevRomAddress); + + // FIX: Confirm that the start address is really + // the one of the previous descriptor. + StartAddress = PrevRomAddress; + Size = PrevRomSize; + Type = UMA_ROM; + } + else + { + // Non-overlapping ROM + + PrevRomAddress = Address; + PrevRomSize = RomSize; + + /* Initialize a new descriptor. We will create it when it's OK */ + StartAddress = Address; + Size = RomSize; + Type = UMA_ROM; + // continue; + } + } +#if 1 // FIXME: ...or there?? + else + { + // We may be looping inside a ROM block, if: Type == 2 and: + // (StartAddress <= Address &&) StartAddress + Size > Address. + // In this case, do nothing (do not increase size either) + // But if we are now outside of a ROM block, then we need + // to create the previous block, and initialize a new empty one! + // (and following the next passages, increase its size). + + // if (StartAddress < Address && Type != UMA_ROM) + if (Type == UMA_ROM && StartAddress + Size > Address) + { + // We are inside ROM, do nothing + } + else if (Type == UMA_ROM && StartAddress + Size <= Address) + { + // We finished a ROM descriptor. + + /* Create descriptor */ + UmaDesc = CreateUmaDescriptor(NULL, StartAddress, Size, Type); + if (!UmaDesc) return FALSE; + + StartAddress = 0; + goto Restart; + } + else if (Type != UMA_ROM) + { + Size += Increment; + } + } +#endif + + // Fixed incroment; we may encounter again overlapping ROMs, etc. + // Address += Increment; + } + + if (StartAddress != 0 && Size != 0) + { + /* Create descriptor */ + UmaDesc = CreateUmaDescriptor(NULL, StartAddress, Size, Type); + if (!UmaDesc) return FALSE; + + StartAddress = 0; + } + + } + } + } + + return TRUE; +} + +VOID UmaMgrCleanup(VOID) +{ + PUMA_DESCRIPTOR UmaDesc; + + while (!IsListEmpty(&UmaDescriptorList)) + { + UmaDesc = (PUMA_DESCRIPTOR)CONTAINING_RECORD(UmaDescriptorList.Flink, UMA_DESCRIPTOR, Entry); + FreeUmaDescriptor(UmaDesc); + } +} + +/* EOF */ Propchange: trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.c ------------------------------------------------------------------------------ svn:eol-style = native Added: trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.h (added) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.h [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -0,0 +1,34 @@ +/* + * COPYRIGHT: GPLv2+ - See COPYING in the top level directory + * PROJECT: ReactOS Virtual DOS Machine + * FILE: umamgr.h + * PURPOSE: Upper Memory Area Manager + * PROGRAMMERS: Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) + */ + +#ifndef _UMAMGR_H_ +#define _UMAMGR_H_ + +/* DEFINITIONS ****************************************************************/ + +typedef enum +{ + UMA_FREE = 0, // Free RAM block + UMA_SYSTEM, // System memory (eg. VGA memory, etc...) + UMA_ROM, // ROM block + UMA_UMB, // Upper memory block + UMA_VDD // VDD-reserved block +} UMA_DESC_TYPE; + +/* FUNCTIONS ******************************************************************/ + +BOOLEAN UmaDescReserve(IN OUT PUSHORT UmbSegment, IN OUT PUSHORT Size); +BOOLEAN UmaDescRelease(IN USHORT UmbSegment); +BOOLEAN UmaDescReallocate(IN USHORT UmbSegment, IN OUT PUSHORT Size); + +BOOLEAN UmaMgrInitialize(VOID); +VOID UmaMgrCleanup(VOID); + +#endif // _UMAMGR_H_ + +/* EOF */ Propchange: trunk/reactos/subsystems/mvdm/ntvdm/bios/umamgr.h ------------------------------------------------------------------------------ svn:eol-style = native Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.c [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -1658,19 +1658,25 @@ case 0x02: { Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; - setAL(DosUmbLinked ? 0x01 : 0x00); + setAL(SysVars->UmbLinked ? 0x01 : 0x00); break; } /* Set UMB link state */ case 0x03: { + BOOLEAN Success; + if (getBX()) - DosLinkUmb(); + Success = DosLinkUmb(); else - DosUnlinkUmb(); - - Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; + Success = DosUnlinkUmb(); + + if (Success) + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; + else + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; } @@ -2306,7 +2312,6 @@ /* Initialize the list of lists */ SysVars = &DosData->SysVars; RtlZeroMemory(SysVars, sizeof(*SysVars)); - SysVars->FirstMcb = FIRST_MCB_SEGMENT; SysVars->FirstSft = MAKELONG(DOS_DATA_OFFSET(Sft), DOS_DATA_SEGMENT); SysVars->CurrentDirs = MAKELONG(DOS_DATA_OFFSET(CurrentDirectories), DOS_DATA_SEGMENT); @@ -2397,9 +2402,6 @@ /* Set the current PSP to the system PSP */ Sda->CurrentPsp = SYSTEM_PSP; - /* Set the initial allocation strategy to "best fit" */ - Sda->AllocStrategy = DOS_ALLOC_BEST_FIT; - /* Initialize the SFT */ Sft = (PDOS_SFT)FAR_POINTER(SysVars->FirstSft); Sft->Link = 0xFFFFFFFF; @@ -2451,11 +2453,14 @@ XmsInitialize(); /* Load the EMS driver */ - if (!EmsDrvInitialize(EMS_TOTAL_PAGES)) + if (!EmsDrvInitialize(EMS_SEGMENT, EMS_TOTAL_PAGES)) { DPRINT1("Could not initialize EMS. EMS will not be available.\n" - "Try reducing the number of EMS pages.\n"); + "Page frame segment or number of EMS pages invalid.\n"); } + + /* Finally initialize the UMBs */ + DosInitializeUmb(); return TRUE; } Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.h [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -34,7 +34,7 @@ #define DOS_DATA_OFFSET(x) FIELD_OFFSET(DOS_DATA, x) -#define SYSTEM_ENV_BLOCK 0x600 // FIXME: Should be dynamic +#define SYSTEM_ENV_BLOCK 0x600 // FIXME: Should be dynamically initialized! #define SYSTEM_PSP 0x0008 @@ -96,9 +96,9 @@ BYTE UseDwordMoves; // 0x44 WORD ExtMemSize; // 0x45 BYTE Reserved4[0x1C]; // 0x47 - BYTE ChainUMB; // 0x63 - 0/1: UMB chain (un)linked to MCB chain + BYTE UmbLinked; // 0x63 - 0/1: UMB chain (un)linked to MCB chain WORD Reserved5; // 0x64 - WORD UMBChainStart; // 0x66 - Segment of the first UMB MCB + WORD UmbChainStart; // 0x66 - Segment of the first UMB MCB WORD MemAllocScanStart; // 0x68 - Segment where allocation scan starts } DOS_SYSVARS, *PDOS_SYSVARS; Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.c [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -12,15 +12,18 @@ #include "ntvdm.h" #include "emulator.h" +#include "../../memory.h" +#include "bios/umamgr.h" #include "dos.h" #include "dos/dem.h" #include "device.h" -#include "../../memory.h" #include "emsdrv.h" -#define EMS_DEVICE_NAME "EMMXXXX0" +#define EMS_DEVICE_NAME "EMMXXXX0" + +#define EMS_SEGMENT_SIZE ((EMS_PHYSICAL_PAGES * EMS_PAGE_SIZE) >> 4) /* PRIVATE VARIABLES **********************************************************/ @@ -30,10 +33,51 @@ static PEMS_PAGE PageTable = NULL; static EMS_HANDLE HandleTable[EMS_MAX_HANDLES]; static PVOID Mapping[EMS_PHYSICAL_PAGES] = { NULL }; +static PVOID MappingBackup[EMS_PHYSICAL_PAGES] = { NULL }; static ULONG EmsTotalPages = 0; static PVOID EmsMemory = NULL; +static USHORT EmsSegment = EMS_SEGMENT; /* PRIVATE FUNCTIONS **********************************************************/ + +static VOID InitHandlesTable(VOID) +{ + ULONG i; + + for (i = 0; i < EMS_MAX_HANDLES; i++) + { + HandleTable[i].Allocated = FALSE; + HandleTable[i].PageCount = 0; + InitializeListHead(&HandleTable[i].PageList); + } +} + +static PEMS_HANDLE CreateHandle(PUSHORT Handle) +{ + PEMS_HANDLE HandleEntry; + ULONG i; + + for (i = 0; i < EMS_MAX_HANDLES; i++) + { + HandleEntry = &HandleTable[i]; + if (!HandleEntry->Allocated) + { + *Handle = i; + break; + } + } + + if (i == EMS_MAX_HANDLES) return NULL; + HandleEntry->Allocated = TRUE; + + return HandleEntry; +} + +static VOID FreeHandle(PEMS_HANDLE HandleEntry) +{ + HandleEntry->Allocated = FALSE; + HandleEntry->PageCount = 0; +} static inline PEMS_HANDLE GetHandleRecord(USHORT Handle) { @@ -41,15 +85,18 @@ return &HandleTable[Handle]; } +static inline BOOLEAN ValidateHandle(PEMS_HANDLE HandleEntry) +{ + return (HandleEntry != NULL && HandleEntry->Allocated); +} + static USHORT EmsFree(USHORT Handle) { PLIST_ENTRY Entry; PEMS_HANDLE HandleEntry = GetHandleRecord(Handle); - if (HandleEntry == NULL || !HandleEntry->Allocated) - { + if (!ValidateHandle(HandleEntry)) return EMS_STATUS_INVALID_HANDLE; - } for (Entry = HandleEntry->PageList.Flink; Entry != &HandleEntry->PageList; @@ -62,9 +109,9 @@ RtlClearBits(&AllocBitmap, PageNumber, 1); } - HandleEntry->Allocated = FALSE; - HandleEntry->PageCount = 0; InitializeListHead(&HandleEntry->PageList); + + FreeHandle(HandleEntry); return EMS_STATUS_OK; } @@ -76,18 +123,8 @@ if (NumPages == 0) return EMS_STATUS_ZERO_PAGES; - for (i = 0; i < EMS_MAX_HANDLES; i++) - { - HandleEntry = &HandleTable[i]; - if (!HandleEntry->Allocated) - { - *Handle = i; - break; - } - } - - if (i == EMS_MAX_HANDLES) return EMS_STATUS_NO_MORE_HANDLES; - HandleEntry->Allocated = TRUE; + HandleEntry = CreateHandle(Handle); + if (!HandleEntry) return EMS_STATUS_NO_MORE_HANDLES; while (HandleEntry->PageCount < NumPages) { @@ -120,13 +157,13 @@ return EMS_STATUS_OK; } -static PEMS_PAGE GetLogicalPage(PEMS_HANDLE Handle, USHORT LogicalPage) -{ - PLIST_ENTRY Entry = Handle->PageList.Flink; +static PEMS_PAGE GetLogicalPage(PEMS_HANDLE HandleEntry, USHORT LogicalPage) +{ + PLIST_ENTRY Entry = HandleEntry->PageList.Flink; while (LogicalPage) { - if (Entry == &Handle->PageList) return NULL; + if (Entry == &HandleEntry->PageList) return NULL; LogicalPage--; Entry = Entry->Flink; } @@ -139,7 +176,12 @@ PEMS_PAGE PageEntry; PEMS_HANDLE HandleEntry = GetHandleRecord(Handle); - if (PhysicalPage >= EMS_PHYSICAL_PAGES) return EMS_STATUS_INV_PHYSICAL_PAGE; + if (!ValidateHandle(HandleEntry)) + return EMS_STATUS_INVALID_HANDLE; + + if (PhysicalPage >= EMS_PHYSICAL_PAGES) + return EMS_STATUS_INV_PHYSICAL_PAGE; + if (LogicalPage == 0xFFFF) { /* Unmap */ @@ -147,11 +189,6 @@ return EMS_STATUS_OK; } - if (HandleEntry == NULL || !HandleEntry->Allocated) - { - return EMS_STATUS_INVALID_HANDLE; - } - PageEntry = GetLogicalPage(HandleEntry, LogicalPage); if (!PageEntry) return EMS_STATUS_INV_LOGICAL_PAGE; @@ -162,8 +199,6 @@ static VOID WINAPI EmsIntHandler(LPWORD Stack) { - static PVOID MappingBackup[EMS_PHYSICAL_PAGES] = { NULL }; - switch (getAH()) { /* Get Manager Status */ @@ -177,11 +212,11 @@ case 0x41: { setAH(EMS_STATUS_OK); - setBX(EMS_SEGMENT); - break; - } - - /* Get Number Of Pages */ + setBX(EmsSegment); + break; + } + + /* Get Number of Pages */ case 0x42: { setAH(EMS_STATUS_OK); @@ -190,7 +225,7 @@ break; } - /* Get Handle And Allocate Memory */ + /* Get Handle and Allocate Memory */ case 0x43: { USHORT Handle; @@ -208,7 +243,7 @@ break; } - /* Release Handle And Memory */ + /* Release Handle and Memory */ case 0x45: { setAH(EmsFree(getDX())); @@ -226,14 +261,14 @@ /* Save Page Map */ case 0x47: { - RtlCopyMemory(MappingBackup, Mapping, sizeof(PVOID) * EMS_PHYSICAL_PAGES); + RtlCopyMemory(MappingBackup, Mapping, sizeof(Mapping)); break; } /* Restore Page Map */ case 0x48: { - RtlCopyMemory(Mapping, MappingBackup, sizeof(PVOID) * EMS_PHYSICAL_PAGES); + RtlCopyMemory(Mapping, MappingBackup, sizeof(Mapping)); break; } @@ -241,7 +276,8 @@ case 0x53: { PEMS_HANDLE HandleEntry = GetHandleRecord(getDX()); - if (HandleEntry == NULL || !HandleEntry->Allocated) + + if (!ValidateHandle(HandleEntry)) { setAL(EMS_STATUS_INVALID_HANDLE); break; @@ -286,7 +322,7 @@ /* Expanded memory */ HandleEntry = GetHandleRecord(Data->SourceHandle); - if (HandleEntry == NULL || !HandleEntry->Allocated) + if (!ValidateHandle(HandleEntry)) { setAL(EMS_STATUS_INVALID_HANDLE); break; @@ -315,7 +351,7 @@ /* Expanded memory */ HandleEntry = GetHandleRecord(Data->DestHandle); - if (HandleEntry == NULL || !HandleEntry->Allocated) + if (!ValidateHandle(HandleEntry)) { setAL(EMS_STATUS_INVALID_HANDLE); break; @@ -405,7 +441,7 @@ static VOID FASTCALL EmsReadMemory(ULONG Address, PVOID Buffer, ULONG Size) { ULONG i; - ULONG RelativeAddress = Address - TO_LINEAR(EMS_SEGMENT, 0); + ULONG RelativeAddress = Address - TO_LINEAR(EmsSegment, 0); ULONG FirstPage = RelativeAddress / EMS_PAGE_SIZE; ULONG LastPage = (RelativeAddress + Size - 1) / EMS_PAGE_SIZE; ULONG Offset, Length; @@ -425,7 +461,7 @@ static BOOLEAN FASTCALL EmsWriteMemory(ULONG Address, PVOID Buffer, ULONG Size) { ULONG i; - ULONG RelativeAddress = Address - TO_LINEAR(EMS_SEGMENT, 0); + ULONG RelativeAddress = Address - TO_LINEAR(EmsSegment, 0); ULONG FirstPage = RelativeAddress / EMS_PAGE_SIZE; ULONG LastPage = (RelativeAddress + Size - 1) / EMS_PAGE_SIZE; ULONG Offset, Length; @@ -444,8 +480,7 @@ return TRUE; } - -WORD NTAPI EmsDrvDispatchIoctlRead(PDOS_DEVICE_NODE Device, DWORD Buffer, PWORD Length) +static WORD NTAPI EmsDrvDispatchIoctlRead(PDOS_DEVICE_NODE Device, DWORD Buffer, PWORD Length) { // TODO: NOT IMPLEMENTED UNIMPLEMENTED; @@ -455,22 +490,26 @@ /* PUBLIC FUNCTIONS ***********************************************************/ -BOOLEAN EmsDrvInitialize(ULONG TotalPages) -{ - ULONG i; - - for (i = 0; i < EMS_MAX_HANDLES; i++) - { - HandleTable[i].Allocated = FALSE; - HandleTable[i].PageCount = 0; - InitializeListHead(&HandleTable[i].PageList); - } +BOOLEAN EmsDrvInitialize(USHORT Segment, ULONG TotalPages) +{ + USHORT Size; + + /* Try to allocate our page table in UMA at the given segment */ + EmsSegment = (Segment != 0 ? Segment : EMS_SEGMENT); + Size = EMS_SEGMENT_SIZE; // Size in paragraphs + if (!UmaDescReserve(&EmsSegment, &Size)) return FALSE; + + InitHandlesTable(); EmsTotalPages = TotalPages; BitmapBuffer = RtlAllocateHeap(RtlGetProcessHeap(), HEAP_ZERO_MEMORY, ((TotalPages + 31) / 32) * sizeof(ULONG)); - if (BitmapBuffer == NULL) return FALSE; + if (BitmapBuffer == NULL) + { + UmaDescRelease(EmsSegment); + return FALSE; + } RtlInitializeBitMap(&AllocBitmap, BitmapBuffer, TotalPages); @@ -482,6 +521,7 @@ RtlFreeHeap(RtlGetProcessHeap(), 0, BitmapBuffer); BitmapBuffer = NULL; + UmaDescRelease(EmsSegment); return FALSE; } @@ -493,10 +533,11 @@ RtlFreeHeap(RtlGetProcessHeap(), 0, BitmapBuffer); BitmapBuffer = NULL; + UmaDescRelease(EmsSegment); return FALSE; } - MemInstallFastMemoryHook((PVOID)TO_LINEAR(EMS_SEGMENT, 0), + MemInstallFastMemoryHook((PVOID)TO_LINEAR(EmsSegment, 0), EMS_PHYSICAL_PAGES * EMS_PAGE_SIZE, EmsReadMemory, EmsWriteMemory); @@ -520,7 +561,7 @@ /* Delete the device */ DosDeleteDevice(Node); - MemRemoveFastMemoryHook((PVOID)TO_LINEAR(EMS_SEGMENT, 0), + MemRemoveFastMemoryHook((PVOID)TO_LINEAR(EmsSegment, 0), EMS_PHYSICAL_PAGES * EMS_PAGE_SIZE); if (EmsMemory) @@ -540,4 +581,6 @@ RtlFreeHeap(RtlGetProcessHeap(), 0, BitmapBuffer); BitmapBuffer = NULL; } -} + + UmaDescRelease(EmsSegment); +} Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/emsdrv.h [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -13,7 +13,7 @@ #define EMS_VERSION_NUM 0x40 #define EMS_INTERRUPT_NUM 0x67 -#define EMS_SEGMENT 0xD000 +#define EMS_SEGMENT 0xD000 // Default segment #define EMS_MAX_HANDLES 16 #define EMS_PAGE_BITS 14 #define EMS_PAGE_SIZE (1 << EMS_PAGE_BITS) @@ -72,9 +72,11 @@ #pragma pack(pop) -#endif - /* FUNCTIONS ******************************************************************/ -BOOLEAN EmsDrvInitialize(ULONG TotalPages); +BOOLEAN EmsDrvInitialize(USHORT Segment, ULONG TotalPages); VOID EmsDrvCleanup(VOID); + +#endif // _EMSDRV_H_ + +/* EOF */ Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/himem.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/himem.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/himem.c [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -2,8 +2,46 @@ * COPYRIGHT: GPLv2+ - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: himem.c - * PURPOSE: DOS XMS Driver + * PURPOSE: DOS XMS Driver and UMB Provider * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> + * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) + * + * DOCUMENTATION: Official specifications: + * XMS v2.0: http://www.phatcode.net/res/219/files/xms20.txt + * XMS v3.0: http://www.phatcode.net/res/219/files/xms30.txt + * + * About the implementation of UMBs in DOS: + * ---------------------------------------- + * DOS needs a UMB provider to be able to use chunks of RAM in the C000-EFF0 + * memory region. A UMB provider detects regions of memory that do not contain + * any ROMs or other system mapped area such as video RAM. + * + * Where can UMB providers be found? + * + * The XMS specification (see himem.c) provides three APIs to create, free, and + * resize UMB blocks. As such, DOS performs calls into the XMS driver chain to + * request UMB blocks and include them into the DOS memory arena. + * However, is it only the HIMEM driver (implementing the XMS specification) + * which can provide UMBs? It appears that this is not necessarily the case: + * for example the MS HIMEM versions do not implement the UMB APIs; instead + * it is the EMS driver (EMM386) which provides them, by hooking into the XMS + * driver chain (see https://support.microsoft.com/en-us/kb/95555 : "MS-DOS 5.0 + * and later EMM386.EXE can also be configured to provide UMBs according to the + * XMS. This causes EMM386.EXE to be a provider of the UMB portion of the XMS."). + * + * Some alternative replacements of HIMEM/EMM386 (for example in FreeDOS) + * implement everything inside only one driver (XMS+EMS+UMB provider). + * Finally there are some UMB providers that exist separately of XMS and EMS + * drivers (for example, UMBPCI): they use hardware-specific tricks to discover + * and provide UMBs. + * + * For more details, see: + * http://www.freedos.org/technotes/technote/txt/169.txt + * http://www.freedos.org/technotes/technote/txt/202.txt + * http://www.uncreativelabs.net/textfiles/system/UMB.TXT + * + * This DOS XMS Driver provides the UMB APIs that are implemented on top + * of the internal Upper Memory Area Manager, in umamgr.c */ /* INCLUDES *******************************************************************/ @@ -14,14 +52,10 @@ #include "emulator.h" #include "cpu/bop.h" #include "../../memory.h" -#include "io.h" -#include "hardware/ps2.h" - -#include "dos.h" -#include "dos/dem.h" +#include "bios/umamgr.h" + #include "device.h" #include "himem.h" -#include "memory.h" #define XMS_DEVICE_NAME "XMSXXXX0" @@ -30,7 +64,8 @@ /* PRIVATE VARIABLES **********************************************************/ -static const BYTE EntryProcedure[] = { +static const BYTE EntryProcedure[] = +{ 0xEB, // jmp short +0x03 0x03, 0x90, // nop @@ -49,7 +84,24 @@ static ULONG BitmapBuffer[(XMS_BLOCKS + 31) / 32]; /* - * Flag used by Global Enable/Disable A20 functions, so that they don't + * This value is associated to HIMEM's "/HMAMIN=" switch. It indicates the + * minimum account of space in the HMA a program can use, and is used in + * conjunction with the "Request HMA" function. + * + * NOTE: The "/HMAMIN=" value is in kilo-bytes, whereas HmaMinSize is in bytes. + * + * Default value: 0. This causes the HMA to be allocated on a first come, + * first served basis. + */ +static WORD HmaMinSize = 0; +/* + * Flag used by "Request/Release HMA" functions, which indicates + * whether the HMA was reserved or not. + */ +static BOOLEAN IsHmaReserved = FALSE; + +/* + * Flag used by "Global Enable/Disable A20" functions, so that they don't * need to re-change the state of A20 if it was already enabled/disabled. */ static BOOLEAN IsA20Enabled = FALSE; @@ -66,7 +118,7 @@ */ static LONG A20EnableCount = 0; -/* HELPERS FOR A20 LINE *******************************************************/ +/* A20 LINE HELPERS ***********************************************************/ static VOID XmsLocalEnableA20(VOID) { @@ -86,6 +138,8 @@ static VOID XmsLocalDisableA20(VOID) { + UCHAR Result = XMS_STATUS_SUCCESS; + /* Disable A20 only if we can do so, otherwise make the caller believe we disabled it */ if (!CanChangeA20) goto Quit; @@ -95,15 +149,18 @@ --A20EnableCount; /* The count is zero so disable A20 */ - if (A20EnableCount == 0) EmulatorSetA20(FALSE); + if (A20EnableCount == 0) + EmulatorSetA20(FALSE); // Result = XMS_STATUS_SUCCESS; + else + Result = XMS_STATUS_A20_STILL_ENABLED; Quit: setAX(0x0001); /* Line successfully disabled */ - setBL(XMS_STATUS_SUCCESS); + setBL(Result); return; Fail: - setAX(0x0000); /* Line failed to be enabled */ + setAX(0x0000); /* Line failed to be disabled */ setBL(XMS_STATUS_A20_ERROR); return; } @@ -117,6 +174,11 @@ Entry = &HandleTable[Handle - 1]; return Entry->Size ? Entry : NULL; +} + +static inline BOOLEAN ValidateHandle(PXMS_HANDLE HandleEntry) +{ + return (HandleEntry != NULL && HandleEntry->Handle != 0); } static WORD XmsGetLargestFreeBlock(VOID) @@ -141,7 +203,7 @@ return Result; } -static CHAR XmsAlloc(WORD Size, PWORD Handle) +static UCHAR XmsAlloc(WORD Size, PWORD Handle) { BYTE i; PXMS_HANDLE HandleEntry; @@ -210,13 +272,16 @@ return XMS_STATUS_OUT_OF_MEMORY; } -static CHAR XmsFree(WORD Handle) +static UCHAR XmsFree(WORD Handle) { DWORD BlockNumber; PXMS_HANDLE HandleEntry = GetHandleRecord(Handle); - if (HandleEntry == NULL || HandleEntry->Handle == 0) return XMS_STATUS_INVALID_HANDLE; - if (HandleEntry->LockCount) return XMS_STATUS_LOCKED; + if (!ValidateHandle(HandleEntry)) + return XMS_STATUS_INVALID_HANDLE; + + if (HandleEntry->LockCount) + return XMS_STATUS_LOCKED; BlockNumber = (HandleEntry->Address - XMS_ADDRESS) / XMS_BLOCK_SIZE; RtlClearBits(&AllocBitmap, BlockNumber, HandleEntry->Size); @@ -227,12 +292,15 @@ return XMS_STATUS_SUCCESS; } -static CHAR XmsLock(WORD Handle, PDWORD Address) +static UCHAR XmsLock(WORD Handle, PDWORD Address) { PXMS_HANDLE HandleEntry = GetHandleRecord(Handle); - if (HandleEntry == NULL) return XMS_STATUS_INVALID_HANDLE; - if (HandleEntry->LockCount == 0xFF) return XMS_STATUS_LOCK_OVERFLOW; + if (!ValidateHandle(HandleEntry)) + return XMS_STATUS_INVALID_HANDLE; + + if (HandleEntry->LockCount == 0xFF) + return XMS_STATUS_LOCK_OVERFLOW; /* Increment the lock count */ HandleEntry->LockCount++; @@ -241,12 +309,15 @@ return XMS_STATUS_SUCCESS; } -static CHAR XmsUnlock(WORD Handle) +static UCHAR XmsUnlock(WORD Handle) { PXMS_HANDLE HandleEntry = GetHandleRecord(Handle); - if (HandleEntry == NULL) return XMS_STATUS_INVALID_HANDLE; - if (!HandleEntry->LockCount) return XMS_STATUS_NOT_LOCKED; + if (!ValidateHandle(HandleEntry)) + return XMS_STATUS_INVALID_HANDLE; + + if (!HandleEntry->LockCount) + return XMS_STATUS_NOT_LOCKED; /* Decrement the lock count */ HandleEntry->LockCount--; @@ -267,6 +338,57 @@ break; } + /* Request HMA */ + case 0x01: + { + /* Check whether HMA is already reserved */ + if (IsHmaReserved) + { + /* It is, bail out */ + setAX(0x0000); + setBL(XMS_STATUS_HMA_IN_USE); + break; + } + + // NOTE: We implicitely suppose that we always have HMA. + // If not, we should fail there with the XMS_STATUS_HMA_DOES_NOT_EXIST + // error code. + + /* Check whether the requested size is above the minimal allowed one */ + if (getDX() < HmaMinSize) + { + /* It is not, bail out */ + setAX(0x0000); + setBL(XMS_STATUS_HMA_MIN_SIZE); + break; + } + + /* Reserve it */ + IsHmaReserved = TRUE; + setAX(0x0001); + setBL(XMS_STATUS_SUCCESS); + break; + } + + /* Release HMA */ + case 0x02: + { + /* Check whether HMA was reserved */ + if (!IsHmaReserved) + { + /* It was not, bail out */ + setAX(0x0000); + setBL(XMS_STATUS_HMA_NOT_ALLOCATED); + break; + } + + /* Release it */ + IsHmaReserved = FALSE; + setAX(0x0001); + setBL(XMS_STATUS_SUCCESS); + break; + } + /* Global Enable A20 */ case 0x03: { @@ -274,7 +396,7 @@ if (!IsA20Enabled) { XmsLocalEnableA20(); - if (getAX() != 1) + if (getAX() != 0x0001) { /* XmsLocalEnableA20 failed and already set AX and BL to their correct values */ break; @@ -291,21 +413,24 @@ /* Global Disable A20 */ case 0x04: { + UCHAR Result = XMS_STATUS_SUCCESS; + /* Disable A20 if needed */ if (IsA20Enabled) { XmsLocalDisableA20(); - if (getAX() != 1) + if (getAX() != 0x0001) { /* XmsLocalDisableA20 failed and already set AX and BL to their correct values */ break; } IsA20Enabled = FALSE; + Result = getBL(); } setAX(0x0001); /* Line successfully disabled */ - setBL(XMS_STATUS_SUCCESS); + setBL(Result); break; } @@ -338,7 +463,12 @@ { setAX(XmsGetLargestFreeBlock()); setDX(FreeBlocks); - setBL(XMS_STATUS_SUCCESS); + + if (FreeBlocks > 0) + setBL(XMS_STATUS_SUCCESS); + else + setBL(XMS_STATUS_OUT_OF_MEMORY); + break; } @@ -346,7 +476,7 @@ case 0x09: { WORD Handle; - CHAR Result = XmsAlloc(getDX(), &Handle); + UCHAR Result = XmsAlloc(getDX(), &Handle); if (Result >= 0) { @@ -365,11 +495,10 @@ /* Free Extended Memory Block */ case 0x0A: { - CHAR Result = XmsFree(getDX()); + UCHAR Result = XmsFree(getDX()); setAX(Result >= 0); setBL(Result); - break; } @@ -378,60 +507,61 @@ { PVOID SourceAddress, DestAddress; PXMS_COPY_DATA CopyData = (PXMS_COPY_DATA)SEG_OFF_TO_PTR(getDS(), getSI()); + PXMS_HANDLE HandleEntry; if (CopyData->SourceHandle) { - PXMS_HANDLE Entry = GetHandleRecord(CopyData->SourceHandle); - if (!Entry) + HandleEntry = GetHandleRecord(CopyData->SourceHandle); + if (!ValidateHandle(HandleEntry)) { setAX(0); setBL(XMS_STATUS_BAD_SRC_HANDLE); break; } - if (CopyData->SourceOffset >= Entry->Size * XMS_BLOCK_SIZE) + if (CopyData->SourceOffset >= HandleEntry->Size * XMS_BLOCK_SIZE) { setAX(0); setBL(XMS_STATUS_BAD_SRC_OFFSET); } - SourceAddress = (PVOID)REAL_TO_PHYS(Entry->Address + CopyData->SourceOffset); + SourceAddress = (PVOID)REAL_TO_PHYS(HandleEntry->Address + CopyData->SourceOffset); } else { /* The offset is actually a 16-bit segment:offset pointer */ - SourceAddress = SEG_OFF_TO_PTR(HIWORD(CopyData->SourceOffset), - LOWORD(CopyData->SourceOffset)); + SourceAddress = FAR_POINTER(CopyData->SourceOffset); } if (CopyData->DestHandle) { - PXMS_HANDLE Entry = GetHandleRecord(CopyData->DestHandle); - if (!Entry) + HandleEntry = GetHandleRecord(CopyData->DestHandle); + if (!ValidateHandle(HandleEntry)) { setAX(0); setBL(XMS_STATUS_BAD_DEST_HANDLE); break; } - if (CopyData->DestOffset >= Entry->Size * XMS_BLOCK_SIZE) + if (CopyData->DestOffset >= HandleEntry->Size * XMS_BLOCK_SIZE) { setAX(0); setBL(XMS_STATUS_BAD_DEST_OFFSET); } - DestAddress = (PVOID)REAL_TO_PHYS(Entry->Address + CopyData->DestOffset); + DestAddress = (PVOID)REAL_TO_PHYS(HandleEntry->Address + CopyData->DestOffset); } else { /* The offset is actually a 16-bit segment:offset pointer */ - DestAddress = SEG_OFF_TO_PTR(HIWORD(CopyData->DestOffset), - LOWORD(CopyData->DestOffset)); - } + DestAddress = FAR_POINTER(CopyData->DestOffset); + } + + /* Perform the move */ + RtlMoveMemory(DestAddress, SourceAddress, CopyData->Count); setAX(1); setBL(XMS_STATUS_SUCCESS); - RtlMoveMemory(DestAddress, SourceAddress, CopyData->Count); break; } @@ -439,7 +569,7 @@ case 0x0C: { DWORD Address; - CHAR Result = XmsLock(getDX(), &Address); + UCHAR Result = XmsLock(getDX(), &Address); if (Result >= 0) { @@ -461,108 +591,133 @@ /* Unlock Extended Memory Block */ case 0x0D: { - CHAR Result = XmsUnlock(getDX()); + UCHAR Result = XmsUnlock(getDX()); setAX(Result >= 0); setBL(Result); - break; } /* Get Handle Information */ case 0x0E: { - PXMS_HANDLE Entry = GetHandleRecord(getDX()); - - if (Entry && Entry->Handle != 0) - { - UINT i; - UCHAR Handles = 0; - - for (i = 0; i < XMS_MAX_HANDLES; i++) - { - if (HandleTable[i].Handle == 0) Handles++; - } - - setAX(1); - setBH(Entry->LockCount); - setBL(Handles); - setDX(Entry->Size); - } - else + PXMS_HANDLE HandleEntry = GetHandleRecord(getDX()); + UINT i; + UCHAR Handles = 0; + + if (!ValidateHandle(HandleEntry)) { setAX(0); setBL(XMS_STATUS_INVALID_HANDLE); - } - - break; - } - - /* Allocate UMB */ + break; + } + + for (i = 0; i < XMS_MAX_HANDLES; i++) + { + if (HandleTable[i].Handle == 0) Handles++; + } + + setAX(1); + setBH(HandleEntry->LockCount); + setBL(Handles); + setDX(HandleEntry->Size); + break; + } + + /* Reallocate Extended Memory Block */ + case 0x0F: + { + PXMS_HANDLE HandleEntry = GetHandleRecord(getDX()); + WORD NewSize = getBX(); + + if (!ValidateHandle(HandleEntry)) + { + setAX(0); + setBL(XMS_STATUS_INVALID_HANDLE); + break; + } + + if (HandleEntry->LockCount) + { + setAX(0); + setBL(XMS_STATUS_LOCKED); + break; + } + + /* Check for reduction or enlargement */ + if (NewSize < HandleEntry->Size) + { + /* Reduction, the easy case: just update the size */ + HandleEntry->Size = NewSize; + } + else + { + /* Enlargement: we need to reallocate elsewhere */ + UNIMPLEMENTED; + setAX(0); + setBL(XMS_STATUS_NOT_IMPLEMENTED); + break; + } + + setAX(1); + setBL(XMS_STATUS_SUCCESS); + break; + } + + /* Request UMB */ case 0x10: { - WORD Segment; - WORD MaxAvailable; - BYTE OldAllocStrategy = Sda->AllocStrategy; - BOOLEAN OldLinkState = DosUmbLinked; - - DosLinkUmb(); - Sda->AllocStrategy = DOS_ALLOC_HIGH | DOS_ALLOC_BEST_FIT; - Segment = DosAllocateMemory(getDX(), &MaxAvailable); - - if (Segment) - { - setAX(1); + BOOLEAN Result; + USHORT Segment = 0x0000; /* No preferred segment */ + USHORT Size = getDX(); /* Size is in paragraphs */ + + Result = UmaDescReserve(&Segment, &Size); + if (Result) setBX(Segment); - } else - { - setAX(0); - setBL(MaxAvailable ? XMS_STATUS_SMALLER_UMB : XMS_STATUS_OUT_OF_UMBS); - setDX(MaxAvailable); - } - - Sda->AllocStrategy = OldAllocStrategy; - if (!OldLinkState) DosUnlinkUmb(); - break; - } - - /* Free UMB */ + setBL(Size > 0 ? XMS_STATUS_SMALLER_UMB : XMS_STATUS_OUT_OF_UMBS); + + setDX(Size); + setAX(Result); + break; + } + + /* Release UMB */ case 0x11: { - if (DosFreeMemory(getDX())) - { - setAX(1); - } - else - { - setAX(0); + BOOLEAN Result; + USHORT Segment = getDX(); + + Result = UmaDescRelease(Segment); + if (!Result) setBL(XMS_STATUS_INVALID_UMB); - } - + + setAX(Result); break; } /* Reallocate UMB */ case 0x12: { - WORD Segment; - WORD MaxAvailable; - - Segment = DosResizeMemory(getDX(), getBX(), &MaxAvailable); - - if (Segment) - { - setAX(1); - } - else - { - setAX(0); - setBL(Sda->LastErrorCode == ERROR_INVALID_HANDLE - ? XMS_STATUS_INVALID_UMB : XMS_STATUS_SMALLER_UMB); - setDX(MaxAvailable); - } - + BOOLEAN Result; + USHORT Segment = getDX(); + USHORT Size = getBX(); /* Size is in paragraphs */ + + Result = UmaDescReallocate(Segment, &Size); + if (!Result) + { + if (Size > 0) + { + setBL(XMS_STATUS_SMALLER_UMB); + setDX(Size); + } + else + { + setBL(XMS_STATUS_INVALID_UMB); + } + } + + setAX(Result); break; } Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/himem.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/himem.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/himem.h [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -2,35 +2,43 @@ * COPYRIGHT: GPLv2+ - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: himem.h - * PURPOSE: DOS XMS Driver + * PURPOSE: DOS XMS Driver and UMB Provider * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> + * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) */ + +#ifndef _HIMEM_H_ +#define _HIMEM_H_ /* DEFINITIONS ****************************************************************/ -#define XMS_ADDRESS 0x110000 -#define XMS_BLOCKS 0x3BC0 +#define XMS_ADDRESS 0x110000 // Just above HMA +#define XMS_BLOCKS 0x3BC0 // XMS_ADDRESS + (XMS_BLOCKS * XMS_BLOCK_SIZE) == 16 MB #define XMS_BLOCK_SIZE 1024 -#define XMS_MAX_HANDLES 16 +#define XMS_MAX_HANDLES 16 // Specification: min 1, max 128, default 32 -#define XMS_STATUS_SUCCESS 0x00 -#define XMS_STATUS_NOT_IMPLEMENTED 0x80 -#define XMS_STATUS_A20_ERROR 0x82 -#define XMS_STATUS_HMA_IN_USE 0x91 -#define XMS_STATUS_OUT_OF_MEMORY 0xA0 -#define XMS_STATUS_OUT_OF_HANDLES 0xA1 -#define XMS_STATUS_INVALID_HANDLE 0xA2 -#define XMS_STATUS_BAD_SRC_HANDLE 0xA3 -#define XMS_STATUS_BAD_DEST_HANDLE 0xA4 -#define XMS_STATUS_BAD_SRC_OFFSET 0xA5 -#define XMS_STATUS_BAD_DEST_OFFSET 0xA6 -#define XMS_STATUS_NOT_LOCKED 0xAA -#define XMS_STATUS_LOCKED 0xAB -#define XMS_STATUS_LOCK_OVERFLOW 0xAC -#define XMS_STATUS_CANNOT_LOCK 0xAD -#define XMS_STATUS_SMALLER_UMB 0xB0 -#define XMS_STATUS_OUT_OF_UMBS 0xB1 -#define XMS_STATUS_INVALID_UMB 0xB2 +#define XMS_STATUS_SUCCESS 0x00 +#define XMS_STATUS_NOT_IMPLEMENTED 0x80 +#define XMS_STATUS_A20_ERROR 0x82 +#define XMS_STATUS_HMA_DOES_NOT_EXIST 0x90 +#define XMS_STATUS_HMA_IN_USE 0x91 +#define XMS_STATUS_HMA_MIN_SIZE 0x92 +#define XMS_STATUS_HMA_NOT_ALLOCATED 0x93 +#define XMS_STATUS_A20_STILL_ENABLED 0x94 +#define XMS_STATUS_OUT_OF_MEMORY 0xA0 +#define XMS_STATUS_OUT_OF_HANDLES 0xA1 +#define XMS_STATUS_INVALID_HANDLE 0xA2 +#define XMS_STATUS_BAD_SRC_HANDLE 0xA3 +#define XMS_STATUS_BAD_DEST_HANDLE 0xA4 +#define XMS_STATUS_BAD_SRC_OFFSET 0xA5 +#define XMS_STATUS_BAD_DEST_OFFSET 0xA6 +#define XMS_STATUS_NOT_LOCKED 0xAA +#define XMS_STATUS_LOCKED 0xAB +#define XMS_STATUS_LOCK_OVERFLOW 0xAC +#define XMS_STATUS_CANNOT_LOCK 0xAD +#define XMS_STATUS_SMALLER_UMB 0xB0 +#define XMS_STATUS_OUT_OF_UMBS 0xB1 +#define XMS_STATUS_INVALID_UMB 0xB2 typedef struct _XMS_HANDLE { @@ -56,3 +64,7 @@ BOOLEAN XmsGetDriverEntry(PDWORD Pointer); VOID XmsInitialize(VOID); VOID XmsCleanup(VOID); + +#endif // _HIMEM_H_ + +/* EOF */ Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/memory.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/memory.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/memory.c [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -4,6 +4,7 @@ * FILE: dos/dos32krnl/memory.c * PURPOSE: DOS32 Memory Manager * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> + * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) */ /* INCLUDES *******************************************************************/ @@ -12,17 +13,61 @@ #include "ntvdm.h" #include "emulator.h" + +#include "bios/umamgr.h" // HACK until we correctly call XMS services for UMBs. #include "dos.h" #include "dos/dem.h" #include "memory.h" #include "process.h" +#include "himem.h" + +// FIXME: Should be dynamically initialized! +#define FIRST_MCB_SEGMENT (SYSTEM_ENV_BLOCK + 0x200) // old value: 0x1000 +#define USER_MEMORY_SIZE (0x9FFE - FIRST_MCB_SEGMENT) + +/* PRIVATE VARIABLES **********************************************************/ /* PUBLIC VARIABLES ***********************************************************/ -BOOLEAN DosUmbLinked = FALSE; - /* PRIVATE FUNCTIONS **********************************************************/ + +static inline BOOLEAN ValidateMcb(PDOS_MCB Mcb) +{ + return (Mcb->BlockType == 'M' || Mcb->BlockType == 'Z'); +} + +/* + * This is a helper function to help us detecting + * when the DOS arena starts to become corrupted. + */ +static VOID DosMemValidate(VOID) +{ + WORD PrevSegment, Segment = SysVars->FirstMcb; + PDOS_MCB CurrentMcb; + + PrevSegment = Segment; + while (TRUE) + { + /* Get a pointer to the MCB */ + CurrentMcb = SEGMENT_TO_MCB(Segment); + + /* Make sure it's valid */ + if (!ValidateMcb(CurrentMcb)) + { + DPRINT1("The DOS memory arena is corrupted! (CurrentMcb = %04x; PreviousMcb = %04X)\n", Segment, PrevSegment); + return; + } + + PrevSegment = Segment; + + /* If this was the last MCB in the chain, quit */ + if (CurrentMcb->BlockType == 'Z') return; + + /* Otherwise, update the segment and continue */ + Segment += CurrentMcb->Size + 1; + } +} static VOID DosCombineFreeBlocks(WORD StartBlock) { @@ -35,6 +80,14 @@ { /* Get a pointer to the next MCB */ NextMcb = SEGMENT_TO_MCB(StartBlock + CurrentMcb->Size + 1); + + /* Make sure it's valid */ + if (!ValidateMcb(NextMcb)) + { + DPRINT1("The DOS memory arena is corrupted!\n"); + // Sda->LastErrorCode = ERROR_ARENA_TRASHED; + return; + } /* Check if the next MCB is free */ if (NextMcb->OwnerPsp == 0) @@ -56,16 +109,19 @@ WORD DosAllocateMemory(WORD Size, WORD *MaxAvailable) { - WORD Result = 0, Segment = FIRST_MCB_SEGMENT, MaxSize = 0; + WORD Result = 0, Segment = SysVars->FirstMcb, MaxSize = 0; PDOS_MCB CurrentMcb; BOOLEAN SearchUmb = FALSE; DPRINT("DosAllocateMemory: Size 0x%04X\n", Size); - if (DosUmbLinked && (Sda->AllocStrategy & (DOS_ALLOC_HIGH | DOS_ALLOC_HIGH_LOW))) + DosMemValidate(); + + if (SysVars->UmbLinked && SysVars->UmbChainStart != 0xFFFF && + (Sda->AllocStrategy & (DOS_ALLOC_HIGH | DOS_ALLOC_HIGH_LOW))) { /* Search UMB first */ - Segment = UMB_START_SEGMENT; + Segment = SysVars->UmbChainStart; SearchUmb = TRUE; } @@ -75,7 +131,7 @@ CurrentMcb = SEGMENT_TO_MCB(Segment); /* Make sure it's valid */ - if (CurrentMcb->BlockType != 'M' && CurrentMcb->BlockType != 'Z') + if (!ValidateMcb(CurrentMcb)) { DPRINT1("The DOS memory arena is corrupted!\n"); Sda->LastErrorCode = ERROR_ARENA_TRASHED; @@ -130,7 +186,7 @@ if ((Result == 0) && SearchUmb && (Sda->AllocStrategy & DOS_ALLOC_HIGH_LOW)) { /* Search low memory */ - Segment = FIRST_MCB_SEGMENT; + Segment = SysVars->FirstMcb; SearchUmb = FALSE; continue; } @@ -144,7 +200,9 @@ Done: - /* If we didn't find a free block, return 0 */ + DosMemValidate(); + + /* If we didn't find a free block, bail out */ if (Result == 0) { Sda->LastErrorCode = ERROR_NOT_ENOUGH_MEMORY; @@ -194,6 +252,9 @@ /* Take ownership of the block */ CurrentMcb->OwnerPsp = Sda->CurrentPsp; + RtlCopyMemory(CurrentMcb->Name, SEGMENT_TO_MCB(Sda->CurrentPsp - 1)->Name, sizeof(CurrentMcb->Name)); + + DosMemValidate(); /* Return the segment of the data portion of the block */ return Result + 1; @@ -209,16 +270,18 @@ BlockData, NewSize); + DosMemValidate(); + /* Make sure this is a valid, allocated block */ - if (BlockData == 0 - || (Mcb->BlockType != 'M' && Mcb->BlockType != 'Z') - || Mcb->OwnerPsp == 0) + if (BlockData == 0 || !ValidateMcb(Mcb) || Mcb->OwnerPsp == 0) { Success = FALSE; Sda->LastErrorCode = ERROR_INVALID_HANDLE; goto Done; } + DosCombineFreeBlocks(SysVars->FirstMcb); + ReturnSize = Mcb->Size; /* Check if we need to expand or contract the block */ @@ -235,10 +298,18 @@ NextSegment = Segment + Mcb->Size + 1; NextMcb = SEGMENT_TO_MCB(NextSegment); + /* Make sure it's valid */ + if (!ValidateMcb(NextMcb)) + { + DPRINT1("The DOS memory arena is corrupted!\n"); + Sda->LastErrorCode = ERROR_ARENA_TRASHED; + return FALSE; + } + /* Make sure the next segment is free */ if (NextMcb->OwnerPsp != 0) { - DPRINT("Cannot expand memory block: next segment is not free!\n"); + DPRINT("Cannot expand memory block 0x%04X: next segment is not free!\n", Segment); Sda->LastErrorCode = ERROR_NOT_ENOUGH_MEMORY; Success = FALSE; goto Done; @@ -252,7 +323,7 @@ if (ReturnSize < NewSize) { - DPRINT("Cannot expand memory block: insufficient free segments available!\n"); + DPRINT("Cannot expand memory block 0x%04X: insufficient free segments available!\n", Segment); Sda->LastErrorCode = ERROR_NOT_ENOUGH_MEMORY; Success = FALSE; goto Done; @@ -269,8 +340,7 @@ if (Mcb->Size > NewSize) { DPRINT("Block too large, reducing size from 0x%04X to 0x%04X\n", - Mcb->Size, - NewSize); + Mcb->Size, NewSize); /* It is, split it into two blocks */ NextMcb = SEGMENT_TO_MCB(Segment + NewSize + 1); @@ -288,8 +358,7 @@ else if (NewSize < Mcb->Size) { DPRINT("Shrinking block from 0x%04X to 0x%04X\n", - Mcb->Size, - NewSize); + Mcb->Size, NewSize); /* Just split the block */ NextSegment = Segment + NewSize + 1; @@ -317,6 +386,8 @@ if (MaxAvailable) *MaxAvailable = ReturnSize; } + DosMemValidate(); + return Success; } @@ -327,8 +398,10 @@ DPRINT("DosFreeMemory: BlockData 0x%04X\n", BlockData); if (BlockData == 0) return FALSE; + DosMemValidate(); + /* Make sure the MCB is valid */ - if (Mcb->BlockType != 'M' && Mcb->BlockType != 'Z') + if (!ValidateMcb(Mcb)) { DPRINT("MCB block type '%c' not valid!\n", Mcb->BlockType); return FALSE; @@ -342,45 +415,55 @@ BOOLEAN DosLinkUmb(VOID) { - DWORD Segment = FIRST_MCB_SEGMENT; + DWORD Segment = SysVars->FirstMcb; PDOS_MCB Mcb = SEGMENT_TO_MCB(Segment); DPRINT("Linking UMB\n"); /* Check if UMBs are already linked */ - if (DosUmbLinked) return FALSE; + if (SysVars->UmbChainStart == 0xFFFF) return FALSE; + if (SysVars->UmbLinked) return TRUE; + + DosMemValidate(); /* Find the last block */ + // FIXME: Use SysVars->UmbChainStart while ((Mcb->BlockType == 'M') && (Segment <= 0xFFFF)) { Segment += Mcb->Size + 1; Mcb = SEGMENT_TO_MCB(Segment); } + DosMemValidate(); + /* Make sure it's valid */ if (Mcb->BlockType != 'Z') return FALSE; /* Connect the MCB with the UMB chain */ Mcb->BlockType = 'M'; - DosUmbLinked = TRUE; + SysVars->UmbLinked = TRUE; return TRUE; } BOOLEAN DosUnlinkUmb(VOID) { - DWORD Segment = FIRST_MCB_SEGMENT; + DWORD Segment = SysVars->FirstMcb; PDOS_MCB Mcb = SEGMENT_TO_MCB(Segment); DPRINT("Unlinking UMB\n"); /* Check if UMBs are already unlinked */ - if (!DosUmbLinked) return FALSE; + if (SysVars->UmbChainStart == 0xFFFF) return FALSE; + if (!SysVars->UmbLinked) return TRUE; + + DosMemValidate(); /* Find the block preceding the MCB that links it with the UMB chain */ while (Segment <= 0xFFFF) { - if ((Segment + Mcb->Size) == (FIRST_MCB_SEGMENT + USER_MEMORY_SIZE)) + // FIXME: Use SysVars->UmbChainStart + if ((Segment + Mcb->Size) == (SysVars->FirstMcb + USER_MEMORY_SIZE)) { /* This is the last non-UMB segment */ break; @@ -394,38 +477,164 @@ /* Mark the MCB as the last MCB */ Mcb->BlockType = 'Z'; - DosUmbLinked = FALSE; + DosMemValidate(); + + SysVars->UmbLinked = FALSE; return TRUE; } VOID DosChangeMemoryOwner(WORD Segment, WORD NewOwner) { PDOS_MCB Mcb = SEGMENT_TO_MCB(Segment - 1); - - /* Just set the owner */ Mcb->OwnerPsp = NewOwner; } +/* + * Some information about DOS UMBs: + * http://textfiles.com/virus/datut010.txt + * http://www.asmcommunity.net/forums/topic/?id=30884 + */ + +WORD DosGetPreviousUmb(WORD UmbSegment) +{ + PDOS_MCB CurrentMcb; + WORD Segment, PrevSegment = 0; + + if (SysVars->UmbChainStart == 0xFFFF) + return 0; + + /* Start scanning the UMB chain */ + Segment = SysVars->UmbChainStart; + while (TRUE) + { + /* Get a pointer to the MCB */ + CurrentMcb = SEGMENT_TO_MCB(Segment); + + /* Make sure it's valid */ + if (!ValidateMcb(CurrentMcb)) + { + DPRINT1("The UMB DOS memory arena is corrupted!\n"); + Sda->LastErrorCode = ERROR_ARENA_TRASHED; + return 0; + } + + if (Segment >= UmbSegment) + break; + + PrevSegment = Segment; + + /* If this was the last MCB in the chain, quit */ + if (CurrentMcb->BlockType == 'Z') + break; + + /* Otherwise, update the segment and continue */ + Segment += CurrentMcb->Size + 1; + } + + return PrevSegment; +} + +VOID DosInitializeUmb(VOID) +{ + BOOLEAN Result; + USHORT UmbSegment = 0x0000, PrevSegment; + USHORT Size; + PDOS_MCB Mcb, PrevMcb; + + ASSERT(SysVars->UmbChainStart == 0xFFFF); + + // SysVars->UmbLinked = FALSE; + + /* Try to allocate all the UMBs */ + while (TRUE) + { + /* Find the maximum amount of memory that can be allocated */ + Size = 0xFFFF; + Result = UmaDescReserve(&UmbSegment, &Size); + + /* If we are out of UMBs, bail out */ + if (!Result && Size == 0) // XMS_STATUS_OUT_OF_UMBS + break; + + /* We should not have succeeded! */ + ASSERT(!Result); + + /* 'Size' now contains the size of the biggest UMB block. Request it. */ + Result = UmaDescReserve(&UmbSegment, &Size); + ASSERT(Result); // XMS_STATUS_SUCCESS + + /* If this is our first UMB block, initialize the UMB chain */ + if (SysVars->UmbChainStart == 0xFFFF) + { + /* Initialize the link MCB to the UMB area */ + // NOTE: We use the fact that UmbChainStart is still == 0xFFFF + // so that we initialize this block from 9FFF:0000 up to FFFF:000F. + // It will be splitted as needed just below. + Mcb = SEGMENT_TO_MCB(SysVars->FirstMcb + USER_MEMORY_SIZE + 1); // '+1': Readjust the fact that USER_MEMORY_SIZE is based using 0x9FFE instead of 0x9FFF + Mcb->BlockType = 'Z'; // At the moment it is really the last block + Mcb->Size = SysVars->UmbChainStart /* UmbSegment */ - SysVars->FirstMcb - USER_MEMORY_SIZE - 2; + Mcb->OwnerPsp = SYSTEM_PSP; + RtlCopyMemory(Mcb->Name, "SC ", sizeof("SC ")-1); + +#if 0 // Keep here for reference; this will be deleted as soon as it becomes unneeded. + /* Initialize the UMB area */ + Mcb = SEGMENT_TO_MCB(SysVars->UmbChainStart); + Mcb->Size = UMB_END_SEGMENT - SysVars->UmbChainStart; +#endif + + // FIXME: We should adjust the size of the previous block!! + + /* Initialize the start of the UMB chain */ + SysVars->UmbChainStart = SysVars->FirstMcb + USER_MEMORY_SIZE + 1; + } + + /* Split the block */ + + /* Get the previous block */ + PrevSegment = DosGetPreviousUmb(UmbSegment); + PrevMcb = SEGMENT_TO_MCB(PrevSegment); + + /* Initialize the next block */ + Mcb = SEGMENT_TO_MCB(UmbSegment + /*Mcb->Size*/(Size - 1) + 0); + // Mcb->BlockType = 'Z'; // FIXME: What if this block happens to be the last one?? + Mcb->BlockType = PrevMcb->BlockType; + Mcb->Size = PrevMcb->Size - (UmbSegment + Size - PrevSegment); + Mcb->OwnerPsp = PrevMcb->OwnerPsp; + RtlCopyMemory(Mcb->Name, PrevMcb->Name, sizeof(PrevMcb->Name)); + + /* The previous block is not the latest one anymore */ + PrevMcb->BlockType = 'M'; + PrevMcb->Size = UmbSegment - PrevSegment - 1; + + /* Initialize the new UMB block */ + Mcb = SEGMENT_TO_MCB(UmbSegment); + Mcb->BlockType = 'M'; // 'Z' // FIXME: What if this block happens to be the last one?? + Mcb->Size = Size - 1 - 1; // minus 2 because we need to have one arena at the beginning and one at the end. + Mcb->OwnerPsp = 0; + // FIXME: Which MCB name should we use? I need to explore more the docs! + RtlCopyMemory(Mcb->Name, "UMB ", sizeof("UMB ")-1); + // RtlCopyMemory(Mcb->Name, "SM ", sizeof("SM ")-1); + } +} + VOID DosInitializeMemory(VOID) { - PDOS_MCB Mcb = SEGMENT_TO_MCB(FIRST_MCB_SEGMENT); + PDOS_MCB Mcb; + + /* Set the initial allocation strategy to "best fit" */ + Sda->AllocStrategy = DOS_ALLOC_BEST_FIT; + + /* Initialize conventional memory; we don't have UMBs yet */ + SysVars->FirstMcb = FIRST_MCB_SEGMENT; // The Arena Head + SysVars->UmbLinked = FALSE; + SysVars->UmbChainStart = 0xFFFF; + + Mcb = SEGMENT_TO_MCB(SysVars->FirstMcb); /* Initialize the MCB */ Mcb->BlockType = 'Z'; Mcb->Size = USER_MEMORY_SIZE; Mcb->OwnerPsp = 0; - - /* Initialize the link MCB to the UMB area */ - Mcb = SEGMENT_TO_MCB(FIRST_MCB_SEGMENT + USER_MEMORY_SIZE + 1); - Mcb->BlockType = 'M'; - Mcb->Size = UMB_START_SEGMENT - FIRST_MCB_SEGMENT - USER_MEMORY_SIZE - 2; - Mcb->OwnerPsp = SYSTEM_PSP; - - /* Initialize the UMB area */ - Mcb = SEGMENT_TO_MCB(UMB_START_SEGMENT); - Mcb->BlockType = 'Z'; - Mcb->Size = UMB_END_SEGMENT - UMB_START_SEGMENT; - Mcb->OwnerPsp = 0; } /* EOF */ Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/memory.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/memory.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/memory.h [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -4,6 +4,7 @@ * FILE: dos/dos32krnl/memory.h * PURPOSE: DOS32 Memory Manager * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> + * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) */ #ifndef _DOS_MEMORY_H_ @@ -12,12 +13,6 @@ /* TYPEDEFS *******************************************************************/ #define SEGMENT_TO_MCB(seg) ((PDOS_MCB)SEG_OFF_TO_PTR((seg), 0)) - -#define FIRST_MCB_SEGMENT 0x1000 -#define USER_MEMORY_SIZE (0x9FFE - FIRST_MCB_SEGMENT) - -#define UMB_START_SEGMENT 0xC000 -#define UMB_END_SEGMENT 0xDFFF #define DOS_ALLOC_HIGH 0x40 #define DOS_ALLOC_HIGH_LOW 0x80 @@ -43,9 +38,6 @@ /* VARIABLES ******************************************************************/ -extern BYTE DosAllocStrategy; -extern BOOLEAN DosUmbLinked; - /* FUNCTIONS ******************************************************************/ WORD DosAllocateMemory(WORD Size, WORD *MaxAvailable); @@ -55,6 +47,7 @@ BOOLEAN DosUnlinkUmb(VOID); VOID DosChangeMemoryOwner(WORD Segment, WORD NewOwner); +VOID DosInitializeUmb(VOID); VOID DosInitializeMemory(VOID); #endif // _DOS_MEMORY_H_ Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/process.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/process.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/process.c [iso-8859-1] Sat Aug 1 17:07:07 2015 @@ -852,7 +852,7 @@ VOID DosTerminateProcess(WORD Psp, BYTE ReturnCode, WORD KeepResident) { WORD i; - WORD McbSegment = FIRST_MCB_SEGMENT; + WORD McbSegment = SysVars->FirstMcb; PDOS_MCB CurrentMcb; LPDWORD IntVecTable = (LPDWORD)((ULONG_PTR)BaseAddress); PDOS_PSP PspBlock = SEGMENT_TO_PSP(Psp);

1 0

[hbelusca] 68585: [NTVDM] - Remove some hardcoded values. - Reshuffle again DOS initialization to better reflect how it is done in the real world (see "Advanced MS-DOS Programming" by Ray Duncan, C...
by hbelusca＠svn.reactos.org 01 Aug '15

01 Aug '15

Author: hbelusca Date: Sat Aug 1 12:41:22 2015 New Revision: 68585 URL: http://svn.reactos.org/svn/reactos?rev=68585&view=rev Log: [NTVDM] - Remove some hardcoded values. - Reshuffle again DOS initialization to better reflect how it is done in the real world (see "Advanced MS-DOS Programming" by Ray Duncan, Chapter 2 "MS-DOS in Operation"): split DOS data stuff into DOS BIOS data, and DOS kernel data areas. In preparation for an upcoming commit. Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/bios.c trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.c trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.h trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dosfiles.h trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/handle.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/bios.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/bios.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/bios.c [iso-8859-1] Sat Aug 1 12:41:22 2015 @@ -32,6 +32,13 @@ /* PRIVATE VARIABLES **********************************************************/ +// CALLBACK16 BiosContext; + +/* PUBLIC VARIABLES ***********************************************************/ + +/* Global DOS BIOS data area */ +PBIOS_DATA BiosData; + /* PRIVATE FUNCTIONS **********************************************************/ /* PUBLIC FUNCTIONS ***********************************************************/ @@ -63,7 +70,7 @@ return FALSE; } - if (Descriptor->DeviceInfo & (1 << 7)) + if (Descriptor->DeviceInfo & FILE_INFO_DEVICE) { WORD Result; PDOS_DEVICE_NODE Node = DosGetDriverNode(Descriptor->DevicePointer); @@ -155,28 +162,61 @@ BOOLEAN DosBIOSInitialize(VOID) { -#if 0 - UCHAR i; - CHAR CurrentDirectory[MAX_PATH]; - CHAR DosDirectory[DOS_DIR_LENGTH]; - LPSTR Path; - FILE *Stream; WCHAR Buffer[256]; -#endif /* Set the data segment */ - setDS(DOS_DATA_SEGMENT); - - /* Initialize the DOS stack */ - // Stack just before FIRST_MCB_SEGMENT and after SYSTEM_ENV_BLOCK - // FIXME: Add a block of fixed size for the stack in DOS_DATA instead! + setDS(BIOS_DATA_SEGMENT); + + /* Initialize the global DOS BIOS data area */ + BiosData = (PBIOS_DATA)SEG_OFF_TO_PTR(BIOS_DATA_SEGMENT, 0x0000); + + /* Initialize the DOS BIOS stack */ + // FIXME: Add a block of fixed size for the stack in BIOS/DOS_DATA instead! setSS(0x0F00); setSP(0x0FF0); - setBP(0x091E); // DOS base stack pointer relic value - - /* Initialize memory management */ - DosInitializeMemory(); + + /* + * Initialize the INT 13h (BIOS Disk Services) handler chain support. + * + * The INT 13h handler chain is some functionality that allows DOS + * to insert disk filter drivers in between the (hooked) INT 13h handler + * and its original handler. + * Typically, those are: + * - filter for detecting disk changes (for floppy disks), + * - filter for tracking formatting calls and correcting DMA boundary errors, + * - a possible filter to work around a bug in a particular version of PC-AT's + * IBM's ROM BIOS (on systems with model byte FCh and BIOS date "01/10/84" only) + * (see http://www.ctyme.com/intr/rb-4453.htm for more details). + * + * This functionality is known to be used by some legitimate programs, + * by Windows 3.x, as well as some illegitimate ones (aka. virii). + * + * See extra information about this support in dos.h + */ + // FIXME: Should be done by the DOS BIOS + BiosData->RomBiosInt13 = ((PULONG)BaseAddress)[0x13]; + BiosData->PrevInt13 = BiosData->RomBiosInt13; +// RegisterDosInt32(0x13, DosInt13h); // Unused at the moment! + + // + // HERE: Do all hardware initialization needed for DOS + // + + /* + * SysInit part... + */ + + // InitializeContext(&DosContext, BIOS_CODE_SEGMENT, 0x0010); + + /* Initialize the DOS kernel (DosInit) */ + if (!DosKRNLInitialize()) + { + DisplayMessage(L"Failed to load the DOS kernel! Exiting..."); + return FALSE; + } + + /* DOS kernel loading succeeded, we can finish the initialization */ /* Build the system master environment block (inherited by the shell) */ if (!DosBuildSysEnvBlock()) @@ -184,44 +224,7 @@ DPRINT1("An error occurred when setting up the system environment block.\n"); } - -#if 0 - - /* Clear the current directory buffer */ - RtlZeroMemory(CurrentDirectories, sizeof(CurrentDirectories)); - - /* Get the current directory */ - if (!GetCurrentDirectoryA(MAX_PATH, CurrentDirectory)) - { - // TODO: Use some kind of default path? - return FALSE; - } - - /* Convert that to a DOS path */ - if (!GetShortPathNameA(CurrentDirectory, DosDirectory, DOS_DIR_LENGTH)) - { - // TODO: Use some kind of default path? - return FALSE; - } - - /* Set the drive */ - Sda->CurrentDrive = DosDirectory[0] - 'A'; - - /* Get the directory part of the path */ - Path = strchr(DosDirectory, '\\'); - if (Path != NULL) - { - /* Skip the backslash */ - Path++; - } - - /* Set the directory */ - if (Path != NULL) - { - strncpy(CurrentDirectories[Sda->CurrentDrive], Path, DOS_DIR_LENGTH); - } - - /* Read CONFIG.SYS */ + /* TODO: Read CONFIG.NT/SYS */ Stream = _wfopen(DOS_CONFIG_PATH, L"r"); if (Stream != NULL) { @@ -232,10 +235,7 @@ fclose(Stream); } -#endif - - /* Initialize the DOS kernel */ - return DosKRNLInitialize(); + return TRUE; } /* EOF */ Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.c [iso-8859-1] Sat Aug 1 12:41:22 2015 @@ -191,7 +191,6 @@ BYTE Character; SYSTEMTIME SystemTime; PCHAR String; - PDOS_INPUT_BUFFER InputBuffer; Sda->InDos++; @@ -351,7 +350,7 @@ case 0x0A: { WORD Count = 0; - InputBuffer = (PDOS_INPUT_BUFFER)SEG_OFF_TO_PTR(getDS(), getDX()); + PDOS_INPUT_BUFFER InputBuffer = (PDOS_INPUT_BUFFER)SEG_OFF_TO_PTR(getDS(), getDX()); DPRINT("Read Buffered Input\n"); @@ -2199,12 +2198,12 @@ case 0x13: { /* Save the old values of PrevInt13 and RomBiosInt13 */ - ULONG OldInt13 = DosData->PrevInt13; - ULONG OldBiosInt13 = DosData->RomBiosInt13; + ULONG OldInt13 = BiosData->PrevInt13; + ULONG OldBiosInt13 = BiosData->RomBiosInt13; /* Set PrevInt13 and RomBiosInt13 to their new values */ - DosData->PrevInt13 = MAKELONG(getDX(), getDS()); - DosData->RomBiosInt13 = MAKELONG(getBX(), getES()); + BiosData->PrevInt13 = MAKELONG(getDX(), getDS()); + BiosData->RomBiosInt13 = MAKELONG(getBX(), getES()); /* Return in DS:DX the old value of PrevInt13 */ setDS(HIWORD(OldInt13)); @@ -2271,8 +2270,6 @@ BOOLEAN DosKRNLInitialize(VOID) { -#if 1 - UCHAR i; PDOS_SFT Sft; LPSTR Path; @@ -2281,7 +2278,8 @@ CHAR CurrentDirectory[MAX_PATH]; CHAR DosDirectory[DOS_DIR_LENGTH]; - const BYTE NullDriverRoutine[] = { + static const BYTE NullDriverRoutine[] = + { /* Strategy routine entry */ 0x26, // mov [Request.Status], DOS_DEVSTAT_DONE 0xC7, @@ -2294,12 +2292,16 @@ 0xCB, // retf }; - FILE *Stream; - WCHAR Buffer[256]; + /* Set the data segment */ + setDS(DOS_DATA_SEGMENT); /* Initialize the global DOS data area */ DosData = (PDOS_DATA)SEG_OFF_TO_PTR(DOS_DATA_SEGMENT, 0x0000); RtlZeroMemory(DosData, sizeof(*DosData)); + + /* Initialize the DOS stack */ + setSS(DOS_DATA_SEGMENT); + setSP(DOS_DATA_OFFSET(DosStack) + sizeof(DosData->DosStack) - sizeof(WORD)); /* Initialize the list of lists */ SysVars = &DosData->SysVars; @@ -2409,18 +2411,8 @@ RtlZeroMemory(&Sft->FileDescriptors[i], sizeof(DOS_FILE_DESCRIPTOR)); } - /* Read CONFIG.SYS */ - Stream = _wfopen(DOS_CONFIG_PATH, L"r"); - if (Stream != NULL) - { - while (fgetws(Buffer, 256, Stream)) - { - // TODO: Parse the line - } - fclose(Stream); - } - -#endif + /* Initialize memory management */ + DosInitializeMemory(); /* Initialize the callback context */ InitializeContext(&DosContext, DOS_CODE_SEGMENT, 0x0000); @@ -2449,29 +2441,6 @@ RegisterDosInt32(0x2C, NULL); RegisterDosInt32(0x2D, NULL); - /* - * Initialize the INT 13h (BIOS Disk Services) handler chain support. - * - * The INT 13h handler chain is some functionality that allows DOS - * to insert disk filter drivers in between the (hooked) INT 13h handler - * and its original handler. - * Typically, those are: - * - filter for detecting disk changes (for floppy disks), - * - filter for tracking formatting calls and correcting DMA boundary errors, - * - a possible filter to work around a bug in a particular version of PC-AT's - * IBM's ROM BIOS (on systems with model byte FCh and BIOS date "01/10/84" only) - * (see http://www.ctyme.com/intr/rb-4453.htm for more details). - * - * This functionality is known to be used by some legitimate programs, - * by Windows 3.x, as well as some illegitimate ones (aka. virii). - * - * See extra information about this support in dos.h - */ - // FIXME: Should be done by the DOS BIOS - DosData->RomBiosInt13 = ((PULONG)BaseAddress)[0x13]; - DosData->PrevInt13 = DosData->RomBiosInt13; -// RegisterDosInt32(0x13, DosInt13h); // Unused at the moment! - /* Initialize country data */ DosCountryInitialize(); Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.h [iso-8859-1] Sat Aug 1 12:41:22 2015 @@ -27,22 +27,26 @@ #define DOS_CONFIG_PATH L"%SystemRoot%\\system32\\CONFIG.NT" #define DOS_COMMAND_INTERPRETER L"%SystemRoot%\\system32\\COMMAND.COM /k %SystemRoot%\\system32\\AUTOEXEC.NT" -#define SYSTEM_PSP 0x08 -#define SYSTEM_ENV_BLOCK 0x800 -#define DOS_CODE_SEGMENT 0x70 -#define DOS_DATA_SEGMENT 0xA0 +#define BIOS_CODE_SEGMENT 0x70 +#define BIOS_DATA_SEGMENT 0x70 +#define DOS_CODE_SEGMENT 0x80 +#define DOS_DATA_SEGMENT 0xA5 #define DOS_DATA_OFFSET(x) FIELD_OFFSET(DOS_DATA, x) + +#define SYSTEM_ENV_BLOCK 0x600 // FIXME: Should be dynamic + +#define SYSTEM_PSP 0x0008 #define INVALID_DOS_HANDLE 0xFFFF #define DOS_INPUT_HANDLE 0 #define DOS_OUTPUT_HANDLE 1 #define DOS_ERROR_HANDLE 2 -#define DOS_SFT_SIZE 255 -#define DOS_DIR_LENGTH 64 -#define NUM_DRIVES ('Z' - 'A' + 1) -#define DOS_CHAR_ATTRIBUTE 0x07 +#define DOS_SFT_SIZE 255 // Value of the 'FILES=' command; maximum 255 +#define DOS_DIR_LENGTH 64 +#define NUM_DRIVES ('Z' - 'A' + 1) +#define DOS_CHAR_ATTRIBUTE 0x07 #pragma pack(push, 1) @@ -230,8 +234,27 @@ DWORD PrevCallFrame; } DOS_SDA, *PDOS_SDA; +/* + * DOS kernel data structure + */ typedef struct _DOS_DATA { + DOS_SYSVARS SysVars; + BYTE NullDriverRoutine[7]; + WORD DosVersion; // DOS version to report to programs (can be different from the true one) + DOS_SDA Sda; + CHAR CurrentDirectories[NUM_DRIVES][DOS_DIR_LENGTH]; + BYTE DosStack[384]; + BYTE Sft[ANYSIZE_ARRAY]; +} DOS_DATA, *PDOS_DATA; + +/* + * DOS BIOS data structure at segment 70h + */ +typedef struct _BIOS_DATA +{ + BYTE StartupCode[20]; // 0x00 - 20 bytes: large enough for now! + /* * INT 13h (BIOS Disk Services) handler chain support. * @@ -256,22 +279,20 @@ * http://repo.hackerzvoice.net/depot_madchat/vxdevl/vdat/tuvd0001.htm * http://vxheaven.org/lib/vsm01.html */ - DWORD RomBiosInt13; - DWORD PrevInt13; // FIXME: Put it at 0070:00B4 - - DOS_SYSVARS SysVars; - BYTE NullDriverRoutine[7]; - WORD DosVersion; // DOS version to report to programs (can be different from the true one) - DOS_SDA Sda; - CHAR CurrentDirectories[NUM_DRIVES][DOS_DIR_LENGTH]; - BYTE Sft[ANYSIZE_ARRAY]; -} DOS_DATA, *PDOS_DATA; + BYTE Padding0[0xB0 - /*FIELD_OFFSET(BIOS_DATA, StartupCode)*/ 20]; + DWORD RomBiosInt13; // 0xb0 + DWORD PrevInt13; // 0xb4 + BYTE Padding1[0x100 - 0xB8]; // 0xb8 +} BIOS_DATA, *PBIOS_DATA; + +C_ASSERT(sizeof(BIOS_DATA) == 0x100); #pragma pack(pop) /* VARIABLES ******************************************************************/ -extern BOOLEAN DoEcho; +extern BOOLEAN DoEcho; // FIXME: Maybe move inside BiosData? (it's set by BIOS but used by CON driver in DOS BIOS) +extern PBIOS_DATA BiosData; extern PDOS_DATA DosData; extern PDOS_SYSVARS SysVars; extern PDOS_SDA Sda; @@ -279,11 +300,13 @@ /* FUNCTIONS ******************************************************************/ extern CALLBACK16 DosContext; -#define RegisterDosInt32(IntNumber, IntHandler) \ +#define RegisterDosInt32(IntNumber, IntHandler) \ do { \ - DosContext.NextOffset += RegisterInt32(MAKELONG(DosContext.NextOffset, \ - DosContext.Segment), \ - (IntNumber), (IntHandler), NULL); \ + ASSERT((0x20 <= IntNumber) && (IntNumber <= 0x2F)); \ + RegisterInt32(DosContext.TrampolineFarPtr + \ + DosContext.TrampolineSize + \ + (IntNumber - 0x20) * Int16To32StubSize, \ + (IntNumber), (IntHandler), NULL); \ } while(0); /* Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dosfiles.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dosfiles.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dosfiles.h [iso-8859-1] Sat Aug 1 12:41:22 2015 @@ -12,6 +12,30 @@ #pragma pack(push, 1) +#if 0 // Real DOS-5 SFT entry, for reference only +typedef struct _DOS_FILE_DESCRIPTOR_DOS5 +{ + WORD RefCount; // 0x00 + WORD OpenMode; // 0x02 + BYTE Attributes; // 0x04 + WORD DeviceInfo; // 0x05 + DWORD DevicePointer; // 0x07 + WORD StartCluster; // 0x0b + WORD Time; // 0x0d + WORD Date; // 0x0f + DWORD Size; // 0x11 + DWORD Position; // 0x15 + BYTE Reserved0[7]; // 0x19 + CHAR FileName[11]; // 0x20 + BYTE Reserved1[6]; // 0x2b + WORD OwnerPsp; // 0x31 + BYTE Reserved2[8]; // 0x33 +} DOS_FILE_DESCRIPTOR_DOS5, *PDOS_FILE_DESCRIPTOR_DOS5; + +C_ASSERT(sizeof(DOS_FILE_DESCRIPTOR_DOS5) == 0x3B); +#endif + +// Modified DOS SFT entry, compatible for NTVDM only typedef struct _DOS_FILE_DESCRIPTOR { WORD RefCount; Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/handle.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/handle.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/handle.c [iso-8859-1] Sat Aug 1 12:41:22 2015 @@ -360,7 +360,7 @@ /* Check if the reference count fell to zero */ if (!Descriptor->RefCount) { - if (Descriptor->DeviceInfo & (1 << 7)) + if (Descriptor->DeviceInfo & FILE_INFO_DEVICE) { PDOS_DEVICE_NODE Node = DosGetDriverNode(Descriptor->DevicePointer); @@ -369,7 +369,7 @@ } else { - /* Close the win32 handle */ + /* Close the Win32 handle */ CloseHandle(Descriptor->Win32Handle); } }

1 0

[bfreisen] 68584: [MSPAINT_NEW] - use optimal buffer size to load null terminated strings using LoadString - add printing-related menu entries and dialogs (no printing functionality yet)
by bfreisen＠svn.reactos.org 31 Jul '15

31 Jul '15

Author: bfreisen Date: Fri Jul 31 17:33:21 2015 New Revision: 68584 URL: http://svn.reactos.org/svn/reactos?rev=68584&view=rev Log: [MSPAINT_NEW] - use optimal buffer size to load null terminated strings using LoadString - add printing-related menu entries and dialogs (no printing functionality yet) Modified: trunk/reactos/base/applications/mspaint_new/definitions.h trunk/reactos/base/applications/mspaint_new/lang/bg-BG.rc trunk/reactos/base/applications/mspaint_new/lang/cs-CZ.rc trunk/reactos/base/applications/mspaint_new/lang/de-DE.rc trunk/reactos/base/applications/mspaint_new/lang/en-GB.rc trunk/reactos/base/applications/mspaint_new/lang/en-US.rc trunk/reactos/base/applications/mspaint_new/lang/es-ES.rc trunk/reactos/base/applications/mspaint_new/lang/eu-ES.rc trunk/reactos/base/applications/mspaint_new/lang/fr-FR.rc trunk/reactos/base/applications/mspaint_new/lang/he-IL.rc trunk/reactos/base/applications/mspaint_new/lang/hu-HU.rc trunk/reactos/base/applications/mspaint_new/lang/it-IT.rc trunk/reactos/base/applications/mspaint_new/lang/ja-JP.rc trunk/reactos/base/applications/mspaint_new/lang/nl-NL.rc trunk/reactos/base/applications/mspaint_new/lang/no-NO.rc trunk/reactos/base/applications/mspaint_new/lang/pl-PL.rc trunk/reactos/base/applications/mspaint_new/lang/pt-BR.rc trunk/reactos/base/applications/mspaint_new/lang/pt-PT.rc trunk/reactos/base/applications/mspaint_new/lang/ro-RO.rc trunk/reactos/base/applications/mspaint_new/lang/ru-RU.rc trunk/reactos/base/applications/mspaint_new/lang/sk-SK.rc trunk/reactos/base/applications/mspaint_new/lang/sq-AL.rc trunk/reactos/base/applications/mspaint_new/lang/sv-SE.rc trunk/reactos/base/applications/mspaint_new/lang/tr-TR.rc trunk/reactos/base/applications/mspaint_new/lang/uk-UA.rc trunk/reactos/base/applications/mspaint_new/lang/zh-CN.rc trunk/reactos/base/applications/mspaint_new/lang/zh-TW.rc trunk/reactos/base/applications/mspaint_new/winproc.cpp Modified: trunk/reactos/base/applications/mspaint_new/definitions.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/definitions.h [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/definitions.h [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -50,11 +50,15 @@ #define IDM_FILESAVE 203 #define IDM_FILESAVEAS 204 +#define IDM_FILEPAGEVIEW 206 +#define IDM_FILEPAGESETUP 207 +#define IDM_FILEPRINT 208 + #define IDM_FILEASWALLPAPERPLANE 210 #define IDM_FILEASWALLPAPERCENTERED 211 #define IDM_FILEASWALLPAPERSTRETCHED 212 -#define IDM_FILEEXIT 205 +#define IDM_FILEEXIT 215 #define IDM_EDITUNDO 220 #define IDM_EDITREDO 221 Modified: trunk/reactos/base/applications/mspaint_new/lang/bg-BG.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/bg-BG.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/bg-BG.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "&ÐÑÐ²Ð°ÑÑÐ½Ðµ...\tAlt+tCtrl+O", IDM_FILEOPEN MENUITEM "&ÐÐ°Ð¿Ð¸Ñ\tCtrl+S", IDM_FILESAVE MENUITEM "ÐÐ°Ð¿Ð¸Ñ &ÐºÐ°ÑÐ¾...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "ÐÐ¾ÑÑÐ°Ð²ÑÐ½Ðµ ÐºÐ°ÑÐ¾ ÑÐ°Ð¿ÐµÑ (Ð½Ð°ÑÑÐµÐ»ÐµÐ½Ð¾)", IDM_FILEASWALLPAPERPLANE MENUITEM "ÐÐ¾ÑÑÐ°Ð²ÑÐ½Ðµ ÐºÐ°ÑÐ¾ ÑÐ°Ð¿ÐµÑ (Ð² ÑÑÐµÐ´Ð°ÑÐ°)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/cs-CZ.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/cs-CZ.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/cs-CZ.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "&OtevÅÃt...\tCtrl+O", IDM_FILEOPEN MENUITEM "UloÅ¾it\tCtrl+S", IDM_FILESAVE MENUITEM "UloÅ¾it jako...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Nastavit jako pozadÃ (vedle sebe)", IDM_FILEASWALLPAPERPLANE MENUITEM "Nastavit jako pozadÃ (na stÅed)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/de-DE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/de-DE.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/de-DE.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "Ãffnen...\tStrg+O", IDM_FILEOPEN MENUITEM "Speichern\tStrg+S", IDM_FILESAVE MENUITEM "Speichern unter...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Seitenansicht", IDM_FILEPAGEVIEW + MENUITEM "Seite einrichten...", IDM_FILEPAGESETUP + MENUITEM "Drucken...\tStrg+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Als Hintergrund (FlÃ¤che)", IDM_FILEASWALLPAPERPLANE MENUITEM "Als Hintergrund (Zentriert)", IDM_FILEASWALLPAPERCENTERED Modified: trunk/reactos/base/applications/mspaint_new/lang/en-GB.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/en-GB.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/en-GB.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "&Open...\tCtrl+O", IDM_FILEOPEN MENUITEM "&Save\tCtrl+S", IDM_FILESAVE MENUITEM "Save &as...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Set as wallpaper (tiled)", IDM_FILEASWALLPAPERPLANE MENUITEM "Set as wallpaper (centered)", IDM_FILEASWALLPAPERCENTERED Modified: trunk/reactos/base/applications/mspaint_new/lang/en-US.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/en-US.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/en-US.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "&Open...\tCtrl+O", IDM_FILEOPEN MENUITEM "&Save\tCtrl+S", IDM_FILESAVE MENUITEM "Save &as...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Set as wallpaper (tiled)", IDM_FILEASWALLPAPERPLANE MENUITEM "Set as wallpaper (centered)", IDM_FILEASWALLPAPERCENTERED Modified: trunk/reactos/base/applications/mspaint_new/lang/es-ES.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/es-ES.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/es-ES.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -17,6 +17,10 @@ MENUITEM "Abrir...\tCtrl+O", IDM_FILEOPEN MENUITEM "Guardar\tCtrl+S", IDM_FILESAVE MENUITEM "Guardar como...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Fondo de pantalla (normal)", IDM_FILEASWALLPAPERPLANE MENUITEM "Fondo de pantalla (centrado)", IDM_FILEASWALLPAPERCENTERED @@ -215,4 +219,5 @@ IDS_PERCENTAGE "El porcentaje debe de estar entre 1 y 500." IDS_ANGLE "El Ã¡ngulo debe de estar entre -89 y 89." IDS_LOADERRORTEXT "No se pudo cargar el archivo %s." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/eu-ES.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/eu-ES.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/eu-ES.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -8,6 +8,10 @@ MENUITEM "Ireki...\tCtrl+O", IDM_FILEOPEN MENUITEM "Gorde\tCtrl+S", IDM_FILESAVE MENUITEM "Gorde honela...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Ezarri azteko plano gisa (Lauza moduan)", IDM_FILEASWALLPAPERPLANE MENUITEM "Ezarri azteko plano gisa (erdian)", IDM_FILEASWALLPAPERCENTERED @@ -206,4 +210,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/fr-FR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/fr-FR.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/fr-FR.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -8,6 +8,10 @@ MENUITEM "&Ouvrir...\tCtrl+O", IDM_FILEOPEN MENUITEM "&Enregistrer\tCtrl+S", IDM_FILESAVE MENUITEM "Enregistrer &sous...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Choisir comme image d'arriÃ¨re-plan (mosaÃ¯que)", IDM_FILEASWALLPAPERPLANE MENUITEM "Choisir comme image d'arriÃ¨re-plan (centrÃ©e)", IDM_FILEASWALLPAPERCENTERED @@ -206,4 +210,5 @@ IDS_PERCENTAGE "Le pourcentage doit Ãªtre entre 1 et 500." IDS_ANGLE "L'angle doit Ãªtre entre -89 et 89." IDS_LOADERRORTEXT "Le fichier %s n'a pas pu Ãªtre chargÃ©." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/he-IL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/he-IL.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/he-IL.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -8,6 +8,10 @@ MENUITEM "&×¤×ª×\tCtrl+O", IDM_FILEOPEN MENUITEM "&×©×××¨\tCtrl+S", IDM_FILESAVE MENUITEM "×©×××¨ ××©&×...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "×§××¢ ××¨×§×¢ ×©×××× ××¢×××× (×¤×¨××©)", IDM_FILEASWALLPAPERPLANE MENUITEM "×§××¢ ××¨×§×¢ ×©×××× ××¢×××× (××¨××)", IDM_FILEASWALLPAPERCENTERED @@ -206,4 +210,5 @@ IDS_PERCENTAGE, "The percentage must be between 1 and 500." IDS_ANGLE, "The angle must be between -89 and 89." IDS_LOADERRORTEXT, "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/hu-HU.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/hu-HU.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/hu-HU.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -8,6 +8,10 @@ MENUITEM "Meg&nyitÃ¡s...\tCtrl+O", IDM_FILEOPEN MENUITEM "&MentÃ©s\tCtrl+S", IDM_FILESAVE MENUITEM "MentÃ©s mÃ¡&skÃ©nt...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "BeÃ¡llÃtÃ¡s tapÃ©takÃ©nt (mozaik)", IDM_FILEASWALLPAPERPLANE MENUITEM "BeÃ¡llÃtÃ¡s tapÃ©takÃ©nt (kÃ¶zÃ©pre)", IDM_FILEASWALLPAPERCENTERED @@ -206,4 +210,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/it-IT.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/it-IT.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/it-IT.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "Apri...\tCtrl+F12", IDM_FILEOPEN MENUITEM "Salva\tMaiusc+F12", IDM_FILESAVE MENUITEM "Salva con nome...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Imposta come sfondo (normale)", IDM_FILEASWALLPAPERPLANE MENUITEM "Imposta come sfondo (centrata)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "La percentuale deve essere compresa tra 1 e 500." IDS_ANGLE "L'angolo deve essere compreso tra -89 e 89." IDS_LOADERRORTEXT "l file %s non puÃ² essere caricato." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/ja-JP.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/ja-JP.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/ja-JP.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "éã(&O)\tCtrl+O", IDM_FILEOPEN MENUITEM "ä¸æ¸ãä¿å(&S)\tCtrl+S", IDM_FILESAVE MENUITEM "ååãä»ãã¦ä¿å(&A)...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "èæ¯ã«è¨å® (ä¸¦ã¹ã¦è¡¨ç¤º)", IDM_FILEASWALLPAPERPLANE MENUITEM "èæ¯ã«è¨å® (ä¸å¤®ã«è¡¨ç¤º)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/nl-NL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/nl-NL.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/nl-NL.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "Openen...\tCtrl+O", IDM_FILEOPEN MENUITEM "Opslaan\tCtrl+S", IDM_FILESAVE MENUITEM "Opslaan als...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Als achtergrond instellen (naast elkaar)", IDM_FILEASWALLPAPERPLANE MENUITEM "Als achtergrond instellen (gecentreerd)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/no-NO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/no-NO.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/no-NO.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "&Ãpne...\tCtrl+O", IDM_FILEOPEN MENUITEM "&Lagre\tCtrl+S", IDM_FILESAVE MENUITEM "Lagre &som...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Bruk som bakgrunn (sidestilt)", IDM_FILEASWALLPAPERPLANE MENUITEM "Bruk som bakgrunn (midstilt)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/pl-PL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/pl-PL.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/pl-PL.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -17,6 +17,10 @@ MENUITEM "&OtwÃ³rz...\tCtrl+O", IDM_FILEOPEN MENUITEM "&Zapisz\tCtrl+S", IDM_FILESAVE MENUITEM "Zapisz &jako...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Ustaw jako tapetÄ (sÄsiadujÄce)", IDM_FILEASWALLPAPERPLANE MENUITEM "Ustaw jako tapetÄ (wyÅrodkowana)", IDM_FILEASWALLPAPERCENTERED @@ -215,4 +219,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/pt-BR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/pt-BR.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/pt-BR.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "A&brir...\tCtrl+O", IDM_FILEOPEN MENUITEM "&Salvar\tCtrl+S", IDM_FILESAVE MENUITEM "Salvar &como...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Definir como plano de fundo (Lado a lado)", IDM_FILEASWALLPAPERPLANE MENUITEM "Definir como plano de fundo (Centralizado)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/pt-PT.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/pt-PT.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/pt-PT.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "&Abrir...\tCtrl+A", IDM_FILEOPEN MENUITEM "&Guardar\tCtrl+G", IDM_FILESAVE MENUITEM "Guardar &como...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Definir como fundo (Mosaico)", IDM_FILEASWALLPAPERPLANE MENUITEM "Definir como fundo (Centrado)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/ro-RO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/ro-RO.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/ro-RO.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "&Deschidereâ¦\tCtrl+O", IDM_FILEOPEN MENUITEM "&PÄstreazÄ\tCtrl+S", IDM_FILESAVE MENUITEM "PÄst&rare Ã®nâ¦", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Decor de birou (Ã®n c&arou)", IDM_FILEASWALLPAPERPLANE MENUITEM "Decor de birou (&centrat)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "Procentajul trebuie sÄ fie Ã®ntre 1 Èi 500." IDS_ANGLE "Unghiul trebuie sÄ fie Ã®ntre -89 Èi 89." IDS_LOADERRORTEXT "FiÈierul %s nu poate fi Ã®ncÄrcat." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/ru-RU.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/ru-RU.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/ru-RU.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -8,6 +8,10 @@ MENUITEM "&ÐÑÐºÑÑÑÑ...\tCtrl+O", IDM_FILEOPEN MENUITEM "&Ð¡Ð¾ÑÑÐ°Ð½Ð¸ÑÑ\tCtrl+S", IDM_FILESAVE MENUITEM "Ð¡Ð¾ÑÑÐ°Ð½Ð¸ÑÑ &ÐºÐ°Ðº...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Ð¡Ð´ÐµÐ»Ð°ÑÑ ÑÐ¾Ð½Ð¾Ð²ÑÐ¼ ÑÐ¸ÑÑÐ½ÐºÐ¾Ð¼ (&Ð·Ð°Ð¼Ð¾ÑÑÐ¸ÑÑ)", IDM_FILEASWALLPAPERPLANE MENUITEM "Ð¡Ð´ÐµÐ»Ð°ÑÑ ÑÐ¾Ð½Ð¾Ð²ÑÐ¼ ÑÐ¸ÑÑÐ½ÐºÐ¾Ð¼ (Ð¿Ð¾ Ñ&ÐµÐ½ÑÑÑ)", IDM_FILEASWALLPAPERCENTERED @@ -206,4 +210,5 @@ IDS_PERCENTAGE, "ÐÑÐ¾ÑÐµÐ½Ñ Ð´Ð¾Ð»Ð¶ÐµÐ½ Ð±ÑÑÑ Ð² Ð¸Ð½ÑÐµÑÐ²Ð°Ð»Ðµ Ð¾Ñ 1 Ð´Ð¾ 500." IDS_ANGLE, "Ð£Ð³Ð¾Ð» Ð´Ð¾Ð»Ð¶ÐµÐ½ Ð±ÑÑÑ Ð² Ð¸Ð½ÑÐµÑÐ²Ð°Ð»Ðµ Ð¾Ñ -89 Ð´Ð¾ 89." IDS_LOADERRORTEXT, "ÐÐµÐ²Ð¾Ð·Ð¼Ð¾Ð¶Ð½Ð¾ Ð·Ð°Ð³ÑÑÐ·Ð¸ÑÑ ÑÐ°Ð¹Ð» %s." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/sk-SK.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/sk-SK.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/sk-SK.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -17,6 +17,10 @@ MENUITEM "&OtvoriÅ¥...\tCtrl+O", IDM_FILEOPEN MENUITEM "&UloÅ¾iÅ¥\tCtrl+S", IDM_FILESAVE MENUITEM "UloÅ¾iÅ¥ &ako...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "NastaviÅ¥ ako pozadie (vydlÃ¡Å¾denÃ©)", IDM_FILEASWALLPAPERPLANE MENUITEM "NastaviÅ¥ ako pozadie (v strede)", IDM_FILEASWALLPAPERCENTERED @@ -215,4 +219,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/sq-AL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/sq-AL.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/sq-AL.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "&Hap...\tCtrl+O", IDM_FILEOPEN MENUITEM "&Ruaj\tCtrl+S", IDM_FILESAVE MENUITEM "Ruaj &si...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Vere si wallpaper (tiled)", IDM_FILEASWALLPAPERPLANE MENUITEM "Vere si wallpaper (centered)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "PÃ«rqindja duhet tÃ« jetÃ« nÃ« mes 1 dhe 500." IDS_ANGLE "KÃ«ndi duhet tÃ« jetÃ« nÃ« mes -89 dhe 89." IDS_LOADERRORTEXT "Dokumenti %s nuk mund te ngarkohej." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/sv-SE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/sv-SE.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/sv-SE.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -8,6 +8,10 @@ MENUITEM "&Ãppna...\tCtrl+O", IDM_FILEOPEN MENUITEM "&Spara\tCtrl+S", IDM_FILESAVE MENUITEM "Spara so&m...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "Ange som skrivbordsbakgrund (kakelklÃ¤dd)", IDM_FILEASWALLPAPERPLANE MENUITEM "Ange som skrivbordsbakgrund (centrerad)", IDM_FILEASWALLPAPERCENTERED @@ -206,4 +210,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/tr-TR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/tr-TR.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/tr-TR.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "&AÃ§...\tCtrl+O", IDM_FILEOPEN MENUITEM "&Sakla\tCtrl+S", IDM_FILESAVE MENUITEM "Ay&rÄ± Sakla...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "&DÃ¶Åeyerek Duvar KaÄÄ±dÄ± Yap", IDM_FILEASWALLPAPERPLANE MENUITEM "&Ortalayarak Duvar KaÄÄ±dÄ± Yap", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "YÃ¼zde 1'den 500'e dek olmalÄ±dÄ±r." IDS_ANGLE "AÃ§Ä± -89'dan 89'a dek olmalÄ±dÄ±r." IDS_LOADERRORTEXT "%s kÃ¼tÃ¼ÄÃ¼ yÃ¼klenemedi." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/uk-UA.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/uk-UA.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/uk-UA.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "Ð&ÑÐ´ÐºÑÐ¸ÑÐ¸...\tCtrl+O", IDM_FILEOPEN MENUITEM "Ð&Ð±ÐµÑÐµÐ³ÑÐ¸\tCtrl+S", IDM_FILESAVE MENUITEM "ÐÐ±ÐµÑÐµÐ³ÑÐ¸ &ÑÐº...", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "&ÐÐ°Ð¼Ð¾ÑÑÐ¸ÑÐ¸ ÑÐ¾Ð±Ð¾ÑÐ¸Ð¹ ÑÑÑÐ» Windows", IDM_FILEASWALLPAPERPLANE MENUITEM "ÐÐ¾ Ñ&ÐµÐ½ÑÑÑ ÑÐ¾Ð±Ð¾ÑÐ¾Ð³Ð¾ ÑÑÐ¾Ð»Ð° Windows", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/zh-CN.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/zh-CN.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/zh-CN.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "æå¼... (&O)\tCtrl+O", IDM_FILEOPEN MENUITEM "ä¿å(&S)\tCtrl+S", IDM_FILESAVE MENUITEM "å¦åä¸º... (&A)", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "ä¿åä¸ºå£çº¸ (æ®é)", IDM_FILEASWALLPAPERPLANE MENUITEM "ä¿åä¸ºå£çº¸ (å±ä¸)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "ç¾åæ¯å¿é¡»å¨1è³500ä¹é´ã" IDS_ANGLE "è¯¥è§åº¦å¿é¡»å¨-89ä¸89ä¹é´ã" IDS_LOADERRORTEXT "è¯¥æä»¶ %s æ æ³å è½½ã" -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/lang/zh-TW.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/lang/zh-TW.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/lang/zh-TW.rc [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -16,6 +16,10 @@ MENUITEM "éå... (&O)\tCtrl+O", IDM_FILEOPEN MENUITEM "å²å(&S)\tCtrl+S", IDM_FILESAVE MENUITEM "å¦åçº... (&A)", IDM_FILESAVEAS + MENUITEM SEPARATOR + MENUITEM "Page View", IDM_FILEPAGEVIEW + MENUITEM "Page Setup...", IDM_FILEPAGESETUP + MENUITEM "Print...\tCtrl+P", IDM_FILEPRINT MENUITEM SEPARATOR MENUITEM "å²åçºå£ç´ (æ®é)", IDM_FILEASWALLPAPERPLANE MENUITEM "å²åçºå£ç´ (å±ä¸)", IDM_FILEASWALLPAPERCENTERED @@ -214,4 +218,5 @@ IDS_PERCENTAGE "The percentage must be between 1 and 500." IDS_ANGLE "The angle must be between -89 and 89." IDS_LOADERRORTEXT "The file %s could not be loaded." -END + IDS_ENLARGEPROMPTTEXT "The image in the clipboard is larger than the bitmap.\nWould you like the bitmap enlarged?" +END Modified: trunk/reactos/base/applications/mspaint_new/winproc.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/mspaint_… ============================================================================== --- trunk/reactos/base/applications/mspaint_new/winproc.cpp [iso-8859-1] (original) +++ trunk/reactos/base/applications/mspaint_new/winproc.cpp [iso-8859-1] Fri Jul 31 17:33:21 2015 @@ -196,11 +196,14 @@ { if (!imageModel.IsImageSaved()) { - TCHAR programname[20]; - TCHAR saveprompttext[100]; - TCHAR temptext[500]; - LoadString(hProgInstance, IDS_PROGRAMNAME, programname, SIZEOF(programname)); - LoadString(hProgInstance, IDS_SAVEPROMPTTEXT, saveprompttext, SIZEOF(saveprompttext)); + TCHAR* tempptr; + TCHAR programname[LoadString(hProgInstance, IDS_PROGRAMNAME, (LPTSTR)&tempptr, 0) + 1]; + CopyMemory(programname, tempptr, sizeof(programname) - sizeof(TCHAR)); + programname[SIZEOF(programname) - 1] = (TCHAR)'\0'; + TCHAR saveprompttext[LoadString(hProgInstance, IDS_SAVEPROMPTTEXT, (LPTSTR)&tempptr, 0) + 1]; + CopyMemory(saveprompttext, tempptr, sizeof(saveprompttext) - sizeof(TCHAR)); + saveprompttext[SIZEOF(saveprompttext) - 1] = (TCHAR)'\0'; + TCHAR temptext[SIZEOF(saveprompttext) + _tcslen(filename)]; _stprintf(temptext, saveprompttext, filename); switch (MessageBox(temptext, programname, MB_YESNOCANCEL | MB_ICONQUESTION)) { @@ -393,6 +396,22 @@ break; case IDM_FILESAVEAS: saveImage(FALSE); + break; + case IDM_FILEPAGESETUP: + // DUMMY: Shows the dialog only, no functionality + PAGESETUPDLG psd; + ZeroMemory(&psd, sizeof(psd)); + psd.lStructSize = sizeof(psd); + psd.hwndOwner = m_hWnd; + PageSetupDlg(&psd); + break; + case IDM_FILEPRINT: + // DUMMY: Shows the dialog only, no functionality + PRINTDLG pd; + ZeroMemory(&pd, sizeof(pd)); + pd.lStructSize = sizeof(pd); + pd.hwndOwner = m_hWnd; + PrintDlg(&pd); break; case IDM_FILEASWALLPAPERPLANE: SetWallpaper(filepathname, 1, 1);

1 0

[jimtabor] 68583: [NtGdi] - Fix locking.
by jimtabor＠svn.reactos.org 31 Jul '15

31 Jul '15

Author: jimtabor Date: Fri Jul 31 06:55:22 2015 New Revision: 68583 URL: http://svn.reactos.org/svn/reactos?rev=68583&view=rev Log: [NtGdi] - Fix locking. Modified: trunk/reactos/win32ss/gdi/ntgdi/line.c Modified: trunk/reactos/win32ss/gdi/ntgdi/line.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/line.c?r… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/line.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/line.c [iso-8859-1] Fri Jul 31 06:55:22 2015 @@ -60,13 +60,16 @@ INT y, LPPOINT pptOut) { + BOOL Ret; PDC dc; if (!(dc = DC_LockDc(hdc))) { EngSetLastError(ERROR_INVALID_HANDLE); return FALSE; } - return IntGdiMoveToEx(dc, x, y, pptOut, TRUE); + Ret = IntGdiMoveToEx(dc, x, y, pptOut, TRUE); + DC_UnlockDc(dc); + return Ret; } // Should use Fx in pt

1 0

[jimtabor] 68582: [Win32SS] - Start ordering functions into separate files.
by jimtabor＠svn.reactos.org 30 Jul '15

30 Jul '15

Author: jimtabor Date: Thu Jul 30 07:49:09 2015 New Revision: 68582 URL: http://svn.reactos.org/svn/reactos?rev=68582&view=rev Log: [Win32SS] - Start ordering functions into separate files. Added: trunk/reactos/win32ss/user/ntuser/nonclient.c - copied, changed from r68579, trunk/reactos/win32ss/user/ntuser/defwnd.c trunk/reactos/win32ss/user/ntuser/scrollex.c - copied, changed from r68579, trunk/reactos/win32ss/user/ntuser/painting.c Modified: trunk/reactos/win32ss/CMakeLists.txt trunk/reactos/win32ss/user/ntuser/defwnd.c trunk/reactos/win32ss/user/ntuser/painting.c trunk/reactos/win32ss/user/ntuser/painting.h trunk/reactos/win32ss/user/ntuser/userfuncs.h [This mail would be too long, it was shortened to contain the URLs only.] Modified: trunk/reactos/win32ss/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/CMakeLists.txt?rev… Modified: trunk/reactos/win32ss/user/ntuser/defwnd.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/defwnd… Copied: trunk/reactos/win32ss/user/ntuser/nonclient.c (from r68579, trunk/reactos/win32ss/user/ntuser/defwnd.c) URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/noncli… Modified: trunk/reactos/win32ss/user/ntuser/painting.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/painti… Modified: trunk/reactos/win32ss/user/ntuser/painting.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/painti… Copied: trunk/reactos/win32ss/user/ntuser/scrollex.c (from r68579, trunk/reactos/win32ss/user/ntuser/painting.c) URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/scroll… Modified: trunk/reactos/win32ss/user/ntuser/userfuncs.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/userfu…

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