Minihwc.c getRegPath() returns null on Win98

[NejcDolicanin]

Hi, if you could help me a little.

Im trying to make it work on win98, but cant seem to figure this one out.

User env variables are not being set, because in minihwc.c, function getRegPath() return null when checked in function hwcGetenv.

if (regPath == NULL) {
regPath = getRegPath();

if (regPath == NULL){
  return NULL;
}

Works fine for winXp. Works if built with DEBUG=1 parameter in win98.
I can hardcode the return string
CM_Get_DevNode_Key( QDevNode.dwDevNode, NULL,
strval, sizeof(strval),
CM_REGISTRY_SOFTWARE );

  //strcat(strval, "\\glide");
  retVal = "SYSTEM\\CurrentControlSet\\Services\\Class\\Display\\0000\\glide";

Ofc that works, but its not really a solution :).

[sezero]

How do you compile glide?

Assuming that you are building with MSVC6 and building h5: If you change the #if 0 to #if 1 at
https://github.com/sezero/glide/blob/glide-devel-sezero/glide3x/h5/minihwc/minihwc.c#L770
(and possibly remove win9x.obj from objects list) and build against win9x ddk: does it work then?

[NejcDolicanin]

Thank you for responding.
Yes h5.

I build on winxp, with winxpddk(2600) with win98sdk, vc6 headers and all other old stuff installed. I dont use win9x.obj, reverted to state on sourceforge glide dev branch.

I tried Koolsmokys built dll
(https://www.3dfxzone.it/news/reader.php?objid=5280)

Has the same behaviour. Xp fine, win98 no-go.

Works fine if built as debug, so hard for me to debug it :)

[sezero]

I tried Koolsmokys built dll (https://www.3dfxzone.it/news/reader.php?objid=5280)

Has the same behaviour. Xp fine, win98 no-go.

Huh. OK, will look at this as soon as possible.

[sezero]

Just to confirm: does this build work properly instead? h5.zip
It is built with mingw, optimized.

[NejcDolicanin]

No this one doesnt work.

[sezero]

That getRegPath returns NULL for you means that ExtEspace is somehow failing, don't know how - yet.

[NejcDolicanin]

Thanks for looking into it, I spent more time than Im willing to admit, with no resolve :)

[sezero]

Two more questions though:

Did you build from my own branch with DEBUG=1 and if you did, does it work too?

The error screenshot you showed above: Unreal uses glide2x, not glide3x: how are
you making it to use glide3x?

[NejcDolicanin]

Just tested it now, from your codebase as debug=1 doesnt work, will ignore user settings(switched about 3 weeks ago, forgot).
This is the exact one Im using now:
sourceForgeDevelop.zip

Wanted as little change as posible and when everything worked, the plan was to add your changes to it.

I use a glide2 to glide3 wrapper from this guy: https://wenchy.net/old/glidexp/, so everything glide will go to glide3x.

[NejcDolicanin]

You trying a different compiler got me thinking. What if really isnt a code problem?

So I tried a couple, and win2kddk compiles it correctly, registry path gets read.
Ofc, as always, something else broke, now asm part is broke.

I can only compile it with "make -f Makefile.win32 FX_GLIDE_HW=h5" without the USE_X86=1.

Come to think of it, could just be that it doesnt find nasm

[sezero]

Those line numbers don't make any sense..

[NejcDolicanin]

Yes, mine has a bit different lines. Its just asm code, I think the way I have it now(with 2kddk istead of xp) it doesnt use nasm. I tried moving it around, not sure how to configure it to make it work.

• Ok its not that, it takes nasm just well, the problem is somewhere else.

Heres the file so you can see what are the lines
minihwc.zip

[sezero]

Nothing to affect build in your changes as far as I can see.

[sezero]

Maybe instead of moving configmg.h stuff into win9x.asm
I should inline them into c code and let compiler do its thing.
Like below:

diff --git a/glide3x/h5/glide3/src/Makefile.mingw b/glide3x/h5/glide3/src/Makefile.mingw
index 3900876..2a05bbc 100644
--- a/glide3x/h5/glide3/src/Makefile.mingw
+++ b/glide3x/h5/glide3/src/Makefile.mingw
@@ -273,5 +273,2 @@ GLIDE_OBJECTS += \
 	../../minihwc/win_mode.o
-# so that we don't need win9x ddk:
-GLIDE_OBJECTS += \
-	../../minihwc/win9x.o
 
@@ -368,5 +365,2 @@ xdrawtri.o: xdrawtri.asm
 
-../../minihwc/win9x.o: ../../minihwc/win9x.asm
-	$(AS) -o $@ -O2 -fwin32 $<
-
 $(GLIDE_OBJECTS): fxinline.h fxgasm.h fxbldno.h
