Merge pull request #354 from justinweiss/move-to-thread-api

3DS: Switch from svc* to the thread* API

3DS: Switch from svc* to the thread* API

svcCreateThread doesn't fully set up thread vars, which causes
svcBreaks / crashes when calling certain functions -- reentrant ones,
for example. threadCreate, etc. are higher-level functions that do all
the correct setup and cleanup.

Since we're treating the thread structure as opaque, calling it an
int_32t seems OK.

Merge pull request #353 from justinweiss/default-thread-cpu-3ds

3DS: Schedule threads on the default core

3DS: Schedule threads on the default core

When creating a thread, pcsx was scheduling threads on core 1, which
is the system core. This is usually prevented by the system. Instead,
it should schedule threads on the default core, as suggested by
3dbrew: https://www.3dbrew.org/wiki/Multi-threading

> Games usually create threads using -2.

In the future, we may be able to schedule threads on core 2 on New 3DS
only, if we find it gives better performance.

This was preventing the CDDA thread from starting, which was
preventing CD Audio from playing.

Merge pull request #349 from negativeExponent/unai_plugin_update

update gpu_unai plugin

Merge branch 'gpu_unai_plugin_update'

Merge pull request #348 from justinweiss/remove-utime

Stop calling utime on 3ds

Stop calling utime on 3ds

The toolchain doesn't have utime defined, so these calls will fail.

Merge pull request #347 from justinweiss/enable-chd-3ds

Enable CHD on 3DS builds

Enable CHD on 3DS builds

I don't know what build errors were happening before, but this built
cleanly for me.

Merge pull request #345 from justinweiss/enable-unai-3ds

Re-enable unai on 3DS

Merge pull request #346 from justinweiss/fix-sound-3ds

Fix sound over-buffering on 3ds

Fix sound over-buffering on 3ds

With iTempo turned on, we generate a frame and a half of sound data
each frame, which causes dropped sound frames and noise.

It looks like GPH hacks are disabled for 3ds in other places, so
disabling it here too seemed like the correct fix.

Re-enable unai on 3DS

It now builds properly, and is much much faster.

Merge pull request #339 from dankcushions/patch-1

Added support for PSXONPSP660.bin and psx* named BIOS files

Added support for PSXONPSP660.bin and psx* named BIOS files

Backport GPU Unai plugin from PCSX4ALL

- backports gpu unai plugin from PCSX4ALL
- sync necessary files with notaz/master to allow building standalone app

Merge pull request #335 from yoshisuga/ios-arm64-buildbot-fix

iOS: support for building using buildbot recipe

Merge pull request #334 from jdgleaver/options-update

Update 'libretro_core_options.h' to v1.3 format

Update 'libretro_core_options.h' to v1.3 format

iOS: support for building using buildbot recipe

Merge pull request #330 from retro-wertz/skip_plugins

Do not compile plugins when using the non-static build method

Do not compile plugins when using the non-static build method

- This is meant for platforms that still uses the non-static build method to compile the libretro core (aka versions of retropie)
- The output file is still libretro.so to preserve any compatibility with existing script
- Using the static makefile is still recommended.

Merge pull request #328 from retro-wertz/sanitize_gpu_adv_options

Sanitize gpu adv options and close plugins upon exiting core

Merge pull request #327 from retro-wertz/patch-4

Fix main RAM size in retro_get_memory_size

Close plugins when exiting core

Update advanced peops gpu core options

Fix main RAM size in retro_get_memory_size

PSX Main RAM should only by 2048KB

Merge pull request #326 from retro-wertz/silence_warnings

silence warnings and a few core fixes

Silence a few more warnings

- silence a few more truncated warnings
- change "disable" to "disabled" in core options

HLE BIOS: Fix corrupted memory card saves

- Save files for games that are suppose to work with HLE bios are now corrupted from the last HLE BIOS improvements PR.
- This only affects memcard 2 slot when enabled and used. Saves on memcard 1 are fine since those are directly handled through the Libretro API.
- This is fixed by adding the missing SaveMcd() to buwrite().

Tested on Wild Arms and Breath of Fire III

NOTE: HLE bios is still not compatible for most games. For example, memory card save functions are broken or does not work at all. Most of Konami games i have tested does not work with HLE BIOS for memory card saves.

psxbios.c: Cleanup buopen()

- This just silences some [-Wpointer-sign] warning from last PR

Fix region detection for Wild Arms

