05408: savquest: regression in protected mode memory mapping

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
05408	Misc.	Major	Always	Dec 31, 2013, 05:03	Aug 17, 2017, 15:35

Tester	peterferrie	View Status	Public	Platform	MAME (Official Binary)
Assigned To		Resolution	Fixed	OS
Status [?]	Resolved			Driver	misc/savquest.cpp
Version	0.151	Fixed in Version		Build	Normal
		Fixed in Git Commit		Github Pull Request #

Summary	05408: savquest: regression in protected mode memory mapping
Description	Savage Quest runs a DOS4GW executable during startup to check for the presence of the HASP dongle. In 0.149 and 0.150 and earlier, the check passes. In 0.151 and 0.152, the execution fails with message "unable to initialize loader".
Steps To Reproduce	mame savquest press 's' when the CMOS failure message appears, and then wait. Memory is checked, files are copied, Sound Blaster is initialized, two "file not found" messages will appear, and then the check will be performed and the loader failure message will appear.
Additional Information	can't find any changes to savquest.c code in that timeframe, so suspect x86 emulation change is responsible instead
Github Commit

Flags
Regression Version
Affected Sets / Systems	savquest

Attached Files

No.10112 peterferrie Developer Jan 1, 2014, 23:37	bad behavior starts much earlier than that - CMOS reads are returning garbage value for floppy type during DOS boot (0xd8 instead of 0x40), DOS can't load high anymore because HMA isn't available, etc.
No.10115 hap Developer Jan 2, 2014, 19:46	copypasted: smf:" NAK. The MC146818 only has 64 bytes. Savage Quest obviously doesn’t have an MC146818. Maybe it should be using the DS12885? "
No.13507 Kale Developer Dec 26, 2016, 00:40	http://mamedev.emulab.it/kale/fast/files/0000_9315221006.png Dies to me here, I guess this can be considered resolved?
No.14108 Osso Moderator Aug 17, 2017, 15:35	As Kale wrote, this particular regression was fixed long ago.