Ros-diffs August 2015

ros-diffs@reactos.org

15 participants
301 discussions

[tfaber] 68615: [WINLOGON] - Load sfc_os.dll because applications have a tendency to create remote threads in winlogon with sfc_os function pointers, assuming that it is loaded. See https://bitsum....
by tfaber＠svn.reactos.org 07 Aug '15

07 Aug '15

Author: tfaber Date: Fri Aug 7 16:28:49 2015 New Revision: 68615 URL: http://svn.reactos.org/svn/reactos?rev=68615&view=rev Log: [WINLOGON] - Load sfc_os.dll because applications have a tendency to create remote threads in winlogon with sfc_os function pointers, assuming that it is loaded. See https://bitsum.com/aboutwfp.asp CORE-9598 #resolve Modified: trunk/reactos/base/system/winlogon/winlogon.c Modified: trunk/reactos/base/system/winlogon/winlogon.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/winlogon/winlo… ============================================================================== --- trunk/reactos/base/system/winlogon/winlogon.c [iso-8859-1] (original) +++ trunk/reactos/base/system/winlogon/winlogon.c [iso-8859-1] Fri Aug 7 16:28:49 2015 @@ -457,6 +457,8 @@ else PostMessageW(WLSession->SASWindow, WLX_WM_SAS, WLX_SAS_TYPE_CTRL_ALT_DEL, 0); + (void)LoadLibraryW(L"sfc_os.dll"); + /* Tell kernel that CurrentControlSet is good (needed * to support Last good known configuration boot) */ NtInitializeRegistry(CM_BOOT_FLAG_ACCEPTED | 1);

1 0

[hbelusca] 68614: Adjust file header according to the previous commit.
by hbelusca＠svn.reactos.org 07 Aug '15

07 Aug '15

Author: hbelusca Date: Fri Aug 7 16:12:06 2015 New Revision: 68614 URL: http://svn.reactos.org/svn/reactos?rev=68614&view=rev Log: Adjust file header according to the previous commit. Modified: trunk/reactos/subsystems/mvdm/ntvdm/hardware/mouse.h Modified: trunk/reactos/subsystems/mvdm/ntvdm/hardware/mouse.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/hard… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/hardware/mouse.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/hardware/mouse.h [iso-8859-1] Fri Aug 7 16:12:06 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: mouse.h - * PURPOSE: Mouse emulation + * PURPOSE: PS/2 Mouse emulation * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> */

1 0

[hbelusca] 68613: [NTVDM] PS2: - By default all the PS/2 ports are disabled. They become enabled by the BIOS at the POST step. - Similarly it is the BIOS POST that sets up the PS/2 controller confi...
by hbelusca＠svn.reactos.org 07 Aug '15

07 Aug '15

