08999: jaguar: Quickload crashes

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
08999	Crash/Freeze	Critical (emulator)	Always	14 days ago	1 day ago

Tester	Robbbert	View Status	Public	Platform	MAME (Self-compiled)
Assigned To	Robbbert	Resolution	Fixed	OS	Windows 10/11 (64-bit)
Status [?]	Resolved			Driver	atari/jaguar.cpp
Version	0.272	Fixed in Version	0.273GIT	Build	64-bit
		Fixed in Git Commit	9f8d3e7	Github Pull Request #

Summary	08999: jaguar: Quickload crashes
Description	A long long time ago, the jaguar driver could run most homebrews. Today that is no longer the case. The first to go was breakage of ".rom" files. They are carts without the system header, which must be supplied by the emulator. This crucial support was removed many years ago, so ".rom" files just end up at a red "jaguar" screen - indicative of an invalid cart. A few years ago I did some testing and found that all the quickload homebrews now started at the wrong resolution, making them unplayable. And now, all of the quickloads crash at the red warning screen, which means that homebrew is entirely unsupported. The only software that works is ".j64", which are mostly commercial cartridges.
Steps To Reproduce	You can choose any quickloads, which can be found at atariage and other places. An example: >mame jaguar -quik e:\data\atari\jaguar\blobs.cof When the red warning screen appears it will instantly crash.
Additional Information	C:\MAME>mame jaguar -quik e:\data\atari\jaguar\blobs.cof ----------------------------------------------------- Exception at EIP=00007ff75ee2b1ce (jaguar_state::quickload_cb[abi:cxx11](snapshot_image_device&)+0x00ee): ACCESS VIOLATION While attempting to read memory at 0000000000000028 ----------------------------------------------------- RAX=0000000000000000 RBX=000002141b878080 RCX=3a6d61696e726f6d RDX=000001a04414a098 RSI=00000073d7929950 RDI=000002141901dbb0 RBP=00000073d7929810 RSP=00000073d79297b0 R8=0000000000000008 R9=0000000000000001 R10=000000000000f9a7 R11=0000021418f80e75 R12=000000000000f9a7 R13=00000073d7929cb0 R14=00007ff76db28520 R15=000000000000f9a7 ----------------------------------------------------- Stack crawl: 00000073d7929900: 00007ff75ee2b1ce (jaguar_state::quickload_cb[abi:cxx11](snapshot_image_device&)+0x00ee) 00000073d7929b50: 00007ff7628fe707 (snapshot_image_device::process_snapshot_or_quickload(int)+0x0047) 00000073d7929bd0: 00007ff7627744db (device_scheduler::timeslice()+0x013b) 00000073d7929d50: 00007ff762771735 (running_machine::run(bool)+0x01c5) 00000073d792f370: 00007ff7658c370c (mame_machine_manager::execute()+0x024c) 00000073d792f760: 00007ff7695e1aba (cli_frontend::start_execution(mame_machine_manager*, std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > const&)+0x03ea) 00000073d792fa70: 00007ff7695e20ea (cli_frontend::execute(std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > >&)+0x007a) 00000073d792fad0: 00007ff7658be4b7 (emulator_info::start_frontend(emu_options&, osd_interface&, std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > >&)+0x0027) 00000073d792fea0: 00007ff76b277271 (luaopen_lfs+0xc5e6b1) 00000073d792fef0: 00007ff75e9d12ee (__tmainCRTStartup+0x016e) 00000073d792ff20: 00007ff75e9d1406 (mainCRTStartup+0x0016) 00000073d792ff50: 00007ffb419c7374 (BaseThreadInitThunk+0x0014) 00000073d792ffd0: 00007ffb4219cc91 (RtlUserThreadStart+0x0021)
Github Commit

Flags
Regression Version	0.217
Affected Sets / Systems	jaguar

Attached Files

No.22578 Robbbert Moderator 10 days ago	Regression commit: f9695c58c0cc5f588b6eb162018c1136414c4193 Quickload is referring to region "maincpu" which doesn't exist after that commit.
No.22579 Robbbert Moderator 9 days ago	I've fixed all the issues of this report in the current git of mameui. The changes are here: https://github.com/Robbbert/mameui/commit/a580645c0ccf7a2ee0b4d980887ab84af5bbac9d#diff-0368bb351f239c2e7ca5ac1f21f7d4c6e589e4ec1f4b745d6758cf8f9e023777 if there's any interest from the devs.