gpu_neon: reserved bpp is 16bpp, not 4bpp

drc: don't abort on game crash

Was not the best user experience, with this the user can load
another game at least.

drc: add apparently missing ROREG loading

sjump_assemble had different loading for taken/not taken cases for
whatever reason, and I added ROREG loading according to INVCP, which
was (maybe?) erroneously missing for 'not taken' case. Fixes CTR crash
shortly after starting driving.

drc: some more debug logging

Merge pull request #598 from pcercuei/lightrec_events

Unlock Lightrec to its full potential

lightrec: Run dynarec until next interrupt

Instead of running the dynarec once per block, which creates a huge
overhead as entering/exiting the dynarec isn't very fast, run only the
number of cycles until the next interrupt.

This boosts performance by a huge margin. On my PC, running the intro
video in MediEvil goes from using 34-36% CPU usage down to 16-17%.

Signed-off-by: Paul Cercueil <paul@crapouillou.net>
Co-developed-by: notaz <notasas@gmail.com>
Signed-off-by: notaz <notasas@gmail.com>

drc: add seemingly missing double-alloc check

seems like it was possible to alloc the same reg on 2 host regs

edit: found after all the debugging: mupen64plus/mupen64plus-core@8f4cc2bdc93c41801077abda3197a3cb3b89cbb7

drc: clear dirty flag when deallocating a reg

In later stages it may place special regs like AGEN there, and things
like AGEN being marked dirty seemed wrong, although probably harmless
as there is no code to writeback AGEN.

drc: init some variables, adjust comments

no proof that is actually needed, but it felt like some path could
pick up something uninitialized

drc: emulate break opcode

sort-of-but-not-really used by "F1 2000"
(SLUS01120, not "Formula One 2000").

drc: don't cache psxRegs.interrupt

cdrom code can now cancel interrupts, so must always use the global

lightrec: Factorize memhandler state restore into a single function

Signed-off-by: Paul Cercueil <paul@crapouillou.net>

drc: another hack to try to get Vita to work

drc: a bit more sophisticated f1 hack

yes it sucks, I know...

always describe to get git hash in log messages

vita: add a seemingly missing include

drc: possibly the missing piece for Vita

Lost during merge of dynarec fixes.
_newlib_vm_size_user might be the critical bit that overrides
a weak symbol in bootstrap/vita/sbrk.c

drc: align size passed to mprotect

Maybe this makes svcControlProcessMemory() not do it's work there?
It doesn't seem to be failing though.

spu: try to eliminate some cdda clicks/pops

cdda feeds less data than consumes due to poor timings,
especially for PAL

spu: save/restore decode_pos

SPUOSSFreeze_t conveniently had padding space for it

cdrom: play cdda regardless of report mode

Vib-Ribbon doesn't use it while playing.

Merge pull request #587 from notaz/libretro_drc_fixes2

Libretro drc fixes2

libretro: bump version for dynarec fixes

... and let 'l' show that it's libretro fork build and not upstream

drc: always clear full decoded_insn

Some stuff was left after INTCALL conversion and broke branch in delay
slot handling, like in Rayman 2.

minor cleanup

fix some issues and warnings seen on ctr build

drc: reapply some libretro changes

Merge pull request #586 from notaz/libretro_overwrite_drc

overwrite dynarec related code with upstream version

overwrite dynarec related code with upstream version

This gives a fast arm64 dynarec and many other ari64 dynarec fixes
from upstream. Although I tried to take care not to overwrite libretro
specific changes like lightrec, some things may have got lost or
broken. Only tested on rpi4 in 64bit mode.

Warning: untested on Android and most other platforms. If there are
issues from this merge, post a comment and tag me and I'll try to take
a look at it while I'm still active, at least for the time being.

Merge pull request #579 from gameblabla/pio_writes_sen_lib

psxmem: Writes to PIO Expansion area have no effect.

psxmem: Writes to PIO Expansion area have no effect.

This is related to the Tetris with Card Captor fix a while back.
However, i forgot to merge this one as well related to writes,
even though it was not really necessary to fix that particular game.

Co-authored-by: senquack <dansilsby@gmail.com>

Merge pull request #578 from gameblabla/counters_lib

One more on odd lines for PAL HSYNC

One more on odd lines for PAL HSYNC

As done by Duckstation
https://github.com/stenzek/duckstation/blob/bbcf1c67d1aefd5de9cdc9c158f92bc7aaecaa63/src/core/gpu.h#L56

Merge pull request #576 from gameblabla/cdrom_regression_remove

Remove CdlGetLocP hack

Merge pull request #577 from gameblabla/cdrom_setloc_lib

Move CdlSetLoc to CdrInterrupt and return invalid arg error

Move CdlSetLoc to CdrInterrupt and return invalid arg error