Author: hbelusca Date: Fri Aug 7 16:10:18 2015 New Revision: 68613 URL: http://svn.reactos.org/svn/reactos?rev=68613&view=rev Log: [NTVDM] PS2: - By default all the PS/2 ports are disabled. They become enabled by the BIOS at the POST step. - Similarly it is the BIOS POST that sets up the PS/2 controller configuration byte. - Synchronize the value of bit 2 "System flag" and bit 4 "Keyboard enable flag" in the status register, according to what is set in the controller configuration register. What is the "keyboard enable flag" ? See http://www.os2museum.com/wp/the-dos-4-0-shell-mouse-mystery/ for more details... HW MOUSE: - Resetting the mouse sends also an ACKnowledge byte too... BIOS32: - Fix the reported number of bytes in the BIOS configuration table. - Enable the PS/2 ports in the POST. - Implement the "Pointing Device BIOS Interface" INT 15h, AH=C2h, AL=00h...09h based on VBox OSE & SeaBIOS; we should make our PS/2 mouse driver use it. The real call to INT 15h, AH=C2h function is still disabled because our mouse driver doesn't react well with it, when we run some applications like MS Diagnostics. PS2 MOUSE DRV: - Update copyright notice; - Remove 2 useless functions; - I've put in comments in the code the places where calls to the BIOS ps/2 mouse interface are needed. Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios.h trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/bios32.c trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/moubios32.c trunk/reactos/subsystems/mvdm/ntvdm/dos/mouse32.c trunk/reactos/subsystems/mvdm/ntvdm/dos/mouse32.h trunk/reactos/subsystems/mvdm/ntvdm/hardware/mouse.c trunk/reactos/subsystems/mvdm/ntvdm/hardware/ps2.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios.h [iso-8859-1] Fri Aug 7 16:10:18 2015 @@ -142,7 +142,7 @@ */ typedef struct _BIOS_CONFIG_TABLE { - WORD Length; // 0x00 + WORD Length; // 0x00 - Number of bytes following BYTE Model; // 0x02 BYTE SubModel; // 0x03 BYTE Revision; // 0x04 Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/bios32.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/bios32.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/bios32.c [iso-8859-1] Fri Aug 7 16:10:18 2015 @@ -4,6 +4,7 @@ * FILE: bios32.c * PURPOSE: VDM 32-bit BIOS * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> + * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) */ /* INCLUDES *******************************************************************/ @@ -109,7 +110,7 @@ static const BIOS_CONFIG_TABLE BiosConfigTable = { - sizeof(BIOS_CONFIG_TABLE), // Length + sizeof(BIOS_CONFIG_TABLE) - sizeof(((BIOS_CONFIG_TABLE*)0)->Length), // Length: Number of bytes following BIOS_MODEL, // BIOS Model BIOS_SUBMODEL, // BIOS Sub-Model @@ -325,7 +326,6 @@ { setAX(0x80); Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; - break; } @@ -394,8 +394,11 @@ /* Pointing Device BIOS Interface (PS) */ case 0xC2: { - BiosMousePs2Interface(Stack); - UNIMPLEMENTED; // Remove it when BiosMousePs2Interface is implemented! + // FIXME: Reenable this call when we understand why + // our included mouse driver doesn't correctly reeanble + // mouse reporting! + // BiosMousePs2Interface(Stack); + // break; goto Default; } @@ -429,7 +432,7 @@ ULONG Above1M = (min(MAX_ADDRESS, 0x01000000) - 0x00100000) >> 10; /* The amount of memory above 16M, in 64K blocks */ - ULONG Above16M = (MAX_ADDRESS > 0x01000000) ? (MAX_ADDRESS - 0x01000000) >> 16: 0; + ULONG Above16M = (MAX_ADDRESS > 0x01000000) ? ((MAX_ADDRESS - 0x01000000) >> 16) : 0; setAX(Above1M); setBX(Above16M); @@ -476,8 +479,7 @@ else { DPRINT1("BIOS Function INT 15h, AH = 0xE8 - unexpected AL = %02X, EDX = %08X\n", - getAL(), - getEDX()); + getAL(), getEDX()); } break; @@ -773,6 +775,16 @@ IOWriteB(PIT_DATA_PORT(2), 0x97); IOWriteB(PIT_DATA_PORT(2), 0x0A); + + /* Initialize PS/2 keyboard port */ + // Enable the port + IOWriteB(PS2_CONTROL_PORT, 0xAE); + IOWriteB(PS2_CONTROL_PORT, 0x60); + // Port interrupts and clock enabled, + // enable keyboard scancode translation. + // POST passed, force keyboard unlocking. + IOWriteB(PS2_DATA_PORT , 0x4D); + EnableHwIRQ(0, BiosTimerIrq); } @@ -785,7 +797,10 @@ /* Register the default BIOS interrupt vectors */ - /* Zero out all of the IVT (0x00 -- 0xFF) */ + /* + * Zero out all of the IVT (0x00 -- 0xFF). Some applications + * indeed expect to have free vectors at the end of the IVT. + */ RtlZeroMemory(BaseAddress, 0x0100 * sizeof(ULONG)); #if defined(ADVANCED_DEBUGGING) && (ADVANCED_DEBUGGING_LEVEL >= 3) Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/moubios32.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/moubios32.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/moubios32.c [iso-8859-1] Fri Aug 7 16:10:18 2015 @@ -2,8 +2,11 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: moubios32.c - * PURPOSE: VDM Mouse 32-bit BIOS + * PURPOSE: VDM 32-bit PS/2 Mouse BIOS * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> + * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) + * + * NOTE: Based from VirtualBox OSE ROM BIOS, and SeaBIOS. */ /* INCLUDES *******************************************************************/ @@ -12,118 +15,512 @@ #include "ntvdm.h" #include "emulator.h" +#include "cpu/cpu.h" // for EMULATOR_FLAG_CF #include "moubios32.h" #include "bios32p.h" #include "io.h" +#include "hardware/mouse.h" #include "hardware/ps2.h" /* PRIVATE VARIABLES **********************************************************/ +#define MOUSE_IRQ_INT 0x74 + +static BOOLEAN MouseEnabled = FALSE; +static DWORD OldIrqHandler; + +/* + * Far pointer to a device handler. In compatible PS/2, it is stored in the EBDA. + * + * See Ralf Brown: http://www.ctyme.com/intr/rb-1603.htm + * for more information. In particular: + * when the subroutine is called, it is given 4 WORD values on the stack; + * the handler should return with a FAR return without popping the stack. + */ +static ULONG DeviceHandler = 0; + /* PRIVATE FUNCTIONS **********************************************************/ + +static VOID DisableMouseInt(VOID) +{ + BYTE ControllerConfig; + + /* Clear the mouse queue */ + while (PS2PortQueueRead(1)) ; // NOTE: Should be a IOReadB! But see r67231 + + /* Disable mouse interrupt and events */ + IOWriteB(PS2_CONTROL_PORT, 0x20); + ControllerConfig = IOReadB(PS2_DATA_PORT); + ControllerConfig &= ~0x02; // Turn off IRQ12 + ControllerConfig |= 0x20; // Disable mouse clock line + IOWriteB(PS2_CONTROL_PORT, 0x60); + IOWriteB(PS2_DATA_PORT, ControllerConfig); +} + +static VOID EnableMouseInt(VOID) +{ + BYTE ControllerConfig; + + /* Clear the mouse queue */ + while (PS2PortQueueRead(1)) ; // NOTE: Should be a IOReadB! But see r67231 + + /* Enable mouse interrupt and events */ + IOWriteB(PS2_CONTROL_PORT, 0x20); + ControllerConfig = IOReadB(PS2_DATA_PORT); + ControllerConfig |= 0x02; // Turn on IRQ12 + ControllerConfig &= ~0x20; // Enable mouse clock line + IOWriteB(PS2_CONTROL_PORT, 0x60); + IOWriteB(PS2_DATA_PORT, ControllerConfig); +} + +static inline +VOID SendMouseCommand(UCHAR Command) +{ + /* Clear the mouse queue */ + while (PS2PortQueueRead(1)) ; // NOTE: Should be a IOReadB! But see r67231 + + /* Send the command */ + IOWriteB(PS2_CONTROL_PORT, 0xD4); + IOWriteB(PS2_DATA_PORT, Command); +} + +static inline +UCHAR ReadMouseData(VOID) +{ + PS2PortQueueRead(1); // NOTE: Should be a IOReadB! But see r67231 + return IOReadB(PS2_DATA_PORT); +} + + +static +VOID BiosMouseEnable(VOID) +{ + if (MouseEnabled) return; + + MouseEnabled = TRUE; + + /* Get the old IRQ handler */ + OldIrqHandler = ((PDWORD)BaseAddress)[MOUSE_IRQ_INT]; + + /* Set the IRQ handler */ + //RegisterInt32(MAKELONG(FIELD_OFFSET(MOUSE_DRIVER, MouseIrqInt16Stub), MouseDataSegment), + // MOUSE_IRQ_INT, DosMouseIrq, NULL); +} + +static +VOID BiosMouseDisable(VOID) +{ + if (!MouseEnabled) return; + + /* Restore the old IRQ handler */ + // ((PDWORD)BaseAddress)[MOUSE_IRQ_INT] = OldIrqHandler; + + MouseEnabled = FALSE; +} + // Mouse IRQ 12 static VOID WINAPI BiosMouseIrq(LPWORD Stack) { + DPRINT1("PS/2 Mouse IRQ! DeviceHandler = 0x%04X:0x%04X\n", + HIWORD(DeviceHandler), LOWORD(DeviceHandler)); + + if (DeviceHandler != 0) + { + /* + * Prepare the stack for the mouse device handler: + * push Status, X and Y data, and a zero word. + */ + setSP(getSP() - sizeof(WORD)); + *((LPWORD)SEG_OFF_TO_PTR(getSS(), getSP())) = 0; // Status + setSP(getSP() - sizeof(WORD)); + *((LPWORD)SEG_OFF_TO_PTR(getSS(), getSP())) = 0; // X data (high byte = 0) + setSP(getSP() - sizeof(WORD)); + *((LPWORD)SEG_OFF_TO_PTR(getSS(), getSP())) = 0; // Y data (high byte = 0) + setSP(getSP() - sizeof(WORD)); + *((LPWORD)SEG_OFF_TO_PTR(getSS(), getSP())) = 0; // Zero + + /* Call the device handler */ + RunCallback16(&BiosContext, DeviceHandler); + + /* Pop the stack */ + setSP(getSP() + 4*sizeof(WORD)); + } + PicIRQComplete(LOBYTE(Stack[STACK_INT_NUM])); } VOID BiosMousePs2Interface(LPWORD Stack) { - DPRINT1("INT 15h, AH = C2h must be implemented in order to support vendor mouse drivers\n"); + /* Disable mouse interrupt and events */ + DisableMouseInt(); switch (getAL()) { /* Enable / Disable */ case 0x00: { - break; + UCHAR State = getBH(); + + if (State > 2) + { + /* Invalid function */ + setAH(0x01); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + if (State == 0x00) + { + BiosMouseDisable(); + + /* Disable packet reporting */ + SendMouseCommand(0xF5); + } + else // if (State == 0x01) + { + /* Check for the presence of the device handler */ + if (DeviceHandler == 0) + { + /* No device handler installed */ + setAH(0x05); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + BiosMouseEnable(); + + /* Enable packet reporting */ + SendMouseCommand(0xF4); + } + + if (ReadMouseData() != MOUSE_ACK) + { + /* Failure */ + setAH(0x03); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + /* Success */ + setAH(0x00); + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; + break; + } + + /* Initialize */ + case 0x05: + { + // Fall through } /* Reset */ case 0x01: { + UCHAR Answer; + + SendMouseCommand(0xFF); + Answer = ReadMouseData(); + /* A "Resend" (0xFE) signal is sent if no mouse is attached */ + if (Answer == 0xFE) + { + /* Resend */ + setAH(0x04); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + else if (Answer != MOUSE_ACK) + { + /* Failure */ + setAH(0x03); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + setBL(ReadMouseData()); // Should be MOUSE_BAT_SUCCESS + setBH(ReadMouseData()); // Mouse ID + + /* Success */ + setAH(0x00); + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; break; } /* Set Sampling Rate */ case 0x02: { + UCHAR SampleRate = 0; + + switch (getBH()) + { + case 0x00: SampleRate = 10; break; // 10 reports/sec + case 0x01: SampleRate = 20; break; // 20 " " + case 0x02: SampleRate = 40; break; // 40 " " + case 0x03: SampleRate = 60; break; // 60 " " + case 0x04: SampleRate = 80; break; // 80 " " + case 0x05: SampleRate = 100; break; // 100 " " + case 0x06: SampleRate = 200; break; // 200 " " + default: SampleRate = 0; + } + + if (SampleRate == 0) + { + /* Invalid input */ + setAH(0x02); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + SendMouseCommand(0xF3); + if (ReadMouseData() != MOUSE_ACK) + { + /* Failure */ + setAH(0x03); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + SendMouseCommand(SampleRate); + if (ReadMouseData() != MOUSE_ACK) + { + /* Failure */ + setAH(0x03); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + /* Success */ + setAH(0x00); + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; break; } /* Set Resolution */ case 0x03: { + UCHAR Resolution = getBH(); + + /* + * 0: 25 dpi, 1 count per millimeter + * 1: 50 dpi, 2 counts per millimeter + * 2: 100 dpi, 4 counts per millimeter + * 3: 200 dpi, 8 counts per millimeter + */ + if (Resolution > 3) + { + /* Invalid input */ + setAH(0x02); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + SendMouseCommand(0xE8); + if (ReadMouseData() != MOUSE_ACK) + { + /* Failure */ + setAH(0x03); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + SendMouseCommand(Resolution); + if (ReadMouseData() != MOUSE_ACK) + { + /* Failure */ + setAH(0x03); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + /* Success */ + setAH(0x00); + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; break; } /* Get Type */ case 0x04: { - break; - } - - /* Initialize */ - case 0x05: - { - break; - } - - /* Extended Commands */ + SendMouseCommand(0xF2); + if (ReadMouseData() != MOUSE_ACK) + { + /* Failure */ + setAH(0x03); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + setBH(ReadMouseData()); + + /* Success */ + setAH(0x00); + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; + break; + } + + /* Extended Commands (Return Status and Set Scaling Factor) */ case 0x06: { + UCHAR Command = getBH(); + + switch (Command) + { + /* Return Status */ + case 0x00: + { + SendMouseCommand(0xE9); + if (ReadMouseData() != MOUSE_ACK) + { + /* Failure */ + setAH(0x03); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + } + + setBL(ReadMouseData()); // Status + setCL(ReadMouseData()); // Resolution + setDL(ReadMouseData()); // Sample rate + + /* Success */ + setAH(0x00); + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; + break; + } + + /* Set Scaling Factor to 1:1 */ + case 0x01: + /* Set Scaling Factor to 2:1 */ + case 0x02: + { + SendMouseCommand(Command == 0x01 ? 0xE6 : 0xE7); + if (ReadMouseData() != MOUSE_ACK) + { + /* Failure */ + setAH(0x03); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + } + + /* Success */ + setAH(0x00); + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; + break; + } + + default: + { + /* Invalid function */ + setAH(0x01); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + } + break; } /* Set Device Handler Address */ case 0x07: { + /* ES:BX == 0000h:0000h removes the device handler */ + DeviceHandler = MAKELONG(getBX(), getES()); + + /* Success */ + setAH(0x00); + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; break; } /* Write to Pointer Port */ case 0x08: { + SendMouseCommand(getBL()); + if (ReadMouseData() != MOUSE_ACK) + { + /* Failure */ + setAH(0x03); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; + break; + } + + /* Success */ + setAH(0x00); + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; break; } /* Read from Pointer Port */ case 0x09: { + setBL(ReadMouseData()); + setCL(ReadMouseData()); + setDL(ReadMouseData()); + + /* Success */ + setAH(0x00); + Stack[STACK_FLAGS] &= ~EMULATOR_FLAG_CF; break; } default: { - DPRINT1("INT 15h, AH = C2h, AL = 0x%02X NOT IMPLEMENTED\n", + DPRINT1("INT 15h, AH = C2h, AL = %02Xh NOT IMPLEMENTED\n", getAL()); + + /* Unknown function */ + setAH(0x01); + Stack[STACK_FLAGS] |= EMULATOR_FLAG_CF; } } + + /* Reenable mouse interrupt and events */ + EnableMouseInt(); } /* PUBLIC FUNCTIONS ***********************************************************/ VOID MouseBios32Post(VOID) { - BYTE ControllerConfig; - - /* Clear the mouse queue */ - while (PS2PortQueueRead(1)) continue; - - /* Enable packet reporting */ - IOWriteB(PS2_CONTROL_PORT, 0xD4); - IOWriteB(PS2_DATA_PORT, 0xF4); - - /* Read the mouse ACK reply */ - PS2PortQueueRead(1); - - /* Enable IRQ12 */ - IOWriteB(PS2_CONTROL_PORT, 0x20); - ControllerConfig = IOReadB(PS2_DATA_PORT); - IOWriteB(PS2_CONTROL_PORT, 0x60); - IOWriteB(PS2_DATA_PORT, ControllerConfig | 0x02); + UCHAR Answer; + + /* Initialize PS/2 mouse port */ + // Enable the port + IOWriteB(PS2_CONTROL_PORT, 0xA8); + + /* Detect mouse presence by attempting a reset */ + SendMouseCommand(0xFF); + Answer = ReadMouseData(); + /* If no mouse attached, it will return RESEND (0xFE) */ + if (Answer == 0xFE) + { + DPRINT1("No mouse present!\n"); + } + else if (Answer != MOUSE_ACK) + { + DPRINT1("Mouse reset failure!\n"); + } + else + { + /* Mouse present, try to completely enable it */ + + // FIXME: The following is temporary until + // this is moved into the mouse driver!! + + /* Enable packet reporting */ + SendMouseCommand(0xF4); + if (ReadMouseData() != MOUSE_ACK) + { + DPRINT1("Failed to enable mouse!\n"); + } + else + { + /* Enable mouse interrupt and events */ + EnableMouseInt(); + } + } + + /* No mouse driver available so far */ + RegisterBiosInt32(0x33, NULL); /* Set up the HW vector interrupts */ EnableHwIRQ(12, BiosMouseIrq); Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/mouse32.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/mouse32.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/mouse32.c [iso-8859-1] Fri Aug 7 16:10:18 2015 @@ -2,13 +2,17 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: mouse32.c - * PURPOSE: VDM 32-bit compatible MOUSE.COM driver + * PURPOSE: VDM 32-bit compatible PS/2 MOUSE.COM driver * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> */ /* INCLUDES *******************************************************************/ #define NDEBUG + +/* Driver Version number and Copyright */ +#include <reactos/buildno.h> +#include <reactos/version.h> #include "ntvdm.h" #include "emulator.h" @@ -30,7 +34,10 @@ /* PRIVATE VARIABLES **********************************************************/ -static const CHAR MouseCopyright[] = "ROS PS/2 16/32-bit Mouse Driver Compatible MS-MOUSE 6.26 Copyright (C) ReactOS Team 1996-2015\0"; +static const CHAR MouseCopyright[] = + "ReactOS PS/2 16/32-bit Mouse Driver Compatible MS-MOUSE 6.26\r\n" + "Version "KERNEL_VERSION_STR" (Build "KERNEL_VERSION_BUILD_STR")\r\n" + "Copyright (C) ReactOS Team 1996-"COPYRIGHT_YEAR"\0"; #pragma pack(push, 1) @@ -58,7 +65,79 @@ static DWORD OldIrqHandler; static DWORD OldIntHandler; +static WORD DefaultGfxScreenMask[16] = +{ + 0xC3FF, // 1100001111111111 + 0xC0FF, // 1100000011111111 + 0xC07F, // 1100000001111111 + 0xC01F, // 1100000000011111 + 0xC00F, // 1100000000001111 + 0xC007, // 1100000000000111 + 0xC003, // 1100000000000011 + 0xC007, // 1100000000000111 + 0xC01F, // 1100000000011111 + 0xC01F, // 1100000000011111 + 0xC00F, // 1100000000001111 + 0xC60F, // 1100011000001111 + 0xFF07, // 1111111100000111 + 0xFF07, // 1111111100000111 + 0xFF87, // 1111111110000111 + 0xFFCF, // 1111111111001111 +}; + +static WORD DefaultGfxCursorMask[16] = +{ + 0x0000, // 0000000000000000 + 0x1C00, // 0001110000000000 + 0x1F00, // 0001111100000000 + 0x1F80, // 0001111110000000 + 0x1FE0, // 0001111111100000 + 0x1FF0, // 0001111111110000 + 0x1FF8, // 0001111111111000 + 0x1FE0, // 0001111111100000 + 0x1FC0, // 0001111111000000 + 0x1FC0, // 0001111111000000 + 0x19E0, // 0001100111100000 + 0x00E0, // 0000000011100000 + 0x0070, // 0000000001110000 + 0x0070, // 0000000001110000 + 0x0030, // 0000000000110000 + 0x0000, // 0000000000000000 +}; + /* PRIVATE FUNCTIONS **********************************************************/ + +/* static */ +VOID BiosPs2Service(UCHAR Function) +{ + /* Save AX and BX */ + USHORT AX = getAX(); + // USHORT BX = getBX(); + + /* + * Set the parameters: + * AL contains the character to print (already set), + * BL contains the character attribute, + * BH contains the video page to use. + */ + // setBL(DOS_CHAR_ATTRIBUTE); + // setBH(Bda->VideoPage); + setAL(Function); + + /* Call the BIOS INT 15h, AH=C2h "Pointing Device BIOS Interface (PS)" */ + setAH(0xC2); + Int32Call(&MouseContext, BIOS_MISC_INTERRUPT); + + /* Restore AX and BX */ + // setBX(BX); + setAX(AX); +} + + + +static VOID DosMouseEnable(VOID); +static VOID DosMouseDisable(VOID); + static VOID PaintMouseCursor(VOID) { @@ -273,7 +352,7 @@ CallMouseUserHandlers(0x0001); // We use MS MOUSE v1.0+ format } -static inline VOID DosUpdateButtons(BYTE ButtonState) +static inline VOID DosUpdateButtons(BYTE ButtonState) // WORD ButtonState { USHORT i; USHORT CallMask = 0x0000; // We use MS MOUSE v1.0+ format @@ -316,9 +395,9 @@ /* Read the whole packet at once */ Flags = IOReadB(PS2_DATA_PORT); - PS2PortQueueRead(1); + PS2PortQueueRead(1); // NOTE: Should be a IOReadB! But see r67231 DeltaX = IOReadB(PS2_DATA_PORT); - PS2PortQueueRead(1); + PS2PortQueueRead(1); // NOTE: Should be a IOReadB! But see r67231 DeltaY = IOReadB(PS2_DATA_PORT); /* Adjust the sign */ @@ -369,39 +448,13 @@ DriverState.GraphicsCursor.HotSpot.X = 3; DriverState.GraphicsCursor.HotSpot.Y = 1; - DriverState.GraphicsCursor.ScreenMask[0] = 0xC3FF; // 1100001111111111 - DriverState.GraphicsCursor.ScreenMask[1] = 0xC0FF; // 1100000011111111 - DriverState.GraphicsCursor.ScreenMask[2] = 0xC07F; // 1100000001111111 - DriverState.GraphicsCursor.ScreenMask[3] = 0xC01F; // 1100000000011111 - DriverState.GraphicsCursor.ScreenMask[4] = 0xC00F; // 1100000000001111 - DriverState.GraphicsCursor.ScreenMask[5] = 0xC007; // 1100000000000111 - DriverState.GraphicsCursor.ScreenMask[6] = 0xC003; // 1100000000000011 - DriverState.GraphicsCursor.ScreenMask[7] = 0xC007; // 1100000000000111 - DriverState.GraphicsCursor.ScreenMask[8] = 0xC01F; // 1100000000011111 - DriverState.GraphicsCursor.ScreenMask[9] = 0xC01F; // 1100000000011111 - DriverState.GraphicsCursor.ScreenMask[10] = 0xC00F; // 1100000000001111 - DriverState.GraphicsCursor.ScreenMask[11] = 0xC60F; // 1100011000001111 - DriverState.GraphicsCursor.ScreenMask[12] = 0xFF07; // 1111111100000111 - DriverState.GraphicsCursor.ScreenMask[13] = 0xFF07; // 1111111100000111 - DriverState.GraphicsCursor.ScreenMask[14] = 0xFF87; // 1111111110000111 - DriverState.GraphicsCursor.ScreenMask[15] = 0xFFCF; // 1111111111001111 - - DriverState.GraphicsCursor.CursorMask[0] = 0x0000; // 0000000000000000 - DriverState.GraphicsCursor.CursorMask[1] = 0x1C00; // 0001110000000000 - DriverState.GraphicsCursor.CursorMask[2] = 0x1F00; // 0001111100000000 - DriverState.GraphicsCursor.CursorMask[3] = 0x1F80; // 0001111110000000 - DriverState.GraphicsCursor.CursorMask[4] = 0x1FE0; // 0001111111100000 - DriverState.GraphicsCursor.CursorMask[5] = 0x1FF0; // 0001111111110000 - DriverState.GraphicsCursor.CursorMask[6] = 0x1FF8; // 0001111111111000 - DriverState.GraphicsCursor.CursorMask[7] = 0x1FE0; // 0001111111100000 - DriverState.GraphicsCursor.CursorMask[8] = 0x1FC0; // 0001111111000000 - DriverState.GraphicsCursor.CursorMask[9] = 0x1FC0; // 0001111111000000 - DriverState.GraphicsCursor.CursorMask[10] = 0x19E0; // 0001100111100000 - DriverState.GraphicsCursor.CursorMask[11] = 0x00E0; // 0000000011100000 - DriverState.GraphicsCursor.CursorMask[12] = 0x0070; // 0000000001110000 - DriverState.GraphicsCursor.CursorMask[13] = 0x0070; // 0000000001110000 - DriverState.GraphicsCursor.CursorMask[14] = 0x0030; // 0000000000110000 - DriverState.GraphicsCursor.CursorMask[15] = 0x0000; // 0000000000000000 + RtlCopyMemory(DriverState.GraphicsCursor.ScreenMask, + DefaultGfxScreenMask, + sizeof(DriverState.GraphicsCursor.ScreenMask)); + + RtlCopyMemory(DriverState.GraphicsCursor.CursorMask, + DefaultGfxCursorMask, + sizeof(DriverState.GraphicsCursor.CursorMask)); /* Initialize the counters */ DriverState.HorizCount = DriverState.VertCount = 0; @@ -436,7 +489,7 @@ break; } - /* Return Position And Button Status */ + /* Return Position and Button Status */ case 0x03: { COORD Position = DriverState.Position; @@ -610,6 +663,13 @@ break; } + /* Set Exclusion Area */ + // http://www.ctyme.com/intr/rb-5972.htm + // http://www.techhelpmanual.com/849-int_33h_0010h__set_exclusion_area.html + //case 0x10: + //{ + //} + /* Define Double-Speed Threshold */ case 0x13: { @@ -625,12 +685,18 @@ DriverState.Handler0.CallMask = getCX(); DriverState.Handler0.Callback = MAKELONG(getDX(), getES()); // Far pointer to the callback + DPRINT1("Exchange old callback 0x%04X, %04X:%04X with new callback 0x%04X, %04X:%04X\n", + OldCallMask, + HIWORD(OldCallback), + LOWORD(OldCallback), + DriverState.Handler0.CallMask, + HIWORD(DriverState.Handler0.Callback), + LOWORD(DriverState.Handler0.Callback)); /* Return old callmask in CX and callback vector in ES:DX */ setCX(OldCallMask); setES(HIWORD(OldCallback)); setDX(LOWORD(OldCallback)); - break; } @@ -644,6 +710,9 @@ /* Save Driver State */ case 0x16: { + /* Check whether the user buffer has correct size and fail if not */ + if (getBX() != sizeof(MOUSE_DRIVER_STATE)) break; + *((PMOUSE_DRIVER_STATE)SEG_OFF_TO_PTR(getES(), getDX())) = DriverState; break; } @@ -651,6 +720,9 @@ /* Restore Driver State */ case 0x17: { + /* Check whether the user buffer has correct size and fail if not */ + if (getBX() != sizeof(MOUSE_DRIVER_STATE)) break; + DriverState = *((PMOUSE_DRIVER_STATE)SEG_OFF_TO_PTR(getES(), getDX())); break; } @@ -803,6 +875,15 @@ case 0x1A: { DPRINT1("INT 33h, AH=1Ah: Mouse sensitivity is UNSUPPORTED\n"); + + // FIXME: Do that at runtime! + + // UCHAR BH = getBH(); + // setBH(0x00); + // BiosPs2Service(0x00); + // FIXME: Check for return status in AH and CF + // setBH(BH); + break; } @@ -815,6 +896,15 @@ setBX(50); // Horizontal speed setCX(50); // Vertical speed setDX(50); // Double speed threshold + + // FIXME: Get that at runtime! + + // UCHAR BH = getBH(); + // setBH(0x00); + // BiosPs2Service(0x00); + // FIXME: Check for return status in AH and CF + // setBH(BH); + break; } @@ -826,6 +916,11 @@ setBX(LOWORD(OldIntHandler)); DosMouseDisable(); + // UCHAR BH = getBH(); + // setBH(0x00); + // BiosPs2Service(0x00); + // FIXME: Check for return status in AH and CF + // setBH(BH); break; } @@ -833,6 +928,11 @@ case 0x20: { DosMouseEnable(); + // UCHAR BH = getBH(); + // setBH(0x01); + // BiosPs2Service(0x00); + // FIXME: Check for return status in AH and CF + // setBH(BH); break; } @@ -876,12 +976,70 @@ { setBX(MOUSE_VERSION); // Version Number + /* + * See Ralf Brown: http://www.ctyme.com/intr/rb-5993.htm + * for the list of possible values. + */ // FIXME: To be determined at runtime! setCH(0x04); // PS/2 Type setCL(0x00); // PS/2 Interrupt break; } + + // BIOS Function INT 33h, AX = 0x0025 NOT IMPLEMENTED + case 0x25: + { + setAX(0); + setBX(0); + setCX(0); + setDX(0); + UNIMPLEMENTED; + break; + } + + /* Get Maximum Virtual Coordinates */ + case 0x26: + { + setBX(!DriverEnabled); + // FIXME: In fact the MaxX and MaxY here are + // theoretical values for the current video mode. + // They therefore can be different from the current + // min/max values. + // See http://www.ctyme.com/intr/rb-5995.htm + // for more details. + setCX(DriverState.MaxX); + setDX(DriverState.MaxY); + break; + } + + /* Get Current Minimum/Maximum Virtual Coordinates */ + case 0x31: + { + setAX(DriverState.MinX); + setBX(DriverState.MinY); + setCX(DriverState.MaxX); + setDX(DriverState.MaxY); + break; + } + +#if 0 + case 0x33: + { + /* + * Related to http://www.ctyme.com/intr/rb-5985.htm + * INT 33h, AX=001Ch "SET INTERRUPT RATE": + + * Values for mouse interrupt rate: + * BX = rate + 00h no interrupts allowed + 01h 30 per second + 02h 50 per second + 03h 100 per second + 04h 200 per second + */ + } +#endif /* Return Pointer to Copyright String */ case 0x4D: @@ -914,6 +1072,7 @@ /* PUBLIC FUNCTIONS ***********************************************************/ +static VOID DosMouseEnable(VOID) { if (DriverEnabled) return; @@ -928,6 +1087,7 @@ MOUSE_IRQ_INT, DosMouseIrq, NULL); } +static VOID DosMouseDisable(VOID) { if (!DriverEnabled) return; @@ -936,61 +1096,6 @@ ((PDWORD)BaseAddress)[MOUSE_IRQ_INT] = OldIrqHandler; DriverEnabled = FALSE; -} - -VOID DosMouseUpdatePosition(PCOORD NewPosition) -{ - SHORT DeltaX = NewPosition->X - DriverState.Position.X; - SHORT DeltaY = NewPosition->Y - DriverState.Position.Y; - - if (!DriverEnabled) return; - - DriverState.HorizCount += (DeltaX * MICKEYS_PER_CELL_HORIZ) / 8; - DriverState.VertCount += (DeltaY * MICKEYS_PER_CELL_VERT) / 8; - - if (DriverState.ShowCount > 0) EraseMouseCursor(); - DriverState.Position = *NewPosition; - if (DriverState.ShowCount > 0) PaintMouseCursor(); - - /* Call the mouse handlers */ - // if (DeltaX || DeltaY) - CallMouseUserHandlers(0x0001); // We use MS MOUSE v1.0+ format -} - -VOID DosMouseUpdateButtons(WORD ButtonState) -{ - USHORT i; - USHORT CallMask = 0x0000; // We use MS MOUSE v1.0+ format - - if (!DriverEnabled) return; - - for (i = 0; i < NUM_MOUSE_BUTTONS; i++) - { - BOOLEAN OldState = (DriverState.ButtonState >> i) & 1; - BOOLEAN NewState = (ButtonState >> i) & 1; - - if (NewState > OldState) - { - /* Mouse press */ - DriverState.PressCount[i]++; - DriverState.LastPress[i] = DriverState.Position; - - CallMask |= (1 << (2 * i + 1)); - } - else if (NewState < OldState) - { - /* Mouse release */ - DriverState.ReleaseCount[i]++; - DriverState.LastRelease[i] = DriverState.Position; - - CallMask |= (1 << (2 * i + 2)); - } - } - - DriverState.ButtonState = ButtonState; - - /* Call the mouse handlers */ - CallMouseUserHandlers(CallMask); } BOOLEAN DosMouseInitialize(VOID) @@ -1020,6 +1125,12 @@ DOS_MOUSE_INTERRUPT, DosMouseService, NULL); DosMouseEnable(); + // UCHAR BH = getBH(); + // setBH(0x01); + // BiosPs2Service(0x00); + // FIXME: Check for return status in AH and CF + // setBH(BH); + return TRUE; } @@ -1027,6 +1138,11 @@ { if (DriverState.ShowCount > 0) EraseMouseCursor(); DosMouseDisable(); + // UCHAR BH = getBH(); + // setBH(0x00); + // BiosPs2Service(0x00); + // FIXME: Check for return status in AH and CF + // setBH(BH); /* Restore the old mouse service interrupt handler */ ((PDWORD)BaseAddress)[DOS_MOUSE_INTERRUPT] = OldIntHandler; Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/mouse32.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/mouse32.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/mouse32.h [iso-8859-1] Fri Aug 7 16:10:18 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: mouse32.h - * PURPOSE: VDM 32-bit compatible MOUSE.COM driver + * PURPOSE: VDM 32-bit compatible PS/2 MOUSE.COM driver * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> */ @@ -12,7 +12,7 @@ /* DEFINES ********************************************************************/ // -// We are ROS PS/2 Mouse Driver Version 6.26, compatible MS-MOUSE 6.26 +// We are ReactOS PS/2 Mouse Driver Version 6.26, compatible MS-MOUSE 6.26 // #define MOUSE_VERSION MAKEWORD(0x26, 0x06) @@ -75,8 +75,6 @@ /* FUNCTIONS ******************************************************************/ -VOID DosMouseEnable(VOID); -VOID DosMouseDisable(VOID); BOOLEAN DosMouseInitialize(VOID); VOID DosMouseCleanup(VOID); Modified: trunk/reactos/subsystems/mvdm/ntvdm/hardware/mouse.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/hard… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/hardware/mouse.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/hardware/mouse.c [iso-8859-1] Fri Aug 7 16:10:18 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: mouse.c - * PURPOSE: Mouse emulation + * PURPOSE: PS/2 Mouse emulation * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> */ @@ -39,11 +39,11 @@ static BYTE DataByteWait = 0; static BYTE ScrollMagicCounter = 0, ExtraButtonMagicCounter = 0; +static UINT MouseCycles = 10; + static BYTE PS2Port = 1; /* PUBLIC VARIABLES ***********************************************************/ - -UINT MouseCycles = 10; /* PRIVATE FUNCTIONS **********************************************************/ @@ -73,6 +73,8 @@ MouseId = 0; ScrollMagicCounter = ExtraButtonMagicCounter = 0; + PS2QueuePush(PS2Port, MOUSE_ACK); + /* Send the Basic Assurance Test success code and the device ID */ PS2QueuePush(PS2Port, MOUSE_BAT_SUCCESS); PS2QueuePush(PS2Port, MouseId); @@ -118,7 +120,7 @@ /* Set the button flags */ if (ButtonState & FROM_LEFT_1ST_BUTTON_PRESSED) Packet->Flags |= MOUSE_LEFT_BUTTON; if (ButtonState & FROM_LEFT_2ND_BUTTON_PRESSED) Packet->Flags |= MOUSE_MIDDLE_BUTTON; - if (ButtonState & RIGHTMOST_BUTTON_PRESSED) Packet->Flags |= MOUSE_RIGHT_BUTTON; + if (ButtonState & RIGHTMOST_BUTTON_PRESSED) Packet->Flags |= MOUSE_RIGHT_BUTTON; if (MouseId == 4) { @@ -250,8 +252,8 @@ /* Set Sample Rate */ case 0xF3: { - PS2QueuePush(PS2Port, MOUSE_ACK); DataByteWait = Command; + PS2QueuePush(PS2Port, MOUSE_ACK); break; } @@ -259,12 +261,12 @@ case 0xE9: { BYTE Status = ButtonState & 7; - PS2QueuePush(PS2Port, MOUSE_ACK); if (Scaling) Status |= 1 << 4; if (Reporting) Status |= 1 << 5; if (Mode == MOUSE_REMOTE_MODE) Status |= 1 << 6; + PS2QueuePush(PS2Port, MOUSE_ACK); PS2QueuePush(PS2Port, Status); PS2QueuePush(PS2Port, Resolution); PS2QueuePush(PS2Port, (BYTE)(1000 / MouseCycles)); @@ -290,7 +292,7 @@ break; } - /* Return From Wrap Mode */ + /* Return from Wrap Mode */ case 0xEC: { if (Mode == MOUSE_WRAP_MODE) @@ -300,7 +302,10 @@ Mode = PreviousMode; PS2QueuePush(PS2Port, MOUSE_ACK); } - else PS2QueuePush(PS2Port, MOUSE_ERROR); + else + { + PS2QueuePush(PS2Port, MOUSE_ERROR); + } break; } Modified: trunk/reactos/subsystems/mvdm/ntvdm/hardware/ps2.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/hard… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/hardware/ps2.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/hardware/ps2.c [iso-8859-1] Fri Aug 7 16:10:18 2015 @@ -45,8 +45,7 @@ #define PS2_PORTS 2 static PS2_PORT Ports[PS2_PORTS]; -#define PS2_DEFAULT_CONFIG 0x45 -static BYTE ControllerConfig = PS2_DEFAULT_CONFIG; +static BYTE ControllerConfig = 0x00; static BYTE ControllerCommand = 0x00; static BYTE StatusRegister = 0x00; @@ -69,8 +68,16 @@ { if (Port == PS2_CONTROL_PORT) { - /* Be sure bit 2 is always set */ - StatusRegister |= 1 << 2; + /* + * Be sure bit 4 "Keyboard enable flag" is always set. + * Keyboard enable (or keyboard lock) flag values are: + * 0: Locked; 1: Not locked. + * On IBM PC-ATs this is the state of the hardware keyboard + * lock mechanism. It is not widely used, but some programs + * still use it: see for example: + * http://www.os2museum.com/wp/the-dos-4-0-shell-mouse-mystery/ + */ + StatusRegister |= (1 << 4); // FIXME: Should clear bits 6 and 7 because there are // no timeouts and no parity errors. @@ -220,6 +227,31 @@ case 0x60: { ControllerConfig = Data; + + /* + * Update bit 2 "System flag" of the status register + * with bit 2 of the controller configuration byte. + * See: http://www.win.tue.nl/~aeb/linux/kbd/scancodes-11.html#kccb2 + * for more details. + */ + if (ControllerConfig & (1 << 2)) + StatusRegister |= (1 << 2); + else + StatusRegister &= ~(1 << 2); + + /* + * Update bit 4 "Keyboard enable flag" of the status register + * with bit 3 "Ignore keyboard lock" of the controller + * configuration byte (if set), then reset the latter one. + * See: http://www.win.tue.nl/~aeb/linux/kbd/scancodes-11.html#kccb3 + * for more details. + */ + if (ControllerConfig & (1 << 3)) + { + StatusRegister |= (1 << 4); + ControllerConfig &= ~(1 << 3); + } + break; } @@ -399,13 +431,13 @@ BOOLEAN PS2Initialize(VOID) { /* Initialize the PS/2 ports */ - Ports[0].IsEnabled = TRUE; + Ports[0].IsEnabled = FALSE; Ports[0].QueueEmpty = TRUE; Ports[0].QueueStart = 0; Ports[0].QueueEnd = 0; Ports[0].QueueMutex = CreateMutex(NULL, FALSE, NULL); - Ports[1].IsEnabled = TRUE; + Ports[1].IsEnabled = FALSE; Ports[1].QueueEmpty = TRUE; Ports[1].QueueStart = 0; Ports[1].QueueEnd = 0;

