Changeset 144 for trunk/src/os2ahci/os2ahci.def.template

Timestamp:

Jan 3, 2013, 1:22:56 PM (13 years ago)

Author:

rousseau

Message:

Fixed the Open Watcom build

Building with Open Watcom produced a non working driver.

Causes & Fixes

o Enum types were 8 bits when using WCC

This caused the functions with variable arguments to fail.
Adding the -ei compiler-flag to WCC solved this issue.

o Incorrect inline assembler syntax to call DeviceHelp in os2ahci.c

DeviceHelp is a 16:16 far pointer residing in memory.
MASM interprets 'call DeviceHelp' differently than WASM.
The correct syntax is: 'call dword ptr [DeviceHelp]'.

o Missing segments from TGROUP in init.asm

WLINK then generates the AUTO class, most probably causing incorrect
relocation fixups.

Enhancements

o WASM (or JWASM) can now be used instead of ALP
o WCC can now be used instead of CL
o The Open Watcom linker (WLINK) can now be used instead of LINK
o Streamlined both makefiles a notch so it's easy to use both tool-chains

To investigate

o CL barks about a missing prototype for DevHelp_Yield() in lib.c

The prototype is actually defined in dhcalls.h but CL still barks.

o Using WCC with ALP and WLINK produces an oversized image

Probaly due to misplaced BSS segments. WCC with ALP and LINK produces
a normal sized image however.

File:

• trunk/src/os2ahci/os2ahci.def.template (modified) (1 diff)

trunk/src/os2ahci/os2ahci.def.template

@@ -4 +4 @@
 
 segments
-  DEVHDR        CLASS 'DATA'
-  _DATA         CLASS 'DATA'
-  CONST         CLASS 'CONST'
-  _BSS          CLASS 'BSS'
-  c_common      CLASS 'BSS'
-  _z_data       CLASS 'BSS'
-  _TEXT         CLASS 'CODE' IOPL
+  'DEVHDR'      CLASS 'DATA'
+  '_DATA'       CLASS 'DATA'
+  'LIBDATA'     CLASS 'DATA'
+  'CONST'       CLASS 'CONST'
+  '_BSS'        CLASS 'BSS'
+  'c_common'    CLASS 'BSS'
+  '_z_data'     CLASS 'BSS'
+  '_TEXT'       CLASS 'CODE' IOPL
   'CODE'        CLASS 'CODE' IOPL
-  RMCode        CLASS 'CODE' IOPL
-  LIBCODE       CLASS 'CODE' IOPL
-  _z_text       CLASS 'CODE' IOPL
-
+  'LIBCODE'     CLASS 'CODE' IOPL
+  'RMCode'      CLASS 'CODE' IOPL
+  '_z_text'     CLASS 'CODE' IOPL