- Wild Arms EU and US version has non-standard string (probably for region protection) causing auto-region detection to fail and wrong CDROM info in log.
- This fix sanitizes the string name to remove unneeded characters ("EXE\" in this case that comes before the serial-number). This would sanitize the CD-ROM info in log as well.

Silence warning -Wstringop-truncation

-strncpy replaced with strcpy and memcpy where appropriate

Silence warning -Wformat-overflow

- Increasing the maximum size of destination char array to x2 to silence this warning.

Merge pull request #325 from retro-wertz/SysPrintf_to_retro_log

SysPrintf to retro_log and silence some warnings

Sanitize Loaded CD image string in log

Silence warning about misleading indent

Silence warning: missing braces around initializer

Rename PACKAGE_VERSION to PCSX_VERSION

- this should silence warning for another PACKAGE_VERSION define used for flac
- PACKAGE_VERSION is not used within pcsx other than version logging. Savestate does not check for this and is using another define for savestate version number.

Use retro_log for SysPrintf

Merge pull request #323 from fpscan/master

Update libretro_core_options.h

Update libretro_core_options.h

option_defs_tr,

Merge pull request #322 from retro-wertz/fix_android_build

Fix android build

Fix android build

Merge pull request #321 from fpscan/master

Turkish Language Support

Merge pull request #320 from retro-wertz/new_core_options

target actual gpu plugin used for advance gpu core options

Turkish Language Support

Silence this log message

- Removing this log message since RETRO_ENVIRONMENT_SET_GEOMETRY will show width and height when there is a change from last callback.

Make advanced gpu options target actual gpu plugin used

- In the case of IOS, the core uses UNAI but can be built with dynarec or interpreter. Other platform does as well. So gpu-specific  defines are added for actual  gpu plugin used and compiled and then the specific gpu-only core options are shown or hidden based on this #define

Update libretro_core_options.h

Merge pull request #319 from retro-wertz/new_core_options

add new core options menu, add advance gpu options.

Add advanced gpu core options

- works for peops gpu plugin only

Merge pull request #318 from retro-wertz/psxbios

HLE BIOS improvements from upstream

Add enhanced core options

HLE: Merge HLE BIOS improvements from upstream
- Merge update from https://github.com/libretro/pcsx_rearmed

Merge pull request #317 from retro-wertz/psxhle_log

psxhle.c: Silence logging when using HLE bios

psxhle.c: Silence logging when using HLE bios

- at least one game (Wild Arms) is continouos spamming the log window. Silence this in a way that its easy to enable when need to.
- Remove unnecessary message in Makefile

add custom issue report template

Merge pull request #137 from gameblabla/hle

HLE BIOS improvements (based on nocash doc, pcsx4all, upstream pcsxr etc...)

psxbios: Don't accept 400h as a valid sector.

Notaz said this could corrupt the emulator (even though
the console allowed this and it was a bug) so limit it to 0x3FF.

psxbios: Properly fix garbage area code.

This should work as intended now.

psxbios: Fixup mistake in card_info function.

This fixes Looney Tunes Sheep Raiders.

psxbios: Return value is always 1 for CloseTh and ChangeTh.

According to nocash PSX doc, ChangeThread and CloseThread should
always return 1. Fix it accordingly.

psxbios: Add checks for memchr.

psxbios: Some games read from address 0, fix that.

See Garbage Area at Address 00000000h in Nocash PSX Specifications for more inf>

R-types will refuse to boot if psxM[0] is zero.
Fade to Black can crash upon memory card access if psxM[5] is set to the wrong >
Crash Team Racing will refuse to boot if psxM[2] and psxM[3] are not zero.
(Previously, that region of memory would be overwritten above. No explanation why.)

It's set as such according to nocash :
[00000000h]=3C1A0000h  ;<-- but overwritten by 00000003h after soon
[00000004h]=275A0C80h  ;<-- or 275A0C50h (in older BIOS)
[00000008h]=03400008h
[0000000Ch]=00000000h

Note that setting 00000000h to 00000003h doesn't work for R-types.

psxbios: Implement get_cd_status stub according to nocash doc.

psxbios: Make HLE bios start from block 1, not 2.

I already fixed this for bufile :
ptr = Mcd##mcd##Data + 128 * (nfile + 1); \
instead of
ptr = Mcd##mcd##Data + 128 * nfile; \

But i forgot to also fix psxBios_firstfile accordingly.
So nfile would still start from block 2 and it would
cause all kinds of issues.

This was the original comment on pcsxr codeplex
as my patch was based on it :
"The HLE-Bios start to read memory card blocks at block 2. You need to change
ptr = Mcd##mcd##Data + 128 * nfile; \ to
ptr = Mcd##mcd##Data + 128 * (nfile + 1); \
for #define bufile(mcd) { \ and changing nfile = 1; to nfile = 0; for psxBios_firstfile() { // 42 Starts numbering slots in the right position."

psxbios: Fix card_status function as to use card_active_chan.

psxbios: Merge interrupt_r26 fix from PCSX4ALL.

Thanks DmitrySmagin for the fix.

psxbios: Do the same for heap_size.

psxbios : Use bfreezel for pad_stopped instead of setting it to 0.

psxbios : Implement stub for wait_card.

Pretty much identical to get_card_status except it's supposed
to wait until a non-busy state occurs.

psxbios : Fix a few mistakes :

- Remove extra ffile, pfile.. as it is declared elsewhere.
- Remove extra condition as it's already being handled by pad_stopped.
- Remove n variable from function (pcsx4all did this) as it's useless
and documentation doesn't seem to specify it's needed.

psxbios: Add note and update copyright.

psxbios: Merge heap fix from PCSX4ALL.

psxbios: Init some vars just to be sure.

psxbios: Merge PCSX4ALL pad_stopped fix.

psxbios.cpp: Add psxBios_card_status(); doesn't seem to affect anything

psxbios: Fixes save issues on Parasite Eve II, Parasite Eve I and others

Merging never-merged fix by Solus for Parasite Eve. I also fixed some GCC warnings
about dir->name going out of bounds too.
We also make sure to only add '/0' if string is not full and right after the last character in the dir array.

This fixes saving/loading in Parasite Eve II, Parasite Eve.
It also fixed regressions for the LEGO Racers where other games would break,
meaning this should be the proper fix for saves.

We're also using our own strlen implementation to avoid any issues or
relying on the libc's own strlen.

psxbios: Callback Events should not set event status to busy in TestEvent.

We are already doing that in WaitEvent. According to nocash documentation,
this also applies to TestEvent as well so let's implement it.

psxbios: Support other syscalls

Since they don't do anything however, let's just add a default and break;
instead. Interestingly, trying to implement SYS(04h+ can actually crash
the HLE bios. Nocash documentation said that it should crash it
but due to other reasons, it doesn't and returns to main program instead.

psxbios: Add checks for strlen.

Per what Nocash documentation says.

psxbios: Add checks for strcpy & strncpy.

This should be closer to documentation.

psxbios: Fixup free.

This is needed for Burning Road. Fix comes from PCSX4ALL by dmitryssmagin.

psxbios: Add checks for strcmp & strncmp.

Yes, it's supposed to have a lot of checks like these.
Also merge some more fixes from PCSX4ALL. (upstream did not have
these as far as i know)

psxbios: Stub for get_error

According to a user on PCSX-Reloaded codeplex (now dead), this fixed loading/saving
in Final Fantasy Tactics.

psxbios : Several fixes to malloc.

- Don't allow malloc to init memory if heap is invalid
According to nocash documentation, malloc will not allocate if heap size
is too small or invalid.

- Also merge some fixes from PCSX4ALL fork by senquack in order to allow
Burning Road to run.

psxbios: Add checks for memmove.

Always returns dst per documentation.

psxbios : Add checks to memset.

This now can be a tricky pony and an improper memset implementation means that
you can't play Tomb Raider 4 properly.
(Trying to put in the eye of horus will just result in Lara sidestepping)

This should hopefully be fixed (and follows nocash doc).

psxbios: Add checks to memcpy.

Always returns dst

psxbios: Add checks for bzero.

Only return 0 if size is invalid. (0 or 0x7FFFFFFF), return dst if not.

psxbios: Add checks for bcopy

Nocash documentation say that function refuses to copy any data
if dst is NULL or len greater than 0x7FFFFFFF.
(In any cases, return value is always dst)

psxbios : Add checks to index and rindex calls

Return 0 for index/rindex if src is 0x00.

psxbios: Merge upstream fix for Deliverevent in firstfile

Looks like it is only executed for memory cards, that makes sense i guess.

psxbios: Should fix input issues in some games.

Someone reported that this fixed Negcon controls in Ridge Racer Revolution
and Grand Tour Racing.
I doubt this will suffer from regressions ?

psxbios: Fix out-of-bounds issue.

Make sure to return 0xff is no TCBs are available.
Thanks senquack for the proper fix.

Only PCSX4ALL uses ResetIoCycle. Remove it for now.

Fixup buread & buwrite being undefined.

psxbios: Improve WaitEvent behaviour according to documentation

Nocash says it should return 0 if Event is unused, which it already does so it might be safe to remove the warning about it.
(see why below)

Then, it says that it should return 1 if event is ready (EvStALREADY). When that happens, it should also set it to ready (EvStACTIVE),
unless event mode is EvMdINTR. (aka Callback Events)
If it is a callback event, then documentation says it should be stuck in a loop forever.

It also says that it can sometime return 0 (a BUG according to nocash),
especially if it switched from EvStALREADY to EvStACTIVE.

Said return value was previously set to 1, for some reasons. Now that we are covering all the corner cases,
we should be returning 0 instead by default to cover the BUG.

This should cover how most games expects it to behave now and should make it more robust.