1 0

[tfaber] 68612: [NTOS:MM] - Arch, don't remove all the page protections CORE-9808
by tfaber＠svn.reactos.org 07 Aug '15

07 Aug '15

Author: tfaber Date: Fri Aug 7 16:09:02 2015 New Revision: 68612 URL: http://svn.reactos.org/svn/reactos?rev=68612&view=rev Log: [NTOS:MM] - Arch, don't remove all the page protections CORE-9808 Modified: trunk/reactos/ntoskrnl/mm/section.c Modified: trunk/reactos/ntoskrnl/mm/section.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/section.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/mm/section.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/section.c [iso-8859-1] Fri Aug 7 16:09:02 2015 @@ -4485,7 +4485,7 @@ } /* FIXME: We should keep this, but it would break code checking equality */ - Protect &= PAGE_NOCACHE; + Protect &= ~PAGE_NOCACHE; Section = (PROS_SECTION_OBJECT)SectionObject; AddressSpace = &Process->Vm;

1 0

[tfaber] 68611: [NTOS:MM] - Allow (and then ignore) PAGE_NOCACHE in the protection flags for MmMapViewOfSection CORE-9808
by tfaber＠svn.reactos.org 07 Aug '15

07 Aug '15

Author: tfaber Date: Fri Aug 7 15:52:04 2015 New Revision: 68611 URL: http://svn.reactos.org/svn/reactos?rev=68611&view=rev Log: [NTOS:MM] - Allow (and then ignore) PAGE_NOCACHE in the protection flags for MmMapViewOfSection CORE-9808 Modified: trunk/reactos/ntoskrnl/include/internal/mm.h trunk/reactos/ntoskrnl/mm/section.c Modified: trunk/reactos/ntoskrnl/include/internal/mm.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/include/internal/… ============================================================================== --- trunk/reactos/ntoskrnl/include/internal/mm.h [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/include/internal/mm.h [iso-8859-1] Fri Aug 7 15:52:04 2015 @@ -115,7 +115,8 @@ PAGE_EXECUTE_READ | \ PAGE_EXECUTE_READWRITE | \ PAGE_EXECUTE_WRITECOPY | \ - PAGE_NOACCESS) + PAGE_NOACCESS | \ + PAGE_NOCACHE) #define PAGE_IS_READABLE \ (PAGE_READONLY | \ Modified: trunk/reactos/ntoskrnl/mm/section.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/section.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/mm/section.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/section.c [iso-8859-1] Fri Aug 7 15:52:04 2015 @@ -4484,6 +4484,8 @@ return STATUS_INVALID_PAGE_PROTECTION; } + /* FIXME: We should keep this, but it would break code checking equality */ + Protect &= PAGE_NOCACHE; Section = (PROS_SECTION_OBJECT)SectionObject; AddressSpace = &Process->Vm;

