[greatlrd] 26159: rewrite old NtGdiDdBlt I wrote so it match with the new caching of Dx pointers. Fixing varius problem in NtGdiDdCreateSurface. But some who the color are not being draw right with DdBlt work in Progress to figout why. But it works now partly yes dx start working. at kmode or syscall for real apps. or if they are using gdi32.dll or d3d8thk.dll
23 Mar
2007
23 Mar
'07
5:07 p.m.
Author: greatlrd
Date: Fri Mar 23 20:07:49 2007
New Revision: 26159
URL: http://svn.reactos.org/svn/reactos?rev=26159&view=rev
Log:
rewrite old NtGdiDdBlt I wrote so it match with the new caching of Dx pointers. Fixing
varius problem in NtGdiDdCreateSurface. But some who the color are not being draw right
with DdBlt work in Progress to figout why. But it works now partly yes dx start working.
at kmode or syscall for real apps. or if they are using gdi32.dll or d3d8thk.dll
Modified:
trunk/reactos/subsystems/win32/win32k/ntddraw/dd.c
trunk/reactos/subsystems/win32/win32k/ntddraw/ddraw.c
trunk/reactos/subsystems/win32/win32k/ntddraw/ddsurf.c
Modified: trunk/reactos/subsystems/win32/win32k/ntddraw/dd.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt…
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/ntddraw/dd.c (original)
+++ trunk/reactos/subsystems/win32/win32k/ntddraw/dd.c Fri Mar 23 20:07:49 2007
@@ -44,49 +44,98 @@
DWORD ddRVal = DDHAL_DRIVER_NOTHANDLED;
NTSTATUS Status = FALSE;
PDD_DIRECTDRAW pDirectDraw;
- HANDLE hsurface;
PDD_SURFACE phsurface;
PDD_SURFACE_LOCAL pLocal;
PDD_SURFACE_MORE pMore;
PDD_SURFACE_GLOBAL pGlobal;
+ DD_CREATESURFACEDATA CreateSurfaceData;
+
/* FIXME alloc so mayne we need */
- PHANDLE *myhSurface[1];
+ PHANDLE *myhSurface;
/* GCC4 warnns on value are unisitaed,
but they are initated in seh
*/
- myhSurface[0] = 0;
DPRINT1("NtGdiDdCreateSurface\n");
- _SEH_TRY
- {
- ProbeForRead(hSurface, sizeof(HANDLE), 1);
- ProbeForRead(hSurface[0], sizeof(HANDLE), 1);
- myhSurface[0] = hSurface[0];
- }
- _SEH_HANDLE
- {
- Status = _SEH_GetExceptionCode();
- }
- _SEH_END;
- if(!NT_SUCCESS(Status))
- {
- SetLastNtError(Status);
- return ddRVal;
- }
-
-
- /* Create a surface */
- hsurface = GDIOBJ_AllocObj(DdHandleTable, GDI_OBJECT_TYPE_DD_SURFACE);
- if (!hsurface)
- {
- return ddRVal;
- }
-
- phsurface = GDIOBJ_LockObj(DdHandleTable, hsurface, GDI_OBJECT_TYPE_DD_SURFACE);
+ DPRINT1("Copy puCreateSurfaceData to kmode CreateSurfaceData\n");
+ _SEH_TRY
+ {
+ ProbeForRead(puCreateSurfaceData, sizeof(DD_CREATESURFACEDATA), 1);
+ RtlCopyMemory( &CreateSurfaceData, puCreateSurfaceData,
+ sizeof( DD_CREATESURFACEDATA ) );
+ }
+ _SEH_HANDLE
+ {
+ Status = _SEH_GetExceptionCode();
+ }
+ _SEH_END;
+ if(!NT_SUCCESS(Status))
+ {
+ SetLastNtError(Status);
+ return ddRVal;
+ }
+
+ /* FIXME we only support one surface at moment
+ this is a hack to prevent more that one surface being create
+ */
+ if (CreateSurfaceData.dwSCnt > 1)
+ {
+ CreateSurfaceData.dwSCnt = 1;
+ }
+
+
+ DPRINT1("Setup surface in put handler\n");
+ myhSurface = ExAllocatePoolWithTag( PagedPool, CreateSurfaceData.dwSCnt *
sizeof(HANDLE), 0);
+
+ _SEH_TRY
+ {
+ ProbeForRead(hSurface, CreateSurfaceData.dwSCnt * sizeof(HANDLE), 1);
+ for (i=0;i<CreateSurfaceData.dwSCnt;i++)
+ {
+ myhSurface[i] = hSurface[i];
+ }
+ }
+ _SEH_HANDLE
+ {
+ Status = _SEH_GetExceptionCode();
+ }
+ _SEH_END;
+ if(!NT_SUCCESS(Status))
+ {
+ SetLastNtError(Status);
+ return ddRVal;
+ }
+
+ /* see if a surface have been create or not */
+
+ for (i=0;i<CreateSurfaceData.dwSCnt;i++)
+ {
+ if (!myhSurface[i])
+ {
+ myhSurface[i] = GDIOBJ_AllocObj(DdHandleTable, GDI_OBJECT_TYPE_DD_SURFACE);
+ if (!myhSurface[i])
+ {
+ /* FIXME lock myhSurface*/
+ /* FIXME free myhSurface, and the contain */
+ /* add to attach list */
+ return ddRVal;
+ }
+ else
+ {
+ /* FIXME lock myhSurface*/
+ /* FIXME add to attach list */
+ }
+ }
+ }
+
+ /* FIXME we need continue fix more that one createsurface */
+ /* FIXME we need release myhSurface before any exits*/
+
+ phsurface = GDIOBJ_LockObj(DdHandleTable, myhSurface[0],
GDI_OBJECT_TYPE_DD_SURFACE);
if (!phsurface)
{
return ddRVal;
@@ -96,7 +145,7 @@
_SEH_TRY
{
ProbeForRead(puCreateSurfaceData, sizeof(DD_CREATESURFACEDATA), 1);
- RtlCopyMemory( &phsurface->CreateSurfaceData, puCreateSurfaceData,
+ RtlCopyMemory( &CreateSurfaceData, puCreateSurfaceData,
sizeof( DD_CREATESURFACEDATA ) );
}
_SEH_HANDLE
@@ -206,12 +255,8 @@
pLocal = &phsurface->Local;
pMore = &phsurface->More;
pGlobal = &phsurface->Global;
-
-
- /* FIXME we only support one surface for now */
- phsurface->CreateSurfaceData.dwSCnt = 1;
-
- for (i = 0; i < phsurface->CreateSurfaceData.dwSCnt; i++)
+
+ for (i = 0; i < CreateSurfaceData.dwSCnt; i++)
{
phsurface->lcllist[i] = (PDD_SURFACE_LOCAL)pLocal;
pLocal->lpGbl = pGlobal;
@@ -226,34 +271,104 @@
}
-
- /* FIXME support for more that one surface */
-
/* setup DD_CREATESURFACEDATA CreateSurfaceData for the driver */
- phsurface->CreateSurfaceData.lplpSList = (PDD_SURFACE_LOCAL *)
&phsurface->lcllist;
- phsurface->CreateSurfaceData.lpDDSurfaceDesc = &phsurface->desc;
- phsurface->CreateSurfaceData.CreateSurface = NULL;
- phsurface->CreateSurfaceData.ddRVal = DDERR_GENERIC;
- phsurface->CreateSurfaceData.lpDD = &pDirectDraw->Global;
+ CreateSurfaceData.lplpSList = (PDD_SURFACE_LOCAL *) &phsurface->lcllist;
+ CreateSurfaceData.lpDDSurfaceDesc = &phsurface->desc;
+ CreateSurfaceData.CreateSurface = NULL;
+ CreateSurfaceData.ddRVal = DDERR_GENERIC;
+ CreateSurfaceData.lpDD = &pDirectDraw->Global;
/* the CreateSurface crash with lcl convering */
if ((pDirectDraw->DD.dwFlags & DDHAL_CB32_CREATESURFACE))
{
DPRINT1("0x%04x",pDirectDraw->DD.CreateSurface);
- ddRVal = pDirectDraw->DD.CreateSurface(&phsurface->CreateSurfaceData);
- }
-
- /* FIXME copy data from DDRAWI_ sturct to DD_ struct */
+ ddRVal = pDirectDraw->DD.CreateSurface(&CreateSurfaceData);
+ }
+
+ DPRINT1("Retun value is %04x and driver return code is
%04x\n",ddRVal,CreateSurfaceData.ddRVal);
+
+ /* FIXME support for more that one surface */
+ _SEH_TRY
+ {
+ ProbeForWrite(puSurfaceDescription, sizeof(DDSURFACEDESC), 1);
+ RtlCopyMemory( puSurfaceDescription, &phsurface->desc, sizeof(
DDSURFACEDESC ) );
+ }
+ _SEH_HANDLE
+ {
+ Status = _SEH_GetExceptionCode();
+ }
+ _SEH_END;
+
+ _SEH_TRY
+ {
+ ProbeForWrite(puCreateSurfaceData, sizeof(DD_CREATESURFACEDATA), 1);
+ puCreateSurfaceData->ddRVal = CreateSurfaceData.ddRVal;
+ }
+ _SEH_HANDLE
+ {
+ Status = _SEH_GetExceptionCode();
+ }
+ _SEH_END;
+
+ for (i = 0; i < CreateSurfaceData.dwSCnt; i++)
+ {
+ _SEH_TRY
+ {
+ ProbeForWrite(puSurfaceGlobalData, sizeof(DD_SURFACE_GLOBAL), 1);
+ RtlCopyMemory( puSurfaceGlobalData, &phsurface->Global, sizeof(
DD_SURFACE_GLOBAL ) );
+ }
+ _SEH_HANDLE
+ {
+ Status = _SEH_GetExceptionCode();
+ }
+ _SEH_END;
+
+ _SEH_TRY
+ {
+ ProbeForWrite(puSurfaceLocalData, sizeof(DD_SURFACE_LOCAL), 1);
+ RtlCopyMemory( puSurfaceLocalData, &phsurface->Local, sizeof(
DD_SURFACE_LOCAL ) );
+ }
+ _SEH_HANDLE
+ {
+ Status = _SEH_GetExceptionCode();
+ }
+ _SEH_END;
+
+ _SEH_TRY
+ {
+ ProbeForWrite(puSurfaceMoreData, sizeof(DD_SURFACE_MORE), 1);
+ RtlCopyMemory( puSurfaceMoreData, &phsurface->More, sizeof(
DD_SURFACE_MORE ) );
+
+ puSurfaceLocalData->lpGbl = puSurfaceGlobalData;
+ puSurfaceLocalData->lpSurfMore = puSurfaceMoreData;
+ }
+ _SEH_HANDLE
+ {
+ Status = _SEH_GetExceptionCode();
+ }
+ _SEH_END;
+
+ DPRINT1("GDIOBJ_UnlockObjByPtr\n");
+ GDIOBJ_UnlockObjByPtr(DdHandleTable, phsurface);
+ _SEH_TRY
+ {
+ ProbeForWrite(puhSurface, sizeof(HANDLE), 1);
+ puhSurface[i] = myhSurface[i];
+ }
+ _SEH_HANDLE
+ {
+ Status = _SEH_GetExceptionCode();
+ }
+ _SEH_END;
+ }
+
/* FIXME fillin the return handler */
DPRINT1("GDIOBJ_UnlockObjByPtr\n");
- GDIOBJ_UnlockObjByPtr(DdHandleTable, phsurface);
- DPRINT1("GDIOBJ_UnlockObjByPtr\n");
GDIOBJ_UnlockObjByPtr(DdHandleTable, pDirectDraw);
-
- DPRINT1("Retun value is %04x and driver return code is
%04x\n",ddRVal,phsurface->CreateSurfaceData.ddRVal);
+ DPRINT1("Retun value is %04x and driver return code is
%04x\n",ddRVal,CreateSurfaceData.ddRVal);
return ddRVal;
}
Modified: trunk/reactos/subsystems/win32/win32k/ntddraw/ddraw.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt…
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/ntddraw/ddraw.c (original)
+++ trunk/reactos/subsystems/win32/win32k/ntddraw/ddraw.c Fri Mar 23 20:07:49 2007
@@ -667,40 +667,7 @@
return ddRVal;
}
-DWORD STDCALL NtGdiDdBlt(
- HANDLE hSurfaceDest,
- HANDLE hSurfaceSrc,
- PDD_BLTDATA puBltData
-)
-{
- DWORD ddRVal;
- PDD_DIRECTDRAW_GLOBAL lgpl;
-
- PDD_DIRECTDRAW pDirectDraw = GDIOBJ_LockObj(DdHandleTable, hSurfaceDest,
GDI_OBJECT_TYPE_DIRECTDRAW);
-#ifdef DX_DEBUG
- DPRINT1("NtGdiDdBlt\n");
-#endif
- if (pDirectDraw == NULL)
- return DDHAL_DRIVER_NOTHANDLED;
-
- /* backup the orignal PDev and info */
- lgpl = puBltData->lpDD;
-
- /* use our cache version instead */
- puBltData->lpDD = &pDirectDraw->Global;
-
- /* make the call */
- if (!(pDirectDraw->Surf.dwFlags & DDHAL_SURFCB32_BLT))
- ddRVal = DDHAL_DRIVER_NOTHANDLED;
- else
- ddRVal = pDirectDraw->Surf.Blt(puBltData);
-
- /* But back the orignal PDev */
- puBltData->lpDD = lgpl;
-
- GDIOBJ_UnlockObjByPtr(DdHandleTable, pDirectDraw);
- return ddRVal;
- }
+
DWORD STDCALL NtGdiDdSetColorKey(
HANDLE hSurface,
Modified: trunk/reactos/subsystems/win32/win32k/ntddraw/ddsurf.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt…
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/ntddraw/ddsurf.c (original)
+++ trunk/reactos/subsystems/win32/win32k/ntddraw/ddsurf.c Fri Mar 23 20:07:49 2007
@@ -14,6 +14,107 @@
#include <debug.h>
#define DdHandleTable GdiHandleTable
+
+
+
+DWORD STDCALL NtGdiDdBlt(
+ HANDLE hSurfaceDest,
+ HANDLE hSurfaceSrc,
+ PDD_BLTDATA puBltData
+)
+{
+ NTSTATUS Status = FALSE;
+ DWORD ddRVal = DDHAL_DRIVER_NOTHANDLED;
+ DD_BLTDATA Blt;
+ PDD_SURFACE pDstSurface = NULL;
+ PDD_SURFACE pSrcSurface = NULL;
+ PDD_DIRECTDRAW pDirectDraw;
+
+ DPRINT1("NtGdiDdBlt\n");
+
+ _SEH_TRY
+ {
+ ProbeForRead(puBltData, sizeof(DD_BLTDATA), 1);
+ RtlCopyMemory( &Blt, puBltData, sizeof( DD_BLTDATA ) );
+ }
+ _SEH_HANDLE
+ {
+ Status = _SEH_GetExceptionCode();
+ }
+ _SEH_END;
+ if(!NT_SUCCESS(Status))
+ {
+ SetLastNtError(Status);
+ return ddRVal;
+ }
+
+ pDstSurface = GDIOBJ_LockObj(DdHandleTable, hSurfaceDest,
GDI_OBJECT_TYPE_DD_SURFACE);
+ if (!pDstSurface)
+ {
+ DPRINT1("Fail\n");
+ return ddRVal;
+ }
+
+ pDirectDraw = GDIOBJ_LockObj(DdHandleTable, pDstSurface->hDirectDrawLocal,
GDI_OBJECT_TYPE_DIRECTDRAW);
+ if (!pDirectDraw)
+ {
+ DPRINT1("Fail\n");
+ GDIOBJ_UnlockObjByPtr(DdHandleTable, pSrcSurface);
+ return ddRVal;
+ }
+
+ if (pSrcSurface)
+ {
+ pSrcSurface = GDIOBJ_LockObj(DdHandleTable, pSrcSurface,
GDI_OBJECT_TYPE_DD_SURFACE);
+ if (!pSrcSurface)
+ {
+ DPRINT1("Fail\n");
+ GDIOBJ_UnlockObjByPtr(DdHandleTable, pDstSurface);
+ GDIOBJ_UnlockObjByPtr(DdHandleTable, pDirectDraw);
+ return ddRVal;
+ }
+ }
+
+ Blt.lpDDDestSurface = &pDstSurface->Local;
+ if (pDstSurface)
+ {
+ Blt.lpDDSrcSurface = &pDstSurface->Local;
+ }
+
+ Blt.ddRVal = DDERR_GENERIC;
+
+ if (pDirectDraw->Surf.dwFlags & DDHAL_SURFCB32_BLT)
+ {
+ Blt.lpDD = &pDirectDraw->Global;
+ Blt.Blt = NULL;
+ ddRVal = pDirectDraw->Surf.Blt(&Blt);
+ }
+
+ DPRINT1("Retun value is %04x and driver return code is
%04x\n",ddRVal,Blt.ddRVal);
+
+ GDIOBJ_UnlockObjByPtr(DdHandleTable, pDirectDraw);
+ GDIOBJ_UnlockObjByPtr(DdHandleTable, pDstSurface);
+
+ if(pSrcSurface)
+ {
+ GDIOBJ_UnlockObjByPtr(DdHandleTable, pSrcSurface);
+ }
+
+ _SEH_TRY
+ {
+ ProbeForWrite(puBltData, sizeof(DD_BLTDATA), 1);
+ puBltData->ddRVal = Blt.ddRVal;
+ }
+ _SEH_HANDLE
+ {
+ Status = _SEH_GetExceptionCode();
+ }
+ _SEH_END;
+
+ DPRINT1("Retun value is %04x and driver return code is
%04x\n",ddRVal,Blt.ddRVal);
+ return ddRVal;
+ }
+
/************************************************************************/
6487
days inactive
6487
days old
0 comments
1 participants
participants (1)
-
greatlrd＠svn.reactos.org