QUsb2snes icon indicating copy to clipboard operation
QUsb2snes copied to clipboard

QUsb2snes v0.7.19 to SD2SNES is not returning correct memory values.

Open trippsc2 opened this issue 3 years ago • 9 comments

I'm running QUsb2snes on Manjaro Linux. Since upgrading to v0.7.19, memory checks are returning incorrect and rapidly changing results when connecting to an SD2SNES. I've attached the debug log below.

Reverting to v0.7.18 returns functionality to normal.

log-debug.txt

trippsc2 avatar Oct 20 '20 14:10 trippsc2

This should be fixed in last git version.

Le mar. 20 oct. 2020 à 16:25, trippsc2 [email protected] a écrit :

I'm running QUsb2snes on Manjaro Linux. Since upgrading to v0.7.19, memory checks are returning incorrect and rapidly changing results when connecting to an SD2SNES. I've attached the debug log below.

Reverting to v0.7.18 returns functionality to normal.

log-debug.txt https://github.com/Skarsnik/QUsb2snes/files/5409295/log-debug.txt

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Skarsnik/QUsb2snes/issues/65, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADDBK7Y3XBYH2S7GGZ2UZQDSLWMT5ANCNFSM4SYNVIQA .

Skarsnik avatar Oct 20 '20 14:10 Skarsnik

The debug log I provided was with the latest commit. The issue does not appear to be fixed.

I'm going to rebuild and retest and provide another log.

trippsc2 avatar Oct 20 '20 14:10 trippsc2

After doing a clean build to verify, I get the same result. New log attached.

log-debug.txt

trippsc2 avatar Oct 20 '20 14:10 trippsc2

I mean the last git commit, not the latest release, I fixed a similar issue but did not do a release yet.

Le mar. 20 oct. 2020 à 16:58, trippsc2 [email protected] a écrit :

After doing a clean build to verify, I get the same result. New log attached.

log-debug.txt https://github.com/Skarsnik/QUsb2snes/files/5409537/log-debug.txt

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Skarsnik/QUsb2snes/issues/65#issuecomment-712914834, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADDBK727M7FMRJUOJJSHHNLSLWQQPANCNFSM4SYNVIQA .

Skarsnik avatar Oct 20 '20 15:10 Skarsnik

I understood what you meant. The original issue was generated with the latest version of the master branch compiled from source. I rebuilt again to verify this. The issue is not fixed.

trippsc2 avatar Oct 20 '20 15:10 trippsc2

I tried running a stress test with 10 clients doing 1000 request or one client doing 10000 request and no issue so far. I will try on my Linux env later

Skarsnik avatar Oct 28 '20 17:10 Skarsnik

Is this still a problem for you?

For me commit 0649ecd fixed most issues on Arch, but a different change in v0.7.19 will make QUsb send partial data back to the application (i.e. application requests 512 bytes, qusb will reply 2x 256 bytes), which is fine with TCP, but may reveal bugs in applications. Menblock's secret of mana rando tracker for example does not like this.

black-sliver avatar Feb 20 '21 13:02 black-sliver

I haven't tested since my last reply, because I hadn't seen any updates. I'll give it a try later today and see if I see something similar. I will also try limiting requests to 256 bytes and see if that makes a difference. It's possible that there is an issue in my app when fetching larger datasets.

trippsc2 avatar Feb 24 '21 16:02 trippsc2

I reduced the maximum data requested to 256 bytes and tested on the latest version of Qusb2snes from the AUR (displays v7.19.3) and still see the same rapidly changing values. This behavior is only occurring on an SD2SNES setup, not using Retroarch.

trippsc2 avatar Feb 25 '21 15:02 trippsc2