1 0

[hbelusca] 68610: [NTVDM]: Fix few file headers.
by hbelusca＠svn.reactos.org 07 Aug '15

07 Aug '15

Author: hbelusca Date: Fri Aug 7 15:50:09 2015 New Revision: 68610 URL: http://svn.reactos.org/svn/reactos?rev=68610&view=rev Log: [NTVDM]: Fix few file headers. Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/kbdbios32.c trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/kbdbios32.h trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/moubios32.h trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vidbios32.c trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vidbios32.h trunk/reactos/subsystems/mvdm/ntvdm/bios/kbdbios.c trunk/reactos/subsystems/mvdm/ntvdm/bios/kbdbios.h trunk/reactos/subsystems/mvdm/ntvdm/bios/vidbios.c trunk/reactos/subsystems/mvdm/ntvdm/bios/vidbios.h Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/kbdbios32.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/kbdbios32.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/kbdbios32.c [iso-8859-1] Fri Aug 7 15:50:09 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: kbdbios32.c - * PURPOSE: VDM Keyboard 32-bit BIOS + * PURPOSE: VDM 32-bit PS/2 Keyboard BIOS * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> */ @@ -209,10 +209,8 @@ * In return, if CF is set we continue processing the scan code * stored in AL, and if not, we skip it. */ - BYTE CF; - WORD AX; - CF = getCF(); - AX = getAX(); + BYTE CF = getCF(); + WORD AX = getAX(); setCF(1); setAL(IOReadB(PS2_DATA_PORT)); Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/kbdbios32.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/kbdbios32.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/kbdbios32.h [iso-8859-1] Fri Aug 7 15:50:09 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: kbdbios32.h - * PURPOSE: VDM Keyboard 32-bit BIOS + * PURPOSE: VDM 32-bit PS/2 Keyboard BIOS * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> */ Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/moubios32.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/moubios32.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/moubios32.h [iso-8859-1] Fri Aug 7 15:50:09 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: moubios32.h - * PURPOSE: VDM Mouse 32-bit BIOS + * PURPOSE: VDM 32-bit PS/2 Mouse BIOS * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> */ Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vidbios32.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vidbios32.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vidbios32.c [iso-8859-1] Fri Aug 7 15:50:09 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: vidbios32.c - * PURPOSE: VDM Video 32-bit BIOS + * PURPOSE: VDM 32-bit Video BIOS * PROGRAMMERS: Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) * * NOTE: All of the real code is in bios/vidbios.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vidbios32.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vidbios32.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/bios32/vidbios32.h [iso-8859-1] Fri Aug 7 15:50:09 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: vidbios32.h - * PURPOSE: VDM Video 32-bit BIOS + * PURPOSE: VDM 32-bit Video BIOS * PROGRAMMERS: Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) * * NOTE: All of the real code is in bios/vidbios.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/kbdbios.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/kbdbios.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/kbdbios.c [iso-8859-1] Fri Aug 7 15:50:09 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: kbdbios.c - * PURPOSE: VDM Keyboard BIOS Support Library + * PURPOSE: VDM 32-bit PS/2 Keyboard BIOS Support Library * PROGRAMMERS: Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) */ Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/kbdbios.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/kbdbios.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/kbdbios.h [iso-8859-1] Fri Aug 7 15:50:09 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: kbdbios.h - * PURPOSE: VDM Keyboard BIOS Support Library + * PURPOSE: VDM 32-bit PS/2 Keyboard BIOS Support Library * PROGRAMMERS: Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) */ Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/vidbios.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/vidbios.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/vidbios.c [iso-8859-1] Fri Aug 7 15:50:09 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: vidbios.c - * PURPOSE: VDM Video BIOS Support Library + * PURPOSE: VDM 32-bit Video BIOS Support Library * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) */ Modified: trunk/reactos/subsystems/mvdm/ntvdm/bios/vidbios.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/bios… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/bios/vidbios.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/bios/vidbios.h [iso-8859-1] Fri Aug 7 15:50:09 2015 @@ -2,7 +2,7 @@ * COPYRIGHT: GPL - See COPYING in the top level directory * PROJECT: ReactOS Virtual DOS Machine * FILE: vidbios.h - * PURPOSE: VDM Video BIOS Support Library + * PURPOSE: VDM 32-bit Video BIOS Support Library * PROGRAMMERS: Aleksandar Andrejevic <theflash AT sdf DOT lonestar DOT org> * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) */

