melonDS-emu / melonDS

DS emulator, sorta

https://melonds.kuribo64.net

GNU General Public License v3.0

3.15k stars 517 forks source link

ARM7 and ARM9 debugger doesn't work at all: #2147

Closed micsthepick closed 10 hours ago

micsthepick commented 2 weeks ago

This line is outdated:

https://github.com/melonDS-emu/melonDS/blob/4f6498c99c5dcdb780371fe936d49e32df148e6e/src/frontend/qt_sdl/EmuSettingsDialog.cpp#L98

Needs to take into account instance number?

2144

I believe that this is the main cause.

micsthepick commented 2 weeks ago

... scratch that, I didn't notice that it's using instcfg instead of cfg.

Arisotura commented 2 weeks ago

yeah, the gdb stuff needs some fixing and reworking. this is weird

On Fri, Aug 30, 2024 at 2:17 AM micsthepick @.***> wrote:

https://github.com/melonDS-emu/melonDS/blob/4f6498c99c5dcdb780371fe936d49e32df148e6e/src/frontend/qt_sdl/EmuSettingsDialog.cpp#L98

Needs to take into account instance number?

— Reply to this email directly, view it on GitHub https://github.com/melonDS-emu/melonDS/issues/2147, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKAI64V2RRZWPHESK7RVHLZT62YBAVCNFSM6AAAAABNLODLJWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ4TKOBXGQ4DQMQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Ukalus commented 3 days ago

i ran into this too, is there a workaround?

RSDuck commented 3 days ago

https://github.com/melonDS-emu/melonDS/commit/74f479ce6d956375f9bf9d2c3fd0b0b7a6f4ad56 fixed this particular issue.

Though it looks like there is a race condition in the GDB protocol (which makes especially break on start buggy). I haven't figured completely out how to fix.

RSDuck commented 10 hours ago

https://github.com/melonDS-emu/melonDS/commit/a3d696121e8d0643bbe45d6bdecdb99e2d62cbe1 should have hopefully done the rest.

If you are still experiencing issues (besides the known one of gdb stub settings not being reflected properly until a restart) please let me know.