So far, i could only find Simple 1500 Series Vol. 31 - The Sound Novel to be affected by this.
In Duckstation, this was causing extra delays without it.
However in our case, this doesn't seem to be the case and i couldn't find much find about it.

Remove CdlGetLocP hack

This was added back in 2013 or so in PCSX Rearmed
and according to some tests against Tomb Raider 1 (which is affected by the GetLocP code),
it works properly without this hack.

So let's just remove it as we are now doing it properly.

Notaz also forgot the reason why he introduced the hack
in the first place, see :
https://github.com/notaz/pcsx_rearmed/pull/235#issuecomment-955026232

Merge pull request #574 from rtissera/rpi3_64

Add Raspberry Pi 3 64-bit target

Add Raspberry Pi 3 64-bit target

Merge pull request #569 from gameblabla/gte_stalling_libretro

GTE stalls/timings

Merge pull request #571 from gameblabla/cdrom_audio_fix_attempt

Fix stutter before speech samples in Castlevania Symphony of the Night

Hack fix for CDROM timings regression (Castlevania, Megaman X4)

See https://github.com/libretro/pcsx_rearmed/issues/557.

The CDROM timings changes from Redux fixed a bunch of games
(Crash Team Racing's intro no longer cuts off too early,
Worms Pinball gets further instead of just crashing,
FF8 Lunar Cry FMV no longer crashes etc...)
but it broke the other games that relied on ADPCM samples,
which is most noticeable in games like Castlevania or Megaman X4.

According to nicolasnoble, we should not cause a delay if seek destination is close to the current location.
However, this would require a lot more work so in the meantime,
let's do just that and add a comment.

Merge pull request #572 from gameblabla/cd_volume_fix_lib

Fix CD Volume issue in Star Wars Dark Forces

Merge pull request #564 from jdorigao/rpi1

Added rpi1 to makefile

Fix CD Volume issue in Star Wars Dark Forces

CD Volume is 16-bits signed, not unsigned.
Otherwise in Star Wars - Dark Forces :
if you lower the music volume slider all the way down,
the volume will wrap around and instead be set at the highest volume.

GTE stalls/timings

This is to fix an issue with Battle Arena Toshinden 1 going too fast.
Zero Divide is also said to be affected by this as well.

Note that this doesn't fix it for gte_arm.S :
We would still need to increase the cycle count here.

Co-authored-by: firnis <firnis@ya.ru>

Merge pull request #570 from Sanaki/drop_ps3

Revert "libretro: Add support for ps1_rom.bin (PS3 BIOS)"

Revert "libretro: Add support for ps1_rom.bin (PS3 BIOS)"

This reverts commit c5c20552af08fd4158cc0e7894c1798d95f0d35b.

Merge pull request #567 from gameblabla/fix_minor_close_iso

Fix minor issue when closing ISO.

Fix minor issue when closing ISO.

Merge pull request #566 from gameblabla/neon_fix_libretro

Always look up verify_dirty literals from offsets by neonloop

Always look up verify_dirty literals from offsets by neonloop

Literals are deduplicated, so there's no guarantee they will be stored
next to each other, even if they're written sequentially. verify_dirty
and get_bounds must use the offsets on each instruction, instead of
assuming values are stored sequentially.

Thanks neonloop for the fix

Co-authored-by: neonloop <neonloop@pm.me>

Added rpi1 to makefile

drflac - assume DRFLAC_NO_STDIO by default

Merge pull request #563 from gameblabla/remove_junk

Remove junk files and fix lack of clearing icache when loading new games

Remove junk files and fix regression in icache mode with Armored Core.

Merge pull request #562 from gameblabla/cdrom_fixes_libretro_precautions

Revert some changes as they were not tested or are not required

Revert some changes as they were not tested or are not required

Per feedback from notaz, thanks to him.

Merge pull request #545 from gameblabla/icache_emulation

ICache emulation (Interpreter) from PCSX redux

Fix comment to be more precise about lightrec's status.

Culling off cache bits from the hardware addresses.

Co-authored-by: Nicolas Noble <nicolasnoble@users.noreply.github.com>

Interpreter fixes for BREAK and a few other instructions.