1 0

[hbelusca] 68609: [NTVDM]: DOS: When we create a new PSP, or terminate a TSR, we *must* use the code segment value of the CALLER!! and not the current one (DOS code segment). Should fix a lot of st...
by hbelusca＠svn.reactos.org 07 Aug '15

07 Aug '15

Author: hbelusca Date: Fri Aug 7 15:14:19 2015 New Revision: 68609 URL: http://svn.reactos.org/svn/reactos?rev=68609&view=rev Log: [NTVDM]: DOS: When we create a new PSP, or terminate a TSR, we *must* use the code segment value of the CALLER!! and not the current one (DOS code segment). Should fix a lot of stuff... Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/dos.c 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] Fri Aug 7 15:14:19 2015 @@ -563,7 +563,7 @@ /* Create New PSP */ case 0x26: { - DosClonePsp(getDX(), getCS()); + DosClonePsp(getDX(), Stack[STACK_CS]); break; } @@ -2096,7 +2096,7 @@ VOID WINAPI DosInt27h(LPWORD Stack) { - DosTerminateProcess(getCS(), 0, (getDX() + 0x0F) >> 4); + DosTerminateProcess(Stack[STACK_CS], 0, (getDX() + 0x0F) >> 4); } VOID WINAPI DosIdle(LPWORD Stack)

