Opened 2 weeks ago

Closed 6 days ago

Last modified 5 days ago

#15513 closed defect (fixed)

M4: black screen (BE/MORPHOS)

Reported by: BeWorld2018 Owned by: ccawley2011
Priority: normal Component: Engine: M4
Version: Keywords: orion burger m4
Cc: Game: Orion Burger

Description

Platform: MorphOS / BE
Game: Orion Burger (DOS/French)

When i launch, blackscreen, i heart sound and voice.
After some time... ScummVM crash

ScummVM: 2.10.0git 19.11.2024

Attachments (5)

{FD5707EB-FD2F-4618-921B-C7971820D314}.png (89.5 KB ) - added by BeWorld2018 2 weeks ago.
{F8E74922-0DB1-4F29-9439-F11E9DEB5183}.png (43.8 KB ) - added by BeWorld2018 2 weeks ago.
{8DE559C9-74AE-4187-893D-7870FF6C4706}.png (449.9 KB ) - added by BeWorld2018 7 days ago.
{B833D6F0-7B2A-431F-A63C-E74944D3A145}.png (521.8 KB ) - added by BeWorld2018 7 days ago.
{010B7C9A-2726-4077-8B38-D272BAE2B965}.png (164.3 KB ) - added by BeWorld2018 7 days ago.

Download all attachments as: .zip

Change History (15)

comment:1 by BeWorld2018, 2 weeks ago

Seem effect "riddle": Ripley's Believe it or not (DOS/French)

comment:2 by BeWorld2018, 2 weeks ago

Summary: M4: ORION BUGER FR black screen (BE/MORPHOS)M4: black screen (BE/MORPHOS)

comment:3 by dwatteau, 10 days ago

Thanks for testing and reporting this.

Indeed, I saw the same symptoms on PS3 and on my big-endian PowerBook G4, this week.

The funny thing is that Orion Burger (and its French release) used to work on that PowerBook G4, back in July or something like that.

I'm investigating this with a Git bisect, to see whether it's a regression, and if so, which commit could have introduced that...

comment:4 by dwatteau, 9 days ago

So, a Git bisect on my big-endian system gave me commit 39f0c165c30118919ba082372acdd18ea72f2b3d ("M4: Avoid unaligned memory access in the memory stash code") (<https://github.com/scummvm/scummvm/commit/39f0c165c30118919ba082372acdd18ea72f2b3d>) as the source of the regression.

i.e. that change should maybe just work with byte instead of uintptr?

by BeWorld2018, 7 days ago

by BeWorld2018, 7 days ago

by BeWorld2018, 7 days ago

comment:5 by BeWorld2018, 7 days ago

ok after change byte instead of uintptr
result with OrionBurger working (check screenshot)
result same with Ripley.. ,ow working

Wow :-)

comment:6 by dwatteau, 6 days ago

In 26c13427:

M4: Fix mem_get_from_stash() causing black screen regression on big-endian hosts (Trac#15513)

This fix is required on big-endian hosts, after the changes introduced in
commit 39f0c165c30118919ba082372acdd18ea72f2b3d.

Issue reported by BeWorld2018, fix suggested by ccawley2011, tested by
myself, and OK'd by dreammaster.

comment:7 by dwatteau, 6 days ago

Owner: set to ccawley2011
Resolution: fixed
Status: newclosed

@BeWorld2018: I've pushed a shorter fix, suggested by ccawley2011. It works for me on my PowerBook G4, so I've pushed it to branch-2-9 as well.

Feel free to reopen this ticket if you still encounter any issue on MorphOS after this change. Thanks!

comment:8 by dwatteau, 6 days ago

In 08b2d938:

M4: Fix mem_get_from_stash() causing black screen regression on big-endian hosts (Trac#15513)

This fix is required on big-endian hosts, after the changes introduced in
commit 39f0c165c30118919ba082372acdd18ea72f2b3d.

Issue reported by BeWorld2018, fix suggested by ccawley2011, tested by
myself, and OK'd by dreammaster.

(cherry picked from commit 26c13427cc6042a120c408148f550478b8682ac8)

comment:9 by dwatteau, 6 days ago

and then a better fix was added in GitHub PR https://github.com/scummvm/scummvm/pull/6267

(mentioning it here for traceability reasons)

comment:10 by BeWorld2018, 5 days ago

Working nice !!! thanks very much to all !

Note: See TracTickets for help on using tickets.