diff --git a/glide3x/h5/glide3/src/Makefile.win32 b/glide3x/h5/glide3/src/Makefile.win32
index 8241ac0..6702b90 100644
--- a/glide3x/h5/glide3/src/Makefile.win32
+++ b/glide3x/h5/glide3/src/Makefile.win32
@@ -222,5 +222,2 @@ GLIDE_OBJECTS += \
 	../../minihwc/win_mode.obj
-# so that we don't need win9x ddk:
-GLIDE_OBJECTS += \
-	../../minihwc/win9x.obj
 
@@ -304,5 +301,2 @@ xtexdl_sse2.obj: xtexdl.asm
 
-../../minihwc/win9x.obj: ../../minihwc/win9x.asm
-	$(AS) -o $@ -O2 -fwin32 $<
-
 $(GLIDE_OBJECTS): fxinline.h fxgasm.h fxbldno.h
diff --git a/glide3x/h5/minihwc/minihwc.c b/glide3x/h5/minihwc/minihwc.c
index ed46249..b51cf93 100644
--- a/glide3x/h5/minihwc/minihwc.c
+++ b/glide3x/h5/minihwc/minihwc.c
@@ -767,19 +767,68 @@
 
 #include <ddraw.h>
 #include "qmodes.h"
-#if 0 /* moved to asm so we don't need w9x ddk headers. */
+#if 0 /* inlined here so we don't need w9x ddk headers. */
 #define IS_32
 #define Not_VxD
 #include <minivdd.h>
 #include <vmm.h>
 #include <configmg.h>
 #else
-extern DWORD __cdecl CM_Get_DevNode_Key(DWORD,PCHAR,PVOID,ULONG,ULONG);
 #define CM_REGISTRY_HARDWARE 0
 #define CM_REGISTRY_SOFTWARE 1
+#define CR_FAILURE 0x00000013
+#define ___CONFIGMG_Get_DevNode_Key 0x0033003d
+struct _CMIOCTLPACKET {
+ DWORD dwStack;
+ DWORD dwServiceNumber;
+};
+static DWORD WINAPI CMIOCTLHandler(struct _CMIOCTLPACKET *pkt)
+{
+  HANDLE hCONFIGMG;
+  DWORD crReturnValue = CR_FAILURE;
+  DWORD dwReturnSize = 0;
+
+  hCONFIGMG = CreateFileA(
+    "\\\\.\\CONFIGMG",
+    GENERIC_READ|GENERIC_WRITE,
+    FILE_SHARE_READ|FILE_SHARE_WRITE,
+    NULL, OPEN_EXISTING, 0, NULL);
+
+  if (hCONFIGMG == INVALID_HANDLE_VALUE) {
+      return CR_FAILURE;
+  }
+  if (!DeviceIoControl(
+        hCONFIGMG, pkt->dwServiceNumber,
+        &(pkt->dwStack), sizeof(pkt->dwStack),
+        &crReturnValue, sizeof(crReturnValue),
+        &dwReturnSize, NULL)) {
+    crReturnValue = CR_FAILURE;
+  }
+  CloseHandle(hCONFIGMG);
+  if (dwReturnSize != sizeof(crReturnValue)) {
+      crReturnValue = CR_FAILURE;
+  }
+  return crReturnValue;
+}
+static DWORD __cdecl CM_Get_DevNode_Key (DWORD devnode, char *subkey, void *buffer, ULONG bufferlen, ULONG flags)
+{
+    struct _CMIOCTLPACKET packet;
+    DWORD dwStack;
+    #if defined(_MSC_VER)
+    _asm {mov dwStack, ebp};
+    #elif defined(__GNUC__)
+    dwStack = (DWORD) __builtin_frame_address(0);
+    #else
+    #error Add support for your compiler here.
+    #endif
+    dwStack += 8;
+    packet.dwStack = dwStack;
+    packet.dwServiceNumber = 0x80000000 + (___CONFIGMG_Get_DevNode_Key & 0xFFFF);
+    return CMIOCTLHandler(&packet);
+}
 #endif
 
-#endif
+#endif /* __WIN32__ */
 
 #ifdef macintosh
 #include <GraphicsPrivHwc.h>

Does the inlined functionality look correct to you? (Sorry, that I still
can't run-test stuff, yet.)

Here is a build using mingw gcc with such a change: h5-2.zip

Please try that build, and also yours if you can e.g. with MSVC etc.

[sezero]

Nothing to affect build in your changes as far as I can see.

It always seems to choke on MovQ mm3, mmword ptr [edx]

I remember to have tried building with MSVC6 with success in the past.
Will try again soon.