1 0

[jimtabor] 68608: [NtGdi] - Add more function for NtUser.
by jimtabor＠svn.reactos.org 07 Aug '15

07 Aug '15

Author: jimtabor Date: Fri Aug 7 05:56:01 2015 New Revision: 68608 URL: http://svn.reactos.org/svn/reactos?rev=68608&view=rev Log: [NtGdi] - Add more function for NtUser. Modified: trunk/reactos/win32ss/gdi/ntgdi/coord.c trunk/reactos/win32ss/gdi/ntgdi/coord.h trunk/reactos/win32ss/gdi/ntgdi/dc.h trunk/reactos/win32ss/gdi/ntgdi/dcutil.c trunk/reactos/win32ss/gdi/ntgdi/text.c trunk/reactos/win32ss/gdi/ntgdi/text.h Modified: trunk/reactos/win32ss/gdi/ntgdi/coord.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/coord.c?… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/coord.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/coord.c [iso-8859-1] Fri Aug 7 05:56:01 2015 @@ -894,6 +894,38 @@ return iPrevMapMode; } +BOOL +FASTCALL +GreSetViewportOrgEx( + HDC hDC, + int X, + int Y, + LPPOINT Point) +{ + PDC dc; + PDC_ATTR pdcattr; + + dc = DC_LockDc(hDC); + if (!dc) + { + EngSetLastError(ERROR_INVALID_HANDLE); + return FALSE; + } + pdcattr = dc->pdcattr; + + if (Point) + { + Point->x = pdcattr->ptlViewportOrg.x; + Point->y = pdcattr->ptlViewportOrg.y; + } + + pdcattr->ptlViewportOrg.x = X; + pdcattr->ptlViewportOrg.y = Y; + pdcattr->flXform |= PAGE_XLATE_CHANGED; + + DC_UnlockDc(dc); + return TRUE; +} BOOL APIENTRY @@ -1239,6 +1271,89 @@ } BOOL APIENTRY +GreGetDCPoint( + HDC hDC, + UINT iPoint, + PPOINTL Point) +{ + BOOL Ret = TRUE; + DC *pdc; + SIZE Size; + PSIZEL pszlViewportExt; + + if (!Point) + { + EngSetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } + + pdc = DC_LockDc(hDC); + if (!pdc) + { + EngSetLastError(ERROR_INVALID_HANDLE); + return FALSE; + } + + switch (iPoint) + { + case GdiGetViewPortExt: + pszlViewportExt = DC_pszlViewportExt(pdc); + Point->x = pszlViewportExt->cx; + Point->y = pszlViewportExt->cy; + break; + + case GdiGetWindowExt: + Point->x = pdc->pdcattr->szlWindowExt.cx; + Point->y = pdc->pdcattr->szlWindowExt.cy; + break; + + case GdiGetViewPortOrg: + *Point = pdc->pdcattr->ptlViewportOrg; + break; + + case GdiGetWindowOrg: + *Point = pdc->pdcattr->ptlWindowOrg; + break; + + case GdiGetDCOrg: + *Point = pdc->ptlDCOrig; + break; + + case GdiGetAspectRatioFilter: + DC_vGetAspectRatioFilter(pdc, &Size); + Point->x = Size.cx; + Point->y = Size.cy; + break; + + default: + EngSetLastError(ERROR_INVALID_PARAMETER); + Ret = FALSE; + break; + } + + DC_UnlockDc(pdc); + return Ret; +} + +BOOL +WINAPI +GreGetWindowExtEx( + _In_ HDC hdc, + _Out_ LPSIZE lpSize) +{ + return GreGetDCPoint(hdc, GdiGetWindowExt, (PPOINTL)lpSize); +} + +BOOL +WINAPI +GreGetViewportExtEx( + _In_ HDC hdc, + _Out_ LPSIZE lpSize) +{ + return GreGetDCPoint(hdc, GdiGetViewPortExt, (PPOINTL)lpSize); +} + +BOOL APIENTRY NtGdiGetDCPoint( HDC hDC, UINT iPoint, Modified: trunk/reactos/win32ss/gdi/ntgdi/coord.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/coord.h?… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/coord.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/coord.h [iso-8859-1] Fri Aug 7 05:56:01 2015 @@ -156,3 +156,7 @@ int APIENTRY IntGdiSetMapMode(PDC, int); BOOL FASTCALL GreLPtoDP(HDC, LPPOINT, INT); BOOL FASTCALL GreDPtoLP(HDC, LPPOINT, INT); +BOOL APIENTRY GreGetDCPoint(HDC,UINT,PPOINTL); +BOOL WINAPI GreGetWindowExtEx( _In_ HDC hdc, _Out_ LPSIZE lpSize); +BOOL WINAPI GreGetViewportExtEx( _In_ HDC hdc, _Out_ LPSIZE lpSize); +BOOL FASTCALL GreSetViewportOrgEx(HDC,int,int,LPPOINT); Modified: trunk/reactos/win32ss/gdi/ntgdi/dc.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dc.h?rev… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/dc.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/dc.h [iso-8859-1] Fri Aug 7 05:56:01 2015 @@ -181,7 +181,7 @@ COLORREF FASTCALL GreGetTextColor(HDC); COLORREF FASTCALL IntSetDCBrushColor(HDC,COLORREF); COLORREF FASTCALL IntSetDCPenColor(HDC,COLORREF); - +int FASTCALL GreGetGraphicsMode(HDC); INIT_FUNCTION NTSTATUS NTAPI InitDcImpl(VOID); PPDEVOBJ FASTCALL IntEnumHDev(VOID); Modified: trunk/reactos/win32ss/gdi/ntgdi/dcutil.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dcutil.c… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/dcutil.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/dcutil.c [iso-8859-1] Fri Aug 7 05:56:01 2015 @@ -239,6 +239,21 @@ DC_UnlockDc(pdc); } return oSMode; +} + +int FASTCALL +GreGetGraphicsMode(HDC hdc) +{ + PDC dc; + int GraphicsMode; + if (!(dc = DC_LockDc(hdc))) + { + EngSetLastError(ERROR_INVALID_HANDLE); + return CLR_INVALID; + } + GraphicsMode = dc->pdcattr->iGraphicsMode;; + DC_UnlockDc(dc); + return GraphicsMode; } VOID Modified: trunk/reactos/win32ss/gdi/ntgdi/text.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/text.c?r… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/text.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/text.c [iso-8859-1] Fri Aug 7 05:56:01 2015 @@ -146,6 +146,18 @@ return Result; } +BOOL +WINAPI +GreGetTextMetricsW( + _In_ HDC hdc, + _Out_ LPTEXTMETRICW lptm) +{ + TMW_INTERNAL tmwi; + if (!ftGdiGetTextMetricsW(hdc, &tmwi)) return FALSE; + *lptm = tmwi.TextMetric; + return TRUE; +} + DWORD APIENTRY NtGdiGetCharSet(HDC hDC) Modified: trunk/reactos/win32ss/gdi/ntgdi/text.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/text.h?r… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/text.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/text.h [iso-8859-1] Fri Aug 7 05:56:01 2015 @@ -132,6 +132,7 @@ BOOL FASTCALL GreGetTextExtentExW(HDC,LPWSTR,ULONG,ULONG,PULONG,PULONG,LPSIZE,FLONG); BOOL FASTCALL GreTextOutW(HDC,int,int,LPCWSTR,int); HFONT FASTCALL GreCreateFontIndirectW( LOGFONTW * ); +BOOL WINAPI GreGetTextMetricsW( _In_ HDC hdc, _Out_ LPTEXTMETRICW lptm); #define IntLockProcessPrivateFonts(W32Process) \ ExEnterCriticalRegionAndAcquireFastMutexUnsafe(&W32Process->PrivateFontListLock)

