[aandrejevic] 59454: [NTVDM] Add video pages support to INT 10h functions and memory access. [SOFTX86] Fix the opcode check for XLAT.
10 Jul
2013
10 Jul
'13
7:41 p.m.
Author: aandrejevic
Date: Wed Jul 10 19:41:43 2013
New Revision: 59454
URL: http://svn.reactos.org/svn/reactos?rev=59454&view=rev
Log:
[NTVDM]
Add video pages support to INT 10h functions and memory access.
[SOFTX86]
Fix the opcode check for XLAT.
Modified:
branches/ntvdm/lib/3rdparty/softx86/softx86/mov.c
branches/ntvdm/subsystems/ntvdm/bios.c
Modified: branches/ntvdm/lib/3rdparty/softx86/softx86/mov.c
URL:
http://svn.reactos.org/svn/reactos/branches/ntvdm/lib/3rdparty/softx86/soft…
==============================================================================
--- branches/ntvdm/lib/3rdparty/softx86/softx86/mov.c [iso-8859-1] (original)
+++ branches/ntvdm/lib/3rdparty/softx86/softx86/mov.c [iso-8859-1] Wed Jul 10 19:41:43
2013
@@ -219,7 +219,7 @@
else
seg = ctx->state->segment_override;
- if ((opcode&0xFC) == 0xD7) { // MOV reg,reg/mem or reg/mem,reg
+ if (opcode == 0xD7) { // MOV reg,reg/mem or reg/mem,reg
sx86_ubyte d;
sx86_udword ofs;
@@ -239,7 +239,7 @@
int Sfx86OpcodeDec_xlat(sx86_ubyte opcode,softx86_ctx* ctx,char buf[128])
{
- if ((opcode&0xFC) == 0xD7) { // XLAT
+ if (opcode == 0xD7) { // XLAT
strcpy(buf,"XLAT");
return 1;
}
Modified: branches/ntvdm/subsystems/ntvdm/bios.c
URL:
http://svn.reactos.org/svn/reactos/branches/ntvdm/subsystems/ntvdm/bios.c?r…
==============================================================================
--- branches/ntvdm/subsystems/ntvdm/bios.c [iso-8859-1] (original)
+++ branches/ntvdm/subsystems/ntvdm/bios.c [iso-8859-1] Wed Jul 10 19:41:43 2013
@@ -72,11 +72,31 @@
return i;
}
+static DWORD BiosGetVideoPageSize()
+{
+ INT i;
+ DWORD BufferSize = VideoModes[CurrentVideoMode].Width
+ * VideoModes[CurrentVideoMode].Height
+ * BiosColorNumberToBits(VideoModes[CurrentVideoMode].Colors)
+ / 8;
+
+ for (i = 0; i < 32; i++) if ((1 << i) >= BufferSize) break;
+
+ return 1 << i;
+}
+
+static BYTE BiosVideoAddressToPage(ULONG Address)
+{
+ return (Address - (VideoModes[CurrentVideoMode].Segment << 4))
+ / BiosGetVideoPageSize();
+}
+
static COORD BiosVideoAddressToCoord(ULONG Address)
{
COORD Result = {0, 0};
INT BitsPerPixel;
- DWORD Offset = Address - (VideoModes[CurrentVideoMode].Segment << 4);
+ BYTE PageStart = BiosVideoAddressToPage(Address) * BiosGetVideoPageSize();
+ DWORD Offset = Address - (VideoModes[CurrentVideoMode].Segment << 4) -
PageStart;
if (VideoModes[CurrentVideoMode].Text)
{
@@ -403,6 +423,7 @@
{
ULONG i;
COORD Coordinates;
+ BYTE Page;
COORD Origin = { 0, 0 };
COORD UnitSize = { 1, 1 };
CHAR_INFO Character;
@@ -418,6 +439,12 @@
{
/* Get the coordinates */
Coordinates = BiosVideoAddressToCoord(i);
+
+ /* Get the page number */
+ Page = BiosVideoAddressToPage(i);
+
+ /* Make sure the page is valid */
+ if (Page >= VideoModes[CurrentVideoMode].Pages) continue;
/* Fill the rectangle structure */
Rect.Left = Coordinates.X;
@@ -430,7 +457,7 @@
Character.Attributes = *((PBYTE)((ULONG_PTR)BaseAddress + i + 1));
/* Write the character */
- WriteConsoleOutputA(BiosConsoleOutput,
+ WriteConsoleOutputA(ConsoleBuffers[Page],
&Character,
UnitSize,
Origin,
@@ -480,6 +507,7 @@
{
ULONG i;
COORD Coordinates;
+ BYTE Page;
WORD Attribute;
DWORD CharsWritten;
@@ -490,12 +518,18 @@
{
/* Get the coordinates */
Coordinates = BiosVideoAddressToCoord(i);
+
+ /* Get the page number */
+ Page = BiosVideoAddressToPage(i);
+
+ /* Make sure the page is valid */
+ if (Page >= VideoModes[CurrentVideoMode].Pages) continue;
/* Check if this is a character byte or an attribute byte */
if ((i - (VideoModes[CurrentVideoMode].Segment << 4)) % 2 == 0)
{
/* This is a regular character */
- ReadConsoleOutputCharacterA(BiosConsoleOutput,
+ ReadConsoleOutputCharacterA(ConsoleBuffers[Page],
(LPSTR)((ULONG_PTR)BaseAddress + i),
sizeof(CHAR),
Coordinates,
@@ -504,7 +538,7 @@
else
{
/* This is an attribute */
- ReadConsoleOutputAttribute(BiosConsoleOutput,
+ ReadConsoleOutputAttribute(ConsoleBuffers[Page],
&Attribute,
sizeof(CHAR),
Coordinates,
@@ -614,7 +648,7 @@
/* Set the cursor */
CursorInfo.dwSize = (CursorHeight * 100) / CONSOLE_FONT_HEIGHT;
CursorInfo.bVisible = !Invisible;
- SetConsoleCursorInfo(BiosConsoleOutput, &CursorInfo);
+ SetConsoleCursorInfo(ConsoleBuffers[CurrentVideoPage], &CursorInfo);
break;
}
@@ -622,10 +656,13 @@
/* Set Cursor Position */
case 0x02:
{
+ /* Make sure the selected video page exists */
+ if (HIBYTE(Ebx) >= VideoModes[CurrentVideoMode].Pages) break;
+
Position.X = LOBYTE(Edx);
Position.Y = HIBYTE(Edx);
- SetConsoleCursorPosition(BiosConsoleOutput, Position);
+ SetConsoleCursorPosition(ConsoleBuffers[HIBYTE(Ebx)], Position);
break;
}
@@ -634,9 +671,13 @@
{
INT StartLine;
+ /* Make sure the selected video page exists */
+ if (HIBYTE(Ebx) >= VideoModes[CurrentVideoMode].Pages) break;
+
/* Retrieve the data */
- GetConsoleCursorInfo(BiosConsoleOutput, &CursorInfo);
- GetConsoleScreenBufferInfo(BiosConsoleOutput, &ScreenBufferInfo);
+ GetConsoleCursorInfo(ConsoleBuffers[HIBYTE(Ebx)], &CursorInfo);
+ GetConsoleScreenBufferInfo(ConsoleBuffers[HIBYTE(Ebx)],
+ &ScreenBufferInfo);
/* Find the first line */
StartLine = 32 - ((CursorInfo.dwSize * 32) / 100);
@@ -647,6 +688,28 @@
EmulatorSetRegister(EMULATOR_REG_DX,
LOWORD(ScreenBufferInfo.dwCursorPosition.Y) << 8
|| LOWORD(ScreenBufferInfo.dwCursorPosition.X));
+ break;
+ }
+
+ /* Select Active Display Page */
+ case 0x05:
+ {
+ /* Check if the page exists */
+ if (LOBYTE(Eax) >= VideoModes[CurrentVideoMode].Pages) break;
+
+ /* Check if this is the same page */
+ if (LOBYTE(Eax) == CurrentVideoPage) break;
+
+ /* Change the video page */
+ CurrentVideoPage = LOBYTE(Eax);
+
+ /* Set the active page console buffer */
+ SetConsoleActiveScreenBuffer(ConsoleBuffers[CurrentVideoPage]);
+
+ /* Restore the cursor to (0, 0) */
+ Position.X = Position.Y = 0;
+ SetConsoleCursorPosition(BiosConsoleOutput, Position);
+
break;
}
@@ -664,7 +727,7 @@
if (HIBYTE(Eax) == 0x06) Position.Y = Rect.Top - LOBYTE(Eax);
else Position.Y = Rect.Top + LOBYTE(Eax);
- ScrollConsoleScreenBuffer(BiosConsoleOutput,
+ ScrollConsoleScreenBuffer(ConsoleBuffers[CurrentVideoPage],
&Rect,
&Rect,
Position,
@@ -677,15 +740,23 @@
{
COORD BufferSize = { 1, 1 }, Origin = { 0, 0 };
+ /* Make sure the selected video page exists */
+ if (HIBYTE(Ebx) >= VideoModes[CurrentVideoMode].Pages) break;
+
/* Get the cursor position */
- GetConsoleScreenBufferInfo(BiosConsoleOutput, &ScreenBufferInfo);
+ GetConsoleScreenBufferInfo(ConsoleBuffers[HIBYTE(Ebx)],
+ &ScreenBufferInfo);
/* Read at cursor position */
Rect.Left = ScreenBufferInfo.dwCursorPosition.X;
Rect.Top = ScreenBufferInfo.dwCursorPosition.Y;
/* Read the console output */
- ReadConsoleOutput(BiosConsoleOutput, &Character, BufferSize, Origin,
&Rect);
+ ReadConsoleOutput(ConsoleBuffers[HIBYTE(Ebx)],
+ &Character,
+ BufferSize,
+ Origin,
+ &Rect);
/* Return the result */
EmulatorSetRegister(EMULATOR_REG_AX,
@@ -700,18 +771,22 @@
{
DWORD CharsWritten;
+ /* Make sure the selected video page exists */
+ if (HIBYTE(Ebx) >= VideoModes[CurrentVideoMode].Pages) break;
+
/* Get the cursor position */
- GetConsoleScreenBufferInfo(BiosConsoleOutput, &ScreenBufferInfo);
+ GetConsoleScreenBufferInfo(ConsoleBuffers[HIBYTE(Ebx)],
+ &ScreenBufferInfo);
/* Write the attribute to the output */
- FillConsoleOutputAttribute(BiosConsoleOutput,
+ FillConsoleOutputAttribute(ConsoleBuffers[HIBYTE(Ebx)],
LOBYTE(Ebx),
LOWORD(Ecx),
ScreenBufferInfo.dwCursorPosition,
&CharsWritten);
/* Write the character to the output */
- FillConsoleOutputCharacterA(BiosConsoleOutput,
+ FillConsoleOutputCharacterA(ConsoleBuffers[HIBYTE(Ebx)],
LOBYTE(Eax),
LOWORD(Ecx),
ScreenBufferInfo.dwCursorPosition,
@@ -725,11 +800,15 @@
{
DWORD CharsWritten;
+ /* Make sure the selected video page exists */
+ if (HIBYTE(Ebx) >= VideoModes[CurrentVideoMode].Pages) break;
+
/* Get the cursor position */
- GetConsoleScreenBufferInfo(BiosConsoleOutput, &ScreenBufferInfo);
+ GetConsoleScreenBufferInfo(ConsoleBuffers[HIBYTE(Ebx)],
+ &ScreenBufferInfo);
/* Write the character to the output */
- FillConsoleOutputCharacterA(BiosConsoleOutput,
+ FillConsoleOutputCharacterA(ConsoleBuffers[HIBYTE(Ebx)],
LOBYTE(Eax),
LOWORD(Ecx),
ScreenBufferInfo.dwCursorPosition,
4140
days inactive
4140
days old
0 comments
1 participants
participants (1)
-
aandrejevic＠svn.reactos.org