This should fix F1 2000. (but it's possible that game
needs some tweaks to the CDROM timings as it's quite sensitive to them)

Co-authored-by: Nicolas Noble <nicolasnoble@users.noreply.github.com>

[Interpreter] Link even if branch is not taken in BGEZAL/BLTZAL

Source :
grumpycoders/pcsx-redux@c1a0569

Co-authored-by: wheremyfoodat <44909372+wheremyfoodat@users.noreply.github.com>

OpenBus changes from PCSX-Redux

Co-authored-by: Nicolas Noble <nicolasnoble@users.noreply.github.com>

Merge Icache emulation from PCSX Redux + Senquack changes from PCSX4ALL

See (Redux) :
https://github.com/grumpycoders/pcsx-redux/commit/1923ce5

See original icache implementation (mirror of PCSX Reloaded):
https://github.com/gameblabla/pcsxr/commit/6916151

Without icache emulation, F1 2001 will greatly misbehave :
if you accelerate, the car will go around like crazy.
With icache emulation, it works as intended.

Note that our code is slightly different from theirs as i found out that
having the icache arrays in psxregs would cause crashes so instead
what i'm doing is to taking them out of there and only allocating them
on the heap (due to their great size).

Known issue(s) : Instruction cache emulation code fails to run
Spyro 2 PAL version. (or is it because of SBI?)

Co-authored-by: Nicolas Noble <nicolasnoble@users.noreply.github.com>
Co-authored-by: senquack <dansilsby@gmail.com>

Merge pull request #552 from libretro/fp_fix_libretro

Implement fix from Mednafen for Fantastic Pinball Kyuutenkai.

Merge pull request #561 from gameblabla/sony_armored_fix_libretro

Fix for Armored Core misdetecting a Link cable being detected

Fix for Armored Core misdetecting a Link cable being detected

For some reason, the game detects that a link cable is plugged in
and disables the local multiplayer as a result.

Thanks @sony for fixing the issue in their PS Classic branch, a
simpler fix is done here instead.

Co-authored-by: Sony

Update custom-issue-report.md

Point out that people should try on PCSX-Redux, PCSX Reloaded... as well in case we suffer from a regression or forgot to merge a fix.

Merge pull request #559 from gameblabla/cdrom_various_fixes

Various CDROM fixes from Duckstation and Mednafen

Use the "magic numbers" from Duckstation for CDLPause

We need to make sure not to break Gundam Battle Assault 2 in PAL mode
as otherwise it will lockup.
Amusingly, Mednafen currently cannot run this game in PAL mode.

Fix CdlGetTD according to Mednafen's implementation.

CdlGetmode is GetParam, fix it according to Mednafen's implementation

The result size for GetParam (previously Getmode) should be only 5 and done this way.

Here's Mednafen implementation :
https://github.com/libretro-mirrors/mednafen-git/blob/1899500078169e787f86eb5302a3f35abdaa8764/src/psx/cdc.cpp#L2095

Nocash's documentation also shows that our previous implementation was wrong :
https://problemkaputt.de/psx-spx.htm
(See CDROM Controller Command Summary section or CDROM - Status Commands)

Don't throw an error for CdlGetQ.

I doubt this is used by any game but who knows.

See Mednafen for reference :
https://github.com/libretro-mirrors/mednafen-git/blob/1899500078169e787f86eb5302a3f35abdaa8764/src/psx/cdc.cpp#L2527

Improve SetLoc CDROM command from Duckstation

See Duckstation's CDROM for reference :
https://github.com/stenzek/duckstation/blob/18241978ea056a81ff9847f0ad89658f886153ea/src/core/cdrom.cpp#L1087

CdlSync should be NULL and return an error.

This has been confirmed by both Mednafen and nocash's documentation.

Implement fix from Mednafen for Fantastic Pinball Kyuutenkai.

This is taken from the way Mednafen implemented it.
https://github.com/libretro-mirrors/mednafen-git/blob/563b72e377fa8284559df4df0271108d4935c9f1/src/psx/cdc.cpp#L941

This properly fixes the freeze issue in Fantastic Pinball Kyuutenkai.
Thanks notaz for assisting me with the code.

It seems not to work properly on ARM though ?

Merge pull request #558 from gameblabla/cdrom_fixes_libretro_popo

Merge several fixes from PCSX Redux and adjust delay for SetLocPending.

Merge several fixes from PCSX Redux and adjust delay for SetLocPending.

There's a game, PoPoLoCrois Monogatari II, that unfortunately locks up
during the intro screen.
I should have known that code was wrong as Mednafen did not have anything
like that in their code either, hence the confusion.

Their fix however still don't include the Driver fix so the game would still
crash if we don't have the "+ Seektime".
To be honest, i'm not sure why the PCSX Reloaded team did it this way...

I noticed that the fastforward and fastbackward code was pretty much unused.
Looked at Mednafen and all they do is just adjust the cursector
and make sure that fastword & backword trigger the AUTO_REPORT code
so i did the latter.

There was also a mysterious leftover code from PCSX Reloaded in Cdlplay:
that had a condition that forced it to SEEK_DONE
(with a comment even saying that it should be set to SEEK instead)
so i set it to SEEK_PENDING instead.
(Mednafen's code seems to suggest so as well)

Fix regression caused by #9361a5aa4fef73e06bc78b7c462eb28a7d12e3d0

Not sure why it caused that but it causes the function to fail.
I don't want to wait months for him to fix it properly
so let's just do that for now.

Merge pull request #553 from gameblabla/football_fix_libretro

CDROM: Rename Reset+Init commands, fix "This is Football 2" lockup

CDROM: Rename Reset+Init commands, fix "This is Football 2" lockup

Upon the reset command being issued, we need to set the Mode to 0x20
(as it was confirmed in Mednafen) and also disable the muting.

We also change the seektime for the reset command appropriately.
Please see Mednafen's implementation as reference :
https://github.com/libretro-mirrors/mednafen-git/blob/1899500078169e787f86eb5302a3f35abdaa8764/src/psx/cdc.cpp#L2007

This fixes the black screen issue in "This is Football 2".
This also fixes the black screen issue in Pooh's Party
after you selected a character.

Properly fix the extern to const void instead of just void.

Merge pull request #551 from gameblabla/hle_compil

Fix broken HLE PR

Fix broken HLE PR.

Merge pull request #543 from gameblabla/hle_fies_corruption_libretro

Properly protect the HLE instructions against corrupted memory.

Merge pull request #547 from gameblabla/blues_clues_fix_audio_fmv

CDROM: Ignore sectors with channel number 255

CDROM: Ignore sectors with channel number 255

This was inspired by the fix in Duckstation :
https://github.com/stenzek/duckstation/commit/0710e3b6d384526ed939f742f8f657623bb354bb

Some games have junk audio sectors with a channel number of 255.
If these are not skipped, then they will play wrong file.

This was tested on "Blue's Clues : Blue's Big Musical" and
it fixed the missing audio there.

Taxi 2 is also said to be affected by this.

Merge pull request #544 from gameblabla/gpu_hle_rewrite_libretro

Rewrite GPU bios functions according to OpenBIOS.

Merge pull request #542 from gameblabla/mdec_fix

mdec.cpp: Dare to set MDEC_BIAS to 10

Merge pull request #541 from gameblabla/spu_fix_libretro

Fixes Diablo 1 SPU bug properly and remove Diablo hack.

Rewrite GPU bios functions according to OpenBIOS.

See PCSX-Redux :
https://github.com/grumpycoders/pcsx-redux/blob/93653ba5281487d3bed57371d7b64c32dfc669f0/src/mips/openbios/gpu/gpu.c

Properly protect the HLE instructions against corrupted memory.

Fix is from PCSX-redux :
grumpycoders/pcsx-redux@99c9508

This should hopefully fix HDHOSHY's experimental patch properly.

Co-authored-by: Nicolas Noble <nicolasnoble@users.noreply.github.com>

Fixes Diablo 1 SPU bug properly and remove Diablo hack.

(Mostly) inspired by the fixes done in PCSX Redux :
https://github.com/grumpycoders/pcsx-redux/blob/93653ba5281487d3bed57371d7b64c32dfc669f0/src/spu/registers.cc#L504

It seems that there was an attempt to initially implement it
in PCSX Reloaded (hence the bIgnoreLoop in freeze.c) but it was never
implemented properly.

Co-authored-by: Nicolas Noble <nicolasnoble@users.noreply.github.com>

Merge pull request #539 from gameblabla/spustate_register_fix_libretro

[SPU] Emulate SPUSTAT[5:0] as a mirror of SPUCNT[5:0]

Merge pull request #540 from gameblabla/gte_senquack_fix_libretro

gte: Fix gteH division and sign extension (from PCSX4ALL)

gte: Fix gteH division and sign extension (from PCSX4ALL)

gteH register is u16, not s16. DIVIDE macro/func assumed it was
s16 for some reason. Behavior now matches Mednafen.

Co-authored-by: senquack <dansilsby@gmail.com>

[SPU] Emulate SPUSTAT[5:0] as a mirror of SPUCNT[5:0]

Fix is from PCSX-Redux :
grumpycoders/pcsx-redux@4e905d7

I have tested the fix against Loonies 8192 (a PSn00bSDK made homebrew game) and it no longer
locks up during loading.

Co-authored-by: wheremyfoodat <44909372+wheremyfoodat@users.noreply.github.com>

Merge pull request #538 from gameblabla/fix_cdom_savestate

Fix save state compatibility that was broken by my CDrom timing changes.

mdec.cpp: Dare to set MDEC_BIAS to 10

This fixes graphic artifacts during cinematics in :
- Vandal Hearts
- R-Types
- Galerians

Dmitrysmagin had a similar fix in PCSX4ALL but he set it to 6.
It turns out MDEC_BIAS set to 6 was still not enough for Galerians
to look properly so it was set to 10.
It seems so far that other games are unaffected.

Co-authored-by: Dmitry Smagin <exmortis@yandex.ru>