1 0

[aandrejevic] 68607: [FASTFAT] Irp->UserBuffer being NULL doesn't indicate any error. It could be that the caller really wants the result stored at address NULL (which can be valid, and is valid by...
by aandrejevic＠svn.reactos.org 07 Aug '15

07 Aug '15

Author: aandrejevic Date: Fri Aug 7 03:30:05 2015 New Revision: 68607 URL: http://svn.reactos.org/svn/reactos?rev=68607&view=rev Log: [FASTFAT] Irp->UserBuffer being NULL doesn't indicate any error. It could be that the caller really wants the result stored at address NULL (which can be valid, and is valid by default for programs like NTVDM). Modified: trunk/reactos/drivers/filesystems/fastfat/rw.c Modified: trunk/reactos/drivers/filesystems/fastfat/rw.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/rw.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/rw.c [iso-8859-1] Fri Aug 7 03:30:05 2015 @@ -656,7 +656,7 @@ } Buffer = VfatGetUserBuffer(IrpContext->Irp, BooleanFlagOn(IrpContext->Irp->Flags, IRP_PAGING_IO)); - if (!Buffer) + if (!Buffer && IrpContext->Irp->MdlAddress) { Status = STATUS_INVALID_USER_BUFFER; goto ByeBye; @@ -927,7 +927,7 @@ OldFileSize = Fcb->RFCB.FileSize; Buffer = VfatGetUserBuffer(IrpContext->Irp, BooleanFlagOn(IrpContext->Irp->Flags, IRP_PAGING_IO)); - if (!Buffer) + if (!Buffer && IrpContext->Irp->MdlAddress) { Status = STATUS_INVALID_USER_BUFFER; goto ByeBye;

1 0

[hbelusca] 68606: [NTVDM] - Copy-paste error fix in the initialization of the collating table. - Fix DosCombineFreeBlocks so that it doesn't say "DOS memory arena corrupted" because while combining...
by hbelusca＠svn.reactos.org 06 Aug '15

06 Aug '15

Author: hbelusca Date: Thu Aug 6 23:00:17 2015 New Revision: 68606 URL: http://svn.reactos.org/svn/reactos?rev=68606&view=rev Log: [NTVDM] - Copy-paste error fix in the initialization of the collating table. - Fix DosCombineFreeBlocks so that it doesn't say "DOS memory arena corrupted" because while combining blocks, the current block became the last one and we tried to merge it with a next block which, by definition, cannot exist (see comment in the code). - Remove a spurious call to DosCombineFreeBlocks in DosResizeMemory which was forgotten in r68596. - Check the validity of the encountered MCBs during the loop for linking or unlinking the UMB chain to the main arena chain. - Fix the size of the created UMB blocks. Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/country.c trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/memory.c Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/country.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… ============================================================================== --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/country.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/country.c [iso-8859-1] Thu Aug 6 23:00:17 2015 @@ -279,8 +279,8 @@ CountryData->FNameTermTbl.Data[21] = ','; CountryData->CollateTbl.Size = ARRAYSIZE(CountryData->CollateTbl.Data); - for (i = 0; i < CountryData->LoCaseTbl.Size; ++i) - CountryData->LoCaseTbl.Data[i] = i; + for (i = 0; i < CountryData->CollateTbl.Size; ++i) + CountryData->CollateTbl.Data[i] = i; CountryData->DBCSLeadTbl.Size = 0; // Empty DBCS table CountryData->DBCSLeadTbl.Data[0] = 0x0000; 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] Thu Aug 6 23:00:17 2015 @@ -63,7 +63,7 @@ /* Make sure it's valid */ if (!ValidateMcb(CurrentMcb)) { - DPRINT1("The DOS memory arena is corrupted! (CurrentMcb = %04x; PreviousMcb = %04X)\n", Segment, PrevSegment); + DPRINT1("The DOS memory arena is corrupted! (CurrentMcb = 0x%04X; PreviousMcb = 0x%04X)\n", Segment, PrevSegment); return; } @@ -82,12 +82,25 @@ static VOID DosCombineFreeBlocks(WORD StartBlock) { + /* NOTE: This function is always called with valid MCB blocks */ + PDOS_MCB CurrentMcb = SEGMENT_TO_MCB(StartBlock), NextMcb; - /* If this is the last block or it's not free, quit */ - if (CurrentMcb->BlockType == 'Z' || CurrentMcb->OwnerPsp != 0) return; - - while (TRUE) + /* If the block is not free, quit */ + if (CurrentMcb->OwnerPsp != 0) return; + + /* + * Loop while the current block is not the last one. It can happen + * that the block is not the last one at the beginning, but becomes + * the last one at the end of the process. This happens in the case + * where its next following blocks are free but not combined yet, + * and they are terminated by a free last block. During the process + * all the blocks are combined together and we end up in the situation + * where the current (free) block is followed by the last (free) block. + * At the last step of the algorithm the current block becomes the + * last one. + */ + while (CurrentMcb->BlockType != 'Z') { /* Get a pointer to the next MCB */ NextMcb = SEGMENT_TO_MCB(StartBlock + CurrentMcb->Size + 1); @@ -210,7 +223,6 @@ } Done: - DosMemValidate(); /* If we didn't find a free block, bail out */ @@ -278,12 +290,11 @@ PDOS_MCB Mcb = SEGMENT_TO_MCB(Segment), NextMcb; DPRINT("DosResizeMemory: BlockData 0x%04X, NewSize 0x%04X\n", - BlockData, - NewSize); - - DosMemValidate(); - - /* Make sure this is a valid, allocated block */ + BlockData, NewSize); + + DosMemValidate(); + + /* Make sure this is a valid and allocated block */ if (BlockData == 0 || !ValidateMcb(Mcb) || Mcb->OwnerPsp == 0) { Success = FALSE; @@ -291,15 +302,13 @@ goto Done; } - DosCombineFreeBlocks(SysVars->FirstMcb); - ReturnSize = Mcb->Size; /* Check if we need to expand or contract the block */ if (NewSize > Mcb->Size) { /* We can't expand the last block */ - if (Mcb->BlockType != 'M') + if (Mcb->BlockType == 'Z') { Success = FALSE; goto Done; @@ -431,21 +440,32 @@ DPRINT("Linking UMB\n"); - /* Check if UMBs are already linked */ + /* Check if UMBs are initialized and already linked */ 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)) - { + /* Find the last block before the start of the UMB chain */ + while (Segment < SysVars->UmbChainStart) + { + /* Get a pointer to the MCB */ + Mcb = SEGMENT_TO_MCB(Segment); + + /* Make sure it's valid */ + if (!ValidateMcb(Mcb)) + { + DPRINT1("The DOS memory arena is corrupted!\n"); + Sda->LastErrorCode = ERROR_ARENA_TRASHED; + return FALSE; + } + + /* If this was the last MCB in the chain, quit */ + if (Mcb->BlockType == 'Z') break; + + /* Otherwise, update the segment and continue */ Segment += Mcb->Size + 1; - Mcb = SEGMENT_TO_MCB(Segment); - } - - DosMemValidate(); + } /* Make sure it's valid */ if (Mcb->BlockType != 'Z') return FALSE; @@ -453,6 +473,8 @@ /* Connect the MCB with the UMB chain */ Mcb->BlockType = 'M'; + DosMemValidate(); + SysVars->UmbLinked = TRUE; return TRUE; } @@ -464,25 +486,28 @@ DPRINT("Unlinking UMB\n"); - /* Check if UMBs are already unlinked */ + /* Check if UMBs are initialized and already unlinked */ 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) - { - // FIXME: Use SysVars->UmbChainStart - if ((Segment + Mcb->Size) == (SysVars->FirstMcb + USER_MEMORY_SIZE)) - { - /* This is the last non-UMB segment */ - break; + /* Find the last block before the start of the UMB chain */ + while (Segment < SysVars->UmbChainStart) + { + /* Get a pointer to the MCB */ + Mcb = SEGMENT_TO_MCB(Segment); + + /* Make sure it's valid */ + if (!ValidateMcb(Mcb)) + { + DPRINT1("The DOS memory arena is corrupted!\n"); + Sda->LastErrorCode = ERROR_ARENA_TRASHED; + return FALSE; } /* Advance to the next MCB */ Segment += Mcb->Size + 1; - Mcb = SEGMENT_TO_MCB(Segment); } /* Mark the MCB as the last MCB */ @@ -509,7 +534,7 @@ WORD DosGetPreviousUmb(WORD UmbSegment) { PDOS_MCB CurrentMcb; - WORD Segment, PrevSegment = 0; + WORD Segment, PrevSegment = 0; // FIXME: or use UmbChainStart ?? if (SysVars->UmbChainStart == 0xFFFF) return 0; @@ -529,14 +554,13 @@ return 0; } - if (Segment >= UmbSegment) - break; + /* We went over the UMB segment, quit */ + if (Segment >= UmbSegment) break; PrevSegment = Segment; /* If this was the last MCB in the chain, quit */ - if (CurrentMcb->BlockType == 'Z') - break; + if (CurrentMcb->BlockType == 'Z') break; /* Otherwise, update the segment and continue */ Segment += CurrentMcb->Size + 1; @@ -583,7 +607,7 @@ // 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->Size = (SysVars->UmbChainStart /* UmbSegment */ - SysVars->FirstMcb - USER_MEMORY_SIZE - 2) + 1; Mcb->OwnerPsp = SYSTEM_PSP; RtlCopyMemory(Mcb->Name, "SC ", sizeof("SC ")-1); @@ -609,7 +633,7 @@ 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->Size = PrevMcb->Size - (UmbSegment + Size - PrevSegment) + 1; Mcb->OwnerPsp = PrevMcb->OwnerPsp; RtlCopyMemory(Mcb->Name, PrevMcb->Name, sizeof(PrevMcb->Name));

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs August 2015